From: Yujie Liu <yujie.liu@intel.com>
To: Randy Dunlap <rdunlap@infradead.org>
Cc: kernel test robot <lkp@intel.com>,
Ruslan Bilovol <ruslan.bilovol@gmail.com>,
<oe-kbuild-all@lists.linux.dev>, <linux-kernel@vger.kernel.org>,
Felipe Balbi <felipe.balbi@linux.intel.com>
Subject: Re: drivers/usb/gadget/function/f_uac1_legacy.c:1: warning: no structured comments found
Date: Tue, 5 Dec 2023 14:26:02 +0800 [thread overview]
Message-ID: <ZW7CepyuiJn5d34K@yujie-X299> (raw)
In-Reply-To: <61f2de49-131f-4619-b62e-dfe2f5b6bf3c@infradead.org>
Hi Randy,
On Mon, Dec 04, 2023 at 02:46:41PM -0800, Randy Dunlap wrote:
> Hi ktr,
>
> On 11/20/23 18:31, kernel test robot wrote:
> > Hi Ruslan,
> >
> > FYI, the error/warning was bisected to this commit, please ignore it if it's irrelevant.
> >
> > tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> > head: 98b1cc82c4affc16f5598d4fa14b1858671b2263
> > commit: d355339eecd986648420e05f8c958fbc78dbb382 usb: gadget: function: make current f_uac1 implementation legacy
> > date: 6 years ago
> > config: i386-randconfig-002-20231120 (https://download.01.org/0day-ci/archive/20231121/202311211052.RsFuk4Gr-lkp@intel.com/config)
> > compiler: gcc-7 (Ubuntu 7.5.0-6ubuntu2) 7.5.0
> > reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20231121/202311211052.RsFuk4Gr-lkp@intel.com/reproduce)
> >
> > If you fix the issue in a separate patch/commit (i.e. not just a new version of
> > the same patch/commit), kindly add following tags
> > | Reported-by: kernel test robot <lkp@intel.com>
> > | Closes: https://lore.kernel.org/oe-kbuild-all/202311211052.RsFuk4Gr-lkp@intel.com/
> >
> > All warnings (new ones prefixed by >>):
> >
>
> I can't reproduce any of these warnings.
>
> I think that you should reevaluate something here.
These warnings are produced by kernel-doc tool:
$ git checkout d355339eecd986648420e05f8c958fbc78dbb382
HEAD is now at d355339eecd9 usb: gadget: function: make current f_uac1 implementation legacy
$ scripts/kernel-doc drivers/usb/gadget/function/u_uac1_legacy.c 1>/dev/null
drivers/usb/gadget/function/u_uac1_legacy.c:33: warning: No description found for parameter 'i'
drivers/usb/gadget/function/u_uac1_legacy.c:33: warning: No description found for parameter 'val'
drivers/usb/gadget/function/u_uac1_legacy.c:96: warning: No description found for parameter 'snd'
drivers/usb/gadget/function/u_uac1_legacy.c:156: warning: No description found for parameter 'card'
drivers/usb/gadget/function/u_uac1_legacy.c:156: warning: No description found for parameter 'buf'
drivers/usb/gadget/function/u_uac1_legacy.c:156: warning: No description found for parameter 'count'
drivers/usb/gadget/function/u_uac1_legacy.c:205: warning: No description found for parameter 'card'
drivers/usb/gadget/function/u_uac1_legacy.c:265: warning: No description found for parameter 'gau'
drivers/usb/gadget/function/u_uac1_legacy.c:294: warning: No description found for parameter 'card'
drivers/usb/gadget/function/u_uac1_legacy.c:311: warning: No description found for parameter 'the_card'
Since this is a "move files around" commit that renames u_uac1.c to
u_uac1_legacy.c, and the warnings already exist before this commit, so
this is a false report. Sorry for this.
We introduced a new feature into the bot recently to bisect old
document warnings, but it is not working very smoothly and produces some
false reports. The "steps to reproduce" was not updated in time,
either. We are investigating these issues and will fix them soon.
Apologies again for any noise reports.
Best Regards,
Yujie
>
>
> >>> drivers/usb/gadget/function/f_uac1_legacy.c:1: warning: no structured comments found
> > --
> >>> drivers/usb/gadget/function/u_uac1_legacy.c:33: warning: No description found for parameter 'i'
> >>> drivers/usb/gadget/function/u_uac1_legacy.c:33: warning: No description found for parameter 'val'
> >>> drivers/usb/gadget/function/u_uac1_legacy.c:96: warning: No description found for parameter 'snd'
> >>> drivers/usb/gadget/function/u_uac1_legacy.c:156: warning: No description found for parameter 'card'
> >>> drivers/usb/gadget/function/u_uac1_legacy.c:156: warning: No description found for parameter 'buf'
> >>> drivers/usb/gadget/function/u_uac1_legacy.c:156: warning: No description found for parameter 'count'
> > drivers/usb/gadget/function/u_uac1_legacy.c:205: warning: No description found for parameter 'card'
> >>> drivers/usb/gadget/function/u_uac1_legacy.c:265: warning: No description found for parameter 'gau'
> > drivers/usb/gadget/function/u_uac1_legacy.c:294: warning: No description found for parameter 'card'
> >>> drivers/usb/gadget/function/u_uac1_legacy.c:311: warning: No description found for parameter 'the_card'
> >
> >
> > vim +1 drivers/usb/gadget/function/f_uac1_legacy.c
> >
> > c6994e6f067cf0 drivers/usb/gadget/f_audio.c Bryan Wu 2009-06-03 @1 /*
> > c6994e6f067cf0 drivers/usb/gadget/f_audio.c Bryan Wu 2009-06-03 2 * f_audio.c -- USB Audio class function driver
> > c6994e6f067cf0 drivers/usb/gadget/f_audio.c Bryan Wu 2009-06-03 3 *
> > c6994e6f067cf0 drivers/usb/gadget/f_audio.c Bryan Wu 2009-06-03 4 * Copyright (C) 2008 Bryan Wu <cooloney@kernel.org>
> > c6994e6f067cf0 drivers/usb/gadget/f_audio.c Bryan Wu 2009-06-03 5 * Copyright (C) 2008 Analog Devices, Inc
> > c6994e6f067cf0 drivers/usb/gadget/f_audio.c Bryan Wu 2009-06-03 6 *
> > c6994e6f067cf0 drivers/usb/gadget/f_audio.c Bryan Wu 2009-06-03 7 * Enter bugs at http://blackfin.uclinux.org/
> > c6994e6f067cf0 drivers/usb/gadget/f_audio.c Bryan Wu 2009-06-03 8 *
> > c6994e6f067cf0 drivers/usb/gadget/f_audio.c Bryan Wu 2009-06-03 9 * Licensed under the GPL-2 or later.
> > c6994e6f067cf0 drivers/usb/gadget/f_audio.c Bryan Wu 2009-06-03 10 */
> > c6994e6f067cf0 drivers/usb/gadget/f_audio.c Bryan Wu 2009-06-03 11
> >
> > :::::: The code at line 1 was first introduced by commit
> > :::::: c6994e6f067cf0fc4c6cca3d164018b1150916f8 USB: gadget: add USB Audio Gadget driver
> >
> > :::::: TO: Bryan Wu <cooloney@kernel.org>
> > :::::: CC: Greg Kroah-Hartman <gregkh@suse.de>
> >
>
> --
> ~Randy
>
prev parent reply other threads:[~2023-12-05 6:30 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-11-21 2:31 kernel test robot
2023-12-04 22:46 ` Randy Dunlap
2023-12-05 6:26 ` Yujie Liu [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=ZW7CepyuiJn5d34K@yujie-X299 \
--to=yujie.liu@intel.com \
--cc=felipe.balbi@linux.intel.com \
--cc=linux-kernel@vger.kernel.org \
--cc=lkp@intel.com \
--cc=oe-kbuild-all@lists.linux.dev \
--cc=rdunlap@infradead.org \
--cc=ruslan.bilovol@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®