Re: [PATCH AUTOSEL 4.9 08/17] drm/edid: Add 6 bpc quirk for BOE panel.

From: Sasha Levin
Date: Thu Nov 22 2018 - 10:53:14 EST


On Wed, Nov 21, 2018 at 10:33:18AM +0100, Daniel Vetter wrote:
On Wed, Nov 21, 2018 at 10:31 AM Daniel Vetter <daniel@xxxxxxxx> wrote:

On Tue, Nov 13, 2018 at 12:52:14AM -0500, Sasha Levin wrote:
> From: "Lee, Shawn C" <shawn.c.lee@xxxxxxxxx>
>
> [ Upstream commit 922dceff8dc1fb4dafc9af78139ba65671408103 ]
>
> BOE panel (ID: 0x0771) that reports "DFP 1.x compliant TMDS".
> But it's 6bpc panel only instead of 8 bpc.
>
> Add panel ID to edid quirk list and set 6 bpc as default to
> work around this issue.
>
> Cc: Jani Nikula <jani.nikula@xxxxxxxxx>
> Cc: Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxxxx>
> Cc: Gustavo Padovan <gustavo@xxxxxxxxxxx>
> Cc: Cooper Chiou <cooper.chiou@xxxxxxxxx>
> Signed-off-by: Lee, Shawn C <shawn.c.lee@xxxxxxxxx>>
> Signed-off-by: Daniel Vetter <daniel.vetter@xxxxxxxx>
> Link: https://patchwork.freedesktop.org/patch/msgid/1540792173-7288-1-git-send-email-shawn.c.lee@xxxxxxxxx
> Signed-off-by: Sasha Levin <sashal@xxxxxxxxxx>

Given that I'm not a fan of AUTOSEL at all: This one here is correctly
cherry-picked for stable, ack.

An idea that just crossed my mind: Could we integrate this into 0day
and suggest cc: stable before the patch even gets merged? Or is the
heuristics not good enough for that kind of automation?

Yes! I've actually tried it before but it seemed that the response rate
was quite low (even for commits that are obviously stable material) so I
turned it off to avoid spamming too much.

If you'd like to be the guinea pig for this, I could enable it for
drivers/gpu/drm/i915/ which I currently completely ignore. If at any
point you want it back off that's easy to do.

If this works well we can extend it to more subsystems where maintainers
might find it useful.

--
Thanks,
Sasha