Thread (7 messages) 7 messages, 2 authors, 2009-11-23

Lifecycle

  1. Posted niej0001@gmail.com (Jun Nie)

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

From: Jun Nie <hidden>
Date: 2009-11-20 02:38:18
Also in: linux-arm-kernel

2009/11/20 Eric Miao [off-list ref]:
quoted
+       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?
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help