Commit 9a9c3e1f authored by Wayne Lin's avatar Wayne Lin Committed by Alex Deucher
Browse files

drm/amd/display: Avoid flooding unnecessary info messages 



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: 3637e457 ("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: default avatarWayne Lin <Wayne.Lin@amd.com>
Acked-by: default avatarAlex Deucher <alexander.deucher@amd.com>
Link: https://lore.kernel.org/r/20250513032026.838036-1-Wayne.Lin@amd.com


Signed-off-by: default avatarMario Limonciello <mario.limonciello@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 73b7fd4b

drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c

+3 −3
Original line number Diff line number Diff line
@@ -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;