[coreboot-gerrit] Change in coreboot[master]: mb/google/kukui: Extend FMAP to 8MB layout

[Julius Werner (Code Review)]

Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35612 )

Change subject: mb/google/kukui: Extend FMAP to 8MB layout
......................................................................

Patch Set 6: Code-Review+2

(3 comments)

https://review.coreboot.org/c/coreboot/+/35612/6/src/mainboard/google/kukui/... 
File src/mainboard/google/kukui/chromeos.fmd:

https://review.coreboot.org/c/coreboot/+/35612/6/src/mainboard/google/kukui/... 
PS6, Line 40: 	RW_SHARED 36K {
nit: why not just leave this unspecified...

https://review.coreboot.org/c/coreboot/+/35612/6/src/mainboard/google/kukui/... 
PS6, Line 42: 		RW_UNUSED
...don't have this...

https://review.coreboot.org/c/coreboot/+/35612/6/src/mainboard/google/kukui/... 
PS6, Line 44: 	RW_LEGACY(CBFS) 1M  # Minimal 1M.
...and leave this unspecified so it fills remaining space?