linux-kernel - Re: [PATCH 06/24] usb: isp1760: isp1760-udc: Provide missing description for 'udc' param

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] [thread-next>] [day] [month] [year] [list]

Date:   Wed, 26 May 2021 15:11:04 +0100
From:   "Rui Miguel Silva" <rui.silva@...aro.org>
To:     "Lee Jones" <lee.jones@...aro.org>
Cc:     <linux-kernel@...r.kernel.org>,
        "Greg Kroah-Hartman" <gregkh@...uxfoundation.org>,
        "Laurent Pinchart" <laurent.pinchart@...asonboard.com>,
        <linux-usb@...r.kernel.org>
Subject: Re: [PATCH 06/24] usb: isp1760: isp1760-udc: Provide missing
 description for 'udc' param

Hi Lee,
On Wed May 26, 2021 at 2:00 PM WEST, Lee Jones wrote:

> Fixes the following W=1 kernel build warning(s):
>
>  drivers/usb/isp1760/isp1760-udc.c:150: warning: Function parameter or member 'udc' not described in 'isp1760_udc_select_ep'
>
> Cc: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
> Cc: Rui Miguel Silva <rui.silva@...aro.org>
> Cc: Laurent Pinchart <laurent.pinchart@...asonboard.com>
> Cc: linux-usb@...r.kernel.org
> Signed-off-by: Lee Jones <lee.jones@...aro.org>
> ---
>  drivers/usb/isp1760/isp1760-udc.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/drivers/usb/isp1760/isp1760-udc.c b/drivers/usb/isp1760/isp1760-udc.c
> index 3e05e36054357..a78da59d6417b 100644
> --- a/drivers/usb/isp1760/isp1760-udc.c
> +++ b/drivers/usb/isp1760/isp1760-udc.c
> @@ -137,6 +137,7 @@ static void __isp1760_udc_select_ep(struct isp1760_udc *udc,
>  /**
>   * isp1760_udc_select_ep - Select an endpoint for register access
>   * @ep: The endpoint
> + * @udc: Reference to the device controller

I had this in my latest patch set, but got lost, thanks for fixing
this.

I'd like to ask you to swap the order to match the order in which
the args appear in the function declaration.

With this changed you can add my:
Reviewed-by: Rui Miguel Silva <rui.silva@...aro.org>

------
Cheers,
     Rui


>   *
>   * The ISP1761 endpoint registers are banked. This function selects the target
>   * endpoint for banked register access. The selection remains valid until the
> -- 
> 2.31.1