ACK: [PATCH 1/1][SRU][Jammy] drm/amd/display: Avoid flooding unnecessary info messages

[Chia-Lin Kao (AceLan)]

On Wed, Jun 25, 2025 at 06:13:40PM +0800, You-Sheng Yang wrote:
> From: Wayne Lin <Wayne.Lin at amd.com>
> 
> BugLink: https://bugs.launchpad.net/bug/2115238
> 
> commit d33724ffb743d3d2698bd969e29253ae0cff9739 upstream.
> 
> 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 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>
> Link: https://lore.kernel.org/r/20250513032026.838036-1-Wayne.Lin@amd.com
> Signed-off-by: Mario Limonciello <mario.limonciello at amd.com>
> Signed-off-by: Alex Deucher <alexander.deucher at amd.com>
> (cherry picked from commit 9a9c3e1fe5256da14a0a307dff0478f90c55fc8c)
> Cc: stable at vger.kernel.org
> Signed-off-by: Greg Kroah-Hartman <gregkh at linuxfoundation.org>
> (backported from commit 0638bad18d49c5da0b80ffb8c0e9172597422302)
> [vicamo: adjust the context due to missing commit d433981385c62]
> Signed-off-by: You-Sheng Yang <vicamo.yang at canonical.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 b3823c8e81b2f..ab88287a5f05a 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
> @@ -101,7 +101,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. Force 0 to retry*/
> -			drm_info(adev_to_drm(adev), "amdgpu: AUX partially written\n");
> +			drm_dbg_dp(adev_to_drm(adev), "amdgpu: AUX partially written\n");
>  			result = 0;
>  		} else if (!payload.reply[0])
>  			/*I2C_ACK|AUX_ACK*/
> @@ -127,11 +127,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.48.1
> 
Acked-by: Chia-Lin Kao (AceLan) <acelan.kao at canonical.com>

> 
> -- 
> kernel-team mailing list
> kernel-team at lists.ubuntu.com
> https://lists.ubuntu.com/mailman/listinfo/kernel-team