[PATCH 2/5] Treo680: remove unused LCD power GPIO
Tomáš 'Sleep_Walker' Čech
sleep_walker at suse.cz
Mon Aug 31 14:34:53 EDT 2009
Hi,
Dne Po 31. srpna 2009 19:34:51 Marek Vasut napsal(a):
> Dne Po 31. srpna 2009 19:25:58 Tomáš Čech napsal(a):
> > From: Tomáš Čech <sleep walker at suse.cz>
> >
> >
> > Signed-off-by: Tomáš Čech <sleep_walker at suse.cz>
> > ---
> > arch/arm/mach-pxa/treo680.c | 10 +---------
> > 1 files changed, 1 insertions(+), 9 deletions(-)
> >
> > diff --git a/arch/arm/mach-pxa/treo680.c b/arch/arm/mach-pxa/treo680.c
> > index 050ad8e..57bccc3 100644
> > --- a/arch/arm/mach-pxa/treo680.c
> > +++ b/arch/arm/mach-pxa/treo680.c
> > @@ -254,16 +254,9 @@ static int treo680_backlight_init(struct device *dev)
> > ret = gpio_direction_output(GPIO_NR_TREO680_BL_POWER, 0);
> > if (ret)
> > goto err2;
> > - ret = gpio_request(GPIO_NR_TREO680_LCD_POWER, "LCD POWER");
> > - if (ret)
> > - goto err2;
> > - ret = gpio_direction_output(GPIO_NR_TREO680_LCD_POWER, 0);
> > - if (ret)
> > - goto err3;
> >
> > return 0;
> > -err3:
> > - gpio_free(GPIO_NR_TREO680_LCD_POWER);
> > +
> > err2:
> > gpio_free(GPIO_NR_TREO680_BL_POWER);
> > err:
> > @@ -279,7 +272,6 @@ static int treo680_backlight_notify(int brightness)
> > static void treo680_backlight_exit(struct device *dev)
> > {
> > gpio_free(GPIO_NR_TREO680_BL_POWER);
> > - gpio_free(GPIO_NR_TREO680_LCD_POWER);
> > }
> >
> > static struct platform_pwm_backlight_data treo680_backlight_data = {
>
> Why was that GPIO there before then? Does this have any effect on the boot
> problems you are experiencing with Centro/T680? Otherwise seems OK to me
I used it first in brightness setting for 0 value. I found it not good idea. I removed it from function, but I wanted to keep GPIO registered and direction set. But it can work well even without that and I'd like to add some function to export some control GPIOs through /sys/class/gpio interface (bluetooth power, GSM power, GSM reset...) - this GPIO will be also exported there...
Best regards,
Tomas Cech
More information about the linux-arm-kernel
mailing list