Hi Tom,

On 5/21/24 8:04 PM, Tom Rini wrote:
On Tue, May 21, 2024 at 07:39:53PM +0200, Quentin Schulz wrote:
PX30 Ringneck ran out of memory in the allocation pool of U-Boot proper
pre-reloc. Something needed to be done. Jonas did migrate a few SoCs
already to this common bss+stack addresses so it made sense to follow
the same route for one additional SoC: PX30.

While at it, also fix a few issues related to UART on the PX30 Mini EVB
I could test.

Boot (to U-Boot CLI) tested on PX30 Ringneck and PX30 Mini-EVB.

Thanks to Jonas for hinting where to look at.

Signed-off-by: Quentin Schulz <quentin.sch...@cherry.de>
---
Quentin Schulz (10):
       rockchip: px30: default TPL_SYS_MALLOC_F_LEN to 0x600 on PX30 Kconfig 
level
       rockchip: Use common bss and stack addresses on PX30
       rockchip: ringneck_px30: Use common bss and stack addresses
       rockchip: evb-px30: Use common bss and stack addresses
       rockchip: firefly-px30: Use common bss and stack addresses
       rockchip: odroid-go2: Use common bss and stack addresses
       rockchip: px30-core-*: Use common bss and stack addresses
       rockchip: px30: make UART pinmux accessible to TPL/SPL DTB
       rockchip: evb-px30: do not remove pinctrl nodes from SPL DTB
       rockchip: evb-px30: make UART5 the debug UART

  arch/arm/dts/px30-u-boot.dtsi                 | 16 ++++++++++++++++
  arch/arm/mach-rockchip/px30/Kconfig           |  8 +++++++-
  configs/evb-px30_defconfig                    | 23 +++++------------------
  configs/firefly-px30_defconfig                | 19 +++----------------
  configs/odroid-go2_defconfig                  | 19 +++----------------
  configs/px30-core-ctouch2-of10-px30_defconfig | 19 +++----------------
  configs/px30-core-ctouch2-px30_defconfig      | 19 +++----------------
  configs/px30-core-edimm2.2-px30_defconfig     | 19 +++----------------
  configs/ringneck-px30_defconfig               | 19 +++----------------
  9 files changed, 46 insertions(+), 115 deletions(-)
---
base-commit: a7f0154c412859323396111dd0c09dbafbc153cb
change-id: 20240521-px30-2024-07-rc-7136f6241d29

As I assume we want to fix the platforms for v2024.07, is this the level
of config changes everyone is comfortable with on the platforms? Or
should we just go with the minimum for release and the rest to -next?


I can tell you that Ringneck doesn't work anymore on v2024.07-rc, but the PX30 Mini EVB was still reaching U-Boot CLI without the changes. I don't know about the other boards.

So I could split this into two series, one for master, one for next.

I could suggest: patch 1 to 3 in master, the rest in next. Patch 1 is just moving things around. Patch 2 is doing nothing if nobody uses ROCKCHIP_COMMON_STACK_ADDR and SPL_SHARES_INIT_SP_ADDR (which is the case for px30 boards). Patch 3 is for fixing Ringneck, which I know is broken.

Heiko having access to the Odroid Go2, maybe he could test without my patches and see if it reaches the CLI to know if we should pull it in for master as well.

Does this make sense?

Thanks,
Quentin

Reply via email to