mfd: max77686: Convert to use regmap_irq
authorJavier Martinez Canillas <javier.martinez@collabora.co.uk>
Fri, 4 Jul 2014 20:24:04 +0000 (22:24 +0200)
committerLee Jones <lee.jones@linaro.org>
Mon, 21 Jul 2014 15:54:26 +0000 (16:54 +0100)
By using the generic IRQ support in the Register map API, it
is possible to get rid max77686-irq.c and simplify the code.

Suggested-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
Signed-off-by: Javier Martinez Canillas <javier.martinez@collabora.co.uk>
Reviewed-by: Doug Anderson <dianders@chromium.org>
Tested-by: Doug Anderson <dianders@chromium.org>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
drivers/mfd/Kconfig
drivers/mfd/Makefile
drivers/mfd/max77686-irq.c [deleted file]
drivers/mfd/max77686.c
drivers/rtc/rtc-max77686.c
include/linux/mfd/max77686-private.h
include/linux/mfd/max77686.h

index b8d9ca0b68e22668b07b6e28a8623db80e209da4..30102042dcafb1564aacfeb37d56d0766018031c 100644 (file)
@@ -384,6 +384,7 @@ config MFD_MAX77686
        depends on I2C=y
        select MFD_CORE
        select REGMAP_I2C
+       select REGMAP_IRQ
        select IRQ_DOMAIN
        help
          Say yes here to add support for Maxim Semiconductor MAX77686.
index 4e2bc255b8b091fe2c280a9a82d85a78656789cc..f00148782d9b3c234609ad10827430ec84c2a56b 100644 (file)
@@ -115,7 +115,7 @@ da9063-objs                 := da9063-core.o da9063-irq.o da9063-i2c.o
 obj-$(CONFIG_MFD_DA9063)       += da9063.o
 
 obj-$(CONFIG_MFD_MAX14577)     += max14577.o
-obj-$(CONFIG_MFD_MAX77686)     += max77686.o max77686-irq.o
+obj-$(CONFIG_MFD_MAX77686)     += max77686.o
 obj-$(CONFIG_MFD_MAX77693)     += max77693.o
 obj-$(CONFIG_MFD_MAX8907)      += max8907.o
 max8925-objs                   := max8925-core.o max8925-i2c.o
