[coreboot-gerrit] Change in coreboot[master]: soc/amd/common: Add func to clear eSPI IO & memory decode ranges

[Furquan Shaikh (Code Review)]

Attention is currently required from: Jason Glenesk, Raul Rangel, Martin Roth, Marshall Dawson, Felix Held.
Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/51748 )

Change subject: soc/amd/common: Add func to clear eSPI IO & memory decode ranges
......................................................................

Patch Set 3:

(1 comment)

File src/soc/amd/common/block/lpc/espi_util.c:

https://review.coreboot.org/c/coreboot/+/51748/comment/ca16ad84_72ed3a91 
PS2, Line 108: for (idx = 0; idx < ESPI_GENERIC_IO_WIN_COUNT; idx++) {
             : 		espi_write16(ESPI_IO_RANGE_BASE(idx), 0);
             : 		espi_write8(ESPI_IO_RANGE_SIZE(idx), 0);
             : 	}
             : 	for (idx = 0; idx < ESPI_GENERIC_MMIO_WIN_COUNT; idx++) {
             : 		espi_write32(ESPI_MMIO_RANGE_BASE(idx), 0);
             : 		espi_write16(ESPI_MMIO_RANGE_SIZE(idx), 0);
             : 	}
> I think `espi_write32(ESPI_DECODE, 0);` right before configure_decodes should be sufficient. […]

Agree with that. espi_write32(ESPI_DECODE, 0); should be sufficient here.