build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/27923 )
Change subject: cpu/amd: Correct number of MCA banks cleared ......................................................................
Patch Set 1:
(5 comments)
https://review.coreboot.org/#/c/27923/1/src/cpu/amd/agesa/family12/model_12_... File src/cpu/amd/agesa/family12/model_12_init.c:
https://review.coreboot.org/#/c/27923/1/src/cpu/amd/agesa/family12/model_12_... PS1, Line 62: for (i = 0; i < num_banks; i++) { braces {} are not necessary for single statement blocks
https://review.coreboot.org/#/c/27923/1/src/cpu/amd/agesa/family14/model_14_... File src/cpu/amd/agesa/family14/model_14_init.c:
https://review.coreboot.org/#/c/27923/1/src/cpu/amd/agesa/family14/model_14_... PS1, Line 85: for (i = 0; i < num_banks; i++) { braces {} are not necessary for single statement blocks
https://review.coreboot.org/#/c/27923/1/src/cpu/amd/agesa/family15tn/model_1... File src/cpu/amd/agesa/family15tn/model_15_init.c:
https://review.coreboot.org/#/c/27923/1/src/cpu/amd/agesa/family15tn/model_1... PS1, Line 80: for (i = 0; i < num_banks; i++) { braces {} are not necessary for single statement blocks
https://review.coreboot.org/#/c/27923/1/src/cpu/amd/agesa/family16kb/model_1... File src/cpu/amd/agesa/family16kb/model_16_init.c:
https://review.coreboot.org/#/c/27923/1/src/cpu/amd/agesa/family16kb/model_1... PS1, Line 78: for (i = 0; i < num_banks; i++) { braces {} are not necessary for single statement blocks
https://review.coreboot.org/#/c/27923/1/src/cpu/amd/family_10h-family_15h/mo... File src/cpu/amd/family_10h-family_15h/model_10xxx_init.c:
https://review.coreboot.org/#/c/27923/1/src/cpu/amd/family_10h-family_15h/mo... PS1, Line 119: for (i = 0; i < num_banks; i++) { braces {} are not necessary for single statement blocks
-
build bot (Jenkins) (Code Review)