RE: [PATCH 1/8] drm/edid: Check the number of detailed timing descriptors in the CEA ext block

2020-02-03 Thread Shankar, Uma
t; > Subject: Re: [PATCH 1/8] drm/edid: Check the number of detailed timing > descriptors > in the CEA ext block > > On Fri, Jan 24, 2020 at 3:03 PM Ville Syrjala > wrote: > > > > From: Ville Syrjälä > > > > CEA-861 says : > > "d = offset f

Re: [PATCH 1/8] drm/edid: Check the number of detailed timing descriptors in the CEA ext block

2020-01-28 Thread Ville Syrjälä
On Tue, Jan 28, 2020 at 05:18:34PM +0100, Daniel Vetter wrote: > On Tue, Jan 28, 2020 at 5:15 PM Ville Syrjälä > wrote: > > > > On Tue, Jan 28, 2020 at 04:17:58PM +0100, Daniel Vetter wrote: > > > On Fri, Jan 24, 2020 at 10:02:24PM +0200, Ville Syrjala wrote: > > > > From: Ville Syrjälä > > > > >

Re: [PATCH 1/8] drm/edid: Check the number of detailed timing descriptors in the CEA ext block

2020-01-28 Thread Daniel Vetter
On Tue, Jan 28, 2020 at 5:15 PM Ville Syrjälä wrote: > > On Tue, Jan 28, 2020 at 04:17:58PM +0100, Daniel Vetter wrote: > > On Fri, Jan 24, 2020 at 10:02:24PM +0200, Ville Syrjala wrote: > > > From: Ville Syrjälä > > > > > > CEA-861 says : > > > "d = offset for the byte following the reserved dat

Re: [PATCH 1/8] drm/edid: Check the number of detailed timing descriptors in the CEA ext block

2020-01-28 Thread Ville Syrjälä
On Tue, Jan 28, 2020 at 04:17:58PM +0100, Daniel Vetter wrote: > On Fri, Jan 24, 2020 at 10:02:24PM +0200, Ville Syrjala wrote: > > From: Ville Syrjälä > > > > CEA-861 says : > > "d = offset for the byte following the reserved data block. > > If no data is provided in the reserved data block, th

Re: [PATCH 1/8] drm/edid: Check the number of detailed timing descriptors in the CEA ext block

2020-01-28 Thread Daniel Vetter
On Fri, Jan 24, 2020 at 10:02:24PM +0200, Ville Syrjala wrote: > From: Ville Syrjälä > > CEA-861 says : > "d = offset for the byte following the reserved data block. > If no data is provided in the reserved data block, then d=4. > If no DTDs are provided, then d=0." > > So let's not look for D

Re: [PATCH 1/8] drm/edid: Check the number of detailed timing descriptors in the CEA ext block

2020-01-27 Thread Alex Deucher
On Fri, Jan 24, 2020 at 3:03 PM Ville Syrjala wrote: > > From: Ville Syrjälä > > CEA-861 says : > "d = offset for the byte following the reserved data block. > If no data is provided in the reserved data block, then d=4. > If no DTDs are provided, then d=0." > > So let's not look for DTDs when

[PATCH 1/8] drm/edid: Check the number of detailed timing descriptors in the CEA ext block

2020-01-24 Thread Ville Syrjala
From: Ville Syrjälä CEA-861 says : "d = offset for the byte following the reserved data block. If no data is provided in the reserved data block, then d=4. If no DTDs are provided, then d=0." So let's not look for DTDs when d==0. In fact let's just make that <4 since those values would just me