Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.34 as permitted sender) client-ip=23.128.96.34;
Message-ID: <e8a7daf5-3205-4103-a02e-59fe415e0be6@amd.com>
Date:   Thu, 12 Oct 2023 16:55:24 -0400
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH] drm/edid: add 8 bpc quirk to the BenQ GW2765
Content-Language: en-US
To:     Hamza Mahfooz <hamza.mahfooz@amd.com>,
        dri-devel@lists.freedesktop.org
Cc:     amd-gfx@lists.freedesktop.org,
        Alex Deucher <alexander.deucher@amd.com>,
        stable@vger.kernel.org,
        Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
        Maxime Ripard <mripard@kernel.org>,
        Thomas Zimmermann <tzimmermann@suse.de>,
        David Airlie <airlied@gmail.com>,
        Daniel Vetter <daniel@ffwll.ch>, linux-kernel@vger.kernel.org
References: <20231012184927.133137-1-hamza.mahfooz@amd.com>
From:   Harry Wentland <harry.wentland@amd.com>
In-Reply-To: <20231012184927.133137-1-hamza.mahfooz@amd.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?T3BqbXA5b0dmNVlIMU93MkdNcCt6QnV2WFh2NFdub29Xa0pHNXpiZTJtMEw0?=
 =?utf-8?B?MmZCejR5SjBoVkh6a2ovUVg0cXIyRmdubXVOSzRUOHJOcGQ5dE02RTZyWDNH?=
 =?utf-8?B?UGNmZHRNUXJoZE52d3BwcVc5RDNFRk5qT2V1Z0liNm03ZUV5ZWcybWhjekZV?=
 =?utf-8?B?bThEY1lFMlNjRU11Nm8yYkV0SVJBa01hZzhKSE44ZGhFMDZRNjk1VEFyWkRS?=
 =?utf-8?B?Wko0ajZyWGhsZlJpRjNoYUltUzVvWFZZS20rZkNYc1dRcEJjSmNveHdQaTNV?=
 =?utf-8?B?eDUwc0h0T2V0bXp4R0d2T2pQU1N0Qkg5WjdPU2R5V2VuOXNXaHEvbVZkdDZl?=
 =?utf-8?B?NjZkTmhLY054Yysxd1V3Z2ZTYXhLb1RmQXdRUVFQQ0srREFvbU1CWWJKdGk0?=
 =?utf-8?B?dDJNQ0k0MlVHRWRWTjZTQ3lWSVZtYTR0bElkeHhHZ0dKRDd6Rko2NFk0K25l?=
 =?utf-8?B?ZDM0Yjd3MTREWURGZUtheWJHeE5Tc01nakZTTWlYcHphWStmaE41MVlFelJM?=
 =?utf-8?B?K3ZoVzlDSklhUjRnS2dJOHFEMjVxbFpMVm9CdjVHMDFvZkV1TWxyWTBkaDg0?=
 =?utf-8?B?dzZpUDZXR2c2cENGalJnWk9NSkdPQTIyOTJrTHJwcU9DM0tEV2ZPR1g2UURX?=
 =?utf-8?B?TzdpMndGLzRwK29nQkRUQjRRd01aZnBPaWJWek9RbGxRYlJtWm56VXBoS1l3?=
 =?utf-8?B?dGU2cVNpend6ellkTnlLSWlLYTljQStXZkR4bmoxYVB0MkZWaTNGaGdkd2Jl?=
 =?utf-8?B?bmowM3FqZ1JaSFFNOWxQVFlrQ3ZIQmlHd2Z2SFpMYkNTck91bUFadzF3NzZk?=
 =?utf-8?B?cW04Qkg5dGxsZE1JN1REVEQvRG1JaTQ4c3ZUVW5FaXErSXVWN3Voc3lCcDZG?=
 =?utf-8?B?ZjR3VzVPY1B2Zk5ENDVMcFUyWWpWN2VTSVFJamFDSFdDbG9pWDZ3ejBjbDFI?=
 =?utf-8?B?cmswK0I1RVZzMkY2aGN3blBSUHFoS09HM1NTSmJQbkpqaGpZVWg0QXROWWhu?=
 =?utf-8?B?MXNzNmdTUHN0ajdxc1I3eUdrbm9hSURNN2FXdUNUZUNyN213VlVOVW82N1dQ?=
 =?utf-8?B?eExlR3Q1NCt4cjFpc2ZtQkcwR24yMG05ZEovTndEYnRsWTgvdGtaT0RrR0FH?=
 =?utf-8?B?cit5M0lOVDQvdVh6RUF4MjFjMTMrVjFtZ2RjcC9Ta3FDMGExQVhobGJnSHVr?=
 =?utf-8?B?ZWFTdTU4L3c4OTMzSFpSWk8xZFQyS1VEVkxVbmoyMlJ4dDJCQU9mOEM0d3c0?=
 =?utf-8?B?ZGViT3VuUGdWWTF5ODVpMTBJdkdZR2hFZG9WYnV5amRUZXZhTFBodytmcTZG?=
 =?utf-8?B?MEtsa1VucGJoQWM5VmdkQUZ5aWk2b1lrRUNYR0ZraVk3RHZnUVJrNzdHZDNT?=
 =?utf-8?B?R3JnNWZJaE9XWVlTQ3dKVng5UTRCd3FjZ21jQ24zZXVmODhkY2J0eTBndkJZ?=
 =?utf-8?B?bmk2bzJlRDFLbDJzSEFoamJKWGhBSGNBZ1ozVjhZenlSM2MzRjFYczlSTXJP?=
 =?utf-8?B?aEtBNFc1aVluZkZQcmppNloxalNyNDVRTThGamhPVm1RaStxRnl3ZDNBdkJF?=
 =?utf-8?B?QytYTWgramxGbzRYcmdmRFMzYjNhdDhMZmN3QmhxQncyTEFHZVpzYkU2dVBS?=
 =?utf-8?B?bjZBRFBobTMwNWIxRDlIS240SS9WaXBkb2JnbUpWY3ozTTlLWDNpQ1VuSUZZ?=
 =?utf-8?B?NXBjT3hwd3BnUmQ3eXVOU0RTZkl2NkRaUmVGNHhJdnBWWXdwcXdJTUIzbUJk?=
 =?utf-8?B?a0k3cldVanhzUTFoVS8xTVAwNDU4M3RrU3BZZG9rL2syam96aU1GSHBPeUxM?=
 =?utf-8?B?MWdNdGprSjVUN0t2cHJkbUhwdWVYN3VGeWRDdmtuSnRrbEJGS0tiQ0kxY0p0?=
 =?utf-8?B?VW9nK1B5cjN3SXNrT3Bqa0w3bCtEd01kMU43RkZGUnRndWlCUm0vY1FLYTJ1?=
 =?utf-8?B?Mjk2MXB5WXBDZVNvVDdZNnR2bGdhcFhQSkZKc3ViVUVxRzdJMkJNT29PSTlH?=
 =?utf-8?B?cmFHeVRsdTYxZWQrTjlRMzlqSHBWV3d2a1M4Zks5THdIR0NwNzQ0TmZxWmlW?=
 =?utf-8?B?OHZ5aDRsN1VYbWdXNmh0Z1Zqamx0TXJJK1VFV3huZTU4Qmx2c29PUThYV2JC?=
 =?utf-8?Q?wTaR5Pk/NuxGwJ1QmLDqCp/yK?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: d6d89916-7f27-47f6-d0ad-08dbcb65912f
