On Wed, 2017-05-10 at 15:55 +0200, Philipp Zabel wrote:
On Wed, 2017-05-10 at 16:17 +0300, Leonard Crestez wrote:
Not having an endpoint bound in DT should not cause a failure here, there are fallbacks. So explicitly accept a missing endpoint.
This behavior change was introduced by refactoring in drm_of parsing code and it should not require dts changes.
In particular this fixes imx6qdl-sabreauto boards.
diff --git a/drivers/gpu/drm/imx/imx-ldb.c b/drivers/gpu/drm/imx/imx-ldb.c index 8fb801f..8b05ecb 100644 --- a/drivers/gpu/drm/imx/imx-ldb.c +++ b/drivers/gpu/drm/imx/imx-ldb.c @@ -673,7 +673,7 @@ static int imx_ldb_bind(struct device *dev, struct device *master, void *data) ret = drm_of_find_panel_or_bridge(child, imx_ldb->lvds_mux ? 4 : 2, 0, &channel->panel, &channel->bridge);
if (ret)
if (ret && ret != -ENODEV)return ret; /* panel ddc only if there is no bridge */
Thank you, applied to imx-drm/fixes.
regards Philipp
How are such fixes going to make their way upstream? It's been almost a month and this patch did not get into either linux-next or torvalds/master or anywhere other than pza/imx-drm/fixes.
-- Regards, Leonard