build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37400 )
Change subject: sb/amd/{agesa,pi,cimx}: use ACPIMMIO common block wherever possible ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/c/coreboot/+/37400/1/src/southbridge/amd/pi/huds... File src/southbridge/amd/pi/hudson/early_setup.c:
https://review.coreboot.org/c/coreboot/+/37400/1/src/southbridge/amd/pi/huds... PS1, Line 248: data |= biosram_read8(nvram_pos) << (i *8); need consistent spacing around '*' (ctx:WxV)