Re: leds: just ignore invalid GPIOs in leds-gpio

[Geert Uytterhoeven]

On Mon, 6 Apr 2009, Linux Kernel Mailing List wrote:
> Gitweb:     http://git.kernel.org/linus/d379ee8acd0719736ee7f1d1ccc3b5765880eaf8
> Commit:     d379ee8acd0719736ee7f1d1ccc3b5765880eaf8
> Parent:     7fbc3a9b132e93b2ba1fd889c1ad8a4135731cc3
> Author:     David Brownell <dbrownell@xxxxxxxxxxxxxxxxxxxxx>
> AuthorDate: Thu Mar 5 16:46:44 2009 -0800
> Committer:  Richard Purdie <rpurdie@xxxxxxxxxxxxxxx>
> CommitDate: Mon Apr 6 16:06:27 2009 +0100
> 
>     leds: just ignore invalid GPIOs in leds-gpio
>     
>     Sometimes it's awkward to make sure that the array in the
>     platform_data handed to the leds-gpio driver has only valid
>     data ... some leds may not be always available, and coping
>     with that currently requires patching or rebuilding the array.
>     
>     This patch fixes that by making it be OK to pass an invalid
>     GPIO (such as "-EINVAL") ... such table entries are skipped.
>     
>     [rpurdie@xxxxxxxxxxxxxxx: adjusted to apply against other led tree changes]
>     Signed-off-by: David Brownell <dbrownell@xxxxxxxxxxxxxxxxxxxxx>
>     Tested-by: Diego Dompe <diego.dompe@xxxxxxxxxxxx>
>     Signed-off-by: Richard Purdie <rpurdie@xxxxxxxxxxxxxxx>
> ---
>  drivers/leds/leds-gpio.c |    9 +++++++++
>  1 files changed, 9 insertions(+), 0 deletions(-)
> 
> diff --git a/drivers/leds/leds-gpio.c b/drivers/leds/leds-gpio.c
> index 8fa352a..102ef4a 100644
> --- a/drivers/leds/leds-gpio.c
> +++ b/drivers/leds/leds-gpio.c
> @@ -78,6 +78,13 @@ static int __devinit create_gpio_led(const struct gpio_led *template,
>  {
>  	int ret;
>  
> +	/* skip leds that aren't available */
> +	if (!gpio_is_valid(template->gpio)) {
> +		printk(KERN_INFO "Skipping unavilable LED gpio %d (%s)\n", 
> +				template->gpio, template->name);
> +		return;

| drivers/leds/leds-gpio.c:85: warning: 'return' with no value, in function
| returning non-void

So what should we return here? -ENODEV? -EINVAL? Anything else?

> +	}
> +
>  	ret = gpio_request(template->gpio, template->name);
>  	if (ret < 0)
>  		return ret;

With kind regards,

Geert Uytterhoeven
Software Architect

Sony Techsoft Centre Europe
The Corporate Village  Da Vincilaan 7-D1  B-1935 Zaventem  Belgium

Phone:    +32 (0)2 700 8453
Fax:      +32 (0)2 700 8622
E-mail:   Geert.Uytterhoeven@xxxxxxxxxxx
Internet: http://www.sony-europe.com/

A division of Sony Europe (Belgium) N.V.
VAT BE 0413.825.160 Â RPR Brussels
Fortis  BIC GEBABEBB  IBAN BE41293037680010
--
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/