]> www.pilppa.org Git - linux-2.6-omap-h63xx.git/blobdiff - drivers/mfd/twl4030-core.c
create twl4030-irq.c
[linux-2.6-omap-h63xx.git] / drivers / mfd / twl4030-core.c
index f7e717a5eff97001ab57914509fc2260d0c10278..ab0a6bda4591c03814691456d8c156d41fd828e5 100644 (file)
  * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307 USA
  */
 
-#include <linux/kernel_stat.h>
 #include <linux/init.h>
 #include <linux/mutex.h>
-#include <linux/interrupt.h>
-#include <linux/irq.h>
-#include <linux/random.h>
-#include <linux/kthread.h>
 #include <linux/platform_device.h>
 #include <linux/clk.h>
+#include <linux/err.h>
 
 #include <linux/i2c.h>
 #include <linux/i2c/twl4030.h>
-#include <linux/i2c/twl4030-madc.h>
+
+
+/*
+ * The TWL4030 "Triton 2" is one of a family of a multi-function "Power
+ * Management and System Companion Device" chips originally designed for
+ * use in OMAP2 and OMAP 3 based systems.  Its control interfaces use I2C,
+ * often at around 3 Mbit/sec, including for interrupt handling.
+ *
+ * This driver core provides genirq support for the interrupts emitted,
+ * by the various modules, and exports register access primitives.
+ *
+ * FIXME this driver currently requires use of the first interrupt line
+ * (and associated registers).
+ */
 
 #define DRIVER_NAME                    "twl4030"
 
 #if defined(CONFIG_TWL4030_BCI_BATTERY) || \
-       defined(CONFIG_TWL4030_BCI_BATTERY_MODUEL)
+       defined(CONFIG_TWL4030_BCI_BATTERY_MODULE)
 #define twl_has_bci()          true
 #else
 #define twl_has_bci()          false
 #define twl_has_usb()  false
 #endif
 
-static inline void activate_irq(int irq)
-{
-#ifdef CONFIG_ARM
-       /* ARM requires an extra step to clear IRQ_NOREQUEST, which it
-        * sets on behalf of every irq_chip.  Also sets IRQ_NOPROBE.
-        */
-       set_irq_flags(irq, IRQF_VALID);
-#else
-       /* same effect on other architectures */
-       set_irq_noprobe(irq);
-#endif
-}
-
-/* Primary Interrupt Handler on TWL4030 Registers */
-
-/* Register Definitions */
-
-#define REG_PIH_ISR_P1                 (0x1)
-#define REG_PIH_ISR_P2                 (0x2)
-#define REG_PIH_SIR                    (0x3)
 
 /* Triton Core internal information (BEGIN) */
 
@@ -108,23 +97,20 @@ static inline void activate_irq(int irq)
 
 #define TWL4030_NUM_SLAVES             4
 
-/* Slave address */
-#define TWL4030_SLAVENUM_NUM0          0x00
-#define TWL4030_SLAVENUM_NUM1          0x01
-#define TWL4030_SLAVENUM_NUM2          0x02
-#define TWL4030_SLAVENUM_NUM3          0x03
 
-/* Base Address defns */
-/* USB ID */
+/* Base Address defns for twl4030_map[] */
+
+/* subchip/slave 0 - USB ID */
 #define TWL4030_BASEADD_USB            0x0000
-/* AUD ID */
+
+/* subchip/slave 1 - AUD ID */
 #define TWL4030_BASEADD_AUDIO_VOICE    0x0000
 #define TWL4030_BASEADD_GPIO           0x0098
-
 #define TWL4030_BASEADD_INTBR          0x0085
 #define TWL4030_BASEADD_PIH            0x0080
 #define TWL4030_BASEADD_TEST           0x004C
-/* AUX ID */
+
+/* subchip/slave 2 - AUX ID */
 #define TWL4030_BASEADD_INTERRUPTS     0x00B9
 #define TWL4030_BASEADD_LED            0x00EE
 #define TWL4030_BASEADD_MADC           0x0000
@@ -135,7 +121,8 @@ static inline void activate_irq(int irq)
 #define TWL4030_BASEADD_PWMA           0x00EF
 #define TWL4030_BASEADD_PWMB           0x00F1
 #define TWL4030_BASEADD_KEYPAD         0x00D2
-/* POWER ID */
+
+/* subchip/slave 3 - POWER ID */
 #define TWL4030_BASEADD_BACKUP         0x0014
 #define TWL4030_BASEADD_INT            0x002E
 #define TWL4030_BASEADD_PM_MASTER      0x0036
@@ -143,185 +130,34 @@ static inline void activate_irq(int irq)
 #define TWL4030_BASEADD_RTC            0x001C
 #define TWL4030_BASEADD_SECURED_REG    0x0000
 
