public inbox for stable@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] drm/edid: add 8 bpc quirk to the BenQ GW2765
@ 2023-10-12 18:49 Hamza Mahfooz
  2023-10-12 20:55 ` Harry Wentland
  2023-10-13 10:30 ` Ville Syrjälä
  0 siblings, 2 replies; 4+ messages in thread
From: Hamza Mahfooz @ 2023-10-12 18:49 UTC (permalink / raw)
  To: dri-devel
  Cc: amd-gfx, Harry Wentland, Alex Deucher, Hamza Mahfooz, stable,
	Maarten Lankhorst, Maxime Ripard, Thomas Zimmermann, David Airlie,
	Daniel Vetter, linux-kernel

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>
---
 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),
 
-- 
2.42.0


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] drm/edid: add 8 bpc quirk to the BenQ GW2765
  2023-10-12 18:49 [PATCH] drm/edid: add 8 bpc quirk to the BenQ GW2765 Hamza Mahfooz
@ 2023-10-12 20:55 ` Harry Wentland
  2023-10-13 10:30 ` Ville Syrjälä
  1 sibling, 0 replies; 4+ messages in thread
From: Harry Wentland @ 2023-10-12 20:55 UTC (permalink / raw)
  To: Hamza Mahfooz, dri-devel
  Cc: amd-gfx, Alex Deucher, stable, Maarten Lankhorst, Maxime Ripard,
	Thomas Zimmermann, David Airlie, Daniel Vetter, linux-kernel



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),
>  


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] drm/edid: add 8 bpc quirk to the BenQ GW2765
  2023-10-12 18:49 [PATCH] drm/edid: add 8 bpc quirk to the BenQ GW2765 Hamza Mahfooz
  2023-10-12 20:55 ` Harry Wentland
@ 2023-10-13 10:30 ` Ville Syrjälä
  2023-10-13 11:37   ` Hamza Mahfooz
  1 sibling, 1 reply; 4+ messages in thread
From: Ville Syrjälä @ 2023-10-13 10:30 UTC (permalink / raw)
  To: Hamza Mahfooz
  Cc: dri-devel, Maxime Ripard, amd-gfx, linux-kernel, stable,
	Thomas Zimmermann, Alex Deucher

On Thu, Oct 12, 2023 at 02:49:27PM -0400, 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.

Bad cable/etc was ruled out as the cause?

> 
> 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>
> ---
>  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),
>  
> -- 
> 2.42.0

-- 
Ville Syrjälä
Intel

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] drm/edid: add 8 bpc quirk to the BenQ GW2765
  2023-10-13 10:30 ` Ville Syrjälä
@ 2023-10-13 11:37   ` Hamza Mahfooz
  0 siblings, 0 replies; 4+ messages in thread
From: Hamza Mahfooz @ 2023-10-13 11:37 UTC (permalink / raw)
  To: Ville Syrjälä
  Cc: Maxime Ripard, dri-devel, linux-kernel, stable, amd-gfx,
	Thomas Zimmermann, Alex Deucher

On 10/13/23 06:30, Ville Syrjälä wrote:
> On Thu, Oct 12, 2023 at 02:49:27PM -0400, 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.
> 
> Bad cable/etc was ruled out as the cause?

Yup, the issue was also reproduced by two different people with same
aforementioned monitor.

> 
>>
>> 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>
>> ---
>>   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),
>>   
>> -- 
>> 2.42.0
> 
-- 
Hamza


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2023-10-13 11:37 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-10-12 18:49 [PATCH] drm/edid: add 8 bpc quirk to the BenQ GW2765 Hamza Mahfooz
2023-10-12 20:55 ` Harry Wentland
2023-10-13 10:30 ` Ville Syrjälä
2023-10-13 11:37   ` Hamza Mahfooz

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox