Re: [PATCH v2 0/4] gpiolib: some fixup patches
From: Alexandre Courbot
Date: Thu Feb 21 2013 - 21:20:15 EST
Grant, will you be able to include these for 3.9? They fix code that
you merged recently, so I'd be glad if they could be squashed into the
patch mentioned in the description.
On Fri, Feb 15, 2013 at 2:46 PM, Alexandre Courbot <acourbot@xxxxxxxxxx> wrote:
> This short series is a fixup to patch 6/9 of my previous series ("gpiolib: use
> descriptors internally"), which mainly addresses issues raised by Linus W. in
> this patch and a few other minor ones caught while proof-reading gpiolib.
> First patch is the most significant. It mostly avoids oopses when passing an
> invalid GPIO to some functions. Also gpio_get/set_value*() now return 0 when
> given an invalid GPIO number instead of oopsing. I tend to think that's an
> improvement, but if it's not please let me know.
> Ideally these patches should be melded into the above-mentioned patch since
> they were intended to become a new revision, but I leave that decision to
> Grant's discretion.
> This patchset has been tested on Tegra 2/Ventana with a GPIO backlight driver
> and the sysfs interface.
> Changes from v1:
> - as suggested by Ryan Mallon, add a dedicated control flow for signaling
> invalid GPIOs to avoid meaningless error messages and ugly ternary operators
> - use pr_warn instead of pr_debug to signal invalid GPIOs
> - do descriptors validity check before acquiring any lock
> Alexandre Courbot (4):
> gpiolib: check descriptors validity before use
> gpiolib: use const parameters when possible
> gpiolib: move comment to right function
> gpiolib: rename local offset variables to "hwgpio"
> drivers/gpio/gpiolib.c | 213 +++++++++++++++++++++++++++----------------------
> 1 file changed, 117 insertions(+), 96 deletions(-)
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/