Re: [Intel-gfx] [PATCH v2 00/20] drm/edid: CEA data block iterators, and more
On Thu, 05 May 2022, Ville Syrjälä 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ä 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
Re: [Intel-gfx] [PATCH v2 00/20] drm/edid: CEA data block iterators, and more
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ä > 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 -- Ville Syrjälä Intel
Re: [Intel-gfx] [PATCH v2 00/20] drm/edid: CEA data block iterators, and more
On Tue, 03 May 2022, 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. Argh, forgot to send this series to dri-devel. Instead of sending the series yet another time, I bounced each message to dri-devel, and at least the archive indicates it worked fine. Fingers crossed. BR, Jani. -- Jani Nikula, Intel Open Source Graphics Center
[Intel-gfx] [PATCH v2 00/20] drm/edid: CEA data block iterators, and more
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: detect color formats and CTA revision in all CTA 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 drivers/gpu/drm/drm_edid.c | 799 + 1 file changed, 458 insertions(+), 341 deletions(-) -- 2.30.2