[coreboot-gerrit] Change in ...coreboot[master]: sb/intel/common/firmware: Don't call GbE binary `firmware`

[Subrata Banik (Code Review)]

Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31108 )

Change subject: sb/intel/common/firmware: Don't call GbE binary `firmware`
......................................................................

Patch Set 1:

(1 comment)

https://review.coreboot.org/#/c/31108/1//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/#/c/31108/1//COMMIT_MSG@8 
PS1, Line 8:
> The GbE binary holds the MAC address the integrated Ethernet uses. […]

can you please point me to the document where you get those details about referring as Gbe "configuration"

This is what I could see at Intel document, not sure if you have a better version ?

SPI programming guide tells me the purpose of this Gbe region that you have modified in this CL:

GbE – Location for Integrated LAN firmware and MAC address