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

[Stefan Bader]

On 25.06.25 12:13, 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>
Acked-by: Stefan Bader <stefan.bader 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;


-- 
- Stefan
-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_0xE8675DEECBEECEA3.asc
Type: application/pgp-keys
Size: 47863 bytes
Desc: OpenPGP public key
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20250710/39078c60/attachment-0001.key>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20250710/39078c60/attachment-0001.sig>