[coreboot-gerrit] Change in coreboot[master]: sb/intel/common: Replace inb/outb()

[Angel Pons (Code Review)]

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38150 )

Change subject: sb/intel/common: Replace inb/outb()
......................................................................

Patch Set 2: Code-Review+2

(1 comment)

https://review.coreboot.org/c/coreboot/+/38150/2//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/38150/2//COMMIT_MSG@9 
PS2, Line 9: At least since ICH10 SMBUS controller implements register
I'm not sure if I follow. I would rephrase this as:

sb/intel/common/smbus.c: Wrap inb/outb()

On Intel, accessing the SMBus register banks can be done via IO and, since at least ICH10, via MMIO. We may want to use the latter in the future.