[coreboot-gerrit] Change in ...coreboot[master]: Kconfig: Make ramstage kconfigs dependent on CONFIG_ARCH_RAMSTAGE_X86...

[Aaron Durbin (Code Review)]

Aaron Durbin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33112 )

Change subject: Kconfig: Make ramstage kconfigs dependent on CONFIG_ARCH_RAMSTAGE_X86_32/64
......................................................................

Patch Set 1:

(1 comment)

https://review.coreboot.org/#/c/33112/1//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/#/c/33112/1//COMMIT_MSG@11 
PS1, Line 11: is not selected.
> In case we would like to deselect CONFIG_ARCH_RAMSTAGE_X86_32 config then ramstage won't be there bu […]

reasoning is not in the commit message. I don't see such a binding current with RELOCATABLE_RAMSTAGE and x86_32. I'm very confused why there is an association being made as one doesn't exist now. If you want to hide such options based on the presence of ramstage there should be a proper kconfig for that. It should not be targeting a specific arch's Kconfig value.