From: Sakari Ailus Date: Wed, 1 Oct 2025 10:43:19 +0000 (+0300) Subject: ACPI: property: Use ACPI functions in acpi_graph_get_next_endpoint() only X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=5d010473cdeaabf6a2d3a9e2aed2186c1b73c213;p=thirdparty%2Flinux.git ACPI: property: Use ACPI functions in acpi_graph_get_next_endpoint() only Calling fwnode_get_next_child_node() in ACPI implementation of the fwnode property API is somewhat problematic as the latter is used in the impelementation of the former. Instead of using fwnode_get_next_child_node() in acpi_graph_get_next_endpoint(), call acpi_get_next_subnode() directly instead. Signed-off-by: Sakari Ailus Reviewed-by: Laurent Pinchart Reviewed-by: Jonathan Cameron Link: https://patch.msgid.link/20251001104320.1272752-3-sakari.ailus@linux.intel.com Signed-off-by: Rafael J. Wysocki --- diff --git a/drivers/acpi/property.c b/drivers/acpi/property.c index dbf86bee62e1d..b88bad197fca9 100644 --- a/drivers/acpi/property.c +++ b/drivers/acpi/property.c @@ -1473,7 +1473,7 @@ static struct fwnode_handle *acpi_graph_get_next_endpoint( if (!prev) { do { - port = fwnode_get_next_child_node(fwnode, port); + port = acpi_get_next_subnode(fwnode, port); /* * The names of the port nodes begin with "port@" * followed by the number of the port node and they also @@ -1491,13 +1491,13 @@ static struct fwnode_handle *acpi_graph_get_next_endpoint( if (!port) return NULL; - endpoint = fwnode_get_next_child_node(port, prev); + endpoint = acpi_get_next_subnode(port, prev); while (!endpoint) { - port = fwnode_get_next_child_node(fwnode, port); + port = acpi_get_next_subnode(fwnode, port); if (!port) break; if (is_acpi_graph_node(port, "port")) - endpoint = fwnode_get_next_child_node(port, NULL); + endpoint = acpi_get_next_subnode(port, NULL); } /*