Bagas Sanjaya <bagasdotme@xxxxxxxxx> writes: > *uclogic_params_pen_inrange_to_str(), uclogic_params_pen_hid_dbg(), and > uclogic_params_frame_hid_dbg() are static functions but contain > double-asterisk (/** */) comment, which confuses kernel-doc as public > function for generating documentation. > > Remove the extraneous asterisk from these comments. So there is nothing wrong with making kerneldoc comments for static functions, that is not where the problem lies. > Link: https://lore.kernel.org/linux-doc/202205272033.XFYlYj8k-lkp@xxxxxxxxx/ > Fixes: a228809fa6f39c ("HID: uclogic: Move param printing to a function") > Reported-by: kernel test robot <lkp@xxxxxxxxx> > Cc: Nikolai Kondrashov <spbnick@xxxxxxxxx> > Cc: Jiri Kosina <jikos@xxxxxxxxxx> > Cc: Benjamin Tissoires <benjamin.tissoires@xxxxxxxxxx> > Cc: "José Expósito" <jose.exposito89@xxxxxxxxx> > Cc: llvm@xxxxxxxxxxxxxxx > Cc: linux-input@xxxxxxxxxxxxxxx > Cc: linux-kernel@xxxxxxxxxxxxxxx > Signed-off-by: Bagas Sanjaya <bagasdotme@xxxxxxxxx> > --- > drivers/hid/hid-uclogic-params.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/hid/hid-uclogic-params.c b/drivers/hid/hid-uclogic-params.c > index db838f16282d64..71501453de12e4 100644 > --- a/drivers/hid/hid-uclogic-params.c > +++ b/drivers/hid/hid-uclogic-params.c > @@ -20,7 +20,7 @@ > #include <linux/ctype.h> > #include <asm/unaligned.h> > > -/** > +/* > * uclogic_params_pen_inrange_to_str() - Convert a pen in-range reporting type > * to a string. > * A more likely problem here is the blank between the function intro and the parameter description. > @@ -44,7 +44,7 @@ static const char *uclogic_params_pen_inrange_to_str( > } > } > > -/** > +/* > * Dump tablet interface pen parameters with hid_dbg(), indented with one tab. > * Here too > * @hdev: The HID device the pen parameters describe. > @@ -76,7 +76,7 @@ static void uclogic_params_pen_hid_dbg(const struct hid_device *hdev, > (pen->tilt_y_flipped ? "true" : "false")); > } > > -/** > +/* > * Dump tablet interface frame parameters with hid_dbg(), indented with two > * tabs. > * Here, instead, this needs to be formatted as a proper kerneldoc comment with the function name. jon
