build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31913 )
Change subject: Fix 'unsigned int' to bare use of 'unsigned' ......................................................................
Patch Set 2:
(6 comments)
https://review.coreboot.org/#/c/31913/2/src/northbridge/amd/amdk8/reset_test... File src/northbridge/amd/amdk8/reset_test.c:
https://review.coreboot.org/#/c/31913/2/src/northbridge/amd/amdk8/reset_test... PS2, Line 64: static unsigned node_link_to_bus(unsigned int node, unsigned int link) Prefer 'unsigned int' to bare use of 'unsigned'
https://review.coreboot.org/#/c/31913/2/src/northbridge/amd/pi/00630F01/nort... File src/northbridge/amd/pi/00630F01/northbridge.c:
https://review.coreboot.org/#/c/31913/2/src/northbridge/amd/pi/00630F01/nort... PS2, Line 171: static int reg_useable(unsigned reg, struct device *goal_dev, unsigned int goal_nodeid, line over 80 characters
https://review.coreboot.org/#/c/31913/2/src/northbridge/amd/pi/00630F01/nort... PS2, Line 171: static int reg_useable(unsigned reg, struct device *goal_dev, unsigned int goal_nodeid, Prefer 'unsigned int' to bare use of 'unsigned'
https://review.coreboot.org/#/c/31913/2/src/northbridge/amd/pi/00630F01/nort... PS2, Line 199: static struct resource *amdfam15_find_iopair(struct device *dev, unsigned int nodeid, unsigned int link) line over 80 characters
https://review.coreboot.org/#/c/31913/2/src/northbridge/amd/pi/00730F01/nort... File src/northbridge/amd/pi/00730F01/northbridge.c:
https://review.coreboot.org/#/c/31913/2/src/northbridge/amd/pi/00730F01/nort... PS2, Line 172: static int reg_useable(unsigned int reg, struct device *goal_dev, unsigned int goal_nodeid, line over 80 characters
https://review.coreboot.org/#/c/31913/2/src/northbridge/amd/pi/00730F01/nort... PS2, Line 200: static struct resource *amdfam16_find_iopair(struct device *dev, unsigned int nodeid, unsigned int link) line over 80 characters