* [PATCH] drm/exynos/dsi: Remove error handling for bridge_node DT parsing [not found] <CGME20170609092307epcas5p3a7851f80bd799626782941a640824521@epcas5p3.samsung.com> @ 2017-06-09 9:23 ` Hoegeun Kwon 2017-06-13 9:25 ` Inki Dae 0 siblings, 1 reply; 3+ messages in thread From: Hoegeun Kwon @ 2017-06-09 9:23 UTC (permalink / raw) To: inki.dae, jy0922.shim, sw0312.kim, airlied, kgene, krzk Cc: javier, a.hajda, dri-devel, linux-arm-kernel, linux-samsung-soc, linux-kernel, Hoegeun Kwon The bridge_node is unnecessary between FIMD and DSIM. If don't remove error handling, it will not work between FIMD and DSIM. So remove error handling. Signed-off-by: Hoegeun Kwon <hoegeun.kwon@samsung.com> --- drivers/gpu/drm/exynos/exynos_drm_dsi.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/exynos/exynos_drm_dsi.c b/drivers/gpu/drm/exynos/exynos_drm_dsi.c index a11b795..6ee0dac 100644 --- a/drivers/gpu/drm/exynos/exynos_drm_dsi.c +++ b/drivers/gpu/drm/exynos/exynos_drm_dsi.c @@ -1651,8 +1651,6 @@ static int exynos_dsi_parse_dt(struct exynos_dsi *dsi) return ret; dsi->bridge_node = of_graph_get_remote_node(node, DSI_PORT_IN, 0); - if (!dsi->bridge_node) - return -EINVAL; return 0; } -- 1.9.1 ^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] drm/exynos/dsi: Remove error handling for bridge_node DT parsing 2017-06-09 9:23 ` [PATCH] drm/exynos/dsi: Remove error handling for bridge_node DT parsing Hoegeun Kwon @ 2017-06-13 9:25 ` Inki Dae 2017-06-13 10:33 ` Hoegeun Kwon 0 siblings, 1 reply; 3+ messages in thread From: Inki Dae @ 2017-06-13 9:25 UTC (permalink / raw) To: Hoegeun Kwon, jy0922.shim, sw0312.kim, airlied, kgene, krzk Cc: javier, a.hajda, dri-devel, linux-arm-kernel, linux-samsung-soc, linux-kernel 2017년 06월 09일 18:23에 Hoegeun Kwon 이(가) 쓴 글: > The bridge_node is unnecessary between FIMD and DSIM. If don't remove > error handling, it will not work between FIMD and DSIM. So remove > error handling. Please make sure to describe why bridge_node is unnecessary. For example, In case of Exynos SoC, a bridge device such as mDNIe and MIC could be placed between Display Controller and MIPI DSI device but the bridge device is required optionally. Thanks, Inki Dae > > Signed-off-by: Hoegeun Kwon <hoegeun.kwon@samsung.com> > --- > drivers/gpu/drm/exynos/exynos_drm_dsi.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/drivers/gpu/drm/exynos/exynos_drm_dsi.c b/drivers/gpu/drm/exynos/exynos_drm_dsi.c > index a11b795..6ee0dac 100644 > --- a/drivers/gpu/drm/exynos/exynos_drm_dsi.c > +++ b/drivers/gpu/drm/exynos/exynos_drm_dsi.c > @@ -1651,8 +1651,6 @@ static int exynos_dsi_parse_dt(struct exynos_dsi *dsi) > return ret; > > dsi->bridge_node = of_graph_get_remote_node(node, DSI_PORT_IN, 0); > - if (!dsi->bridge_node) > - return -EINVAL; > > return 0; > } > ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] drm/exynos/dsi: Remove error handling for bridge_node DT parsing 2017-06-13 9:25 ` Inki Dae @ 2017-06-13 10:33 ` Hoegeun Kwon 0 siblings, 0 replies; 3+ messages in thread From: Hoegeun Kwon @ 2017-06-13 10:33 UTC (permalink / raw) To: Inki Dae, jy0922.shim, sw0312.kim, airlied, kgene, krzk Cc: javier, a.hajda, dri-devel, linux-arm-kernel, linux-samsung-soc, linux-kernel, Hoegeun Kwon On 06/13/2017 06:25 PM, Inki Dae wrote: > > 2017년 06월 09일 18:23에 Hoegeun Kwon 이(가) 쓴 글: >> The bridge_node is unnecessary between FIMD and DSIM. If don't remove >> error handling, it will not work between FIMD and DSIM. So remove >> error handling. > Please make sure to describe why bridge_node is unnecessary. > For example, > In case of Exynos SoC, a bridge device such as mDNIe and MIC could be placed between Display Controller and MIPI DSI device but the bridge device is required optionally. > > Thanks, > Inki Dae Remove the error handling for bridge_node because the bridge_node is required optionally. It has already been described in commit message. For example, In case of Exynos 3250 SoC, no bridge_node is required between FIMD and MIPI DSI. Best regards, Hoegeun > >> Signed-off-by: Hoegeun Kwon <hoegeun.kwon@samsung.com> >> --- >> drivers/gpu/drm/exynos/exynos_drm_dsi.c | 2 -- >> 1 file changed, 2 deletions(-) >> >> diff --git a/drivers/gpu/drm/exynos/exynos_drm_dsi.c b/drivers/gpu/drm/exynos/exynos_drm_dsi.c >> index a11b795..6ee0dac 100644 >> --- a/drivers/gpu/drm/exynos/exynos_drm_dsi.c >> +++ b/drivers/gpu/drm/exynos/exynos_drm_dsi.c >> @@ -1651,8 +1651,6 @@ static int exynos_dsi_parse_dt(struct exynos_dsi *dsi) >> return ret; >> >> dsi->bridge_node = of_graph_get_remote_node(node, DSI_PORT_IN, 0); >> - if (!dsi->bridge_node) >> - return -EINVAL; >> >> return 0; >> } >> > ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-06-13 10:33 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <CGME20170609092307epcas5p3a7851f80bd799626782941a640824521@epcas5p3.samsung.com>
2017-06-09 9:23 ` [PATCH] drm/exynos/dsi: Remove error handling for bridge_node DT parsing Hoegeun Kwon
2017-06-13 9:25 ` Inki Dae
2017-06-13 10:33 ` Hoegeun Kwon
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox