Re: [PATCH v6 11/27] PCI: of: Use fw_devlink_set_device()

Bjorn Helgaas
2026-03-25T20:10:50Z

On Wed, Mar 25, 2026 at 03:35:38PM +0100, Herve Codina wrote:
> The code set directly fwnode.dev field.
>
> Use the dedicated fw_devlink_set_device() helper to perform this
> operation.
>
> Signed-off-by: Herve Codina <herve.codina@bootlin.com>
> Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>

Acked-by: Bjorn Helgaas <bhelgaas@google.com>

> ---
> drivers/pci/of.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/pci/of.c b/drivers/pci/of.c
> index 9f8eb5df279e..5fb7f501fd35 100644
> --- a/drivers/pci/of.c
> +++ b/drivers/pci/of.c
> @@ -803,7 +803,7 @@ void of_pci_make_host_bridge_node(struct pci_host_bridge *bridge)
> * bus. Avoid any new device creation.
> */
> of_node_set_flag(np, OF_POPULATED);
> - np->fwnode.dev = &bridge->dev;
> + fw_devlink_set_device(&np->fwnode, &bridge->dev);
> fwnode_dev_initialized(&np->fwnode, true);
>
> ret = of_changeset_apply(cset);
> --
> 2.53.0
>