On Thu, Mar 23, 2017 at 09:33:06PM +0100, Greg Kroah-Hartman wrote:
> On Thu, Mar 23, 2017 at 06:06:45PM +0100, Johan Hovold wrote:
> > On Thu, Mar 23, 2017 at 10:57:05PM +0800, kbuild test robot wrote:
> > > tree:   https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb.git 
> > > usb-testing
> > > head:   5370860a1f0e6b4197982dc1d323f988f7b1b11d
> > > commit: 279daf4e053470f22c9421a4ab05f8e5a9e9eeec [41/58] USB: core: add 
> > > helpers to retrieve endpoints in reverse order
> > > reproduce: make htmldocs
> > > 
> > > All warnings (new ones prefixed by >>):
> > > 
> > > >> drivers/usb/core/usb.c:178: warning: No description found for 
> > > >> parameter 'alt'
> > > >> drivers/usb/core/usb.c:178: warning: No description found for 
> > > >> parameter 'bulk_in'
> > > >> drivers/usb/core/usb.c:178: warning: No description found for 
> > > >> parameter 'bulk_out'
> > > >> drivers/usb/core/usb.c:178: warning: No description found for 
> > > >> parameter 'int_in'
> > > >> drivers/usb/core/usb.c:178: warning: No description found for 
> > > >> parameter 'int_out'
> > 
> > > vim +/alt +178 drivers/usb/core/usb.c
> > > 
> > >    162            return -ENXIO;
> > >    163    }
> > >    164    EXPORT_SYMBOL_GPL(usb_find_common_endpoints);
> > >    165    
> > >    166    /**
> > >    167     * usb_find_common_endpoints_reverse() -- look up common 
> > > endpoint descriptors
> > >    168     *
> > >    169     * Same as usb_find_common_endpoints(), but the endpoint 
> > > descriptors are
> > >    170     * searched in reverse order (see usb_find_common_endpoints() 
> > > for details).
> > >    171     */
> > >    172    int usb_find_common_endpoints_reverse(struct usb_host_interface 
> > > *alt,
> > >    173                    struct usb_endpoint_descriptor **bulk_in,
> > >    174                    struct usb_endpoint_descriptor **bulk_out,
> > >    175                    struct usb_endpoint_descriptor **int_in,
> > >    176                    struct usb_endpoint_descriptor **int_out)
> > >    177    {
> > 
> > Greg, do prefer I remove the kerneldoc formatting, add the missing
> > descriptions, or leave things as they are here?
> 
> Fix up the kernel doc to be the correct format would be best, that way
> it shows up correctly in the kernel documentation output.

Ok, thanks. Just sent a follow-up fix to address this.

Johan
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to