[PATCH 7/8] pxa: add frame buffer on pxa168 aspenite platform

Jun Nie niej0001 at gmail.com
Mon Nov 23 00:52:42 EST 2009


2009/11/20 Eric Miao <eric.y.miao at gmail.com>:
> On Thu, Nov 19, 2009 at 8:38 PM, Jun Nie <niej0001 at gmail.com> wrote:
>> 2009/11/20 Eric Miao <eric.y.miao at gmail.com>:
>>>> +       int err = 0;
>>>> +       if (on) {
>>>> +               if (spi_gpio_reset != -1) {
>>>> +                       err = gpio_request(spi_gpio_reset, "TPO_LCD_SPI_RESET");
>>>> +                       if (err) {
>>>> +                               printk(KERN_ERR "failed to request GPIO for TPO LCD RESET\n");
>>>> +                               return err;
>>>> +                       }
>>>> +                       gpio_direction_output(spi_gpio_reset, 0);
>>>> +                       msleep(100);
>>>
>>> This is really a loooong delay, there any specific reason for such a long
>>> delay, I'm seeing this to slow down the boot up significantly?
>>>
>>> Otherwise looks OK to me, and I prefer the defconfig file change to
>>> be a separate patch please.
>>>
>>
>> Actually, This is just an example for other real products. There is no
>> such spi_reset pin on Aspenite , so machine info set it spi_gpio_reset
>> == -1. Or do you think we should delete these lines?
>>
>
> I see no reason for it to stay there then, please remove.
>

rebased to topic/pxa168fb branch of pxa git repository. changes include:
1. Delete redundant gpio operation in aspenite.c
2. change to include <linux/gpio.h>


More information about the linux-arm-kernel mailing list