Felix Singer submitted this change.

View Change

Approvals: Jan Samek: Looks good to me, approved build bot (Jenkins): Verified Marvin Drees: Looks good to me, but someone else must approve
util/ifdtool.c: Fix long_options for platform

Platform has argument, but has_arg was mistakenly set to 0.

Change-Id: I7d5c31c2b1da544cb73d9e213d463332fcdba7df
Signed-off-by: Vojtech Vesely <vojtech.vesely@9elements.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/80432
Reviewed-by: Jan Samek <jan.samek@siemens.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Marvin Drees <marvin.drees@9elements.com>
---
M util/ifdtool/ifdtool.c
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/util/ifdtool/ifdtool.c b/util/ifdtool/ifdtool.c
index e020a8a..516570e 100644
--- a/util/ifdtool/ifdtool.c
+++ b/util/ifdtool/ifdtool.c
@@ -2188,7 +2188,7 @@
{"gpr0-enable", 0, NULL, 'E'},
{"version", 0, NULL, 'v'},
{"help", 0, NULL, 'h'},
- {"platform", 0, NULL, 'p'},
+ {"platform", 1, NULL, 'p'},
{"validate", 0, NULL, 't'},
{"setpchstrap", 1, NULL, 'S'},
{"newvalue", 1, NULL, 'V'},

To view, visit change 80432. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I7d5c31c2b1da544cb73d9e213d463332fcdba7df
Gerrit-Change-Number: 80432
Gerrit-PatchSet: 4
Gerrit-Owner: AtomicFS <vojtech.vesely@9elements.com>
Gerrit-Reviewer: Felix Singer <service+coreboot-gerrit@felixsinger.de>
Gerrit-Reviewer: Jan Samek <jan.samek@siemens.com>
Gerrit-Reviewer: Marvin Drees <marvin.drees@9elements.com>
Gerrit-Reviewer: Maximilian Brune <maximilian.brune@9elements.com>
Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer@coreboot.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-Reviewer: deleted
Gerrit-CC: Felix Held <felix-coreboot@felixheld.de>
Gerrit-MessageType: merged