On Tue, 14 Jul 2020, Jacopo Mondi wrote: > Hi Lee, > > On Mon, Jul 13, 2020 at 03:49:25PM +0100, Lee Jones wrote: > > Some description blocks are void of any description/documentation, > > others are missing 'struct' identifiers, there are also a couple of > > misspellings of function parameter names. Fix all of them. > > > > Fixes the following W=1 kernel build warning(s): > > > > drivers/pinctrl/pinctrl-rza1.c:81: warning: cannot understand function > > prototype: 'struct rza1_bidir_pin ' > > drivers/pinctrl/pinctrl-rza1.c:90: warning: cannot understand function > > prototype: 'struct rza1_bidir_entry ' > > drivers/pinctrl/pinctrl-rza1.c:98: warning: cannot understand function > > prototype: 'struct rza1_swio_pin ' > > drivers/pinctrl/pinctrl-rza1.c:108: warning: cannot understand function > > prototype: 'struct rza1_swio_entry ' > > drivers/pinctrl/pinctrl-rza1.c:116: warning: cannot understand function > > prototype: 'struct rza1_pinmux_conf ' > > drivers/pinctrl/pinctrl-rza1.c:443: warning: cannot understand function > > prototype: 'struct rza1_mux_conf ' > > drivers/pinctrl/pinctrl-rza1.c:462: warning: cannot understand function > > prototype: 'struct rza1_port ' > > drivers/pinctrl/pinctrl-rza1.c:482: warning: cannot understand function > > prototype: 'struct rza1_pinctrl ' > > drivers/pinctrl/pinctrl-rza1.c:546: warning: Function parameter or member > > 'port' not described in 'rza1_pinmux_get_flags' > > drivers/pinctrl/pinctrl-rza1.c:546: warning: Function parameter or member > > 'pin' not described in 'rza1_pinmux_get_flags' > > drivers/pinctrl/pinctrl-rza1.c:546: warning: Function parameter or member > > 'func' not described in 'rza1_pinmux_get_flags' > > drivers/pinctrl/pinctrl-rza1.c:546: warning: Function parameter or member > > 'rza1_pctl' not described in 'rza1_pinmux_get_flags' > > drivers/pinctrl/pinctrl-rza1.c:575: warning: Function parameter or member > > 'port' not described in 'rza1_set_bit' > > drivers/pinctrl/pinctrl-rza1.c:575: warning: Function parameter or member > > 'reg' not described in 'rza1_set_bit' > > drivers/pinctrl/pinctrl-rza1.c:575: warning: Function parameter or member > > 'bit' not described in 'rza1_set_bit' > > drivers/pinctrl/pinctrl-rza1.c:575: warning: Function parameter or member > > 'set' not described in 'rza1_set_bit' > > drivers/pinctrl/pinctrl-rza1.c:672: warning: Function parameter or member > > 'rza1_pctl' not described in 'rza1_pin_mux_single' > > drivers/pinctrl/pinctrl-rza1.c:672: warning: Excess function parameter > > 'pinctrl' description in 'rza1_pin_mux_single' > > > > Cc: Geert Uytterhoeven <geert+rene...@glider.be> > > Cc: Jacopo Mondi <jacopo+rene...@jmondi.org> > > Cc: linux-renesas-...@vger.kernel.org > > Signed-off-by: Lee Jones <lee.jo...@linaro.org> > > Thanks, at the time I had no real idea about the implications of > documenting with kerneldoc :)
You and 1000 others! :D Don't lose any sleep over it. > Acked-by: Jacopo Mondi <jac...@jmondi.org> Thanks. -- Lee Jones [李琼斯] Senior Technical Lead - Developer Services Linaro.org │ Open source software for Arm SoCs Follow Linaro: Facebook | Twitter | Blog