build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36331 )
Change subject: src/[northbridge,security]: change "unsigned" to "unsigned int" ......................................................................
Patch Set 2:
(11 comments)
https://review.coreboot.org/c/coreboot/+/36331/2/src/northbridge/amd/agesa/f... File src/northbridge/amd/agesa/family15tn/northbridge.c:
https://review.coreboot.org/c/coreboot/+/36331/2/src/northbridge/amd/agesa/f... PS2, Line 648: mem_hole.hole_startk = (unsigned int)limitk_pri; // must beblow 4G line over 96 characters
https://review.coreboot.org/c/coreboot/+/36331/2/src/northbridge/amd/agesa/f... File src/northbridge/amd/agesa/family16kb/northbridge.c:
https://review.coreboot.org/c/coreboot/+/36331/2/src/northbridge/amd/agesa/f... PS2, Line 664: mem_hole.hole_startk = (unsigned int)limitk_pri; // must beblow 4G line over 96 characters
https://review.coreboot.org/c/coreboot/+/36331/2/src/northbridge/amd/amdfam1... File src/northbridge/amd/amdfam10/northbridge.c:
https://review.coreboot.org/c/coreboot/+/36331/2/src/northbridge/amd/amdfam1... PS2, Line 392: static struct resource *amdfam10_find_iopair(struct device *dev, unsigned int nodeid, unsigned int link) line over 96 characters
https://review.coreboot.org/c/coreboot/+/36331/2/src/northbridge/amd/amdfam1... PS2, Line 860: mem_hole.hole_startk = (unsigned int)limitk_pri; // must beblow 4G line over 96 characters
https://review.coreboot.org/c/coreboot/+/36331/2/src/northbridge/amd/amdmct/... File src/northbridge/amd/amdmct/mct/mct.h:
https://review.coreboot.org/c/coreboot/+/36331/2/src/northbridge/amd/amdmct/... PS2, Line 511: void Get_ChannelPS_Cfg0(unsigned int MAAdimms, unsigned int Speed, unsigned int MAAload, unsigned int DATAAload, line over 96 characters
https://review.coreboot.org/c/coreboot/+/36331/2/src/northbridge/amd/amdmct/... PS2, Line 535: unsigned int K8FCalcFinalDQSRcvValue(struct MCTStatStruc *pMCTstat, struct DCTStatStruc *pDCTstat, unsigned int LeftRcvEn, unsigned int RightRcvEn, unsigned int *valid); line over 96 characters
https://review.coreboot.org/c/coreboot/+/36331/2/src/northbridge/amd/pi/0063... File src/northbridge/amd/pi/00630F01/northbridge.c:
https://review.coreboot.org/c/coreboot/+/36331/2/src/northbridge/amd/pi/0063... PS2, Line 661: mem_hole.hole_startk = (unsigned int)limitk_pri; // must be below 4G line over 96 characters
https://review.coreboot.org/c/coreboot/+/36331/2/src/northbridge/amd/pi/0066... File src/northbridge/amd/pi/00660F01/northbridge.c:
https://review.coreboot.org/c/coreboot/+/36331/2/src/northbridge/amd/pi/0066... PS2, Line 199: static struct resource *amdfam15_find_iopair(struct device *dev, unsigned int nodeid, unsigned int link) line over 96 characters
https://review.coreboot.org/c/coreboot/+/36331/2/src/northbridge/amd/pi/0066... PS2, Line 661: mem_hole.hole_startk = (unsigned int)limitk_pri; // must beblow 4G line over 96 characters
https://review.coreboot.org/c/coreboot/+/36331/2/src/northbridge/amd/pi/0073... File src/northbridge/amd/pi/00730F01/northbridge.c:
https://review.coreboot.org/c/coreboot/+/36331/2/src/northbridge/amd/pi/0073... PS2, Line 885: mem_hole.hole_startk = (unsigned int)limitk_pri; // must beblow 4G line over 96 characters
https://review.coreboot.org/c/coreboot/+/36331/2/src/northbridge/intel/e7505... File src/northbridge/intel/e7505/raminit.c:
https://review.coreboot.org/c/coreboot/+/36331/2/src/northbridge/intel/e7505... PS2, Line 760: static uint8_t configure_dimm_row_boundaries(struct dimm_size dimm_log2_num_bits, uint8_t total_dram_64M_multiple, unsigned int dimm_index) line over 96 characters