drm/amd/display: Return error code on DSC atomic check failure
authorHersen Wu <hersenxs.wu@amd.com>
Sun, 29 May 2022 14:54:30 +0000 (10:54 -0400)
committerAlex Deucher <alexander.deucher@amd.com>
Tue, 18 Apr 2023 20:28:53 +0000 (16:28 -0400)
[Why&How]
We were not returning -EINVAL on DSC atomic check fail. Add it.

Fixes: 71be4b16d39a ("drm/amd/display: dsc validate fail not pass to atomic check")
Reviewed-by: Aurabindo Pillai <aurabindo.pillai@amd.com>
Signed-off-by: Hersen Wu <hersenxs.wu@amd.com>
Tested-by: Daniel Wheeler <daniel.wheeler@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c

index 6e9d2e680d502102e9bf3c8df641463a598768ef..73b52bf76461f93f282aa25d558e84aa6dd7e13d 100644 (file)
@@ -10170,6 +10170,7 @@ static int amdgpu_dm_atomic_check(struct drm_device *dev,
                ret = compute_mst_dsc_configs_for_state(state, dm_state->context, vars);
                if (ret) {
                        DRM_DEBUG_DRIVER("compute_mst_dsc_configs_for_state() failed\n");
+                       ret = -EINVAL;
                        goto fail;
                }
 
index 5dc79b753d5fba855dafa7e9957310cdb9edcda1..810ab682f424f94e7f450e82651a18b61016c7b7 100644 (file)
@@ -1410,6 +1410,7 @@ int pre_validate_dsc(struct drm_atomic_state *state,
        ret = pre_compute_mst_dsc_configs_for_state(state, local_dc_state, vars);
        if (ret != 0) {
                DRM_INFO_ONCE("pre_compute_mst_dsc_configs_for_state() failed\n");
+               ret = -EINVAL;
                goto clean_exit;
        }