-/* TWL4030 BCI registers */
-#define TWL4030_INTERRUPTS_BCIIMR1A    0x2
-#define TWL4030_INTERRUPTS_BCIIMR2A    0x3
-#define TWL4030_INTERRUPTS_BCIIMR1B    0x6
-#define TWL4030_INTERRUPTS_BCIIMR2B    0x7
-#define TWL4030_INTERRUPTS_BCIISR1A    0x0
-#define TWL4030_INTERRUPTS_BCIISR2A    0x1
-#define TWL4030_INTERRUPTS_BCIISR1B    0x4
-#define TWL4030_INTERRUPTS_BCIISR2B    0x5
-
-/* TWL4030 keypad registers */
-#define TWL4030_KEYPAD_KEYP_IMR1       0x12
-#define TWL4030_KEYPAD_KEYP_IMR2       0x14
-#define TWL4030_KEYPAD_KEYP_ISR1       0x11
-#define TWL4030_KEYPAD_KEYP_ISR2       0x13
-
-
 /* Triton Core internal information (END) */
 
+
 /* Few power values */
 #define R_CFG_BOOT                     0x05
 #define R_PROTECT_KEY                  0x0E
 
-/* access control */
+/* access control values for R_PROTECT_KEY */
 #define KEY_UNLOCK1                    0xce
 #define KEY_UNLOCK2                    0xec
 #define KEY_LOCK                       0x00
 
+/* some fields in R_CFG_BOOT */
 #define HFCLK_FREQ_19p2_MHZ            (1 << 0)
 #define HFCLK_FREQ_26_MHZ              (2 << 0)
 #define HFCLK_FREQ_38p4_MHZ            (3 << 0)
 #define HIGH_PERF_SQ                   (1 << 3)
 
