build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38004 )
Change subject: src/{soc,southbridge}/amd: Fix typo ......................................................................
Patch Set 1:
(4 comments)
https://review.coreboot.org/c/coreboot/+/38004/1/src/southbridge/amd/agesa/h... File src/southbridge/amd/agesa/hudson/smbus.c:
https://review.coreboot.org/c/coreboot/+/38004/1/src/southbridge/amd/agesa/h... PS1, Line 192: outl((reg_space & 0x7) << 29 | reg_addr, AB_INDX); /* probably we don't have to do it again. */ line over 96 characters
https://review.coreboot.org/c/coreboot/+/38004/1/src/southbridge/amd/agesa/h... PS1, Line 214: outl((reg_space & 0x7) << 29 | (port & 3) << 24 | reg_addr, AB_INDX); /* probably we don't have to do it again. */ line over 96 characters
https://review.coreboot.org/c/coreboot/+/38004/1/src/southbridge/amd/pi/huds... File src/southbridge/amd/pi/hudson/smbus.c:
https://review.coreboot.org/c/coreboot/+/38004/1/src/southbridge/amd/pi/huds... PS1, Line 192: outl((reg_space & 0x7) << 29 | reg_addr, AB_INDX); /* probably we don't have to do it again. */ line over 96 characters
https://review.coreboot.org/c/coreboot/+/38004/1/src/southbridge/amd/pi/huds... PS1, Line 214: outl((reg_space & 0x7) << 29 | (port & 3) << 24 | reg_addr, AB_INDX); /* probably we don't have to do it again. */ line over 96 characters