diff --git a/drivers/mfd/max77686-irq.c b/drivers/mfd/max77686-irq.c
deleted file mode 100644 (file)
index cdc3280..0000000
+++ /dev/null
@@ -1,319 +0,0 @@
-/*
- * max77686-irq.c - Interrupt controller support for MAX77686
- *
- * Copyright (C) 2012 Samsung Electronics Co.Ltd
- * Chiwoong Byun <woong.byun@samsung.com>
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
- *
- * This driver is based on max8997-irq.c
- */
-
-#include <linux/err.h>
-#include <linux/irq.h>
-#include <linux/interrupt.h>
-#include <linux/gpio.h>
-#include <linux/mfd/max77686.h>
-#include <linux/mfd/max77686-private.h>
-#include <linux/irqdomain.h>
-#include <linux/regmap.h>
-
-enum {
-       MAX77686_DEBUG_IRQ_INFO = 1 << 0,
-       MAX77686_DEBUG_IRQ_MASK = 1 << 1,
-       MAX77686_DEBUG_IRQ_INT = 1 << 2,
-};
-
-static int debug_mask = 0;
-module_param(debug_mask, int, 0);
-MODULE_PARM_DESC(debug_mask, "Set debug_mask : 0x0=off 0x1=IRQ_INFO  0x2=IRQ_MASK 0x4=IRQ_INI)");
-
-static const u8 max77686_mask_reg[] = {
-       [PMIC_INT1] = MAX77686_REG_INT1MSK,
-       [PMIC_INT2] = MAX77686_REG_INT2MSK,
-       [RTC_INT] = MAX77686_RTC_INTM,
-};
-
-static struct regmap *max77686_get_regmap(struct max77686_dev *max77686,
-                               enum max77686_irq_source src)
-{
-       switch (src) {
-       case PMIC_INT1 ... PMIC_INT2:
-               return max77686->regmap;
-       case RTC_INT:
-               return max77686->rtc_regmap;
-       default:
-               return ERR_PTR(-EINVAL);
-       }
-}
-
-struct max77686_irq_data {
-       int mask;
-       enum max77686_irq_source group;
-};
-
-#define DECLARE_IRQ(idx, _group, _mask)                \
-       [(idx)] = { .group = (_group), .mask = (_mask) }
-static const struct max77686_irq_data max77686_irqs[] = {
-       DECLARE_IRQ(MAX77686_PMICIRQ_PWRONF,    PMIC_INT1, 1 << 0),
-       DECLARE_IRQ(MAX77686_PMICIRQ_PWRONR,    PMIC_INT1, 1 << 1),
-       DECLARE_IRQ(MAX77686_PMICIRQ_JIGONBF,   PMIC_INT1, 1 << 2),
-       DECLARE_IRQ(MAX77686_PMICIRQ_JIGONBR,   PMIC_INT1, 1 << 3),
-       DECLARE_IRQ(MAX77686_PMICIRQ_ACOKBF,    PMIC_INT1, 1 << 4),
-       DECLARE_IRQ(MAX77686_PMICIRQ_ACOKBR,    PMIC_INT1, 1 << 5),
-       DECLARE_IRQ(MAX77686_PMICIRQ_ONKEY1S,   PMIC_INT1, 1 << 6),
-       DECLARE_IRQ(MAX77686_PMICIRQ_MRSTB,             PMIC_INT1, 1 << 7),
-       DECLARE_IRQ(MAX77686_PMICIRQ_140C,              PMIC_INT2, 1 << 0),
-       DECLARE_IRQ(MAX77686_PMICIRQ_120C,              PMIC_INT2, 1 << 1),
-       DECLARE_IRQ(MAX77686_RTCIRQ_RTC60S,             RTC_INT, 1 << 0),
-       DECLARE_IRQ(MAX77686_RTCIRQ_RTCA1,              RTC_INT, 1 << 1),
-       DECLARE_IRQ(MAX77686_RTCIRQ_RTCA2,              RTC_INT, 1 << 2),
-       DECLARE_IRQ(MAX77686_RTCIRQ_SMPL,               RTC_INT, 1 << 3),
-       DECLARE_IRQ(MAX77686_RTCIRQ_RTC1S,              RTC_INT, 1 << 4),
-       DECLARE_IRQ(MAX77686_RTCIRQ_WTSR,               RTC_INT, 1 << 5),
-};
-
-static void max77686_irq_lock(struct irq_data *data)
-{
-       struct max77686_dev *max77686 = irq_get_chip_data(data->irq);
-
-       if (debug_mask & MAX77686_DEBUG_IRQ_MASK)
-               pr_info("%s\n", __func__);
-
-       mutex_lock(&max77686->irqlock);
-}
-
-static void max77686_irq_sync_unlock(struct irq_data *data)
-{
-       struct max77686_dev *max77686 = irq_get_chip_data(data->irq);
-       int i;
-
-       for (i = 0; i < MAX77686_IRQ_GROUP_NR; i++) {
-               u8 mask_reg = max77686_mask_reg[i];
-               struct regmap *map = max77686_get_regmap(max77686, i);
-
-               if (debug_mask & MAX77686_DEBUG_IRQ_MASK)
-                       pr_debug("%s: mask_reg[%d]=0x%x, cur=0x%x\n",
-                       __func__, i, mask_reg, max77686->irq_masks_cur[i]);
-
-               if (mask_reg == MAX77686_REG_INVALID ||
-                               IS_ERR_OR_NULL(map))
-                       continue;
-
-               max77686->irq_masks_cache[i] = max77686->irq_masks_cur[i];
-
-               regmap_write(map, max77686_mask_reg[i],
-                               max77686->irq_masks_cur[i]);
-       }
-
-       mutex_unlock(&max77686->irqlock);
-}
-
-static const inline struct max77686_irq_data *to_max77686_irq(int irq)
-{
-       struct irq_data *data = irq_get_irq_data(irq);
-       return &max77686_irqs[data->hwirq];
-}
-
-static void max77686_irq_mask(struct irq_data *data)
-{
-       struct max77686_dev *max77686 = irq_get_chip_data(data->irq);
-       const struct max77686_irq_data *irq_data = to_max77686_irq(data->irq);
-
-       max77686->irq_masks_cur[irq_data->group] |= irq_data->mask;
-
-       if (debug_mask & MAX77686_DEBUG_IRQ_MASK)
-               pr_info("%s: group=%d, cur=0x%x\n",
-                       __func__, irq_data->group,
-                       max77686->irq_masks_cur[irq_data->group]);
-}
-
-static void max77686_irq_unmask(struct irq_data *data)
-{
-       struct max77686_dev *max77686 = irq_get_chip_data(data->irq);
-       const struct max77686_irq_data *irq_data = to_max77686_irq(data->irq);
-
-       max77686->irq_masks_cur[irq_data->group] &= ~irq_data->mask;
-
-       if (debug_mask & MAX77686_DEBUG_IRQ_MASK)
-               pr_info("%s: group=%d, cur=0x%x\n",
-                       __func__, irq_data->group,
-                       max77686->irq_masks_cur[irq_data->group]);
-}
-
-static struct irq_chip max77686_irq_chip = {
-       .name                   = "max77686",
-       .irq_bus_lock           = max77686_irq_lock,
-       .irq_bus_sync_unlock    = max77686_irq_sync_unlock,
-       .irq_mask               = max77686_irq_mask,
-       .irq_unmask             = max77686_irq_unmask,
-};
-
-static irqreturn_t max77686_irq_thread(int irq, void *data)
-{
-       struct max77686_dev *max77686 = data;
-       unsigned int irq_reg[MAX77686_IRQ_GROUP_NR] = {};
-       unsigned int irq_src;
-       int ret;
-       int i, cur_irq;
-
-       ret = regmap_read(max77686->regmap,  MAX77686_REG_INTSRC, &irq_src);
-       if (ret < 0) {
-               dev_err(max77686->dev, "Failed to read interrupt source: %d\n",
-                               ret);
-               return IRQ_NONE;
-       }
-
-       if (debug_mask & MAX77686_DEBUG_IRQ_INT)
-               pr_info("%s: irq_src=0x%x\n", __func__, irq_src);
-
-       if (irq_src == MAX77686_IRQSRC_PMIC) {
-               ret = regmap_bulk_read(max77686->regmap,
-                                        MAX77686_REG_INT1, irq_reg, 2);
-               if (ret < 0) {
-                       dev_err(max77686->dev, "Failed to read interrupt source: %d\n",
-                                       ret);
-                       return IRQ_NONE;
-               }
-
-               if (debug_mask & MAX77686_DEBUG_IRQ_INT)
-                       pr_info("%s: int1=0x%x, int2=0x%x\n", __func__,
-                                irq_reg[PMIC_INT1], irq_reg[PMIC_INT2]);
-       }
-
-       if (irq_src & MAX77686_IRQSRC_RTC) {
-               ret = regmap_read(max77686->rtc_regmap,
-                                       MAX77686_RTC_INT, &irq_reg[RTC_INT]);
-               if (ret < 0) {
-                       dev_err(max77686->dev, "Failed to read interrupt source: %d\n",
-                                       ret);
-                       return IRQ_NONE;
-               }
-
-               if (debug_mask & MAX77686_DEBUG_IRQ_INT)
-                       pr_info("%s: rtc int=0x%x\n", __func__,
-                                                        irq_reg[RTC_INT]);
-
-       }
-
-       for (i = 0; i < MAX77686_IRQ_GROUP_NR; i++)
-               irq_reg[i] &= ~max77686->irq_masks_cur[i];
-
-       for (i = 0; i < MAX77686_IRQ_NR; i++) {
-               if (irq_reg[max77686_irqs[i].group] & max77686_irqs[i].mask) {
-                       cur_irq = irq_find_mapping(max77686->irq_domain, i);
-                       if (cur_irq)
-                               handle_nested_irq(cur_irq);
-               }
-       }
-
-       return IRQ_HANDLED;
-}
-
-static int max77686_irq_domain_map(struct irq_domain *d, unsigned int irq,
-                                       irq_hw_number_t hw)
-{
-       struct max77686_dev *max77686 = d->host_data;
-
-       irq_set_chip_data(irq, max77686);
-       irq_set_chip_and_handler(irq, &max77686_irq_chip, handle_edge_irq);
-       irq_set_nested_thread(irq, 1);
-#ifdef CONFIG_ARM
-       set_irq_flags(irq, IRQF_VALID);
-#else
-       irq_set_noprobe(irq);
-#endif
-       return 0;
-}
-
-static struct irq_domain_ops max77686_irq_domain_ops = {
-       .map = max77686_irq_domain_map,
-};
-
-int max77686_irq_init(struct max77686_dev *max77686)
-{
-       struct irq_domain *domain;
-       int i;
-       int ret;
-       int val;
-       struct regmap *map;
-
-       mutex_init(&max77686->irqlock);
-
-       if (max77686->irq_gpio && !max77686->irq) {
-               max77686->irq = gpio_to_irq(max77686->irq_gpio);
-
-               if (debug_mask & MAX77686_DEBUG_IRQ_INT) {
-                       ret = gpio_request(max77686->irq_gpio, "pmic_irq");
-                       if (ret < 0) {
-                               dev_err(max77686->dev,
-                                       "Failed to request gpio %d with ret:"
-                                       "%d\n", max77686->irq_gpio, ret);
-                               return IRQ_NONE;
-                       }
-
-                       gpio_direction_input(max77686->irq_gpio);
-                       val = gpio_get_value(max77686->irq_gpio);
-                       gpio_free(max77686->irq_gpio);
-                       pr_info("%s: gpio_irq=%x\n", __func__, val);
-               }
-       }
-
-       if (!max77686->irq) {
-               dev_err(max77686->dev, "irq is not specified\n");
-               return -ENODEV;
-       }
-
-       /* Mask individual interrupt sources */
-       for (i = 0; i < MAX77686_IRQ_GROUP_NR; i++) {
-               max77686->irq_masks_cur[i] = 0xff;
-               max77686->irq_masks_cache[i] = 0xff;
-               map = max77686_get_regmap(max77686, i);
-
-               if (IS_ERR_OR_NULL(map))
-                       continue;
-               if (max77686_mask_reg[i] == MAX77686_REG_INVALID)
-                       continue;
-
-               regmap_write(map, max77686_mask_reg[i], 0xff);
-       }
-       domain = irq_domain_add_linear(NULL, MAX77686_IRQ_NR,
-                                       &max77686_irq_domain_ops, max77686);
-       if (!domain) {
-               dev_err(max77686->dev, "could not create irq domain\n");
-               return -ENODEV;
-       }
-       max77686->irq_domain = domain;
-
-       ret = request_threaded_irq(max77686->irq, NULL, max77686_irq_thread,
-                                  IRQF_TRIGGER_FALLING | IRQF_ONESHOT,
-                                  "max77686-irq", max77686);
-
-       if (ret)
-               dev_err(max77686->dev, "Failed to request IRQ %d: %d\n",
-                       max77686->irq, ret);
-
-
-       if (debug_mask & MAX77686_DEBUG_IRQ_INFO)
-               pr_info("%s-\n", __func__);
-
-       return 0;
-}
-
-void max77686_irq_exit(struct max77686_dev *max77686)
-{
-       if (max77686->irq)
-               free_irq(max77686->irq, max77686);
-}
index ce869acf27aebea28dcee07465c3677727b72f43..3cb41d02cd3da733caf195c1485b0c2ec46453c1 100644 (file)
@@ -25,6 +25,8 @@
 #include <linux/export.h>
 #include <linux/slab.h>
 #include <linux/i2c.h>
