[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20200714171824.GB7260@Mani-XPS-13-9360>
Date: Tue, 14 Jul 2020 22:48:24 +0530
From: Manivannan Sadhasivam <manivannan.sadhasivam@...aro.org>
To: Lee Jones <lee.jones@...aro.org>
Cc: linus.walleij@...aro.org, linux-kernel@...r.kernel.org,
linux-gpio@...r.kernel.org,
Andreas Färber <afaerber@...e.de>,
David Liu <liuwei@...ions-semi.com>
Subject: Re: [PATCH 01/25] pinctrl: actions: pinctrl-owl: Supply missing
'struct owl_pinctrl' attribute descriptions
On Mon, Jul 13, 2020 at 03:49:06PM +0100, Lee Jones wrote:
> Fixes the following W=1 kernel build warning(s):
>
> drivers/pinctrl/actions/pinctrl-owl.c:52: warning: Function parameter or member 'clk' not described in 'owl_pinctrl'
> drivers/pinctrl/actions/pinctrl-owl.c:52: warning: Function parameter or member 'irq_chip' not described in 'owl_pinctrl'
> drivers/pinctrl/actions/pinctrl-owl.c:52: warning: Function parameter or member 'num_irq' not described in 'owl_pinctrl'
> drivers/pinctrl/actions/pinctrl-owl.c:52: warning: Function parameter or member 'irq' not described in 'owl_pinctrl'
>
> Cc: "Andreas Färber" <afaerber@...e.de>
> Cc: Manivannan Sadhasivam <manivannan.sadhasivam@...aro.org>
> Cc: David Liu <liuwei@...ions-semi.com>
> Signed-off-by: Lee Jones <lee.jones@...aro.org>
Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@...aro.org>
Thanks,
Mani
> ---
> drivers/pinctrl/actions/pinctrl-owl.c | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/drivers/pinctrl/actions/pinctrl-owl.c b/drivers/pinctrl/actions/pinctrl-owl.c
> index 5a0c8e87aa7cd..7efdfb4f3e9ba 100644
> --- a/drivers/pinctrl/actions/pinctrl-owl.c
> +++ b/drivers/pinctrl/actions/pinctrl-owl.c
> @@ -35,8 +35,12 @@
> * @pctrldev: pinctrl handle
> * @chip: gpio chip
> * @lock: spinlock to protect registers
> + * @clk: clock control
> * @soc: reference to soc_data
> * @base: pinctrl register base address
> + * @irq_chip: IRQ chip information
> + * @num_irq: number of possible interrupts
> + * @irq: interrupt numbers
> */
> struct owl_pinctrl {
> struct device *dev;
> --
> 2.25.1
>
Powered by blists - more mailing lists