[coreboot-gerrit] Change in ...coreboot[master]: payload/seabios: Fix CBFS location on APL platform

[Patrick Rudolph (Code Review)]

Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32327 )

Change subject: payload/seabios: Fix CBFS location on APL platform
......................................................................

Patch Set 3:

(1 comment)

https://review.coreboot.org/#/c/32327/3/payloads/external/SeaBIOS/Makefile 
File payloads/external/SeaBIOS/Makefile:

https://review.coreboot.org/#/c/32327/3/payloads/external/SeaBIOS/Makefile@4... 
PS3, Line 43: 	echo "CONFIG_CBFS_LOCATION=0xfffc0000" >> seabios/.config
> Sorry, I should think before submitting. Disregard the above. […]

as only a part of the boot media is memory mapped on x86 (the bios region), and we don't know the size nor the offset, the boot media params are useless to find the CBFS in that memory mapped region.
Only a software that is able to access the whole boot media contents can make use if it and seabios isn't.