build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31356 )
Change subject: WIP: libpayload: add initial support for RISC-V ......................................................................
Patch Set 2:
(8 comments)
https://review.coreboot.org/#/c/31356/2/payloads/libpayload/include/riscv/ar... File payloads/libpayload/include/riscv/arch/asm.h:
https://review.coreboot.org/#/c/31356/2/payloads/libpayload/include/riscv/ar... PS2, Line 14: #define ALIGN .align 2 Macros with complex values should be enclosed in parentheses
https://review.coreboot.org/#/c/31356/2/payloads/libpayload/include/riscv/ar... PS2, Line 16: #define ENDPROC(name) \ Macros with multiple statements should be enclosed in a do - while loop
https://review.coreboot.org/#/c/31356/2/payloads/libpayload/include/riscv/ar... PS2, Line 17: .type name, %function; \ need consistent spacing around '%' (ctx:WxV)
https://review.coreboot.org/#/c/31356/2/payloads/libpayload/include/riscv/ar... PS2, Line 20: #define ENTRY(name) \ Macros with multiple statements should be enclosed in a do - while loop
https://review.coreboot.org/#/c/31356/2/payloads/libpayload/include/riscv/ar... PS2, Line 21: .section .text.name, "ax", %progbits; \ need consistent spacing around '%' (ctx:WxV)
https://review.coreboot.org/#/c/31356/2/payloads/libpayload/include/riscv/ar... PS2, Line 24: name: labels should not be indented
https://review.coreboot.org/#/c/31356/2/payloads/libpayload/include/riscv/ar... PS2, Line 26: #define END(name) \ Macros with complex values should be enclosed in parentheses
https://review.coreboot.org/#/c/31356/2/payloads/libpayload/include/riscv/ar... PS2, Line 27: .size name, .-name space required before that '-' (ctx:VxV)