Hello Inki, On 12/03/2015 02:45 AM, Inki Dae wrote: > This patch makes it to return -EINVAL instead of -ENXIO > when getting a port or endpoint node failed. > > Signed-off-by: Inki Dae <[email protected]> > --- > drivers/gpu/drm/exynos/exynos_drm_dsi.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/exynos/exynos_drm_dsi.c > b/drivers/gpu/drm/exynos/exynos_drm_dsi.c > index 7c3606a..a24bf8b 100644 > --- a/drivers/gpu/drm/exynos/exynos_drm_dsi.c > +++ b/drivers/gpu/drm/exynos/exynos_drm_dsi.c > @@ -1735,13 +1735,13 @@ static int exynos_dsi_parse_dt(struct exynos_dsi *dsi) > > ep = of_graph_get_next_endpoint(node, NULL); > if (!ep) { > - ret = -ENXIO; > + ret = -EINVAL; > goto end; > } > > dsi->bridge_node = of_graph_get_remote_port_parent(ep); > if (!dsi->bridge_node) { > - ret = -ENXIO; > + ret = -EINVAL; > goto end; > } > end: >
Yes, I also think that -EINVAL is a better error code in this case. Reviewed-by: Javier Martinez Canillas <[email protected]> Best regards, -- Javier Martinez Canillas Open Source Group Samsung Research America -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
