[U-Boot] [PATCH] usb: add static to local symbols

Marek Vasut marex at denx.de
Thu Jun 22 09:20:04 UTC 2017


On 06/22/2017 09:35 AM, Masahiro Yamada wrote:
> Sparse reports "... was not declared. Should it be static?"
> 
> Signed-off-by: Masahiro Yamada <yamada.masahiro at socionext.com>

Applied, thanks.

> ---
> 
>  drivers/usb/host/ehci-hcd.c   |  2 +-
>  drivers/usb/host/usb-uclass.c | 18 +++++++++---------
>  drivers/usb/host/xhci-dwc3.c  |  2 +-
>  drivers/usb/host/xhci.c       |  4 ++--
>  4 files changed, 13 insertions(+), 13 deletions(-)
> 
> diff --git a/drivers/usb/host/ehci-hcd.c b/drivers/usb/host/ehci-hcd.c
> index 13aa70d60661..f08709d0212d 100644
> --- a/drivers/usb/host/ehci-hcd.c
> +++ b/drivers/usb/host/ehci-hcd.c
> @@ -937,7 +937,7 @@ unknown:
>  	return -1;
>  }
>  
> -const struct ehci_ops default_ehci_ops = {
> +static const struct ehci_ops default_ehci_ops = {
>  	.set_usb_mode		= ehci_set_usbmode,
>  	.get_port_speed		= ehci_get_port_speed,
>  	.powerup_fixup		= ehci_powerup_fixup,
> diff --git a/drivers/usb/host/usb-uclass.c b/drivers/usb/host/usb-uclass.c
> index 110ddc92fa3a..efc5d4e7d7b0 100644
> --- a/drivers/usb/host/usb-uclass.c
> +++ b/drivers/usb/host/usb-uclass.c
> @@ -373,8 +373,8 @@ int usb_setup_ehci_gadget(struct ehci_ctrl **ctlrp)
>  }
>  
>  /* returns 0 if no match, 1 if match */
> -int usb_match_device(const struct usb_device_descriptor *desc,
> -		     const struct usb_device_id *id)
> +static int usb_match_device(const struct usb_device_descriptor *desc,
> +			    const struct usb_device_id *id)
>  {
>  	if ((id->match_flags & USB_DEVICE_ID_MATCH_VENDOR) &&
>  	    id->idVendor != le16_to_cpu(desc->idVendor))
> @@ -410,9 +410,9 @@ int usb_match_device(const struct usb_device_descriptor *desc,
>  }
>  
>  /* returns 0 if no match, 1 if match */
> -int usb_match_one_id_intf(const struct usb_device_descriptor *desc,
> -			  const struct usb_interface_descriptor *int_desc,
> -			  const struct usb_device_id *id)
> +static int usb_match_one_id_intf(const struct usb_device_descriptor *desc,
> +			const struct usb_interface_descriptor *int_desc,
> +			const struct usb_device_id *id)
>  {
>  	/* The interface class, subclass, protocol and number should never be
>  	 * checked for a match if the device class is Vendor Specific,
> @@ -445,9 +445,9 @@ int usb_match_one_id_intf(const struct usb_device_descriptor *desc,
>  }
>  
>  /* returns 0 if no match, 1 if match */
> -int usb_match_one_id(struct usb_device_descriptor *desc,
> -		     struct usb_interface_descriptor *int_desc,
> -		     const struct usb_device_id *id)
> +static int usb_match_one_id(struct usb_device_descriptor *desc,
> +			    struct usb_interface_descriptor *int_desc,
> +			    const struct usb_device_id *id)
>  {
>  	if (!usb_match_device(desc, id))
>  		return 0;
> @@ -680,7 +680,7 @@ int usb_detect_change(void)
>  	return change;
>  }
>  
> -int usb_child_post_bind(struct udevice *dev)
> +static int usb_child_post_bind(struct udevice *dev)
>  {
>  	struct usb_dev_platdata *plat = dev_get_parent_platdata(dev);
>  	int val;
> diff --git a/drivers/usb/host/xhci-dwc3.c b/drivers/usb/host/xhci-dwc3.c
> index 33961cd63455..a4591f2e02ca 100644
> --- a/drivers/usb/host/xhci-dwc3.c
> +++ b/drivers/usb/host/xhci-dwc3.c
> @@ -19,7 +19,7 @@ void dwc3_set_mode(struct dwc3 *dwc3_reg, u32 mode)
>  			DWC3_GCTL_PRTCAPDIR(mode));
>  }
>  
> -void dwc3_phy_reset(struct dwc3 *dwc3_reg)
> +static void dwc3_phy_reset(struct dwc3 *dwc3_reg)
>  {
>  	/* Assert USB3 PHY reset */
>  	setbits_le32(&dwc3_reg->g_usb3pipectl[0], DWC3_GUSB3PIPECTL_PHYSOFTRST);
> diff --git a/drivers/usb/host/xhci.c b/drivers/usb/host/xhci.c
> index 32011774768f..bd8f4c43b4b2 100644
> --- a/drivers/usb/host/xhci.c
> +++ b/drivers/usb/host/xhci.c
> @@ -192,7 +192,7 @@ static int xhci_start(struct xhci_hcor *hcor)
>   * @param hcor	pointer to host controller operation registers
>   * @return -EBUSY if XHCI Controller is not halted else status of handshake
>   */
> -int xhci_reset(struct xhci_hcor *hcor)
> +static int xhci_reset(struct xhci_hcor *hcor)
>  {
>  	u32 cmd;
>  	u32 state;
> @@ -481,7 +481,7 @@ static int xhci_address_device(struct usb_device *udev, int root_portnr)
>   * @param udev	pointer to the Device Data Structure
>   * @return Returns 0 on succes else return error code on failure
>   */
> -int _xhci_alloc_device(struct usb_device *udev)
> +static int _xhci_alloc_device(struct usb_device *udev)
>  {
>  	struct xhci_ctrl *ctrl = xhci_get_ctrl(udev);
>  	union xhci_trb *event;
> 


-- 
Best regards,
Marek Vasut


More information about the U-Boot mailing list