[coreboot-gerrit] Change in coreboot[master]: soc/amd/stoneyridge: move to using smbus_host.h definitions

[Kyösti Mälkki (Code Review)]

Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38222 )

Change subject: soc/amd/stoneyridge: move to using smbus_host.h definitions
......................................................................

Patch Set 8:

(1 comment)

https://review.coreboot.org/c/coreboot/+/38222/4/src/soc/amd/stoneyridge/smb... 
File src/soc/amd/stoneyridge/smbus.c:

https://review.coreboot.org/c/coreboot/+/38222/4/src/soc/amd/stoneyridge/smb... 
PS4, Line 28: 		return asf_read8(reg);
> Inspecting the assembly it looked like the compiler is not able to do a good optimisation of this ev […]

The SMBus slave side code should not be required to know the base address, slave-side sources are not allowed to include platform-specific host-side headerfiles.

To be addressed at some later date when needs to support SMBus multiplexers is evaluated.