-/* SIH_CTRL registers that aren't defined elsewhere */
-#define TWL4030_INTERRUPTS_BCISIHCTRL  0x0d
-#define TWL4030_MADC_MADC_SIH_CTRL     0x67
-#define TWL4030_KEYPAD_KEYP_SIH_CTRL   0x17
-
-
-/**
- * struct twl4030_mod_iregs - TWL module IMR/ISR regs to mask/clear at init
- * @mod_no: TWL4030 module number (e.g., TWL4030_MODULE_GPIO)
- * @sih_ctrl: address of module SIH_CTRL register
- * @reg_cnt: number of IMR/ISR regs
- * @imrs: pointer to array of TWL module interrupt mask register indices
- * @isrs: pointer to array of TWL module interrupt status register indices
- *
- * Ties together TWL4030 modules and lists of IMR/ISR registers to mask/clear
- * during twl_init_irq().
- */
-struct twl4030_mod_iregs {
-       const u8 mod_no;
-       const u8 sih_ctrl;
-       const u8 reg_cnt;
-       const u8 *imrs;
-       const u8 *isrs;
-};
-
-/* TWL4030 INT module interrupt mask registers */
-static const u8 __initconst twl4030_int_imr_regs[] = {
-       TWL4030_INT_PWR_IMR1,
-       TWL4030_INT_PWR_IMR2,
-};
-
-/* TWL4030 INT module interrupt status registers */
-static const u8 __initconst twl4030_int_isr_regs[] = {
-       TWL4030_INT_PWR_ISR1,
-       TWL4030_INT_PWR_ISR2,
-};
-
-/* TWL4030 INTERRUPTS module interrupt mask registers */
-static const u8 __initconst twl4030_interrupts_imr_regs[] = {
-       TWL4030_INTERRUPTS_BCIIMR1A,
-       TWL4030_INTERRUPTS_BCIIMR1B,
-       TWL4030_INTERRUPTS_BCIIMR2A,
-       TWL4030_INTERRUPTS_BCIIMR2B,
-};
-
-/* TWL4030 INTERRUPTS module interrupt status registers */
-static const u8 __initconst twl4030_interrupts_isr_regs[] = {
-       TWL4030_INTERRUPTS_BCIISR1A,
-       TWL4030_INTERRUPTS_BCIISR1B,
-       TWL4030_INTERRUPTS_BCIISR2A,
-       TWL4030_INTERRUPTS_BCIISR2B,
-};
-
-/* TWL4030 MADC module interrupt mask registers */
-static const u8 __initconst twl4030_madc_imr_regs[] = {
-       TWL4030_MADC_IMR1,
-       TWL4030_MADC_IMR2,
-};
-
-/* TWL4030 MADC module interrupt status registers */
-static const u8 __initconst twl4030_madc_isr_regs[] = {
-       TWL4030_MADC_ISR1,
-       TWL4030_MADC_ISR2,
-};
-
-/* TWL4030 keypad module interrupt mask registers */
-static const u8 __initconst twl4030_keypad_imr_regs[] = {
-       TWL4030_KEYPAD_KEYP_IMR1,
-       TWL4030_KEYPAD_KEYP_IMR2,
-};
-
-/* TWL4030 keypad module interrupt status registers */
-static const u8 __initconst twl4030_keypad_isr_regs[] = {
-       TWL4030_KEYPAD_KEYP_ISR1,
-       TWL4030_KEYPAD_KEYP_ISR2,
-};
-
-/* TWL4030 GPIO module interrupt mask registers */
-static const u8 __initconst twl4030_gpio_imr_regs[] = {
-       REG_GPIO_IMR1A,
-       REG_GPIO_IMR1B,
-       REG_GPIO_IMR2A,
-       REG_GPIO_IMR2B,
-       REG_GPIO_IMR3A,
-       REG_GPIO_IMR3B,
-};
-
-/* TWL4030 GPIO module interrupt status registers */
-static const u8 __initconst twl4030_gpio_isr_regs[] = {
-       REG_GPIO_ISR1A,
-       REG_GPIO_ISR1B,
-       REG_GPIO_ISR2A,
-       REG_GPIO_ISR2B,
-       REG_GPIO_ISR3A,
-       REG_GPIO_ISR3B,
-};
-
-/* TWL4030 modules that have IMR/ISR registers that must be masked/cleared */
-static const struct twl4030_mod_iregs __initconst twl4030_mod_regs[] = {
-       {
-               .mod_no   = TWL4030_MODULE_INT,
-               .sih_ctrl = TWL4030_INT_PWR_SIH_CTRL,
-               .reg_cnt  = ARRAY_SIZE(twl4030_int_imr_regs),
-               .imrs     = twl4030_int_imr_regs,
-               .isrs     = twl4030_int_isr_regs,
-       },
-       {
-               .mod_no   = TWL4030_MODULE_INTERRUPTS,
-               .sih_ctrl = TWL4030_INTERRUPTS_BCISIHCTRL,
-               .reg_cnt  = ARRAY_SIZE(twl4030_interrupts_imr_regs),
-               .imrs     = twl4030_interrupts_imr_regs,
-               .isrs     = twl4030_interrupts_isr_regs,
-       },
-       {
-               .mod_no   = TWL4030_MODULE_MADC,
-               .sih_ctrl = TWL4030_MADC_MADC_SIH_CTRL,
-               .reg_cnt  = ARRAY_SIZE(twl4030_madc_imr_regs),
-               .imrs     = twl4030_madc_imr_regs,
-               .isrs     = twl4030_madc_isr_regs,
-       },
-       {
-               .mod_no   = TWL4030_MODULE_KEYPAD,
-               .sih_ctrl = TWL4030_KEYPAD_KEYP_SIH_CTRL,
-               .reg_cnt  = ARRAY_SIZE(twl4030_keypad_imr_regs),
-               .imrs     = twl4030_keypad_imr_regs,
-               .isrs     = twl4030_keypad_isr_regs,
-       },
-       {
-               .mod_no   = TWL4030_MODULE_GPIO,
-               .sih_ctrl = REG_GPIO_SIH_CTRL,
-               .reg_cnt  = ARRAY_SIZE(twl4030_gpio_imr_regs),
-               .imrs     = twl4030_gpio_imr_regs,
-               .isrs     = twl4030_gpio_isr_regs,
-       },
-};
 
+/*----------------------------------------------------------------------*/
 
-/* Data Structures */
-/* To have info on T2 IRQ substem activated or not */
-static struct completion irq_event;
+/* is driver active, bound to a chip? */
+static bool inuse;
 
