build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/30007 )
Change subject: sb/amd/sr5650/cmn.h: Don't use device_t ......................................................................
Patch Set 1:
(6 comments)
https://review.coreboot.org/#/c/30007/1/src/southbridge/amd/sr5650/cmn.h File src/southbridge/amd/sr5650/cmn.h:
https://review.coreboot.org/#/c/30007/1/src/southbridge/amd/sr5650/cmn.h@185 PS1, Line 185: if (reg != reg_old) { braces {} are not necessary for single statement blocks
https://review.coreboot.org/#/c/30007/1/src/southbridge/amd/sr5650/cmn.h@217 PS1, Line 217: if (reg != reg_old) { braces {} are not necessary for single statement blocks
https://review.coreboot.org/#/c/30007/1/src/southbridge/amd/sr5650/cmn.h@229 PS1, Line 229: if (reg != reg_old) { braces {} are not necessary for single statement blocks
https://review.coreboot.org/#/c/30007/1/src/southbridge/amd/sr5650/cmn.h@241 PS1, Line 241: if (reg != reg_old) { braces {} are not necessary for single statement blocks
https://review.coreboot.org/#/c/30007/1/src/southbridge/amd/sr5650/cmn.h@253 PS1, Line 253: if (reg != reg_old) { braces {} are not necessary for single statement blocks
https://review.coreboot.org/#/c/30007/1/src/southbridge/amd/sr5650/cmn.h@265 PS1, Line 265: if (reg != reg_old) { braces {} are not necessary for single statement blocks