[coreboot-gerrit] Change in coreboot[master]: [WIP] soc/amd/common: Place guards for ACPIMMIO on ENV_ARM

[Aaron Durbin (Code Review)]

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

Change subject: [WIP] soc/amd/common: Place guards for ACPIMMIO on ENV_ARM
......................................................................

Patch Set 7:

(1 comment)

https://review.coreboot.org/c/coreboot/+/42523/7/src/soc/amd/picasso/i2c.c 
File src/soc/amd/picasso/i2c.c:

https://review.coreboot.org/c/coreboot/+/42523/7/src/soc/amd/picasso/i2c.c@5... 
PS7, Line 55: 	switch (dev->path.mmio.addr) {
> If we continue with the ideological view that no x86 memory space symbols should appear in ARCH_VERS […]

You could just use the i2c_bus_address[x] values? I don't think those case statements will hold up, but that approach should work.