build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/27409 )
Change subject: src/southbridge: Use "foo *bar" instead of "foo* bar" ......................................................................
Patch Set 3:
(8 comments)
https://review.coreboot.org/#/c/27409/3/src/southbridge/amd/cimx/sb800/Amd.h File src/southbridge/amd/cimx/sb800/Amd.h:
https://review.coreboot.org/#/c/27409/3/src/southbridge/amd/cimx/sb800/Amd.h... PS3, Line 64: typedef AGESA_STATUS (*CALLOUT_ENTRY) (unsigned int Param1, unsigned int Param2, space prohibited between function name and open parenthesis '('
https://review.coreboot.org/#/c/27409/3/src/southbridge/amd/cimx/sb800/Amd.h... PS3, Line 66: typedef AGESA_STATUS (*IMAGE_ENTRY) (IN OUT void *ConfigPtr); space prohibited between function name and open parenthesis '('
https://review.coreboot.org/#/c/27409/3/src/southbridge/amd/cimx/sb800/Amd.h... PS3, Line 67: typedef AGESA_STATUS (*MODULE_ENTRY) (IN OUT void *ConfigPtr); space prohibited between function name and open parenthesis '('
https://review.coreboot.org/#/c/27409/3/src/southbridge/amd/cimx/sb900/Amd.h File src/southbridge/amd/cimx/sb900/Amd.h:
https://review.coreboot.org/#/c/27409/3/src/southbridge/amd/cimx/sb900/Amd.h... PS3, Line 64: typedef AGESA_STATUS (*CALLOUT_ENTRY) (unsigned int Param1, unsigned int Param2, space prohibited between function name and open parenthesis '('
https://review.coreboot.org/#/c/27409/3/src/southbridge/amd/cimx/sb900/Amd.h... PS3, Line 66: typedef AGESA_STATUS (*IMAGE_ENTRY) (IN OUT void *ConfigPtr); space prohibited between function name and open parenthesis '('
https://review.coreboot.org/#/c/27409/3/src/southbridge/amd/cimx/sb900/Amd.h... PS3, Line 67: typedef AGESA_STATUS (*MODULE_ENTRY) (IN OUT void *ConfigPtr); space prohibited between function name and open parenthesis '('
https://review.coreboot.org/#/c/27409/3/src/southbridge/intel/i82870/ioapic.... File src/southbridge/intel/i82870/ioapic.c:
https://review.coreboot.org/#/c/27409/3/src/southbridge/intel/i82870/ioapic.... PS3, Line 47: volatile uint32_t *pIndexRegister; /* io apic io memory space command address */ line over 80 characters
https://review.coreboot.org/#/c/27409/3/src/southbridge/intel/i82870/ioapic.... PS3, Line 48: volatile uint32_t *pWindowRegister; /* io apic io memory space data address */ line over 80 characters
-
build bot (Jenkins) (Code Review)