Re: [PATCH 2/5] gpio: gpiolib: Print error number if gpio hog failed

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 03/08/2016 08:32 AM, Laxman Dewangan wrote:

On Tuesday 08 March 2016 07:52 PM, Thierry Reding wrote:
On Tue, Mar 08, 2016 at 05:32:05PM +0530, Laxman Dewangan wrote:

Signed-off-by: Laxman Dewangan <ldewangan@xxxxxxxxxx>
---
  drivers/gpio/gpiolib.c | 9 +++++----
  1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/drivers/gpio/gpiolib.c b/drivers/gpio/gpiolib.c
index bc788b9..7575ebb 100644
--- a/drivers/gpio/gpiolib.c
+++ b/drivers/gpio/gpiolib.c
@@ -2621,15 +2621,16 @@ int gpiod_hog(struct gpio_desc *desc, const
char *name,
      local_desc = gpiochip_request_own_desc(chip, hwnum, name);
      if (IS_ERR(local_desc)) {
-        pr_err("requesting hog GPIO %s (chip %s, offset %d) failed\n",
-               name, chip->label, hwnum);
+        status = PTR_ERR(local_desc);
+        pr_err("requesting hog GPIO %s, chip %s, offset %d failed
%d\n",
+               name, chip->label, hwnum, status);
I find this type of format hard to read. I prefer a semi-colon to
separate the message from the failure reason (i.e. error code).

Besides that I don't understand why you're dropping the parentheses
around the "chip %s, offset %d", I found that easier on the eye.

I did to accommodate the  3 extra character ( %d) for string format on
that line as it was already near to 80 column.
Just did not want to split in multiple lines.

Note that strings shouldn't be split across lines since it makes it harder to grep for them. This is one case where the 80-column limit isn't strict, within reason.
--
To unsubscribe from this list: send the line "unsubscribe linux-gpio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux SPI]     [Linux Kernel]     [Linux ARM (vger)]     [Linux ARM MSM]     [Linux Omap]     [Linux Arm]     [Linux Tegra]     [Fedora ARM]     [Linux for Samsung SOC]     [eCos]     [Linux Fastboot]     [Gcc Help]     [Git]     [DCCP]     [IETF Announce]     [Security]     [Linux MIPS]     [Yosemite Campsites]

  Powered by Linux