Re: [PATCH 00/30] Fix a bunch of W=1 issues in USB

From: Greg KH
Date: Fri Jul 03 2020 - 03:37:23 EST


On Thu, Jul 02, 2020 at 03:45:55PM +0100, Lee Jones wrote:
> This set is part of a larger effort attempting to clean-up W=1
> kernel builds, which are currently overwhelmingly riddled with
> niggly little warnings.
>
> Lee Jones (30):
> usb: phy: phy: Fix-up a whole bunch of formatting issues
> usb: host: pci-quirks: Demote function header from kerneldoc to
> comment block
> usb: common: debug: Demote comment blocks which are obviously not
> kerneldoc
> usb: common: usb-conn-gpio: Demote comment block which is clearly not
> kerneldoc
> usb: dwc3: drd: File headers are not doc headers
> usb: dwc3: ulpi: File headers are not doc headers
> usb: common: ulpi: Fix a few kerneldoc related issues
> usb: dwc3: dwc3-omap: Do not read DMA status
> usb: dwc2: gadget: Do not read GINTMSK2
> usb: dwc2: gadget: Remove assigned but never used 'maxsize'
> usb: dwc2: gadget: Avoid pointless read of EP control register
> usb: host: ehci-omap: Provide documentation for
> ehci_hcd_omap_probe()'s arg 'pdev'
> usb: cdns3: core: Fix incorrect formatting and misspelled function arg
> docs
> usb: cdns3: ep0: Fix a bunch of kerneldoc issues
> usb: dwc3: dwc3-haps: Function headers are not suitable for kerneldoc
> usb: mtu3: mtu3_trace: Function headers are not suitable for kerneldoc
> usb: cdns3: gadget: Fix a bunch of kernel doc issues
> usb: dwc3: dwc3-of-simple: Function headers are not good candidates
> for kerneldoc
> usb: host: isp1362: Mark the many unused ISP1362_REG entries as
> __maybe_unused
> usb: host: ohci-at91: Demote kerneldoc headers down to basic comment
> blocks
> usb: host: ohci: Mark cc_to_error as __maybe_unused
> usb: cdns3: ep0: Move 'zlp' description to appropriate function header
> usb: host: oxu210hp-hcd: Move declaration of 'qtd' into 'ifdef
> OXU_URB_TRACE'
> usb: mtu3: mtu3_trace: Supply missing mtu3_debug.h include file
> usb: mtu3: mtu3_core: Demote obvious misuse of kerneldoc to standard
> comment block
> usb: c67x00: c67x00-ll-hpi: Demote obvious misuse of kerneldoc to
> standard comment blocks
> usb: class: cdc-wdm: Provide description for usb_cdc_wdm_register()'s
> manage_power arg
> usb: c67x00: c67x00-hcd: Demote obvious misuse of kerneldoc to
> standard comment blocks
> usb: class: usbtmc: File headers are not good candidates for kerneldoc
> usb: c67x00: c67x00-sched: Demote obvious misuse of kerneldoc to
> standard comment blocks

I've applied 25 of these, if you could rebase and resend the remaining
ones, after fixing up the review comments made on them, that would be
wonderful.

thanks,

greg k-h