It seems that below patch should be modifed for only one of two outbound nodes 
- panel
and bridge - to be bound because in case of using bridge device, the bridge 
driver will
bind the panel. I will fix and post it again.

Thanks,
Inki Dae

2015년 12월 02일 20:57에 Inki Dae 이(가) 쓴 글:
> This patch adds of_graph dt binding support for panel device
> and also keeps the backward compatibility.
> 
> i.e.,
> The dts file for Exynos5800 based peach pi board
> has a panel property so we need to keep the backward compatibility.
> 
> Changelog v2:
> - return -EINVAL if getting a port node failed.
> 
> Signed-off-by: Inki Dae <inki....@samsung.com>
> ---
>  drivers/gpu/drm/exynos/exynos_dp_core.c | 21 +++++++++++++++++++--
>  1 file changed, 19 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/gpu/drm/exynos/exynos_dp_core.c 
> b/drivers/gpu/drm/exynos/exynos_dp_core.c
> index 94f02a0..0b53045 100644
> --- a/drivers/gpu/drm/exynos/exynos_dp_core.c
> +++ b/drivers/gpu/drm/exynos/exynos_dp_core.c
> @@ -1392,7 +1392,7 @@ static const struct component_ops exynos_dp_ops = {
>  static int exynos_dp_probe(struct platform_device *pdev)
>  {
>       struct device *dev = &pdev->dev;
> -     struct device_node *panel_node, *bridge_node, *endpoint;
> +     struct device_node *panel_node = NULL, *bridge_node, *endpoint = NULL;
>       struct exynos_dp_device *dp;
>       int ret;
>  
> @@ -1403,15 +1403,32 @@ static int exynos_dp_probe(struct platform_device 
> *pdev)
>  
>       platform_set_drvdata(pdev, dp);
>  
> +     /* This is for the backward compatibility. */
>       panel_node = of_parse_phandle(dev->of_node, "panel", 0);
>       if (panel_node) {
>               dp->panel = of_drm_find_panel(panel_node);
>               of_node_put(panel_node);
>               if (!dp->panel)
>                       return -EPROBE_DEFER;
> +     } else {
> +             endpoint = of_graph_get_next_endpoint(dev->of_node, NULL);
> +             if (endpoint) {
> +                     panel_node = of_graph_get_remote_port_parent(endpoint);
> +                     if (panel_node) {
> +                             dp->panel = of_drm_find_panel(panel_node);
> +                             of_node_put(panel_node);
> +                             if (!dp->panel)
> +                                     return -EPROBE_DEFER;
> +                     } else {
> +                             DRM_ERROR("no port node for panel device.\n");
> +                             return -EINVAL;
> +                     }
> +             }
>       }
>  
> -     endpoint = of_graph_get_next_endpoint(dev->of_node, NULL);
> +     panel_node = !endpoint ? NULL : panel_node;
> +
> +     endpoint = of_graph_get_next_endpoint(dev->of_node, panel_node);
>       if (endpoint) {
>               bridge_node = of_graph_get_remote_port_parent(endpoint);
>               if (bridge_node) {
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to