Re: [PATCH v3 10/19] LoongArch: Boot Image bits

On Sat, Aug 22, 2026 at 01:03:16PM -0600, Tom Rini wrote:
> On Sat, Aug 22, 2026 at 05:52:14PM +0000, Yao Zi wrote:
> > On Fri, Aug 21, 2026 at 10:20:48AM -0600, Tom Rini wrote:
> > > On Fri, Aug 21, 2026 at 08:42:10AM +0000, Yao Zi wrote:
> > > 
> > > > From: Jiaxun Yang <[email protected]>
> > > > 
> > > > Implement loading and booting functions for LoongArch
> > > > standard kernel image as per spec.
> > > > 
> > > > LoongArch kernel do expect us to fake a efi systemtable
> > 
> > I occasionally noticed this typo ('a' -> 'an'), will fix it.
> > 
> > > > for passing fdt to kernel, we don't need to implement any
> > > > EFI functions for kernel because it won't look into anything
> > > > beside devicetree from that table if we tell kernel we are
> > > > not efi compatible by setting a0 boot argument to zero.
> > > > 
> > > > Link: https://docs.kernel.org/arch/loongarch/booting.html
> > > > Signed-off-by: Jiaxun Yang <[email protected]>
> > > > Signed-off-by: Yao Zi <[email protected]>
> > > [snip]
> > > > diff --git a/arch/loongarch/Kconfig b/arch/loongarch/Kconfig
> > > > index ac4be2c837c0..5654de5a58c7 100644
> > > > --- a/arch/loongarch/Kconfig
> > > > +++ b/arch/loongarch/Kconfig
> > > > @@ -56,4 +56,7 @@ config STACK_SIZE_SHIFT
> > > >  config OF_BOARD_FIXUP
> > > >  	default y if OF_SEPARATE
> > > >  
> > > > +config LIB_BOOTM
> > > > +	select CRC32
> > > 
> > > If this is needed, it's a generic problem (and to be fixed outside this
> > > series) where LIB_BOOTM (and possibly LIB_BOOTI/Z since you know what to
> > > look for) need to select CRC32 and SPL_LIB_BOOTM SPL_CRC32. Thanks.
> > 
> > I think the dependency is specific to LoongArch, since CRC32 is
> > necessary only because we need to fake an EFI system table, which is
> > CRC32-checksum'ed, to pass FDT to the kernel on LoongArch. AFAIK other
> > architectures don't require such treatment, so I keep this
> > LoongArch-only.
> 
> Ah, I think where you select LIB_BOOTM you should also select CRC32
> then.

Or, finishing the thought, "select CRC32 if LOONGARCH" in the main
entry for LIB_BOOTM.

-- 
Tom

-----BEGIN PGP SIGNATURE-----

iHUEABYKAB0WIQTzzqh0PWDgGS+bTHor4qD1Cr/kCgUCaonzSgAKCRAr4qD1Cr/k
Ch1AAQDzezeTRff+wuUaW7PmodVNTtcr+IG/LLs9Lxv4ZCRM5AEA76KmpQhbOrK9
HD871TxNyd6WIXjLjQ4Ixi0D7NKN1Qg=
=dPvL
-----END PGP SIGNATURE-----