drm/amd/display: Avoid flooding unnecessary info messages
authorWayne Lin <Wayne.Lin@amd.com>
Tue, 13 May 2025 03:20:24 +0000 (11:20 +0800)
committerAlex Deucher <alexander.deucher@amd.com>
Tue, 13 May 2025 18:36:27 +0000 (14:36 -0400)
It's expected that we'll encounter temporary exceptions
during aux transactions. Adjust logging from drm_info to
drm_dbg_dp to prevent flooding with unnecessary log messages.

Fixes: 3637e457eb00 ("drm/amd/display: Fix wrong handling for AUX_DEFER case")
Cc: Mario Limonciello <mario.limonciello@amd.com>
Cc: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Wayne Lin <Wayne.Lin@amd.com>
Acked-by: Alex Deucher <alexander.deucher@amd.com>
Link: https://lore.kernel.org/r/20250513032026.838036-1-Wayne.Lin@amd.com
Signed-off-by: Mario Limonciello <mario.limonciello@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
(cherry picked from commit 9a9c3e1fe5256da14a0a307dff0478f90c55fc8c)
Cc: stable@vger.kernel.org
drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c

index c93eef6f7a6b98cfa36cc30265909fa527dbe711..5cdbc86ef8f5a93085911ad5659738ca588b44e4 100644 (file)
@@ -107,7 +107,7 @@ static ssize_t dm_dp_aux_transfer(struct drm_dp_aux *aux,
        if (payload.write && result >= 0) {
                if (result) {
                        /*one byte indicating partially written bytes*/
-                       drm_info(adev_to_drm(adev), "amdgpu: AUX partially written\n");
+                       drm_dbg_dp(adev_to_drm(adev), "amdgpu: AUX partially written\n");
                        result = payload.data[0];
                } else if (!payload.reply[0])
                        /*I2C_ACK|AUX_ACK*/
@@ -133,11 +133,11 @@ static ssize_t dm_dp_aux_transfer(struct drm_dp_aux *aux,
                        break;
                }
 
-               drm_info(adev_to_drm(adev), "amdgpu: DP AUX transfer fail:%d\n", operation_result);
+               drm_dbg_dp(adev_to_drm(adev), "amdgpu: DP AUX transfer fail:%d\n", operation_result);
        }
 
        if (payload.reply[0])
-               drm_info(adev_to_drm(adev), "amdgpu: AUX reply command not ACK: 0x%02x.",
+               drm_dbg_dp(adev_to_drm(adev), "amdgpu: AUX reply command not ACK: 0x%02x.",
                        payload.reply[0]);
 
        return result;