mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Thomas Zimmermann <tzimmermann@suse.de>
To: Maxime Ripard <mripard@kernel.org>,
	Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
	David Airlie <airlied@gmail.com>, Simona Vetter <simona@ffwll.ch>
Cc: Dmitry Baryshkov <lumag@kernel.org>,
	dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 2/3] drm/tests: edid: Update CTA-861 HDMI Vendor Specific Data Block
Date: Mon, 14 Jul 2025 13:02:33 +0200	[thread overview]
Message-ID: <046193df-bb11-4d84-98a4-c6d46d359316@suse.de> (raw)
In-Reply-To: <20250625-drm-update-edid-v1-2-2d963743ab9e@kernel.org>

Hi

Am 25.06.25 um 17:14 schrieb Maxime Ripard:
> For some reason, the HDMI VSDBs in our kunit EDIDs had a length longer
> than expected.
>
> While this was harmless, we should get rid of it to make it somewhat
> predictable.

Dump question: should these errors be kept in another test specifically 
for detecting this problem?

Best regards
Thomas

>
> Signed-off-by: Maxime Ripard <mripard@kernel.org>
> ---
>   drivers/gpu/drm/tests/drm_kunit_edid.h | 85 ++++++++++++++++------------------
>   1 file changed, 40 insertions(+), 45 deletions(-)
>
> diff --git a/drivers/gpu/drm/tests/drm_kunit_edid.h b/drivers/gpu/drm/tests/drm_kunit_edid.h
> index ac311804b36ccb21f865ecec90dd2557bff0e4b2..14782a3840f00c50c7daee38bba98585003c13a2 100644
> --- a/drivers/gpu/drm/tests/drm_kunit_edid.h
> +++ b/drivers/gpu/drm/tests/drm_kunit_edid.h
> @@ -71,18 +71,18 @@ static const unsigned char test_edid_dvi_1080p[] = {
>    * 45 00 40 84 63 00 00 1e 00 00 00 fc 00 54 65 73
>    * 74 20 45 44 49 44 0a 20 20 20 00 00 00 fd 00 32
>    * 46 1e 46 0f 00 0a 20 20 20 20 20 20 00 00 00 10
>    * 00 00 00 00 00 00 00 00 00 00 00 00 00 00 01 92
>    *
> - * 02 03 1b 81 e3 05 00 20 41 10 e2 00 4a 6d 03 0c
> - * 00 12 34 00 14 20 00 00 00 00 00 00 00 00 00 00
> + * 02 03 15 81 e3 05 00 20 41 10 e2 00 4a 67 03 0c
> + * 00 12 34 00 14 00 00 00 00 00 00 00 00 00 00 00
>    * 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
>    * 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
>    * 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
>    * 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
>    * 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
> - * 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 e4
> + * 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 10
>    *
>    * ----------------
>    *
>    * Block 0, Base EDID:
>    *   EDID Structure Version & Revision: 1.3
> @@ -133,12 +133,11 @@ static const unsigned char test_edid_dvi_1080p[] = {
>    *     IT scan behavior: Always Underscanned
>    *     CE scan behavior: Always Underscanned
>    *   Vendor-Specific Data Block (HDMI), OUI 00-0C-03:
>    *     Source physical address: 1.2.3.4
>    *     Maximum TMDS clock: 100 MHz
> - *     Extended HDMI video details:
> - * Checksum: 0xe4  Unused space in Extension Block: 100 bytes
> + * Checksum: 0x10  Unused space in Extension Block: 106 bytes
>    */
>   static const unsigned char test_edid_hdmi_1080p_rgb_max_100mhz[] = {
>   	0x00, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0x00, 0x31, 0xd8, 0x2a, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x21, 0x01, 0x03, 0x81, 0xa0, 0x5a, 0x78,
>   	0x02, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x20,
> @@ -147,22 +146,22 @@ static const unsigned char test_edid_hdmi_1080p_rgb_max_100mhz[] = {
>   	0x2d, 0x40, 0x58, 0x2c, 0x45, 0x00, 0x40, 0x84, 0x63, 0x00, 0x00, 0x1e,
>   	0x00, 0x00, 0x00, 0xfc, 0x00, 0x54, 0x65, 0x73, 0x74, 0x20, 0x45, 0x44,
>   	0x49, 0x44, 0x0a, 0x20, 0x20, 0x20, 0x00, 0x00, 0x00, 0xfd, 0x00, 0x32,
>   	0x46, 0x1e, 0x46, 0x0f, 0x00, 0x0a, 0x20, 0x20, 0x20, 0x20, 0x20, 0x20,
>   	0x00, 0x00, 0x00, 0x10, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
> -	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x01, 0x92, 0x02, 0x03, 0x1b, 0x81,
> -	0xe3, 0x05, 0x00, 0x20, 0x41, 0x10, 0xe2, 0x00, 0x4a, 0x6d, 0x03, 0x0c,
> -	0x00, 0x12, 0x34, 0x00, 0x14, 0x20, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
> +	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x01, 0x92, 0x02, 0x03, 0x15, 0x81,
> +	0xe3, 0x05, 0x00, 0x20, 0x41, 0x10, 0xe2, 0x00, 0x4a, 0x67, 0x03, 0x0c,
> +	0x00, 0x12, 0x34, 0x00, 0x14, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
> -	0x00, 0x00, 0x00, 0xe4
> +	0x00, 0x00, 0x00, 0x10
>   };
>   
>   /*
>    * edid-decode (hex):
>    *
> @@ -173,18 +172,18 @@ static const unsigned char test_edid_hdmi_1080p_rgb_max_100mhz[] = {
>    * 45 00 40 84 63 00 00 1e 00 00 00 fc 00 54 65 73
>    * 74 20 45 44 49 44 0a 20 20 20 00 00 00 fd 00 32
>    * 46 1e 46 0f 00 0a 20 20 20 20 20 20 00 00 00 10
>    * 00 00 00 00 00 00 00 00 00 00 00 00 00 00 01 92
>    *
> - * 02 03 1b 81 e3 05 00 20 41 10 e2 00 4a 6d 03 0c
> - * 00 12 34 00 28 20 00 00 00 00 00 00 00 00 00 00
> + * 02 03 15 81 e3 05 00 20 41 10 e2 00 4a 67 03 0c
> + * 00 12 34 00 28 00 00 00 00 00 00 00 00 00 00 00
>    * 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
>    * 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
>    * 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
>    * 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
>    * 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
> - * 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 d0
> + * 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 fc
>    *
>    * ----------------
>    *
>    * Block 0, Base EDID:
>    *   EDID Structure Version & Revision: 1.3
> @@ -235,12 +234,11 @@ static const unsigned char test_edid_hdmi_1080p_rgb_max_100mhz[] = {
>    *     IT scan behavior: Always Underscanned
>    *     CE scan behavior: Always Underscanned
>    *   Vendor-Specific Data Block (HDMI), OUI 00-0C-03:
>    *     Source physical address: 1.2.3.4
>    *     Maximum TMDS clock: 200 MHz
> - *     Extended HDMI video details:
> - * Checksum: 0xd0  Unused space in Extension Block: 100 bytes
> + * Checksum: 0xfc  Unused space in Extension Block: 106 bytes
>    */
>   static const unsigned char test_edid_hdmi_1080p_rgb_max_200mhz[] = {
>   	0x00, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0x00, 0x31, 0xd8, 0x2a, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x21, 0x01, 0x03, 0x81, 0xa0, 0x5a, 0x78,
>   	0x02, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x20,
> @@ -249,22 +247,22 @@ static const unsigned char test_edid_hdmi_1080p_rgb_max_200mhz[] = {
>   	0x2d, 0x40, 0x58, 0x2c, 0x45, 0x00, 0x40, 0x84, 0x63, 0x00, 0x00, 0x1e,
>   	0x00, 0x00, 0x00, 0xfc, 0x00, 0x54, 0x65, 0x73, 0x74, 0x20, 0x45, 0x44,
>   	0x49, 0x44, 0x0a, 0x20, 0x20, 0x20, 0x00, 0x00, 0x00, 0xfd, 0x00, 0x32,
>   	0x46, 0x1e, 0x46, 0x0f, 0x00, 0x0a, 0x20, 0x20, 0x20, 0x20, 0x20, 0x20,
>   	0x00, 0x00, 0x00, 0x10, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
> -	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x01, 0x92, 0x02, 0x03, 0x1b, 0x81,
> -	0xe3, 0x05, 0x00, 0x20, 0x41, 0x10, 0xe2, 0x00, 0x4a, 0x6d, 0x03, 0x0c,
> -	0x00, 0x12, 0x34, 0x00, 0x28, 0x20, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
> +	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x01, 0x92, 0x02, 0x03, 0x15, 0x81,
> +	0xe3, 0x05, 0x00, 0x20, 0x41, 0x10, 0xe2, 0x00, 0x4a, 0x67, 0x03, 0x0c,
> +	0x00, 0x12, 0x34, 0x00, 0x28, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
> -	0x00, 0x00, 0x00, 0xd0
> +	0x00, 0x00, 0x00, 0xfc
>   };
>   
>   /*
>    * edid-decode (hex):
>    *
> @@ -275,18 +273,18 @@ static const unsigned char test_edid_hdmi_1080p_rgb_max_200mhz[] = {
>    * 45 00 40 84 63 00 00 1e 00 00 00 fc 00 54 65 73
>    * 74 20 45 44 49 44 0a 20 20 20 00 00 00 fd 00 32
>    * 46 1e 46 0f 00 0a 20 20 20 20 20 20 00 00 00 10
>    * 00 00 00 00 00 00 00 00 00 00 00 00 00 00 01 92
>    *
> - * 02 03 1b 81 e3 05 00 20 41 10 e2 00 4a 6d 03 0c
> - * 00 12 34 00 28 20 00 00 00 00 00 00 00 00 00 00
> + * 02 03 15 81 e3 05 00 20 41 10 e2 00 4a 67 03 0c
> + * 00 12 34 00 44 00 00 00 00 00 00 00 00 00 00 00
>    * 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
>    * 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
>    * 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
>    * 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
>    * 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
> - * 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 d0
> + * 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 e0
>    *
>    * ----------------
>    *
>    * Block 0, Base EDID:
>    *   EDID Structure Version & Revision: 1.3
> @@ -337,12 +335,11 @@ static const unsigned char test_edid_hdmi_1080p_rgb_max_200mhz[] = {
>    *     IT scan behavior: Always Underscanned
>    *     CE scan behavior: Always Underscanned
>    *   Vendor-Specific Data Block (HDMI), OUI 00-0C-03:
>    *     Source physical address: 1.2.3.4
>    *     Maximum TMDS clock: 340 MHz
> - *     Extended HDMI video details:
> - * Checksum: 0xd0  Unused space in Extension Block: 100 bytes
> + * Checksum: 0xe0  Unused space in Extension Block: 106 bytes
>    */
>   static const unsigned char test_edid_hdmi_1080p_rgb_max_340mhz[] = {
>   	0x00, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0x00, 0x31, 0xd8, 0x2a, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x21, 0x01, 0x03, 0x81, 0xa0, 0x5a, 0x78,
>   	0x02, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x20,
> @@ -351,22 +348,22 @@ static const unsigned char test_edid_hdmi_1080p_rgb_max_340mhz[] = {
>   	0x2d, 0x40, 0x58, 0x2c, 0x45, 0x00, 0x40, 0x84, 0x63, 0x00, 0x00, 0x1e,
>   	0x00, 0x00, 0x00, 0xfc, 0x00, 0x54, 0x65, 0x73, 0x74, 0x20, 0x45, 0x44,
>   	0x49, 0x44, 0x0a, 0x20, 0x20, 0x20, 0x00, 0x00, 0x00, 0xfd, 0x00, 0x32,
>   	0x46, 0x1e, 0x46, 0x0f, 0x00, 0x0a, 0x20, 0x20, 0x20, 0x20, 0x20, 0x20,
>   	0x00, 0x00, 0x00, 0x10, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
> -	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x01, 0x92, 0x02, 0x03, 0x1b, 0x81,
> -	0xe3, 0x05, 0x00, 0x20, 0x41, 0x10, 0xe2, 0x00, 0x4a, 0x6d, 0x03, 0x0c,
> -	0x00, 0x12, 0x34, 0x00, 0x44, 0x20, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
> +	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x01, 0x92, 0x02, 0x03, 0x15, 0x81,
> +	0xe3, 0x05, 0x00, 0x20, 0x41, 0x10, 0xe2, 0x00, 0x4a, 0x67, 0x03, 0x0c,
> +	0x00, 0x12, 0x34, 0x00, 0x44, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
> -	0x00, 0x00, 0x00, 0xd0
> +	0x00, 0x00, 0x00, 0xe0
>   };
>   
>   /*
>    * edid-decode (hex):
>    *
> @@ -377,18 +374,18 @@ static const unsigned char test_edid_hdmi_1080p_rgb_max_340mhz[] = {
>    * 45 00 40 84 63 00 00 1e 00 00 00 fc 00 54 65 73
>    * 74 20 45 44 49 44 0a 20 20 20 00 00 00 fd 00 32
>    * 46 1e 46 0f 00 0a 20 20 20 20 20 20 00 00 00 10
>    * 00 00 00 00 00 00 00 00 00 00 00 00 00 00 01 7a
>    *
> - * 02 03 1b b1 e3 05 00 20 41 10 e2 00 ca 6d 03 0c
> - * 00 12 34 78 28 20 00 00 00 00 00 00 00 00 00 00
> + * 02 03 15 b1 e3 05 00 20 41 10 e2 00 ca 67 03 0c
> + * 00 12 34 78 28 00 00 00 00 00 00 00 00 00 00 00
>    * 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
>    * 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
>    * 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
>    * 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
>    * 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
> - * 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 a8
> + * 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 d4
>    *
>    * ----------------
>    *
>    * Block 0, Base EDID:
>    *   EDID Structure Version & Revision: 1.3
> @@ -445,12 +442,11 @@ static const unsigned char test_edid_hdmi_1080p_rgb_max_340mhz[] = {
>    *     DC_48bit
>    *     DC_36bit
>    *     DC_30bit
>    *     DC_Y444
>    *     Maximum TMDS clock: 200 MHz
> - *     Extended HDMI video details:
> - * Checksum: 0xa8  Unused space in Extension Block: 100 bytes
> + * Checksum: 0xd4  Unused space in Extension Block: 106 bytes
>    */
>   static const unsigned char test_edid_hdmi_1080p_rgb_yuv_dc_max_200mhz[] = {
>   	0x00, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0x00, 0x31, 0xd8, 0x2a, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x21, 0x01, 0x03, 0x81, 0xa0, 0x5a, 0x78,
>   	0x1a, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x20,
> @@ -459,22 +455,22 @@ static const unsigned char test_edid_hdmi_1080p_rgb_yuv_dc_max_200mhz[] = {
>   	0x2d, 0x40, 0x58, 0x2c, 0x45, 0x00, 0x40, 0x84, 0x63, 0x00, 0x00, 0x1e,
>   	0x00, 0x00, 0x00, 0xfc, 0x00, 0x54, 0x65, 0x73, 0x74, 0x20, 0x45, 0x44,
>   	0x49, 0x44, 0x0a, 0x20, 0x20, 0x20, 0x00, 0x00, 0x00, 0xfd, 0x00, 0x32,
>   	0x46, 0x1e, 0x46, 0x0f, 0x00, 0x0a, 0x20, 0x20, 0x20, 0x20, 0x20, 0x20,
>   	0x00, 0x00, 0x00, 0x10, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
> -	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x01, 0x7a, 0x02, 0x03, 0x1b, 0xb1,
> -	0xe3, 0x05, 0x00, 0x20, 0x41, 0x10, 0xe2, 0x00, 0xca, 0x6d, 0x03, 0x0c,
> -	0x00, 0x12, 0x34, 0x78, 0x28, 0x20, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
> +	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x01, 0x7a, 0x02, 0x03, 0x15, 0xb1,
> +	0xe3, 0x05, 0x00, 0x20, 0x41, 0x10, 0xe2, 0x00, 0xca, 0x67, 0x03, 0x0c,
> +	0x00, 0x12, 0x34, 0x78, 0x28, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
> -	0x00, 0x00, 0x00, 0xa8
> +	0x00, 0x00, 0x00, 0xd4
>   };
>   
>   /*
>    * edid-decode (hex):
>    *
> @@ -485,18 +481,18 @@ static const unsigned char test_edid_hdmi_1080p_rgb_yuv_dc_max_200mhz[] = {
>    * 45 00 40 84 63 00 00 1e 00 00 00 fc 00 54 65 73
>    * 74 20 45 44 49 44 0a 20 20 20 00 00 00 fd 00 32
>    * 46 1e 46 0f 00 0a 20 20 20 20 20 20 00 00 00 10
>    * 00 00 00 00 00 00 00 00 00 00 00 00 00 00 01 8a
>    *
> - * 02 03 1b b1 e3 05 00 20 41 10 e2 00 ca 6d 03 0c
> - * 00 12 34 78 44 20 00 00 00 00 00 00 00 00 00 00
> + * 02 03 15 b1 e3 05 00 20 41 10 e2 00 ca 67 03 0c
> + * 00 12 34 78 44 00 00 00 00 00 00 00 00 00 00 00
>    * 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
>    * 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
>    * 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
>    * 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
>    * 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
> - * 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 8c
> + * 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 b8
>    *
>    * ----------------
>    *
>    * Block 0, Base EDID:
>    *   EDID Structure Version & Revision: 1.3
> @@ -553,12 +549,11 @@ static const unsigned char test_edid_hdmi_1080p_rgb_yuv_dc_max_200mhz[] = {
>    *     DC_48bit
>    *     DC_36bit
>    *     DC_30bit
>    *     DC_Y444
>    *     Maximum TMDS clock: 340 MHz
> - *     Extended HDMI video details:
> - * Checksum: 0x8c  Unused space in Extension Block: 100 bytes
> + * Checksum: 0xb8  Unused space in Extension Block: 106 bytes
>    */
>   static const unsigned char test_edid_hdmi_1080p_rgb_yuv_dc_max_340mhz[] = {
>   	0x00, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0x00, 0x31, 0xd8, 0x2a, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x21, 0x01, 0x03, 0x81, 0xa0, 0x5a, 0x78,
>   	0x0a, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x20,
> @@ -567,20 +562,20 @@ static const unsigned char test_edid_hdmi_1080p_rgb_yuv_dc_max_340mhz[] = {
>   	0x2d, 0x40, 0x58, 0x2c, 0x45, 0x00, 0x40, 0x84, 0x63, 0x00, 0x00, 0x1e,
>   	0x00, 0x00, 0x00, 0xfc, 0x00, 0x54, 0x65, 0x73, 0x74, 0x20, 0x45, 0x44,
>   	0x49, 0x44, 0x0a, 0x20, 0x20, 0x20, 0x00, 0x00, 0x00, 0xfd, 0x00, 0x32,
>   	0x46, 0x1e, 0x46, 0x0f, 0x00, 0x0a, 0x20, 0x20, 0x20, 0x20, 0x20, 0x20,
>   	0x00, 0x00, 0x00, 0x10, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
> -	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x01, 0x8a, 0x02, 0x03, 0x1b, 0xb1,
> -	0xe3, 0x05, 0x00, 0x20, 0x41, 0x10, 0xe2, 0x00, 0xca, 0x6d, 0x03, 0x0c,
> -	0x00, 0x12, 0x34, 0x78, 0x44, 0x20, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
> +	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x01, 0x8a, 0x02, 0x03, 0x15, 0xb1,
> +	0xe3, 0x05, 0x00, 0x20, 0x41, 0x10, 0xe2, 0x00, 0xca, 0x67, 0x03, 0x0c,
> +	0x00, 0x12, 0x34, 0x78, 0x44, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
>   	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
> -	0x00, 0x00, 0x00, 0x8c
> +	0x00, 0x00, 0x00, 0xb8
>   };
>   
>   #endif // DRM_KUNIT_EDID_H_
>

-- 
--
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Frankenstrasse 146, 90461 Nuernberg, Germany
GF: Ivo Totev, Andrew Myers, Andrew McDonald, Boudien Moerman
HRB 36809 (AG Nuernberg)


  parent reply	other threads:[~2025-07-14 11:02 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-06-25 15:14 [PATCH 0/3] drm/tests: edid: Improve our kunit EDIDs Maxime Ripard
2025-06-25 15:14 ` [PATCH 1/3] drm/tests: edid: Fix monitor range limits Maxime Ripard
2025-07-10 12:09   ` Javier Martinez Canillas
2025-06-25 15:14 ` [PATCH 2/3] drm/tests: edid: Update CTA-861 HDMI Vendor Specific Data Block Maxime Ripard
2025-07-10 12:13   ` Javier Martinez Canillas
2025-07-14 11:02   ` Thomas Zimmermann [this message]
2025-07-16 15:06     ` Maxime Ripard
2025-07-16 15:34       ` Thomas Zimmermann
2025-07-28  8:37         ` Jani Nikula
2025-06-25 15:14 ` [PATCH 3/3] drm/tests: edid: Add edid-decode --check output Maxime Ripard
2025-07-10 12:16   ` Javier Martinez Canillas

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=046193df-bb11-4d84-98a4-c6d46d359316@suse.de \
    --to=tzimmermann@suse.de \
    --cc=airlied@gmail.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lumag@kernel.org \
    --cc=maarten.lankhorst@linux.intel.com \
    --cc=mripard@kernel.org \
    --cc=simona@ffwll.ch \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®