On Sat, Oct 03, 2020 at 01:28:18PM -0400, Brian Gerst wrote: > On Sat, Oct 3, 2020 at 2:05 AM Heinrich Schuchardt <[email protected]> wrote: > > > > CONFIG_EFI_MIXED depends on CONFIG_X86_64=y. > > There is no need to check CONFIG_X86_64 again. > > > > Signed-off-by: Heinrich Schuchardt <[email protected]> > > --- > > arch/x86/include/asm/efi.h | 2 -- > > 1 file changed, 2 deletions(-) > > > > diff --git a/arch/x86/include/asm/efi.h b/arch/x86/include/asm/efi.h > > index b9c2667ac46c..ab28bf1c74cf 100644 > > --- a/arch/x86/include/asm/efi.h > > +++ b/arch/x86/include/asm/efi.h > > @@ -223,8 +223,6 @@ static inline bool efi_is_64bit(void) > > > > static inline bool efi_is_native(void) > > { > > - if (!IS_ENABLED(CONFIG_X86_64)) > > - return true; > > return efi_is_64bit(); > > } > > This would then return false for native 32-bit. > > -- > Brian Gerst
32-bit doesn't use this implementation: it's #define'd to true in drivers/firmware/efi/libstub/efistub.h. Thanks.
