[Public]
Hi Michel,
I want to double check if we're identifying the correct monitor for applying
the workaround. Could you please try the attached patch and let me know the
panel id ?
--
Regards,
Jay
________________________________
From: Michel Dänzer <mic...@daenzer.net>
Sent: Thursday, June 8, 2023 11:18 AM
To: Pillai, Aurabindo <aurabindo.pil...@amd.com>; Zhuo, Qingqing (Lillian)
<qingqing.z...@amd.com>; Chalmers, Wesley <wesley.chalm...@amd.com>
Cc: Wang, Chao-kai (Stylon) <stylon.w...@amd.com>; Li, Sun peng (Leo)
<sunpeng...@amd.com>; Wentland, Harry <harry.wentl...@amd.com>; Siqueira,
Rodrigo <rodrigo.sique...@amd.com>; Li, Roman <roman...@amd.com>;
amd-gfx@lists.freedesktop.org <amd-gfx@lists.freedesktop.org>; Chiu, Solomon
<solomon.c...@amd.com>; Lin, Wayne <wayne....@amd.com>; Lakha, Bhawanpreet
<bhawanpreet.la...@amd.com>; Gutierrez, Agustin <agustin.gutier...@amd.com>;
Kotarac, Pavle <pavle.kota...@amd.com>
Subject: Re: [PATCH 10/66] drm/amd/display: Do not set drr on pipe commit
On 6/8/23 16:31, Pillai, Aurabindo wrote:
>
> Thanks Michel,
>
> I reached out to windows driver team, and they have a monitor specific quirk
> to disable FAMS on this model. I suspect the issue is only present on certain
> fw revisions on the monitor which is why we cant see your issue.
>
> Unfortunately, having the patches in question reverted causes hangs with 3
> monitor setups. So I will push that monitor specific quirk and bring back the
> reverted patches.
Sounds good, thanks.
--
Earthling Michel Dänzer | https://redhat.com
Libre software enthusiast | Mesa and Xwayland developer
From 3dfcb5e60ec9fc9ec6c573231e5b6aa4edca2ed6 Mon Sep 17 00:00:00 2001
From: Aurabindo Pillai <aurabindo.pil...@amd.com>
Date: Mon, 12 Jun 2023 12:44:00 -0400
Subject: [PATCH] drm/amd/display: Add monitor specific edid quirk
Disable FAMS on a Samsung Odyssey G9 monitor. Experiments show that this
monitor does not work well under some use cases, and is likely
implementation specific bug on some revisions of the device.
Signed-off-by: Aurabindo Pillai <aurabindo.pil...@amd.com>
---
.../amd/display/amdgpu_dm/amdgpu_dm_helpers.c | 24 +++++++++++++++++++
1 file changed, 24 insertions(+)
diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_helpers.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_helpers.c
index cd20cfc04996..e7e545665007 100644
--- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_helpers.c
+++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_helpers.c
@@ -44,6 +44,28 @@
#include "dm_helpers.h"
#include "ddc_service_types.h"
+static u32 edid_extract_panel_id(struct edid *edid)
+{
+ return (u32)edid->mfg_id[0] << 24 |
+ (u32)edid->mfg_id[1] << 16 |
+ (u32)EDID_PRODUCT_ID(edid);
+}
+
+static void apply_edid_quirks(struct edid *edid, struct dc_edid_caps *edid_caps) {
+ uint32_t panel_id = edid_extract_panel_id(edid);
+
+ switch (panel_id) {
+ case drm_edid_encode_panel_id('S', 'A', 'M', 0x0E5E):
+ case drm_edid_encode_panel_id('S', 'A', 'M', 0x7053):
+ pr_err("### Applying any edid quirk for panel %x\n", panel_id);
+ edid_caps->panel_patch.disable_fams = true;
+ break;
+ default:
+ pr_err("### Not applying any edid quirk for panel %x\n", panel_id);
+ return;
+ }
+}
+
/* dm_helpers_parse_edid_caps
*
* Parse edid caps
@@ -115,6 +137,8 @@ enum dc_edid_status dm_helpers_parse_edid_caps(
else
edid_caps->speaker_flags = DEFAULT_SPEAKER_LOCATION;
+ apply_edid_quirks(edid_buf, edid_caps);
+
kfree(sads);
kfree(sadb);
--
2.40.1
