Felix Singer has submitted this change. ( https://review.coreboot.org/c/coreboot/+/80432?usp=email )
Change subject: util/ifdtool.c: Fix long_options for platform ......................................................................
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(-)
Approvals: Jan Samek: Looks good to me, approved build bot (Jenkins): Verified Marvin Drees: Looks good to me, but someone else must approve
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'},