Re: [PATCH v2 05/26] arm: dts: k3-am654: copy bootph properties to a53 dts

2024-01-02 Thread Nishanth Menon
On 11:47-20231229, Bryan Brattlof wrote:
> In order to unify the R5 board dtb file with the Linux board dtb file,
> we will need to copy all bootph-pre-ram properties to the *-u-boot.dtsi
> overlay.
> 
> Tested-by: Tom Rini 
> Signed-off-by: Bryan Brattlof 
> ---
>  arch/arm/dts/k3-am654-base-board-u-boot.dtsi | 160 +++
>  arch/arm/dts/k3-am654-r5-base-board.dts  |  85 +-
>  2 files changed, 162 insertions(+), 83 deletions(-)
> 
> diff --git a/arch/arm/dts/k3-am654-base-board-u-boot.dtsi 
> b/arch/arm/dts/k3-am654-base-board-u-boot.dtsi
> index f29cecf870bcd..4b1e8ce2c920c 100644
> --- a/arch/arm/dts/k3-am654-base-board-u-boot.dtsi
> +++ b/arch/arm/dts/k3-am654-base-board-u-boot.dtsi
> @@ -5,6 +5,166 @@
>  
>  #include "k3-am65x-binman.dtsi"
>  
> +&vtt_supply {
> + bootph-pre-ram;
> +};
> +
> +&cbass_main {
> + bootph-pre-ram;
> +};

I understand this in an intermediate state and the final dts looks fine.
So,
[...]
Reviewed-by: Nishanth Menon 
-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 
849D 1736 249D


[PATCH v2 05/26] arm: dts: k3-am654: copy bootph properties to a53 dts

2023-12-29 Thread Bryan Brattlof
In order to unify the R5 board dtb file with the Linux board dtb file,
we will need to copy all bootph-pre-ram properties to the *-u-boot.dtsi
overlay.

Tested-by: Tom Rini 
Signed-off-by: Bryan Brattlof 
---
 arch/arm/dts/k3-am654-base-board-u-boot.dtsi | 160 +++
 arch/arm/dts/k3-am654-r5-base-board.dts  |  85 +-
 2 files changed, 162 insertions(+), 83 deletions(-)

diff --git a/arch/arm/dts/k3-am654-base-board-u-boot.dtsi 
b/arch/arm/dts/k3-am654-base-board-u-boot.dtsi
index f29cecf870bcd..4b1e8ce2c920c 100644
--- a/arch/arm/dts/k3-am654-base-board-u-boot.dtsi
+++ b/arch/arm/dts/k3-am654-base-board-u-boot.dtsi
@@ -5,6 +5,166 @@
 
 #include "k3-am65x-binman.dtsi"
 
+&vtt_supply {
+   bootph-pre-ram;
+};
+
+&cbass_main {
+   bootph-pre-ram;
+};
+
+&main_navss {
+   bootph-pre-ram;
+};
+
+&cbass_mcu {
+   bootph-pre-ram;
+};
+
+&mcu_navss {
+   bootph-pre-ram;
+};
+
+&mcu_ringacc {
+   bootph-pre-ram;
+};
+
+&mcu_udmap {
+   bootph-pre-ram;
+};
+
+&wkup_gpio0 {
+   bootph-pre-ram;
+};
+
+&secure_proxy_main {
+   bootph-pre-ram;
+};
+
+&cbass_wakeup {
+   bootph-pre-ram;
+
+   chipid@4314 {
+   bootph-pre-ram;
+   };
+};
+
+&dmsc {
+   bootph-pre-ram;
+};
+
+&k3_pds {
+   bootph-pre-ram;
+};
+
+&k3_clks {
+   bootph-pre-ram;
+};
+
+&k3_reset {
+   bootph-pre-ram;
+};
+
+&main_uart0 {
+   bootph-pre-ram;
+};
+
+&wkup_vtm0 {
+   bootph-pre-ram;
+};
+
+&wkup_pmx0 {
+   bootph-pre-ram;
+};
+
+&wkup_uart0_pins_default {
+   bootph-pre-ram;
+};
+
+&wkup_vtt_pins_default {
+   bootph-pre-ram;
+};
+
+&mcu_uart0_pins_default {
+   bootph-pre-ram;
+};
+
+&wkup_i2c0_pins_default {
+   bootph-pre-ram;
+};
+
+&mcu_fss0_ospi0_pins_default {
+   bootph-pre-ram;
+};
+
+&main_pmx0 {
+   bootph-pre-ram;
+};
+
+&main_uart0_pins_default {
+   bootph-pre-ram;
+};
+
+&main_mmc0_pins_default {
+   bootph-pre-ram;
+};
+
+&main_mmc1_pins_default {
+   bootph-pre-ram;
+};
+
+&usb0_pins_default {
+   bootph-pre-ram;
+};
+
+&main_pmx1 {
+   bootph-pre-ram;
+};
+
+&sdhci0 {
+   bootph-pre-ram;
+};
+
+&sdhci1 {
+   bootph-pre-ram;
+};
+
+&wkup_i2c0 {
+   bootph-pre-ram;
+};
+
+&vdd_mpu {
+   bootph-pre-ram;
+};
+
+&ospi0 {
+   bootph-pre-ram;
+
+   flash@0 {
+   bootph-pre-ram;
+   };
+};
+
+&dwc3_0 {
+   bootph-pre-ram;
+};
+
+&usb0_phy {
+   bootph-pre-ram;
+};
+
+&usb0 {
+   bootph-pre-ram;
+};
+
+&scm_conf {
+   bootph-pre-ram;
+};
+
+&fss {
+   bootph-pre-ram;
+};
+
 &pru0_0 {
remoteproc-name = "pru0_0";
 };
