[coreboot-gerrit] Change in coreboot[master]: fsp_broadwell_de: Enable early write access to the SPI flash

[Arthur Heymans (Code Review)]

Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35598 )

Change subject: fsp_broadwell_de: Enable early write access to the SPI flash
......................................................................

Patch Set 1: Code-Review+2

(1 comment)

https://review.coreboot.org/c/coreboot/+/35598/1/src/soc/intel/fsp_broadwell... 
File src/soc/intel/fsp_broadwell_de/Kconfig:

https://review.coreboot.org/c/coreboot/+/35598/1/src/soc/intel/fsp_broadwell... 
PS1, Line 32: 	select BOOT_DEVICE_SUPPORTS_WRITES
> shouldn't this depend on VBOOT?

It's the opposite. VBOOT_VBNV_FLASH depends on BOOT_DEVICE_SUPPORTS_WRITES.