Hi Saket,
On Sun, Aug 23, 2015 at 7:33 PM, Saket Sinha saket.sinha89@gmail.com wrote:
Hi Bin,
On Sun, Aug 23, 2015 at 3:03 PM, Bin Meng bmeng.cn@gmail.com wrote:
Hi Saket,
On Sat, Aug 22, 2015 at 2:50 PM, Saket Sinha saket.sinha89@gmail.com wrote:
This patch mainly adds ACPI support to QEMU. Verified by booting Linux kernel on QEMU i440FX and Q35.
I don't think ACPI is for i440FX. Although it boots, but that does not mean ACPI really works. These ASL files describing Q35 platform, not i440FX.
Signed-off-by: Saket Sinha saket.sinha89@gmail.com
arch/x86/cpu/qemu/Makefile | 1 + arch/x86/cpu/qemu/acpi.c | 179 +++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 180 insertions(+) create mode 100644 arch/x86/cpu/qemu/acpi.c
diff --git a/arch/x86/cpu/qemu/Makefile b/arch/x86/cpu/qemu/Makefile index 9a66b16..8c3884c 100644 --- a/arch/x86/cpu/qemu/Makefile +++ b/arch/x86/cpu/qemu/Makefile @@ -8,4 +8,5 @@ ifndef CONFIG_EFI_STUB obj-y += car.o dram.o endif obj-y += qemu.o +obj-$(CONFIG_GENERATE_ACPI_TABLE) += acpi.o obj-$(CONFIG_PCI) += pci.o diff --git a/arch/x86/cpu/qemu/acpi.c b/arch/x86/cpu/qemu/acpi.c new file mode 100644 index 0000000..2496da7 --- /dev/null +++ b/arch/x86/cpu/qemu/acpi.c @@ -0,0 +1,179 @@ +/*
- Copyright (C) 2015, Saket Sinha saket.sinha89@gmail.com
- SPDX-License-Identifier: GPL-2.0+
- */
+#include <common.h> +#include <asm/acpi_table.h> +#include <asm/ioapic.h> +#include <asm/tables.h>
+void acpi_create_fadt(struct acpi_fadt *fadt, struct acpi_facs *facs,
void *dsdt)+{
acpi_header_t *header = &(fadt->header);u16 pmbase;pci_dev_t bdf = PCI_BDF(0, 0x1f, 0);pci_read_config_word(bdf, 0x40, &pmbase);/*Nits: misaligned
* TODO(saket.sinha89@gmail.com): wrong value* of pmbase by above function. Harcoding it toTypo: Hardcoding
* correct value. Since no PCI register is* programmed Power Management Interface is* not working*/You still did not address this issue before sending v6. I don't understand why don't you just prepare a separate patch to program this PMBASE register by yourself. Are you waiting for me or Simon to do this, or are you looking for something else?
As discussed with Simon, he wants me to come up with ACPI support on Minnowmax first and look up for QEMU fixes later before the deadline of this program(GSOC) ends.
Currently, I have been working on Minnowmax ACPI support and post a workable patch for the same on mailing list ASAP.
If you want me to currently stop on this activity and work on the Qemu patch which programs PMBASE register first, let us discuss it with Simon and plan accordingly.
OK, thanks for the update. It's better you'd send this clarification in your v5 patch thread which should have addressed my concern before sending v6.
Frankly speaking, working on MinnowMax now without a basically working ACPI platform does not save us time. So far ACPI on QEMU is just partially working. We only addressed the Configuration part, but nothing in the Power management part was looked at. That's where this PMBASE is concerned.
Regards, Bin