Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33324 )
Change subject: arch/x86/Kconfig: Hide the prefix option on all but BOOTBLOCK_NORMAL ......................................................................
Patch Set 3:
Patch Set 3:
Doesn't this mean that the stages are all going to be called "/ramstage", "/payload", etc. (i.e. with leading slash)? I don't think we want that? If you want to do this, I think you need to first make the slash part of the prefix.
Also, while I do agree that the prefix is ugly (especially on non-x86 boards which can't even do the switching), I'd also caution that this is a biiig change in how things work that's going to effect a lot of people's muscle memory and scripts. There are plenty of hardcoded "fallback/..." in the Chrome OS build system and I bet in other setups as well. If you really want to do this, you should probably at least announce it on the mailing list first and give everyone time to prepare.
While getting rid of the prefix altogether should be done in the long run, that is not what this patch does.
All it does is removing/hiding the ability to configure the prefix when you're not using the bootblock_normal.c romcc bootblock, which is the only codepath that currently makes use of the prefix (switch between normal/fallback depending on rtc nvram bits).