+#include <linux/irq.h>
+#include <linux/interrupt.h>
 #include <linux/pm_runtime.h>
 #include <linux/module.h>
 #include <linux/mfd/core.h>
@@ -46,6 +48,54 @@ static struct regmap_config max77686_regmap_config = {
        .val_bits = 8,
 };
 
+static struct regmap_config max77686_rtc_regmap_config = {
+       .reg_bits = 8,
+       .val_bits = 8,
+};
+
+static const struct regmap_irq max77686_irqs[] = {
+       /* INT1 interrupts */
+       { .reg_offset = 0, .mask = MAX77686_INT1_PWRONF_MSK, },
+       { .reg_offset = 0, .mask = MAX77686_INT1_PWRONR_MSK, },
+       { .reg_offset = 0, .mask = MAX77686_INT1_JIGONBF_MSK, },
+       { .reg_offset = 0, .mask = MAX77686_INT1_JIGONBR_MSK, },
+       { .reg_offset = 0, .mask = MAX77686_INT1_ACOKBF_MSK, },
+       { .reg_offset = 0, .mask = MAX77686_INT1_ACOKBR_MSK, },
+       { .reg_offset = 0, .mask = MAX77686_INT1_ONKEY1S_MSK, },
+       { .reg_offset = 0, .mask = MAX77686_INT1_MRSTB_MSK, },
+       /* INT2 interrupts */
+       { .reg_offset = 1, .mask = MAX77686_INT2_140C_MSK, },
+       { .reg_offset = 1, .mask = MAX77686_INT2_120C_MSK, },
+};
+
+static const struct regmap_irq_chip max77686_irq_chip = {
+       .name                   = "max77686-pmic",
+       .status_base            = MAX77686_REG_INT1,
+       .mask_base              = MAX77686_REG_INT1MSK,
+       .num_regs               = 2,
+       .irqs                   = max77686_irqs,
+       .num_irqs               = ARRAY_SIZE(max77686_irqs),
+};
+
+static const struct regmap_irq max77686_rtc_irqs[] = {
+       /* RTC interrupts */
+       { .reg_offset = 0, .mask = MAX77686_RTCINT_RTC60S_MSK, },
+       { .reg_offset = 0, .mask = MAX77686_RTCINT_RTCA1_MSK, },
+       { .reg_offset = 0, .mask = MAX77686_RTCINT_RTCA2_MSK, },
+       { .reg_offset = 0, .mask = MAX77686_RTCINT_SMPL_MSK, },
+       { .reg_offset = 0, .mask = MAX77686_RTCINT_RTC1S_MSK, },
+       { .reg_offset = 0, .mask = MAX77686_RTCINT_WTSR_MSK, },
+};
+
+static const struct regmap_irq_chip max77686_rtc_irq_chip = {
+       .name                   = "max77686-rtc",
+       .status_base            = MAX77686_RTC_INT,
+       .mask_base              = MAX77686_RTC_INTM,
+       .num_regs               = 1,
+       .irqs                   = max77686_rtc_irqs,
+       .num_irqs               = ARRAY_SIZE(max77686_rtc_irqs),
+};
+
 #ifdef CONFIG_OF
 static const struct of_device_id max77686_pmic_dt_match[] = {
        {.compatible = "maxim,max77686", .data = NULL},
@@ -101,7 +151,6 @@ static int max77686_i2c_probe(struct i2c_client *i2c,
        max77686->type = id->driver_data;
 
        max77686->wakeup = pdata->wakeup;
-       max77686->irq_gpio = pdata->irq_gpio;
        max77686->irq = i2c->irq;
 
        max77686->regmap = devm_regmap_init_i2c(i2c, &max77686_regmap_config);
@@ -117,8 +166,7 @@ static int max77686_i2c_probe(struct i2c_client *i2c,
                dev_err(max77686->dev,
                        "device not found on this channel (this is not an error)\n");
                return -ENODEV;
-       } else
-               dev_info(max77686->dev, "device found\n");
+       }
 
        max77686->rtc = i2c_new_dummy(i2c->adapter, I2C_ADDR_RTC);
        if (!max77686->rtc) {
@@ -127,15 +175,48 @@ static int max77686_i2c_probe(struct i2c_client *i2c,
        }
        i2c_set_clientdata(max77686->rtc, max77686);
 
-       max77686_irq_init(max77686);
+       max77686->rtc_regmap = devm_regmap_init_i2c(max77686->rtc,
+                                                   &max77686_rtc_regmap_config);
+       if (IS_ERR(max77686->rtc_regmap)) {
+               ret = PTR_ERR(max77686->rtc_regmap);
+               dev_err(max77686->dev, "failed to allocate RTC regmap: %d\n",
+                       ret);
+               goto err_unregister_i2c;
+       }
+
+       ret = regmap_add_irq_chip(max77686->regmap, max77686->irq,
+                                 IRQF_TRIGGER_FALLING | IRQF_ONESHOT |
+                                 IRQF_SHARED, 0, &max77686_irq_chip,
+                                 &max77686->irq_data);
+       if (ret != 0) {
+               dev_err(&i2c->dev, "failed to add PMIC irq chip: %d\n", ret);
+               goto err_unregister_i2c;
+       }
+       ret = regmap_add_irq_chip(max77686->rtc_regmap, max77686->irq,
+                                 IRQF_TRIGGER_FALLING | IRQF_ONESHOT |
+                                 IRQF_SHARED, 0, &max77686_rtc_irq_chip,
+                                 &max77686->rtc_irq_data);
+       if (ret != 0) {
+               dev_err(&i2c->dev, "failed to add RTC irq chip: %d\n", ret);
+               goto err_del_irqc;
+       }
 
        ret = mfd_add_devices(max77686->dev, -1, max77686_devs,
                              ARRAY_SIZE(max77686_devs), NULL, 0, NULL);
        if (ret < 0) {
-               mfd_remove_devices(max77686->dev);
-               i2c_unregister_device(max77686->rtc);
+               dev_err(&i2c->dev, "failed to add MFD devices: %d\n", ret);
+               goto err_del_rtc_irqc;
        }
 
+       return 0;
+
+err_del_rtc_irqc:
+       regmap_del_irq_chip(max77686->irq, max77686->rtc_irq_data);
+err_del_irqc:
+       regmap_del_irq_chip(max77686->irq, max77686->irq_data);
+err_unregister_i2c:
+       i2c_unregister_device(max77686->rtc);
+
        return ret;
 }
 
@@ -144,6 +225,10 @@ static int max77686_i2c_remove(struct i2c_client *i2c)
        struct max77686_dev *max77686 = i2c_get_clientdata(i2c);
 
        mfd_remove_devices(max77686->dev);
+
+       regmap_del_irq_chip(max77686->irq, max77686->rtc_irq_data);
+       regmap_del_irq_chip(max77686->irq, max77686->irq_data);
+
        i2c_unregister_device(max77686->rtc);
 
        return 0;
index 9efe118a28bae7bf42a7c6564640fe8967ecf411..d20a7f0786ebd9540625caa57c7609412569354f 100644 (file)
@@ -492,16 +492,11 @@ static int max77686_rtc_init_reg(struct max77686_rtc_info *info)
        return ret;
 }
 
