Re: [Intel-gfx] [PATCH v2 00/20] drm/edid: CEA data block iterators, and more

[Jani Nikula <jani.nikula@intel.com>]

On Thu, 05 May 2022, Ville Syrjälä <ville.syrjala@linux.intel.com> wrote:
> On Tue, May 03, 2022 at 12:23:45PM +0300, Jani Nikula wrote:
>> I've kind of lost track of the version numbers on some of the iterator
>> patches, but this is the next version (or mostly a resend) of
>> [1]. There's an additional rename patch for SCDS.
>> 
>> BR,
>> Jani.
>> 
>> 
>> [1] https://patchwork.freedesktop.org/series/102703/
>> 
>> 
>> Jani Nikula (19):
>>   drm/edid: reset display info in drm_add_edid_modes() for NULL edid
>>   drm/edid: rename HDMI Forum VSDB to SCDS
>>   drm/edid: clean up CTA data block tag definitions
>>   drm/edid: add iterator for EDID base and extension blocks
>>   drm/edid: add iterator for CTA data blocks
>>   drm/edid: clean up cea_db_is_*() functions
>>   drm/edid: convert add_cea_modes() to use cea db iter
>>   drm/edid: convert drm_edid_to_speaker_allocation() to use cea db iter
>>   drm/edid: convert drm_edid_to_sad() to use cea db iter
>>   drm/edid: convert drm_detect_hdmi_monitor() to use cea db iter
>>   drm/edid: convert drm_detect_monitor_audio() to use cea db iter
>>   drm/edid: convert drm_parse_cea_ext() to use cea db iter
>>   drm/edid: convert drm_edid_to_eld() to use cea db iter
>>   drm/edid: sunset the old unused cea data block iterators
>>   drm/edid: restore some type safety to cea_db_*() functions
>>   drm/edid: detect basic audio in all CEA extensions
>>   drm/edid: skip CTA extension scan in drm_edid_to_eld() just for CTA
>>     rev
>>   drm/edid: sunset drm_find_cea_extension()
>> 
>> Lee Shawn C (1):
>>   drm/edid: check for HF-SCDB block
>
> All of the above patches look OK to me.
> Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>

Thanks a bunch, pushed the lot to drm-misc-next.

BR,
Jani.


>
>>   drm/edid: detect color formats and CTA revision in all CTA extensions
>
> For this one I'm not entirely convinced the behavioural change
> for the no-CTA ext case is what we want. Replied to that one
> individually.
>
>> 
>>  drivers/gpu/drm/drm_edid.c | 799 +++++++++++++++++++++----------------
>>  1 file changed, 458 insertions(+), 341 deletions(-)
>> 
>> -- 
>> 2.30.2

-- 
Jani Nikula, Intel Open Source Graphics Center