On Tue, 2014-07-08 at 11:14 +0100, Matt Fleming wrote:
> On Tue, 08 Jul, at 11:00:58AM, Lee, Chun-Yi wrote:
[...]
> > --- a/arch/x86/platform/efi/efi.c
> > +++ b/arch/x86/platform/efi/efi.c
> > @@ -44,6 +44,7 @@
> >  #include <linux/io.h>
> >  #include <linux/reboot.h>
> >  #include <linux/bcd.h>
> > +#include <linux/platform_device.h>
> >  
> >  #include <asm/setup.h>
> >  #include <asm/efi.h>
> > @@ -780,6 +781,20 @@ void __init efi_late_init(void)
> >     efi_bgrt_init();
> >  }
> >  
> > +#ifdef CONFIG_EFI_VARS_MODULE
> > +static int __init efi_load_efivars(void)
> > +{
> > +   struct platform_device *pdev;
> > +
> > +   if (!efi_enabled(EFI_RUNTIME_SERVICES))
> > +           return 0;
> > +
> > +   pdev = platform_device_register_simple("efivars", 0, NULL, 0);
> > +   return IS_ERR(pdev) ? PTR_ERR(pdev) : 0;
> > +}
> > +device_initcall(efi_load_efivars);
> > +#endif
> > +
> 
> Could this be moved to drivers/firmware/efi/efi.c? That way the arm64
> folks could benefit from it too.

It seems like that should work now that efi_enabled() is not specific to
x86.

Ben.

> >  void __init efi_set_executable(efi_memory_desc_t *md, bool executable)
> >  {
> >     u64 addr, npages;
> > diff --git a/drivers/firmware/efi/efivars.c b/drivers/firmware/efi/efivars.c
> > index 463c565..f256ecd 100644
> > --- a/drivers/firmware/efi/efivars.c
> > +++ b/drivers/firmware/efi/efivars.c
> > @@ -78,6 +78,7 @@ MODULE_AUTHOR("Matt Domsch <[email protected]>");
> >  MODULE_DESCRIPTION("sysfs interface to EFI Variables");
> >  MODULE_LICENSE("GPL");
> >  MODULE_VERSION(EFIVARS_VERSION);
> > +MODULE_ALIAS("platform:efivars");
> >  
> >  LIST_HEAD(efivar_sysfs_list);
> >  EXPORT_SYMBOL_GPL(efivar_sysfs_list);
> > -- 
> > 1.8.4.5
> > 
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-efi" in
> > the body of a message to [email protected]
> > More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 

-- 
Ben Hutchings
Any smoothly functioning technology is indistinguishable from a rigged demo.

Attachment: signature.asc
Description: This is a digitally signed message part

Reply via email to