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

[Aaron Durbin (Code Review)]

Aaron Durbin 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: Code-Review+2

(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
> Well, I thought I did, but just changed it in this patch version because Kyosti said he preferred it […]

fwiw, I think Kyosti's comment was about using char * because of pointer aliasing rules using void *.