Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34141 )
Change subject: riscv: Remove unused headers ......................................................................
Patch Set 2: Code-Review+2
(3 comments)
Don't want to bikeshed this further.
https://review.coreboot.org/c/coreboot/+/34141/1/src/arch/riscv/stages.c File src/arch/riscv/stages.c:
https://review.coreboot.org/c/coreboot/+/34141/1/src/arch/riscv/stages.c@27 PS1, Line 27: #include <arch/encoding.h> : #include <arch/stages.h> : #include <arch/smp/smp.h> : #include <rules.h> : #include <mcall.h>
Done
Ack
https://review.coreboot.org/c/coreboot/+/34141/2/src/arch/riscv/stages.c File src/arch/riscv/stages.c:
https://review.coreboot.org/c/coreboot/+/34141/2/src/arch/riscv/stages.c@29 PS2, Line 29: #include <rules.h> Seems unused.
https://review.coreboot.org/c/coreboot/+/34141/2/src/arch/riscv/stages.c@43 PS2, Line 43: main(); Indirectly included through <arch/stages.h>, I don't know why.