diff --git a/arch/arm/dts/k3-am654-r5-base-board.dts 
b/arch/arm/dts/k3-am654-r5-base-board.dts
index d75c7bf3fe662..8f55dab508ee6 100644
--- a/arch/arm/dts/k3-am654-r5-base-board.dts
+++ b/arch/arm/dts/k3-am654-r5-base-board.dts
@@ -53,13 +53,10 @@
regulator-max-microvolt = <330>;
gpios = <&wkup_gpio0 28 GPIO_ACTIVE_HIGH>;
states = <0 0x0 330 0x1>;
-   bootph-pre-ram;
};
 };
 
 &cbass_main {
-   bootph-pre-ram;
-
timer1: timer@4040 {
compatible = "ti,omap5430-timer";
reg = <0x0 0x4040 0x0 0x80>;
@@ -67,15 +64,9 @@
clock-frequency = <2500>;
bootph-all;
};
-
-   main_navss: bus@3080 {
-   bootph-pre-ram;
-   };
 };
 
 &cbass_mcu {
-   bootph-pre-ram;
-
mcu_secproxy: secproxy@2838 {
compatible = "ti,am654-secure-proxy";
reg = <0x0 0x2a38 0x0 0x8>,
@@ -87,8 +78,6 @@
};
 
mcu_navss: bus@2838 {
-   bootph-pre-ram;
-
ringacc@2b80 {
reg =   <0x0 0x2b80 0x0 0x40>,
<0x0 0x2b00 0x0 0x40>,
@@ -96,7 +85,6 @@
<0x0 0x2a50 0x0 0x4>,
<0x0 0x2844 0x0 0x4>;
reg-names = "rt", "fifos", "proxy_gcfg", 
"proxy_target", "cfg";
-   bootph-pre-ram;
ti,dma-ring-reset-quirk;
};
 
@@ -109,34 +97,11 @@
<0x0 0x2840 0x0 0x2000>;
reg-names = "gcfg", "rchan", "rchanrt", "tchan",
"tchanrt", "rflow";
-   bootph-pre-ram;
};
};
 };
 
-&k3_pds {
-   bootph-pre-ram;
-};
-
-&k3_clks {
-   bootph-pre-ram;
-};
-
-&k3_reset {
-   bootph-pre-ram;
-};
-
-&wkup_gpio0 {
-   bootph-pre-ram;
-};
-
-&secure_proxy_main {
-   bootph-pre-ram;
-};
-
 &cbass_wakeup {
-   bootph-pre-ram;
-
sysctrler: sysctrler {
compatible = "ti,am654-system-controller";
mboxes= <&mcu_secproxy 4>, <&mcu_secproxy 5>;
@@ -150,40 +115,29 @