[Buildroot] [PATCH v2 1/1] fwts: Enable optional efi_runtime kernel module build

Erico Nunes nunes.erico at gmail.com
Mon Nov 5 12:52:12 UTC 2018

On Mon, Nov 5, 2018 at 1:03 PM Sumit Garg <sumit.garg at linaro.org> wrote:
> Firmware test suite does provides efi_runtime kernel module required
> to run UEFI tests. So optionally enable this module build.
> Signed-off-by: Sumit Garg <sumit.garg at linaro.org>
> ---
> Changes in v2:
> 1. Keep efi_runtime config option inside if block.
> 2. Remove linux dependency as $(kernel-module) takes care of this.
> 3. Correct placement of config and $(kernel-module) blocks.

v2 looks good to me and I gave it a quick test, seems to work fine.

> +       bool "efi_runtime module"
> +       depends on BR2_LINUX_KERNEL
> +       help
> +         Firmware Test Suite (FWTS) also provides EFI runtime kernel
> +         module required to run UEFI tests.
> +
> +comment "efi-runtime module needs a Linux kernel to be built"
> +       depends on !BR2_LINUX_KERNEL

These efi_runtime and efi-runtime strings should at least be
consistent in the menu, either with dash or underscore. If nobody has
other issues, maybe this can be fixed while applying?

Other than that:

Reviewed-by: Erico Nunes <nunes.erico at gmail.com>

More information about the buildroot mailing list