During the review of a patch for pci_endpoint_test driver [1], Greg spotted the wrong usage of the return value of IOCTLs in the driver. This series fixes that by returning 0 for success and negative error code for failure. Relevant change is also made to the userspace tool and the Documentation. Along with those, there are couple more patches fixing other small issues I noted. NOTE: I have just compile tested this series. So it'd be good if someone can test it on the PCI endpoint setup. Thanks, Mani [1] https://lore.kernel.org/all/20220816100617.90720-1-mie@xxxxxxxxxx/ Manivannan Sadhasivam (5): misc: pci_endpoint_test: Remove unnecessary WARN_ON misc: pci_endpoint_test: Fix the return value of IOCTL tools: PCI: Fix parsing the return value of IOCTLs tools: PCI: Fix memory leak Documentation: PCI: endpoint: Use the correct return value of pcitest.sh Documentation/PCI/endpoint/pci-test-howto.rst | 152 ++++++++-------- drivers/misc/pci_endpoint_test.c | 167 ++++++++---------- tools/pci/pcitest.c | 48 ++--- 3 files changed, 179 insertions(+), 188 deletions(-) -- 2.25.1
