The RPC HF might be locked by ATF, and any access to its register space would result in complete hang. Disable the RPC HF by default. The ATF should be patched to set RPC node status = "okay" in the DT fragment it passes to U-Boot in case the RPC HF access is unlocked, and that way U-Boot could access the RPC HF safely.
Signed-off-by: Marek Vasut marek.vasut+renesas@gmail.com --- ATF patch: https://review.trustedfirmware.org/c/TF-A/trusted-firmware-a/+/10622 --- arch/arm/dts/r8a77950-salvator-x-u-boot.dts | 2 +- arch/arm/dts/r8a77950-ulcb-u-boot.dts | 2 +- arch/arm/dts/r8a77960-salvator-x-u-boot.dts | 2 +- arch/arm/dts/r8a77960-ulcb-u-boot.dts | 2 +- arch/arm/dts/r8a77965-salvator-x-u-boot.dts | 2 +- arch/arm/dts/r8a77965-ulcb-u-boot.dts | 2 +- arch/arm/dts/r8a77990-ebisu-u-boot.dts | 2 +- arch/arm/dts/r8a77995-draak-u-boot.dts | 2 +- 8 files changed, 8 insertions(+), 8 deletions(-)
diff --git a/arch/arm/dts/r8a77950-salvator-x-u-boot.dts b/arch/arm/dts/r8a77950-salvator-x-u-boot.dts index bb83bc1d7b..d94ad91973 100644 --- a/arch/arm/dts/r8a77950-salvator-x-u-boot.dts +++ b/arch/arm/dts/r8a77950-salvator-x-u-boot.dts @@ -27,7 +27,7 @@
&rpc { reg = <0 0xee200000 0 0x100>, <0 0x08000000 0 0x04000000>; - status = "okay"; + status = "disabled"; };
&sdhi0 { diff --git a/arch/arm/dts/r8a77950-ulcb-u-boot.dts b/arch/arm/dts/r8a77950-ulcb-u-boot.dts index 16d9e38b70..ff00ccdb5b 100644 --- a/arch/arm/dts/r8a77950-ulcb-u-boot.dts +++ b/arch/arm/dts/r8a77950-ulcb-u-boot.dts @@ -39,7 +39,7 @@
&rpc { reg = <0 0xee200000 0 0x100>, <0 0x08000000 0 0x04000000>; - status = "okay"; + status = "disabled"; };
&sdhi0 { diff --git a/arch/arm/dts/r8a77960-salvator-x-u-boot.dts b/arch/arm/dts/r8a77960-salvator-x-u-boot.dts index dda3567e9e..79a54f38c1 100644 --- a/arch/arm/dts/r8a77960-salvator-x-u-boot.dts +++ b/arch/arm/dts/r8a77960-salvator-x-u-boot.dts @@ -27,7 +27,7 @@
&rpc { reg = <0 0xee200000 0 0x100>, <0 0x08000000 0 0x04000000>; - status = "okay"; + status = "disabled"; };
&sdhi0 { diff --git a/arch/arm/dts/r8a77960-ulcb-u-boot.dts b/arch/arm/dts/r8a77960-ulcb-u-boot.dts index e4f77ae262..1e9e8b87d5 100644 --- a/arch/arm/dts/r8a77960-ulcb-u-boot.dts +++ b/arch/arm/dts/r8a77960-ulcb-u-boot.dts @@ -39,7 +39,7 @@
&rpc { reg = <0 0xee200000 0 0x100>, <0 0x08000000 0 0x04000000>; - status = "okay"; + status = "disabled"; };
&sdhi0 { diff --git a/arch/arm/dts/r8a77965-salvator-x-u-boot.dts b/arch/arm/dts/r8a77965-salvator-x-u-boot.dts index e651e889bc..4272ecc110 100644 --- a/arch/arm/dts/r8a77965-salvator-x-u-boot.dts +++ b/arch/arm/dts/r8a77965-salvator-x-u-boot.dts @@ -27,7 +27,7 @@
&rpc { reg = <0 0xee200000 0 0x100>, <0 0x08000000 0 0x04000000>; - status = "okay"; + status = "disabled"; };
&sdhi0 { diff --git a/arch/arm/dts/r8a77965-ulcb-u-boot.dts b/arch/arm/dts/r8a77965-ulcb-u-boot.dts index 3574d6da6f..d9c680b171 100644 --- a/arch/arm/dts/r8a77965-ulcb-u-boot.dts +++ b/arch/arm/dts/r8a77965-ulcb-u-boot.dts @@ -39,7 +39,7 @@
&rpc { reg = <0 0xee200000 0 0x100>, <0 0x08000000 0 0x04000000>; - status = "okay"; + status = "disabled"; };
&sdhi0 { diff --git a/arch/arm/dts/r8a77990-ebisu-u-boot.dts b/arch/arm/dts/r8a77990-ebisu-u-boot.dts index 7022a2ba84..55699bafc4 100644 --- a/arch/arm/dts/r8a77990-ebisu-u-boot.dts +++ b/arch/arm/dts/r8a77990-ebisu-u-boot.dts @@ -43,7 +43,7 @@
&rpc { reg = <0 0xee200000 0 0x100>, <0 0x08000000 0 0x04000000>; - status = "okay"; + status = "disabled"; };
&sdhi0 { diff --git a/arch/arm/dts/r8a77995-draak-u-boot.dts b/arch/arm/dts/r8a77995-draak-u-boot.dts index 10fb9cb3b6..0ea2570c1d 100644 --- a/arch/arm/dts/r8a77995-draak-u-boot.dts +++ b/arch/arm/dts/r8a77995-draak-u-boot.dts @@ -10,5 +10,5 @@
&rpc { reg = <0 0xee200000 0 0x100>, <0 0x08000000 0 0x04000000>; - status = "okay"; + status = "disabled"; };