diff mbox

[5/8] arm64: defconfig: enable EFI_VARS and EFI_CAPSULE_LOADER

Message ID 1492012980-11650-6-git-send-email-timur@codeaurora.org (mailing list archive)
State New, archived
Headers show

Commit Message

Timur Tabi April 12, 2017, 4:02 p.m. UTC
Many ARM64 server systems boot via EFI, so enable support for some EFI
options to allow applications to interact with EFI.

CONFIG_EFI_VARS enables sysfs support for reading and writing variables

CONFIG_EFI_CAPSULE_LOADER allows the user to update the EFI firmware.

Signed-off-by: Timur Tabi <timur@codeaurora.org>
---
 arch/arm64/configs/defconfig | 2 ++
 1 file changed, 2 insertions(+)

Comments

Robin Murphy April 12, 2017, 4:13 p.m. UTC | #1
On 12/04/17 17:02, Timur Tabi wrote:
> Many ARM64 server systems boot via EFI, so enable support for some EFI
> options to allow applications to interact with EFI.
> 
> CONFIG_EFI_VARS enables sysfs support for reading and writing variables

Isn't efivarfs preferred for that these days, or have I imagined that?

Robin.

> CONFIG_EFI_CAPSULE_LOADER allows the user to update the EFI firmware.
> 
> Signed-off-by: Timur Tabi <timur@codeaurora.org>
> ---
>  arch/arm64/configs/defconfig | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
> index 1e5c516..cdc8698 100644
> --- a/arch/arm64/configs/defconfig
> +++ b/arch/arm64/configs/defconfig
> @@ -485,6 +485,8 @@ CONFIG_PHY_XGENE=y
>  CONFIG_PHY_TEGRA_XUSB=y
>  CONFIG_ARM_SCPI_PROTOCOL=y
>  CONFIG_RASPBERRYPI_FIRMWARE=y
> +CONFIG_EFI_VARS=y
> +CONFIG_EFI_CAPSULE_LOADER=y
>  CONFIG_ACPI=y
>  CONFIG_EXT2_FS=y
>  CONFIG_EXT3_FS=y
>
Mark Rutland April 12, 2017, 4:14 p.m. UTC | #2
On Wed, Apr 12, 2017 at 05:13:20PM +0100, Robin Murphy wrote:
> On 12/04/17 17:02, Timur Tabi wrote:
> > Many ARM64 server systems boot via EFI, so enable support for some EFI
> > options to allow applications to interact with EFI.
> > 
> > CONFIG_EFI_VARS enables sysfs support for reading and writing variables
> 
> Isn't efivarfs preferred for that these days, or have I imagined that?
> 
> Robin.

Ard, Leif?

Mark.

> > CONFIG_EFI_CAPSULE_LOADER allows the user to update the EFI firmware.
> > 
> > Signed-off-by: Timur Tabi <timur@codeaurora.org>
> > ---
> >  arch/arm64/configs/defconfig | 2 ++
> >  1 file changed, 2 insertions(+)
> > 
> > diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
> > index 1e5c516..cdc8698 100644
> > --- a/arch/arm64/configs/defconfig
> > +++ b/arch/arm64/configs/defconfig
> > @@ -485,6 +485,8 @@ CONFIG_PHY_XGENE=y
> >  CONFIG_PHY_TEGRA_XUSB=y
> >  CONFIG_ARM_SCPI_PROTOCOL=y
> >  CONFIG_RASPBERRYPI_FIRMWARE=y
> > +CONFIG_EFI_VARS=y
> > +CONFIG_EFI_CAPSULE_LOADER=y
> >  CONFIG_ACPI=y
> >  CONFIG_EXT2_FS=y
> >  CONFIG_EXT3_FS=y
> > 
> 
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Timur Tabi April 12, 2017, 8:58 p.m. UTC | #3
On 04/12/2017 11:14 AM, Mark Rutland wrote:
>>> > > Many ARM64 server systems boot via EFI, so enable support for some EFI
>>> > > options to allow applications to interact with EFI.
>>> > > 
>>> > > CONFIG_EFI_VARS enables sysfs support for reading and writing variables
>> > 
>> > Isn't efivarfs preferred for that these days, or have I imagined that?
>> > 
>> > Robin.
> Ard, Leif?

So my research suggests that efivarfs is a replacement for CONFIG_EFI_VARS.
We may still have some older programs that use the sysfs entries, but I'm
not sure.

Regardless, I will drop EFI_VARS from v2 of the patset, and just set
EFI_CAPSULE_LOADER.
diff mbox

Patch

diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
index 1e5c516..cdc8698 100644
--- a/arch/arm64/configs/defconfig
+++ b/arch/arm64/configs/defconfig
@@ -485,6 +485,8 @@  CONFIG_PHY_XGENE=y
 CONFIG_PHY_TEGRA_XUSB=y
 CONFIG_ARM_SCPI_PROTOCOL=y
 CONFIG_RASPBERRYPI_FIRMWARE=y
+CONFIG_EFI_VARS=y
+CONFIG_EFI_CAPSULE_LOADER=y
 CONFIG_ACPI=y
 CONFIG_EXT2_FS=y
 CONFIG_EXT3_FS=y