On 20/05/2014 07:49, Alexandre Courbot wrote:
On Tue, May 20, 2014 at 2:49 AM, Anthony Fee <anthony.fee@xxxxxxxxxx>
wrote:
gpiolib will require all gpio drivers to expicitly set the request
function pointer
in future. To encourage gpio driver developers to adhere to this
standard
gpio-generic.c now sets this function pointer.
This one applies. ;)
Acked-by: Alexandre Courbot <acourbot@xxxxxxxxxx>
(btw, please carry the acks you received to the next revisions of your
patches, that will be that much that the maintainer does not need to
do).
Thanks for the tip. I'll keep this in mind in the future.
Signed-off-by: Anthony Fee <anthony.fee@xxxxxxxxxx>
---
drivers/gpio/gpio-generic.c | 9 +++++++++
1 file changed, 9 insertions(+)
diff --git a/drivers/gpio/gpio-generic.c b/drivers/gpio/gpio-generic.c
index b5dff9e..fea8c82 100644
--- a/drivers/gpio/gpio-generic.c
+++ b/drivers/gpio/gpio-generic.c
@@ -388,6 +388,14 @@ static int bgpio_setup_direction(struct
bgpio_chip *bgc,
return 0;
}
+static int bgpio_request(struct gpio_chip *chip, unsigned gpio_pin)
+{
+ if (gpio_pin < chip->ngpio)
+ return 0;
+
+ return -EINVAL;
Nit: I might have preferred it turned that way:
if (gpio_pin >= chip->ngpio)
return -EINVAL;
return 0;
That makes the error case the exception and the success case the
default, which looks more natural to me. But the statement is simple
enough to be easily understood anyway.
I see your point. It makes sense to end the function on the non-error
condition. Another thing I'll remember in future :)
+}
+
int bgpio_remove(struct bgpio_chip *bgc)
{
return gpiochip_remove(&bgc->gc);
@@ -413,6 +421,7 @@ int bgpio_init(struct bgpio_chip *bgc, struct
device *dev,
bgc->gc.label = dev_name(dev);
bgc->gc.base = -1;
bgc->gc.ngpio = bgc->bits;
+ bgc->gc.request = bgpio_request;
ret = bgpio_setup_io(bgc, dat, set, clr);
if (ret)
--
1.9.0
--
Anthony Fee
Software Engineer
EMUTEX LTD.
Callan Centre, National Technology Park, Limerick, Ireland
Phone: +353 (0)61 514496, Mobile: +353 (0)86 3197614
Web: www.emutex.com, Email: anthony.fee@xxxxxxxxxx
This email may contain information, which is confidential and/or
privileged. The information is intended solely for the use of the
individual or entity named above. If you are not the intended recipient,
be
aware that any disclosure, copying, distribution or use of the contents
is
prohibited. If you have received this electronic transmission in error,
please notify the sender by telephone or return email and delete the
material from your computer. Emutex Ltd is registered in Ireland, No
256238, at Callan Centre, National Technology Park, Limerick, Ireland.
--
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
