On Thu, Mar 28, 2013 at 04:23:30PM +0100, Philipp Zabel wrote: > +static void imx_ldb_encoder_prepare(struct drm_encoder *encoder) > +{ > + struct imx_ldb_channel *imx_ldb_ch = enc_to_imx_ldb_ch(encoder); > + struct imx_ldb *ldb = imx_ldb_ch->ldb; > + struct drm_display_mode *mode = &encoder->crtc->mode; > + unsigned long serial_clk; > + unsigned long di_clk = mode->clock * 1000; > + int mux = imx_drm_encoder_get_mux_id(imx_ldb_ch->imx_drm_encoder, > + encoder->crtc); > + > + if (ldb->ldb_ctrl & LDB_SPLIT_MODE_EN) { > + /* dual channel LVDS mode */ > + serial_clk = 3500UL * mode->clock; > + imx_ldb_set_clock(ldb, mux, 0, serial_clk, di_clk); > + imx_ldb_set_clock(ldb, mux, 1, serial_clk, di_clk); > + } else { > + serial_clk = 7000UL * mode->clock; > + imx_ldb_set_clock(ldb, mux, imx_ldb_ch->chno, serial_clk, di_clk); > + } > + > + imx_drm_crtc_panel_format(encoder->crtc, DRM_MODE_ENCODER_LVDS, > + V4L2_PIX_FMT_RGB24); I have panel which needs it to be V4L2_PIX_FMT_BGR666. We should probably have a device tree property for that like interface_pix_fmt in parallel display support? Shawn > +} _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel