[coreboot-gerrit] Change in coreboot[master]: [WIP] soc/amd/common: Avoid MMIO aliasing on SMBus

[Aaron Durbin (Code Review)]

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

Change subject: [WIP] soc/amd/common: Avoid MMIO aliasing on SMBus
......................................................................

Patch Set 2:

(2 comments)

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

https://review.coreboot.org/c/coreboot/+/42074/2/src/soc/amd/common/block/sm... 
PS2, Line 20: u32
make this a uintptr_t from the get-go?

https://review.coreboot.org/c/coreboot/+/42074/2/src/soc/amd/common/block/sm... 
PS2, Line 23: 	return controller->reg8[reg];
What's the reasoning for not using read8()? pointer aliasing rules?