All of lore.kernel.org
 help / color / mirror / Atom feed
From: Javier Martinez Canillas <javier@osg.samsung.com>
To: Inki Dae <inki.dae@samsung.com>, dri-devel@lists.freedesktop.org
Cc: mark.rutland@arm.com, devicetree@vger.kernel.org,
	k.kozlowski@samsung.com, linux-samsung-soc@vger.kernel.org,
	pawel.moll@arm.com, ijc+devicetree@hellion.org.uk,
	robh+dt@kernel.org, galak@codeaurora.org, kgene.kim@samsung.com
Subject: Re: [PATCH 4/4] ARM: dts: Use OF graph for DP to panel connection in exynos5800-peach-pi
Date: Mon, 7 Dec 2015 10:41:12 -0300	[thread overview]
Message-ID: <56658C78.8090509@osg.samsung.com> (raw)
In-Reply-To: <1449492758-19989-5-git-send-email-inki.dae@samsung.com>

Hello Inki,

On 12/07/2015 09:52 AM, Inki Dae wrote:
> From: Javier Martinez Canillas <javier@osg.samsung.com>
>

Thanks a lot for posting this patch.
 
> The DT binding for the Exynos DRM Display Port (DP) driver isn't consistent
> since it uses a phandle to describe the connection between the DP port and
> the display panel but uses the OF graph ports and endpoints to describe the
> connection betwen the DP port, a bridge chip and the panel.
> 
> The Exynos DP driver and the DT binding have been changed to allow also to
> describe the DP port to panel connection using ports / endpoints (OF graph)
> so this patch changes the Exynos5800 Peach Pi DT to make it consistent with
> the Exynos5420 Peach Pit that has a eDP to LVDS chip and uses OF graph too.
> 
> Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
> Tested-by: Javier Martinez Canillas <javier@osg.samsung.com>

This tag was not in my original patch, it's true that I tested
it but will someone believe me? ;)

> Reviewed-by: Inki Dae <inki.dae@samsung.com>

Thanks for the review.

Best regards,
-- 
Javier Martinez Canillas
Open Source Group
Samsung Research America
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

  reply	other threads:[~2015-12-07 13:41 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-12-07 12:52 [PATCH v2 0/4] drm/exynos: dp: consider port node outbound for panel Inki Dae
2015-12-07 12:52 ` [PATCH v3 1/4] drm/exynos: dp: add of_graph dt binding support " Inki Dae
2015-12-07 12:52 ` [PATCH v2 2/4] drm/exynos: dp: fix wrong return type Inki Dae
2015-12-07 12:52 ` [PATCH v2 3/4] dt-bindings: exynos-dp: update ports node binding for panel Inki Dae
2015-12-08 15:29   ` Rob Herring
2015-12-07 12:52 ` [PATCH 4/4] ARM: dts: Use OF graph for DP to panel connection in exynos5800-peach-pi Inki Dae
2015-12-07 13:41   ` Javier Martinez Canillas [this message]
2015-12-07 15:36     ` Inki Dae
2015-12-08  0:48       ` Krzysztof Kozlowski
2015-12-08  1:44         ` Javier Martinez Canillas
2015-12-08  2:09         ` Inki Dae
2015-12-08  0:45   ` Krzysztof Kozlowski
2015-12-08  1:33     ` Javier Martinez Canillas
2015-12-08  1:47       ` Krzysztof Kozlowski
2015-12-08  2:13         ` Inki Dae

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=56658C78.8090509@osg.samsung.com \
    --to=javier@osg.samsung.com \
    --cc=devicetree@vger.kernel.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=galak@codeaurora.org \
    --cc=ijc+devicetree@hellion.org.uk \
    --cc=inki.dae@samsung.com \
    --cc=k.kozlowski@samsung.com \
    --cc=kgene.kim@samsung.com \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=mark.rutland@arm.com \
    --cc=pawel.moll@arm.com \
    --cc=robh+dt@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.