When the commit fbca0b284bd0 ("tools: PCI: Add 'h' in optstring of getopt()") introduced a fix to support -h command line option, it, at the same time, brought a regression to the code which produces a compile-time warning: pcitest.c:203:4: warning: too many arguments for format [-Wformat-extra-args] "usage: %s [options]\n" ^~~~~~~~~~~~~~~~~~~~~~~ Remove trailing comma to make it correct. Fixes: fbca0b284bd0 ("tools: PCI: Add 'h' in optstring of getopt()") Cc: Kishon Vijay Abraham I <kishon@xxxxxx> Cc: Sekhar Nori <nsekhar@xxxxxx> Cc: Lorenzo Pieralisi <lorenzo.pieralisi@xxxxxxx> Signed-off-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> --- tools/pci/pcitest.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/pci/pcitest.c b/tools/pci/pcitest.c index 81b89260e80f..c495e5b5fd7f 100644 --- a/tools/pci/pcitest.c +++ b/tools/pci/pcitest.c @@ -212,7 +212,7 @@ int main(int argc, char **argv) "\t-r Read buffer test\n" "\t-w Write buffer test\n" "\t-c Copy buffer test\n" - "\t-s <size> Size of buffer {default: 100KB}\n", + "\t-s <size> Size of buffer {default: 100KB}\n" "\t-h Print this help message\n", argv[0]); return -EINVAL; -- 2.20.1