[PATCH 2/5] Treo680: remove unused LCD power GPIO
Eric Miao
eric.y.miao at gmail.com
Tue Sep 1 07:42:14 EDT 2009
Tomáš 'Sleep_Walker' Čech wrote:
> Hi,
>
> Dne Út 1. září 2009 11:49:24 Eric Miao napsal(a):
>> Tomáš 'Sleep_Walker' Čech wrote:
>>> 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...
>>>
>> The name sounds like a GPIO for the power of the LCD panel itself,
>> does it make sense to have this somewhere in the LCD power_{on,off}
>> hooks?
>>
>
> Yes, it would be much better.
> I was looking for the right place - did you mean pxafb_mach_info.pxafb_lcd_power?
>
I think so.
More information about the linux-arm-kernel
mailing list