All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2] USB: hub: Fix the broken detection of USB3 device in SMSC hub
@ 2020-01-30 17:06 Hardik Gajjar
  2020-01-30 18:09 ` Alan Stern
  0 siblings, 1 reply; 4+ messages in thread
From: Hardik Gajjar @ 2020-01-30 17:06 UTC (permalink / raw)
  To: gregkh, stern, thinhn, Kento.A.Kobayashi, atmgnd, linux-usb
  Cc: andrew_gabbasov, erosca, linux-renesas-soc, hgajjar

Renesas R-Car H3ULCB + Kingfisher Infotainment Board is either not able
to detect the USB3.0 mass storage devices or is detecting those as
USB2.0 high speed devices.

The explanation given by Renesas is that, due to a HW issue, the XHCI
driver does not wake up after going to sleep on connecting a USB3.0
device.

In order to mitigate that, disable the auto-suspend feature
specifically for SMSC hubs from hub_probe() function, as a quirk.

Renesas Kingfisher Infotainment Board has two USB3.0 ports (CN2) which
are connected via USB5534B 4-port SuperSpeed/Hi-Speed, low-power,
configurable hub controller.

[1] SanDisk USB 3.0 device detected as USB-2.0 before the patch
 [   74.036390] usb 5-1.1: new high-speed USB device number 4 using xhci-hcd
 [   74.061598] usb 5-1.1: New USB device found, idVendor=0781, idProduct=5581, bcdDevice= 1.00
 [   74.069976] usb 5-1.1: New USB device strings: Mfr=1, Product=2, SerialNumber=3
 [   74.077303] usb 5-1.1: Product: Ultra
 [   74.080980] usb 5-1.1: Manufacturer: SanDisk
 [   74.085263] usb 5-1.1: SerialNumber: 4C530001110208116550

[2] SanDisk USB 3.0 device detected as USB-3.0 after the patch
 [   34.565078] usb 6-1.1: new SuperSpeed Gen 1 USB device number 3 using xhci-hcd
 [   34.588719] usb 6-1.1: New USB device found, idVendor=0781, idProduct=5581, bcdDevice= 1.00
 [   34.597098] usb 6-1.1: New USB device strings: Mfr=1, Product=2, SerialNumber=3
 [   34.604430] usb 6-1.1: Product: Ultra
 [   34.608110] usb 6-1.1: Manufacturer: SanDisk
 [   34.612397] usb 6-1.1: SerialNumber: 4C530001110208116550

Signed-off-by: Hardik Gajjar <hgajjar@de.adit-jv.com>
---
Changes in v2:
 - [Alan Stern] Switched from pm_runtime_set_autosuspend_delay()
   to usb_autopm_get_interface()
 - Improved commit description
 - Rebased against v5.5
 - https://lore.kernel.org/linux-renesas-soc/1579876573-13741-1-git-send-email-hgajjar@de.adit-jv.com/

 drivers/usb/core/hub.c | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/drivers/usb/core/hub.c b/drivers/usb/core/hub.c
index 3405b14..4152f44 100644
--- a/drivers/usb/core/hub.c
+++ b/drivers/usb/core/hub.c
@@ -38,7 +38,9 @@
 #include "otg_whitelist.h"
 
 #define USB_VENDOR_GENESYS_LOGIC		0x05e3
+#define USB_VENDOR_SMSC				0x0424
 #define HUB_QUIRK_CHECK_PORT_AUTOSUSPEND	0x01
+#define HUB_QUIRK_DISABLE_AUTOSUSPEND		0x02
 
 #define USB_TP_TRANSMISSION_DELAY	40	/* ns */
 #define USB_TP_TRANSMISSION_DELAY_MAX	65535	/* ns */
@@ -1863,6 +1865,9 @@ static int hub_probe(struct usb_interface *intf, const struct usb_device_id *id)
 	if (id->driver_info & HUB_QUIRK_CHECK_PORT_AUTOSUSPEND)
 		hub->quirk_check_port_auto_suspend = 1;
 
+	if (id->driver_info & HUB_QUIRK_DISABLE_AUTOSUSPEND)
+		usb_autopm_get_interface(intf);
+
 	if (hub_configure(hub, &desc->endpoint[0].desc) >= 0)
 		return 0;
 
@@ -5599,6 +5604,10 @@ static void hub_event(struct work_struct *work)
 }
 
 static const struct usb_device_id hub_id_table[] = {
+    { .match_flags = USB_DEVICE_ID_MATCH_VENDOR | USB_DEVICE_ID_MATCH_INT_CLASS,
+      .idVendor = USB_VENDOR_SMSC,
+      .bInterfaceClass = USB_CLASS_HUB,
+      .driver_info = HUB_QUIRK_DISABLE_AUTOSUSPEND},
     { .match_flags = USB_DEVICE_ID_MATCH_VENDOR
 			| USB_DEVICE_ID_MATCH_INT_CLASS,
       .idVendor = USB_VENDOR_GENESYS_LOGIC,
-- 
2.7.4


^ permalink raw reply related	[flat|nested] 4+ messages in thread
* Re: [PATCH v2] USB: hub: Fix the broken detection of USB3 device in SMSC hub
  2020-01-30 17:06 [PATCH v2] USB: hub: Fix the broken detection of USB3 device in SMSC hub Hardik Gajjar
@ 2020-01-30 18:09 ` Alan Stern
  2020-01-31 10:32   ` Hardik Gajjar
  0 siblings, 1 reply; 4+ messages in thread
From: Alan Stern @ 2020-01-30 18:09 UTC (permalink / raw)
  To: Hardik Gajjar
  Cc: gregkh, thinhn, Kento.A.Kobayashi, atmgnd, linux-usb,
	andrew_gabbasov, erosca, linux-renesas-soc

On Thu, 30 Jan 2020, Hardik Gajjar wrote:

> Renesas R-Car H3ULCB + Kingfisher Infotainment Board is either not able
> to detect the USB3.0 mass storage devices or is detecting those as
> USB2.0 high speed devices.
> 
> The explanation given by Renesas is that, due to a HW issue, the XHCI
> driver does not wake up after going to sleep on connecting a USB3.0
> device.
> 
> In order to mitigate that, disable the auto-suspend feature
> specifically for SMSC hubs from hub_probe() function, as a quirk.
> 
> Renesas Kingfisher Infotainment Board has two USB3.0 ports (CN2) which
> are connected via USB5534B 4-port SuperSpeed/Hi-Speed, low-power,
> configurable hub controller.
> 
> [1] SanDisk USB 3.0 device detected as USB-2.0 before the patch
>  [   74.036390] usb 5-1.1: new high-speed USB device number 4 using xhci-hcd
>  [   74.061598] usb 5-1.1: New USB device found, idVendor=0781, idProduct=5581, bcdDevice= 1.00
>  [   74.069976] usb 5-1.1: New USB device strings: Mfr=1, Product=2, SerialNumber=3
>  [   74.077303] usb 5-1.1: Product: Ultra
>  [   74.080980] usb 5-1.1: Manufacturer: SanDisk
>  [   74.085263] usb 5-1.1: SerialNumber: 4C530001110208116550
> 
> [2] SanDisk USB 3.0 device detected as USB-3.0 after the patch
>  [   34.565078] usb 6-1.1: new SuperSpeed Gen 1 USB device number 3 using xhci-hcd
>  [   34.588719] usb 6-1.1: New USB device found, idVendor=0781, idProduct=5581, bcdDevice= 1.00
>  [   34.597098] usb 6-1.1: New USB device strings: Mfr=1, Product=2, SerialNumber=3
>  [   34.604430] usb 6-1.1: Product: Ultra
>  [   34.608110] usb 6-1.1: Manufacturer: SanDisk
>  [   34.612397] usb 6-1.1: SerialNumber: 4C530001110208116550
> 
> Signed-off-by: Hardik Gajjar <hgajjar@de.adit-jv.com>
> ---
> Changes in v2:
>  - [Alan Stern] Switched from pm_runtime_set_autosuspend_delay()
>    to usb_autopm_get_interface()
>  - Improved commit description
>  - Rebased against v5.5
>  - https://lore.kernel.org/linux-renesas-soc/1579876573-13741-1-git-send-email-hgajjar@de.adit-jv.com/
> 
>  drivers/usb/core/hub.c | 9 +++++++++
>  1 file changed, 9 insertions(+)
> 
> diff --git a/drivers/usb/core/hub.c b/drivers/usb/core/hub.c
> index 3405b14..4152f44 100644
> --- a/drivers/usb/core/hub.c
> +++ b/drivers/usb/core/hub.c
> @@ -38,7 +38,9 @@
>  #include "otg_whitelist.h"
>  
>  #define USB_VENDOR_GENESYS_LOGIC		0x05e3
> +#define USB_VENDOR_SMSC				0x0424
>  #define HUB_QUIRK_CHECK_PORT_AUTOSUSPEND	0x01
> +#define HUB_QUIRK_DISABLE_AUTOSUSPEND		0x02
>  
>  #define USB_TP_TRANSMISSION_DELAY	40	/* ns */
>  #define USB_TP_TRANSMISSION_DELAY_MAX	65535	/* ns */
> @@ -1863,6 +1865,9 @@ static int hub_probe(struct usb_interface *intf, const struct usb_device_id *id)
>  	if (id->driver_info & HUB_QUIRK_CHECK_PORT_AUTOSUSPEND)
>  		hub->quirk_check_port_auto_suspend = 1;
>  
> +	if (id->driver_info & HUB_QUIRK_DISABLE_AUTOSUSPEND)
> +		usb_autopm_get_interface(intf);
> +
>  	if (hub_configure(hub, &desc->endpoint[0].desc) >= 0)
>  		return 0;
>  

This isn't right.  If you call usb_autopm_get_interface() then at some 
later point you _must_ call usb_autopm_put_interface().  In this case, 
you would have to add these calls following the hub_configure() call 
(in the case where it returns an error) and in the hub_disconnect() 
routine.

Alan Stern

> @@ -5599,6 +5604,10 @@ static void hub_event(struct work_struct *work)
>  }
>  
>  static const struct usb_device_id hub_id_table[] = {
> +    { .match_flags = USB_DEVICE_ID_MATCH_VENDOR | USB_DEVICE_ID_MATCH_INT_CLASS,
> +      .idVendor = USB_VENDOR_SMSC,
> +      .bInterfaceClass = USB_CLASS_HUB,
> +      .driver_info = HUB_QUIRK_DISABLE_AUTOSUSPEND},
>      { .match_flags = USB_DEVICE_ID_MATCH_VENDOR
>  			| USB_DEVICE_ID_MATCH_INT_CLASS,
>        .idVendor = USB_VENDOR_GENESYS_LOGIC,
> 


^ permalink raw reply	[flat|nested] 4+ messages in thread
* Re: [PATCH v2] USB: hub: Fix the broken detection of USB3 device in SMSC hub
  2020-01-30 18:09 ` Alan Stern