-/* Structure to define on TWL4030 Slave ID */
+/* Structure for each TWL4030 Slave */
 struct twl4030_client {
        struct i2c_client *client;
        u8 address;
-       bool inuse;
 
        /* max numb of i2c_msg required is for read =2 */
        struct i2c_msg xfer_msg[2];
@@ -330,13 +166,15 @@ struct twl4030_client {
        struct mutex xfer_lock;
 };
 
-/* Module Mapping */
+static struct twl4030_client twl4030_modules[TWL4030_NUM_SLAVES];
+
+
+/* mapping the module id to slave id and base address */
 struct twl4030mapping {
        unsigned char sid;      /* Slave ID */
        unsigned char base;     /* base address */
 };
 
-/* mapping the module id to slave id and base address */
 static struct twl4030mapping twl4030_map[TWL4030_MODULE_LAST + 1] = {
        /*
         * NOTE:  don't change this table without updating the
@@ -344,61 +182,36 @@ static struct twl4030mapping twl4030_map[TWL4030_MODULE_LAST + 1] = {
         * so they continue to match the order in this table.
         */
 
-       { TWL4030_SLAVENUM_NUM0, TWL4030_BASEADD_USB },
-
-       { TWL4030_SLAVENUM_NUM1, TWL4030_BASEADD_AUDIO_VOICE },
-       { TWL4030_SLAVENUM_NUM1, TWL4030_BASEADD_GPIO },
-       { TWL4030_SLAVENUM_NUM1, TWL4030_BASEADD_INTBR },
-       { TWL4030_SLAVENUM_NUM1, TWL4030_BASEADD_PIH },
-       { TWL4030_SLAVENUM_NUM1, TWL4030_BASEADD_TEST },
-
-       { TWL4030_SLAVENUM_NUM2, TWL4030_BASEADD_KEYPAD },
-       { TWL4030_SLAVENUM_NUM2, TWL4030_BASEADD_MADC },
-       { TWL4030_SLAVENUM_NUM2, TWL4030_BASEADD_INTERRUPTS },
-       { TWL4030_SLAVENUM_NUM2, TWL4030_BASEADD_LED },
-       { TWL4030_SLAVENUM_NUM2, TWL4030_BASEADD_MAIN_CHARGE },
-       { TWL4030_SLAVENUM_NUM2, TWL4030_BASEADD_PRECHARGE },
-       { TWL4030_SLAVENUM_NUM2, TWL4030_BASEADD_PWM0 },
-       { TWL4030_SLAVENUM_NUM2, TWL4030_BASEADD_PWM1 },
-       { TWL4030_SLAVENUM_NUM2, TWL4030_BASEADD_PWMA },
-       { TWL4030_SLAVENUM_NUM2, TWL4030_BASEADD_PWMB },
-
-       { TWL4030_SLAVENUM_NUM3, TWL4030_BASEADD_BACKUP },
-       { TWL4030_SLAVENUM_NUM3, TWL4030_BASEADD_INT },
-       { TWL4030_SLAVENUM_NUM3, TWL4030_BASEADD_PM_MASTER },
-       { TWL4030_SLAVENUM_NUM3, TWL4030_BASEADD_PM_RECEIVER },
-       { TWL4030_SLAVENUM_NUM3, TWL4030_BASEADD_RTC },
-       { TWL4030_SLAVENUM_NUM3, TWL4030_BASEADD_SECURED_REG },
+       { 0, TWL4030_BASEADD_USB },
+
+       { 1, TWL4030_BASEADD_AUDIO_VOICE },
+       { 1, TWL4030_BASEADD_GPIO },
+       { 1, TWL4030_BASEADD_INTBR },
+       { 1, TWL4030_BASEADD_PIH },
+       { 1, TWL4030_BASEADD_TEST },
+
+       { 2, TWL4030_BASEADD_KEYPAD },
+       { 2, TWL4030_BASEADD_MADC },
+       { 2, TWL4030_BASEADD_INTERRUPTS },
+       { 2, TWL4030_BASEADD_LED },
+       { 2, TWL4030_BASEADD_MAIN_CHARGE },
+       { 2, TWL4030_BASEADD_PRECHARGE },
+       { 2, TWL4030_BASEADD_PWM0 },
+       { 2, TWL4030_BASEADD_PWM1 },
+       { 2, TWL4030_BASEADD_PWMA },
+       { 2, TWL4030_BASEADD_PWMB },
+
+       { 3, TWL4030_BASEADD_BACKUP },
+       { 3, TWL4030_BASEADD_INT },
+       { 3, TWL4030_BASEADD_PM_MASTER },
+       { 3, TWL4030_BASEADD_PM_RECEIVER },
+       { 3, TWL4030_BASEADD_RTC },
+       { 3, TWL4030_BASEADD_SECURED_REG },
 };
 
-static struct twl4030_client twl4030_modules[TWL4030_NUM_SLAVES];
-
-/*
- * TWL4030 doesn't have PIH mask, hence dummy function for mask
- * and unmask.
- */
-
-static void twl4030_i2c_ackirq(unsigned int irq)
-{
-}
-
-static void twl4030_i2c_disableint(unsigned int irq)
-{
-}
-
-static void twl4030_i2c_enableint(unsigned int irq)
-{
-}
-
-/* information for processing in the Work Item */
-static struct irq_chip twl4030_irq_chip = {
-       .name   = "twl4030",
-       .ack    = twl4030_i2c_ackirq,
-       .mask   = twl4030_i2c_disableint,
-       .unmask = twl4030_i2c_enableint,
-};
+/*----------------------------------------------------------------------*/
 
-/* Global Functions */
+/* Exported Functions */
 
 /**
  * twl4030_i2c_write - Writes a n bit register in TWL4030
@@ -426,7 +239,7 @@ int twl4030_i2c_write(u8 mod_no, u8 *value, u8 reg, u8 num_bytes)
        sid = twl4030_map[mod_no].sid;
        twl = &twl4030_modules[sid];
 
-       if (unlikely(!twl->inuse)) {
+       if (unlikely(!inuse)) {
                pr_err("%s: client %d is not initialized\n", DRIVER_NAME, sid);
                return -EPERM;
        }
@@ -476,7 +289,7 @@ int twl4030_i2c_read(u8 mod_no, u8 *value, u8 reg, u8 num_bytes)
        sid = twl4030_map[mod_no].sid;
        twl = &twl4030_modules[sid];
 
-       if (unlikely(!twl->inuse)) {
+       if (unlikely(!inuse)) {
                pr_err("%s: client %d is not initialized\n", DRIVER_NAME, sid);
                return -EPERM;
        }
@@ -537,156 +350,7 @@ int twl4030_i2c_read_u8(u8 mod_no, u8 *value, u8 reg)
 }
 EXPORT_SYMBOL(twl4030_i2c_read_u8);
 
-/* Helper Functions */
-
-/*
- * do_twl4030_module_irq() is the desc->handle method for each of the twl4030
- * module interrupts.  It executes in kernel thread context.
- * On entry, cpu interrupts are disabled.
- */
-static void do_twl4030_module_irq(unsigned int irq, irq_desc_t *desc)
-{
-       struct irqaction *action;
-       const unsigned int cpu = smp_processor_id();
-
-       /*
-        * Earlier this was desc->triggered = 1;
-        */
-       desc->status |= IRQ_LEVEL;
-
-       /*
-        * The desc->handle method would normally call the desc->chip->ack
-        * method here, but we won't bother since our ack method is NULL.
-        */
-
-       if (!desc->depth) {
-               kstat_cpu(cpu).irqs[irq]++;
-
-               action = desc->action;
-               if (action) {
-                       int ret;
-                       int status = 0;
-                       int retval = 0;
-
-                       local_irq_enable();
-
-                       do {
-                               /* Call the ISR with cpu interrupts enabled */
-                               ret = action->handler(irq, action->dev_id);
-                               if (ret == IRQ_HANDLED)
-                                       status |= action->flags;
-                               retval |= ret;
-                               action = action->next;
-                       } while (action);
-
-                       if (status & IRQF_SAMPLE_RANDOM)
-                               add_interrupt_randomness(irq);
-
-                       local_irq_disable();
-
-                       if (retval != IRQ_HANDLED)
-                               printk(KERN_ERR "ISR for TWL4030 module"
-                                       " irq %d can't handle interrupt\n",
-                                       irq);
-
-                       /*
-                        * Here is where we should call the unmask method, but
-                        * again we won't bother since it is NULL.
-                        */
-               } else
-                       printk(KERN_CRIT "TWL4030 module irq %d has no ISR"
-                                       " but can't be masked!\n", irq);
-       } else
-               printk(KERN_CRIT "TWL4030 module irq %d is disabled but can't"
-                               " be masked!\n", irq);
-}
-
-static unsigned twl4030_irq_base;
-
-/*
- * twl4030_irq_thread() runs as a kernel thread.  It queries the twl4030
- * interrupt controller to see which modules are generating interrupt requests
- * and then calls the desc->handle method for each module requesting service.
- */
-static int twl4030_irq_thread(void *data)
-{
-       long irq = (long)data;
-       irq_desc_t *desc = irq_desc + irq;
-       static unsigned i2c_errors;
-       const static unsigned max_i2c_errors = 100;
-
-       daemonize("twl4030-irq");
-       current->flags |= PF_NOFREEZE;
-
-       while (!kthread_should_stop()) {
-               int ret;
-               int module_irq;
-               u8 pih_isr;
-
-               wait_for_completion_interruptible(&irq_event);
-
-               ret = twl4030_i2c_read_u8(TWL4030_MODULE_PIH, &pih_isr,
-                                         REG_PIH_ISR_P1);
-               if (ret) {
-                       printk(KERN_WARNING "I2C error %d while reading TWL4030"
-                                       " PIH ISR register.\n", ret);
-                       if (++i2c_errors >= max_i2c_errors) {
-                               printk(KERN_ERR "Maximum I2C error count"
-                                               " exceeded.  Terminating %s.\n",
-                                               __func__);
-                               break;
-                       }
-                       continue;
-               }
-
-               for (module_irq = twl4030_irq_base; 0 != pih_isr;
-                        pih_isr >>= 1, module_irq++) {
-                       if (pih_isr & 0x1) {
-                               irq_desc_t *d = irq_desc + module_irq;
-
-                               local_irq_disable();
-
-                               d->handle_irq(module_irq, d);
-
-                               local_irq_enable();
-                       }
-               }
-
-               desc->chip->unmask(irq);
-       }
-
-       return 0;
-}
-
-/*
- * do_twl4030_irq() is the desc->handle method for the twl4030 interrupt.
- * This is a chained interrupt, so there is no desc->action method for it.
- * Now we need to query the interrupt controller in the twl4030 to determine
- * which module is generating the interrupt request.  However, we can't do i2c
- * transactions in interrupt context, so we must defer that work to a kernel
- * thread.  All we do here is acknowledge and mask the interrupt and wakeup
- * the kernel thread.
- */
-static void do_twl4030_irq(unsigned int irq, irq_desc_t *desc)
-{
-       const unsigned int cpu = smp_processor_id();
-
-       /*
-        * Earlier this was desc->triggered = 1;
-        */
-       desc->status |= IRQ_LEVEL;
-
-       /*
-        * Acknowledge, clear _AND_ disable the interrupt.
-        */
-       desc->chip->ack(irq);
-
-       if (!desc->depth) {
-               kstat_cpu(cpu).irqs[irq]++;
-
-               complete(&irq_event);
-       }
-}
+/*----------------------------------------------------------------------*/
 
 static int add_children(struct twl4030_platform_data *pdata)
 {
@@ -695,7 +359,7 @@ static int add_children(struct twl4030_platform_data *pdata)
        int                     status = 0;
 
        if (twl_has_bci() && pdata->bci) {
-               twl = &twl4030_modules[TWL4030_SLAVENUM_NUM3];
+               twl = &twl4030_modules[3];
 
                pdev = platform_device_alloc("twl4030_bci", -1);
                if (!pdev) {
@@ -738,7 +402,7 @@ static int add_children(struct twl4030_platform_data *pdata)
        }
 
        if (twl_has_gpio() && pdata->gpio) {
-               twl = &twl4030_modules[TWL4030_SLAVENUM_NUM1];
+               twl = &twl4030_modules[1];
 
                pdev = platform_device_alloc("twl4030_gpio", -1);
                if (!pdev) {
@@ -787,7 +451,7 @@ static int add_children(struct twl4030_platform_data *pdata)
        if (twl_has_keypad() && pdata->keypad) {
                pdev = platform_device_alloc("twl4030_keypad", -1);
                if (pdev) {
-                       twl = &twl4030_modules[TWL4030_SLAVENUM_NUM2];
+                       twl = &twl4030_modules[2];
                        pdev->dev.parent = &twl->client->dev;
                        device_init_wakeup(&pdev->dev, 1);
                        status = platform_device_add_data(pdev, pdata->keypad,
@@ -817,7 +481,7 @@ static int add_children(struct twl4030_platform_data *pdata)
        if (twl_has_madc() && pdata->madc) {
                pdev = platform_device_alloc("twl4030_madc", -1);
                if (pdev) {
-                       twl = &twl4030_modules[TWL4030_SLAVENUM_NUM2];
+                       twl = &twl4030_modules[2];
                        pdev->dev.parent = &twl->client->dev;
                        device_init_wakeup(&pdev->dev, 1);
                        status = platform_device_add_data(pdev, pdata->madc,
@@ -845,7 +509,7 @@ static int add_children(struct twl4030_platform_data *pdata)
        }
 
        if (twl_has_rtc()) {
-               twl = &twl4030_modules[TWL4030_SLAVENUM_NUM3];
+               twl = &twl4030_modules[3];
 
                pdev = platform_device_alloc("twl4030_rtc", -1);
                if (!pdev) {
@@ -888,7 +552,7 @@ static int add_children(struct twl4030_platform_data *pdata)
        }
 
        if (twl_has_usb() && pdata->usb) {
-               twl = &twl4030_modules[TWL4030_SLAVENUM_NUM0];
+               twl = &twl4030_modules[0];
 
                pdev = platform_device_alloc("twl4030_usb", -1);
                if (!pdev) {
@@ -937,25 +601,14 @@ err:
        return status;
 }
 
-static struct task_struct * __init start_twl4030_irq_thread(long irq)
-{
-       struct task_struct *thread;
-
-       init_completion(&irq_event);
-       thread = kthread_run(twl4030_irq_thread, (void *)irq,
-                            "twl4030 irq %ld", irq);
-       if (!thread)
-               pr_err("%s: could not create twl4030 irq %ld thread!\n",
-                      DRIVER_NAME, irq);
-
-       return thread;
-}
+/*----------------------------------------------------------------------*/
 
 /*
- * These three functions should be part of Voltage frame work
- * added here to complete the functionality for now.
+ * These three functions initialize the on-chip clock framework,
+ * letting it generate the right frequencies for USB, MADC, and
+ * other purposes.
  */
-static int __init protect_pm_master(void)
+static inline int __init protect_pm_master(void)
 {
        int e = 0;
 
@@ -964,7 +617,7 @@ static int __init protect_pm_master(void)
        return e;
 }
 
-static int __init unprotect_pm_master(void)
+static inline int __init unprotect_pm_master(void)
 {
        int e = 0;
 
@@ -975,7 +628,7 @@ static int __init unprotect_pm_master(void)
        return e;
 }
 
-static int __init power_companion_init(void)
+static void __init clocks_init(void)
 {
        int e = 0;
        struct clk *osc;
@@ -988,12 +641,15 @@ static int __init power_companion_init(void)
        else
                osc = clk_get(NULL, "osc_sys_ck");
 #else
+       /* REVISIT for non-OMAP systems, pass the clock rate from
+        * board init code, using platform_data.
+        */
        osc = ERR_PTR(-EIO);
 #endif
        if (IS_ERR(osc)) {
                printk(KERN_WARNING "Skipping twl4030 internal clock init and "
                                "using bootloader value (unknown osc rate)\n");
-               return 0;
+               return;
        }
 
        rate = clk_get_rate(osc);
@@ -1017,135 +673,23 @@ static int __init power_companion_init(void)
        e |= twl4030_i2c_write_u8(TWL4030_MODULE_PM_MASTER, ctrl, R_CFG_BOOT);
        e |= protect_pm_master();
 
-       return e;
-}
-
-/**
- * twl4030_i2c_clear_isr - clear TWL4030 SIH ISR regs via read + write
- * @mod_no: TWL4030 module number
- * @reg: register index to clear
- * @cor: value of the <module>_SIH_CTRL.COR bit (1 or 0)
- *
- * Either reads (cor == 1) or writes (cor == 0) to a TWL4030 interrupt
- * status register to ensure that any prior interrupts are cleared.
- * Returns the status from the I2C read operation.
- */
-static int __init twl4030_i2c_clear_isr(u8 mod_no, u8 reg, u8 cor)
-{
-       u8 tmp;
-
-       return (cor) ? twl4030_i2c_read_u8(mod_no, &tmp, reg) :
-               twl4030_i2c_write_u8(mod_no, 0xff, reg);
-}
-
-/**
- * twl4030_read_cor_bit - are TWL module ISRs cleared by reads or writes?
- * @mod_no: TWL4030 module number
- * @reg: register index to clear
- *
- * Returns 1 if the TWL4030 SIH interrupt status registers (ISRs) for
- * the specified TWL module are cleared by reads, or 0 if cleared by
- * writes.
- */
-static int twl4030_read_cor_bit(u8 mod_no, u8 reg)
-{
-       u8 tmp = 0;
-
-       WARN_ON(twl4030_i2c_read_u8(mod_no, &tmp, reg) < 0);
-
-       tmp &= TWL4030_SIH_CTRL_COR_MASK;
-       tmp >>= __ffs(TWL4030_SIH_CTRL_COR_MASK);
-
-       return tmp;
-}
-
-/**
- * twl4030_mask_clear_intrs - mask and clear all TWL4030 interrupts
- * @t: pointer to twl4030_mod_iregs array
- * @t_sz: ARRAY_SIZE(t) (starting at 1)
- *
- * Mask all TWL4030 interrupt mask registers (IMRs) and clear all
- * interrupt status registers (ISRs).  No return value, but will WARN if
- * any I2C operations fail.
- */
-static void __init twl4030_mask_clear_intrs(const struct twl4030_mod_iregs *t,
-                                           const u8 t_sz)
-{
-       int i, j;
-
-       /*
-        * N.B. - further efficiency is possible here.  Eight I2C
-        * operations on BCI and GPIO modules are avoidable if I2C
-        * burst read/write transactions were implemented.  Would
-        * probably save about 1ms of boot time and a small amount of
-        * power.
-        */
-       for (i = 0; i < t_sz; i++) {
-               const struct twl4030_mod_iregs tmr = t[i];
-               int cor;
-
-               /* Are ISRs cleared by reads or writes? */
-               cor = twl4030_read_cor_bit(tmr.mod_no, tmr.sih_ctrl);
-
-               for (j = 0; j < tmr.reg_cnt; j++) {
-
-                       /* Mask interrupts at the TWL4030 */
-                       WARN_ON(twl4030_i2c_write_u8(tmr.mod_no, 0xff,
-                                                    tmr.imrs[j]) < 0);
-
-                       /* Clear TWL4030 ISRs */
-                       WARN_ON(twl4030_i2c_clear_isr(tmr.mod_no,
-                                                     tmr.isrs[j], cor) < 0);
-               }
-       }
-
-       return;
-}
-
-
-static void twl_init_irq(int irq_num, unsigned irq_base, unsigned irq_end)
-{
-       int     i;
-       int     res = 0;
-       char    *msg = "Unable to register interrupt subsystem";
-
-       /*
-        * Mask and clear all TWL4030 interrupts since initially we do
-        * not have any TWL4030 module interrupt handlers present
-        */
-       twl4030_mask_clear_intrs(twl4030_mod_regs,
-                                ARRAY_SIZE(twl4030_mod_regs));
-
-       twl4030_irq_base = irq_base;
-
-       /* install an irq handler for each of the PIH modules */
-       for (i = irq_base; i < irq_end; i++) {
-               set_irq_chip_and_handler(i, &twl4030_irq_chip,
-                               do_twl4030_module_irq);
-               activate_irq(i);
-       }
-
-       /* install an irq handler to demultiplex the TWL4030 interrupt */
-       set_irq_data(irq_num, start_twl4030_irq_thread(irq_num));
-       set_irq_type(irq_num, IRQ_TYPE_EDGE_FALLING);
-       set_irq_chained_handler(irq_num, do_twl4030_irq);
-
-       res = power_companion_init();
-       if (res < 0)
-               pr_err("%s: %s[%d]\n", DRIVER_NAME, msg, res);
+       if (e < 0)
+               pr_err("%s: clock init err [%d]\n", DRIVER_NAME, e);
 }
 
 /*----------------------------------------------------------------------*/
 
+int twl_init_irq(int irq_num, unsigned irq_base, unsigned irq_end);
+int twl_exit_irq(void);
+
 static int twl4030_remove(struct i2c_client *client)
 {
        unsigned i;
+       int status;
 
-       /* FIXME undo twl_init_irq() */
-       if (twl4030_irq_base) {
-               dev_err(&client->dev, "can't yet clean up IRQs?\n");
-               return -ENOSYS;
-       }
+       status = twl_exit_irq();
+       if (status < 0)
+               return status;
 
        for (i = 0; i < TWL4030_NUM_SLAVES; i++) {
                struct twl4030_client   *twl = &twl4030_modules[i];
@@ -1153,8 +697,8 @@ static int twl4030_remove(struct i2c_client *client)
                if (twl->client && twl->client != client)
                        i2c_unregister_device(twl->client);
                twl4030_modules[i].client = NULL;
-               twl4030_modules[i].inuse = false;
        }
+       inuse = false;
        return 0;
 }
 
@@ -1176,11 +720,9 @@ twl4030_probe(struct i2c_client *client, const struct i2c_device_id *id)
                return -EIO;
        }
 
-       for (i = 0; i < TWL4030_NUM_SLAVES; i++) {
-               if (twl4030_modules[i].inuse || twl4030_irq_base) {
-                       dev_dbg(&client->dev, "driver is already in use\n");
-                       return -EBUSY;
-               }
+       if (inuse) {
+               dev_dbg(&client->dev, "driver is already in use\n");
+               return -EBUSY;
        }
 
        for (i = 0; i < TWL4030_NUM_SLAVES; i++) {
@@ -1201,20 +743,20 @@ twl4030_probe(struct i2c_client *client, const struct i2c_device_id *id)
                        strlcpy(twl->client->name, id->name,
                                        sizeof(twl->client->name));
                }
-               twl->inuse = true;
                mutex_init(&twl->xfer_lock);
        }
+       inuse = true;
 
-       /*
-        * Check if the PIH module is initialized, if yes, then init
-        * the T2 Interrupt subsystem
-        */
-       if (twl4030_modules[twl4030_map[TWL4030_MODULE_PIH].sid].inuse
-                       && twl4030_irq_base == 0
-                       && client->irq
+       /* setup clock framework */
+       clocks_init();
+
+       /* Maybe init the T2 Interrupt subsystem */
+       if (client->irq
                        && pdata->irq_base
                        && pdata->irq_end > pdata->irq_base) {
-               twl_init_irq(client->irq, pdata->irq_base, pdata->irq_end);
+               status = twl_init_irq(client->irq, pdata->irq_base, pdata->irq_end);
+               if (status < 0)
+                       goto fail;
                dev_info(&client->dev, "IRQ %d chains IRQs %d..%d\n",
                                client->irq, pdata->irq_base, pdata->irq_end - 1);
        }
@@ -1231,6 +773,7 @@ static const struct i2c_device_id twl4030_ids[] = {
        { "tps65950", 0 },      /* catalog version of twl4030 */
        { "tps65930", 0 },      /* fewer LDOs and DACs; no charger */
        { "tps65920", 0 },      /* fewer LDOs; no codec or charger */
+       { "twl5030", 0 },       /* T2 updated */
        { /* end of list */ },
 };
 MODULE_DEVICE_TABLE(i2c, twl4030_ids);