X-MS-Exchange-CrossTenant-AuthSource: CO6PR12MB5427.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 12 Oct 2023 20:55:29.2922
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: lZtzub8KISD5NFHbuo98oJl0i5QsHFrTPFM4cksWvGmXYiiHWyfoOSbew4n9ulLWtdt5Xo5t71CBIaIjp/NcgA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BL1PR12MB5779
X-Spam-Status: No, score=-0.9 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED,
	DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,
	MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=unavailable
	autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on howler.vger.email
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (howler.vger.email [0.0.0.0]); Thu, 12 Oct 2023 13:55:55 -0700 (PDT)


On 2023-10-12 14:49, Hamza Mahfooz wrote:
> The BenQ GW2765 reports that it supports higher (> 8) bpc modes, but
> when trying to set them we end up with a black screen. So, limit it to 8
> bpc modes.
> 
> Cc: stable@vger.kernel.org # 6.5+
> Link: https://gitlab.freedesktop.org/drm/amd/-/issues/2610
> Signed-off-by: Hamza Mahfooz <hamza.mahfooz@amd.com>

Reviewed-by: Harry Wentland <harry.wentland@amd.com>

Harry

> ---
>  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 0454da505687..bca2af4fe1fc 100644
> --- a/drivers/gpu/drm/drm_edid.c
> +++ b/drivers/gpu/drm/drm_edid.c
> @@ -123,6 +123,9 @@ static const struct edid_quirk {
>  	/* AEO model 0 reports 8 bpc, but is a 6 bpc panel */
>  	EDID_QUIRK('A', 'E', 'O', 0, EDID_QUIRK_FORCE_6BPC),
>  
> +	/* BenQ GW2765 */
> +	EDID_QUIRK('B', 'N', 'Q', 0x78d6, EDID_QUIRK_FORCE_8BPC),
> +
>  	/* BOE model on HP Pavilion 15-n233sl reports 8 bpc, but is a 6 bpc panel */
>  	EDID_QUIRK('B', 'O', 'E', 0x78b, EDID_QUIRK_FORCE_6BPC),
>