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 > <[email protected]> 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