@ 2020-01-31 10:32   ` Hardik Gajjar
  2020-01-31 15:12     ` Alan Stern
  0 siblings, 1 reply; 4+ messages in thread
From: Hardik Gajjar @ 2020-01-31 10:32 UTC (permalink / raw)
  To: Alan Stern
  Cc: Hardik Gajjar, gregkh, thinhn, Kento.A.Kobayashi, atmgnd,
	linux-usb, andrew_gabbasov, erosca, linux-renesas-soc

Hi Alan,

On Thu, Jan 30, 2020 at 01:09:10PM -0500, Alan Stern wrote:
> On Thu, 30 Jan 2020, Hardik Gajjar wrote:
> 
> > Renesas R-Car H3ULCB + Kingfisher Infotainment Board is either not able
> > to detect the USB3.0 mass storage devices or is detecting those as
> > USB2.0 high speed devices.
> > 
> > The explanation given by Renesas is that, due to a HW issue, the XHCI
> > driver does not wake up after going to sleep on connecting a USB3.0
> > device.
> > 
> > In order to mitigate that, disable the auto-suspend feature
> > specifically for SMSC hubs from hub_probe() function, as a quirk.
> > 
> > Renesas Kingfisher Infotainment Board has two USB3.0 ports (CN2) which
> > are connected via USB5534B 4-port SuperSpeed/Hi-Speed, low-power,
> > configurable hub controller.
> > 
> > [1] SanDisk USB 3.0 device detected as USB-2.0 before the patch
> >  [   74.036390] usb 5-1.1: new high-speed USB device number 4 using xhci-hcd
> >  [   74.061598] usb 5-1.1: New USB device found, idVendor=0781, idProduct=5581, bcdDevice= 1.00
> >  [   74.069976] usb 5-1.1: New USB device strings: Mfr=1, Product=2, SerialNumber=3
> >  [   74.077303] usb 5-1.1: Product: Ultra
> >  [   74.080980] usb 5-1.1: Manufacturer: SanDisk
> >  [   74.085263] usb 5-1.1: SerialNumber: 4C530001110208116550
> > 
> > [2] SanDisk USB 3.0 device detected as USB-3.0 after the patch
> >  [   34.565078] usb 6-1.1: new SuperSpeed Gen 1 USB device number 3 using xhci-hcd
> >  [   34.588719] usb 6-1.1: New USB device found, idVendor=0781, idProduct=5581, bcdDevice= 1.00
> >  [   34.597098] usb 6-1.1: New USB device strings: Mfr=1, Product=2, SerialNumber=3
> >  [   34.604430] usb 6-1.1: Product: Ultra
> >  [   34.608110] usb 6-1.1: Manufacturer: SanDisk
> >  [   34.612397] usb 6-1.1: SerialNumber: 4C530001110208116550
> > 
> > Signed-off-by: Hardik Gajjar <hgajjar@de.adit-jv.com>
> > ---
> > Changes in v2:
> >  - [Alan Stern] Switched from pm_runtime_set_autosuspend_delay()
> >    to usb_autopm_get_interface()
> >  - Improved commit description
> >  - Rebased against v5.5
> >  - https://lore.kernel.org/linux-renesas-soc/1579876573-13741-1-git-send-email-hgajjar@de.adit-jv.com/
> > 
> >  drivers/usb/core/hub.c | 9 +++++++++
> >  1 file changed, 9 insertions(+)
> > 
> > diff --git a/drivers/usb/core/hub.c b/drivers/usb/core/hub.c
> > index 3405b14..4152f44 100644
> > --- a/drivers/usb/core/hub.c
> > +++ b/drivers/usb/core/hub.c
> > @@ -38,7 +38,9 @@
> >  #include "otg_whitelist.h"
> >  
> >  #define USB_VENDOR_GENESYS_LOGIC		0x05e3
> > +#define USB_VENDOR_SMSC				0x0424
> >  #define HUB_QUIRK_CHECK_PORT_AUTOSUSPEND	0x01
> > +#define HUB_QUIRK_DISABLE_AUTOSUSPEND		0x02
> >  
> >  #define USB_TP_TRANSMISSION_DELAY	40	/* ns */
> >  #define USB_TP_TRANSMISSION_DELAY_MAX	65535	/* ns */
> > @@ -1863,6 +1865,9 @@ static int hub_probe(struct usb_interface *intf, const struct usb_device_id *id)
> >  	if (id->driver_info & HUB_QUIRK_CHECK_PORT_AUTOSUSPEND)
> >  		hub->quirk_check_port_auto_suspend = 1;
> >  
> > +	if (id->driver_info & HUB_QUIRK_DISABLE_AUTOSUSPEND)
> > +		usb_autopm_get_interface(intf);
> > +
> >  	if (hub_configure(hub, &desc->endpoint[0].desc) >= 0)
> >  		return 0;
> >  
> 
> This isn't right.  If you call usb_autopm_get_interface() then at some 
> later point you _must_ call usb_autopm_put_interface().  In this case, 
> you would have to add these calls following the hub_configure() call 
> (in the case where it returns an error) and in the hub_disconnect() 
> routine.
> 