-static struct regmap_config max77686_rtc_regmap_config = {
-       .reg_bits = 8,
-       .val_bits = 8,
-};
-
 static int max77686_rtc_probe(struct platform_device *pdev)
 {
        struct max77686_dev *max77686 = dev_get_drvdata(pdev->dev.parent);
        struct max77686_rtc_info *info;
-       int ret, virq;
+       int ret;
 
        dev_info(&pdev->dev, "%s\n", __func__);
 
@@ -514,14 +509,7 @@ static int max77686_rtc_probe(struct platform_device *pdev)
        info->dev = &pdev->dev;
        info->max77686 = max77686;
        info->rtc = max77686->rtc;
-       info->max77686->rtc_regmap = devm_regmap_init_i2c(info->max77686->rtc,
-                                        &max77686_rtc_regmap_config);
-       if (IS_ERR(info->max77686->rtc_regmap)) {
-               ret = PTR_ERR(info->max77686->rtc_regmap);
-               dev_err(info->max77686->dev, "Failed to allocate register map: %d\n",
-                               ret);
-               return ret;
-       }
+
        platform_set_drvdata(pdev, info);
 
        ret = max77686_rtc_init_reg(info);
@@ -550,15 +538,16 @@ static int max77686_rtc_probe(struct platform_device *pdev)
                        ret = -EINVAL;
                goto err_rtc;
        }
