diff mbox

[U-Boot,02/22] usb: Drop device-model specific copy of usb_legacy_port_reset

Message ID 1434569645-30322-3-git-send-email-hdegoede@redhat.com
State Accepted
Delegated to: Simon Glass
Headers show

Commit Message

Hans de Goede June 17, 2015, 7:33 p.m. UTC
The device-model usb_legacy_port_reset function calls the device-model
usb_port_reset function which is a 1 on 1 copy of the non dm
usb_legacy_port_reset and this is the only use of usb_port_reset in all
of u-boot.

Drop both, and alway use the usb_legacy_port_reset() version in
common/usb.c .

Also while at it make it static as it is only used in common/usb.c .

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
 common/usb.c                  |  4 +---
 drivers/usb/host/usb-uclass.c | 29 -----------------------------
 include/usb.h                 |  8 --------
 3 files changed, 1 insertion(+), 40 deletions(-)

Comments

Simon Glass June 29, 2015, 3:44 a.m. UTC | #1
On 17 June 2015 at 13:33, Hans de Goede <hdegoede@redhat.com> wrote:
> The device-model usb_legacy_port_reset function calls the device-model
> usb_port_reset function which is a 1 on 1 copy of the non dm
> usb_legacy_port_reset and this is the only use of usb_port_reset in all
> of u-boot.
>
> Drop both, and alway use the usb_legacy_port_reset() version in
> common/usb.c .
>
> Also while at it make it static as it is only used in common/usb.c .
>
> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
> ---
>  common/usb.c                  |  4 +---
>  drivers/usb/host/usb-uclass.c | 29 -----------------------------
>  include/usb.h                 |  8 --------
>  3 files changed, 1 insertion(+), 40 deletions(-)

Acked-by: Simon Glass <sjg@chromium.org>
Simon Glass July 7, 2015, 6:33 p.m. UTC | #2
On 28 June 2015 at 21:44, Simon Glass <sjg@chromium.org> wrote:
> On 17 June 2015 at 13:33, Hans de Goede <hdegoede@redhat.com> wrote:
>> The device-model usb_legacy_port_reset function cApplied to u-boot-dm, thanks!
alls the device-model
>> usb_port_reset function which is a 1 on 1 copy of the non dm
>> usb_legacy_port_reset and this is the only use of usb_port_reset in all
>> of u-boot.
>>
>> Drop both, and alway use the usb_legacy_port_reset() version in
>> common/usb.c .
>>
>> Also while at it make it static as it is only used in common/usb.c .
>>
>> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
>> ---
>>  common/usb.c                  |  4 +---
>>  drivers/usb/host/usb-uclass.c | 29 -----------------------------
>>  include/usb.h                 |  8 --------
>>  3 files changed, 1 insertion(+), 40 deletions(-)
>
> Acked-by: Simon Glass <sjg@chromium.org>

Applied to u-boot-dm/next, thanks!
diff mbox

Patch

diff --git a/common/usb.c b/common/usb.c
index 7ff8ac5..4ddf98f 100644
--- a/common/usb.c
+++ b/common/usb.c
@@ -911,8 +911,7 @@  __weak int usb_alloc_device(struct usb_device *udev)
 }
 #endif /* !CONFIG_DM_USB */
 
-#ifndef CONFIG_DM_USB
-int usb_legacy_port_reset(struct usb_device *hub, int portnr)
+static int usb_legacy_port_reset(struct usb_device *hub, int portnr)
 {
 	if (hub) {
 		unsigned short portstatus;
@@ -930,7 +929,6 @@  int usb_legacy_port_reset(struct usb_device *hub, int portnr)
 
 	return 0;
 }
-#endif
 
 static int get_descriptor_len(struct usb_device *dev, int len, int expect_len)
 {
diff --git a/drivers/usb/host/usb-uclass.c b/drivers/usb/host/usb-uclass.c
index 6e86f4a..10d4712 100644
--- a/drivers/usb/host/usb-uclass.c
+++ b/drivers/usb/host/usb-uclass.c
@@ -310,35 +310,6 @@  int usb_post_bind(struct udevice *dev)
 	return dm_scan_fdt_node(dev, gd->fdt_blob, dev->of_offset, false);
 }
 
-int usb_port_reset(struct usb_device *parent, int portnr)
-{
-	unsigned short portstatus;
-	int ret;
-
-	debug("%s: start\n", __func__);
-
-	if (parent) {
-		/* reset the port for the second time */
-		assert(portnr > 0);
-		debug("%s: reset %d\n", __func__, portnr - 1);
-		ret = legacy_hub_port_reset(parent, portnr - 1, &portstatus);
-		if (ret < 0) {
-			printf("\n     Couldn't reset port %i\n", portnr);
-			return ret;
-		}
-	} else {
-		debug("%s: reset root\n", __func__);
-		usb_reset_root_port();
-	}
-
-	return 0;
-}
-
-int usb_legacy_port_reset(struct usb_device *parent, int portnr)
-{
-	return usb_port_reset(parent, portnr);
-}
-
 int usb_setup_ehci_gadget(struct ehci_ctrl **ctlrp)
 {
 	struct usb_platdata *plat;
diff --git a/include/usb.h b/include/usb.h
index dca512d..6b5d536 100644
--- a/include/usb.h
+++ b/include/usb.h
@@ -730,14 +730,6 @@  int usb_reset_root_port(void);
 struct usb_device *usb_get_dev_index(struct udevice *bus, int index);
 
 /**
- * usb_legacy_port_reset() - Legacy function to reset a hub port
- *
- * @hub:	Hub device
- * @portnr:	Port number (1=first)
- */
-int usb_legacy_port_reset(struct usb_device *hub, int portnr);
-
-/**
  * usb_setup_device() - set up a device ready for use
  *
  * @dev:	USB device pointer. This need not be a real device - it is