On Sat, Sep 12, 2026 at 08:30:35AM +0200, leow149 wrote:
> Board bring-up for the TrimUI Smart Pro handheld (Allwinner A133 Plus):
> 
> - New board DTS (sun50i-a133-trimui-smartpro.dts) and defconfig.
> - SUNXI_BL31_BASE: A133 selects SUN50I_GEN_H6 (shared H6-family
>   peripheral IP) but its TF-A platform.mk sets SUNXI_BL31_IN_DRAM,
>   unlike other SUN50I_GEN_H6 members which run BL31 from SRAM. Without
>   its own case here, BL31 silently falls through to the wrong load
>   address and crashes on real hardware.
> 
> Boots mainline U-Boot end to end on real hardware (SPL -> BL31 ->
> U-Boot -> distro-boot).
> 
> Signed-off-by: leow149 <[email protected]>
> ---
>  arch/arm/mach-sunxi/Kconfig                   |   7 +
>  configs/trimui-smartpro_defconfig             |  64 ++++
>  .../allwinner/sun50i-a133-trimui-smartpro.dts | 285 ++++++++++++++++++
>  3 files changed, 356 insertions(+)
>  create mode 100644 configs/trimui-smartpro_defconfig
>  create mode 100644 
> dts/upstream/src/arm64/allwinner/sun50i-a133-trimui-smartpro.dts

Hi. Changes under dts/upstream/ must be done following
https://docs.u-boot-project.org/en/latest/develop/devicetree/control.html#resyncing-with-devicetree-rebasing
and so as their own standalone patch, thanks. Next:

> diff --git a/arch/arm/mach-sunxi/Kconfig b/arch/arm/mach-sunxi/Kconfig
> index b0c5d3cd..c12f9b4c 100644
> --- a/arch/arm/mach-sunxi/Kconfig
> +++ b/arch/arm/mach-sunxi/Kconfig
> @@ -268,6 +268,13 @@ config SUNXI_BL31_BASE
>       hex
>       default 0x00044000 if MACH_SUN50I || MACH_SUN50I_H5
>       default 0x40000000 if MACH_SUN50I_H616
> +     # A133 selects SUN50I_GEN_H6 (shared H6-family peripheral IP) but its
> +     # TF-A platform.mk sets SUNXI_BL31_IN_DRAM := 1, unlike other
> +     # SUN50I_GEN_H6 members which run BL31 from SRAM at 0x00104000 -- needs
> +     # its own case here or it silently falls through to the wrong address
> +     # and BL31 crashes on real hardware, jumping into garbage near its
> +     # actual linked base instead.

This kind of information belongs somewhere in
doc/board/allwinner/sunxi.rst I believe and not a comment.

[snip]
> diff --git a/configs/trimui-smartpro_defconfig 
> b/configs/trimui-smartpro_defconfig
> new file mode 100644
> index 00000000..64d26459
> --- /dev/null
> +++ b/configs/trimui-smartpro_defconfig
> @@ -0,0 +1,64 @@
> +# TrimUI Smart Pro (Allwinner A133 Plus, sun50iw10p1) — out-of-tree hobbyist 
> port

This file is auto-generated from "make savedefconfig" and re-generated
as needed, so all of these comments will be lost. Any information that
needs to be kept and available to users/developers should also go in the
doc I mentioned above. Thanks.

-- 
Tom

Attachment: signature.asc
Description: PGP signature

Reply via email to