From: Eugeniu Rosca <erosca@de.adit-jv.com>
To: Hardik Gajjar <hgajjar@de.adit-jv.com>
Cc: <gregkh@linuxfoundation.org>, <stern@rowland.harvard.edu>,
<thinhn@synopsys.com>, <Kento.A.Kobayashi@sony.com>,
<atmgnd@outlook.com>, <linux-usb@vger.kernel.org>,
<andrew_gabbasov@mentor.com>, <erosca@de.adit-jv.com>,
<linux-renesas-soc@vger.kernel.org>,
Eugeniu Rosca <roscaeugeniu@gmail.com>
Subject: Re: [PATCH v3] USB: hub: Fix the broken detection of USB3 device in SMSC hub
Date: Thu, 6 Feb 2020 12:06:16 +0100 [thread overview]
Message-ID: <20200206110616.GA13629@lxhi-065.adit-jv.com> (raw)
In-Reply-To: <1580838253-31822-1-git-send-email-hgajjar@de.adit-jv.com>
Hi Hardik,
On Tue, Feb 04, 2020 at 06:44:13PM +0100, 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
Thanks for the comprehensible and accurate write-up.
> diff --git a/drivers/usb/core/hub.h b/drivers/usb/core/hub.h
> index a9e24e4..2fe9c9f 100644
> --- a/drivers/usb/core/hub.h
> +++ b/drivers/usb/core/hub.h
> @@ -61,6 +61,7 @@ struct usb_hub {
> unsigned quiescing:1;
> unsigned disconnected:1;
> unsigned in_reset:1;
> + unsigned quirk_disable_autosuspend:1;
I think it would make sense to respin the patch once again with the
above spaces converted to tabs, for consistency.
I would also include Alan's Acked-by (thanks!) and since Alan reshaped
the patch significantly in the last iteration, I would add his
Suggested-by too.
In addition to the above (please, also include):
Tested-by: Eugeniu Rosca <erosca@de.adit-jv.com>
Thanks!
--
Best Regards
Eugeniu Rosca
prev parent reply other threads:[~2020-02-06 11:06 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-02-04 17:44 [PATCH v3] USB: hub: Fix the broken detection of USB3 device in SMSC hub Hardik Gajjar
2020-02-04 18:10 ` Alan Stern
2020-02-06 11:06 ` Eugeniu Rosca [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200206110616.GA13629@lxhi-065.adit-jv.com \
--to=erosca@de.adit-jv.com \
--cc=Kento.A.Kobayashi@sony.com \
--cc=andrew_gabbasov@mentor.com \
--cc=atmgnd@outlook.com \
--cc=gregkh@linuxfoundation.org \
--cc=hgajjar@de.adit-jv.com \
--cc=linux-renesas-soc@vger.kernel.org \
--cc=linux-usb@vger.kernel.org \
--cc=roscaeugeniu@gmail.com \
--cc=stern@rowland.harvard.edu \
--cc=thinhn@synopsys.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).