On Wed, Oct 19, 2022 at 05:28:54PM +0300, Heikki Krogerus wrote:
> Replacing the direct fwnode_property_get_reference_args()
> call will this wrapper function.
>
> No functional changes intended.
>
> Signed-off-by: Heikki Krogerus <heikki.krogerus@xxxxxxxxxxxxxxx>
> ---
> drivers/thunderbolt/acpi.c | 15 ++++-----------
> 1 file changed, 4 insertions(+), 11 deletions(-)
>
> diff --git a/drivers/thunderbolt/acpi.c b/drivers/thunderbolt/acpi.c
> index 7a8adf5ad5a09..48ac227e6d8a0 100644
> --- a/drivers/thunderbolt/acpi.c
> +++ b/drivers/thunderbolt/acpi.c
> @@ -15,24 +15,17 @@ static acpi_status tb_acpi_add_link(acpi_handle handle, u32 level, void *data,
> void **return_value)
> {
> struct acpi_device *adev = acpi_fetch_acpi_dev(handle);
> - struct fwnode_reference_args args;
> struct fwnode_handle *fwnode;
> struct tb_nhi *nhi = data;
> struct pci_dev *pdev;
> struct device *dev;
> - int ret;
> -
> - if (!adev)
> - return AE_OK;
Oh, I'm sorry, I did not mean to remove that check. I'll resend.
> - fwnode = acpi_fwnode_handle(adev);
> - ret = fwnode_property_get_reference_args(fwnode, "usb4-host-interface",
> - NULL, 0, 0, &args);
> - if (ret)
> + fwnode = fwnode_find_reference(acpi_fwnode_handle(adev), "usb4-host-interface", 0);
> + if (IS_ERR(fwnode))
> return AE_OK;
>
> /* It needs to reference this NHI */
> - if (dev_fwnode(&nhi->pdev->dev) != args.fwnode)
> + if (dev_fwnode(&nhi->pdev->dev) != fwnode)
> goto out_put;
>
> /*
> @@ -100,7 +93,7 @@ static acpi_status tb_acpi_add_link(acpi_handle handle, u32 level, void *data,
> }
>
> out_put:
> - fwnode_handle_put(args.fwnode);
> + fwnode_handle_put(fwnode);
> return AE_OK;
> }
>
thanks,
--
heikki
