[coreboot-gerrit] [XS] Change in coreboot[main]: util/ifdtool: Enable Read Access for SPI device expansion 2 region

[Subrata Banik (Code Review)]

Attention is currently required from: Kane Chen, Kapil Porwal, Kun Liu, Reka Norman, Stefan Reinauer.

Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/79902?usp=email )

Change subject: util/ifdtool: Enable Read Access for SPI device expansion 2 region
......................................................................

Patch Set 1:

(1 comment)

File util/ifdtool/ifdtool.c:

https://review.coreboot.org/c/coreboot/+/79902/comment/aac15bad_17fb7df1 :
PS1, Line 65: "Device Exp2
> Does the name need to be updated or do they mean the same thing? (I'm not really sure what either device expansion or SPI padding regions are used for.)

I believe this name field is kind of volatile meaning it's like a reserved range for Intel and keeps on getting change over generation. Let me make some modification to keep "Device Exp2" uniformly w/o adding "SPI padding region"

I have lost ur CR+2 in this process. requesting you to please take a relook.