drm/amd/display: Clear dpcd_sink_ext_caps if not set
authorPaul Hsieh <paul.hsieh@amd.com>
Wed, 25 Oct 2023 02:53:35 +0000 (10:53 +0800)
committerAlex Deucher <alexander.deucher@amd.com>
Fri, 17 Nov 2023 05:51:35 +0000 (00:51 -0500)
[WHY]
Some eDP panels' ext caps don't set initial values
and the value of dpcd_addr (0x317) is random.
It means that sometimes the eDP can be OLED, miniLED and etc,
and cause incorrect backlight control interface.

[HOW]
Add remove_sink_ext_caps to remove sink ext caps (HDR, OLED and etc)

Cc: Mario Limonciello <mario.limonciello@amd.com>
Cc: Alex Deucher <alexander.deucher@amd.com>
Cc: stable@vger.kernel.org
Reviewed-by: Anthony Koo <anthony.koo@amd.com>
Acked-by: Alex Hung <alex.hung@amd.com>
Signed-off-by: Paul Hsieh <paul.hsieh@amd.com>
Tested-by: Daniel Wheeler <daniel.wheeler@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/display/dc/dc_types.h
drivers/gpu/drm/amd/display/dc/link/link_detection.c

index cea666ea66c6144cad038aa9a8d833b2d36b0a78..fcb825e4f1bb8f8dfba4107bc3bb9c642aa726f1 100644 (file)
@@ -177,6 +177,7 @@ struct dc_panel_patch {
        unsigned int disable_fams;
        unsigned int skip_avmute;
        unsigned int mst_start_top_delay;
+       unsigned int remove_sink_ext_caps;
 };
 
 struct dc_edid_caps {
index d6f0f857c05af8d6d9298225b58f10291cd5c92a..f2fe523f914f1179106426684e010cfea1b170ef 100644 (file)
@@ -1088,6 +1088,9 @@ static bool detect_link_and_local_sink(struct dc_link *link,
                if (sink->edid_caps.panel_patch.skip_scdc_overwrite)
                        link->ctx->dc->debug.hdmi20_disable = true;
 
+               if (sink->edid_caps.panel_patch.remove_sink_ext_caps)
+                       link->dpcd_sink_ext_caps.raw = 0;
+
                if (dc_is_hdmi_signal(link->connector_signal))
                        read_scdc_caps(link->ddc, link->local_sink);