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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Date:	Tue, 7 Apr 2009 14:00:22 +0200 (CEST)
From:	Geert Uytterhoeven <Geert.Uytterhoeven@...ycom.com>
To:	David Brownell <dbrownell@...rs.sourceforge.net>,
	Richard Purdie <rpurdie@...ux.intel.com>
cc:	Linux Kernel Development <linux-kernel@...r.kernel.org>
Subject: Re: leds: just ignore invalid GPIOs in leds-gpio

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@...rs.sourceforge.net>
> AuthorDate: Thu Mar 5 16:46:44 2009 -0800
> Committer:  Richard Purdie <rpurdie@...ux.intel.com>
> 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@...ux.intel.com: adjusted to apply against other led tree changes]
>     Signed-off-by: David Brownell <dbrownell@...rs.sourceforge.net>
>     Tested-by: Diego Dompe <diego.dompe@...gerun.com>
>     Signed-off-by: Richard Purdie <rpurdie@...ux.intel.com>
> ---
>  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@...ycom.com
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@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/