Attention is currently required from: Arthur Heymans, Paul Menzel.
Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/69747?usp=email )
Change subject: arch/arm: Build test all arm targets with clang ......................................................................
Patch Set 20:
(4 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/69747/comment/02afd7cb_0987ab34 : PS20, Line 7: arch/arm: Build test all arm targets with clang This says "build test" but it looks like this CL is more about adding the ARCH_SUPPORTS_CLANG Kconfig?
File src/Kconfig:
https://review.coreboot.org/c/coreboot/+/69747/comment/297de493_74513037 : PS20, Line 1544: default n Should this be set for x86 somewhere as well? I thought that was already supporting clang?
File src/soc/nvidia/tegra124/Kconfig:
https://review.coreboot.org/c/coreboot/+/69747/comment/6f6ba39c_991636aa : PS20, Line 20: default n if SOC_NVIDIA_TEGRA124 && CHROMEOS Why not move these into the `if SOC_...` blocks below?
File src/soc/qualcomm/ipq806x/Kconfig:
https://review.coreboot.org/c/coreboot/+/69747/comment/842ec24a_7c4de807 : PS20, Line 16: SOC_NVIDIA_TEGRA124 Wrong SoC