Re: [PATCH v2 6/7] of: property: fw_devlink: Follow clock nexus maps
From: Herve Codina
Date: Wed Sep 16 2026 - 14:57:29 EST
Hi Ernest,
On Thu, 10 Sep 2026 16:16:47 +0200
Ernest Van Hoecke <ernestvanhoecke@xxxxxxxxx> wrote:
> From: Ernest Van Hoecke <ernest.vanhoecke@xxxxxxxxxxx>
>
> Clock consumers use of_parse_phandle_with_args_map() to resolve
> specifiers through clock-map properties. fw_devlink instead parses the
> clocks property as a direct phandle reference, so it records the nexus
> node as the supplier rather than the mapped clock provider.
>
> A nexus is a translation node, not the clock provider, and need not be
> populated as a device. The resulting fwnode link can therefore remain
> unresolved and indefinitely defer the consumer while
> device_links_check_suppliers() waits for the nexus.
>
> Use the map-aware parser for clocks so fw_devlink resolves the same
> provider as the clock consumer API. Direct clock references continue to
> work because the map-aware parser returns the original provider when no
> clock-map property is present.
>
> Fixes: aacc54c67869 ("clk: Add support for clock nexus dt bindings")
This tag is only in linux-next.
Will it be the same when merged on next release?
> Link: https://lore.kernel.org/all/20260807-schneider-v7-2-rc1-eip201-upstream-v5-0-8785f9531cf0@xxxxxxxxxxx/
> Signed-off-by: Ernest Van Hoecke <ernest.vanhoecke@xxxxxxxxxxx>
> ---
> drivers/of/property.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/of/property.c b/drivers/of/property.c
> index a9c51d639a2b..48e05f7bae2f 100644
Best regards,
Hervé