All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] [PATCH] usb/xhci: remove XHCI init code from other usb controllers
@ 2014-03-24  1:52 Zhao Qiang
  2014-10-22 18:56 ` Marek Vasut
  0 siblings, 1 reply; 4+ messages in thread
From: Zhao Qiang @ 2014-03-24  1:52 UTC (permalink / raw)
  To: u-boot

The patch with commit id "020bbcb76b5be0d5406d2ae7c26dbdb013ead812"
adds some init codes for XHCI but not for other usb controllers.
And it will causes a bug for EHCI.
so use macro "#ifdef CONFIG_USB_XHCI" to run it just for XHCI.

Signed-off-by: Zhao Qiang <B45475@freescale.com>
---
 common/usb_hub.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/common/usb_hub.c b/common/usb_hub.c
index ffac0e7..7f791f1 100644
--- a/common/usb_hub.c
+++ b/common/usb_hub.c
@@ -90,12 +90,16 @@ static void usb_hub_power_on(struct usb_hub_device *hub)
 	int i;
 	struct usb_device *dev;
 	unsigned pgood_delay = hub->desc.bPwrOn2PwrGood * 2;
+
+#ifdef CONFIG_USB_XHCI
 	ALLOC_CACHE_ALIGN_BUFFER(struct usb_port_status, portsts, 1);
 	unsigned short portstatus;
 	int ret;
+#endif
 
 	dev = hub->pusb_dev;
 
+#ifdef CONFIG_USB_XHCI
 	/*
 	 * Enable power to the ports:
 	 * Here we Power-cycle the ports: aka,
@@ -132,6 +136,7 @@ static void usb_hub_power_on(struct usb_hub_device *hub)
 			continue;
 		}
 	}
+#endif
 
 	for (i = 0; i < dev->maxchild; i++) {
 		usb_set_port_feature(dev, i + 1, USB_PORT_FEAT_POWER);
-- 
1.8.5

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [U-Boot] [PATCH] usb/xhci: remove XHCI init code from other usb controllers
  2014-03-24  1:52 [U-Boot] [PATCH] usb/xhci: remove XHCI init code from other usb controllers Zhao Qiang
@ 2014-10-22 18:56 ` Marek Vasut
  2014-10-22 19:33   ` York Sun
  0 siblings, 1 reply; 4+ messages in thread
From: Marek Vasut @ 2014-10-22 18:56 UTC (permalink / raw)
  To: u-boot

On Monday, March 24, 2014 at 02:52:27 AM, Zhao Qiang wrote:
> The patch with commit id "020bbcb76b5be0d5406d2ae7c26dbdb013ead812"
> adds some init codes for XHCI but not for other usb controllers.
> And it will causes a bug for EHCI.
> so use macro "#ifdef CONFIG_USB_XHCI" to run it just for XHCI.
> 
> Signed-off-by: Zhao Qiang <B45475@freescale.com>

Is this still relevant? What kind of bug does this fix exactly?

Also, next time please CC me.

Best regards,
Marek Vasut

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [U-Boot] [PATCH] usb/xhci: remove XHCI init code from other usb controllers
  2014-10-22 18:56 ` Marek Vasut
@ 2014-10-22 19:33   ` York Sun
  2014-10-22 19:37     ` Marek Vasut
  0 siblings, 1 reply; 4+ messages in thread
From: York Sun @ 2014-10-22 19:33 UTC (permalink / raw)
  To: u-boot

On 10/22/2014 11:56 AM, Marek Vasut wrote:
> On Monday, March 24, 2014 at 02:52:27 AM, Zhao Qiang wrote:
>> The patch with commit id "020bbcb76b5be0d5406d2ae7c26dbdb013ead812"
>> adds some init codes for XHCI but not for other usb controllers.
>> And it will causes a bug for EHCI.
>> so use macro "#ifdef CONFIG_USB_XHCI" to run it just for XHCI.
>>
>> Signed-off-by: Zhao Qiang <B45475@freescale.com>
> 
> Is this still relevant? What kind of bug does this fix exactly?
> 
> Also, next time please CC me.
> 

I think so. Qiang just pinged me yesterday. Do you think it is reasonable to
conditionally power cycle the ports?

York

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [U-Boot] [PATCH] usb/xhci: remove XHCI init code from other usb controllers
  2014-10-22 19:33   ` York Sun
@ 2014-10-22 19:37     ` Marek Vasut
  0 siblings, 0 replies; 4+ messages in thread
From: Marek Vasut @ 2014-10-22 19:37 UTC (permalink / raw)
  To: u-boot

On Wednesday, October 22, 2014 at 09:33:21 PM, York Sun wrote:
> On 10/22/2014 11:56 AM, Marek Vasut wrote:
> > On Monday, March 24, 2014 at 02:52:27 AM, Zhao Qiang wrote:
> >> The patch with commit id "020bbcb76b5be0d5406d2ae7c26dbdb013ead812"
> >> adds some init codes for XHCI but not for other usb controllers.
> >> And it will causes a bug for EHCI.
> >> so use macro "#ifdef CONFIG_USB_XHCI" to run it just for XHCI.
> >> 
> >> Signed-off-by: Zhao Qiang <B45475@freescale.com>
> > 
> > Is this still relevant? What kind of bug does this fix exactly?
> > 
> > Also, next time please CC me.
> 
> I think so. Qiang just pinged me yesterday. Do you think it is reasonable
> to conditionally power cycle the ports?

CCing samsung guys. I have yet to hear what issue do you observe with this 
patch.

Best regards,
Marek Vasut

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2014-10-22 19:37 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-03-24  1:52 [U-Boot] [PATCH] usb/xhci: remove XHCI init code from other usb controllers Zhao Qiang
2014-10-22 18:56 ` Marek Vasut
2014-10-22 19:33   ` York Sun
2014-10-22 19:37     ` Marek Vasut

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.