diff mbox series

arm64: dts: ti: k3-am62-main: Remove power-domains from crypto node

Message ID 20230417133308.1990057-1-kamlesh@ti.com
State Accepted
Commit b573bf35ef3f113c1717fa22cefdfdfbb83aec70
Headers show
Series arm64: dts: ti: k3-am62-main: Remove power-domains from crypto node | expand

Commit Message

Kamlesh Gurudasani April 17, 2023, 1:33 p.m. UTC
From: Kamlesh Gurudasani <kamlesh@ti.com>

With latest firmware update, we can no longer control power of SA3UL from
main domain.

Remove power-domains property from the crypto node.

Signed-off-by: Kamlesh Gurudasani <kamlesh@ti.com>
---
 arch/arm64/boot/dts/ti/k3-am62-main.dtsi | 1 -
 1 file changed, 1 deletion(-)

Comments

Nishanth Menon April 17, 2023, 1:42 p.m. UTC | #1
On 19:03-20230417, kamlesh@ti.com wrote:
> From: Kamlesh Gurudasani <kamlesh@ti.com>
> 
> With latest firmware update, we can no longer control power of SA3UL from
> main domain.

How is this backward compatible?

> 
> Remove power-domains property from the crypto node.
> 
> Signed-off-by: Kamlesh Gurudasani <kamlesh@ti.com>
> ---
>  arch/arm64/boot/dts/ti/k3-am62-main.dtsi | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/arch/arm64/boot/dts/ti/k3-am62-main.dtsi b/arch/arm64/boot/dts/ti/k3-am62-main.dtsi
> index b3e4857bbbe4..18a6e9ffaf58 100644
> --- a/arch/arm64/boot/dts/ti/k3-am62-main.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-am62-main.dtsi
> @@ -174,7 +174,6 @@ k3_reset: reset-controller {
>  	crypto: crypto@40900000 {
>  		compatible = "ti,am62-sa3ul";
>  		reg = <0x00 0x40900000 0x00 0x1200>;
> -		power-domains = <&k3_pds 70 TI_SCI_PD_SHARED>;
>  		#address-cells = <2>;
>  		#size-cells = <2>;
>  		ranges = <0x00 0x40900000 0x00 0x40900000 0x00 0x30000>;
> -- 
> 2.34.1
>
Vignesh Raghavendra May 27, 2023, 6:31 a.m. UTC | #2
On 25/05/23 8:14 pm, Francesco Dolcini wrote:
> On Mon, Apr 17, 2023 at 08:42:41AM -0500, Nishanth Menon wrote:
>> On 19:03-20230417, kamlesh@ti.com wrote:
>>> From: Kamlesh Gurudasani <kamlesh@ti.com>
>>>
>>> With latest firmware update, we can no longer control power of SA3UL from
>>> main domain.
>>
>> How is this backward compatible?
> 
> I just stumbled across this commit since I had an issue with crypto@40900000
> not being probed on Verdin AM62.
> 
> Not sure how this is supposed to work about backward/forward
> compatibility, however this is needed for having it working with the
> latest firmware.
>
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/ti/k3-am62-main.dtsi b/arch/arm64/boot/dts/ti/k3-am62-main.dtsi
index b3e4857bbbe4..18a6e9ffaf58 100644
--- a/arch/arm64/boot/dts/ti/k3-am62-main.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am62-main.dtsi
@@ -174,7 +174,6 @@  k3_reset: reset-controller {
 	crypto: crypto@40900000 {
 		compatible = "ti,am62-sa3ul";
 		reg = <0x00 0x40900000 0x00 0x1200>;
-		power-domains = <&k3_pds 70 TI_SCI_PD_SHARED>;
 		#address-cells = <2>;
 		#size-cells = <2>;
 		ranges = <0x00 0x40900000 0x00 0x40900000 0x00 0x30000>;