Re: [PATCH] arm: dts: k3-am65-iot2050: Fix boot

[Tom Rini]

On Thu, Oct 05, 2023 at 06:37:17AM +0200, Jan Kiszka wrote:

> From: Jan Kiszka 
> 
> Since commit [1] A53 u-boot proper is broken. This is because nodes
> marked as 'bootph-pre-ram' are not available at u-boot proper before
> relocation.
> 
> To fix this we mark all nodes in u-boot.dtsi as 'bootph-all'.
> 
> [1] 9e644284ab812 ("dm: core: Report bootph-pre-ram/sram node as pre-reloc 
> after relocation")
> 
> Signed-off-by: Jan Kiszka 
> Reviewed-by: Roger Quadros 
> Reviewed-by: Nishanth Menon 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: PGP signature

Re: [PATCH] arm: dts: k3-am65-iot2050: Fix boot

[Nishanth Menon]

On 06:37-20231005, Jan Kiszka wrote:
> From: Jan Kiszka 
> 
> Since commit [1] A53 u-boot proper is broken. This is because nodes
> marked as 'bootph-pre-ram' are not available at u-boot proper before
> relocation.
> 
> To fix this we mark all nodes in u-boot.dtsi as 'bootph-all'.
> 
> [1] 9e644284ab812 ("dm: core: Report bootph-pre-ram/sram node as pre-reloc 
> after relocation")
> 
> Signed-off-by: Jan Kiszka 

Reviewed-by: Nishanth Menon 

> ---
>  .../dts/k3-am65-iot2050-common-u-boot.dtsi| 44 +--
>  1 file changed, 22 insertions(+), 22 deletions(-)
> 
> diff --git a/arch/arm/dts/k3-am65-iot2050-common-u-boot.dtsi 
> b/arch/arm/dts/k3-am65-iot2050-common-u-boot.dtsi
> index 9fd64809a63..30fc7a78d89 100644
> --- a/arch/arm/dts/k3-am65-iot2050-common-u-boot.dtsi
> +++ b/arch/arm/dts/k3-am65-iot2050-common-u-boot.dtsi
> @@ -37,18 +37,18 @@
>   };
>  
>   leds {
> - bootph-pre-ram;
> + bootph-all;
>   status-led-red {
> - bootph-pre-ram;
> + bootph-all;
>   };
>   status-led-green {
> - bootph-pre-ram;
> + bootph-all;
>   };
>   };
>  };
>  
>  &cbass_mcu {
> - bootph-pre-ram;
> + bootph-all;
>  
>   mcu_navss: bus@2838 {
>   ringacc@2b80 {
> @@ -75,70 +75,70 @@
>  };
>  
>  &cbass_wakeup {
> - bootph-pre-ram;
> + bootph-all;
>  };
>  
>  &cbass_main {
> - bootph-pre-ram;
> + bootph-all;
>   main_navss: bus@3080 {
> - bootph-pre-ram;
> + bootph-all;
>   };
>  };
>  
>  &wkup_pmx0 {
> - bootph-pre-ram;
> + bootph-all;
>   mcu-fss0-ospi0-pins-default {
> - bootph-pre-ram;
> + bootph-all;
>   };
>  };
>  
>  &main_pmx0 {
> - bootph-pre-ram;
> + bootph-all;
>   main-uart1-pins-default {
> - bootph-pre-ram;
> + bootph-all;
>   };
>  };
>  
>  &main_uart1 {
> - bootph-pre-ram;
> + bootph-all;
>   current-speed = <115200>;
>  };
>  
>  &wkup_gpio0 {
> - bootph-pre-ram;
> + bootph-all;
>  };
>  
>  &ospi0 {
> - bootph-pre-ram;
> + bootph-all;
>   flash@0 {
> - bootph-pre-ram;
> + bootph-all;
>   };
>  };
>  
>  &secure_proxy_main {
> - bootph-pre-ram;
> + bootph-all;
>  };
>  
>  &dmsc {
> - bootph-pre-ram;
> + bootph-all;
>   k3_sysreset: sysreset-controller {
>   compatible = "ti,sci-sysreset";
> - bootph-pre-ram;
> + bootph-all;
>   };
>  };
>  
>  &k3_pds {
> - bootph-pre-ram;
> + bootph-all;
>  };
>  
>  &k3_clks {
> - bootph-pre-ram;
> + bootph-all;
>  };
>  
>  &k3_reset {
> - bootph-pre-ram;
> + bootph-all;
>  };
>  
>  &fss {
> - bootph-pre-ram;
> + bootph-all;
>  };
> -- 
> 2.35.3

-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 
849D 1736 249D

Re: [PATCH] arm: dts: k3-am65-iot2050: Fix boot

[Roger Quadros]

On 05/10/2023 07:37, Jan Kiszka wrote:
> From: Jan Kiszka 
> 
> Since commit [1] A53 u-boot proper is broken. This is because nodes
> marked as 'bootph-pre-ram' are not available at u-boot proper before
> relocation.
> 
> To fix this we mark all nodes in u-boot.dtsi as 'bootph-all'.
> 
> [1] 9e644284ab812 ("dm: core: Report bootph-pre-ram/sram node as pre-reloc 
> after relocation")
> 
> Signed-off-by: Jan Kiszka 

Reviewed-by: Roger Quadros 

[PATCH] arm: dts: k3-am65-iot2050: Fix boot

[Jan Kiszka]

From: Jan Kiszka 

Since commit [1] A53 u-boot proper is broken. This is because nodes
marked as 'bootph-pre-ram' are not available at u-boot proper before
relocation.

To fix this we mark all nodes in u-boot.dtsi as 'bootph-all'.

[1] 9e644284ab812 ("dm: core: Report bootph-pre-ram/sram node as pre-reloc 
after relocation")

Signed-off-by: Jan Kiszka 
---
 .../dts/k3-am65-iot2050-common-u-boot.dtsi| 44 +--
 1 file changed, 22 insertions(+), 22 deletions(-)

diff --git a/arch/arm/dts/k3-am65-iot2050-common-u-boot.dtsi 
b/arch/arm/dts/k3-am65-iot2050-common-u-boot.dtsi
index 9fd64809a63..30fc7a78d89 100644
--- a/arch/arm/dts/k3-am65-iot2050-common-u-boot.dtsi
+++ b/arch/arm/dts/k3-am65-iot2050-common-u-boot.dtsi
@@ -37,18 +37,18 @@
};
 
leds {
-   bootph-pre-ram;
+   bootph-all;
status-led-red {
-   bootph-pre-ram;
+   bootph-all;
};
status-led-green {
-   bootph-pre-ram;
+   bootph-all;
};
};
 };
 
 &cbass_mcu {
-   bootph-pre-ram;
+   bootph-all;
 
mcu_navss: bus@2838 {
ringacc@2b80 {
@@ -75,70 +75,70 @@
 };
 
 &cbass_wakeup {
-   bootph-pre-ram;
+   bootph-all;
 };
 
 &cbass_main {
-   bootph-pre-ram;
+   bootph-all;
main_navss: bus@3080 {
-   bootph-pre-ram;
+   bootph-all;
};
 };
 
 &wkup_pmx0 {
-   bootph-pre-ram;
+   bootph-all;
mcu-fss0-ospi0-pins-default {
-   bootph-pre-ram;
+   bootph-all;
};
 };
 
 &main_pmx0 {
-   bootph-pre-ram;
+   bootph-all;
main-uart1-pins-default {
-   bootph-pre-ram;
+   bootph-all;
};
 };
 
 &main_uart1 {
-   bootph-pre-ram;
+   bootph-all;
current-speed = <115200>;
 };
 
 &wkup_gpio0 {
-   bootph-pre-ram;
+   bootph-all;
 };
 
 &ospi0 {
-   bootph-pre-ram;
+   bootph-all;
flash@0 {
-   bootph-pre-ram;
+   bootph-all;
};
 };
 
 &secure_proxy_main {
-   bootph-pre-ram;
+   bootph-all;
 };
 
 &dmsc {
-   bootph-pre-ram;
+   bootph-all;
k3_sysreset: sysreset-controller {
compatible = "ti,sci-sysreset";
-   bootph-pre-ram;
+   bootph-all;
};
 };
 
 &k3_pds {
-   bootph-pre-ram;
+   bootph-all;
 };
 
 &k3_clks {
-   bootph-pre-ram;
+   bootph-all;
 };
 
 &k3_reset {
-   bootph-pre-ram;
+   bootph-all;
 };
 
 &fss {
-   bootph-pre-ram;
+   bootph-all;
 };
-- 
2.35.3