efi: gsmi: fix false dependency on CONFIG_EFI_VARS
authorArd Biesheuvel <ardb@kernel.org>
Wed, 23 Sep 2020 08:18:31 +0000 (10:18 +0200)
committerArd Biesheuvel <ardb@kernel.org>
Tue, 29 Sep 2020 17:40:57 +0000 (19:40 +0200)
The gsmi code does not actually rely on CONFIG_EFI_VARS, since it only
uses the efivars abstraction that is included unconditionally when
CONFIG_EFI is defined. CONFIG_EFI_VARS controls the inclusion of the
code that exposes the sysfs entries, and which has been deprecated for
some time.

Signed-off-by: Ard Biesheuvel <ardb@kernel.org>
drivers/firmware/google/Kconfig
drivers/firmware/google/gsmi.c

index a3a6ca6..97968ae 100644 (file)
@@ -15,7 +15,7 @@ config GOOGLE_SMI
        help
          Say Y here if you want to enable SMI callbacks for Google
          platforms.  This provides an interface for writing to and
-         clearing the event log.  If EFI_VARS is also enabled this
+         clearing the event log.  If CONFIG_EFI is also enabled this
          driver provides an interface for reading and writing NVRAM
          variables.
 
index 5b2011e..7d9367b 100644 (file)
@@ -302,7 +302,7 @@ static int gsmi_exec(u8 func, u8 sub)
        return rc;
 }
 
-#ifdef CONFIG_EFI_VARS
+#ifdef CONFIG_EFI
 
 static struct efivars efivars;
 
@@ -483,7 +483,7 @@ static const struct efivar_operations efivar_ops = {
        .get_next_variable = gsmi_get_next_variable,
 };
 
-#endif /* CONFIG_EFI_VARS */
+#endif /* CONFIG_EFI */
 
 static ssize_t eventlog_write(struct file *filp, struct kobject *kobj,
                               struct bin_attribute *bin_attr,
@@ -1007,7 +1007,7 @@ static __init int gsmi_init(void)
                goto out_remove_bin_file;
        }
 
-#ifdef CONFIG_EFI_VARS
+#ifdef CONFIG_EFI
        ret = efivars_register(&efivars, &efivar_ops, gsmi_kobj);
        if (ret) {
                printk(KERN_INFO "gsmi: Failed to register efivars\n");
@@ -1047,7 +1047,7 @@ static void __exit gsmi_exit(void)
        unregister_die_notifier(&gsmi_die_notifier);
        atomic_notifier_chain_unregister(&panic_notifier_list,
                                         &gsmi_panic_notifier);
-#ifdef CONFIG_EFI_VARS
+#ifdef CONFIG_EFI
        efivars_unregister(&efivars);
 #endif