| Message ID | 20200828074251.3788165-1-vkoul@kernel.org (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | drm/bridge: Fix the dsi remote end-points | expand |
On Fri, Aug 28, 2020 at 01:12:50PM +0530, Vinod Koul wrote: > DSI end-points are supposed to be at node 0 and node 1 as per binding. > So fix this and use node 0 and node 1 for dsi. > > Reported-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> > Fixes: 23278bf54afe ("drm/bridge: Introduce LT9611 DSI to HDMI bridge") > Signed-off-by: Vinod Koul <vkoul@kernel.org> Thanks, applied to drm-misc-next. Sam > --- > drivers/gpu/drm/bridge/lontium-lt9611.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/bridge/lontium-lt9611.c b/drivers/gpu/drm/bridge/lontium-lt9611.c > index 1009fc4ed4ed..d734d9402c35 100644 > --- a/drivers/gpu/drm/bridge/lontium-lt9611.c > +++ b/drivers/gpu/drm/bridge/lontium-lt9611.c > @@ -960,13 +960,13 @@ static const struct drm_bridge_funcs lt9611_bridge_funcs = { > static int lt9611_parse_dt(struct device *dev, > struct lt9611 *lt9611) > { > - lt9611->dsi0_node = of_graph_get_remote_node(dev->of_node, 1, -1); > + lt9611->dsi0_node = of_graph_get_remote_node(dev->of_node, 0, -1); > if (!lt9611->dsi0_node) { > dev_err(lt9611->dev, "failed to get remote node for primary dsi\n"); > return -ENODEV; > } > > - lt9611->dsi1_node = of_graph_get_remote_node(dev->of_node, 2, -1); > + lt9611->dsi1_node = of_graph_get_remote_node(dev->of_node, 1, -1); > > lt9611->ac_mode = of_property_read_bool(dev->of_node, "lt,ac-mode"); > > -- > 2.26.2
diff --git a/drivers/gpu/drm/bridge/lontium-lt9611.c b/drivers/gpu/drm/bridge/lontium-lt9611.c index 1009fc4ed4ed..d734d9402c35 100644 --- a/drivers/gpu/drm/bridge/lontium-lt9611.c +++ b/drivers/gpu/drm/bridge/lontium-lt9611.c @@ -960,13 +960,13 @@ static const struct drm_bridge_funcs lt9611_bridge_funcs = { static int lt9611_parse_dt(struct device *dev, struct lt9611 *lt9611) { - lt9611->dsi0_node = of_graph_get_remote_node(dev->of_node, 1, -1); + lt9611->dsi0_node = of_graph_get_remote_node(dev->of_node, 0, -1); if (!lt9611->dsi0_node) { dev_err(lt9611->dev, "failed to get remote node for primary dsi\n"); return -ENODEV; } - lt9611->dsi1_node = of_graph_get_remote_node(dev->of_node, 2, -1); + lt9611->dsi1_node = of_graph_get_remote_node(dev->of_node, 1, -1); lt9611->ac_mode = of_property_read_bool(dev->of_node, "lt,ac-mode");
DSI end-points are supposed to be at node 0 and node 1 as per binding. So fix this and use node 0 and node 1 for dsi. Reported-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> Fixes: 23278bf54afe ("drm/bridge: Introduce LT9611 DSI to HDMI bridge") Signed-off-by: Vinod Koul <vkoul@kernel.org> --- drivers/gpu/drm/bridge/lontium-lt9611.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)