Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41248 )
Change subject: soc/amd/common/block/spi: Add support for common SPI configuration ......................................................................
Patch Set 3:
(2 comments)
https://review.coreboot.org/c/coreboot/+/41248/1/src/soc/amd/common/block/in... File src/soc/amd/common/block/include/amdblocks/chip.h:
https://review.coreboot.org/c/coreboot/+/41248/1/src/soc/amd/common/block/in... PS1, Line 12: * Default values if not overriden by mainboard:
'overriden' may be misspelled - perhaps 'overridden'?
Done
https://review.coreboot.org/c/coreboot/+/41248/1/src/soc/amd/common/block/in... File src/soc/amd/common/block/include/amdblocks/spi.h:
https://review.coreboot.org/c/coreboot/+/41248/1/src/soc/amd/common/block/in... PS1, Line 63: * Default values if not overriden by mainboard:
'overriden' may be misspelled - perhaps 'overridden'?
Done