1. 09 Jul, 2020 5 commits
  2. 07 Jul, 2020 1 commit
  3. 03 Jul, 2020 27 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
    • Lee Jones's avatar
      usb: host: ehci-omap: Provide documentation for ehci_hcd_omap_probe()'s arg 'pdev' · ede175a5
      Lee Jones authored
      Description for 'pdev' argument was missing from the function header.
      
      Fixes the following W=1 warning:
      
       drivers/usb/host/ehci-omap.c:87: warning: Function parameter or member 'pdev' not described in 'ehci_hcd_omap_probe'
      
      Cc: Alan Stern <stern@rowland.harvard.edu>
      Cc: Vikram Pandita <vikram.pandita@ti.com>
      Cc: Anand Gadiyar <gadiyar@ti.com>
      Cc: Keshava Munegowda <keshava_mgowda@ti.com>
      Cc: Roger Quadros <rogerq@ti.com>
      Cc: Felipe Balbi <felipe.balbi@nokia.com>
      Cc: linux-omap@vger.kernel.org
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200702144625.2533530-13-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      ede175a5
    • Lee Jones's avatar
      usb: dwc3: dwc3-omap: Do not read DMA status · 3c373454
      Lee Jones authored
      Commit af566a0b ("usb: dwc3: omap: get rid of dma_status") rendered
      reading DMA status from the H/W even more redundant.  The variable hasn't
      been read/used since 2016.  Remove the set but unused variable and the call
      which populates it.
      
      Fixes the following W=1 warning:
      
       drivers/usb/dwc3/dwc3-omap.c: In function ‘dwc3_omap_probe’:
       drivers/usb/dwc3/dwc3-omap.c:460:8: warning: variable ‘reg’ set but not used [-Wunused-but-set-variable]
       460 | u32 reg;
       | ^~~
      
      Cc: Felipe Balbi <balbi@kernel.org>
      Cc: Andrzej Siewior <bigeasy@linutronix.de>
      Cc: linux-omap@vger.kernel.org
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200702144625.2533530-9-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      3c373454
    • Lee Jones's avatar
      usb: common: ulpi: Fix a few kerneldoc related issues · 246d7a11
      Lee Jones authored
      Firstly, demote function header to standard comment block as
      they are not suitable for kerneldoc.  Then provide description
      for ulpi_register_driver()'s argument 'module'.  Finally rename
      description for ulpi_unregister_interface()'s 'ulpi' arg.
      
      Fixes the following W=1 warnings:
      
       drivers/usb/common/ulpi.c:23: warning: Function parameter or member 'ulpi' not described in 'ulpi_read'
       drivers/usb/common/ulpi.c:23: warning: Function parameter or member 'addr' not described in 'ulpi_read'
       drivers/usb/common/ulpi.c:150: warning: Function parameter or member 'module' not described in '__ulpi_register_driver'
       drivers/usb/common/ulpi.c:299: warning: Function parameter or member 'ulpi' not described in 'ulpi_unregister_interface'
       drivers/usb/common/ulpi.c:299: warning: Excess function parameter 'intrf' description in 'ulpi_unregister_interface'
      
      Cc: Felipe Balbi <balbi@kernel.org>
      Cc: Heikki Krogerus <heikki.krogerus@linux.intel.com>
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200702144625.2533530-8-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      246d7a11
    • Lee Jones's avatar
      usb: dwc3: ulpi: File headers are not doc headers · fb678a5a
      Lee Jones authored
      Demote ulpi.c's file header to a standard comment block.
      
      Fixes the following W=1 build warning:
      
       drivers/usb/dwc3/ulpi.c:18: warning: Function parameter or member 'a' not described in 'DWC3_ULPI_ADDR'
      
      Cc: Felipe Balbi <balbi@kernel.org>
      Cc: Heikki Krogerus <heikki.krogerus@linux.intel.com>
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200702144625.2533530-7-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      fb678a5a
    • Lee Jones's avatar
      usb: dwc3: drd: File headers are not doc headers · 7679defc
      Lee Jones authored
      Demote drd.c's file header to a standard comment block.
      
      Fixes the following W=1 build warnings:
      
       drivers/usb/dwc3/drd.c:20: warning: Function parameter or member 'dwc' not described in 'dwc3_otg_disable_events'
       drivers/usb/dwc3/drd.c:20: warning: Function parameter or member 'disable_mask' not described in 'dwc3_otg_disable_events'
      
      Cc: Felipe Balbi <balbi@kernel.org>
      Cc: Roger Quadros <rogerq@ti.com>
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200702144625.2533530-6-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      7679defc
    • Lee Jones's avatar
      usb: common: usb-conn-gpio: Demote comment block which is clearly not kerneldoc · 11d96a59
      Lee Jones authored
      This block lacks a title and argument descriptions.
      
      Fixes the following W=1 kernel build warning:
      
       drivers/usb/common/usb-conn-gpio.c:44: warning: Cannot understand  * "DEVICE" = VBUS and "HOST" = !ID, so we have:
       on line 44 - I thought it was a doc line
      
      Cc: Chunfeng Yun <chunfeng.yun@mediatek.com>
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200702144625.2533530-5-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      11d96a59
    • Lee Jones's avatar
      usb: common: debug: Demote comment blocks which are obviously not kerneldoc · 3e682e6f
      Lee Jones authored
      File headers and simple comments are not kerneldoc worthy.
      
      Fixes the following W=1 warnings:
      
       drivers/usb/common/debug.c:15: warning: Function parameter or member 'bRequestType' not described in 'usb_decode_get_status'
       drivers/usb/common/debug.c:15: warning: Function parameter or member 'wIndex' not described in 'usb_decode_get_status'
       drivers/usb/common/debug.c:15: warning: Function parameter or member 'wLength' not described in 'usb_decode_get_status'
       drivers/usb/common/debug.c:15: warning: Function parameter or member 'str' not described in 'usb_decode_get_status'
       drivers/usb/common/debug.c:15: warning: Function parameter or member 'size' not described in 'usb_decode_get_status'
       drivers/usb/common/debug.c:216: warning: Function parameter or member 'str' not described in 'usb_decode_ctrl'
       drivers/usb/common/debug.c:216: warning: Function parameter or member 'size' not described in 'usb_decode_ctrl'
       drivers/usb/common/debug.c:216: warning: Function parameter or member 'bRequestType' not described in 'usb_decode_ctrl'
       drivers/usb/common/debug.c:216: warning: Function parameter or member 'bRequest' not described in 'usb_decode_ctrl'
       drivers/usb/common/debug.c:216: warning: Function parameter or member 'wValue' not described in 'usb_decode_ctrl'
       drivers/usb/common/debug.c:216: warning: Function parameter or member 'wIndex' not described in 'usb_decode_ctrl'
       drivers/usb/common/debug.c:216: warning: Function parameter or member 'wLength' not described in 'usb_decode_ctrl'
      
      Cc: Pawel Laszczak <pawell@cadence.com>
      Cc: Felipe Balbi <felipe.balbi@linux.intel.com>
      Cc: Andrzej Siewior <bigeasy@linutronix.de>
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200702144625.2533530-4-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      3e682e6f
    • Lee Jones's avatar
      usb: host: pci-quirks: Demote function header from kerneldoc to comment block · 82511e2d
      Lee Jones authored
      quirk_usb_handoff_xhci()'s function header is the only one across
      the sourcefile which is denoted as a kerneldoc header.  Despite
      no attempt to document its arguments.  Drop it down in status from
      kerneldoc to a standard comment block to match the other headers
      in the file.
      
      Fixes the following W=1 kernel build warning:
      
       drivers/usb/host/pci-quirks.c:1145: warning: Function parameter or member 'pdev' not described in 'quirk_usb_handoff_xhci'
      
      Cc: Mathias Nyman <mathias.nyman@intel.com>
      Cc: Martin Mares <mj@ucw.cz>
      Cc: aleksey_gorelov@phoenix.com
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200702144625.2533530-3-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      82511e2d
    • Lee Jones's avatar
      usb: phy: phy: Fix-up a whole bunch of formatting issues · 529427b9
      Lee Jones authored
      Kerneldoc expects arg descriptions to be in the format '@.*: '.  If either
      the '@' or the ':' is omitted then kerneldoc complains that the description
      is missing.  Add the missing ':'s here.
      
      Also provide a new description for 'event'.
      
      Fixes the following kernel build W=1 warnings:
      
       drivers/usb/phy/phy.c:106: warning: Function parameter or member 'work' not described in 'usb_phy_notify_charger_work'
       drivers/usb/phy/phy.c:172: warning: Function parameter or member 'nb' not described in 'usb_phy_get_charger_type'
       drivers/usb/phy/phy.c:172: warning: Function parameter or member 'state' not described in 'usb_phy_get_charger_type'
       drivers/usb/phy/phy.c:172: warning: Function parameter or member 'data' not described in 'usb_phy_get_charger_type'
       drivers/usb/phy/phy.c:194: warning: Function parameter or member 'usb_phy' not described in 'usb_phy_set_charger_current'
       drivers/usb/phy/phy.c:194: warning: Function parameter or member 'mA' not described in 'usb_phy_set_charger_current'
       drivers/usb/phy/phy.c:244: warning: Function parameter or member 'usb_phy' not described in 'usb_phy_get_charger_current'
       drivers/usb/phy/phy.c:244: warning: Function parameter or member 'min' not described in 'usb_phy_get_charger_current'
       drivers/usb/phy/phy.c:244: warning: Function parameter or member 'max' not described in 'usb_phy_get_charger_current'
       drivers/usb/phy/phy.c:281: warning: Function parameter or member 'usb_phy' not described in 'usb_phy_set_charger_state'
       drivers/usb/phy/phy.c:281: warning: Function parameter or member 'state' not described in 'usb_phy_set_charger_state'
       drivers/usb/phy/phy.c:427: warning: Function parameter or member 'dev' not described in 'devm_usb_get_phy'
       drivers/usb/phy/phy.c:427: warning: Function parameter or member 'type' not described in 'devm_usb_get_phy'
       drivers/usb/phy/phy.c:456: warning: Function parameter or member 'type' not described in 'usb_get_phy'
       drivers/usb/phy/phy.c:500: warning: Function parameter or member 'dev' not described in 'devm_usb_get_phy_by_node'
       drivers/usb/phy/phy.c:500: warning: Function parameter or member 'node' not described in 'devm_usb_get_phy_by_node'
       drivers/usb/phy/phy.c:500: warning: Function parameter or member 'nb' not described in 'devm_usb_get_phy_by_node'
       drivers/usb/phy/phy.c:558: warning: Function parameter or member 'dev' not described in 'devm_usb_get_phy_by_phandle'
       drivers/usb/phy/phy.c:558: warning: Function parameter or member 'phandle' not described in 'devm_usb_get_phy_by_phandle'
       drivers/usb/phy/phy.c:558: warning: Function parameter or member 'index' not described in 'devm_usb_get_phy_by_phandle'
       drivers/usb/phy/phy.c:590: warning: Function parameter or member 'dev' not described in 'devm_usb_put_phy'
       drivers/usb/phy/phy.c:590: warning: Function parameter or member 'phy' not described in 'devm_usb_put_phy'
       drivers/usb/phy/phy.c:627: warning: Function parameter or member 'type' not described in 'usb_add_phy'
       drivers/usb/phy/phy.c:721: warning: Function parameter or member 'event' not described in 'usb_phy_set_event'
      
      Cc: Felipe Balbi <balbi@kernel.org>
      Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
      Link: https://lore.kernel.org/r/20200702144625.2533530-2-lee.jones@linaro.orgSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      529427b9
  4. 02 Jul, 2020 1 commit
  5. 01 Jul, 2020 6 commits