Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/47573 )
Change subject: util/cbfstool/amdcompress: fix argument requirement ......................................................................
util/cbfstool/amdcompress: fix argument requirement
The compress and uncompress options don't have arguments and shouldn't consume the next token. So replace required_argument with no_argument for the two options.
Change-Id: Ib9b190f2cf606109f82a65d00327871d6ffb7082 Signed-off-by: Felix Held felix-coreboot@felixheld.de Reviewed-on: https://review.coreboot.org/c/coreboot/+/47573 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Kyösti Mälkki kyosti.malkki@gmail.com Reviewed-by: Marshall Dawson marshalldawson3rd@gmail.com --- M util/cbfstool/amdcompress.c 1 file changed, 2 insertions(+), 2 deletions(-)
Approvals: build bot (Jenkins): Verified Kyösti Mälkki: Looks good to me, but someone else must approve Marshall Dawson: Looks good to me, approved
diff --git a/util/cbfstool/amdcompress.c b/util/cbfstool/amdcompress.c index ad6a039..76089ea 100644 --- a/util/cbfstool/amdcompress.c +++ b/util/cbfstool/amdcompress.c @@ -29,9 +29,9 @@ static struct option long_options[] = { {"infile", required_argument, 0, 'i' }, {"outfile", required_argument, 0, 'o' }, - {"compress", required_argument, 0, 'c' }, + {"compress", no_argument, 0, 'c' }, {"maxsize", required_argument, 0, 'm' }, - {"uncompress", required_argument, 0, 'u' }, + {"uncompress", no_argument, 0, 'u' }, {"help", no_argument, 0, 'h' }, };