From mboxrd@z Thu Jan  1 00:00:00 1970
From: Shawn Guo <shawn.guo@linaro.org>
Date: Fri, 18 Jan 2019 09:02:32 +0800
Subject: [U-Boot] [PATCH v1 1/1] arm64: dt: poplar: add optee node
In-Reply-To: <1547735860-5298-1-git-send-email-igor.opaniuk@linaro.org>
References: <1547735860-5298-1-git-send-email-igor.opaniuk@linaro.org>
Message-ID: <20190118010230.GB9575@dragon>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Igor,

On Thu, Jan 17, 2019 at 04:37:40PM +0200, Igor Opaniuk wrote:
> As Poplar supports running TF-A with OP-TEE as BL32
> payload, add op-tee node in DT, which enables usage of
> OP-TEE driver (which provides an interface for requesting services
> from OP-TEE).
> 
> Signed-off-by: Igor Opaniuk <igor.opaniuk@linaro.org>

I'm fine with the change.  But rather than pushing changes to u-boot,
can you please push it to kernel repository, which is the DTS upstream
right now?  We sync DTS changes from kernel when needed, and only
maintains u-boot specific changes in hi3798cv200-u-boot.dtsi.

Shawn

> ---
>  arch/arm/dts/hi3798cv200-poplar.dts | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/arch/arm/dts/hi3798cv200-poplar.dts b/arch/arm/dts/hi3798cv200-poplar.dts
> index 964326e..6bc2a23 100644
> --- a/arch/arm/dts/hi3798cv200-poplar.dts
> +++ b/arch/arm/dts/hi3798cv200-poplar.dts
> @@ -28,6 +28,13 @@
>  		reg = <0x0 0x0 0x0 0x80000000>;
>  	};
>  
> +	firmware {
> +		optee {
> +			compatible = "linaro,optee-tz";
> +			method = "smc";
> +		};
> +	};
> +
>  	leds {
>  		compatible = "gpio-leds";
>  
> -- 
> 2.7.4
>