[coreboot-gerrit] [S] Change in coreboot[master]: arch/x86/Kconfig: Move AMD stages arch to common code

[Arthur Heymans (Code Review)]

Arthur Heymans has posted comments on this change by Arthur Heymans. ( https://review.coreboot.org/c/coreboot/+/69505?usp=email )

Change subject: arch/x86/Kconfig: Move AMD stages arch to common code
......................................................................

Patch Set 2:

(1 comment)

Patchset:

PS2:
> Okay. Could explain the reason? From the commit message, this patch looks like refactoring-only patch.
>
> Also, the `ARCH_ALL_STAGES_X86_32` name is misleading, because on AMD not all stages are x86_32 even if `ARCH_ALL_STAGES_X86_32=y`.

I suppose I didn't explain this in the commit message. I was working on getting both 64bit working on this platform as well as getting clang working across all platfoms. So unless there are problems with building ARM verstage with clang, I'd like to keep that option.

The name is only somewhat misleading as all stages can run in principle on x86 except that if you enable VBOOT verstage can be an ARM stage on chromeos platforms.