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

[Anastasia Klimchuk (Code Review)]

Attention is currently required from: Edward O'Callaghan, Felix Singer, Nico Huber, Angel Pons, Michael Niewöhner.
Anastasia Klimchuk 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)

File sfdp.c:

https://review.coreboot.org/c/flashrom/+/60055/comment/f8ac5ed6_451c9bdb 
PS2, Line 133: 	msg_cdbg("Parsing JEDEC flash parameter table... ");
             : 	msg_cdbg2("\n");
Looks like these two message can be merged into one now. It is not very clear why first one is cdbg and second (which is just a new line) is cdbg2?
Since this is now only informational, not an error message, that's probably cdbg2?

https://review.coreboot.org/c/flashrom/+/60055/comment/b84834da_af35021d 
PS2, Line 372: } else if (sfdp_fill_flash(flash->chip, tbuf, len) == 0)
Oh so length of table was checked twice?