[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 07/11] acpi: enumerate SMB bridge automatically along with other
From: |
Igor Mammedov |
Subject: |
[PATCH 07/11] acpi: enumerate SMB bridge automatically along with other PCI devices |
Date: |
Mon, 17 Oct 2022 12:21:42 +0200 |
to make that happen (bridge sits at _ADR: 0x001F0003),
relax PCI enumeration logic to include devices with *function* > 0
if device has something to say about itself (i.e. has build_dev_aml
callback set).
Signed-off-by: Igor Mammedov <imammedo@redhat.com>
---
hw/i386/acpi-build.c | 27 +++------------------------
1 file changed, 3 insertions(+), 24 deletions(-)
diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c
index e1483bb11a..916343d8d6 100644
--- a/hw/i386/acpi-build.c
+++ b/hw/i386/acpi-build.c
@@ -448,9 +448,10 @@ static void build_append_pci_bus_devices(Aml
*parent_scope, PCIBus *bus,
/*
* allow describing coldplugged bridges in ACPI even if they are
not
* on function 0, as they are not unpluggable, for all other
devices
- * generate description only for function 0 per slot
+ * generate description only for function 0 per slot, and for other
+ * functions if device on function provides its own AML
*/
- if (func && !bridge_in_acpi) {
+ if (func && !bridge_in_acpi && !get_dev_aml_func(DEVICE(pdev))) {
continue;
}
} else {
@@ -1319,25 +1320,6 @@ static Aml *build_q35_osc_method(bool
enable_native_pcie_hotplug)
return method;
}
-static void build_smb0(Aml *table, int devnr, int func)
-{
- Aml *scope = aml_scope("_SB.PCI0");
- Aml *dev = aml_device("SMB0");
- bool ambiguous;
- Object *obj;
- /*
- * temporarily fish out device hosting SMBUS, build_smb0 will be gone once
- * PCI enumeration will be switched to call_dev_aml_func()
- */
- obj = object_resolve_path_type("", TYPE_ICH9_SMB_DEVICE, &ambiguous);
- assert(obj && !ambiguous);
-
- aml_append(dev, aml_name_decl("_ADR", aml_int(devnr << 16 | func)));
- call_dev_aml_func(DEVICE(obj), dev);
- aml_append(scope, dev);
- aml_append(table, scope);
-}
-
static void build_acpi0017(Aml *table)
{
Aml *dev, *scope, *method;
@@ -1440,9 +1422,6 @@ build_dsdt(GArray *table_data, BIOSLinker *linker,
build_x86_acpi_pci_hotplug(dsdt, pm->pcihp_io_base);
}
build_q35_pci0_int(dsdt);
- if (pcms->smbus) {
- build_smb0(dsdt, ICH9_SMB_DEV, ICH9_SMB_FUNC);
- }
}
if (misc->has_hpet) {
--
2.31.1
- [PATCH 00/11] x86: clean up ACPI PCI code part 2, Igor Mammedov, 2022/10/17
- [PATCH 05/11] tests: acpi: whitelist DSDT before generating ICH9_SMB AML automatically, Igor Mammedov, 2022/10/17
- [PATCH 01/11] acpi: pc: vga: use AcpiDevAmlIf interface to build VGA device descriptors, Igor Mammedov, 2022/10/17
- [PATCH 02/11] tests: acpi: whitelist DSDT before generating PCI-ISA bridge AML automatically, Igor Mammedov, 2022/10/17
- [PATCH 07/11] acpi: enumerate SMB bridge automatically along with other PCI devices,
Igor Mammedov <=
- [PATCH 04/11] tests: acpi: update expected DSDT after ISA bridge is moved directly under PCI host bridge, Igor Mammedov, 2022/10/17
- [PATCH 06/11] acpi: add get_dev_aml_func() helper, Igor Mammedov, 2022/10/17
- [PATCH 03/11] acpi: pc/q35: drop ad-hoc PCI-ISA bridge AML routines and let bus ennumeration generate AML, Igor Mammedov, 2022/10/17
- [PATCH 10/11] acpi: pc/35: sanitize _GPE declaration order, Igor Mammedov, 2022/10/17
- [PATCH 08/11] tests: acpi: update expected blobs, Igor Mammedov, 2022/10/17
- [PATCH 09/11] tests: acpi: pc/q35 whitelist DSDT before \_GPE cleanup, Igor Mammedov, 2022/10/17
- [PATCH 11/11] tests: acpi: update expected blobs, Igor Mammedov, 2022/10/17