1. 09 Jul, 2020 21 commits
    • Lee Jones's avatar
      usb: typec: altmodes: displayport: Supply missing displayport.h include file · 318ebed2
      Lee Jones authored
      If the header file containing a function's prototype isn't included by
      the sourcefile containing the associated function, the build system
      complains of missing prototypes.
      
      Fixes the following W=1 kernel build warning(s):
      
       drivers/usb/typec/altmodes/displayport.c:511:5: warning: no previous prototype for ‘dp_altmode_probe’ [-Wmissing-prototypes]
       drivers/usb/typec/altmodes/displayport.c:551:6: warning: no previous prototype for ‘dp_altmode_remove’ [-Wmissing-prototypes]
      
      Cc: Heikki Krogerus <heikki.krogerus@linux.intel.com>
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200703174148.2749969-22-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      318ebed2
    • Lee Jones's avatar
      usb: typec: altmodes: displayport: File headers are not good candidates for kerneldoc · 50c9bd05
      Lee Jones authored
      Demote displayport's file header to a standard comment block.
      
      Fixes the following W=1 kernel build warning(s):
      
       drivers/usb/typec/altmodes/displayport.c:18: warning: Function parameter or member '_dp' not described in 'DP_HEADER'
       drivers/usb/typec/altmodes/displayport.c:18: warning: Function parameter or member 'cmd' not described in 'DP_HEADER'
      
      Cc: Heikki Krogerus <heikki.krogerus@linux.intel.com>
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200703174148.2749969-21-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      50c9bd05
    • Lee Jones's avatar
      usb: storage: alauda: Remove set but unchecked variable · 7565fce1
      Lee Jones authored
      The return value of alauda_get_media_status() hasn't been checked
      since the driver's inception back in 2005.  If nothing have gone
      wrong/been detected until this point, it's probably safe to just
      remove the variable.
      
      Fixes the following W=1 kernel build warning(s):
      
       drivers/usb/storage/alauda.c: In function ‘alauda_check_media’:
       drivers/usb/storage/alauda.c:456:6: warning: variable ‘rc’ set but not used [-Wunused-but-set-variable]
       456 | int rc;
       | ^~
      
      Cc: Alan Stern <stern@rowland.harvard.edu>
      Cc: Daniel Drake <dsd@gentoo.org>
      Cc: usb-storage@lists.one-eyed-alien.net
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200703174148.2749969-20-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      7565fce1
    • Lee Jones's avatar
      usb: host: xhci-debugfs: Use 'gnu_printf' format notation · 142c8bb6
      Lee Jones authored
      Fixes the following W=1 kernel build warning(s):
      
       drivers/usb/host/xhci-debugfs.c:128:2: warning: function ‘xhci_debugfs_regset’ might be a candidate for ‘gnu_printf’ format attribute [-Wsuggest-attribute=format]
       128 | vsnprintf(rgs->name, sizeof(rgs->name), fmt, args);
       | ^~~~~~~~~
      
      Cc: Mathias Nyman <mathias.nyman@intel.com>
      Cc: Lu Baolu <baolu.lu@linux.intel.com>
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200703174148.2749969-19-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      142c8bb6
    • Lee Jones's avatar
      usb: chipidea: udc: Fix a few kerneldoc issues · f2926dd5
      Lee Jones authored
      Descriptions were missing for 'ci' almost throughout.  There was
      one instance of over-documenting.  Finally one function argument
      was incorrectly documented (probably down to bitrot).
      
      Fixes the following W=1 kernel build warning(s):
      
       drivers/usb/chipidea/udc.c:245: warning: Function parameter or member 'ci' not described in 'hw_port_is_high_speed'
       drivers/usb/chipidea/udc.c:271: warning: Function parameter or member 'ci' not described in 'hw_test_and_clear_intr_active'
       drivers/usb/chipidea/udc.c:285: warning: Function parameter or member 'ci' not described in 'hw_test_and_clear_setup_guard'
       drivers/usb/chipidea/udc.c:296: warning: Function parameter or member 'ci' not described in 'hw_test_and_set_setup_guard'
       drivers/usb/chipidea/udc.c:321: warning: Function parameter or member 'ci' not described in 'hw_usb_reset'
       drivers/usb/chipidea/udc.c:628: warning: Excess function parameter 'ci' description in 'free_pending_td'
       drivers/usb/chipidea/udc.c:655: warning: Function parameter or member 'hwep' not described in '_hardware_dequeue'
       drivers/usb/chipidea/udc.c:655: warning: Excess function parameter 'gadget' description in '_hardware_dequeue'
      
      Cc: Peter Chen <Peter.Chen@nxp.com>
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200703174148.2749969-18-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      f2926dd5
    • Lee Jones's avatar
      usb: host: xhci-dbgtty: File headers are not good candidates for kerneldoc · 0e1aceca
      Lee Jones authored
      Demote xhci-dbgtty's file header to a standard comment block.
      
      Fixes the following W=1 kernel build warning(s):
      
       drivers/usb/host/xhci-dbgtty.c:19: warning: Function parameter or member 'port' not described in 'dbc_send_packet'
       drivers/usb/host/xhci-dbgtty.c:19: warning: Function parameter or member 'packet' not described in 'dbc_send_packet'
       drivers/usb/host/xhci-dbgtty.c:19: warning: Function parameter or member 'size' not described in 'dbc_send_packet'
      
      Cc: Mathias Nyman <mathias.nyman@intel.com>
      Cc: Lu Baolu <baolu.lu@linux.intel.com>
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200703174148.2749969-17-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      0e1aceca
    • Lee Jones's avatar
      usb: host: xhci-dbgcap: File headers are not good candidates for kerneldoc · e57bde58
      Lee Jones authored
      Demote xhci-dbgcap's file header to a standard comment block.
      
      Fixes the following W=1 kernel build warning(s):
      
       drivers/usb/host/xhci-dbgcap.c:20: warning: Function parameter or member 'xhci' not described in 'dbc_dma_alloc_coherent'
       drivers/usb/host/xhci-dbgcap.c:20: warning: Function parameter or member 'size' not described in 'dbc_dma_alloc_coherent'
       drivers/usb/host/xhci-dbgcap.c:20: warning: Function parameter or member 'dma_handle' not described in 'dbc_dma_alloc_coherent'
       drivers/usb/host/xhci-dbgcap.c:20: warning: Function parameter or member 'flags' not described in 'dbc_dma_alloc_coherent'
      
      Cc: Mathias Nyman <mathias.nyman@intel.com>
      Cc: Lu Baolu <baolu.lu@linux.intel.com>
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200703174148.2749969-16-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      e57bde58
    • Lee Jones's avatar
      usb: host: xhci-mem: Demote obvious misuse of kerneldoc to standard comment block · c3fa4e04
      Lee Jones authored
      No attempt has been made to document the demoted function here.
      
      Fixes the following W=1 kernel build warning(s):
      
       drivers/usb/host/xhci-mem.c:365: warning: Function parameter or member 'xhci' not described in 'xhci_ring_alloc'
       drivers/usb/host/xhci-mem.c:365: warning: Function parameter or member 'num_segs' not described in 'xhci_ring_alloc'
       drivers/usb/host/xhci-mem.c:365: warning: Function parameter or member 'cycle_state' not described in 'xhci_ring_alloc'
       drivers/usb/host/xhci-mem.c:365: warning: Function parameter or member 'type' not described in 'xhci_ring_alloc'
       drivers/usb/host/xhci-mem.c:365: warning: Function parameter or member 'max_packet' not described in 'xhci_ring_alloc'
       drivers/usb/host/xhci-mem.c:365: warning: Function parameter or member 'flags' not described in 'xhci_ring_alloc'
      
      Cc: Mathias Nyman <mathias.nyman@intel.com>
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200703174148.2749969-15-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      c3fa4e04
    • Lee Jones's avatar
      usb: host: xhci: Demote obvious misuse of kerneldoc to standard comment block · f3a9492b
      Lee Jones authored
      No attempt has been made to document the demoted function here.
      
      Fixes the following W=1 kernel build warning(s):
      
       drivers/usb/host/xhci.c:1285: warning: Function parameter or member 'desc' not described in 'xhci_get_endpoint_index'
      
      Cc: Mathias Nyman <mathias.nyman@intel.com>
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200703174148.2749969-14-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      f3a9492b
    • Lee Jones's avatar
      usb: chipidea: udc: Help out kerneldoc headers that have tried, demote the others · 92d08e07
      Lee Jones authored
      Help with adding the odd description where they have been omitted or
      where the format isn't quite right.  Demote all function headers which
      are lacking any attempt of describing their arguments.
      
      Fixes the following W=1 kernel build warning(s):
      
       drivers/usb/chipidea/udc.c:645: warning: Function parameter or member 'hwreq' not described in '_hardware_dequeue'
       drivers/usb/chipidea/udc.c:645: warning: Excess function parameter 'gadget' description in '_hardware_dequeue'
       drivers/usb/chipidea/udc.c:1326: warning: Function parameter or member 'ep' not described in 'ep_enable'
       drivers/usb/chipidea/udc.c:1326: warning: Function parameter or member 'desc' not described in 'ep_enable'
       drivers/usb/chipidea/udc.c:1393: warning: Function parameter or member 'ep' not described in 'ep_disable'
       drivers/usb/chipidea/udc.c:1433: warning: Function parameter or member 'ep' not described in 'ep_alloc_request'
       drivers/usb/chipidea/udc.c:1433: warning: Function parameter or member 'gfp_flags' not described in 'ep_alloc_request'
       drivers/usb/chipidea/udc.c:1454: warning: Function parameter or member 'ep' not described in 'ep_free_request'
       drivers/usb/chipidea/udc.c:1454: warning: Function parameter or member 'req' not described in 'ep_free_request'
       drivers/usb/chipidea/udc.c:1488: warning: Function parameter or member 'ep' not described in 'ep_queue'
       drivers/usb/chipidea/udc.c:1488: warning: Function parameter or member 'req' not described in 'ep_queue'
       drivers/usb/chipidea/udc.c:1488: warning: Function parameter or member 'gfp_flags' not described in 'ep_queue'
       drivers/usb/chipidea/udc.c:1512: warning: Function parameter or member 'ep' not described in 'ep_dequeue'
       drivers/usb/chipidea/udc.c:1512: warning: Function parameter or member 'req' not described in 'ep_dequeue'
       drivers/usb/chipidea/udc.c:1556: warning: Function parameter or member 'ep' not described in 'ep_set_halt'
       drivers/usb/chipidea/udc.c:1556: warning: Function parameter or member 'value' not described in 'ep_set_halt'
       drivers/usb/chipidea/udc.c:1566: warning: Function parameter or member 'ep' not described in 'ep_set_wedge'
       drivers/usb/chipidea/udc.c:1586: warning: Function parameter or member 'ep' not described in 'ep_fifo_flush'
       drivers/usb/chipidea/udc.c:1610: warning: cannot understand function prototype: 'const struct usb_ep_ops usb_ep_ops = '
       drivers/usb/chipidea/udc.c:1629: warning: Function parameter or member '_gadget' not described in 'ci_hdrc_gadget_connect'
       drivers/usb/chipidea/udc.c:1629: warning: Function parameter or member 'is_active' not described in 'ci_hdrc_gadget_connect'
       drivers/usb/chipidea/udc.c:1780: warning: cannot understand function prototype: 'const struct usb_gadget_ops usb_gadget_ops = '
       drivers/usb/chipidea/udc.c:1931: warning: Function parameter or member 'gadget' not described in 'ci_udc_stop'
       drivers/usb/chipidea/udc.c:1965: warning: Function parameter or member 'ci' not described in 'udc_irq'
       drivers/usb/chipidea/udc.c:2095: warning: Function parameter or member 'ci' not described in 'ci_hdrc_gadget_destroy'
       drivers/usb/chipidea/udc.c:2144: warning: Function parameter or member 'ci' not described in 'ci_hdrc_gadget_init'
      
      Cc: Peter Chen <Peter.Chen@nxp.com>
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200703174148.2749969-13-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      92d08e07
    • Lee Jones's avatar
      usb: chipidea: udc: Add missing descriptions for function arg 'ci' · 22dfe657
      Lee Jones authored
      Looks like a very popular argument to omit descriptions for.
      
      Fixes the following W=1 kernel build warning(s):
      
       drivers/usb/chipidea/udc.c:80: warning: Function parameter or member 'ci' not described in 'hw_device_state'
       drivers/usb/chipidea/udc.c:100: warning: Function parameter or member 'ci' not described in 'hw_ep_flush'
       drivers/usb/chipidea/udc.c:121: warning: Function parameter or member 'ci' not described in 'hw_ep_disable'
       drivers/usb/chipidea/udc.c:136: warning: Function parameter or member 'ci' not described in 'hw_ep_enable'
       drivers/usb/chipidea/udc.c:170: warning: Function parameter or member 'ci' not described in 'hw_ep_get_halt'
       drivers/usb/chipidea/udc.c:185: warning: Function parameter or member 'ci' not described in 'hw_ep_prime'
       drivers/usb/chipidea/udc.c:215: warning: Function parameter or member 'ci' not described in 'hw_ep_set_halt'
       drivers/usb/chipidea/udc.c:238: warning: Function parameter or member 'ci' not described in 'hw_port_is_high_speed'
       drivers/usb/chipidea/udc.c:251: warning: Function parameter or member 'ci' not described in 'hw_test_and_clear_complete'
       drivers/usb/chipidea/udc.c:263: warning: Function parameter or member 'ci' not described in 'hw_test_and_clear_intr_active'
       drivers/usb/chipidea/udc.c:277: warning: Function parameter or member 'ci' not described in 'hw_test_and_clear_setup_guard'
       drivers/usb/chipidea/udc.c:288: warning: Function parameter or member 'ci' not described in 'hw_test_and_set_setup_guard'
       drivers/usb/chipidea/udc.c:300: warning: Function parameter or member 'ci' not described in 'hw_usb_set_address'
       drivers/usb/chipidea/udc.c:312: warning: Function parameter or member 'ci' not described in 'hw_usb_reset'
      
      Cc: Peter Chen <Peter.Chen@nxp.com>
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200703174148.2749969-12-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      22dfe657
    • Lee Jones's avatar
      usb: chipidea: debug: Demote obvious misuse of kerneldoc to standard comment blocks · 5722a8ef
      Lee Jones authored
      No attempt has been made to document any of the demoted functions here.
      
      Fixes the following W=1 kernel build warning(s):
      
       drivers/usb/chipidea/debug.c:25: warning: Function parameter or member 's' not described in 'ci_device_show'
       drivers/usb/chipidea/debug.c:25: warning: Function parameter or member 'data' not described in 'ci_device_show'
       drivers/usb/chipidea/debug.c:54: warning: Function parameter or member 's' not described in 'ci_port_test_show'
       drivers/usb/chipidea/debug.c:54: warning: Function parameter or member 'data' not described in 'ci_port_test_show'
       drivers/usb/chipidea/debug.c:75: warning: Function parameter or member 'file' not described in 'ci_port_test_write'
       drivers/usb/chipidea/debug.c:75: warning: Function parameter or member 'ubuf' not described in 'ci_port_test_write'
       drivers/usb/chipidea/debug.c:75: warning: Function parameter or member 'count' not described in 'ci_port_test_write'
       drivers/usb/chipidea/debug.c:75: warning: Function parameter or member 'ppos' not described in 'ci_port_test_write'
       drivers/usb/chipidea/debug.c:122: warning: Function parameter or member 's' not described in 'ci_qheads_show'
       drivers/usb/chipidea/debug.c:122: warning: Function parameter or member 'data' not described in 'ci_qheads_show'
       drivers/usb/chipidea/debug.c:154: warning: Function parameter or member 's' not described in 'ci_requests_show'
       drivers/usb/chipidea/debug.c:154: warning: Function parameter or member 'data' not described in 'ci_requests_show'
      
      Cc: Peter Chen <Peter.Chen@nxp.com>
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200703174148.2749969-11-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      5722a8ef
    • Lee Jones's avatar
      usb: chipidea: otg: Fix kerneldoc issues relating to description of 'ci' · 6a005f0f
      Lee Jones authored
      Firstly we add some missing descriptions.  Then we fix some formatting
      issues.  Kerneldoc expects arguments to be in the format '@.*: '.  If
      either the '@' or ':' is omitted, kerneldoc gets confused.
      
      Fixes the following W=1 kernel build warning(s):
      
       drivers/usb/chipidea/otg.c:29: warning: Function parameter or member 'ci' not described in 'hw_read_otgsc'
       drivers/usb/chipidea/otg.c:82: warning: Function parameter or member 'ci' not described in 'hw_write_otgsc'
       drivers/usb/chipidea/otg.c:235: warning: Function parameter or member 'ci' not described in 'ci_hdrc_otg_init'
       drivers/usb/chipidea/otg.c:254: warning: Function parameter or member 'ci' not described in 'ci_hdrc_otg_destroy'
      
      Cc: Peter Chen <Peter.Chen@nxp.com>
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200703174148.2749969-10-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      6a005f0f
    • Lee Jones's avatar
      usb: gadget: udc: core: Fix a bunch of kerneldoc misdemeanours · e21cd08f
      Lee Jones authored
      Firstly, file headers should not be in kerneldoc format.  Function
      args should be in the format '@.*: '.  We also take the time to
      add some descriptions for various argument which have been
      previously left out.  Finally we remove descriptions for absent
      arguments.
      
      Fixes the following W=1 kernel build warning(s):
      
       drivers/usb/gadget/udc/core.c:25: warning: Incorrect use of kernel-doc format:  * struct usb_udc - describes one usb device controller
       drivers/usb/gadget/udc/core.c:36: warning: cannot understand function prototype: 'struct usb_udc '
       drivers/usb/gadget/udc/core.c:901: warning: Function parameter or member 'ep' not described in 'usb_gadget_giveback_request'
       drivers/usb/gadget/udc/core.c:901: warning: Function parameter or member 'req' not described in 'usb_gadget_giveback_request'
       drivers/usb/gadget/udc/core.c:1098: warning: Function parameter or member 'udc' not described in 'usb_gadget_udc_stop'
       drivers/usb/gadget/udc/core.c:1098: warning: Excess function parameter 'gadget' description in 'usb_gadget_udc_stop'
       drivers/usb/gadget/udc/core.c:1098: warning: Excess function parameter 'driver' description in 'usb_gadget_udc_stop'
      
      Cc: Felipe Balbi <balbi@kernel.org>
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200703174148.2749969-9-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      e21cd08f
    • Lee Jones's avatar
      usb: chipidea: core: Document hw_port_test_set()'s missing 'ci' argument · 8ac326ff
      Lee Jones authored
      Fixes the following W=1 kernel build warning(s):
      
       drivers/usb/chipidea/core.c:163: warning: Function parameter or member 'ci' not described in 'hw_port_test_set'
      
      Cc: Peter Chen <Peter.Chen@nxp.com>
      Cc: David Lopo <dlopo@chipidea.mips.com>
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200703174148.2749969-8-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      8ac326ff
    • Lee Jones's avatar
      usb: misc: adutux: Demote obvious misuse of kerneldoc to standard comment blocks · 874ae838
      Lee Jones authored
      No attempt has been made to document any of the functions here.
      
      Fixes the following W=1 kernel build warning(s):
      
       drivers/usb/misc/adutux.c:117: warning: Function parameter or member 'dev' not described in 'adu_abort_transfers'
       drivers/usb/misc/adutux.c:653: warning: Function parameter or member 'interface' not described in 'adu_probe'
       drivers/usb/misc/adutux.c:653: warning: Function parameter or member 'id' not described in 'adu_probe'
       drivers/usb/misc/adutux.c:762: warning: Function parameter or member 'interface' not described in 'adu_disconnect'
      
      Cc: Johan Hovold <johan@kernel.org>
      Cc: David Glance <davidgsf@sourceforge.net>
      Cc: Juergen Stuber <stuber@loria.fr>
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200703174148.2749969-7-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      874ae838
    • Lee Jones's avatar
      usb: atm: ueagle-atm: Demote obvious misuse of kerneldoc to standard comment blocks · b2e3fa1a
      Lee Jones authored
      No attempt has been made to document any of the functions here.
      
      Fixes the following W=1 kernel build warning(s):
      
       drivers/usb/atm/ueagle-atm.c:578: warning: Function parameter or member 'usb' not described in 'uea_send_modem_cmd'
       drivers/usb/atm/ueagle-atm.c:578: warning: Function parameter or member 'addr' not described in 'uea_send_modem_cmd'
       drivers/usb/atm/ueagle-atm.c:578: warning: Function parameter or member 'size' not described in 'uea_send_modem_cmd'
       drivers/usb/atm/ueagle-atm.c:578: warning: Function parameter or member 'buff' not described in 'uea_send_modem_cmd'
       drivers/usb/atm/ueagle-atm.c:679: warning: Function parameter or member 'usb' not described in 'uea_load_firmware'
       drivers/usb/atm/ueagle-atm.c:679: warning: Function parameter or member 'ver' not described in 'uea_load_firmware'
      
      Cc: Matthieu CASTET <castet.matthieu@free.fr>
      Cc: Stanislaw Gruszka <stf_xl@wp.pl>
      Cc: Damien Bergamini <damien.bergamini@free.fr>
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200703174148.2749969-6-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      b2e3fa1a
    • Lee Jones's avatar
      usb: core: ledtrig-usbport: Demote obvious misuse of kerneldoc to standard comment blocks · 5bef1132
      Lee Jones authored
      No attempt has been made to document any of the functions here.
      
      Fixes the following W=1 kernel build warning(s):
      
       drivers/usb/core/ledtrig-usbport.c:42: warning: Function parameter or member 'usbport_data' not described in 'usbport_trig_usb_dev_observed'
       drivers/usb/core/ledtrig-usbport.c:42: warning: Function parameter or member 'usb_dev' not described in 'usbport_trig_usb_dev_observed'
       drivers/usb/core/ledtrig-usbport.c:71: warning: Function parameter or member 'usbport_data' not described in 'usbport_trig_update_count'
       drivers/usb/core/ledtrig-usbport.c:131: warning: Function parameter or member 'usbport_data' not described in 'usbport_trig_port_observed'
       drivers/usb/core/ledtrig-usbport.c:131: warning: Function parameter or member 'usb_dev' not described in 'usbport_trig_port_observed'
       drivers/usb/core/ledtrig-usbport.c:131: warning: Function parameter or member 'port1' not described in 'usbport_trig_port_observed'
      
      Cc: Rafał Miłecki <rafal@milecki.pl>
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200703174148.2749969-4-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      5bef1132
    • Lee Jones's avatar
      usb: host: ohci-s3c2410: Demote obvious misuse of kerneldoc to standard comment block · 0459268a
      Lee Jones authored
      No attempt has been made to document the function here.
      
      Fixes the following W=1 kernel build warning(s):
      
       drivers/usb/host/ohci-s3c2410.c:356: warning: Function parameter or member 'dev' not described in 'ohci_hcd_s3c2410_probe'
      
      Cc: Alan Stern <stern@rowland.harvard.edu>
      Cc: Kukjin Kim <kgene@kernel.org>
      Cc: Krzysztof Kozlowski <krzk@kernel.org>
      Cc: Roman Weissgaerber <weissg@vienna.at>
      Cc: David Brownell <dbrownell@users.sourceforge.net>
      Cc: Christopher Hoover <ch@hpl.hp.com>
      Cc: Ben Dooks <ben@simtec.co.uk>
      Cc: linux-samsung-soc@vger.kernel.org
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200703174148.2749969-3-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      0459268a
    • Lee Jones's avatar
      usb: mtu3: mtu3_debug: Add forward declaration of 'struct ssusb_mtk' · 16eb9c0a
      Lee Jones authored
      Without it, the build system complains that it was declared inside
      the parameter list.
      
      Fixes the following W=1 kernel build warning(s):
      
       In file included from drivers/usb/mtu3/mtu3_trace.c:11:
       drivers/usb/mtu3/mtu3_debug.h:29:36: warning: ‘struct ssusb_mtk’ declared inside parameter list will not be visible outside of this definition or declaration
       29 | void ssusb_dev_debugfs_init(struct ssusb_mtk *ssusb);
       | ^~~~~~~~~
       drivers/usb/mtu3/mtu3_debug.h:30:35: warning: ‘struct ssusb_mtk’ declared inside parameter list will not be visible outside of this definition or declaration
       30 | void ssusb_dr_debugfs_init(struct ssusb_mtk *ssusb);
       | ^~~~~~~~~
       drivers/usb/mtu3/mtu3_debug.h:31:39: warning: ‘struct ssusb_mtk’ declared inside parameter list will not be visible outside of this definition or declaration
       31 | void ssusb_debugfs_create_root(struct ssusb_mtk *ssusb);
       | ^~~~~~~~~
       drivers/usb/mtu3/mtu3_debug.h:32:39: warning: ‘struct ssusb_mtk’ declared inside parameter list will not be visible outside of this definition or declaration
       32 | void ssusb_debugfs_remove_root(struct ssusb_mtk *ssusb);
       | ^~~~~~~~~
      
      Cc: Chunfeng Yun <chunfeng.yun@mediatek.com>
      Cc: linux-mediatek@lists.infradead.org
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200703174148.2749969-2-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      16eb9c0a
    • Linus Walleij's avatar
      usb: ehci-omap: Drop surplus include · acf916c2
      Linus Walleij authored
      The EHCI OMAP driver includes <linux/gpio.h> but does not use
      any symbols from this file, so drop it.
      
      Cc: Roger Quadros <rogerq@ti.com>
      Cc: Tony Lindgren <tony@atomide.com>
      Signed-off-by: default avatarLinus Walleij <linus.walleij@linaro.org>
      Reviewed-by: default avatarPeter Ujfalusi <peter.ujfalusi@ti.com>
      Link: https://lore.kernel.org/r/20200703131059.515436-1-linus.walleij@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      acf916c2
  2. 07 Jul, 2020 1 commit
  3. 03 Jul, 2020 18 commits
    • Lee Jones's avatar
      usb: dwc2: gadget: Remove assigned but never used 'maxsize' · 768a0741
      Lee Jones authored
      The value of 'maxsize' has not been checked since commit 729cac69
      ("usb: dwc2: Change ISOC DDMA flow") back in 2018, so remove the set but
      unused variable, which fixes the following W=1 kernel build warning:
      
       drivers/usb/dwc2/gadget.c: In function ‘dwc2_gadget_fill_isoc_desc’:
       drivers/usb/dwc2/gadget.c:885:6: warning: variable ‘maxsize’ set but not used [-Wunused-but-set-variable]
       885 | u32 maxsize = 0;
       | ^~~~~~~
      
      Cc: Minas Harutyunyan <hminas@synopsys.com>
      Cc: Ben Dooks <ben@simtec.co.uk>
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200702144625.2533530-11-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      768a0741
    • Greg Kroah-Hartman's avatar
      USB: phy: fsl-usb: remove character device usage · 8c4a09cb
      Greg Kroah-Hartman authored
      No idea why this driver is using a char device node, statically
      allocated, with no dynamic allocation or hook up with devtmpfs, along
      with a reserverd major number, for "special" operations, not all of
      which ever were implemented.
      
      So just rip it out, as no one must be using it because no modern system
      will ever actually create the /dev/ node it needs.
      
      Cc: Felipe Balbi <balbi@kernel.org>
      Reviewed-by: default avatarRan Wang <ran.wang_1@nxp.com>
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      Link: https://lore.kernel.org/r/20200702072914.1072878-2-gregkh@linuxfoundation.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      8c4a09cb
    • Greg Kroah-Hartman's avatar
      USB: phy: fsl-usb: remove sysfs abuse · cc72a2ca
      Greg Kroah-Hartman authored
      This file has a HUGE debugging sysfs file that spews out a lot of
      information all at once, which violates the one-value-per-file rule for
      sysfs.  If this is really needed, it should go into debugfs, but given
      the age of this driver, I strongly doubt anyone is using it anymore.
      
      So just remove the file entirely, it was never documented, so obviously,
      no one actually needed it :)
      
      Cc: Felipe Balbi <balbi@kernel.org>
      Reviewed-by: default avatarRan Wang <ran.wang_1@nxp.com>
      Link: https://lore.kernel.org/r/20200702072914.1072878-1-gregkh@linuxfoundation.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      cc72a2ca
    • Peter Chen's avatar
    • Lee Jones's avatar
      usb: host: oxu210hp-hcd: Move declaration of 'qtd' into 'ifdef OXU_URB_TRACE' · 6181aa1b
      Lee Jones authored
      If we assign 'epnum' during the declaration we can also avoid "ISO
      C90 forbids mixed declarations" issues.  So it does looks like we
      can have our cake and eat it in this scenario.
      
      Fixes the following W=1 kernel build warning(s):
      
       drivers/usb/host/oxu210hp-hcd.c: In function ‘submit_async’:
       drivers/usb/host/oxu210hp-hcd.c:2040:19: warning: variable ‘qtd’ set but not used [-Wunused-but-set-variable]
       2040 | struct ehci_qtd *qtd;
       | ^~~
      
      Cc: Masahiro Yamada <yamada.masahiro@socionext.com>
      Cc: Rodolfo Giometti <giometti@linux.it>
      Cc: "Eurotech S.p.A" <info@eurtech.it>
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200702144625.2533530-24-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      6181aa1b
    • Lee Jones's avatar
      usb: c67x00: c67x00-sched: Demote obvious misuse of kerneldoc to standard comment blocks · aa88cddf
      Lee Jones authored
      No attempt has been made to document any of the functions here.
      
      Fixes the following W=1 kernel build warning(s):
      
       drivers/usb/c67x00/c67x00-sched.c:35: warning: Function parameter or member 'queue' not described in 'c67x00_ep_data'
       drivers/usb/c67x00/c67x00-sched.c:35: warning: Function parameter or member 'node' not described in 'c67x00_ep_data'
       drivers/usb/c67x00/c67x00-sched.c:35: warning: Function parameter or member 'hep' not described in 'c67x00_ep_data'
       drivers/usb/c67x00/c67x00-sched.c:35: warning: Function parameter or member 'dev' not described in 'c67x00_ep_data'
       drivers/usb/c67x00/c67x00-sched.c:35: warning: Function parameter or member 'next_frame' not described in 'c67x00_ep_data'
       drivers/usb/c67x00/c67x00-sched.c:71: warning: Function parameter or member 'ly_base_addr' not described in 'c67x00_td'
       drivers/usb/c67x00/c67x00-sched.c:71: warning: Function parameter or member 'port_length' not described in 'c67x00_td'
       drivers/usb/c67x00/c67x00-sched.c:71: warning: Function parameter or member 'pid_ep' not described in 'c67x00_td'
       drivers/usb/c67x00/c67x00-sched.c:71: warning: Function parameter or member 'dev_addr' not described in 'c67x00_td'
       drivers/usb/c67x00/c67x00-sched.c:71: warning: Function parameter or member 'ctrl_reg' not described in 'c67x00_td'
       drivers/usb/c67x00/c67x00-sched.c:71: warning: Function parameter or member 'status' not described in 'c67x00_td'
       drivers/usb/c67x00/c67x00-sched.c:71: warning: Function parameter or member 'retry_cnt' not described in 'c67x00_td'
       drivers/usb/c67x00/c67x00-sched.c:71: warning: Function parameter or member 'residue' not described in 'c67x00_td'
       drivers/usb/c67x00/c67x00-sched.c:71: warning: Function parameter or member 'next_td_addr' not described in 'c67x00_td'
       drivers/usb/c67x00/c67x00-sched.c:71: warning: Function parameter or member 'td_list' not described in 'c67x00_td'
       drivers/usb/c67x00/c67x00-sched.c:71: warning: Function parameter or member 'td_addr' not described in 'c67x00_td'
       drivers/usb/c67x00/c67x00-sched.c:71: warning: Function parameter or member 'data' not described in 'c67x00_td'
       drivers/usb/c67x00/c67x00-sched.c:71: warning: Function parameter or member 'urb' not described in 'c67x00_td'
       drivers/usb/c67x00/c67x00-sched.c:71: warning: Function parameter or member 'privdata' not described in 'c67x00_td'
       drivers/usb/c67x00/c67x00-sched.c:71: warning: Function parameter or member 'ep_data' not described in 'c67x00_td'
       drivers/usb/c67x00/c67x00-sched.c:71: warning: Function parameter or member 'pipe' not described in 'c67x00_td'
       drivers/usb/c67x00/c67x00-sched.c:137: warning: Function parameter or member 'c67x00' not described in 'dbg_td'
       drivers/usb/c67x00/c67x00-sched.c:137: warning: Function parameter or member 'td' not described in 'dbg_td'
       drivers/usb/c67x00/c67x00-sched.c:137: warning: Function parameter or member 'msg' not described in 'dbg_td'
       drivers/usb/c67x00/c67x00-sched.c:169: warning: Function parameter or member 'a' not described in 'frame_add'
       drivers/usb/c67x00/c67x00-sched.c:169: warning: Function parameter or member 'b' not described in 'frame_add'
       drivers/usb/c67x00/c67x00-sched.c:177: warning: Function parameter or member 'a' not described in 'frame_after'
       drivers/usb/c67x00/c67x00-sched.c:177: warning: Function parameter or member 'b' not described in 'frame_after'
       drivers/usb/c67x00/c67x00-sched.c:186: warning: Function parameter or member 'a' not described in 'frame_after_eq'
       drivers/usb/c67x00/c67x00-sched.c:186: warning: Function parameter or member 'b' not described in 'frame_after_eq'
       drivers/usb/c67x00/c67x00-sched.c:199: warning: Function parameter or member 'c67x00' not described in 'c67x00_release_urb'
       drivers/usb/c67x00/c67x00-sched.c:199: warning: Function parameter or member 'urb' not described in 'c67x00_release_urb'
       drivers/usb/c67x00/c67x00-sched.c:566: warning: Function parameter or member 'c67x00' not described in 'c67x00_create_td'
       drivers/usb/c67x00/c67x00-sched.c:566: warning: Function parameter or member 'urb' not described in 'c67x00_create_td'
       drivers/usb/c67x00/c67x00-sched.c:566: warning: Function parameter or member 'data' not described in 'c67x00_create_td'
       drivers/usb/c67x00/c67x00-sched.c:566: warning: Function parameter or member 'len' not described in 'c67x00_create_td'
       drivers/usb/c67x00/c67x00-sched.c:566: warning: Function parameter or member 'pid' not described in 'c67x00_create_td'
       drivers/usb/c67x00/c67x00-sched.c:566: warning: Function parameter or member 'toggle' not described in 'c67x00_create_td'
       drivers/usb/c67x00/c67x00-sched.c:566: warning: Function parameter or member 'privdata' not described in 'c67x00_create_td'
       drivers/usb/c67x00/c67x00-sched.c:692: warning: Function parameter or member 'c67x00' not described in 'c67x00_add_ctrl_urb'
       drivers/usb/c67x00/c67x00-sched.c:692: warning: Function parameter or member 'urb' not described in 'c67x00_add_ctrl_urb'
       drivers/usb/c67x00/c67x00-sched.c:830: warning: Function parameter or member 'c67x00' not described in 'c67x00_parse_td'
       drivers/usb/c67x00/c67x00-sched.c:830: warning: Function parameter or member 'td' not described in 'c67x00_parse_td'
       drivers/usb/c67x00/c67x00-sched.c:978: warning: Function parameter or member 'c67x00' not described in 'c67x00_check_td_list'
       drivers/usb/c67x00/c67x00-sched.c:1052: warning: Function parameter or member 'c67x00' not described in 'c67x00_send_td'
       drivers/usb/c67x00/c67x00-sched.c:1052: warning: Function parameter or member 'td' not described in 'c67x00_send_td'
       drivers/usb/c67x00/c67x00-sched.c:1088: warning: Function parameter or member 'c67x00' not described in 'c67x00_do_work'
      
      Cc: Peter Korsgaard <jacmet@sunsite.dk>
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200702144625.2533530-31-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      aa88cddf
    • Lee Jones's avatar
      usb: class: usbtmc: File headers are not good candidates for kerneldoc · aa37c246
      Lee Jones authored
      Demote usbtmc's file header to a standard comment block.
      
      Fixes the following W=1 kernel build warning(s):
      
       drivers/usb/class/usbtmc.c:11: warning: Function parameter or member 'fmt' not described in 'pr_fmt'
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200702144625.2533530-30-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      aa37c246
    • Lee Jones's avatar
      usb: c67x00: c67x00-ll-hpi: Demote obvious misuse of kerneldoc to standard comment blocks · b5993881
      Lee Jones authored
      No attempt has been made to document any of the functions here.
      
      Fixes the following W=1 kernel build warning(s):
      
       drivers/usb/c67x00/c67x00-ll-hpi.c:269: warning: Function parameter or member 'sie' not described in 'c67x00_ll_usb_clear_status'
       drivers/usb/c67x00/c67x00-ll-hpi.c:269: warning: Function parameter or member 'bits' not described in 'c67x00_ll_usb_clear_status'
       drivers/usb/c67x00/c67x00-ll-hpi.c:404: warning: Function parameter or member 'dev' not described in 'c67x00_ll_write_mem_le16'
       drivers/usb/c67x00/c67x00-ll-hpi.c:404: warning: Function parameter or member 'addr' not described in 'c67x00_ll_write_mem_le16'
       drivers/usb/c67x00/c67x00-ll-hpi.c:404: warning: Function parameter or member 'data' not described in 'c67x00_ll_write_mem_le16'
       drivers/usb/c67x00/c67x00-ll-hpi.c:404: warning: Function parameter or member 'len' not described in 'c67x00_ll_write_mem_le16'
       drivers/usb/c67x00/c67x00-ll-hpi.c:443: warning: Function parameter or member 'dev' not described in 'c67x00_ll_read_mem_le16'
       drivers/usb/c67x00/c67x00-ll-hpi.c:443: warning: Function parameter or member 'addr' not described in 'c67x00_ll_read_mem_le16'
       drivers/usb/c67x00/c67x00-ll-hpi.c:443: warning: Function parameter or member 'data' not described in 'c67x00_ll_read_mem_le16'
       drivers/usb/c67x00/c67x00-ll-hpi.c:443: warning: Function parameter or member 'len' not described in 'c67x00_ll_read_mem_le16'
      
      Cc: Peter Korsgaard <jacmet@sunsite.dk>
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200702144625.2533530-27-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      b5993881
    • Lee Jones's avatar
      usb: mtu3: mtu3_core: Demote obvious misuse of kerneldoc to standard comment block · 8709e367
      Lee Jones authored
      No attempt has been made to document the function.
      
      Fixes the following W=1 kernel build warning(s):
      
       drivers/usb/mtu3/mtu3_core.c:805: warning: Function parameter or member 'mtu' not described in 'mtu3_set_dma_mask'
      
      Cc: Chunfeng Yun <chunfeng.yun@mediatek.com>
      Cc: linux-arm-kernel@lists.infradead.org
      Cc: linux-mediatek@lists.infradead.org
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200702144625.2533530-26-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      8709e367
    • Lee Jones's avatar
      usb: cdns3: ep0: Move 'zlp' description to appropriate function header · ff5e445e
      Lee Jones authored
      'zlp' was documented, but in the wrong place.
      
      Fixes the following W=1 kernel build warning(s):
      
       drivers/usb/cdns3/ep0.c:36: warning: Function parameter or member 'zlp' not described in 'cdns3_ep0_run_transfer'
       drivers/usb/cdns3/ep0.c:705: warning: Excess function parameter 'zlp' description in 'cdns3_gadget_ep0_queue'
      
      Cc: Pawel Laszczak <pawell@cadence.com>
      Cc: Pawel Jez <pjez@cadence.com>
      Cc: Peter Chen <peter.chen@nxp.com>
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200702144625.2533530-23-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      ff5e445e
    • Lee Jones's avatar
      usb: host: ohci: Mark cc_to_error as __maybe_unused · 9dac16e4
      Lee Jones authored
      drivers/usb/host/ohci.h in included by a whole much of different
      sourcefiles.  Not all of them make use of cc_to_error.  So mark
      it as __maybe_used to convey that this behaviour is not only
      acceptable it's expected.
      
      Fixes the following kernel build warning(s):
      
       In file included from drivers/usb/host/ohci-pci.c:25:
       drivers/usb/host/ohci.h:165:18: warning: ‘cc_to_error’ defined but not used [-Wunused-const-variable=]
       165 | static const int cc_to_error 1 = {
       | ^~~~~~~~~~~
       In file included from drivers/usb/host/ohci-platform.c:32:
       drivers/usb/host/ohci.h:165:18: warning: ‘cc_to_error’ defined but not used [-Wunused-const-variable=]
       165 | static const int cc_to_error 1 = {
       | ^~~~~~~~~~~
       In file included from drivers/usb/host/ohci-exynos.c:20:
       drivers/usb/host/ohci.h:165:18: warning: ‘cc_to_error’ defined but not used [-Wunused-const-variable=]
       165 | static const int cc_to_error 1 = {
       | ^~~~~~~~~~~
       In file included from drivers/usb/host/ohci-spear.c:22:
       drivers/usb/host/ohci.h:165:18: warning: ‘cc_to_error’ defined but not used [-Wunused-const-variable=]
       165 | static const int cc_to_error 1 = {
       | ^~~~~~~~~~~
       In file included from drivers/usb/host/ohci-st.c:26:
       drivers/usb/host/ohci.h:165:18: warning: ‘cc_to_error’ defined but not used [-Wunused-const-variable=]
       165 | static const int cc_to_error 1 = {
       | ^~~~~~~~~~~
       In file included from drivers/usb/host/ohci-at91.c:31:
       drivers/usb/host/ohci.h:165:18: warning: ‘cc_to_error’ defined but not used [-Wunused-const-variable=]
       165 | static const int cc_to_error 1 = {
       | ^~~~~~~~~~~
       In file included from drivers/usb/host/ohci-s3c2410.c:32:
       drivers/usb/host/ohci.h:165:18: warning: ‘cc_to_error’ defined but not used [-Wunused-const-variable=]
       165 | static const int cc_to_error 1 = {
       | ^~~~~~~~~~~
      
      Cc: Alan Stern <stern@rowland.harvard.edu>
      Cc: Roman Weissgaerber <weissg@vienna.at>
      Cc: David Brownell <dbrownell@users.sourceforge.net>
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200702144625.2533530-22-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      9dac16e4
    • Lee Jones's avatar
      usb: host: ohci-at91: Demote kerneldoc headers down to basic comment blocks · f73a8db0
      Lee Jones authored
      Neither usb_hcd_at91_probe() nor usb_hcd_at91_remove()'s function
      headers reach the standards required of a kerneldoc entry.  Only
      one attempt at describing an argument was made, as it is not
      correct (we're removing that too to save confusion).
      
      Fixes the following kernel build warning(s):
      
       drivers/usb/host/ohci-at91.c:166: warning: Function parameter or member 'driver' not described in 'usb_hcd_at91_probe'
       drivers/usb/host/ohci-at91.c:166: warning: Function parameter or member 'pdev' not described in 'usb_hcd_at91_probe'
       drivers/usb/host/ohci-at91.c:259: warning: Function parameter or member 'hcd' not described in 'usb_hcd_at91_remove'
       drivers/usb/host/ohci-at91.c:259: warning: Function parameter or member 'pdev' not described in 'usb_hcd_at91_remove'
       drivers/usb/host/ohci-at91.c:259: warning: Excess function parameter 'dev' description in 'usb_hcd_at91_remove'
      
      Cc: Alan Stern <stern@rowland.harvard.edu>
      Cc: Nicolas Ferre <nicolas.ferre@microchip.com>
      Cc: Alexandre Belloni <alexandre.belloni@bootlin.com>
      Cc: Ludovic Desroches <ludovic.desroches@microchip.com>
      Cc: Thibaut VARENE <varenet@parisc-linux.org>
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200702144625.2533530-21-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      f73a8db0
    • Lee Jones's avatar
      usb: host: isp1362: Mark the many unused ISP1362_REG entries as __maybe_unused · 048715c0
      Lee Jones authored
      It would seem a shame to strip out all of the unused register entries,
      since they can act as a fair source of documentation.  Instead, mark
      them all as __maybe_unused to show the build system that this behaviour
      is known and intentional.
      
      Fixes the following kernel build warning(s):
      
       In file included from drivers/usb/host/isp1362-hcd.c:96:
       drivers/usb/host/isp1362.h:59:22: warning: ‘ISP1362_REG_OTGALTTMR’ defined but not used [-Wunused-const-variable=]
       59 | static isp1362_reg_t ISP1362_REG_##name = addr
       | ^~~~~~~~~~~~
       drivers/usb/host/isp1362.h:199:1: note: in expansion of macro ‘ISP1362_REG’
       199 | ISP1362_REG(OTGALTTMR, 0x6C, REG_WIDTH_16, REG_ACCESS_RW);
       | ^~~~~~~~~~~
       drivers/usb/host/isp1362.h:59:22: warning: ‘ISP1362_REG_OTGTIMER’ defined but not used [-Wunused-const-variable=]
       59 | static isp1362_reg_t ISP1362_REG_##name = addr
       | ^~~~~~~~~~~~
       drivers/usb/host/isp1362.h:198:1: note: in expansion of macro ‘ISP1362_REG’
       198 | ISP1362_REG(OTGTIMER, 0x6A, REG_WIDTH_16, REG_ACCESS_RW);
       | ^~~~~~~~~~~
       drivers/usb/host/isp1362.h:59:22: warning: ‘ISP1362_REG_OTGINTENB’ defined but not used [-Wunused-const-variable=]
       59 | static isp1362_reg_t ISP1362_REG_##name = addr
       | ^~~~~~~~~~~~
       drivers/usb/host/isp1362.h:197:1: note: in expansion of macro ‘ISP1362_REG’
       197 | ISP1362_REG(OTGINTENB, 0x69, REG_WIDTH_16, REG_ACCESS_RW);
       | ^~~~~~~~~~~
       drivers/usb/host/isp1362.h:59:22: warning: ‘ISP1362_REG_OTGINT’ defined but not used [-Wunused-const-variable=]
       59 | static isp1362_reg_t ISP1362_REG_##name = addr
       | ^~~~~~~~~~~~
       drivers/usb/host/isp1362.h:196:1: note: in expansion of macro ‘ISP1362_REG’
       196 | ISP1362_REG(OTGINT, 0x68, REG_WIDTH_16, REG_ACCESS_RW);
       | ^~~~~~~~~~~
       drivers/usb/host/isp1362.h:59:22: warning: ‘ISP1362_REG_OTGSTATUS’ defined but not used [-Wunused-const-variable=]
       59 | static isp1362_reg_t ISP1362_REG_##name = addr
       | ^~~~~~~~~~~~
       drivers/usb/host/isp1362.h:195:1: note: in expansion of macro ‘ISP1362_REG’
       195 | ISP1362_REG(OTGSTATUS, 0x67, REG_WIDTH_16, REG_ACCESS_R);
       | ^~~~~~~~~~~
       drivers/usb/host/isp1362.h:59:22: warning: ‘ISP1362_REG_OTGCONTROL’ defined but not used [-Wunused-const-variable=]
       59 | static isp1362_reg_t ISP1362_REG_##name = addr
       | ^~~~~~~~~~~~
       drivers/usb/host/isp1362.h:194:1: note: in expansion of macro ‘ISP1362_REG’
       194 | ISP1362_REG(OTGCONTROL, 0x62, REG_WIDTH_16, REG_ACCESS_RW);
       | ^~~~~~~~~~~
       drivers/usb/host/isp1362.h:59:22: warning: ‘ISP1362_REG_HCATLPORT’ defined but not used [-Wunused-const-variable=]
       59 | static isp1362_reg_t ISP1362_REG_##name = addr
       | ^~~~~~~~~~~~
       drivers/usb/host/isp1362.h:183:1: note: in expansion of macro ‘ISP1362_REG’
       183 | ISP1362_REG(HCATLPORT, 0x44, REG_WIDTH_16, REG_ACCESS_RW);
       | ^~~~~~~~~~~
       drivers/usb/host/isp1362.h:59:22: warning: ‘ISP1362_REG_HCINTLPORT’ defined but not used [-Wunused-const-variable=]
       59 | static isp1362_reg_t ISP1362_REG_##name = addr
       | ^~~~~~~~~~~~
       drivers/usb/host/isp1362.h:175:1: note: in expansion of macro ‘ISP1362_REG’
       175 | ISP1362_REG(HCINTLPORT, 0x43, REG_WIDTH_16, REG_ACCESS_RW);
       | ^~~~~~~~~~~
       drivers/usb/host/isp1362.h:59:22: warning: ‘ISP1362_REG_HCISTL1PORT’ defined but not used [-Wunused-const-variable=]
       59 | static isp1362_reg_t ISP1362_REG_##name = addr
       | ^~~~~~~~~~~~
       drivers/usb/host/isp1362.h:171:1: note: in expansion of macro ‘ISP1362_REG’
       171 | ISP1362_REG(HCISTL1PORT, 0x42, REG_WIDTH_16, REG_ACCESS_RW);
       | ^~~~~~~~~~~
       drivers/usb/host/isp1362.h:59:22: warning: ‘ISP1362_REG_HCISTL0PORT’ defined but not used [-Wunused-const-variable=]
       59 | static isp1362_reg_t ISP1362_REG_##name = addr
       | ^~~~~~~~~~~~
       drivers/usb/host/isp1362.h:170:1: note: in expansion of macro ‘ISP1362_REG’
       170 | ISP1362_REG(HCISTL0PORT, 0x40, REG_WIDTH_16, REG_ACCESS_RW);
       | ^~~~~~~~~~~
       drivers/usb/host/isp1362.h:59:22: warning: ‘ISP1362_REG_HCINTDIS’ defined but not used [-Wunused-const-variable=]
       59 | static isp1362_reg_t ISP1362_REG_##name = addr
       | ^~~~~~~~~~~~
       drivers/usb/host/isp1362.h:77:1: note: in expansion of macro ‘ISP1362_REG’
       77 | ISP1362_REG(HCINTDIS, 0x05, REG_WIDTH_32, REG_ACCESS_RW);
       | ^~~~~~~~~~~
      
      Cc: "by L. Wassmann" <LW@KARO-electronics.de>
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200702144625.2533530-20-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      048715c0
    • Lee Jones's avatar
      usb: dwc3: dwc3-of-simple: Function headers are not good candidates for kerneldoc · 26ef796f
      Lee Jones authored
      Fixes the following kernel build warning(s):
      
       drivers/usb/dwc3/dwc3-of-simple.c:25: warning: cannot understand function prototype: 'struct dwc3_of_simple '
      
      Cc: Felipe Balbi <balbi@kernel.org>
      Cc: iivanov@mm-sol.com
      Cc: Sundeep Bhatta <subbaraya.sundeep.bhatta@xilinx.com>
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200702144625.2533530-19-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      26ef796f
    • Lee Jones's avatar
      usb: cdns3: gadget: Fix a bunch of kernel doc issues · 4a35aa6d
      Lee Jones authored
      Mainline misspelled function argument descriptions.  Also one
      formatting issue with a missing '@' identifier.
      
      Fixes the following W=1 build warnings:
      
       drivers/usb/cdns3/gadget.c:653: warning: Function parameter or member 'priv_ep' not described in 'cdns3_wa2_descmissing_packet'
       drivers/usb/cdns3/gadget.c:653: warning: Excess function parameter 'priv_dev' description in 'cdns3_wa2_descmissing_packet'
       drivers/usb/cdns3/gadget.c:1088: warning: Function parameter or member 'request' not described in 'cdns3_ep_run_transfer'
       drivers/usb/cdns3/gadget.c:2574: warning: Function parameter or member 'priv_ep' not described in '__cdns3_gadget_ep_set_halt'
       drivers/usb/cdns3/gadget.c:2574: warning: Excess function parameter 'ep' description in '__cdns3_gadget_ep_set_halt'
       drivers/usb/cdns3/gadget.c:2595: warning: Function parameter or member 'priv_ep' not described in '__cdns3_gadget_ep_clear_halt'
       drivers/usb/cdns3/gadget.c:2595: warning: Excess function parameter 'ep' description in '__cdns3_gadget_ep_clear_halt'
       drivers/usb/cdns3/gadget.c:2898: warning: Function parameter or member 'priv_dev' not described in 'cdns3_init_eps'
       drivers/usb/cdns3/gadget.c:2898: warning: Excess function parameter 'cdns3' description in 'cdns3_init_eps'
       drivers/usb/cdns3/gadget.c:3210: warning: Function parameter or member 'cdns' not described in 'cdns3_gadget_init'
      
      Cc: Pawel Laszczak <pawell@cadence.com>
      Cc: Pawel Jez <pjez@cadence.com>
      Cc: Peter Chen <peter.chen@nxp.com>
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200702144625.2533530-18-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      4a35aa6d
    • Lee Jones's avatar
      usb: dwc3: dwc3-haps: Function headers are not suitable for kerneldoc · 4d72cf0c
      Lee Jones authored
      Fixes the following W=1 kernel build warnings:
      
       drivers/usb/dwc3/dwc3-haps.c:19: warning: Incorrect use of kernel-doc format:  * struct dwc3_haps - Driver private structure
       drivers/usb/dwc3/dwc3-haps.c:23: warning: cannot understand function prototype: 'struct dwc3_haps '
      
      Cc: Felipe Balbi <balbi@kernel.org>
      Cc: Thinh Nguyen <thinhn@synopsys.com>
      Cc: John Youn <johnyoun@synopsys.com>
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200702144625.2533530-16-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      4d72cf0c
    • Lee Jones's avatar
      usb: cdns3: ep0: Fix a bunch of kerneldoc issues · 9293b7db
      Lee Jones authored
      Add some missing function argument descriptions for 'ep', 'desc'
      and 'zlp', and correct spelling/bitrot issues surrounding the
      correct spelling of present args.
      
      Fixes the following W=1 warnings:
      
       drivers/usb/cdns3/ep0.c:36: warning: Function parameter or member 'zlp' not described in 'cdns3_ep0_run_transfer'
       drivers/usb/cdns3/ep0.c:236: warning: Function parameter or member 'ctrl' not described in 'cdns3_req_ep0_get_status'
       drivers/usb/cdns3/ep0.c:236: warning: Excess function parameter 'ctrl_req' description in 'cdns3_req_ep0_get_status'
       drivers/usb/cdns3/ep0.c:411: warning: Function parameter or member 'ctrl' not described in 'cdns3_req_ep0_handle_feature'
       drivers/usb/cdns3/ep0.c:411: warning: Excess function parameter 'ctrl_req' description in 'cdns3_req_ep0_handle_feature'
       drivers/usb/cdns3/ep0.c:661: warning: Function parameter or member 'ep' not described in 'cdns3_gadget_ep0_enable'
       drivers/usb/cdns3/ep0.c:661: warning: Function parameter or member 'desc' not described in 'cdns3_gadget_ep0_enable'
       drivers/usb/cdns3/ep0.c:671: warning: Function parameter or member 'ep' not described in 'cdns3_gadget_ep0_disable'
       drivers/usb/cdns3/ep0.c:867: warning: Function parameter or member 'priv_ep' not described in 'cdns3_init_ep0'
       drivers/usb/cdns3/ep0.c:867: warning: Excess function parameter 'ep_priv' description in 'cdns3_init_ep0'
      
      Cc: Pawel Laszczak <pawell@cadence.com>
      Cc: Pawel Jez <pjez@cadence.com>
      Cc: Peter Chen <peter.chen@nxp.com>
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200702144625.2533530-15-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      9293b7db
    • Lee Jones's avatar
      usb: cdns3: core: Fix incorrect formatting and misspelled function arg docs · 6e29619f
      Lee Jones authored
      There are 3 misspellings and 1 incorrect format used in this file's
      kerneldoc function headers.  Fixing them squashes the following W=1
      kernel build warnings:
      
       drivers/usb/cdns3/gadget.c:653: warning: Function parameter or member 'priv_ep' not described in 'cdns3_wa2_descmissing_packet'
       drivers/usb/cdns3/gadget.c:653: warning: Excess function parameter 'priv_dev' description in 'cdns3_wa2_descmissing_packet'
       drivers/usb/cdns3/gadget.c:1088: warning: Function parameter or member 'request' not described in 'cdns3_ep_run_transfer'
       drivers/usb/cdns3/gadget.c:2574: warning: Function parameter or member 'priv_ep' not described in '__cdns3_gadget_ep_set_halt'
       drivers/usb/cdns3/gadget.c:2574: warning: Excess function parameter 'ep' description in '__cdns3_gadget_ep_set_halt'
       drivers/usb/cdns3/gadget.c:2595: warning: Function parameter or member 'priv_ep' not described in '__cdns3_gadget_ep_clear_halt'
       drivers/usb/cdns3/gadget.c:2595: warning: Excess function parameter 'ep' description in '__cdns3_gadget_ep_clear_halt'
       drivers/usb/cdns3/gadget.c:2898: warning: Function parameter or member 'priv_dev' not described in 'cdns3_init_eps'
       drivers/usb/cdns3/gadget.c:2898: warning: Excess function parameter 'cdns3' description in 'cdns3_init_eps'
       drivers/usb/cdns3/gadget.c:3210: warning: Function parameter or member 'cdns' not described in 'cdns3_gadget_init'
      
      Cc: Pawel Laszczak <pawell@cadence.com>
      Cc: Peter Chen <peter.chen@nxp.com>
      Cc: Roger Quadros <rogerq@ti.com>
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200702144625.2533530-14-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      6e29619f