| Message ID | 20210426193520.4115528-5-philmd@redhat.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | hw/nvram/fw_cfg: Do not build device if not needed (Spring cleanup) | expand |
On 04/26/21 21:35, Philippe Mathieu-Daudé wrote: > The TYPE_VMGENID device depends on fw_cfg: > > $ git grep \ fw_cfg hw/acpi/vmgenid.c > hw/acpi/vmgenid.c:128: fw_cfg_add_file(s, VMGENID_GUID_FW_CFG_FILE, guid->data, > hw/acpi/vmgenid.c:131: fw_cfg_add_file_callback(s, VMGENID_ADDR_FW_CFG_FILE, NULL, NULL, NULL, > > Add the proper Kconfig dependency. > > Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com> > --- > hw/acpi/Kconfig | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/hw/acpi/Kconfig b/hw/acpi/Kconfig > index 1932f66af8d..b9dc932d2a7 100644 > --- a/hw/acpi/Kconfig > +++ b/hw/acpi/Kconfig > @@ -40,5 +40,6 @@ config ACPI_VMGENID > bool > default y > depends on PC > + select FW_CFG > > config ACPI_HW_REDUCED > The intent is certainly correct and the implementation looks plausible. Reviewed-by: Laszlo Ersek <lersek@redhat.com>
diff --git a/hw/acpi/Kconfig b/hw/acpi/Kconfig index 1932f66af8d..b9dc932d2a7 100644 --- a/hw/acpi/Kconfig +++ b/hw/acpi/Kconfig @@ -40,5 +40,6 @@ config ACPI_VMGENID bool default y depends on PC + select FW_CFG config ACPI_HW_REDUCED
The TYPE_VMGENID device depends on fw_cfg: $ git grep \ fw_cfg hw/acpi/vmgenid.c hw/acpi/vmgenid.c:128: fw_cfg_add_file(s, VMGENID_GUID_FW_CFG_FILE, guid->data, hw/acpi/vmgenid.c:131: fw_cfg_add_file_callback(s, VMGENID_ADDR_FW_CFG_FILE, NULL, NULL, NULL, Add the proper Kconfig dependency. Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com> --- hw/acpi/Kconfig | 1 + 1 file changed, 1 insertion(+)