Devicetree
 help / color / mirror / Atom feed
From: Rob Herring <robh@kernel.org>
To: Inki Dae <inki.dae@samsung.com>
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,
	dri-devel@lists.freedesktop.org, javier@osg.samsung.com,
	kgene.kim@samsung.com, galak@codeaurora.org
Subject: Re: [PATCH v2 3/4] dt-bindings: exynos-dp: update ports node binding for panel
Date: Tue, 8 Dec 2015 09:29:12 -0600	[thread overview]
Message-ID: <20151208152912.GA31338@rob-hp-laptop> (raw)
In-Reply-To: <1449492758-19989-4-git-send-email-inki.dae@samsung.com>

On Mon, Dec 07, 2015 at 09:52:37PM +0900, Inki Dae wrote:
> This patch updates a ports node binding for panel.
> 
> With this, dp node can have a ports node which describes
> a remote endpoint node that can be connected to panel or bridge
> node.
> 
> Changelog v2:
> - remove unnecessary properties and numbering.
> - update description about eDP device.
> 
> Signed-off-by: Inki Dae <inki.dae@samsung.com>
> Reviewed-by: Javier Martinez Canillas <javier@osg.samsung.com>

Acked-by: Rob Herring <robh@kernel.org>

> ---
>  .../bindings/display/exynos/exynos_dp.txt          | 41 +++++++++++++++++++---
>  1 file changed, 37 insertions(+), 4 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/display/exynos/exynos_dp.txt b/Documentation/devicetree/bindings/display/exynos/exynos_dp.txt
> index 64693f2..22efeba 100644
> --- a/Documentation/devicetree/bindings/display/exynos/exynos_dp.txt
> +++ b/Documentation/devicetree/bindings/display/exynos/exynos_dp.txt
> @@ -1,3 +1,20 @@
> +Device-Tree bindings for Samsung Exynos Embedded DisplayPort Transmitter(eDP)
> +
> +DisplayPort is industry standard to accommodate the growing board adoption
> +of digital display technology within the PC and CE industries.
> +It consolidates the internal and external connection methods to reduce device
> +complexity and cost. It also supports necessary features for important cross
> +industry applications and provides performance scalability to enable the next
> +generation of displays that feature higher color depths, refresh rates, and
> +display resolutions.
> +
> +eDP (embedded display port) device is compliant with Embedded DisplayPort
> +standard as follows,
> +- DisplayPort standard 1.1a for Exynos5250 and Exynos5260.
> +- DisplayPort standard 1.3 for Exynos5422s and Exynos5800.
> +
> +eDP resides between FIMD and panel or FIMD and bridge such as LVDS.
> +
>  The Exynos display port interface should be configured based on
>  the type of panel connected to it.
>  
> @@ -66,8 +83,15 @@ Optional properties for dp-controller:
>  		Hotplug detect GPIO.
>  			Indicates which GPIO should be used for hotplug
>  			detection
> -	-video interfaces: Device node can contain video interface port
> -			    nodes according to [1].
> +Video interfaces:
> +  Device node can contain video interface port nodes according to [1].
> +  The following are properties specific to those nodes:
> +
> +  endpoint node connected to bridge or panel node:
> +   - remote-endpoint: specifies the endpoint in panel or bridge node.
> +		      This node is required in all kinds of exynos dp
> +		      to represent the connection between dp and bridge
> +		      or dp and panel.
>  
>  [1]: Documentation/devicetree/bindings/media/video-interfaces.txt
>  
> @@ -111,9 +135,18 @@ Board Specific portion:
>  		};
>  
>  		ports {
> -			port@0 {
> +			port {
>  				dp_out: endpoint {
> -					remote-endpoint = <&bridge_in>;
> +					remote-endpoint = <&dp_in>;
> +				};
> +			};
> +		};
> +
> +		panel {
> +			...
> +			port {
> +				dp_in: endpoint {
> +					remote-endpoint = <&dp_out>;
>  				};
>  			};
>  		};
> -- 
> 1.9.1
> 
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

  reply	other threads:[~2015-12-08 15:29 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 [this message]
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
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=20151208152912.GA31338@rob-hp-laptop \
    --to=robh@kernel.org \
    --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=javier@osg.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 \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox