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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20181121093126.GJ4266@phenom.ffwll.local>
Date:   Wed, 21 Nov 2018 10:31:26 +0100
From:   Daniel Vetter <daniel@...ll.ch>
To:     Sasha Levin <sashal@...nel.org>
Cc:     stable@...r.kernel.org, linux-kernel@...r.kernel.org,
        "Lee, Shawn C" <shawn.c.lee@...el.com>,
        Jani Nikula <jani.nikula@...el.com>,
        Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
        Gustavo Padovan <gustavo@...ovan.org>,
        Cooper Chiou <cooper.chiou@...el.com>, Lee@...nom.ffwll.local,
        Daniel Vetter <daniel.vetter@...ll.ch>,
        dri-devel@...ts.freedesktop.org
Subject: Re: [PATCH AUTOSEL 4.9 08/17] drm/edid: Add 6 bpc quirk for BOE
 panel.

On Tue, Nov 13, 2018 at 12:52:14AM -0500, Sasha Levin wrote:
> From: "Lee, Shawn C" <shawn.c.lee@...el.com>
> 
> [ 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@...el.com>
> Cc: Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>
> Cc: Gustavo Padovan <gustavo@...ovan.org>
> Cc: Cooper Chiou <cooper.chiou@...el.com>
> Signed-off-by: Lee, Shawn C <shawn.c.lee@...el.com>>
> Signed-off-by: Daniel Vetter <daniel.vetter@...ll.ch>
> Link: https://patchwork.freedesktop.org/patch/msgid/1540792173-7288-1-git-send-email-shawn.c.lee@intel.com
> Signed-off-by: Sasha Levin <sashal@...nel.org>

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

> ---
>  drivers/gpu/drm/drm_edid.c | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
> index 83d2f43b5a2f..c93dcfedc219 100644
> --- a/drivers/gpu/drm/drm_edid.c
> +++ b/drivers/gpu/drm/drm_edid.c
> @@ -116,6 +116,9 @@ static const struct edid_quirk {
>  	/* SDC panel of Lenovo B50-80 reports 8 bpc, but is a 6 bpc panel */
>  	{ "SDC", 0x3652, EDID_QUIRK_FORCE_6BPC },
>  
> +	/* BOE model 0x0771 reports 8 bpc, but is a 6 bpc panel */
> +	{ "BOE", 0x0771, EDID_QUIRK_FORCE_6BPC },
> +
>  	/* Belinea 10 15 55 */
>  	{ "MAX", 1516, EDID_QUIRK_PREFER_LARGE_60 },
>  	{ "MAX", 0x77e, EDID_QUIRK_PREFER_LARGE_60 },
> -- 
> 2.17.1
> 

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch