Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/68496 )
Change subject: mb/Kconfig: Add a prompt string for MAINBOARD_PART_NUMBER ......................................................................
mb/Kconfig: Add a prompt string for MAINBOARD_PART_NUMBER
For some nissa variants, there are build configurations that need to use different blobs, but otherwise are identical. Currently, they use the same choice of board config and configure blob paths appropriately. This avoids duplication within the Kconfig file, but the resulting firmware images can be difficult to distinguish, since they report the same FRID.
Add a prompt string for MAINBOARD_PART_NUMBER so it can be overridden and round-tripped through make oldconfig, allowing customisation of the reported FRID and other MAINBOARD_PART_NUMBER-derived values with minimal overhead.
BUG=b:253966060 BRANCH=None TEST=CL:3960290 MAINBOARD_PART_NUMBER configs apply successfully
Signed-off-by: Sam McNally sammc@chromium.org Change-Id: I3497d7fa1c04c8fa2592025c771d9dbc65632e6e Reviewed-on: https://review.coreboot.org/c/coreboot/+/68496 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Angel Pons th3fanbus@gmail.com Reviewed-by: Reka Norman rekanorman@chromium.org --- M src/mainboard/Kconfig 1 file changed, 31 insertions(+), 1 deletion(-)
Approvals: build bot (Jenkins): Verified Angel Pons: Looks good to me, but someone else must approve Reka Norman: Looks good to me, approved
diff --git a/src/mainboard/Kconfig b/src/mainboard/Kconfig index 8f9b48c..aec996a 100644 --- a/src/mainboard/Kconfig +++ b/src/mainboard/Kconfig @@ -14,7 +14,7 @@ string
config MAINBOARD_PART_NUMBER - string + string "Mainboard part number"
config MAINBOARD_VENDOR string "Mainboard vendor name"