Thanks for feedback.

I tried to call the usb_autopm_put_interface() after hub_configure()[1] but then,
detection of USB3 device is stopped. Perhaps, It will increase the counter
again and allow the hub to go into suspend/sleep Mode. I need to disable auto
suspend permanently for SMSC hub. 

Please,  give some suggestion. 

[1] diff --git a/drivers/usb/core/hub.c b/drivers/usb/core/hub.c
index 30977da..6b2a84b 100644
--- a/drivers/usb/core/hub.c
+++ b/drivers/usb/core/hub.c
@@ -1718,6 +1718,7 @@ static int hub_probe(struct usb_interface *intf, const struct usb_device_id *id)
        struct usb_host_interface *desc;
        struct usb_device *hdev;
        struct usb_hub *hub;
+       int ret;

        desc = intf->cur_altsetting;
        hdev = interface_to_usbdev(intf);
@@ -1827,11 +1828,17 @@ static int hub_probe(struct usb_interface *intf, const struct usb_device_id *id)
        if (id->driver_info & HUB_QUIRK_DISABLE_AUTOSUSPEND)
                usb_autopm_get_interface(intf);

-       if (hub_configure(hub, &desc->endpoint[0].desc) >= 0)
-               return 0;
+       ret = hub_configure(hub, &desc->endpoint[0].desc);

