Re: [PATCH 2/5] arm: include asm-generic/int-ll64.h in setjmp.h

[Yao Zi <ziyao@disroot.org>]

On Sun, Mar 02, 2025 at 05:18:21PM +0100, Heinrich Schuchardt wrote:
> Yao Zi <ziyao@disroot.org> schrieb am So., 2. März 2025, 17:02:
> 
> > On Sun, Mar 02, 2025 at 03:21:18PM +0100, Heinrich Schuchardt wrote:
> > > Don't assume that u32 and u64 are already defined.
> > >
> > > Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
> > > ---
> > >  arch/arm/include/asm/setjmp.h | 2 ++
> > >  1 file changed, 2 insertions(+)
> > >
> > > diff --git a/arch/arm/include/asm/setjmp.h
> > b/arch/arm/include/asm/setjmp.h
> > > index 662bec86321..9a7f5af9f8f 100644
> > > --- a/arch/arm/include/asm/setjmp.h
> > > +++ b/arch/arm/include/asm/setjmp.h
> > > @@ -7,6 +7,8 @@
> > >  #ifndef _SETJMP_H_
> > >  #define _SETJMP_H_   1
> > >
> > > +#include <asm-generic/int-ll64.h>
> > > +
> >
> > >  /*
> > >   * This really should be opaque, but the EFI implementation wrongly
> > >   * assumes that a 'struct jmp_buf_data' is defined.
> >
> > With PATCH 4 of the series, this comment isn't true anymore. Should we
> > fix it up at the same time?
> >
> 
> The comment is removed in patch 3.

Oops, I didn't notice that. Sorry for the noise.

> https://lore.kernel.org/u-boot/20250302142121.150480-4-heinrich.schuchardt@canonical.com/T/#u
> 
> Best regards
> 
> Heinrich

Thanks,
Yao Zi