Lee Jones
61a140f08e
usb: gadget: udc: udc-xilinx: Place correct function names into the headers
...
Fixes the following W=1 kernel build warning(s):
drivers/usb/gadget/udc/udc-xilinx.c:802: warning: expecting prototype for xudc_ep_enable(). Prototype was for __xudc_ep_enable() instead
drivers/usb/gadget/udc/udc-xilinx.c:997: warning: expecting prototype for xudc_ep0_queue(). Prototype was for __xudc_ep0_queue() instead
Cc: Felipe Balbi <balbi@kernel.org >
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
Cc: Michal Simek <michal.simek@xilinx.com >
Cc: linux-usb@vger.kernel.org
Cc: linux-arm-kernel@lists.infradead.org
Signed-off-by: Lee Jones <lee.jones@linaro.org >
Link: https://lore.kernel.org/r/20210526130037.856068-20-lee.jones@linaro.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-27 09:47:22 +02:00
Lee Jones
e0fbc1c0ba
usb: gadget: udc: pxa27x_udc: Fix documentation for 'pxa27x_udc_start()'
...
Fixes the following W=1 kernel build warning(s):
drivers/usb/gadget/udc/pxa27x_udc.c:1749: warning: expecting prototype for pxa27x_start(). Prototype was for pxa27x_udc_start() instead
Cc: Daniel Mack <daniel@zonque.org >
Cc: Haojian Zhuang <haojian.zhuang@gmail.com >
Cc: Robert Jarzmik <robert.jarzmik@free.fr >
Cc: Felipe Balbi <balbi@kernel.org >
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
Cc: linux-arm-kernel@lists.infradead.org
Cc: linux-usb@vger.kernel.org
Acked-by: Daniel Mack <daniel@zonque.org >
Signed-off-by: Lee Jones <lee.jones@linaro.org >
Link: https://lore.kernel.org/r/20210526130037.856068-19-lee.jones@linaro.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-27 09:47:22 +02:00
Lee Jones
5aff197ffe
usb: typec: ucsi: Fix copy/paste issue for 'ucsi_set_drvdata()'
...
Fixes the following W=1 kernel build warning(s):
drivers/usb/typec/ucsi/ucsi.c:1287: warning: expecting prototype for ucsi_get_drvdata(). Prototype was for ucsi_set_drvdata() instead
Cc: Heikki Krogerus <heikki.krogerus@linux.intel.com >
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
Cc: linux-usb@vger.kernel.org
Reviewed-by: Heikki Krogerus <heikki.krogerus@linux.intel.com >
Signed-off-by: Lee Jones <lee.jones@linaro.org >
Link: https://lore.kernel.org/r/20210526130037.856068-25-lee.jones@linaro.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-27 09:47:22 +02:00
Lee Jones
58aff959fc
usb: dwc2: gadget: Repair 'dwc2_hsotg_core_init_disconnected()'s documentation
...
Fixes the following W=1 kernel build warning(s):
drivers/usb/dwc2/gadget.c:3349: warning: expecting prototype for dwc2_hsotg_core_init(). Prototype was for dwc2_hsotg_core_init_disconnected() instead
Cc: Minas Harutyunyan <hminas@synopsys.com >
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
Cc: Ben Dooks <ben@simtec.co.uk >
Cc: linux-usb@vger.kernel.org
Acked-by: Minas Harutyunyan <Minas.Harutyunyan@synopsys.com >
Signed-off-by: Lee Jones <lee.jones@linaro.org >
Link: https://lore.kernel.org/r/20210526130037.856068-16-lee.jones@linaro.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-27 09:45:27 +02:00
Lee Jones
81d708bc13
usb: dwc2: pci: Fix possible copy/paste issue
...
Fixes the following W=1 kernel build warning(s):
drivers/usb/dwc2/pci.c:73: warning: expecting prototype for dwc2_pci_probe(). Prototype was for dwc2_pci_remove() instead
Cc: Minas Harutyunyan <hminas@synopsys.com >
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
Cc: linux-usb@vger.kernel.org
Acked-by: Minas Harutyunyan <Minas.Harutyunyan@synopsys.com >
Signed-off-by: Lee Jones <lee.jones@linaro.org >
Link: https://lore.kernel.org/r/20210526130037.856068-13-lee.jones@linaro.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-27 09:45:27 +02:00
Lee Jones
a63acbde82
usb: dwc2: hcd_queue: Fix typeo in function name 'dwc2_hs_pmap_unschedule()'
...
Fixes the following W=1 kernel build warning(s):
drivers/usb/dwc2/hcd_queue.c:686: warning: expecting prototype for dwc2_ls_pmap_unschedule(). Prototype was for dwc2_hs_pmap_unschedule() instead
Cc: Minas Harutyunyan <hminas@synopsys.com >
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
Cc: linux-usb@vger.kernel.org
Acked-by: Minas Harutyunyan <Minas.Harutyunyan@synopsys.com >
Signed-off-by: Lee Jones <lee.jones@linaro.org >
Link: https://lore.kernel.org/r/20210526130037.856068-12-lee.jones@linaro.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-27 09:45:27 +02:00
Lee Jones
8268acfe1c
usb: isp1760: isp1760-udc: Provide missing description for 'udc' param
...
Fixes the following W=1 kernel build warning(s):
drivers/usb/isp1760/isp1760-udc.c:150: warning: Function parameter or member 'udc' not described in 'isp1760_udc_select_ep'
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
Cc: Rui Miguel Silva <rui.silva@linaro.org >
Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com >
Cc: linux-usb@vger.kernel.org
Reviewed-by: Rui Miguel Silva <rui.silva@linaro.org >
Signed-off-by: Lee Jones <lee.jones@linaro.org >
Link: https://lore.kernel.org/r/20210526130037.856068-7-lee.jones@linaro.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-27 09:45:27 +02:00
Lee Jones
bd37fbd5f5
usb: dwc2: params: Fix naming of 'dwc2_get_hwparams()' in the docs
...
Fixes the following W=1 kernel build warning(s):
drivers/usb/dwc2/params.c:787: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
Cc: Minas Harutyunyan <hminas@synopsys.com >
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
Cc: linux-usb@vger.kernel.org
Acked-by: Minas Harutyunyan <Minas.Harutyunyan@synopsys.com >
Signed-off-by: Lee Jones <lee.jones@linaro.org >
Link: https://lore.kernel.org/r/20210526130037.856068-6-lee.jones@linaro.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-27 09:45:27 +02:00
Lee Jones
826e9c4497
usb: common: ulpi: Add leading underscores for function name '__ulpi_register_driver()'
...
Fixes the following W=1 kernel build warning(s):
drivers/usb/common/ulpi.c:151: warning: expecting prototype for ulpi_register_driver(). Prototype was for __ulpi_register_driver() instead
Cc: Heikki Krogerus <heikki.krogerus@linux.intel.com >
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
Cc: linux-usb@vger.kernel.org
Reviewed-by: Heikki Krogerus <heikki.krogerus@linux.intel.com >
Signed-off-by: Lee Jones <lee.jones@linaro.org >
Link: https://lore.kernel.org/r/20210526130037.856068-4-lee.jones@linaro.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-27 09:45:27 +02:00
Lee Jones
c1fb8640e8
usb: dwc2: platform: Provide function name for 'dwc2_check_core_version()'
...
Fixes the following W=1 kernel build warning(s):
drivers/usb/dwc2/platform.c:411: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
Cc: Minas Harutyunyan <hminas@synopsys.com >
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
Cc: Philipp Zabel <p.zabel@pengutronix.de >
Cc: Liam Girdwood <lgirdwood@gmail.com >
Cc: Mark Brown <broonie@kernel.org >
Cc: Matthijs Kooijman <matthijs@stdin.nl >
Cc: linux-usb@vger.kernel.org
Acked-by: Minas Harutyunyan <Minas.Harutyunyan@synopsys.com >
Signed-off-by: Lee Jones <lee.jones@linaro.org >
Link: https://lore.kernel.org/r/20210526130037.856068-3-lee.jones@linaro.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-27 09:45:26 +02:00
Lee Jones
632d234b0b
usb: cdns3: cdnsp-gadget: Provide function name for 'cdnsp_find_next_ext_cap()'
...
Fixes the following W=1 kernel build warning(s):
drivers/usb/cdns3/cdnsp-gadget.c:59: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
Cc: Pawel Laszczak <pawell@cadence.com >
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
Cc: linux-usb@vger.kernel.org
Acked-by: Peter Chen <peter.chen@kernel.org >
Signed-off-by: Lee Jones <lee.jones@linaro.org >
Link: https://lore.kernel.org/r/20210526130037.856068-24-lee.jones@linaro.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-27 09:43:45 +02:00
Lee Jones
00dfda2db2
usb: cdns3: cdns3-gadget: Provide correct function naming for '__cdns3_gadget_ep_queue()'
...
Fixes the following W=1 kernel build warning(s):
drivers/usb/cdns3/cdns3-gadget.c:2499: warning: expecting prototype for cdns3_gadget_ep_queue(). Prototype was for __cdns3_gadget_ep_queue() instead
Cc: Peter Chen <peter.chen@kernel.org >
Cc: Pawel Laszczak <pawell@cadence.com >
Cc: Roger Quadros <rogerq@kernel.org >
Cc: Aswath Govindraju <a-govindraju@ti.com >
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
Cc: Pawel Jez <pjez@cadence.com >
Cc: linux-usb@vger.kernel.org
Acked-by: Peter Chen <peter.chen@kernel.org >
Signed-off-by: Lee Jones <lee.jones@linaro.org >
Link: https://lore.kernel.org/r/20210526130037.856068-21-lee.jones@linaro.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-27 09:43:45 +02:00
Lee Jones
9b3c1c90d6
usb: chipidea: udc: Fix incorrectly documented function 'hw_port_is_high_speed()'
...
Fixes the following W=1 kernel build warning(s):
drivers/usb/chipidea/udc.c:247: warning: expecting prototype for hw_is_port_high_speed(). Prototype was for hw_port_is_high_speed() instead
Cc: Peter Chen <peter.chen@kernel.org >
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
Cc: linux-usb@vger.kernel.org
Acked-by: Peter Chen <peter.chen@kernel.org >
Signed-off-by: Lee Jones <lee.jones@linaro.org >
Link: https://lore.kernel.org/r/20210526130037.856068-17-lee.jones@linaro.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-27 09:43:45 +02:00
Lee Jones
953c3a3c31
usb: chipidea: otg: Fix formatting and missing documentation issues
...
Fixes the following W=1 kernel build warning(s):
drivers/usb/chipidea/otg.c:25: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
drivers/usb/chipidea/otg.c:78: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
drivers/usb/chipidea/otg.c:143: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
Cc: Peter Chen <peter.chen@kernel.org >
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
Cc: linux-usb@vger.kernel.org
Acked-by: Peter Chen <peter.chen@kernel.org >
Signed-off-by: Lee Jones <lee.jones@linaro.org >
Link: https://lore.kernel.org/r/20210526130037.856068-15-lee.jones@linaro.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-27 09:43:44 +02:00
Lee Jones
b1f562f1c4
usb: chipidea: core: Fix incorrectly documented function 'ci_usb_phy_exit()'
...
Fixes the following W=1 kernel build warning(s):
drivers/usb/chipidea/core.c:343: warning: expecting prototype for _ci_usb_phy_exit(). Prototype was for ci_usb_phy_exit() instead
Cc: Peter Chen <peter.chen@kernel.org >
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
Cc: Liam Girdwood <lgirdwood@gmail.com >
Cc: Mark Brown <broonie@kernel.org >
Cc: David Lopo <dlopo@chipidea.mips.com >
Cc: linux-usb@vger.kernel.org
Acked-by: Peter Chen <peter.chen@kernel.org >
Signed-off-by: Lee Jones <lee.jones@linaro.org >
Link: https://lore.kernel.org/r/20210526130037.856068-14-lee.jones@linaro.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-27 09:43:44 +02:00
Lee Jones
c23e55e668
usb: cdns3: cdns3-imx: File headers are not good candidates for kernel-doc
...
Fixes the following W=1 kernel build warning(s):
drivers/usb/cdns3/cdns3-imx.c:21: warning: expecting prototype for cdns3(). Prototype was for USB3_CORE_CTRL1() instead
Cc: Peter Chen <peter.chen@kernel.org >
Cc: Pawel Laszczak <pawell@cadence.com >
Cc: Roger Quadros <rogerq@kernel.org >
Cc: Aswath Govindraju <a-govindraju@ti.com >
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
Cc: Shawn Guo <shawnguo@kernel.org >
Cc: Sascha Hauer <s.hauer@pengutronix.de >
Cc: Pengutronix Kernel Team <kernel@pengutronix.de >
Cc: Fabio Estevam <festevam@gmail.com >
Cc: NXP Linux Team <linux-imx@nxp.com >
Cc: linux-usb@vger.kernel.org
Cc: linux-arm-kernel@lists.infradead.org
Acked-by: Peter Chen <peter.chen@kernel.org >
Signed-off-by: Lee Jones <lee.jones@linaro.org >
Link: https://lore.kernel.org/r/20210526130037.856068-11-lee.jones@linaro.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-27 09:43:44 +02:00
Lee Jones
e1ecf7582f
usb: cdns3: cdns3-ep0: Fix a few kernel-doc formatting issues
...
Fixes the following W=1 kernel build warning(s):
drivers/usb/cdns3/cdns3-ep0.c:680: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
drivers/usb/cdns3/cdns3-ep0.c:775: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
drivers/usb/cdns3/cdns3-ep0.c:868: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
Cc: Peter Chen <peter.chen@kernel.org >
Cc: Pawel Laszczak <pawell@cadence.com >
Cc: Roger Quadros <rogerq@kernel.org >
Cc: Aswath Govindraju <a-govindraju@ti.com >
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
Cc: Pawel Jez <pjez@cadence.com >
Cc: linux-usb@vger.kernel.org
Acked-by: Peter Chen <peter.chen@kernel.org >
Signed-off-by: Lee Jones <lee.jones@linaro.org >
Link: https://lore.kernel.org/r/20210526130037.856068-10-lee.jones@linaro.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-27 09:43:44 +02:00
Lee Jones
a945fd0a58
usb: cdns3: cdns3-ti: File headers are not good candidates for kernel-doc
...
Fixes the following W=1 kernel build warning(s):
drivers/usb/cdns3/cdns3-ti.c:20: warning: expecting prototype for cdns3(). Prototype was for USBSS_PID() instead
Cc: Peter Chen <peter.chen@kernel.org >
Cc: Pawel Laszczak <pawell@cadence.com >
Cc: Roger Quadros <rogerq@kernel.org >
Cc: Aswath Govindraju <a-govindraju@ti.com >
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
Cc: linux-usb@vger.kernel.org
Acked-by: Peter Chen <peter.chen@kernel.org >
Signed-off-by: Lee Jones <lee.jones@linaro.org >
Link: https://lore.kernel.org/r/20210526130037.856068-9-lee.jones@linaro.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-27 09:43:44 +02:00
Lee Jones
56480a03f1
usb: cdns3: cdns3-gadget: Fix a bunch of kernel-doc related formatting issues
...
Fixes the following W=1 kernel build warning(s):
drivers/usb/cdns3/cdns3-gadget.c:163: warning: expecting prototype for select_ep(). Prototype was for cdns3_select_ep() instead
drivers/usb/cdns3/cdns3-gadget.c:2025: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
drivers/usb/cdns3/cdns3-gadget.c:2224: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
drivers/usb/cdns3/cdns3-gadget.c:2247: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
drivers/usb/cdns3/cdns3-gadget.c:2264: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
drivers/usb/cdns3/cdns3-gadget.c:2399: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
drivers/usb/cdns3/cdns3-gadget.c:2489: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
drivers/usb/cdns3/cdns3-gadget.c:2589: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
drivers/usb/cdns3/cdns3-gadget.c:2656: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
drivers/usb/cdns3/cdns3-gadget.c:2677: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
drivers/usb/cdns3/cdns3-gadget.c:2722: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
drivers/usb/cdns3/cdns3-gadget.c:2768: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
drivers/usb/cdns3/cdns3-gadget.c:2877: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
drivers/usb/cdns3/cdns3-gadget.c:2923: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
drivers/usb/cdns3/cdns3-gadget.c:2986: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
Cc: Peter Chen <peter.chen@kernel.org >
Cc: Pawel Laszczak <pawell@cadence.com >
Cc: Roger Quadros <rogerq@kernel.org >
Cc: Aswath Govindraju <a-govindraju@ti.com >
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
Cc: Pawel Jez <pjez@cadence.com >
Cc: linux-usb@vger.kernel.org
Acked-by: Peter Chen <peter.chen@kernel.org >
Signed-off-by: Lee Jones <lee.jones@linaro.org >
Link: https://lore.kernel.org/r/20210526130037.856068-8-lee.jones@linaro.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-27 09:43:44 +02:00
Lee Jones
6dd1efeb18
usb: cdns3: cdns3-plat: Fix incorrect naming of function 'cdns3_plat_remove()'
...
Fixes the following W=1 kernel build warning(s):
drivers/usb/cdns3/cdns3-plat.c:179: warning: expecting prototype for cdns3_remove(). Prototype was for cdns3_plat_remove() instead
Cc: Peter Chen <peter.chen@kernel.org >
Cc: Pawel Laszczak <pawell@cadence.com >
Cc: Roger Quadros <rogerq@kernel.org >
Cc: Aswath Govindraju <a-govindraju@ti.com >
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
Cc: linux-usb@vger.kernel.org
Acked-by: Peter Chen <peter.chen@kernel.org >
Signed-off-by: Lee Jones <lee.jones@linaro.org >
Link: https://lore.kernel.org/r/20210526130037.856068-5-lee.jones@linaro.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-27 09:43:44 +02:00
Lee Jones
47a4edc7ac
usb: cdns3: core: Fix a couple of incorrectly documented function names
...
Fixes the following W=1 kernel build warning(s):
drivers/usb/cdns3/core.c:342: warning: expecting prototype for cdsn3_role_get(). Prototype was for cdns_role_get() instead
drivers/usb/cdns3/core.c:428: warning: expecting prototype for cdns_probe(). Prototype was for cdns_init() instead
Cc: Peter Chen <peter.chen@kernel.org >
Cc: Pawel Laszczak <pawell@cadence.com >
Cc: Roger Quadros <rogerq@kernel.org >
Cc: Aswath Govindraju <a-govindraju@ti.com >
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
Cc: linux-usb@vger.kernel.org
Acked-by: Peter Chen <peter.chen@kernel.org >
Signed-off-by: Lee Jones <lee.jones@linaro.org >
Link: https://lore.kernel.org/r/20210526130037.856068-2-lee.jones@linaro.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-27 09:43:44 +02:00
Rajat Jain
c037b6c818
PCI: Add sysfs "removable" attribute
...
A PCI device is "external_facing" if it's a Root Port with the ACPI
"ExternalFacingPort" property or if it has the DT "external-facing"
property. We consider everything downstream from such a device to
be removable by user.
We're mainly concerned with consumer platforms with user accessible
Thunderbolt ports that are vulnerable to DMA attacks, and we expect those
ports to be identified by firmware as "ExternalFacingPort". Devices in
traditional hotplug slots can technically be removed, but the expectation
is that unless the port is marked with "ExternalFacingPort", such devices
are less accessible to user / may not be removed by end user, and thus not
exposed as "removable" to userspace.
This can be used to implement userspace policies tailored for
user removable devices. Eg usage:
https://chromium-review.googlesource.com/c/chromiumos/platform2/+/2591812
https://chromium-review.googlesource.com/c/chromiumos/platform2/+/2795038
(code uses such an attribute to remove external PCI devices or disable
features on them as needed by the policy desired)
Acked-by: Bjorn Helgaas <bhelgaas@google.com >
Signed-off-by: Rajat Jain <rajatja@google.com >
Link: https://lore.kernel.org/r/20210524171812.18095-2-rajatja@google.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-27 09:36:31 +02:00
Rajat Jain
70f400d4d9
driver core: Move the "removable" attribute from USB to core
...
Move the "removable" attribute from USB to core in order to allow it to be
supported by other subsystem / buses. Individual buses that want to support
this attribute can populate the removable property of the device while
enumerating it with the 3 possible values -
- "unknown"
- "fixed"
- "removable"
Leaving the field unchanged (i.e. "not supported") would mean that the
attribute would not show up in sysfs for that device. The UAPI (location,
symantics etc) for the attribute remains unchanged.
Move the "removable" attribute from USB to the device core so it can be
used by other subsystems / buses.
By default, devices do not have a "removable" attribute in sysfs.
If a subsystem or bus driver wants to support a "removable" attribute, it
should call device_set_removable() before calling device_register() or
device_add(), e.g.:
device_set_removable(dev, DEVICE_REMOVABLE);
device_register(dev);
The possible values and the resulting sysfs attribute contents are:
DEVICE_REMOVABLE_UNKNOWN -> "unknown"
DEVICE_REMOVABLE -> "removable"
DEVICE_FIXED -> "fixed"
Convert the USB "removable" attribute to use this new device core
functionality. There should be no user-visible change in the location or
semantics of attribute for USB devices.
Reviewed-by: Bjorn Helgaas <bhelgaas@google.com >
Signed-off-by: Rajat Jain <rajatja@google.com >
Link: https://lore.kernel.org/r/20210524171812.18095-1-rajatja@google.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-27 09:36:31 +02:00
Greg Kroah-Hartman
0f60203d21
USB: fotg210-hcd: remove dentry storage for debugfs file
...
There is no need to store the dentry pointer for a debugfs file that we
only use to remove it when the device goes away. debugfs can do the
lookup for us instead, saving us some trouble, and making things smaller
overall.
Link: https://lore.kernel.org/r/20210525172534.848775-1-gregkh@linuxfoundation.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-27 09:35:16 +02:00
Greg Kroah-Hartman
8efd88f946
USB: gadget: pxa27x_udc: remove dentry storage for debugfs file
...
There is no need to store the dentry pointer for a debugfs file that we
only use to remove it when the device goes away. debugfs can do the
lookup for us instead, saving us some trouble, and making things smaller
overall.
Cc: Daniel Mack <daniel@zonque.org >
Cc: Haojian Zhuang <haojian.zhuang@gmail.com >
Cc: Robert Jarzmik <robert.jarzmik@free.fr >
Cc: Felipe Balbi <balbi@kernel.org >
Link: https://lore.kernel.org/r/20210525171636.758758-1-gregkh@linuxfoundation.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-27 09:35:04 +02:00
Greg Kroah-Hartman
0cac357717
USB: gadget: bcm63xx_udc: remove dentry storage for debugfs file
...
There is no need to store the dentry pointer for a debugfs file that we
only use to remove it when the device goes away. debugfs can do the
lookup for us instead, saving us some trouble, and making things smaller
overall.
Cc: Kevin Cernekee <cernekee@gmail.com >
Cc: Felipe Balbi <balbi@kernel.org >
Cc: bcm-kernel-feedback-list@broadcom.com
Link: https://lore.kernel.org/r/20210525171508.758365-1-gregkh@linuxfoundation.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-27 09:34:44 +02:00
Greg Kroah-Hartman
8f6c7c5a11
USB: chipidea: remove dentry storage for debugfs file
...
There is no need to store the dentry pointer for a debugfs file that we
only use to remove it when the device goes away. debugfs can do the
lookup for us instead, saving us some trouble, and making things smaller
overall.
Cc: Peter Chen <peter.chen@kernel.org >
Link: https://lore.kernel.org/r/20210525171419.758146-1-gregkh@linuxfoundation.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-27 09:34:34 +02:00
Mayank Rana
ab00a41e73
usb: dwc3: trace: Remove unused fields in dwc3_log_trb
...
Commit 0bd0f6d201 ("usb: dwc3: gadget: remove allocated/queued request
tracking") removed the allocated & queued fields from struct dwc3_ep
but neglected to also remove them from the dwc3_log_trb event class's
TP_STRUCT definition which are now unused. Remove them to save eight
bytes per trace event entry.
Signed-off-by: Mayank Rana <mrana@codeaurora.org >
Signed-off-by: Jack Pham <jackp@codeaurora.org >
Link: https://lore.kernel.org/r/20210527012924.3596-1-jackp@codeaurora.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-27 09:30:01 +02:00
Heikki Krogerus
acad3e9c72
usb: typec: mux: Remove requirement for the "orientation-switch" device property
...
The additional boolean device property "orientation-switch"
is not needed when the connection is described with device
graph, so removing the check and the requirement for it.
Reviewed-by: Hans de Goede <hdegoede@redhat.com >
Signed-off-by: Heikki Krogerus <heikki.krogerus@linux.intel.com >
Link: https://lore.kernel.org/r/20210526153548.61276-3-heikki.krogerus@linux.intel.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-27 09:23:57 +02:00
Heikki Krogerus
7bf991eab8
usb: typec: mux: Use device type instead of device name for matching
...
Both the USB Type-C switch and mux have already a device
type defined for them. We can use those types instead of the
device name to differentiate the two.
Reviewed-by: Hans de Goede <hdegoede@redhat.com >
Signed-off-by: Heikki Krogerus <heikki.krogerus@linux.intel.com >
Link: https://lore.kernel.org/r/20210526153548.61276-2-heikki.krogerus@linux.intel.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-27 09:23:57 +02:00
Chunfeng Yun
baabd69492
usb: common: usb-conn-gpio: use usb_role_string() to print role status
...
Use usb_role_string() to print role status, make the log readable.
Signed-off-by: Chunfeng Yun <chunfeng.yun@mediatek.com >
Link: https://lore.kernel.org/r/1621932786-9335-2-git-send-email-chunfeng.yun@mediatek.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-27 09:17:18 +02:00
Chunfeng Yun
73e33008e8
usb: roles: add helper usb_role_string()
...
Introduces usb_role_string() function, which returns a
human-readable name of provided usb role, it's useful to
make the log readable.
Reviewed-by: Heikki Krogerus <heikki.krogerus@linux.intel.com >
Signed-off-by: Chunfeng Yun <chunfeng.yun@mediatek.com >
Link: https://lore.kernel.org/r/1621932786-9335-1-git-send-email-chunfeng.yun@mediatek.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-27 09:17:18 +02:00
Wei Yongjun
a0765597c9
usb: typec: tcpci: Make symbol 'tcpci_apply_rc' static
...
The sparse tool complains as follows:
drivers/usb/typec/tcpm/tcpci.c:118:5: warning:
symbol 'tcpci_apply_rc' was not declared. Should it be static?
This symbol is not used outside of tcpci.c, so marks it static.
Fixes: 7257fbc7c5 ("usb: typec: tcpci: Implement callback for apply_rc")
Reported-by: Hulk Robot <hulkci@huawei.com >
Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com >
Link: https://lore.kernel.org/r/20210524133704.2432555-1-weiyongjun1@huawei.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-24 15:35:15 +02:00
Andy Shevchenko
ca82c06788
usb: phy: isp1301: Deduplicate of_find_i2c_device_by_node()
...
The driver is using open-coded variant of of_find_i2c_device_by_node().
Replace it by the actual call to the above mentioned API.
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com >
Link: https://lore.kernel.org/r/20210521145243.87911-1-andriy.shevchenko@linux.intel.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-24 15:27:17 +02:00
Geoffrey D. Bennett
08377263a9
USB: usbfs: remove double evaluation of usb_sndctrlpipe()
...
usb_sndctrlpipe() is evaluated in do_proc_control(), saved in a
variable, then evaluated again. Use the saved variable instead, to
match the use of usb_rcvctrlpipe().
Acked-by: Alan Stern <stern@rowland.harvard.edu >
Signed-off-by: Geoffrey D. Bennett <g@b4.vu >
Link: https://lore.kernel.org/r/20210521174027.GA116484@m.b4.vu
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-24 15:27:14 +02:00
Aditya Srivastava
1eef795312
USB: gadget: udc: fix kernel-doc syntax in file headers
...
The opening comment mark '/**' is used for highlighting the beginning of
kernel-doc comments.
The header for drivers/usb/gadget/udc/trace files follows this syntax, but
the content inside does not comply with kernel-doc.
This line was probably not meant for kernel-doc parsing, but is parsed
due to the presence of kernel-doc like comment syntax(i.e, '/**'), which
causes unexpected warning from kernel-doc.
For e.g., running scripts/kernel-doc -none drivers/usb/gadget/udc/trace.h
emits:
warning: expecting prototype for udc.c(). Prototype was for TRACE_SYSTEM() instead
Provide a simple fix by replacing this occurrence with general comment
format, i.e. '/*', to prevent kernel-doc from parsing it.
Acked-by: Randy Dunlap <rdunlap@infradead.org >
Signed-off-by: Aditya Srivastava <yashsri421@gmail.com >
Link: https://lore.kernel.org/r/20210522115227.9977-1-yashsri421@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-24 15:27:03 +02:00
Alan Stern
5cc59c418f
USB: core: WARN if pipe direction != setup packet direction
...
When a control URB is submitted, the direction indicated by URB's pipe
member is supposed to match the direction indicated by the setup
packet's bRequestType member. A mismatch could lead to trouble,
depending on which field the host controller drivers use for
determining the actual direction.
This shouldn't ever happen; it would represent a careless bug in a
kernel driver somewhere. This patch adds a dev_WARN_ONCE to let
people know about the potential problem.
Suggested-by: "Geoffrey D. Bennett" <g@b4.vu >
Reviewed-by: Johan Hovold <johan@kernel.org >
Signed-off-by: Alan Stern <stern@rowland.harvard.edu >
Link: https://lore.kernel.org/r/20210522021623.GB1260282@rowland.harvard.edu
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-22 10:37:25 +02:00
Badhri Jagan Sridharan
7257fbc7c5
usb: typec: tcpci: Implement callback for apply_rc
...
APPLY RC is defined as ROLE_CONTROL.CC1 != ROLE_CONTROL.CC2 and
POWER_CONTROL.AutodischargeDisconnect is 0. When ROLE_CONTROL.CC1 ==
ROLE_CONTROL.CC2, set the other CC to OPEN.
Fixes: f321a02cae ("usb: typec: tcpm: Implement enabling Auto Discharge disconnect support")
Reviewed-by: Guenter Roeck <linux@roeck-us.net >
Acked-by: Heikki Krogerus <heikki.krogerus@linux.intel.com >
Signed-off-by: Badhri Jagan Sridharan <badhri@google.com >
Link: https://lore.kernel.org/r/20210517192112.40934-4-badhri@google.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-21 20:05:56 +02:00
Badhri Jagan Sridharan
59d4d06c8a
usb: typec: tcpm: Move TCPC to APPLY_RC state during PR_SWAP
...
When vbus auto discharge is enabled, TCPCI based TCPC transitions
into Attached.SNK/Attached.SRC state. During PR_SWAP, TCPCI based
TCPC would disconnect when partner changes power roles. TCPC has
to be moved APPLY RC state during PR_SWAP. This is done by
ROLE_CONTROL.CC1 != ROLE_CONTROL.CC2 and
POWER_CONTROL.AutodischargeDisconnect is 0. Once the swap sequence
is done, AutoDischargeDisconnect is re-enabled.
Fixes: f321a02cae ("usb: typec: tcpm: Implement enabling Auto Discharge disconnect support")
Reviewed-by: Guenter Roeck <linux@roeck-us.net >
Acked-by: Heikki Krogerus <heikki.krogerus@linux.intel.com >
Signed-off-by: Badhri Jagan Sridharan <badhri@google.com >
Link: https://lore.kernel.org/r/20210517192112.40934-3-badhri@google.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-21 20:05:56 +02:00
Badhri Jagan Sridharan
dea6f87e60
usb: typec: tcpm: Refactor logic to enable/disable auto vbus dicharge
...
The logic to enable vbus auto discharge on disconnect is used in
more than one place. Since this is repetitive code, moving this into
its own method.
Fixes: f321a02cae ("usb: typec: tcpm: Implement enabling Auto Discharge disconnect support")
Reviewed-by: Guenter Roeck <linux@roeck-us.net >
Acked-by: Heikki Krogerus <heikki.krogerus@linux.intel.com >
Signed-off-by: Badhri Jagan Sridharan <badhri@google.com >
Link: https://lore.kernel.org/r/20210517192112.40934-2-badhri@google.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-21 20:05:56 +02:00
Badhri Jagan Sridharan
d112efbe6d
usb: typec: tcpm: Fix up PR_SWAP when vsafe0v is signalled
...
During PR_SWAP, When TCPM is in PR_SWAP_SNK_SRC_SINK_OFF, vbus is
expected to reach VSAFE0V when source turns off vbus. Do not move
to SNK_UNATTACHED state when this happens.
Fixes: 28b43d3d74 ("usb: typec: tcpm: Introduce vsafe0v for vbus")
Reviewed-by: Guenter Roeck <linux@roeck-us.net >
Acked-by: Heikki Krogerus <heikki.krogerus@linux.intel.com >
Signed-off-by: Badhri Jagan Sridharan <badhri@google.com >
Link: https://lore.kernel.org/r/20210517192112.40934-1-badhri@google.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-21 20:05:56 +02:00
Tobias Schramm
56d426146c
dt-bindings: usb: dwc2: add compatible for RK3308 USB controller
...
The USB controller in the RK3308 is compatible with the RK3066 USB
controller.
This patch adds a compatible string for it.
Signed-off-by: Tobias Schramm <t.schramm@manjaro.org >
Acked-by: Rob Herring <robh@kernel.org >
Link: https://lore.kernel.org/r/20210514150044.2099298-4-t.schramm@manjaro.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-21 20:05:56 +02:00
Johan Hovold
c9c5f057d0
USB: gadget: drop irq-flags initialisations
...
There's no need to initialise irq-flags variables before saving the
interrupt state.
Drop the redundant initialisations from drivers that got this wrong.
Cc: Li Yang <leoyang.li@nxp.com >
Cc: Felipe Balbi <balbi@kernel.org >
Signed-off-by: Johan Hovold <johan@kernel.org >
Link: https://lore.kernel.org/r/20210519093303.10789-4-johan@kernel.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-21 20:05:56 +02:00
Johan Hovold
8879904b19
USB: dwc2: drop irq-flags initialisations
...
There's no need to initialise irq-flags variables before saving the
interrupt state.
While at it drop two redundant return-value initialisations from two of
the functions that got it wrong.
Cc: Minas Harutyunyan <hminas@synopsys.com >
Signed-off-by: Johan Hovold <johan@kernel.org >
Link: https://lore.kernel.org/r/20210519093303.10789-3-johan@kernel.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-21 20:05:56 +02:00
Johan Hovold
18538a5023
USB: cdnsp: drop irq-flags initialisations
...
There's no need to initialise irq-flags variables before saving the
interrupt state.
Cc: Pawel Laszczak <pawell@cadence.com >
Signed-off-by: Johan Hovold <johan@kernel.org >
Link: https://lore.kernel.org/r/20210519093303.10789-2-johan@kernel.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-21 20:05:56 +02:00
Jon Hunter
77b57218ac
usb: gadget: tegra-xudc: Use dev_err_probe()
...
Rather than testing if the error code is -EPROBE_DEFER before printing
an error message, use dev_err_probe() instead to simplify the code.
Signed-off-by: Jon Hunter <jonathanh@nvidia.com >
Link: https://lore.kernel.org/r/20210519163553.212682-2-jonathanh@nvidia.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-21 20:05:56 +02:00
Jon Hunter
80a3c7f70e
usb: gadget: tegra-xudc: Don't print error on probe deferral
...
The Tegra XUDC driver prints the following error when deferring probe
if the USB PHY is not found ...
ERR KERN tegra-xudc 3550000.usb: failed to get usbphy-0: -517
Deferring probe can be normal and so update to driver to avoid printing
this error if probe is being deferred.
Signed-off-by: Jon Hunter <jonathanh@nvidia.com >
Link: https://lore.kernel.org/r/20210519163553.212682-1-jonathanh@nvidia.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-21 20:05:56 +02:00
zuoqilin
b274e2a44e
usb: atm: cxacru: Fix typo in comment
...
Change 'contol' to 'control'.
Signed-off-by: zuoqilin <zuoqilin@yulong.com >
Link: https://lore.kernel.org/r/20210521095804.773-1-zuoqilin1@163.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-21 20:05:40 +02:00
Rui Miguel Silva
d369c9187c
usb: isp1763: add peripheral mode
...
Besides the already host mode support add peripheral mode support for
the isp1763 IP from the isp1760 family.
Signed-off-by: Rui Miguel Silva <rui.silva@linaro.org >
Link: https://lore.kernel.org/r/20210513084717.2487366-10-rui.silva@linaro.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-21 20:05:32 +02:00
Rui Miguel Silva
e7a990e00c
dt-bindings: usb: nxp,isp1760: add bindings
...
The nxp,isp1760 driver is old in the tree, but did not had a bindings
entry, since I am extend it to support isp1763 in the same family, use
this to add a proper yaml bindings file.
Reviewed-by: Rob Herring <robh@kernel.org >
Signed-off-by: Rui Miguel Silva <rui.silva@linaro.org >
Link: https://lore.kernel.org/r/20210513084717.2487366-9-rui.silva@linaro.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2021-05-21 20:05:32 +02:00