[coreboot-gerrit] Change in coreboot[master]: soc/intel/braswell: Use common sb code for SPI lockdown configuration

[Arthur Heymans (Code Review)]

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

Change subject: soc/intel/braswell: Use common sb code for SPI lockdown configuration
......................................................................

Patch Set 3:

(1 comment)

https://review.coreboot.org/c/coreboot/+/36007/3/src/mainboard/facebook/fbg1... 
File src/mainboard/facebook/fbg1701/w25q64.c:

https://review.coreboot.org/c/coreboot/+/36007/3/src/mainboard/facebook/fbg1... 
PS3, Line 66: 	.preop = CMD_W25_WREN,
> The pre-op for this platform is WREN only! 
> Why changing this?

I didn't notice it, since all those files are called w25q64.c and made assumptions on the flash used. I could use the callback function to override the defaults. Why is it needed?