[flashrom-gerrit] Change in flashrom[master]: SFDP: drop static table length check to make newer SFDP revisions work

[Nico Huber (Code Review)]

Attention is currently required from: Edward O'Callaghan, Felix Singer, Angel Pons, Michael Niewöhner, Anastasia Klimchuk.
Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/60055 )

Change subject: SFDP: drop static table length check to make newer SFDP revisions work
......................................................................

Patch Set 2:

(2 comments)

Commit Message:

https://review.coreboot.org/c/flashrom/+/60055/comment/82ec5a45_539cb5e4 
PS1, Line 28: ***RFC*** We could maintain a list of revisions/lengths but is it really
            : worth it?
            :
> Good question. I think there are three cases: […]

Sounds much like we'd want to check that the length is at least that of 1.0 (or
more specifically the minimum length that covers the last parameter that flashrom
consumes) and that the major version is 1.

File sfdp.c:

https://review.coreboot.org/c/flashrom/+/60055/comment/84654f46_05025282 
PS2, Line 133: 	msg_cdbg("Parsing JEDEC flash parameter table... ");
             : 	msg_cdbg2("\n");
> Looks like these two message can be merged into one now. […]

The line break is only wanted if the next message is at level dbg2 too.