spi: spidev_test: Use perror() only if errno is not 0
authorTiezhu Yang <yangtiezhu@loongson.cn>
Thu, 13 Feb 2020 04:16:07 +0000 (12:16 +0800)
committerMark Brown <broonie@kernel.org>
Thu, 13 Feb 2020 13:17:39 +0000 (13:17 +0000)
commit470a072e12200576788dc622d58894609feae2d7
treefdf990e9665f3f1dbc29fcf73140ad8b5b08e0ed
parent1f3c36328a487059beebd1f7be042e3b7abf7d34
spi: spidev_test: Use perror() only if errno is not 0

It is better to use perror() only if errno is not 0, it should use printf()
when errno is 0, otherwise there exists redudant ": Success".

E.g. without this patch:

$ ./spidev_test -p 1234 --input test.bin
only one of -p and --input may be selected: Success
Aborted (core dumped)

With this patch:

$ ./spidev_test -p 1234 --input test.bin
only one of -p and --input may be selected
Aborted (core dumped)

Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Link: https://lore.kernel.org/r/1581567368-8055-3-git-send-email-yangtiezhu@loongson.cn
Signed-off-by: Mark Brown <broonie@kernel.org>
tools/spi/spidev_test.c