[PATCH] drm/imx: imx-ldb: store LVDS bus configuration in ldb channel

2016-07-22 Thread Ying Liu
Hi Philipp, On Fri, Jul 22, 2016 at 4:59 PM, Philipp Zabel wrote: > Hi Liu, > > thank you for your comments. > > Am Freitag, den 22.07.2016, 12:01 +0800 schrieb Ying Liu: >> Hi Philipp, >> >> This patch's headline doesn't exactly reflect what the patch actually >> does - retrieve lvds bus

[PATCH] drm/imx: imx-ldb: store LVDS bus configuration in ldb channel

2016-07-22 Thread Ying Liu
Hi Philipp, This patch's headline doesn't exactly reflect what the patch actually does - retrieve lvds bus format from imx_crtc_state during encoder mode_set. On Thu, Jul 21, 2016 at 9:25 PM, Philipp Zabel wrote: > The code in imx_ldb_encoder_mode_set crashes trying to access the >

[PATCH] drm/imx: imx-ldb: store LVDS bus configuration in ldb channel

2016-07-22 Thread Philipp Zabel
Hi Liu, thank you for your comments. Am Freitag, den 22.07.2016, 12:01 +0800 schrieb Ying Liu: > Hi Philipp, > > This patch's headline doesn't exactly reflect what the patch actually > does - retrieve lvds bus format from imx_crtc_state during encoder > mode_set. Yes. I initially stored

[PATCH] drm/imx: imx-ldb: store LVDS bus configuration in ldb channel

2016-07-21 Thread Philipp Zabel
The code in imx_ldb_encoder_mode_set crashes trying to access the crtc->state->state drm_atomic_state pointer if that was previously cleared by drm_atomic_helper_swap_state. Instead of trying to walk all connectors during encoder mode_set to determine the LVDS bus format from panel or external