Re: [PATCH v2 01/11] staging: drm/imx: Add LDB support

2013-03-28 Thread Shawn Guo
On Wed, Mar 27, 2013 at 06:30:35PM +0100, Philipp Zabel wrote: +config DRM_IMX_LDB + tristate Support for LVDS displays + help + Choose this to enable the internal LVDS Display Bridge (LDB) + found on i.MX53 and i.MX6 processors. + depends on DRM_IMX + Shouldn't we

Re: [PATCH v2 01/11] staging: drm/imx: Add LDB support

2013-03-28 Thread Philipp Zabel
Am Donnerstag, den 28.03.2013, 14:08 +0800 schrieb Shawn Guo: On Wed, Mar 27, 2013 at 06:30:35PM +0100, Philipp Zabel wrote: +config DRM_IMX_LDB + tristate Support for LVDS displays + help + Choose this to enable the internal LVDS Display Bridge (LDB) + found on i.MX53 and

Re: [PATCH v2 01/11] staging: drm/imx: Add LDB support

2013-03-28 Thread Philipp Zabel
Hi Martin, Am Mittwoch, den 27.03.2013, 19:40 +0100 schrieb Martin Fuzzey: Hi Philipp, On 27/03/13 18:30, Philipp Zabel wrote: +static bool imx_ldb_encoder_mode_fixup(struct drm_encoder *encoder, + const struct drm_display_mode *mode, + struct

[PATCH v2 01/11] staging: drm/imx: Add LDB support

2013-03-27 Thread Philipp Zabel
From: Sascha Hauer s.ha...@pengutronix.de This adds support for the LVDS Display Bridge contained in i.MX5 and i.MX6 SoCs. Bit mapping, data width, and video timings are configurable via device tree. Dual-channel mode is supported for a single high-resolution source. Signed-off-by: Sascha Hauer

Re: [PATCH v2 01/11] staging: drm/imx: Add LDB support

2013-03-27 Thread Martin Fuzzey
Hi Philipp, On 27/03/13 18:30, Philipp Zabel wrote: +static bool imx_ldb_encoder_mode_fixup(struct drm_encoder *encoder, + const struct drm_display_mode *mode, + struct drm_display_mode *adjusted_mode) +{ +/* + struct imx_ldb_channel