2022-09-08 07:20:59

by Horatiu Vultur

[permalink] [raw]
Subject: [PATCH] ARM: dts: lan966x: disable aes

Disable AES node on lan966x pcb8290, pcb891 and pcb8309 because these
boards have lan966x that uses secure OS which reserves the AES block.
Therefore it can't be exposed to non-secure world.

Signed-off-by: Horatiu Vultur <[email protected]>
---
arch/arm/boot/dts/lan966x-pcb8290.dts | 4 ++++
arch/arm/boot/dts/lan966x-pcb8291.dts | 4 ++++
arch/arm/boot/dts/lan966x-pcb8309.dts | 4 ++++
3 files changed, 12 insertions(+)

diff --git a/arch/arm/boot/dts/lan966x-pcb8290.dts b/arch/arm/boot/dts/lan966x-pcb8290.dts
index 8a304210653a..2ed53da914ac 100644
--- a/arch/arm/boot/dts/lan966x-pcb8290.dts
+++ b/arch/arm/boot/dts/lan966x-pcb8290.dts
@@ -27,6 +27,10 @@ phy_external_reset: phy_external_reset {
};
};

+&aes {
+ status = "disabled"; /* Reserved by secure OS */
+};
+
&gpio {
miim_a_pins: mdio-pins {
/* MDC, MDIO */
diff --git a/arch/arm/boot/dts/lan966x-pcb8291.dts b/arch/arm/boot/dts/lan966x-pcb8291.dts
index 289a31649422..f4f054cdf2a8 100644
--- a/arch/arm/boot/dts/lan966x-pcb8291.dts
+++ b/arch/arm/boot/dts/lan966x-pcb8291.dts
@@ -53,6 +53,10 @@ led-s1-green {
};
};

+&aes {
+ status = "disabled"; /* Reserved by secure OS */
+};
+
&gpio {
fc3_b_pins: fc3-b-pins {
/* RX, TX */
diff --git a/arch/arm/boot/dts/lan966x-pcb8309.dts b/arch/arm/boot/dts/lan966x-pcb8309.dts
index cfd7c8933703..c436cd20d4b4 100644
--- a/arch/arm/boot/dts/lan966x-pcb8309.dts
+++ b/arch/arm/boot/dts/lan966x-pcb8309.dts
@@ -97,6 +97,10 @@ sfp3: sfp3 {
};
};

+&aes {
+ status = "disabled"; /* Reserved by secure OS */
+};
+
&flx3 {
atmel,flexcom-mode = <ATMEL_FLEXCOM_MODE_USART>;
status = "okay";
--
2.33.0


2022-09-13 07:22:54

by Claudiu Beznea

[permalink] [raw]
Subject: Re: [PATCH] ARM: dts: lan966x: disable aes

On 08.09.2022 10:04, Horatiu Vultur wrote:
> Disable AES node on lan966x pcb8290, pcb891 and pcb8309 because these
> boards have lan966x that uses secure OS which reserves the AES block.
> Therefore it can't be exposed to non-secure world.
>
> Signed-off-by: Horatiu Vultur <[email protected]>

Applied to at91-dt, thanks!

> ---
> arch/arm/boot/dts/lan966x-pcb8290.dts | 4 ++++
> arch/arm/boot/dts/lan966x-pcb8291.dts | 4 ++++
> arch/arm/boot/dts/lan966x-pcb8309.dts | 4 ++++
> 3 files changed, 12 insertions(+)
>
> diff --git a/arch/arm/boot/dts/lan966x-pcb8290.dts b/arch/arm/boot/dts/lan966x-pcb8290.dts
> index 8a304210653a..2ed53da914ac 100644
> --- a/arch/arm/boot/dts/lan966x-pcb8290.dts
> +++ b/arch/arm/boot/dts/lan966x-pcb8290.dts
> @@ -27,6 +27,10 @@ phy_external_reset: phy_external_reset {
> };
> };
>
> +&aes {
> + status = "disabled"; /* Reserved by secure OS */
> +};
> +
> &gpio {
> miim_a_pins: mdio-pins {
> /* MDC, MDIO */
> diff --git a/arch/arm/boot/dts/lan966x-pcb8291.dts b/arch/arm/boot/dts/lan966x-pcb8291.dts
> index 289a31649422..f4f054cdf2a8 100644
> --- a/arch/arm/boot/dts/lan966x-pcb8291.dts
> +++ b/arch/arm/boot/dts/lan966x-pcb8291.dts
> @@ -53,6 +53,10 @@ led-s1-green {
> };
> };
>
> +&aes {
> + status = "disabled"; /* Reserved by secure OS */
> +};
> +
> &gpio {
> fc3_b_pins: fc3-b-pins {
> /* RX, TX */
> diff --git a/arch/arm/boot/dts/lan966x-pcb8309.dts b/arch/arm/boot/dts/lan966x-pcb8309.dts
> index cfd7c8933703..c436cd20d4b4 100644
> --- a/arch/arm/boot/dts/lan966x-pcb8309.dts
> +++ b/arch/arm/boot/dts/lan966x-pcb8309.dts
> @@ -97,6 +97,10 @@ sfp3: sfp3 {
> };
> };
>
> +&aes {
> + status = "disabled"; /* Reserved by secure OS */
> +};
> +
> &flx3 {
> atmel,flexcom-mode = <ATMEL_FLEXCOM_MODE_USART>;
> status = "okay";