[coreboot-gerrit] Change in coreboot[master]: soc/amd/common: Refactor SMBus base arguments

[Felix Held (Code Review)]

Felix Held has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42074 )

Change subject: soc/amd/common: Refactor SMBus base arguments
......................................................................

Patch Set 18:

(1 comment)

https://review.coreboot.org/c/coreboot/+/42074/17/src/soc/amd/common/block/s... 
File src/soc/amd/common/block/smbus/sm.c:

https://review.coreboot.org/c/coreboot/+/42074/17/src/soc/amd/common/block/s... 
PS17, Line 28: 	res = find_resource(pbus->dev, 0x90);
> I saw 0:14.0 PCI config register 0x90 defined in SB600 and SB700, dropped from SB800 documentation.

just had a look and that register is neither defined in the stoneyridge bkdg not in the picasso ppr. as far as i've looked this function seems to be rather wrong to me