[XS] Change in flashrom[main]: fmap: Update major/minor version check

[Brian Norris (Code Review)]

Attention is currently required from: Anastasia Klimchuk, Brian Norris, Hsuan-ting Chen.

Hello Anastasia Klimchuk, Arthur Heymans, Edward O'Callaghan, Hsuan-ting Chen, build bot (Jenkins),

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/flashrom/+/79060?usp=email

to look at the new patch set (#3).

The following approvals got outdated and were removed:
Code-Review+1 by Anastasia Klimchuk, Code-Review+2 by Hsuan-ting Chen, Verified+1 by build bot (Jenkins)

Change subject: fmap: Update major/minor version check
......................................................................

fmap: Update major/minor version check

It's not valid to separately check the major and minor versions. The
proper minor check would be something like:

if (fmap->ver_major == FMAP_VER_MAJOR &&
      fmap->ver_minor > FMAP_VER_MINOR)
    ERROR();

But this code was alleged (at introduction in [1]) to have come from
cbfstool, and cbfstool doesn't bother with a minor version check. This
check is only for finding the FMAP while searching the flash; it isn't
actually here for integrity and compatibility purpose.

Drop the MINOR version check; align with cbfstool on the MAJOR version
check; and match the cbfstool comments for is_valid_fmap(), to emphasize
the lack of precision.

[1] Commit c82900b66142 ("Add support to get layout from fmap (e.g.
    coreboot rom)")

BRANCH=none
BUG=b:288327526
TEST=libflashrom + ChromiumOS flashmap

Change-Id: I984835579d3b257a2462906f1f5091b179891bd0
Signed-off-by: Brian Norris 
---
M fmap.c
1 file changed, 2 insertions(+), 3 deletions(-)

git pull ssh://review.coreboot.org:29418/flashrom refs/changes/60/79060/3