[00/28] bug fixes, additions, changes

Message

joevt Sept. 24, 2021, 11:31 p.m. UTC

- Patch 02/28 replaces my previous patch 05/11.
It adds a new warning message.

- The problem with my previous patch 07/11 is addressed in patch 01/28 and 04/28 (and maybe elsewhere).

- Patches 06/28 to 26/28 replace my previous patch 11/11 (broken down into smaller steps this time).
They contain some corrections.

- Includes some other changes.


joevt (28):
  edid-decode: remove unnecessary length check
  edid-decode: fix standard timing vertical pixels
  edid-decode: exclude oui from _block functions
  edid-decode: check cta_hdr10plus length
  edid-decode: Capitalize fail sentence
  edid-decode: Replace return with break in switch
  edid-decode: extended tag length check
  edid-decode: Output block type before fail
  edid-decode: update Microsoft expected length
  edid-decode: Capitalize fail sentence
  edid-decode: make all OUI handlers the same
  edid-decode: move OUI parsing to separate function
  edid-decode: move unknown block warning
  edid-decode: remove cta_ext_block
  edid-decode: change unknown CTA block names
  edid-decode: move audio fail/warn messages
  edid-decode: replace first_block with block_number
  edid-decode: move parse_displayid_block inner loop
  edid-decode: remove offset from displayid_block
  edid-decode: displayid_block len fixes
  edid-decode: ignore DisplayID version for OUI check.
  edid-decode: DisplayID non-0 filler fixes
  edid-decode: DisplayID length checks
  edid-decode: make OUI enum
  edid-decode: more OUI changes
  edid-decode: remove extra vendor field
  edid-decode: re-add one EDID
  edid-decode: add interesting EDID

 data/apple-imac-retina-4k-21.5-inch-late-2015 | Bin 0 -> 256 bytes
 data/vizio-m60c3-hdmi-onkyo-txnr555           | Bin 0 -> 256 bytes
 edid-decode.cpp                               | 106 +++-
 edid-decode.h                                 |  32 +-
 oui.h                                         |  20 +
 parse-base-block.cpp                          |   5 +-
 parse-cta-block.cpp                           | 482 +++++++----------
 parse-displayid-block.cpp                     | 498 +++++++++---------
 8 files changed, 553 insertions(+), 590 deletions(-)
 create mode 100644 data/apple-imac-retina-4k-21.5-inch-late-2015
 create mode 100644 oui.h

Comments

Hans Verkuil Sept. 25, 2021, 9:05 a.m. UTC | #1

Hi Joe,

This series is missing patches 20-28 (at least, I never received them, and neither
did they appear on the linux-media mailinglist).

It also needs to be rebased to the latest edid-decode git repo master branch since
it no longer applies.

Regards,

	Hans

On 25/09/2021 01:31, joevt wrote:
> - Patch 02/28 replaces my previous patch 05/11.
> It adds a new warning message.
> 
> - The problem with my previous patch 07/11 is addressed in patch 01/28 and 04/28 (and maybe elsewhere).
> 
> - Patches 06/28 to 26/28 replace my previous patch 11/11 (broken down into smaller steps this time).
> They contain some corrections.
> 
> - Includes some other changes.
> 
> 
> joevt (28):
>   edid-decode: remove unnecessary length check
>   edid-decode: fix standard timing vertical pixels
>   edid-decode: exclude oui from _block functions
>   edid-decode: check cta_hdr10plus length
>   edid-decode: Capitalize fail sentence
>   edid-decode: Replace return with break in switch
>   edid-decode: extended tag length check
>   edid-decode: Output block type before fail
>   edid-decode: update Microsoft expected length
>   edid-decode: Capitalize fail sentence
>   edid-decode: make all OUI handlers the same
>   edid-decode: move OUI parsing to separate function
>   edid-decode: move unknown block warning
>   edid-decode: remove cta_ext_block
>   edid-decode: change unknown CTA block names
>   edid-decode: move audio fail/warn messages
>   edid-decode: replace first_block with block_number
>   edid-decode: move parse_displayid_block inner loop
>   edid-decode: remove offset from displayid_block
>   edid-decode: displayid_block len fixes
>   edid-decode: ignore DisplayID version for OUI check.
>   edid-decode: DisplayID non-0 filler fixes
>   edid-decode: DisplayID length checks
>   edid-decode: make OUI enum
>   edid-decode: more OUI changes
>   edid-decode: remove extra vendor field
>   edid-decode: re-add one EDID
>   edid-decode: add interesting EDID
> 
>  data/apple-imac-retina-4k-21.5-inch-late-2015 | Bin 0 -> 256 bytes
>  data/vizio-m60c3-hdmi-onkyo-txnr555           | Bin 0 -> 256 bytes
>  edid-decode.cpp                               | 106 +++-
>  edid-decode.h                                 |  32 +-
>  oui.h                                         |  20 +
>  parse-base-block.cpp                          |   5 +-
>  parse-cta-block.cpp                           | 482 +++++++----------
>  parse-displayid-block.cpp                     | 498 +++++++++---------
>  8 files changed, 553 insertions(+), 590 deletions(-)
>  create mode 100644 data/apple-imac-retina-4k-21.5-inch-late-2015
>  create mode 100644 oui.h
>