| Message ID | 20190628131218.10244-2-andriy.shevchenko@linux.intel.com (mailing list archive) |
|---|---|
| State | New, archived |
| Delegated to: | Bjorn Helgaas |
| Headers | show |
| Series | [v1,1/2] tools: PCI: Fix compilation error | expand |
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;
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@ti.com> Cc: Sekhar Nori <nsekhar@ti.com> Cc: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> --- tools/pci/pcitest.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)