On Wed, 2012-08-15 at 11:26 -0400, Raphaël Assénat wrote:
> > +
> > + /* ChiMei G121S1-L01 */
> > + {
> > + {
>
> ...
>
> > + .vsync_level = OMAPDSS_SIG_ACTIVE_HIGH,
> > + .hsync_level = OMAPDSS_SIG_ACTIVE_HIGH,
> > + .data_pclk_edge = OMAPDSS_DRIVE_SIG_RISING_EDGE,
> > + .de_level = OMAPDSS_SIG_ACTIVE_HIGH,
> > + .sync_pclk_edge = OMAPDSS_DRIVE_SIG_OPPOSITE_EDGES,
>
> Actually those 3 panels only use the DE signal. The hsync/vsync signals
> are not used and on our system we mux them out to make sure they are
> kept low as recommended in the panel datasheets.
>
> Since vsync/hsync are not used, I think the vsync_level, hsync_level and
> sync_pclk_edge entries could be removed. Otherwise the updated patch
> works fine as is.Okay. How do panels like that work? How can they know where a new frame starts? Actually, I now googled for those panels, and they are all LVDS panels, not DPI panels. So the patch doesn't look correct at all. Do you have a DPI-to-LVDS converter chip on your board? Tomi
signature.asc
Description: This is a digitally signed message part
