[PATCH 0/2] Fix boot w/ scriptaddr & minor newline issue

[PATCH 0/2] Fix boot w/ scriptaddr & minor newline issue

[Wadim Egorov]

First patch fixes boot issues with script boot method and the
second patch is a minor formatting fix in our board detection code.

@TI: I think the scriptaddr should be also updated for most TI/K3 and
beagle board boards.

Wadim Egorov (2):
  board: phytec: phycore_am6xx: Update scriptaddr
  board: phytec: common: Fix missing newline in error message

 board/phytec/common/phytec_som_detection.c     | 2 +-
 board/phytec/phycore_am62ax/phycore_am62ax.env | 2 +-
 board/phytec/phycore_am62x/phycore_am62x.env   | 2 +-
 board/phytec/phycore_am64x/phycore_am64x.env   | 2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

-- 
2.48.1

[PATCH 1/2] board: phytec: phycore_am6xx: Update scriptaddr

[Wadim Egorov]

After switching our boards to standard boot, we observed that the
kernel hangs when booting with the "script" boot method over the
network.

The original scriptaddr value was copied from ti_common.env and
remained unused for some time. On phycore-am62x and phycore-am62ax,
however, this address conflicts with the current location where
ATF is loaded (CONFIG_K3_ATF_LOAD_ADDR).

Move scriptaddr to 0x89100000, directly after fdtoverlay_addr_r.
The phycore-am64x is not affected by this issue, but we update it
as well to keep all phycore-am6xx boards consistent.

Signed-off-by: Wadim Egorov <w.egorov@phytec.de>
---
 board/phytec/phycore_am62ax/phycore_am62ax.env | 2 +-
 board/phytec/phycore_am62x/phycore_am62x.env   | 2 +-
 board/phytec/phycore_am64x/phycore_am64x.env   | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/board/phytec/phycore_am62ax/phycore_am62ax.env b/board/phytec/phycore_am62ax/phycore_am62ax.env
index ff4ab8c87b8..47e90f1b7c6 100644
--- a/board/phytec/phycore_am62ax/phycore_am62ax.env
+++ b/board/phytec/phycore_am62ax/phycore_am62ax.env
@@ -5,7 +5,7 @@
 
 fdtaddr=0x88000000
 loadaddr=0x82000000
-scriptaddr=0x80000000
+scriptaddr=0x89100000
 fdt_addr_r=0x88000000
 kernel_addr_r=0x82000000
 ramdisk_addr_r=0x88080000
diff --git a/board/phytec/phycore_am62x/phycore_am62x.env b/board/phytec/phycore_am62x/phycore_am62x.env
index ff4ab8c87b8..47e90f1b7c6 100644
--- a/board/phytec/phycore_am62x/phycore_am62x.env
+++ b/board/phytec/phycore_am62x/phycore_am62x.env
@@ -5,7 +5,7 @@
 
 fdtaddr=0x88000000
 loadaddr=0x82000000
-scriptaddr=0x80000000
+scriptaddr=0x89100000
 fdt_addr_r=0x88000000
 kernel_addr_r=0x82000000
 ramdisk_addr_r=0x88080000
diff --git a/board/phytec/phycore_am64x/phycore_am64x.env b/board/phytec/phycore_am64x/phycore_am64x.env
index cbaf45b3ace..f4136ed237d 100644
--- a/board/phytec/phycore_am64x/phycore_am64x.env
+++ b/board/phytec/phycore_am64x/phycore_am64x.env
@@ -4,7 +4,7 @@
 
 fdtaddr=0x88000000
 loadaddr=0x82000000
-scriptaddr=0x80000000
+scriptaddr=0x89100000
 fdt_addr_r=0x88000000
 kernel_addr_r=0x82000000
 ramdisk_addr_r=0x88080000
-- 
2.48.1

Re: [PATCH 1/2] board: phytec: phycore_am6xx: Update scriptaddr

[Anshul Dalal]

On Fri Sep 19, 2025 at 12:09 PM IST, Wadim Egorov wrote:
> After switching our boards to standard boot, we observed that the
> kernel hangs when booting with the "script" boot method over the
> network.
>
> The original scriptaddr value was copied from ti_common.env and
> remained unused for some time. On phycore-am62x and phycore-am62ax,
> however, this address conflicts with the current location where
> ATF is loaded (CONFIG_K3_ATF_LOAD_ADDR).
>

We should look into doing similar for K3 as well, would be useful when
we switch to std boot as well. 

> Move scriptaddr to 0x89100000, directly after fdtoverlay_addr_r.
> The phycore-am64x is not affected by this issue, but we update it
> as well to keep all phycore-am6xx boards consistent.
>
> Signed-off-by: Wadim Egorov <w.egorov@phytec.de>
> ---

Reviewed-by: Anshul Dalal <anshuld@ti.com>

Regards,
Anshul

[PATCH 2/2] board: phytec: common: Fix missing newline in error message

[Wadim Egorov]

The error message in phytec_get_product_name() was missing a newline,
causing log output to be concatenated with subsequent messages. Add
the newline to improve readability.

Signed-off-by: Wadim Egorov <w.egorov@phytec.de>
---
 board/phytec/common/phytec_som_detection.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/board/phytec/common/phytec_som_detection.c b/board/phytec/common/phytec_som_detection.c
index 4d7c9b9f80f..136f4486eb0 100644
--- a/board/phytec/common/phytec_som_detection.c
+++ b/board/phytec/common/phytec_som_detection.c
@@ -309,7 +309,7 @@ static int phytec_get_product_name(struct phytec_eeprom_data *data,
 		som_type = 1;
 		break;
 	default:
-		pr_err("%s: Invalid SOM type: %i", __func__, api2->som_type);
+		pr_err("%s: Invalid SOM type: %i\n", __func__, api2->som_type);
 		return -EINVAL;
 	};
 
-- 
2.48.1

Re: [PATCH 0/2] Fix boot w/ scriptaddr & minor newline issue

[Tom Rini]

On Fri, 19 Sep 2025 08:39:46 +0200, Wadim Egorov wrote:

> First patch fixes boot issues with script boot method and the
> second patch is a minor formatting fix in our board detection code.
> 
> @TI: I think the scriptaddr should be also updated for most TI/K3 and
> beagle board boards.
> 
> Wadim Egorov (2):
>   board: phytec: phycore_am6xx: Update scriptaddr
>   board: phytec: common: Fix missing newline in error message
> 
> [...]

Applied to u-boot/master, thanks!

[1/2] board: phytec: phycore_am6xx: Update scriptaddr
      commit: ac046ad1873cb3d85f4c81c0bed592a4ed7a5903
[2/2] board: phytec: common: Fix missing newline in error message
      commit: 96971e5090cd6975023d0556b3d47b195442902e
-- 
Tom