From patchwork Wed Apr 19 03:59:47 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: wuliangfeng X-Patchwork-Id: 9686801 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 84EC560375 for ; Wed, 19 Apr 2017 04:00:41 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 653612838E for ; Wed, 19 Apr 2017 04:00:41 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 57A07283F1; Wed, 19 Apr 2017 04:00:41 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-1.9 required=2.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID autolearn=ham version=3.3.1 Received: from bombadil.infradead.org (bombadil.infradead.org [65.50.211.133]) (using TLSv1.2 with cipher AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id B57C62838E for ; Wed, 19 Apr 2017 04:00:40 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender:Content-Type: Content-Transfer-Encoding:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:Message-ID:From: References:To:Subject:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=ixyyrpxkmemQI2I+PLrdVH/DtlWXtgkdxkHEzqZ3zVI=; b=uB/WqyIJOnKXSPNTG+Cod/zzb XEXP9RthqmdKg2Lt2Oh4xxj/3OfGWHQZT7hSdBihP5i1/4PRTF0TlmHZtT/Fud7BGeJQY0rCfEKk6 oHFFDer8bDMPOtR2YeNgqdE0JAaX93S8cd3Y5Hh36LUGXbgRLkg0upZn1iLgrXbaPqHV0llXxE9kc ObILMZm1ThqE5P02eIffuJxkLQTd8AqKuo8DR2Fa5fkCRtM4xDPWZohaUn1O54G8CiQZPNP7QQ2R0 n1LEnjvKukR1ybjDMwyULL/RCnWuMloQ/09ovFyaDbh6+2x/wHv/LJhsOhtFNiSb/D+nD7ejaVDer LJ3syc1vg==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux)) id 1d0gnH-0004hx-0U; Wed, 19 Apr 2017 04:00:39 +0000 Received: from regular1.263xmail.com ([211.150.99.133]) by bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux)) id 1d0gnD-0004MF-CE for linux-rockchip@lists.infradead.org; Wed, 19 Apr 2017 04:00:37 +0000 Received: from wulf?rock-chips.com (unknown [192.168.165.105]) by regular1.263xmail.com (Postfix) with ESMTP id 4DD739058; Wed, 19 Apr 2017 11:59:59 +0800 (CST) X-263anti-spam: KSV:0; X-MAIL-GRAY: 0 X-MAIL-DELIVERY: 1 X-KSVirus-check: 0 X-ABS-CHECKED: 4 Received: from smtp.263.net (sg-smtp01.263.net [54.255.195.220]) by smtp.263.net (Postfix) with ESMTP id 9073E422; Wed, 19 Apr 2017 11:59:50 +0800 (CST) Received: from [192.168.60.27] (localhost.localdomain [127.0.0.1]) by smtp.263.net (Postfix) with ESMTP id 360FADF30E; Wed, 19 Apr 2017 11:59:50 +0800 (CST) X-RL-SENDER: wulf@rock-chips.com X-FST-TO: daniel.meng@rock-chips.com X-SENDER-IP: 103.29.142.67 X-LOGIN-NAME: wulf@rock-chips.com X-UNIQUE-TAG: <85f6d68fd178b04bdeef52c0890f035f> X-ATTACHMENT-NUM: 0 X-SENDER: wulf@rock-chips.com X-DNS-TYPE: 0 Received: from [192.168.60.27] (unknown [103.29.142.67]) by smtp.263.net (Postfix) whith ESMTP id 5944O4YJV0; Wed, 19 Apr 2017 11:59:51 +0800 (CST) Subject: Re: [PATCH v2] usb: dwc3: add disable u2mac linestate check quirk To: Guenter Roeck , William Wu , Felipe Balbi , johnyoun@synopsys.com References: <1492492659-19266-1-git-send-email-william.wu@rock-chips.com> From: wlf Message-ID: <67e21e78-2f4e-47eb-38f5-734fe13dccab@rock-chips.com> Date: Wed, 19 Apr 2017 11:59:47 +0800 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.8.0 MIME-Version: 1.0 In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20170418_210035_979726_F36DA569 X-CRM114-Status: GOOD ( 16.79 ) X-BeenThere: linux-rockchip@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: Upstream kernel work for Rockchip platforms List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Tao Huang , devicetree@vger.kernel.org, =?UTF-8?Q?Heiko_St=c3=bcbner?= , Frank Wang , gregkh@linuxfoundation.org, linux-usb@vger.kernel.org, linux-kernel , Doug Anderson , "open list:ARM/Rockchip SoC..." , Rob Herring , Brian Norris , daniel.meng@rock-chips.com Sender: "Linux-rockchip" Errors-To: linux-rockchip-bounces+patchwork-linux-rockchip=patchwork.kernel.org@lists.infradead.org X-Virus-Scanned: ClamAV using ClamSMTP Dear Guenter, 在 2017年04月18日 21:18, Guenter Roeck 写道: > On Mon, Apr 17, 2017 at 10:17 PM, William Wu wrote: >> This patch adds a quirk to disable USB 2.0 MAC linestate check >> during HS transmit. Refer the dwc3 databook, we can use it for >> some special platforms if the linestate not reflect the expected >> line state(J) during transmission. >> >> When use this quirk, the controller implements a fixed 40-bit >> TxEndDelay after the packet is given on UTMI and ignores the >> linestate during the transmit of a token (during token-to-token >> and token-to-data IPGAP). >> >> On some rockchip platforms (e.g. rk3399), it requires to disable >> the u2mac linestate check to decrease the SSPLIT token to SETUP >> token inter-packet delay from 566ns to 466ns, and fix the issue >> that FS/LS devices not recognized if inserted through USB 3.0 HUB. >> >> Signed-off-by: William Wu >> --- >> Changes in v2: >> - fix coding style >> >> Documentation/devicetree/bindings/usb/dwc3.txt | 2 ++ >> drivers/usb/dwc3/core.c | 14 ++++++++++---- >> drivers/usb/dwc3/core.h | 4 ++++ >> 3 files changed, 16 insertions(+), 4 deletions(-) >> >> diff --git a/Documentation/devicetree/bindings/usb/dwc3.txt b/Documentation/devicetree/bindings/usb/dwc3.txt >> index f658f39..6a89f0c 100644 >> --- a/Documentation/devicetree/bindings/usb/dwc3.txt >> +++ b/Documentation/devicetree/bindings/usb/dwc3.txt >> @@ -45,6 +45,8 @@ Optional properties: >> a free-running PHY clock. >> - snps,dis-del-phy-power-chg-quirk: when set core will change PHY power >> from P0 to P1/P2/P3 without delay. >> + - snps,tx-ipgap-linecheck-dis-quirk: when set, disable u2mac linestate check >> + during HS transmit. > All other disable-something quirks are named > "snps,dis-something-quirk". Maybe use the same naming convention ? Yes, good idea！ I will fix it with "snps,dis-tx-ipgap-linecheck-quirk" in next patch verison. Thanks:-) > >> - snps,is-utmi-l1-suspend: true when DWC3 asserts output signal >> utmi_l1_suspend_n, false when asserts utmi_sleep_n >> - snps,hird-threshold: HIRD threshold >> diff --git a/drivers/usb/dwc3/core.c b/drivers/usb/dwc3/core.c >> index 455d89a..03429c5 100644 >> --- a/drivers/usb/dwc3/core.c >> +++ b/drivers/usb/dwc3/core.c >> @@ -796,15 +796,19 @@ static int dwc3_core_init(struct dwc3 *dwc) >> dwc3_writel(dwc->regs, DWC3_GUCTL2, reg); >> } >> >> + reg = dwc3_readl(dwc->regs, DWC3_GUCTL1); >> + > My understanding is that the register was only introduced with dwc3 > revision 2.50a. Is it ok to read and write it unconditionally ? Yes, refer to dwc3 databook, the DWC3_GUCTL1 was introduced since 2.50a. Maybe it's better to read and write it only when we know our controller version. Is it good to fix it like the following patch? But this patch has a problem that we need to read and write the register twice if our controller verison > = 2.90a, and need this quirk. Thank you！ >> /* >> * Enable hardware control of sending remote wakeup in HS when >> * the device is in the L1 state. >> */ >> - if (dwc->revision >= DWC3_REVISION_290A) { >> - reg = dwc3_readl(dwc->regs, DWC3_GUCTL1); >> + if (dwc->revision >= DWC3_REVISION_290A) >> reg |= DWC3_GUCTL1_DEV_L1_EXIT_BY_HW; >> - dwc3_writel(dwc->regs, DWC3_GUCTL1, reg); >> - } >> + >> + if (dwc->tx_ipgap_linecheck_dis_quirk) >> + reg |= DWC3_GUCTL1_TX_IPGAP_LINECHECK_DIS; >> + >> + dwc3_writel(dwc->regs, DWC3_GUCTL1, reg); >> >> return 0; >> >> @@ -1023,6 +1027,8 @@ static void dwc3_get_properties(struct dwc3 *dwc) >> "snps,dis-u2-freeclk-exists-quirk"); >> dwc->dis_del_phy_power_chg_quirk = device_property_read_bool(dev, >> "snps,dis-del-phy-power-chg-quirk"); >> + dwc->tx_ipgap_linecheck_dis_quirk = device_property_read_bool(dev, >> + "snps,tx-ipgap-linecheck-dis-quirk"); >> >> dwc->tx_de_emphasis_quirk = device_property_read_bool(dev, >> "snps,tx_de_emphasis_quirk"); >> diff --git a/drivers/usb/dwc3/core.h b/drivers/usb/dwc3/core.h >> index 981c77f..3c2537b 100644 >> --- a/drivers/usb/dwc3/core.h >> +++ b/drivers/usb/dwc3/core.h >> @@ -204,6 +204,7 @@ >> #define DWC3_GCTL_DSBLCLKGTNG BIT(0) >> >> /* Global User Control 1 Register */ >> +#define DWC3_GUCTL1_TX_IPGAP_LINECHECK_DIS BIT(28) >> #define DWC3_GUCTL1_DEV_L1_EXIT_BY_HW BIT(24) >> >> /* Global USB2 PHY Configuration Register */ >> @@ -850,6 +851,8 @@ struct dwc3_scratchpad_array { >> * provide a free-running PHY clock. >> * @dis_del_phy_power_chg_quirk: set if we disable delay phy power >> * change quirk. >> + * @tx_ipgap_linecheck_dis_quirk: set if we disable u2mac linestate >> + * check during HS transmit. >> * @tx_de_emphasis_quirk: set if we enable Tx de-emphasis quirk >> * @tx_de_emphasis: Tx de-emphasis value >> * 0 - -6dB de-emphasis >> @@ -1004,6 +1007,7 @@ struct dwc3 { >> unsigned dis_rxdet_inp3_quirk:1; >> unsigned dis_u2_freeclk_exists_quirk:1; >> unsigned dis_del_phy_power_chg_quirk:1; >> + unsigned tx_ipgap_linecheck_dis_quirk:1; >> >> unsigned tx_de_emphasis_quirk:1; >> unsigned tx_de_emphasis:2; >> -- >> 2.0.0 >> >> > > --- a/drivers/usb/dwc3/core.c +++ b/drivers/usb/dwc3/core.c @@ -806,6 +806,12 @@ static int dwc3_core_init(struct dwc3 *dwc) dwc3_writel(dwc->regs, DWC3_GUCTL1, reg); } + if (dwc->dis_tx_ipgap_linecheck_quirk) { + reg = dwc3_readl(dwc->regs, DWC3_GUCTL1); + reg |= DWC3_GUCTL1_TX_IPGAP_LINECHECK_DIS; + dwc3_writel(dwc->regs, DWC3_GUCTL1, reg); + } + Hi John & Felipe, Could you provide me some suggestion？