From patchwork Wed Sep 2 14:34:57 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Felipe Balbi X-Patchwork-Id: 52982 Return-Path: X-Original-To: linaro@patches.linaro.org Delivered-To: linaro@patches.linaro.org Received: from mail-wi0-f200.google.com (mail-wi0-f200.google.com [209.85.212.200]) by patches.linaro.org (Postfix) with ESMTPS id 3A04423028 for ; Wed, 2 Sep 2015 14:35:31 +0000 (UTC) Received: by wicuu12 with SMTP id uu12sf6806365wic.2 for ; Wed, 02 Sep 2015 07:35:30 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:delivered-to:date:from:to:cc:subject:message-id :reply-to:references:mime-version:content-type:content-disposition :in-reply-to:user-agent:sender:precedence:list-id:x-original-sender :x-original-authentication-results:mailing-list:list-post:list-help :list-archive:list-unsubscribe; bh=S19DK0t3RytMGuDYb7xBMjJtbmib84rT+PfyRBuphzM=; b=XSrOwOi7tJH7+71km94Rl78FikbMGwoJ96uwJqXpOc0I9sAIyLxX1FGSWOvpYWCanl 2GQqUBWKev6DM91OezEMoTMLbzIkh3bhJgOOoJNz+rxno4UBHsJhuPCFbZoKlQOAiQv9 Pe25AClOA6d4KXyeNJYxkOE56MRpiZA4f/QxqWkMc54WHb3UgrIxWofyIOA89ixH+ePA cBfK/k99bAAk07qumtZHGV2symeJpYlF1X0i8LB7xKVEB4ssIaV8+5y/maZ7zgc8Q0jQ UdZRBpxJ7DIBoCMxCILwpUmNL5Am8SRDp9Jd3s5KooBbng8UjeMLqbBcrbIN2CvUr5mH bELQ== X-Gm-Message-State: ALoCoQmS1a/YB8ysjep7xJWWEmdhXFO5u+NurIPzi5Z/0L9QK0VDSXejJz+81dGK7tIn+Vl0rom9 X-Received: by 10.152.27.10 with SMTP id p10mr4881712lag.1.1441204530171; Wed, 02 Sep 2015 07:35:30 -0700 (PDT) X-BeenThere: patchwork-forward@linaro.org Received: by 10.152.224.129 with SMTP id rc1ls87101lac.6.gmail; Wed, 02 Sep 2015 07:35:30 -0700 (PDT) X-Received: by 10.152.2.200 with SMTP id 8mr15965587law.115.1441204530037; Wed, 02 Sep 2015 07:35:30 -0700 (PDT) Received: from mail-la0-f43.google.com (mail-la0-f43.google.com. [209.85.215.43]) by mx.google.com with ESMTPS id q6si10419934lag.166.2015.09.02.07.35.30 for (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 02 Sep 2015 07:35:30 -0700 (PDT) Received-SPF: pass (google.com: domain of patch+caf_=patchwork-forward=linaro.org@linaro.org designates 209.85.215.43 as permitted sender) client-ip=209.85.215.43; Received: by lagj9 with SMTP id j9so8618839lag.2 for ; Wed, 02 Sep 2015 07:35:29 -0700 (PDT) X-Received: by 10.112.146.135 with SMTP id tc7mr16334106lbb.35.1441204529907; Wed, 02 Sep 2015 07:35:29 -0700 (PDT) X-Forwarded-To: patchwork-forward@linaro.org X-Forwarded-For: patch@linaro.org patchwork-forward@linaro.org Delivered-To: patch@linaro.org Received: by 10.112.164.42 with SMTP id yn10csp621401lbb; Wed, 2 Sep 2015 07:35:28 -0700 (PDT) X-Received: by 10.68.104.98 with SMTP id gd2mr4254583pbb.130.1441204527769; Wed, 02 Sep 2015 07:35:27 -0700 (PDT) Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id x10si30385816pas.107.2015.09.02.07.35.26; Wed, 02 Sep 2015 07:35:27 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-omap-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755136AbbIBOfG (ORCPT + 5 others); Wed, 2 Sep 2015 10:35:06 -0400 Received: from comal.ext.ti.com ([198.47.26.152]:35241 "EHLO comal.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755074AbbIBOfD (ORCPT ); Wed, 2 Sep 2015 10:35:03 -0400 Received: from dlelxv90.itg.ti.com ([172.17.2.17]) by comal.ext.ti.com (8.13.7/8.13.7) with ESMTP id t82EYw9B005778; Wed, 2 Sep 2015 09:34:58 -0500 Received: from DLEE70.ent.ti.com (dlee70.ent.ti.com [157.170.170.113]) by dlelxv90.itg.ti.com (8.14.3/8.13.8) with ESMTP id t82EYwW4030772; Wed, 2 Sep 2015 09:34:58 -0500 Received: from dflp33.itg.ti.com (10.64.6.16) by DLEE70.ent.ti.com (157.170.170.113) with Microsoft SMTP Server id 14.3.224.2; Wed, 2 Sep 2015 09:34:58 -0500 Received: from localhost (ileax41-snat.itg.ti.com [10.172.224.153]) by dflp33.itg.ti.com (8.14.3/8.13.8) with ESMTP id t82EYv40030046; Wed, 2 Sep 2015 09:34:58 -0500 Date: Wed, 2 Sep 2015 09:34:57 -0500 From: Felipe Balbi To: Roger Quadros CC: , , , , , , , , Subject: Re: [PATCH v4 4/9] usb: dwc3: core: Adapt to named interrupts Message-ID: <20150902143457.GF8299@saruman.tx.rr.com> Reply-To: References: <1441203864-15786-1-git-send-email-rogerq@ti.com> <1441203864-15786-5-git-send-email-rogerq@ti.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <1441203864-15786-5-git-send-email-rogerq@ti.com> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-omap-owner@vger.kernel.org Precedence: list List-ID: X-Mailing-List: linux-omap@vger.kernel.org X-Removed-Original-Auth: Dkim didn't pass. X-Original-Sender: balbi@ti.com X-Original-Authentication-Results: mx.google.com; spf=pass (google.com: domain of patch+caf_=patchwork-forward=linaro.org@linaro.org designates 209.85.215.43 as permitted sender) smtp.mailfrom=patch+caf_=patchwork-forward=linaro.org@linaro.org Mailing-list: list patchwork-forward@linaro.org; contact patchwork-forward+owners@linaro.org X-Google-Group-Id: 836684582541 List-Post: , List-Help: , List-Archive: List-Unsubscribe: , On Wed, Sep 02, 2015 at 05:24:19PM +0300, Roger Quadros wrote: > From: Felipe Balbi > > Add support to use interrupt names, > > Following are the interrupt names > > Peripheral Interrupt - peripheral > HOST Interrupt - host > OTG Interrupt - otg > > [Roger Q] > - If any of these are missing we use the > first available IRQ resource so that we don't > break with older DTBs. this is what original commit did: commit ecd5f71cfd663bcd4efd2f29824acd8b2ba9715d Author: Felipe Balbi Date: Fri Jan 3 13:49:38 2014 -0600 usb: dwc3: cleanup IRQ resources In order to make it easier for the driver to figure out which modes of operation it has, and because some dwc3 integrations have rather fuzzy IRQ routing, we now require three different IRQ numbers (peripheral, host, otg). In order to do that and maintain backwards compatibility, we still maintain support for the old IRQ resource name, but if you *really* want to have proper peripheral/host/otg support, you should make sure your resources are correct. Signed-off-by: Felipe Balbi > > - Use gadget_irq in gadget driver. > > Signed-off-by: Felipe Balbi > Signed-off-by: Roger Quadros > --- > drivers/usb/dwc3/core.c | 12 ++++++++++++ > drivers/usb/dwc3/core.h | 7 +++++++ > drivers/usb/dwc3/gadget.c | 2 +- > 3 files changed, 20 insertions(+), 1 deletion(-) > > diff --git a/drivers/usb/dwc3/core.c b/drivers/usb/dwc3/core.c > index 2e36a9b..38b31df 100644 > --- a/drivers/usb/dwc3/core.c > +++ b/drivers/usb/dwc3/core.c > @@ -958,6 +958,18 @@ static int dwc3_probe(struct platform_device *pdev) > dwc->xhci_resources[1].flags = res->flags; > dwc->xhci_resources[1].name = res->name; > > + dwc->otg_irq = platform_get_irq_byname(pdev, "otg"); > + if (dwc->otg_irq < 0) > + dwc->otg_irq = res->start; > + > + dwc->gadget_irq = platform_get_irq_byname(pdev, "peripheral"); > + if (dwc->gadget_irq < 0) > + dwc->gadget_irq = res->start; > + > + dwc->xhci_irq = platform_get_irq_byname(pdev, "host"); > + if (dwc->xhci_irq < 0) > + dwc->xhci_irq = res->start; > + > res = platform_get_resource(pdev, IORESOURCE_MEM, 0); > if (!res) { > dev_err(dev, "missing memory resource\n"); > diff --git a/drivers/usb/dwc3/core.h b/drivers/usb/dwc3/core.h > index 4b85330..bd32cb2 100644 > --- a/drivers/usb/dwc3/core.h > +++ b/drivers/usb/dwc3/core.h > @@ -742,6 +742,9 @@ struct dwc3_scratchpad_array { > * @maximum_speed: maximum speed requested (mainly for testing purposes) > * @revision: revision register contents > * @dr_mode: requested mode of operation > + * @xhci_irq: IRQ number for XHCI IRQs > + * @gadget_irq: IRQ number for Peripheral IRQs > + * @otg_irq: IRQ number for OTG IRQs > * @usb2_phy: pointer to USB2 PHY > * @usb3_phy: pointer to USB3 PHY > * @usb2_generic_phy: pointer to USB2 PHY > @@ -847,6 +850,10 @@ struct dwc3 { > > enum usb_dr_mode dr_mode; > > + int gadget_irq; > + int xhci_irq; > + int otg_irq; > + > /* used for suspend/resume */ > u32 dcfg; > u32 gctl; > diff --git a/drivers/usb/dwc3/gadget.c b/drivers/usb/dwc3/gadget.c > index 0c25704..1148ec8 100644 > --- a/drivers/usb/dwc3/gadget.c > +++ b/drivers/usb/dwc3/gadget.c > @@ -1533,7 +1533,7 @@ static int dwc3_gadget_start(struct usb_gadget *g, > int irq; > u32 reg; > > - irq = platform_get_irq(to_platform_device(dwc->dev), 0); > + irq = dwc->gadget_irq; > ret = request_threaded_irq(irq, dwc3_interrupt, dwc3_thread_interrupt, > IRQF_SHARED, "dwc3", dwc); > if (ret) { > -- > 2.1.4 > diff --git a/drivers/usb/dwc3/core.c b/drivers/usb/dwc3/core.c index 60580a01cdd2..1f01031873b7 100644 --- a/drivers/usb/dwc3/core.c +++ b/drivers/usb/dwc3/core.c @@ -556,10 +556,22 @@ static int dwc3_core_get_phy(struct dwc3 *dwc) static int dwc3_core_init_mode(struct dwc3 *dwc) { struct device *dev = dwc->dev; + struct platform_device *pdev = to_platform_device(dev); int ret; switch (dwc->dr_mode) { case USB_DR_MODE_PERIPHERAL: + dwc->gadget_irq = platform_get_irq_byname(pdev, "dwc3_peripheral"); + if (dwc->gadget_irq < 0) { + dwc->gadget_irq = platform_get_irq_byname(pdev, "dwc_usb3"); + if (dwc->gadget_irq < 0) { + dev_err(dev, "missing IRQ\n"); + return dwc->gadget_irq; + } else { + dev_warn(dev, "dwc_usb3 resource is deprecated\n"); + } + } + dwc3_set_mode(dwc, DWC3_GCTL_PRTCAP_DEVICE); ret = dwc3_gadget_init(dwc); if (ret) { @@ -568,6 +580,22 @@ static int dwc3_core_init_mode(struct dwc3 *dwc) } break; case USB_DR_MODE_HOST: + dwc->xhci_irq = platform_get_irq_byname(pdev, "dwc3_host"); + if (dwc->xhci_irq < 0) { + dwc->xhci_irq = platform_get_irq_byname(pdev, "dwc_usb3"); + if (dwc->xhci_irq < 0) { + dev_err(dev, "missing Host IRQ\n"); + return dwc->xhci_irq; + } else { + dev_warn(dev, "dwc_usb3 resource is deprecated\n"); + } + + dwc->xhci_resources[1].start = dwc->xhci_irq; + dwc->xhci_resources[1].end = dwc->xhci_irq; + dwc->xhci_resources[1].flags = IORESOURCE_IRQ; + dwc->xhci_resources[1].name = "xhci"; + } + dwc3_set_mode(dwc, DWC3_GCTL_PRTCAP_HOST); ret = dwc3_host_init(dwc); if (ret) { @@ -576,6 +604,28 @@ static int dwc3_core_init_mode(struct dwc3 *dwc) } break; case USB_DR_MODE_OTG: + dwc->gadget_irq = platform_get_irq_byname(pdev, "dwc3_peripheral"); + if (dwc->gadget_irq < 0) { + dwc->gadget_irq = platform_get_irq_byname(pdev, "dwc_usb3"); + if (dwc->gadget_irq < 0) { + dev_err(dev, "missing IRQ\n"); + return dwc->gadget_irq; + } else { + dev_warn(dev, "dwc_usb3 resource is deprecated\n"); + } + + dwc->xhci_irq = dwc->gadget_irq; + dwc->otg_irq = dwc->gadget_irq; + } else { + dwc->xhci_irq = platform_get_irq_byname(pdev, "dwc3_host"); + if (dwc->xhci_irq < 0) { + dev_err(dev, "missing Host IRQ\n"); + return dwc->xhci_irq; + } + + dwc->otg_irq = platform_get_irq_byname(pdev, "dwc3_otg"); + } + dwc3_set_mode(dwc, DWC3_GCTL_PRTCAP_OTG); ret = dwc3_otg_init(dwc); if (ret) { @@ -647,18 +697,6 @@ static int dwc3_probe(struct platform_device *pdev) dwc->mem = mem; dwc->dev = dev; - res = platform_get_resource(pdev, IORESOURCE_IRQ, 0); - if (!res) { - dev_err(dev, "missing IRQ\n"); - return -ENODEV; - } - dwc->xhci_resources[1].start = res->start; - dwc->xhci_resources[1].end = res->end; - dwc->xhci_resources[1].flags = res->flags; - dwc->xhci_resources[1].name = res->name; - - dwc->otg_irq = platform_get_irq_byname(pdev, "dwc3_otg"); - res = platform_get_resource(pdev, IORESOURCE_MEM, 0); if (!res) { dev_err(dev, "missing memory resource\n"); diff --git a/drivers/usb/dwc3/core.h b/drivers/usb/dwc3/core.h index 8e2e579b4b1e..0b02186fad6f 100644 --- a/drivers/usb/dwc3/core.h +++ b/drivers/usb/dwc3/core.h @@ -671,6 +671,8 @@ struct dwc3_scratchpad_array { * @maximum_speed: maximum speed requested (mainly for testing purposes) * @revision: revision register contents * @dr_mode: requested mode of operation + * @gadget_irq: IRQ number for Peripheral IRQs + * @xhci_irq: IRQ number for Host IRQs * @otg_irq: IRQ number for OTG IRQs * @usb2_phy: pointer to USB2 PHY * @usb3_phy: pointer to USB3 PHY @@ -747,6 +749,8 @@ struct dwc3 { enum usb_dr_mode dr_mode; + int gadget_irq; + int xhci_irq; int otg_irq; /* used for suspend/resume */ diff --git a/drivers/usb/dwc3/gadget.c b/drivers/usb/dwc3/gadget.c index a8cf87b3de01..2a6e5155fc89 100644 --- a/drivers/usb/dwc3/gadget.c +++ b/drivers/usb/dwc3/gadget.c @@ -1567,15 +1567,13 @@ static int dwc3_gadget_start(struct usb_gadget *g, struct dwc3_ep *dep; unsigned long flags; int ret = 0; - int irq; u32 reg; - irq = platform_get_irq(to_platform_device(dwc->dev), 0); - ret = request_threaded_irq(irq, dwc3_interrupt, dwc3_thread_interrupt, - IRQF_SHARED, "dwc3", dwc); + ret = request_threaded_irq(dwc->gadget_irq, dwc3_interrupt, + dwc3_thread_interrupt, IRQF_SHARED, "dwc3", dwc); if (ret) { dev_err(dwc->dev, "failed to request irq #%d --> %d\n", - irq, ret); + dwc->gadget_irq, ret); goto err0; } @@ -1668,7 +1666,7 @@ err2: err1: spin_unlock_irqrestore(&dwc->lock, flags); - free_irq(irq, dwc); + free_irq(dwc->gadget_irq, dwc); err0: return ret; @@ -1679,7 +1677,6 @@ static int dwc3_gadget_stop(struct usb_gadget *g, { struct dwc3 *dwc = gadget_to_dwc(g); unsigned long flags; - int irq; spin_lock_irqsave(&dwc->lock, flags); @@ -1691,8 +1688,7 @@ static int dwc3_gadget_stop(struct usb_gadget *g, spin_unlock_irqrestore(&dwc->lock, flags); - irq = platform_get_irq(to_platform_device(dwc->dev), 0); - free_irq(irq, dwc); + free_irq(dwc->gadget_irq, dwc); return 0; }