build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31813 )
Change subject: Fix up remaining boolean uses of CONFIG_XXX to CONFIG(XXX) ......................................................................
Patch Set 1:
(4 comments)
https://review.coreboot.org/#/c/31813/1/src/northbridge/intel/i945/gma.c File src/northbridge/intel/i945/gma.c:
https://review.coreboot.org/#/c/31813/1/src/northbridge/intel/i945/gma.c@680 PS1, Line 680: if (CONFIG(NORTHBRIDGE_INTEL_SUBTYPE_I945GM)) { braces {} are not necessary for single statement blocks
https://review.coreboot.org/#/c/31813/1/src/northbridge/intel/i945/gma.c@683 PS1, Line 683: if (CONFIG(NORTHBRIDGE_INTEL_SUBTYPE_I945GC)) { braces {} are not necessary for single statement blocks
https://review.coreboot.org/#/c/31813/1/src/northbridge/intel/sandybridge/ro... File src/northbridge/intel/sandybridge/romstage.c:
https://review.coreboot.org/#/c/31813/1/src/northbridge/intel/sandybridge/ro... PS1, Line 72: if (CONFIG(USE_NATIVE_RAMINIT)) { braces {} are not necessary for single statement blocks
https://review.coreboot.org/#/c/31813/1/src/southbridge/amd/cimx/sb900/early... File src/southbridge/amd/cimx/sb900/early.c:
https://review.coreboot.org/#/c/31813/1/src/southbridge/amd/cimx/sb900/early... PS1, Line 44: if (!CONFIG(PCIB_ENABLE)) { braces {} are not necessary for single statement blocks