From patchwork Fri Jul 1 09:47:13 2011 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Tarun Kanti DebBarma X-Patchwork-Id: 935572 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by demeter1.kernel.org (8.14.4/8.14.4) with ESMTP id p619lvoA019210 for ; Fri, 1 Jul 2011 09:47:57 GMT Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755438Ab1GAJr4 (ORCPT ); Fri, 1 Jul 2011 05:47:56 -0400 Received: from arroyo.ext.ti.com ([192.94.94.40]:51350 "EHLO arroyo.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755995Ab1GAJr4 (ORCPT ); Fri, 1 Jul 2011 05:47:56 -0400 Received: from dbdp20.itg.ti.com ([172.24.170.38]) by arroyo.ext.ti.com (8.13.7/8.13.7) with ESMTP id p619lqgn000411 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Fri, 1 Jul 2011 04:47:54 -0500 Received: from dbde71.ent.ti.com (localhost [127.0.0.1]) by dbdp20.itg.ti.com (8.13.8/8.13.8) with ESMTP id p619lphE004774; Fri, 1 Jul 2011 15:17:51 +0530 (IST) Received: from dbdp31.itg.ti.com (172.24.170.98) by DBDE71.ent.ti.com (172.24.170.149) with Microsoft SMTP Server id 8.3.106.1; Fri, 1 Jul 2011 15:17:51 +0530 Received: from localhost.localdomain ([172.24.190.145]) by dbdp31.itg.ti.com (8.13.8/8.13.8) with ESMTP id p619l7mk023254; Fri, 1 Jul 2011 15:17:48 +0530 (IST) From: Tarun Kanti DebBarma To: CC: , , , Tarun Kanti DebBarma Subject: [PATCH v3 19/20] GPIO: OMAP: optimize suspend and resume functions Date: Fri, 1 Jul 2011 15:17:13 +0530 Message-ID: <1309513634-20971-20-git-send-email-tarun.kanti@ti.com> X-Mailer: git-send-email 1.6.0.4 In-Reply-To: <1309513634-20971-1-git-send-email-tarun.kanti@ti.com> References: <1309513634-20971-1-git-send-email-tarun.kanti@ti.com> MIME-Version: 1.0 Sender: linux-omap-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-omap@vger.kernel.org X-Greylist: IP, sender and recipient auto-whitelisted, not delayed by milter-greylist-4.2.6 (demeter1.kernel.org [140.211.167.41]); Fri, 01 Jul 2011 09:47:57 +0000 (UTC) There is no need to operate on all the banks every time the function is called. Just operate on the current bank passed by the framework. Signed-off-by: Tarun Kanti DebBarma --- drivers/gpio/gpio-omap.c | 56 ++++++++++++++++++++++----------------------- 1 files changed, 27 insertions(+), 29 deletions(-) diff --git a/drivers/gpio/gpio-omap.c b/drivers/gpio/gpio-omap.c index 66eff1d..e9d2e1e 100644 --- a/drivers/gpio/gpio-omap.c +++ b/drivers/gpio/gpio-omap.c @@ -1082,6 +1082,8 @@ static int __devinit omap_gpio_probe(struct platform_device *pdev) goto err_free; } + platform_set_drvdata(pdev, bank); + pm_runtime_enable(bank->dev); pm_runtime_irq_safe(bank->dev); if (IS_ERR_VALUE(pm_runtime_get_sync(bank->dev) < 0)) { @@ -1114,26 +1116,24 @@ err_exit: static int omap_gpio_suspend(struct device *dev) { - struct gpio_bank *bank; - - pm_runtime_get_sync(dev); + struct platform_device *pdev = to_platform_device(dev); + struct gpio_bank *bank = platform_get_drvdata(pdev); + void __iomem *base = bank->base; + void __iomem *wake_status; + unsigned long flags; - list_for_each_entry(bank, &omap_gpio_list, node) { - void __iomem *base = bank->base; - void __iomem *wake_status; - unsigned long flags; + if (!bank->regs->wkup_status) + return 0; - if (!bank->regs->wkup_status) - return 0; + wake_status = bank->base + bank->regs->wkup_status; - wake_status = bank->base + bank->regs->wkup_status; + pm_runtime_get_sync(dev); - spin_lock_irqsave(&bank->lock, flags); - bank->saved_wakeup = __raw_readl(wake_status); - MOD_REG_BIT(bank->regs->wkup_status, 0xffffffff, 0); - MOD_REG_BIT(bank->regs->wkup_status, bank->suspend_wakeup, 1); - spin_unlock_irqrestore(&bank->lock, flags); - } + spin_lock_irqsave(&bank->lock, flags); + bank->saved_wakeup = __raw_readl(wake_status); + MOD_REG_BIT(bank->regs->wkup_status, 0xffffffff, 0); + MOD_REG_BIT(bank->regs->wkup_status, bank->suspend_wakeup, 1); + spin_unlock_irqrestore(&bank->lock, flags); pm_runtime_put_sync(dev); @@ -1142,22 +1142,20 @@ static int omap_gpio_suspend(struct device *dev) static int omap_gpio_resume(struct device *dev) { - struct gpio_bank *bank; + struct platform_device *pdev = to_platform_device(dev); + struct gpio_bank *bank = platform_get_drvdata(pdev); + void __iomem *base = bank->base; + unsigned long flags; - pm_runtime_get_sync(dev); + if (!bank->regs->wkup_status) + return 0; - list_for_each_entry(bank, &omap_gpio_list, node) { - void __iomem *base = bank->base; - unsigned long flags; - - if (!bank->regs->wkup_status) - return 0; + pm_runtime_get_sync(dev); - spin_lock_irqsave(&bank->lock, flags); - MOD_REG_BIT(bank->regs->wkup_status, 0xffffffff, 0); - MOD_REG_BIT(bank->regs->wkup_status, bank->saved_wakeup, 1); - spin_unlock_irqrestore(&bank->lock, flags); - } + spin_lock_irqsave(&bank->lock, flags); + MOD_REG_BIT(bank->regs->wkup_status, 0xffffffff, 0); + MOD_REG_BIT(bank->regs->wkup_status, bank->saved_wakeup, 1); + spin_unlock_irqrestore(&bank->lock, flags); pm_runtime_put_sync(dev);