[coreboot-gerrit] Change in coreboot[master]: soc/amd/common: Refactor and consolidate code for spi base

[Martin Roth (Code Review)]

Martin Roth has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43307 )

Change subject: soc/amd/common: Refactor and consolidate code for spi base
......................................................................

Patch Set 6:

(1 comment)

https://review.coreboot.org/c/coreboot/+/43307/6/src/soc/amd/common/block/sp... 
File src/soc/amd/common/block/spi/fch_spi_ctrl.c:

https://review.coreboot.org/c/coreboot/+/43307/6/src/soc/amd/common/block/sp... 
PS6, Line 35: spi_base
> Don't you want spi_get_bar()?

Well, I thought I did, but just changed it in this patch version because Kyosti said he preferred it this way.  It seems as safe as it was before this change where we read the spibar variable out of the LPC BAR.