[PATCH] drm/amd/display: Avoid flooding unnecessary info messages

[Alex Deucher]

On Mon, May 12, 2025 at 11:28 PM Wayne Lin <Wayne.Lin at amd.com> wrote:
>
> 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: 6285f12bc54c ("drm/amd/display: Fix wrong handling for AUX_DEFER case")
> Cc: stable at vger.kernel.org
> Cc: Mario Limonciello <mario.limonciello at amd.com>
> Cc: Alex Deucher <alexander.deucher at amd.com>
> Signed-off-by: Wayne Lin <Wayne.Lin at amd.com>

Acked-by: Alex Deucher <alexander.deucher at amd.com>

> ---
>  drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c
> index 0d7b72c75802..25e8befbcc47 100644
> --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c
> +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c
> @@ -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;
> --
> 2.43.0
>