[coreboot-gerrit] [S] Change in coreboot[master]: soc/intel/common/block/fast_spi: Add SPI BIOS decode lock

[David Hendricks (Code Review)]

Attention is currently required from: Simon Chou, TimLiu-SMCI, Jonathan Zhang, Ryback Hung, Paul Menzel, Angel Pons, Tim Chu.

David Hendricks has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/71953 )

Change subject: soc/intel/common/block/fast_spi: Add SPI BIOS decode lock
......................................................................

Patch Set 8: -Code-Review

(1 comment)

File src/soc/intel/common/block/fast_spi/fast_spi.c:

https://review.coreboot.org/c/coreboot/+/71953/comment/952d64f1_82131435 
PS6, Line 467: 	pci_write_config32(dev, SPI_BIOS_DECODE_EN, bde);
> Done

Sorry, looking back at this I think what Paul meant is:
`pci_or_config32(dev, SPI_BIOS_DECODE_EN, SPI_BIOS_DECODE_LOCK);`

Then we can get rid of the `bde` variable and reduce the size of this function by 3 lines.