Hello build bot (Jenkins), Lijian Zhao, Furquan Shaikh,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/33273
to look at the new patch set (#2).
Change subject: ifdtool: Enable GbE/PDR region access if they exist ......................................................................
ifdtool: Enable GbE/PDR region access if they exist
Instead of assuming GbE region is used and PDR region is not, check if there is a valid region defined in the descriptor and set the region access permissions based on that.
This enables the use of the PDR region on the sarien platform, which also uses the GbE region.
This results in the following example changes:
mb/google/sarien (GbE and PDR) . DESC BIOS ME GbE PDR EC -BIOS r rw rw r +BIOS r rw rw rw r
mb/google/eve: (no GbE, no PDR) . DESC BIOS ME GbE PDR EC -BIOS r rw rw +BIOS r rw
BUG=b:134703987
Change-Id: I7aeffc8f8194638c6012340b43aea8f8460d268a Signed-off-by: Duncan Laurie dlaurie@google.com --- M util/ifdtool/ifdtool.c M util/ifdtool/ifdtool.h 2 files changed, 40 insertions(+), 8 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/73/33273/2