Re: [PATCH v1] arm: xea: Modify board code to generate single binary u-boot

[Lukasz Majewski <lukma@denx.de>]

[-- Attachment #1: Type: text/plain, Size: 1931 bytes --]

Hi Simon,

> Hi Lukasz,
> 
> On Sat, 23 Oct 2021 at 09:04, Lukasz Majewski <lukma@denx.de> wrote:
> >
> > This change provides the possibility to build XEA (imx287 based)
> > board U-Boot as a single binary (without support for
> > CONFIG_SPL_FRAMEWORK).
> >
> > The generated u-boot.sb can be used in the factory environment to
> > for example perform initial setup or HW testing.
> >
> > It can be used with 'uuu' utility
> > (SDPS: boot -f /srv/tftp/xea/u-boot.sb)
> >
> > In the configs/imx28_xea_defconfig one needs to disable following
> > configs:  
> 
> Something is missing here.

I've posted the v2 of this patch with this information added.

> 
> >
> > The board_init_ll() is used in arch/arm/cpu/arm926ejs/mxs/start.S,
> > which is utilized when CONFIG_SPL_FRAMEWORK is disabled.
> >
> > However, when it is enabled the arch/arm/cpu/arm926ejs/start.S is
> > used, which requires the lowlevel_init() function.
> >
> > Signed-off-by: Lukasz Majewski <lukma@denx.de>
> >
> > ---
> >
> >  board/liebherr/xea/spl_xea.c | 7 +++++++
> >  board/liebherr/xea/xea.c     | 2 +-
> >  2 files changed, 8 insertions(+), 1 deletion(-)  
> 
> Could you add a Kconifig to control this feature and add something to
> the board docs?

This don't need any special Kconfig option. You just undefine the
CONFIG_SPL_FRAMEWORK, and that's all.

As fair as I can tell - the CONFIG_SPL_FRAMEWORK option is used with
for example imx28evk board. The issue here is that imx28 SoC needs
u-boot.sb to boot up when USB is used to debrick it (this is somewhat
the legacy approach with a single binary u-boot).

> 
> Regards,
> Simon




Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma@denx.de

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]