On 14/1/23 23:27, Bernhard Beschow wrote:
Signed-off-by: Bernhard Beschow <shen...@gmail.com>
---
  include/hw/acpi/acpi_aml_interface.h | 11 +++++++++++
  hw/i2c/smbus_ich9.c                  |  5 +----
  hw/isa/lpc_ich9.c                    |  5 +----
  hw/isa/piix3.c                       |  5 +----
  4 files changed, 14 insertions(+), 12 deletions(-)

diff --git a/include/hw/acpi/acpi_aml_interface.h 
b/include/hw/acpi/acpi_aml_interface.h
index 436da069d6..cb2f1c9910 100644
--- a/include/hw/acpi/acpi_aml_interface.h
+++ b/include/hw/acpi/acpi_aml_interface.h

+static inline void qbus_build_aml(BusState *bus, Aml *scope)
+{
+    BusChild *kid;
+
+    QTAILQ_FOREACH(kid, &bus->children, sibling) {
+        call_dev_aml_func(DEVICE(kid->child), scope);
+    }
+}

Inline not really justified, can we move this to hw/acpi/aml-build.c?

Otherwise:
Reviewed-by: Philippe Mathieu-Daudé <phi...@linaro.org>


Reply via email to