linux-kernel - Re: [PATCH] thunderbolt: ACPI: Use the helper fwnode_find

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <Y1ARlOfUYW7OpczT@kuha.fi.intel.com>
Date:   Wed, 19 Oct 2022 18:02:44 +0300
From:   Heikki Krogerus <heikki.krogerus@...ux.intel.com>
To:     Mika Westerberg <mika.westerberg@...ux.intel.com>
Cc:     Andreas Noever <andreas.noever@...il.com>,
        Michael Jamet <michael.jamet@...el.com>,
        Yehezkel Bernat <YehezkelShB@...il.com>,
        linux-usb@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] thunderbolt: ACPI: Use the helper fwnode_find_reference()

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@...ux.intel.com>
> ---
>  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