On 2025-05-12 23:20, Wayne Lin 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: sta...@vger.kernel.org
> Cc: Mario Limonciello <mario.limoncie...@amd.com>
> Cc: Alex Deucher <alexander.deuc...@amd.com>
> Signed-off-by: Wayne Lin <wayne....@amd.com>
Reviewed-by: Harry Wentland <harry.wentl...@amd.com>
Harry
> ---
> 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;
