build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/30116 )
Change subject: [WIP]pci_drivers/cpu_drivers: Fix constructed arrays on 64bit platforms ......................................................................
Patch Set 3:
(14 comments)
https://review.coreboot.org/#/c/30116/3/src/arch/arm/include/armv7/arch/cpu.... File src/arch/arm/include/armv7/arch/cpu.h:
https://review.coreboot.org/#/c/30116/3/src/arch/arm/include/armv7/arch/cpu.... PS3, Line 29: } __attribute__ ((aligned (CONFIG_DRIVER_ALIGN))); space prohibited between function name and open parenthesis '('
https://review.coreboot.org/#/c/30116/3/src/arch/arm/include/armv7/arch/cpu.... PS3, Line 30: _Static_assert(sizeof(struct cpu_driver) <= CONFIG_DRIVER_ALIGN, \ Avoid unnecessary line continuations
https://review.coreboot.org/#/c/30116/3/src/arch/arm64/include/armv8/arch/cp... File src/arch/arm64/include/armv8/arch/cpu.h:
https://review.coreboot.org/#/c/30116/3/src/arch/arm64/include/armv8/arch/cp... PS3, Line 26: struct cpu_driver { } __attribute__ ((aligned (CONFIG_DRIVER_ALIGN))); space prohibited between function name and open parenthesis '('
https://review.coreboot.org/#/c/30116/3/src/arch/arm64/include/armv8/arch/cp... PS3, Line 27: _Static_assert(sizeof(struct cpu_driver) <= CONFIG_DRIVER_ALIGN, \ Avoid unnecessary line continuations
https://review.coreboot.org/#/c/30116/3/src/arch/mips/include/arch/cpu.h File src/arch/mips/include/arch/cpu.h:
https://review.coreboot.org/#/c/30116/3/src/arch/mips/include/arch/cpu.h@28 PS3, Line 28: } __attribute__ ((aligned (CONFIG_DRIVER_ALIGN))); space prohibited between function name and open parenthesis '('
https://review.coreboot.org/#/c/30116/3/src/arch/mips/include/arch/cpu.h@29 PS3, Line 29: _Static_assert(sizeof(struct cpu_driver) <= CONFIG_DRIVER_ALIGN, \ Avoid unnecessary line continuations
https://review.coreboot.org/#/c/30116/3/src/arch/ppc64/include/arch/cpu.h File src/arch/ppc64/include/arch/cpu.h:
https://review.coreboot.org/#/c/30116/3/src/arch/ppc64/include/arch/cpu.h@27 PS3, Line 27: } __attribute__ ((aligned (CONFIG_DRIVER_ALIGN))); space prohibited between function name and open parenthesis '('
https://review.coreboot.org/#/c/30116/3/src/arch/ppc64/include/arch/cpu.h@28 PS3, Line 28: _Static_assert(sizeof(struct cpu_driver) <= CONFIG_DRIVER_ALIGN, \ Avoid unnecessary line continuations
https://review.coreboot.org/#/c/30116/3/src/arch/riscv/include/arch/cpu.h File src/arch/riscv/include/arch/cpu.h:
https://review.coreboot.org/#/c/30116/3/src/arch/riscv/include/arch/cpu.h@29 PS3, Line 29: } __attribute__ ((aligned (CONFIG_DRIVER_ALIGN))); space prohibited between function name and open parenthesis '('
https://review.coreboot.org/#/c/30116/3/src/arch/riscv/include/arch/cpu.h@30 PS3, Line 30: _Static_assert(sizeof(struct cpu_driver) <= CONFIG_DRIVER_ALIGN, \ Avoid unnecessary line continuations
https://review.coreboot.org/#/c/30116/3/src/arch/x86/include/arch/cpu.h File src/arch/x86/include/arch/cpu.h:
https://review.coreboot.org/#/c/30116/3/src/arch/x86/include/arch/cpu.h@183 PS3, Line 183: } __attribute__ ((aligned (CONFIG_DRIVER_ALIGN))); space prohibited between function name and open parenthesis '('
https://review.coreboot.org/#/c/30116/3/src/arch/x86/include/arch/cpu.h@184 PS3, Line 184: _Static_assert(sizeof(struct cpu_driver) <= CONFIG_DRIVER_ALIGN, \ Avoid unnecessary line continuations
https://review.coreboot.org/#/c/30116/3/src/include/device/pci.h File src/include/device/pci.h:
https://review.coreboot.org/#/c/30116/3/src/include/device/pci.h@57 PS3, Line 57: } __attribute__ ((aligned (CONFIG_DRIVER_ALIGN))); space prohibited between function name and open parenthesis '('
https://review.coreboot.org/#/c/30116/3/src/include/device/pci.h@58 PS3, Line 58: _Static_assert(sizeof(struct pci_driver) <= CONFIG_DRIVER_ALIGN, \ Avoid unnecessary line continuations