[coreboot-gerrit] Change in coreboot[master]: soc/amd/picasso: Allow mainboard to configure SPI settings

[Raul Rangel (Code Review)]

Raul Rangel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40421 )

Change subject: soc/amd/picasso: Allow mainboard to configure SPI settings
......................................................................

Patch Set 8:

(1 comment)

https://review.coreboot.org/c/coreboot/+/40421/8/src/soc/amd/picasso/chip.h 
File src/soc/amd/picasso/chip.h:

https://review.coreboot.org/c/coreboot/+/40421/8/src/soc/amd/picasso/chip.h@... 
PS8, Line 84: spi_read_mode
Instead of placing all these in the main picasso config, would it be cleaner if we defined a SPI chip in the device tree?