ASoC: WM8903: Get default irq_active_low from IRQ controller

If the WM8903 is hooked up to an interrupt, set the irq_active_low flag
in the default platform data based on the IRQ's IRQ_TYPE. Map IRQ_TYPE_NONE
(a lack of explicit configuration/restriction) to irq_active_low = false;
the previous default.

This code is mainly added to support device tree interrupt bindings,
although will work perfectly well in a non device tree system too.

Any interrupt controller that supports only a single IRQ_TYPE could
set each IRQ's type based on that restriction. This applies equally
with and without device tree. To cater for interrupt controllers
that don't do this, for which irqd_get_trigger_type() will return
IRQ_TYPE_NONE, the platform data irq_active_low field may be used
in systems that don't use device tree.

With device tree, every IRQ must have some IRQ_TYPE set.

Controllers that support DT and multiple IRQ_TYPEs must define the
interrupts property (as used in interrupt source nodes) such that it
defines the IRQ_TYPE to use. When the core DT setup code initializes
wm8903->irq, the interrupts property will be parsed, and as a side-
effect, set the IRQ's IRQ_TYPE for the WM8903 probe() function to read.

Controllers that support DT and a single IRQ_TYPE could arrange to
set the IRQ_TYPE somehow during their initialization, or hard-code
it during the processing of the child interrupts property.

Signed-off-by: Stephen Warren <swarren@nvidia.com>
Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>

diff --git a/sound/soc/codecs/wm8903.c b/sound/soc/codecs/wm8903.c
index b114468d6453bb0f06ccdbac7b4c38c227402ff0..b4f2c906b2f335914227b905537ec18c1c1f3663 100644 (file)
--- a/sound/soc/codecs/wm8903.c
+++ b/sound/soc/codecs/wm8903.c
@@ -25,6 +25,7 @@
 #include <linux/i2c.h>
 #include <linux/regmap.h>
 #include <linux/slab.h>
+#include <linux/irq.h>
 #include <sound/core.h>
 #include <sound/jack.h>
 #include <sound/pcm.h>
@@ -2036,6 +2037,39 @@ static const struct regmap_config wm8903_regmap = {
        .num_reg_defaults = ARRAY_SIZE(wm8903_reg_defaults),
 };
 
+static int wm8903_set_pdata_irq_trigger(struct i2c_client *i2c,
+                                       struct wm8903_platform_data *pdata)
+{
+       struct irq_data *irq_data = irq_get_irq_data(i2c->irq);
+       if (!irq_data) {
+               dev_err(&i2c->dev, "Invalid IRQ: %d\n",
+                       i2c->irq);
+               return -EINVAL;
+       }
+
+       switch (irqd_get_trigger_type(irq_data)) {
+       case IRQ_TYPE_NONE:
+               /*
+               * We assume the controller imposes no restrictions,
+               * so we are able to select active-high
+               */
+               /* Fall-through */
+       case IRQ_TYPE_LEVEL_HIGH:
+               pdata->irq_active_low = false;
+               break;
+       case IRQ_TYPE_LEVEL_LOW:
+               pdata->irq_active_low = true;
+               break;
+       default:
+               dev_err(&i2c->dev,
+                       "Unsupported IRQ_TYPE %x\n",
+                       irqd_get_trigger_type(irq_data));
+               return -EINVAL;
+       }
+
+       return 0;
+}
+
 static __devinit int wm8903_i2c_probe(struct i2c_client *i2c,
                                      const struct i2c_device_id *id)
 {
@@ -2071,6 +2105,12 @@ static __devinit int wm8903_i2c_probe(struct i2c_client *i2c,
                        dev_err(&i2c->dev, "Failed to allocate pdata\n");
                        return -ENOMEM;
                }
+
+               if (i2c->irq) {
+                       ret = wm8903_set_pdata_irq_trigger(i2c, wm8903->pdata);
+                       if (ret != 0)
+                               return ret;
+               }
        }
 
        ret = regmap_read(wm8903->regmap, WM8903_SW_RESET_AND_ID, &val);