[PATCH 1/4] arm: msm: gpio support

On Tue, 2010-03-30 at 18:14 -0700, Arve Hj?nnev?g wrote:

On Tue, Mar 30, 2010 at 4:58 PM, H Hartley Sweeten
[off-list ref] wrote:

quoted

On Tuesday, March 30, 2010 4:12 PM, Daniel Walker wrote:

quoted

From: Daniel Walker <redacted>

...

quoted

quoted

+
+static void msm_gpio_irq_ack(unsigned int irq)
+{
+     unsigned long irq_flags;
+     struct msm_gpio_chip *msm_chip = get_irq_chip_data(irq);
+     unsigned b;
+
+     spin_lock_irqsave(&msm_chip->lock, irq_flags);
+
+     b = 1U << (irq - FIRST_GPIO_IRQ - msm_chip->chip.base);

Urk... That's a bit confusing...

You might want to make this a macro or an inline function with some kind
of comment.

On a related note, why did you inline msm_gpio_clear_detect_status? It
is used from two other functions.

You mean other places in the Android tree?

Daniel