drm/amd/display: Don't fill up the logs
authorLuben Tuikov <luben.tuikov@amd.com>
Thu, 24 Feb 2022 21:29:52 +0000 (16:29 -0500)
committerAlex Deucher <alexander.deucher@amd.com>
Wed, 2 Mar 2022 23:38:51 +0000 (18:38 -0500)
Don't fill up the logs with:

[253557.859575] [drm:amdgpu_dm_atomic_check [amdgpu]] DSC precompute is not needed.
[253557.892966] [drm:amdgpu_dm_atomic_check [amdgpu]] DSC precompute is not needed.
[253557.926070] [drm:amdgpu_dm_atomic_check [amdgpu]] DSC precompute is not needed.
[253557.959344] [drm:amdgpu_dm_atomic_check [amdgpu]] DSC precompute is not needed.

which prints many times a second, when the kernel is run with
drm.debug=2.

Instead of DRM_DEBUG_DRIVER(), make it DRM_INFO_ONCE().

Cc: Alex Deucher <Alexander.Deucher@amd.com>
Cc: Roman Li <Roman.Li@amd.com>
Cc: Felix Kuehling <felix.kuehling@amd.com>
Cc: Hersen Wu <hersenwu@amd.com>
Cc: Daniel Wheeler <daniel.wheeler@amd.com>
Fixes: 17ce8a6907f7 ("drm/amd/display: Add dsc pre-validation in atomic check")
Signed-off-by: Luben Tuikov <luben.tuikov@amd.com>
Reviewed-by: Roman Li <Roman.Li@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c

index 740435a..31ac1fc 100644 (file)
@@ -1198,11 +1198,11 @@ void pre_validate_dsc(struct drm_atomic_state *state,
        struct dc_state *local_dc_state = NULL;
 
        if (!is_dsc_precompute_needed(state)) {
-               DRM_DEBUG_DRIVER("DSC precompute is not needed.\n");
+               DRM_INFO_ONCE("DSC precompute is not needed.\n");
                return;
        }
        if (dm_atomic_get_state(state, dm_state_ptr)) {
-               DRM_DEBUG_DRIVER("dm_atomic_get_state() failed\n");
+               DRM_INFO_ONCE("dm_atomic_get_state() failed\n");
                return;
        }
        dm_state = *dm_state_ptr;
@@ -1245,7 +1245,7 @@ void pre_validate_dsc(struct drm_atomic_state *state,
        }
 
        if (!pre_compute_mst_dsc_configs_for_state(state, local_dc_state, vars)) {
-               DRM_DEBUG_DRIVER("pre_compute_mst_dsc_configs_for_state() failed\n");
+               DRM_INFO_ONCE("pre_compute_mst_dsc_configs_for_state() failed\n");
                goto clean_exit;
        }
 
@@ -1258,7 +1258,7 @@ void pre_validate_dsc(struct drm_atomic_state *state,
 
                if (local_dc_state->streams[i] &&
                    is_timing_changed(stream, local_dc_state->streams[i])) {
-                       DRM_DEBUG_DRIVER("crtc[%d] needs mode_changed\n", i);
+                       DRM_INFO_ONCE("crtc[%d] needs mode_changed\n", i);
                } else {
                        int ind = find_crtc_index_in_state_by_stream(state, stream);