Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
From:   Thinh Nguyen <Thinh.Nguyen@synopsys.com>
To:     "Claus H. Stovgaard" <cst@phaseone.com>,
        "linux-usb@vger.kernel.org" <linux-usb@vger.kernel.org>
CC:     Felipe Balbi <balbi@kernel.org>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        Anurag Kumar Vulisha <anurag.kumar.vulisha@xilinx.com>
Subject: Re: [PATCH] usb: gadget: configfs: Add lpm_Ux_disable
Thread-Topic: [PATCH] usb: gadget: configfs: Add lpm_Ux_disable
Thread-Index: AQHVBLXbiuLSRZBxxUeout6SXcNukg==
Date:   Tue, 7 May 2019 18:53:15 +0000
Message-ID: <30102591E157244384E984126FC3CB4F639E9057@us01wembx1.internal.synopsys.com>
References: <1557220655-123090-1-git-send-email-cst@phaseone.com>
 <1557221830.114189.8.camel@phaseone.com>
Accept-Language: en-US
Content-Language: en-US
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

Hi Claus,=0A=
=0A=
Claus H. Stovgaard wrote:=0A=
> On Tue, 2019-05-07 at 11:09 +0200, Claus H. Stovgaard wrote:=0A=
>> When combining dwc3 with an redriver for a USB Type-C device=0A=
>> solution, it=0A=
>> sometimes have problems with leaving U1/U2 for certain hosts,=0A=
>> resulting in=0A=
>> link training errors and reconnects. This create an interface via=0A=
>> configfs for disabling U1/U2, enabling a workaround for devices based=0A=
>> on=0A=
>> dwc3.=0A=
>>=0A=
> Sorry messed up a bit with a missing cover letter.=0A=
> This feature relates to Anurag patch [1] and a thread [2] from earlier =
=0A=
>=0A=
> Where Anurags patch focus on setting U1/U2 latency in the BOS=0A=
> descriptor from the devicetree, this patch focuses on having a configfs=
=0A=
> interface for forcing the UDC (here the dwc3) to not enable U1/U2 and=0A=
> reject the SET_SEL(U1/U2).=0A=
>=0A=
> Looking forward to input.=0A=
>=0A=
> [1] https://urldefense.proofpoint.com/v2/url?u=3Dhttps-3A__www.spinics.ne=
t_lists_linux-2Dusb_msg179732.html&d=3DDwIDaQ&c=3DDPL6_X_6JkXFx7AXWqB0tg&r=
=3Du9FYoxKtyhjrGFcyixFYqTjw1ZX0VsG2d8FCmzkTY-w&m=3DwKdyWmYpbW791LAm7rYwvFYx=
5E0bjENyXZzHvK4vyFo&s=3Des7kki6iuLJUp2rJnzP9alXKyfJPNSfyxTVCKKDd_rQ&e=3D=0A=
> [2] https://urldefense.proofpoint.com/v2/url?u=3Dhttps-3A__www.spinics.ne=
t_lists_linux-2Dusb_msg179393.html&d=3DDwIDaQ&c=3DDPL6_X_6JkXFx7AXWqB0tg&r=
=3Du9FYoxKtyhjrGFcyixFYqTjw1ZX0VsG2d8FCmzkTY-w&m=3DwKdyWmYpbW791LAm7rYwvFYx=
5E0bjENyXZzHvK4vyFo&s=3DcFTmO9wPf7b6TZxFUAAIJM0Z_wM1ttNIc1rct0uR6co&e=3D=0A=
>=0A=
>=0A=
=0A=
I'm not sure who will submit the patch to make change to DWC3 for=0A=
disabling U1/U2 (Anurag or you), but can you split your patch between=0A=
dwc3 and configfs.=0A=
=0A=
Thanks,=0A=
Thinh=0A=