drm/amd/display: downgrade HDMI infoframe error to one time warning
authorAurabindo Pillai <aurabindo.pillai@amd.com>
Thu, 24 Apr 2025 19:15:10 +0000 (15:15 -0400)
committerAlex Deucher <alexander.deucher@amd.com>
Wed, 30 Apr 2025 22:06:19 +0000 (18:06 -0400)
In certain config, a modeprobe test triggers too many instances of the
error related to infoframe. Make it print only once, and also make it a
warning.

Fixes: 6027cbee1900 ("drm/amd/display: Add error check for avi and vendor infoframe setup function")
Signed-off-by: Aurabindo Pillai <aurabindo.pillai@amd.com>
Reviewed-by: Chengjun Yao <Chengjun.Yao@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c

index 89a43f33e8a1317a47cd79752e111fbb018ee4f2..51255016e5fbba6c1cc6ee86d552c62f0c3b92c7 100644 (file)
@@ -6469,13 +6469,13 @@ static void fill_stream_properties_from_drm_display_mode(
                                                               (struct drm_connector *)connector,
                                                               mode_in);
                if (err < 0)
-                       drm_err(connector->dev, "Failed to setup avi infoframe: %zd\n", err);
+                       drm_warn_once(connector->dev, "Failed to setup avi infoframe on connector %s: %zd \n", connector->name, err);
                timing_out->vic = avi_frame.video_code;
                err = drm_hdmi_vendor_infoframe_from_display_mode(&hv_frame,
                                                                  (struct drm_connector *)connector,
                                                                  mode_in);
                if (err < 0)
-                       drm_err(connector->dev, "Failed to setup vendor infoframe: %zd\n", err);
+                       drm_warn_once(connector->dev, "Failed to setup vendor infoframe on connector %s: %zd \n", connector->name, err);
                timing_out->hdmi_vic = hv_frame.vic;
        }