On Thu, 20 Jun 2024 17:03:12 +0100
Jonathan Cameron <jonathan.came...@huawei.com> wrote:
> Igor noted that this function only builds one instance, so was rather
> misleadingly named. Fix that.
>
> Suggested-by: Igor Mammedov <imamm...@redhat.com>
> Signed-off-by: Jonathan Cameron <jonathan.came...@huawei.com>
Reviewed-by: Igor Mammedov <imamm...@redhat.com>
>
> ---
> v3: New patch
> ---
> hw/acpi/acpi_generic_initiator.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/hw/acpi/acpi_generic_initiator.c
> b/hw/acpi/acpi_generic_initiator.c
> index 7665b16107..73bafaaaea 100644
> --- a/hw/acpi/acpi_generic_initiator.c
> +++ b/hw/acpi/acpi_generic_initiator.c
> @@ -74,7 +74,7 @@ static void acpi_generic_initiator_class_init(ObjectClass
> *oc, void *data)
> acpi_generic_initiator_set_node, NULL, NULL);
> }
>
> -static int build_all_acpi_generic_initiators(Object *obj, void *opaque)
> +static int build_acpi_generic_initiator(Object *obj, void *opaque)
> {
> MachineState *ms = MACHINE(qdev_get_machine());
> AcpiGenericInitiator *gi;
> @@ -111,6 +111,6 @@ static int build_all_acpi_generic_initiators(Object *obj,
> void *opaque)
> void build_srat_generic_pci_initiator(GArray *table_data)
> {
> object_child_foreach_recursive(object_get_root(),
> - build_all_acpi_generic_initiators,
> + build_acpi_generic_initiator,
> table_data);
> }