-       virq = irq_create_mapping(max77686->irq_domain, MAX77686_RTCIRQ_RTCA1);
-       if (!virq) {
+
+       info->virq = regmap_irq_get_virq(max77686->rtc_irq_data,
+                                        MAX77686_RTCIRQ_RTCA1);
+       if (!info->virq) {
                ret = -ENXIO;
                goto err_rtc;
        }
-       info->virq = virq;
 
-       ret = devm_request_threaded_irq(&pdev->dev, virq, NULL,
-                               max77686_rtc_alarm_irq, 0, "rtc-alarm0", info);
+       ret = devm_request_threaded_irq(&pdev->dev, info->virq, NULL,
+                               max77686_rtc_alarm_irq, 0, "rtc-alarm1", info);
        if (ret < 0)
                dev_err(&pdev->dev, "Failed to request alarm IRQ: %d: %d\n",
                        info->virq, ret);
index 8c75a9c8dfab30085d9190c3283a6b56d7fc30b8..8e177806cba1e1d85ca400952837e351b31639d1 100644 (file)
@@ -181,9 +181,6 @@ enum max77686_rtc_reg {
        MAX77686_ALARM2_DATE            = 0x1B,
 };
 
-#define MAX77686_IRQSRC_PMIC   (0)
-#define MAX77686_IRQSRC_RTC            (1 << 0)
-
 enum max77686_irq_source {
        PMIC_INT1 = 0,
        PMIC_INT2,
@@ -205,16 +202,33 @@ enum max77686_irq {
        MAX77686_PMICIRQ_140C,
        MAX77686_PMICIRQ_120C,
 
-       MAX77686_RTCIRQ_RTC60S,
+       MAX77686_RTCIRQ_RTC60S = 0,
        MAX77686_RTCIRQ_RTCA1,
        MAX77686_RTCIRQ_RTCA2,
        MAX77686_RTCIRQ_SMPL,
        MAX77686_RTCIRQ_RTC1S,
        MAX77686_RTCIRQ_WTSR,
-
-       MAX77686_IRQ_NR,
 };
 
+#define MAX77686_INT1_PWRONF_MSK       BIT(0)
+#define MAX77686_INT1_PWRONR_MSK       BIT(1)
+#define MAX77686_INT1_JIGONBF_MSK      BIT(2)
+#define MAX77686_INT1_JIGONBR_MSK      BIT(3)
+#define MAX77686_INT1_ACOKBF_MSK       BIT(4)
+#define MAX77686_INT1_ACOKBR_MSK       BIT(5)
+#define MAX77686_INT1_ONKEY1S_MSK      BIT(6)
+#define MAX77686_INT1_MRSTB_MSK                BIT(7)
+
+#define MAX77686_INT2_140C_MSK         BIT(0)
+#define MAX77686_INT2_120C_MSK         BIT(1)
+
+#define MAX77686_RTCINT_RTC60S_MSK     BIT(0)
+#define MAX77686_RTCINT_RTCA1_MSK      BIT(1)
+#define MAX77686_RTCINT_RTCA2_MSK      BIT(2)
+#define MAX77686_RTCINT_SMPL_MSK       BIT(3)
+#define MAX77686_RTCINT_RTC1S_MSK      BIT(4)
+#define MAX77686_RTCINT_WTSR_MSK       BIT(5)
+
 struct max77686_dev {
        struct device *dev;
        struct i2c_client *i2c; /* 0xcc / PMIC, Battery Control, and FLASH */
@@ -224,11 +238,10 @@ struct max77686_dev {
 
        struct regmap *regmap;          /* regmap for mfd */
        struct regmap *rtc_regmap;      /* regmap for rtc */
-
-       struct irq_domain *irq_domain;
+       struct regmap_irq_chip_data *irq_data;
+       struct regmap_irq_chip_data *rtc_irq_data;
 
        int irq;
-       int irq_gpio;
        bool wakeup;
        struct mutex irqlock;
        int irq_masks_cur[MAX77686_IRQ_GROUP_NR];
index 46c0f320ed764ab33b4dfe9d18ad796c1bfa722f..4cbcc13e8a2a051c479b07d6482ae9a95483209b 100644 (file)
@@ -89,8 +89,6 @@ struct max77686_opmode_data {
 };
 
 struct max77686_platform_data {
-       /* IRQ */
-       int irq_gpio;
        int ono;
        int wakeup;