Hi, On Thu, 2011-03-03 at 17:46 -0600, Steve Sakoman wrote: > This patch adds support for the Gumstix Palo35 expansion board > which utilizes the 320 x 240 pixel LG.Philips LB035Q02 LCD Panel > > Signed-off-by: Steve Sakoman <steve@xxxxxxxxxxx> This looks about as good as the other panel drivers. Which means it works for now but it does has faults =). One thing to fix: - There should be locking here. Check panel-taal.c. Mutexes around the xxx_panel_enable() etc calls should do the thing. And another thing to think about, but there's probably not a simple fix for this right now: This wouldn't work with two panel devices. If you think the case where you have to panels, meaning two spi devices and two dss devices. You first register the spi driver, which would call probe for both of the spi devices. The first spi probe call would registers the dss driver, which would call probe for both of the dss devices. At this point only the first spi device would have been probed... This is a bit tricky, and I'm not sure if there's any ready-made solution for this. But what we need is a way to link a spesific spi device to a specific dss device in the board file. Then this panel module would register both dss driver and spi driver in the module_init(), and after both dss and spi probe has been called for the particular dss+spi device pair, the actual display driver would start. So this needs some work, and also some careful checks in, for example, panel_enable, because nothing says that the spi device is ready when panel_enable is called. Tomi -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