-       hub_disconnect(intf);
-       return -ENODEV;
+       if (id->driver_info & HUB_QUIRK_DISABLE_AUTOSUSPEND)
+               usb_autopm_put_interface(intf);
+
+       if (ret < 0) {
+               hub_disconnect(intf);
+               ret = -ENODEV;
+       }
+
+       return ret;
 }

> Alan Stern
> 

-- 
Best Regards,
Hardik Gajjar

^ permalink raw reply related	[flat|nested] 4+ messages in thread
* Re: [PATCH v2] USB: hub: Fix the broken detection of USB3 device in SMSC hub
  2020-01-31 10:32   ` Hardik Gajjar
@ 2020-01-31 15:12     ` Alan Stern
  0 siblings, 0 replies; 4+ messages in thread
From: Alan Stern @ 2020-01-31 15:12 UTC (permalink / raw)
  To: Hardik Gajjar
  Cc: gregkh, thinhn, Kento.A.Kobayashi, atmgnd, linux-usb,
	andrew_gabbasov, erosca, linux-renesas-soc

On Fri, 31 Jan 2020, Hardik Gajjar wrote:

> > > @@ -1863,6 +1865,9 @@ static int hub_probe(struct usb_interface *intf, const struct usb_device_id *id)
> > >  	if (id->driver_info & HUB_QUIRK_CHECK_PORT_AUTOSUSPEND)
> > >  		hub->quirk_check_port_auto_suspend = 1;
> > >  
> > > +	if (id->driver_info & HUB_QUIRK_DISABLE_AUTOSUSPEND)
> > > +		usb_autopm_get_interface(intf);
> > > +
> > >  	if (hub_configure(hub, &desc->endpoint[0].desc) >= 0)
> > >  		return 0;
> > >  
> > 
> > This isn't right.  If you call usb_autopm_get_interface() then at some 
> > later point you _must_ call usb_autopm_put_interface().  In this case, 
> > you would have to add these calls following the hub_configure() call 
> > (in the case where it returns an error) and in the hub_disconnect() 
> > routine.
> > 
> 
> Thanks for feedback.
> 
> I tried to call the usb_autopm_put_interface() after hub_configure()[1] but then,
> detection of USB3 device is stopped. Perhaps, It will increase the counter
> again and allow the hub to go into suspend/sleep Mode. I need to disable auto
> suspend permanently for SMSC hub. 
> 
> Please,  give some suggestion. 

