From: Srinath Mannam <srinath.mannam@broadcom.com> To: Rob Herring <robh@kernel.org> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>, Mathias Nyman <mathias.nyman@intel.com>, Mark Rutland <mark.rutland@arm.com>, Linux USB List <linux-usb@vger.kernel.org>, devicetree@vger.kernel.org, Linux Kernel Mailing List <linux-kernel@vger.kernel.org> Subject: Re: [PATCH 1/2] dt-bindings: usb-xhci: Add usb-phy-port-reset property Date: Wed, 27 Feb 2019 22:38:27 +0530 [thread overview] Message-ID: <CABe79T7=uhVfy4x+L0Vi6g5fiT=-WKnnZmNn5XpZrvOiUzJOaA@mail.gmail.com> (raw) In-Reply-To: <CAL_Jsq+ZOWyY7p=ZjRnB85RY=fcNXu7xiH5j-1-OkuAmXqCFxw@mail.gmail.com> Hi Rob, Thanks for the information. Please find my comments below. Regards, Srinath. On Tue, Feb 26, 2019 at 11:33 PM Rob Herring <robh@kernel.org> wrote: > > On Mon, Feb 25, 2019 at 10:57 PM Srinath Mannam > <srinath.mannam@broadcom.com> wrote: > > > > Hi Rob, > > Thanks for the review, Please see my comments below in line. > > > > Regards, > > Srinath. > > On Tue, Feb 26, 2019 at 3:08 AM Rob Herring <robh@kernel.org> wrote: > > > > > > On Tue, Feb 05, 2019 at 11:48:53AM +0530, Srinath Mannam wrote: > > > > Add usb-phy-port-reset optional property to set quirk in xhci platform > > > > driver which forces USB port PHY reset on port disconnect event. > > > > > > > > Signed-off-by: Srinath Mannam <srinath.mannam@broadcom.com> > > > > Reviewed-by: Ray Jui <ray.jui@broadcom.com> > > > > --- > > > > Documentation/devicetree/bindings/usb/usb-xhci.txt | 1 + > > > > 1 file changed, 1 insertion(+) > > > > > > > > diff --git a/Documentation/devicetree/bindings/usb/usb-xhci.txt b/Documentation/devicetree/bindings/usb/usb-xhci.txt > > > > index fea8b15..ecbdb15 100644 > > > > --- a/Documentation/devicetree/bindings/usb/usb-xhci.txt > > > > +++ b/Documentation/devicetree/bindings/usb/usb-xhci.txt > > > > @@ -40,6 +40,7 @@ Optional properties: > > > > - usb3-lpm-capable: determines if platform is USB3 LPM capable > > > > - quirk-broken-port-ped: set if the controller has broken port disable mechanism > > > > - imod-interval-ns: default interrupt moderation interval is 5000ns > > > > + - usb-phy-port-reset: set this to do USB PORT PHY reset while disconnect > > > > - phys : see usb-hcd.txt in the current directory > > > > > > This should be implied by the HCI or phy compatible string (depending > > > on who exactly needs the quirky behavior). > > Stingray USB HS PHY connected to xHCI port has an issue, if full speed > > devices connected to this port then > > after all High Speed devices connected to this port are detected at > > full speed instead of high speed. > > So that we need to do PHY (which is connected to port) reset on xHCI > > port disconnect event. > > That is the reason we required to add quirk in xHCI. > > So, by looking at the xhci host and phy compatible strings (or maybe > just the phy) you can determine whether you need to reset the port or > not. All the information you need is in DT already. xHCI controller in our SOC has three ports each port has one PHY(SS/HS) connected to it. HS PHY has to reset on its corresponding port disconnect event. port disconnect event is captured in xHCI host framework so, quirk has to be registered in xHCI framework only. But we are using "generic-xhci" generic compatible string for our xHCI controller. As per your advice, we will add new compatible string in xhci-plat.c driver for our xHCI controller and will add quirk part of that. Thank you. > > Rob
WARNING: multiple messages have this Message-ID (diff)
From: Srinath Mannam <srinath.mannam@broadcom.com> To: Rob Herring <robh@kernel.org> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>, Mathias Nyman <mathias.nyman@intel.com>, Mark Rutland <mark.rutland@arm.com>, Linux USB List <linux-usb@vger.kernel.org>, devicetree@vger.kernel.org, Linux Kernel Mailing List <linux-kernel@vger.kernel.org> Subject: [1/2] dt-bindings: usb-xhci: Add usb-phy-port-reset property Date: Wed, 27 Feb 2019 22:38:27 +0530 [thread overview] Message-ID: <CABe79T7=uhVfy4x+L0Vi6g5fiT=-WKnnZmNn5XpZrvOiUzJOaA@mail.gmail.com> (raw) Hi Rob, Thanks for the information. Please find my comments below. Regards, Srinath. On Tue, Feb 26, 2019 at 11:33 PM Rob Herring <robh@kernel.org> wrote: > > On Mon, Feb 25, 2019 at 10:57 PM Srinath Mannam > <srinath.mannam@broadcom.com> wrote: > > > > Hi Rob, > > Thanks for the review, Please see my comments below in line. > > > > Regards, > > Srinath. > > On Tue, Feb 26, 2019 at 3:08 AM Rob Herring <robh@kernel.org> wrote: > > > > > > On Tue, Feb 05, 2019 at 11:48:53AM +0530, Srinath Mannam wrote: > > > > Add usb-phy-port-reset optional property to set quirk in xhci platform > > > > driver which forces USB port PHY reset on port disconnect event. > > > > > > > > Signed-off-by: Srinath Mannam <srinath.mannam@broadcom.com> > > > > Reviewed-by: Ray Jui <ray.jui@broadcom.com> > > > > --- > > > > Documentation/devicetree/bindings/usb/usb-xhci.txt | 1 + > > > > 1 file changed, 1 insertion(+) > > > > > > > > diff --git a/Documentation/devicetree/bindings/usb/usb-xhci.txt b/Documentation/devicetree/bindings/usb/usb-xhci.txt > > > > index fea8b15..ecbdb15 100644 > > > > --- a/Documentation/devicetree/bindings/usb/usb-xhci.txt > > > > +++ b/Documentation/devicetree/bindings/usb/usb-xhci.txt > > > > @@ -40,6 +40,7 @@ Optional properties: > > > > - usb3-lpm-capable: determines if platform is USB3 LPM capable > > > > - quirk-broken-port-ped: set if the controller has broken port disable mechanism > > > > - imod-interval-ns: default interrupt moderation interval is 5000ns > > > > + - usb-phy-port-reset: set this to do USB PORT PHY reset while disconnect > > > > - phys : see usb-hcd.txt in the current directory > > > > > > This should be implied by the HCI or phy compatible string (depending > > > on who exactly needs the quirky behavior). > > Stingray USB HS PHY connected to xHCI port has an issue, if full speed > > devices connected to this port then > > after all High Speed devices connected to this port are detected at > > full speed instead of high speed. > > So that we need to do PHY (which is connected to port) reset on xHCI > > port disconnect event. > > That is the reason we required to add quirk in xHCI. > > So, by looking at the xhci host and phy compatible strings (or maybe > just the phy) you can determine whether you need to reset the port or > not. All the information you need is in DT already. xHCI controller in our SOC has three ports each port has one PHY(SS/HS) connected to it. HS PHY has to reset on its corresponding port disconnect event. port disconnect event is captured in xHCI host framework so, quirk has to be registered in xHCI framework only. But we are using "generic-xhci" generic compatible string for our xHCI controller. As per your advice, we will add new compatible string in xhci-plat.c driver for our xHCI controller and will add quirk part of that. Thank you. > > Rob
next prev parent reply other threads:[~2019-02-27 17:08 UTC|newest] Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-02-05 6:18 [PATCH 0/2] Reset xHCI port PHY on disconnect Srinath Mannam 2019-02-05 6:18 ` [PATCH 1/2] dt-bindings: usb-xhci: Add usb-phy-port-reset property Srinath Mannam 2019-02-05 6:18 ` [1/2] " Srinath Mannam 2019-02-25 21:38 ` [PATCH 1/2] " Rob Herring 2019-02-25 21:38 ` [1/2] " Rob Herring 2019-02-26 4:57 ` [PATCH 1/2] " Srinath Mannam 2019-02-26 4:57 ` [1/2] " Srinath Mannam 2019-02-26 18:02 ` [PATCH 1/2] " Rob Herring 2019-02-26 18:02 ` [1/2] " Rob Herring 2019-02-27 17:08 ` Srinath Mannam [this message] 2019-02-27 17:08 ` Srinath Mannam 2019-02-05 6:18 ` [PATCH 2/2] drivers: xhci: Add quirk to reset xHCI port PHY Srinath Mannam 2019-02-05 6:18 ` [2/2] " Srinath Mannam 2019-02-07 15:06 ` [PATCH 2/2] " Mathias Nyman 2019-02-07 15:06 ` [2/2] " Mathias Nyman 2019-02-07 15:17 ` [PATCH 2/2] " Srinath Mannam 2019-02-07 15:17 ` [2/2] " Srinath Mannam 2019-02-08 12:35 ` [PATCH 2/2] " Mathias Nyman 2019-02-08 12:35 ` [2/2] " Mathias Nyman 2019-02-08 18:48 ` [PATCH 2/2] " Srinath Mannam 2019-02-08 18:48 ` [2/2] " Srinath Mannam
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='CABe79T7=uhVfy4x+L0Vi6g5fiT=-WKnnZmNn5XpZrvOiUzJOaA@mail.gmail.com' \ --to=srinath.mannam@broadcom.com \ --cc=devicetree@vger.kernel.org \ --cc=gregkh@linuxfoundation.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-usb@vger.kernel.org \ --cc=mark.rutland@arm.com \ --cc=mathias.nyman@intel.com \ --cc=robh@kernel.org \ /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: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
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.