
Hi Bin,
2016-01-20 16:46 GMT+08:00 Bin Meng bmeng.cn@gmail.com:
On Wed, Jan 20, 2016 at 12:24 PM, Miao Yan yanmiaobest@gmail.com wrote:
This patch adds a config option for loading ACPI table from QEMU. When enabled, U-Boot won't generate ACPI tables, but use those provided by QEMU.
Signed-off-by: Miao Yan yanmiaobest@gmail.com
arch/x86/Kconfig | 9 +++++++++ arch/x86/cpu/qemu/Makefile | 2 ++ arch/x86/lib/Makefile | 2 ++ 3 files changed, 13 insertions(+)
Reviewed-by: Bin Meng bmeng.cn@gmail.com Tested-by: Bin Meng bmeng.cn@gmail.com
But please see one nits below:
diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig index f07567c..26c8d83 100644 --- a/arch/x86/Kconfig +++ b/arch/x86/Kconfig @@ -358,6 +358,15 @@ config GENERATE_ACPI_TABLE by the operating system. It defines platform-independent interfaces for configuration and power management monitoring.
+config QEMU_ACPI_TABLE
bool "load ACPI table from QEMU fw_cfg interface"
nits: load -> Load
I can fix this when applying.
Thanks, I just found this patch breaks git bisect, I'll submit v3 to fix this as well as the typo.
depends on GENERATE_ACPI_TABLE && QEMU
default y
help
By default, U-Boot generates its own ACPI tables. This option, if
enabled, disables U-Boot's version and loads ACPI tables generated
by QEMU.
[snip]
Regards, Bin