Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Thu, 10 Jun 2021 04:35:02 +0300
From:   Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To:     Michal Simek <michal.simek@xilinx.com>
Cc:     linux-kernel@vger.kernel.org, monstr@monstr.eu, git@xilinx.com,
        Viresh Kumar <viresh.kumar@linaro.org>,
        Quanyang Wang <quanyang.wang@windriver.com>,
        Rob Herring <robh+dt@kernel.org>, devicetree@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 21/31] arm64: zynqmp: Move DP nodes to the end of file on
 zcu106
Message-ID: <YMFsRjXuYm0KMRMD@pendragon.ideasonboard.com>
References: <cover.1623239033.git.michal.simek@xilinx.com>
 <4d1c79f6f184367687f61608bb8e0f18d9121802.1623239033.git.michal.simek@xilinx.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
In-Reply-To: <4d1c79f6f184367687f61608bb8e0f18d9121802.1623239033.git.michal.simek@xilinx.com>
Precedence: bulk

Hi Michal,

Thank you for the patch.

On Wed, Jun 09, 2021 at 01:44:57PM +0200, Michal Simek wrote:
> This location is used by others DTs files that's why this move.

I like alphabetical order :-)

> Signed-off-by: Michal Simek <michal.simek@xilinx.com>

Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

> ---
> 
>  .../boot/dts/xilinx/zynqmp-zcu106-revA.dts    | 22 +++++++++----------
>  1 file changed, 11 insertions(+), 11 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/xilinx/zynqmp-zcu106-revA.dts b/arch/arm64/boot/dts/xilinx/zynqmp-zcu106-revA.dts
> index dbb8bfbb5c7f..4a0f3370bf7f 100644
> --- a/arch/arm64/boot/dts/xilinx/zynqmp-zcu106-revA.dts
> +++ b/arch/arm64/boot/dts/xilinx/zynqmp-zcu106-revA.dts
> @@ -160,17 +160,6 @@ &dcc {
>  	status = "okay";
>  };
>  
> -&zynqmp_dpdma {
> -	status = "okay";
> -};
> -
> -&zynqmp_dpsub {
> -	status = "okay";
> -	phy-names = "dp-phy0", "dp-phy1";
> -	phys = <&psgtr 1 PHY_TYPE_DP 0 3>,
> -	       <&psgtr 0 PHY_TYPE_DP 1 3>;
> -};
> -
>  /* fpd_dma clk 667MHz, lpd_dma 500MHz */
>  &fpd_dma_chan1 {
>  	status = "okay";
> @@ -994,3 +983,14 @@ &usb0 {
>  &watchdog0 {
>  	status = "okay";
>  };
> +
> +&zynqmp_dpdma {
> +	status = "okay";
> +};
> +
> +&zynqmp_dpsub {
> +	status = "okay";
> +	phy-names = "dp-phy0", "dp-phy1";
> +	phys = <&psgtr 1 PHY_TYPE_DP 0 3>,
> +	       <&psgtr 0 PHY_TYPE_DP 1 3>;
> +};

-- 
Regards,

Laurent Pinchart