Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.31 as permitted sender) client-ip=23.128.96.31;
Message-ID: <9d371f71-b7f9-4c4e-b42b-20cabaa42567@amd.com>
Date:   Fri, 13 Oct 2023 07:37:06 -0400
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH] drm/edid: add 8 bpc quirk to the BenQ GW2765
To:     =?UTF-8?B?VmlsbGUgU3lyasOkbMOk?= <ville.syrjala@linux.intel.com>
Cc:     Maxime Ripard <mripard@kernel.org>,
        dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org,
        stable@vger.kernel.org, amd-gfx@lists.freedesktop.org,
        Thomas Zimmermann <tzimmermann@suse.de>,
        Alex Deucher <alexander.deucher@amd.com>
References: <20231012184927.133137-1-hamza.mahfooz@amd.com>
 <ZSkcX1nJ4Ipf2ICd@intel.com>
Content-Language: en-US
From:   Hamza Mahfooz <hamza.mahfooz@amd.com>
In-Reply-To: <ZSkcX1nJ4Ipf2ICd@intel.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?Rmp6OHRwcU9BTnNONGlHY3N0UmoyK2E5VWZiMGhMZk1rVzlCRGh3eGd5Q0Vw?=
 =?utf-8?B?eGJHcnBPMFF1Y2MzaWZ3YWZ0ZmVyZzg5SWk5Mno1Wi9BdGpUNldYS0hqWE9P?=
 =?utf-8?B?UGdaS0RiZVZRMFk2ZTJqLzNjZVBwcjlXVzNpVUw1Y2lCSGIxU0Y1TWZhTTdC?=
 =?utf-8?B?dU9Eczh2NlRmU2I3cC9xUjlUVUpNMTV0RklsVkZXWkdyOHBDdW1jL1RlSEhl?=
 =?utf-8?B?eEtXRDFDQWJzVkdlT0cyRHF1T0JyZzlla3IxUGNBTHFsVlJSc2p1QVlGbnNG?=
 =?utf-8?B?ZE5WOWwvTTFnVmRWTWZmMWhyaHF6SVk0bE5YNCtrUmNZMXMrb1FwYjNxcG5r?=
 =?utf-8?B?LzFyT2F5ejVHdWI3eEZlZjhZTmdsalFRMm5iUDNIR3BneHJzL2xMd2Vpb2JN?=
 =?utf-8?B?UkNCYTZBRHZyOUxRUWRUd1U1VURpd1o2a0JONXBxWXUwY3BqN2VEVkl2c3dn?=
 =?utf-8?B?MHNzRnBhVEJXUFdVcGVXREdvdmZLOWRtV3ZuK2h0YVRZekFlNjA4ZXFpZlQ5?=
 =?utf-8?B?Qi9SdEN1bzlncDFURUZ1NWkxMk9uWWptdldLRzdOUThPN2YrVVhPbit3VHU5?=
 =?utf-8?B?NnV1NTd6bkpaUjNEWGQ0bGxOYm5sTEZKYk54V084ZkNONkFCNjIzSXhBMXlW?=
 =?utf-8?B?Q2lrWDAvbDJmOXR4QlE2TmpNSXphM3RQQnozQ0QzaHd0cnk0K25BZm84M29I?=
 =?utf-8?B?bjcxZXBibnhqdThoeEhhdEJLaFpYd01ZYTk0aDgvZFo5a0luT0g5amxrYkdE?=
 =?utf-8?B?UGQwYWRhZUF2YjAzUmxwbytvNHVjc3N0aUtpQktLb2ZGRFNMU3JwNjhTVzBJ?=
 =?utf-8?B?bDVQWGZ3ZDFrb2NSNDJvU0NBMjhDNXN6b0FwdEMvbVRMYUxsc0czTHRoQkR0?=
 =?utf-8?B?OFRZUkJFYTJvZW1DMHZPcHlDeXNxcms5UEJJaGpjdnFDcHNZeFI4NWZhR09E?=
 =?utf-8?B?aEtCK0xhRXU1VzY2eVVvMkptdXhucjdyc2ZKM1ljNFVLd2h1SUJ2R3daSVhn?=
 =?utf-8?B?a2x2elR4aXZNUXFYS3J2TlFrdkQ4UEFpN0tucEFHSUVWaUtLRER2NjJsRlBH?=
 =?utf-8?B?bDFNY3pFNG40TkpmMXEzZkZZYXNvalZwbVFiWHF6NHBGdVZyWmttdTVJMHZX?=
 =?utf-8?B?cmdpaFgvTDcwWVlHV0sxdmp4b1hGc2R2NlIwWGdETEdTZklpbU1VL1BPWHZs?=
 =?utf-8?B?dXc0RURRLzFpMEFxdGF5SWI3MnJMNlppY0ZTdDJFTW5kbmZwbzA0RFJkR1gx?=
 =?utf-8?B?NCtpdG0wWnd4T2ZzVzF0T1J5T1E4TE43OU5iUk5mU2xDTURMelhXTWpoVU51?=
 =?utf-8?B?NldNMnJ2SU5lT2Y5dGYwM0JVakFYV2ZxbTFscVdJa2Z6UEVodlhheDdjZ21H?=
 =?utf-8?B?V0hMR2t2TmNuT01nNy82ZWlCeUtzTGZWc3JwdndCUHVSZDNrZm51SU85N2g0?=
 =?utf-8?B?aFd4K09lMVFsRlhLbE1la3orcERIR2cxeERFeURBQkFTMUhuMkJhMmQ2Z2g3?=
 =?utf-8?B?dGt1WjdMUExGMXBZdFJNaE9jQTVjcGdZbTJBVyt5Qng0Szg0UlZuaGV0WHpj?=
 =?utf-8?B?MEEvZHhpSVo4YTMrTk5MMDhZdnMxd3lkYTJhV0dSTzY0ZkpkN0d4ZCtDVTk1?=
 =?utf-8?B?OWtjd1d2cVl6cWsyOUFITnAxVTJDUGt2SHJnR1JxbEkvWlFWYnYvL1gyZ0Jo?=
 =?utf-8?B?aHY3dkw2RFlaTExIQXJzRWpvOXJIcXdUUFBhcWQxampCY3E0WlFkRmJtZzNV?=
 =?utf-8?B?TnhpZHVUNUdiRGx6djhLdEcrSFZieGo2Q0U4aTRlSWxwMVJmSm82ay9kOUZu?=
 =?utf-8?B?NkpJNGtnY2F0b2lZTmwzUmRCTTh2MWlYY1pmdjl3eURXQmNwUXJRQ3FzRlN5?=
 =?utf-8?B?anZVVks4T0EzbHp6d0VGcW1RYkJuMmFCTThacTNSZ3B2MnVlTmNsSUdlZ2ZO?=
 =?utf-8?B?NGdqV3ljcVlGMTd3am1JN0t1YkM1WlZITjJiV204c0tMMkRRKzhpSVQwQ0ZI?=
 =?utf-8?B?RFR6SGFhTkErUHl1UmIxY2NzbTZaOUduQldYOUh2N2xIbEVLRlJUdmNsK0c0?=
 =?utf-8?B?ZlQ0cG9JTExhYjc2WWdCUmR6a0VFQlB4aHl2Z0JtcVRQTS80bWp1VFU0L091?=
 =?utf-8?Q?t1Tx/u7EIqNyZ0hdC4k6GyCAV?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: a29d453e-c095-4d8d-409b-08dbcbe0bc8f
X-MS-Exchange-CrossTenant-AuthSource: DM4PR12MB6280.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 13 Oct 2023 11:37:10.1446
 (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: 1Jn0B3zQGmHEosataAYoPBF5GRaomaTAHJcEnCdrrA8x+ISH3xSgoP9x2bMznAM9Q5HCFOujdoRlMGFLDp8OGQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH3PR12MB8534
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 morse.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 (morse.vger.email [0.0.0.0]); Fri, 13 Oct 2023 04:37:26 -0700 (PDT)

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