build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33042 )
Change subject: sb/intel/common: Add a function to cleanly lock down the SPI config ......................................................................
Patch Set 1:
(3 comments)
https://review.coreboot.org/#/c/33042/1/src/southbridge/intel/common/spi.c File src/southbridge/intel/common/spi.c:
https://review.coreboot.org/#/c/33042/1/src/southbridge/intel/common/spi.c@1... PS1, Line 1103: writew_(readw_(&ich7_spi->spis) | HSFS_FLOCKDN, &ich7_spi->spis); line over 80 characters
https://review.coreboot.org/#/c/33042/1/src/southbridge/intel/common/spi.c@1... PS1, Line 1106: writew_(readw_(&ich9_spi->hsfs) | HSFS_FLOCKDN, &ich9_spi->hsfs) ; line over 80 characters
https://review.coreboot.org/#/c/33042/1/src/southbridge/intel/common/spi.c@1... PS1, Line 1106: writew_(readw_(&ich9_spi->hsfs) | HSFS_FLOCKDN, &ich9_spi->hsfs) ; space prohibited before semicolon