[coreboot-gerrit] Change in ...coreboot[master]: riscv: Add initial support for 32bit boards

[Philipp Hug (Code Review)]

Philipp Hug has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31253 )

Change subject: riscv: Add initial support for 32bit boards
......................................................................

Patch Set 3:

(2 comments)

https://review.coreboot.org/#/c/31253/3/src/lib/libgcc.c 
File src/lib/libgcc.c:

https://review.coreboot.org/#/c/31253/3/src/lib/libgcc.c@24 
PS3, Line 24: #if !IS_ENABLED(CONFIG_ARCH_X86) && !IS_ENABLED(CONFIG_ARCH_RISCV_RV32)
Analyze why this is needed on RV32

https://review.coreboot.org/#/c/31253/3/src/soc/ucb/riscv/Kconfig 
File src/soc/ucb/riscv/Kconfig:

https://review.coreboot.org/#/c/31253/3/src/soc/ucb/riscv/Kconfig@37 
PS3, Line 37: 	default "rv32im"
is this correct?