[PATCH 1/6] gpio/omap: remove saved_fallingdetect, saved_risingdetect fields

2012-02-23 Thread Tarun Kanti DebBarma
Since we already have context.fallingdetect and context.risingdetect
there is no more need to have these additional fields. Also, getting
rid of extra reads associated with them.

Signed-off-by: Tarun Kanti DebBarma tarun.ka...@ti.com
---
 drivers/gpio/gpio-omap.c |   19 ---
 1 files changed, 8 insertions(+), 11 deletions(-)

diff --git a/drivers/gpio/gpio-omap.c b/drivers/gpio/gpio-omap.c
index f29252f..40a1fb2 100644
--- a/drivers/gpio/gpio-omap.c
+++ b/drivers/gpio/gpio-omap.c
@@ -59,8 +59,6 @@ struct gpio_bank {
u32 enabled_non_wakeup_gpios;
struct gpio_regs context;
u32 saved_datain;
-   u32 saved_fallingdetect;
-   u32 saved_risingdetect;
u32 level_mask;
u32 toggle_mask;
spinlock_t lock;
@@ -1224,11 +1222,9 @@ static int omap_gpio_runtime_suspend(struct device *dev)
 
bank-saved_datain = __raw_readl(bank-base +
bank-regs-datain);
-   l1 = __raw_readl(bank-base + bank-regs-fallingdetect);
-   l2 = __raw_readl(bank-base + bank-regs-risingdetect);
+   l1 = bank-context.fallingdetect;
+   l2 = bank-context.risingdetect;
 
-   bank-saved_fallingdetect = l1;
-   bank-saved_risingdetect = l2;
l1 = ~bank-enabled_non_wakeup_gpios;
l2 = ~bank-enabled_non_wakeup_gpios;
 
@@ -1287,9 +1283,9 @@ static int omap_gpio_runtime_resume(struct device *dev)
}
}
 
-   __raw_writel(bank-saved_fallingdetect,
+   __raw_writel(bank-context.fallingdetect,
bank-base + bank-regs-fallingdetect);
-   __raw_writel(bank-saved_risingdetect,
+   __raw_writel(bank-context.risingdetect,
bank-base + bank-regs-risingdetect);
l = __raw_readl(bank-base + bank-regs-datain);
 
@@ -1306,14 +1302,15 @@ static int omap_gpio_runtime_resume(struct device *dev)
 * No need to generate IRQs for the rising edge for gpio IRQs
 * configured with falling edge only; and vice versa.
 */
-   gen0 = l  bank-saved_fallingdetect;
+   gen0 = l  bank-context.fallingdetect;
gen0 = bank-saved_datain;
 
-   gen1 = l  bank-saved_risingdetect;
+   gen1 = l  bank-context.risingdetect;
gen1 = ~(bank-saved_datain);
 
/* FIXME: Consider GPIO IRQs with level detections properly! */
-   gen = l  (~(bank-saved_fallingdetect)  ~(bank-saved_risingdetect));
+   gen = l  (~(bank-context.fallingdetect) 
+~(bank-context.risingdetect));
/* Consider all GPIO IRQs needed to be updated */
gen |= gen0 | gen1;
 
-- 
1.7.0.4

--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 1/6] gpio/omap: remove saved_fallingdetect, saved_risingdetect fields

2012-02-23 Thread Felipe Balbi
On Thu, Feb 23, 2012 at 05:40:26PM +0530, Tarun Kanti DebBarma wrote:
 Since we already have context.fallingdetect and context.risingdetect
 there is no more need to have these additional fields. Also, getting
 rid of extra reads associated with them.
 
 Signed-off-by: Tarun Kanti DebBarma tarun.ka...@ti.com

Looks trivial and correct:

Acked-by: Felipe Balbi ba...@ti.com

-- 
balbi


signature.asc
Description: Digital signature