Re: [meta-rockchip] [PATCH v3] rk3288: use u-boot.img as bootloader image

[Trevor Woerner <twoerner@gmail.com>]

On Fri 2025-11-14 @ 09:53:08 AM, Trevor Woerner wrote:
> My build testing shows that this patch fails for the rock2-square
> MACHINE. Are others seeing this as well? In the v1 patch Quentin noticed
> this and mentioned building for all rk3288 devices, was this tested?

Actually, never mind, the fix is so small and obvious, I'll just roll it
into the patch and commit it.

> On Sun 2025-11-09 @ 01:56:41 AM, Peter Shkenev wrote:
> > Currently, u-boot.bin is used for supported rk3288-based boards. But on
> > some boards, such as tinker-board, it can not be loaded, since
> > u-boot.bin is raw and those boards have raw image support
> > (CONFIG_SPL_RAW_IMAGE_SUPPORT) disabled in favour of FIT images.
> > 
> > Use u-boot.img which is FIT when FIT support is enabled, and raw
> > otherwise.
> > 
> > Also add a note that u-boot.itb should be used if one wantes to utilize
> > OP-TEE.
> > 
> > Suggested-by: Quentin Schulz <quentin.schulz@cherry.de>
> > 
> > Signed-off-by: Peter Shkenev <mustela@erminea.space>
> > ---
> >  conf/machine/include/rk3288.inc | 3 ++-
> >  1 file changed, 2 insertions(+), 1 deletion(-)
> > 
> > diff --git a/conf/machine/include/rk3288.inc b/conf/machine/include/rk3288.inc
> > index a648b44..00cc04b 100644
> > --- a/conf/machine/include/rk3288.inc
> > +++ b/conf/machine/include/rk3288.inc
> > @@ -13,6 +13,7 @@ SERIAL_CONSOLES = "115200;ttyS2"
> >  KBUILD_DEFCONFIG ?= "multi_v7_defconfig"
> >  KERNEL_FEATURES:append:rk3288 = " bsp/rockchip/remove-non-rockchip-arch-arm.scc"
> >  
> > -UBOOT_SUFFIX ?= "bin"
> > +# Set UBOOT_SUFFIX to "itb" if you are going to use OP-TEE
> > +UBOOT_SUFFIX ?= "img"
> >  
> >  ENABLE_STATELESS_VPU_GST ?= "1"
> > -- 
> > 2.51.1
> >