[PATCH 1/4] arm: mach-k3: Allow for calling get_reset_reason() twice

Sverdlin, Alexander alexander.sverdlin at siemens.com
Tue May 12 08:33:49 CEST 2026


Hi Anshul!

On Tue, 2026-05-12 at 11:23 +0530, Anshul Dalal wrote:
> On Mon, 11 May 2026 22:12:30 +0200, A. Sverdlin <alexander.sverdlin at siemens.com> wrote:
> > diff --git a/arch/arm/include/asm/global_data.h b/arch/arm/include/asm/global_data.h
> > index b2ec450f900..c2b20a1dde6 100644
> > --- a/arch/arm/include/asm/global_data.h
> > +++ b/arch/arm/include/asm/global_data.h
> > @@ -105,6 +105,9 @@ struct arch_global_data {
> >   #ifdef CONFIG_ARCH_IMX8ULP
> >   	bool m33_handshake_done;
> >   #endif
> > +#ifdef CONFIG_ARCH_K3
> > +	u32 reset_src;			/* Cached reset source register */
> > +#endif
> 
> Since we're trying to make this an architecture agnostic hook that users can
> override, I don't think we should have it conditional on CONFIG_ARCH_K3.
> 
> IMO having it in bd_info while making it accessible to user via 'bdinfo' command
> could be better instead of only displaying it at U-Boot splash as we do now.

I agree, integrating into bdinfo sounds beneficial!
The above variable is a raw register value, specific to K3 family, not necessary
making sense for, say, i.MX. A `char*` pointing to a decoded reset reason
could of course make sense for all platforms.

-- 
Alexander Sverdlin
Siemens AG
www.siemens.com


More information about the U-Boot mailing list