From patchwork Wed Mar 6 09:19:43 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Felipe Balbi X-Patchwork-Id: 2224111 Return-Path: X-Original-To: patchwork-linux-arm@patchwork.kernel.org Delivered-To: patchwork-process-083081@patchwork1.kernel.org Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) by patchwork1.kernel.org (Postfix) with ESMTP id 0561B3FC8A for ; Wed, 6 Mar 2013 09:23:19 +0000 (UTC) Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.80.1 #2 (Red Hat Linux)) id 1UDAWG-00060U-5Q; Wed, 06 Mar 2013 09:20:16 +0000 Received: from comal.ext.ti.com ([198.47.26.152]) by merlin.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1UDAWC-0005zm-6o for linux-arm-kernel@lists.infradead.org; Wed, 06 Mar 2013 09:20:12 +0000 Received: from dlelxv30.itg.ti.com ([172.17.2.17]) by comal.ext.ti.com (8.13.7/8.13.7) with ESMTP id r269K0u7002360; Wed, 6 Mar 2013 03:20:00 -0600 Received: from DFLE72.ent.ti.com (dfle72.ent.ti.com [128.247.5.109]) by dlelxv30.itg.ti.com (8.13.8/8.13.8) with ESMTP id r269JxpO011683; Wed, 6 Mar 2013 03:19:59 -0600 Received: from dlelxv22.itg.ti.com (172.17.1.197) by dfle72.ent.ti.com (128.247.5.109) with Microsoft SMTP Server id 14.1.323.3; Wed, 6 Mar 2013 03:19:59 -0600 Received: from localhost (h78-4.vpn.ti.com [172.24.78.4]) by dlelxv22.itg.ti.com (8.13.8/8.13.8) with ESMTP id r269Jw9c015348; Wed, 6 Mar 2013 03:19:59 -0600 Date: Wed, 6 Mar 2013 11:19:43 +0200 From: Felipe Balbi To: kishon Subject: Re: [PATCH v2 8/8] ARM: dts: omap5: add dwc3 core dt data Message-ID: <20130306091942.GJ28587@arwen.pp.htv.fi> References: <1362494223-14151-1-git-send-email-kishon@ti.com> <1362494223-14151-9-git-send-email-kishon@ti.com> <20130305143934.GB12123@arwen.pp.htv.fi> <51370494.7090902@ti.com> MIME-Version: 1.0 In-Reply-To: <51370494.7090902@ti.com> User-Agent: Mutt/1.5.21 (2010-09-15) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20130306_042012_492700_4E1B9F4B X-CRM114-Status: GOOD ( 18.44 ) X-Spam-Score: -7.5 (-------) X-Spam-Report: SpamAssassin version 3.3.2 on merlin.infradead.org summary: Content analysis details: (-7.5 points) pts rule name description ---- ---------------------- -------------------------------------------------- -5.0 RCVD_IN_DNSWL_HI RBL: Sender listed at http://www.dnswl.org/, high trust [198.47.26.152 listed in list.dnswl.org] -0.0 SPF_PASS SPF: sender matches SPF record -0.6 RP_MATCHES_RCVD Envelope sender domain matches handover relay domain -1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1% [score: 0.0000] Cc: linux@arm.linux.org.uk, b-cousson@ti.com, linux-doc@vger.kernel.org, tony@atomide.com, devicetree-discuss@lists.ozlabs.org, linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org, rob.herring@calxeda.com, grant.likely@secretlab.ca, balbi@ti.com, rob@landley.net, linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.15 Precedence: list Reply-To: balbi@ti.com List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org Hi, On Wed, Mar 06, 2013 at 02:25:48PM +0530, kishon wrote: > Hi, > > On Tuesday 05 March 2013 08:09 PM, Felipe Balbi wrote: > >Hi, > > > >On Tue, Mar 05, 2013 at 08:07:03PM +0530, Kishon Vijay Abraham I wrote: > >>Add dwc3 core dt data as a subnode to dwc3 omap glue data in omap5 dt > >>data file. The information for the entered data node is available @ > >>Documentation/devicetree/bindings/usb/dwc3.txt > >> > >>Signed-off-by: Kishon Vijay Abraham I > >>--- > >> arch/arm/boot/dts/omap5.dtsi | 7 +++++++ > >> 1 file changed, 7 insertions(+) > >> > >>diff --git a/arch/arm/boot/dts/omap5.dtsi b/arch/arm/boot/dts/omap5.dtsi > >>index c4eb2ec..24dd69f 100644 > >>--- a/arch/arm/boot/dts/omap5.dtsi > >>+++ b/arch/arm/boot/dts/omap5.dtsi > >>@@ -514,6 +514,13 @@ > >> #size-cells = <1>; > >> utmi-mode = <2>; > >> ranges; > >>+ dwc3@4a030000 { > >>+ compatible = "synopsys,dwc3"; > >>+ reg = <0x4a030000 0x1000>; > >>+ interrupts = <0 92 4>; > >>+ usb-phy = <&usb2_phy>, <&usb3_phy>; > >>+ tx-fifo-resize; > > > >I think this should be merged with previous patch. Also, we need to > >check whether tx-fifo-resize is needed with ES2 or not ;-) > > yeah. ES2 works without tx-fifo-resize. Great, I guess we can deprecate that attribute. Can you check if ES2 still works with: starting on 1.94a (IIRC) core was supposed to control PHYs directly and since there's nobody in linux mainline using any version prior to 1.94a, we could drop that too. diff --git a/drivers/usb/dwc3/gadget.c b/drivers/usb/dwc3/gadget.c index 82e160e..cad0131 100644 --- a/drivers/usb/dwc3/gadget.c +++ b/drivers/usb/dwc3/gadget.c @@ -2029,6 +2029,8 @@ static void dwc3_gadget_usb3_phy_suspend(struct dwc3 *dwc, int suspend) { u32 reg; + return; + reg = dwc3_readl(dwc->regs, DWC3_GUSB3PIPECTL(0)); if (suspend) @@ -2043,6 +2045,8 @@ static void dwc3_gadget_usb2_phy_suspend(struct dwc3 *dwc, int suspend) { u32 reg; + return; + reg = dwc3_readl(dwc->regs, DWC3_GUSB2PHYCFG(0)); if (suspend)