What I mean is, you need to do something like this...

Alan Stern


Index: usb-devel/drivers/usb/core/hub.c
===================================================================
--- usb-devel.orig/drivers/usb/core/hub.c
+++ usb-devel/drivers/usb/core/hub.c
@@ -1730,6 +1730,9 @@ static void hub_disconnect(struct usb_in
 	kfree(hub->buffer);
 
 	pm_suspend_ignore_children(&intf->dev, false);
+	if (hub->quirk_disable_autosuspend)
+		usb_autopm_put_interface(intf);
+
 	kref_put(&hub->kref, hub_release);
 }
 
@@ -1862,6 +1865,11 @@ static int hub_probe(struct usb_interfac
 	if (id->driver_info & HUB_QUIRK_CHECK_PORT_AUTOSUSPEND)
 		hub->quirk_check_port_auto_suspend = 1;
 
+	if (id->driver_info & HUB_QUIRK_DISABLE_AUTOSUSPEND) {
+		hub->quirk_disable_autosuspend = 1;
+		usb_autopm_get_interface(intf);
+	}
+
 	if (hub_configure(hub, &desc->endpoint[0].desc) >= 0)
 		return 0;
 
Index: usb-devel/drivers/usb/core/hub.h
===================================================================
--- usb-devel.orig/drivers/usb/core/hub.h
+++ usb-devel/drivers/usb/core/hub.h
@@ -63,6 +63,7 @@ struct usb_hub {
 	unsigned		in_reset:1;
 
 	unsigned		quirk_check_port_auto_suspend:1;
+	unsigned		quirk_disable_autosuspend:1;
 
 	unsigned		has_indicators:1;
 	u8			indicator[USB_MAXCHILDREN];


^ permalink raw reply	[flat|nested] 4+ messages in thread
end of thread, other threads:[~2020-01-31 15:12 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-01-30 17:06 [PATCH v2] USB: hub: Fix the broken detection of USB3 device in SMSC hub Hardik Gajjar
2020-01-30 18:09 ` Alan Stern
2020-01-31 10:32   ` Hardik Gajjar
2020-01-31 15:12     ` Alan Stern

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.