Linux-ARM-Kernel Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
To: kostap@marvell.com, linux-arm-kernel@lists.infradead.org,
	devicetree@vger.kernel.org, linux-kernel@vger.kernel.org
Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
	Rob Herring <robh+dt@kernel.org>,
	Stefan Chulski <stefanc@marvell.com>,
	Nadav Haklai <nadavh@marvell.com>, Andrew Lunn <andrew@lunn.ch>,
	Gregory Clement <gregory.clement@bootlin.com>,
	Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>
Subject: Re: [PATCH] arch/arm64/boot/dts/marvell: add optee FW definitions
Date: Wed, 9 Nov 2022 15:34:45 +0100	[thread overview]
Message-ID: <f079ebb0-2161-61bb-818e-840b74ba1821@linaro.org> (raw)
In-Reply-To: <20221109143054.1440158-1-kostap@marvell.com>

On 09/11/2022 15:30, kostap@marvell.com wrote:
> From: Konstantin Porotchkin <kostap@marvell.com>
> 
> Add reserved memory and ARM firmware definitions for optee
> memory region in Marvell Armada SoCs to avoid protected memory
> access.
> 
> Signed-off-by: Konstantin Porotchkin <kostap@marvell.com>

Use subject prefixes matching the subsystem (git log --oneline -- ...).

> Cc: Andrew Lunn <andrew@lunn.ch>
> Cc: Gregory Clement <gregory.clement@bootlin.com>
> Cc: Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>
> ---
>  arch/arm64/boot/dts/marvell/armada-37xx.dtsi  | 11 +++++++++++
>  arch/arm64/boot/dts/marvell/armada-ap80x.dtsi | 13 +++++++++++++
>  2 files changed, 24 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/marvell/armada-37xx.dtsi b/arch/arm64/boot/dts/marvell/armada-37xx.dtsi
> index df152c72276b..c7a5a25826ea 100644
> --- a/arch/arm64/boot/dts/marvell/armada-37xx.dtsi
> +++ b/arch/arm64/boot/dts/marvell/armada-37xx.dtsi
> @@ -35,6 +35,11 @@ psci-area@4000000 {
>  			reg = <0 0x4000000 0 0x200000>;
>  			no-map;
>  		};
> +
> +		tee@4400000 {
> +			reg = <0 0x4400000 0 0x1000000>;
> +			no-map;
> +		};
>  	};
>  
>  	cpus {
> @@ -531,5 +536,11 @@ armada-3700-rwtm {
>  			mboxes = <&rwtm 0>;
>  			status = "okay";
>  		};
> +
> +		optee {
> +			compatible = "linaro,optee-tz";
> +			method = "smc";
> +			status = "okay";

Why? It's by default.

> +		 };
>  	};
>  };
> diff --git a/arch/arm64/boot/dts/marvell/armada-ap80x.dtsi b/arch/arm64/boot/dts/marvell/armada-ap80x.dtsi
> index a06a0a889c43..abf373b39463 100644
> --- a/arch/arm64/boot/dts/marvell/armada-ap80x.dtsi
> +++ b/arch/arm64/boot/dts/marvell/armada-ap80x.dtsi
> @@ -41,6 +41,11 @@ psci-area@4000000 {
>  			reg = <0x0 0x4000000 0x0 0x200000>;
>  			no-map;
>  		};
> +
> +		tee@4400000 {
> +			reg = <0 0x4400000 0 0x1000000>;
> +			no-map;
> +		};
>  	};
>  
>  	AP_NAME {
> @@ -461,4 +466,12 @@ map3_emerg: map3-emerg {
>  			};
>  		};
>  	};
> +
> +	firmware {
> +		optee {
> +			compatible = "linaro,optee-tz";
> +			method = "smc";
> +			status = "okay";

Drop.

> +		 };
> +	};
>  };

Best regards,
Krzysztof


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

      reply	other threads:[~2022-11-09 14:36 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-11-09 14:30 [PATCH] arch/arm64/boot/dts/marvell: add optee FW definitions kostap
2022-11-09 14:34 ` Krzysztof Kozlowski [this message]

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=f079ebb0-2161-61bb-818e-840b74ba1821@linaro.org \
    --to=krzysztof.kozlowski@linaro.org \
    --cc=andrew@lunn.ch \
    --cc=devicetree@vger.kernel.org \
    --cc=gregory.clement@bootlin.com \
    --cc=kostap@marvell.com \
    --cc=krzysztof.kozlowski+dt@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=nadavh@marvell.com \
    --cc=robh+dt@kernel.org \
    --cc=sebastian.hesselbarth@gmail.com \
    --cc=stefanc@marvell.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