[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <29cb4426-a15e-7186-1713-427f749bc187@linaro.org>
Date: Wed, 12 Jul 2023 21:07:32 +0200
From: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
To: Pankaj Gupta <pankaj.gupta@....com>, shawnguo@...nel.org,
s.hauer@...gutronix.de, kernel@...gutronix.de, clin@...e.com,
conor+dt@...nel.org, pierre.gondois@....com, ping.bai@....com,
xiaoning.wang@....com, wei.fang@....com, peng.fan@....com,
haibo.chen@....com, festevam@...il.com, linux-imx@....com,
davem@...emloft.net, robh+dt@...nel.org,
krzysztof.kozlowski+dt@...aro.org,
linux-arm-kernel@...ts.infradead.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org, gaurav.jain@....com,
alexander.stein@...tq-group.com, sahil.malhotra@....com,
aisheng.dong@....com, V.Sethi@....com
Subject: Re: [PATCH v4 4/7] arm64: dts: imx93-11x11-evk: added nxp secure
enclave fw
On 12/07/2023 14:12, Pankaj Gupta wrote:
> Added support for NXP secure enclave called EdgeLock Enclave
> firmware (se-fw) for imx93-11x11-evk.
>
> Signed-off-by: Pankaj Gupta <pankaj.gupta@....com>
> ---
> arch/arm64/boot/dts/freescale/imx93.dtsi | 11 ++++++++++-
> 1 file changed, 10 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/freescale/imx93.dtsi b/arch/arm64/boot/dts/freescale/imx93.dtsi
> index 8643612ace8c..2b0f901d2709 100644
> --- a/arch/arm64/boot/dts/freescale/imx93.dtsi
> +++ b/arch/arm64/boot/dts/freescale/imx93.dtsi
> @@ -1,6 +1,6 @@
> // SPDX-License-Identifier: (GPL-2.0+ OR MIT)
> /*
> - * Copyright 2022 NXP
> + * Copyright 2022-2023 NXP
> */
>
> #include <dt-bindings/clock/imx93-clock.h>
> @@ -863,5 +863,14 @@ ddr-pmu@...00dc0 {
> reg = <0x4e300dc0 0x200>;
> interrupts = <GIC_SPI 90 IRQ_TYPE_LEVEL_HIGH>;
> };
> +
> + ele_fw: se-fw {
> + compatible = "fsl,imx93-ele";
> + mboxes = <&s4muap 0 0 &s4muap 1 0>;
This should be two entries.
> + mbox-names = "tx", "rx";
> + fsl,mu-did = <3>;
> + fsl,mu-id = <2>;
Drop both. Since you put it into the DTSI, it means it is compatible
specific.
> + status = "okay";
Drop.
Best regards,
Krzysztof
Powered by blists - more mailing lists