twl4030_charger: remember to clean up USB regulator
[pandora-kernel.git] / drivers / power / twl4030_charger.c
index 248a23b..575407a 100644 (file)
 #include <linux/power_supply.h>
 #include <linux/notifier.h>
 #include <linux/usb/otg.h>
+#include <linux/ratelimit.h>
+#include <linux/regulator/machine.h>
+#include <linux/leds.h>
 
 #define TWL4030_BCIMSTATEC     0x02
 #define TWL4030_BCIICHG                0x08
 #define TWL4030_BCIVAC         0x0a
 #define TWL4030_BCIVBUS                0x0c
+#define TWL4030_BCIMFSTS3      0x0f
 #define TWL4030_BCIMFSTS4      0x10
+#define TWL4030_BCIMFKEY       0x11
 #define TWL4030_BCICTL1                0x23
+#define TWL4030_BCIIREF1       0x27
+#define TWL4030_BCIIREF2       0x28
 
 #define TWL4030_BCIAUTOWEN     BIT(5)
 #define TWL4030_CONFIG_DONE    BIT(4)
+#define TWL4030_CVENAC         BIT(2)
 #define TWL4030_BCIAUTOUSB     BIT(1)
 #define TWL4030_BCIAUTOAC      BIT(0)
 #define TWL4030_CGAIN          BIT(5)
 #define TWL4030_USBFASTMCHG    BIT(2)
 #define TWL4030_STS_VBUS       BIT(7)
 #define TWL4030_STS_USB_ID     BIT(2)
+#define TWL4030_STS_CHG                BIT(1)
 
 /* BCI interrupts */
 #define TWL4030_WOVF           BIT(0) /* Watchdog overflow */
 #define TWL4030_MSTATEC_COMPLETE1      0x0b
 #define TWL4030_MSTATEC_COMPLETE4      0x0e
 
+#define TWL4030_KEY_IIREF              0xe7
+#define TWL4030_BATSTSMCHG             BIT(6)
+
+#define IRQ_CHECK_PERIOD       (3 * HZ)
+#define IRQ_CHECK_THRESHOLD    4
+
 static bool allow_usb = 1;
 module_param(allow_usb, bool, 0644);
 MODULE_PARM_DESC(allow_usb, "Allow USB charge drawing default current");
@@ -74,8 +89,24 @@ struct twl4030_bci {
        struct work_struct      work;
        int                     irq_chg;
        int                     irq_bci;
+       bool                    ac_charge_enable;
+       bool                    usb_charge_enable;
+       int                     usb_current;
+       int                     ac_current;
+       enum power_supply_type  current_supply;
+       struct regulator        *usb_reg;
+       int                     usb_enabled;
+       int                     irq_had_charger;
+
+       unsigned long           irq_check_count_time;
+       int                     irq_check_count;
+       int                     irq_check_ac_disabled;
+
+       struct led_trigger      *charging_any_trig;
+       int                     was_charging_any;
 
        unsigned long           event;
+       struct ratelimit_state  ratelimit;
 };
 
 /*
@@ -101,9 +132,14 @@ static int twl4030_bci_read(u8 reg, u8 *val)
        return twl_i2c_read_u8(TWL4030_MODULE_MAIN_CHARGE, val, reg);
 }
 
+static int twl4030_bci_write(u8 reg, u8 val)
+{
+       return twl_i2c_write_u8(TWL4030_MODULE_MAIN_CHARGE, val, reg);
+}
+
 static int twl4030_clear_set_boot_bci(u8 clear, u8 set)
 {
-       return twl4030_clear_set(TWL4030_MODULE_PM_MASTER, 0,
+       return twl4030_clear_set(TWL4030_MODULE_PM_MASTER, clear,
                        TWL4030_CONFIG_DONE | TWL4030_BCIAUTOWEN | set,
                        TWL4030_PM_MASTER_BOOT_BCI);
 }
@@ -151,13 +187,16 @@ static int twl4030_bci_have_vbus(struct twl4030_bci *bci)
 }
 
 /*
- * Enable/Disable USB Charge funtionality.
+ * Enable/Disable USB Charge functionality.
  */
 static int twl4030_charger_enable_usb(struct twl4030_bci *bci, bool enable)
 {
        int ret;
 
        if (enable) {
+               if (!bci->usb_charge_enable)
+                       return -EACCES;
+
                /* Check for USB charger conneted */
                if (!twl4030_bci_have_vbus(bci))
                        return -ENODEV;
@@ -171,6 +210,12 @@ static int twl4030_charger_enable_usb(struct twl4030_bci *bci, bool enable)
                        return -EACCES;
                }
 
+               /* Need to keep regulator on */
+               if (!bci->usb_enabled &&
+                   bci->usb_reg &&
+                   regulator_enable(bci->usb_reg) == 0)
+                       bci->usb_enabled = 1;
+
                /* forcing the field BCIAUTOUSB (BOOT_BCI[1]) to 1 */
                ret = twl4030_clear_set_boot_bci(0, TWL4030_BCIAUTOUSB);
                if (ret < 0)
@@ -181,6 +226,9 @@ static int twl4030_charger_enable_usb(struct twl4030_bci *bci, bool enable)
                        TWL4030_USBFASTMCHG, TWL4030_BCIMFSTS4);
        } else {
                ret = twl4030_clear_set_boot_bci(TWL4030_BCIAUTOUSB, 0);
+               if (bci->usb_enabled &&
+                   regulator_disable(bci->usb_reg) == 0)
+                       bci->usb_enabled = 0;
        }
 
        return ret;
@@ -201,17 +249,133 @@ static int twl4030_charger_enable_ac(bool enable)
        return ret;
 }
 
+static int set_charge_current(struct twl4030_bci *bci, int new_current)
+{
+       u8 val, boot_bci_prev, cgain_set, cgain_clear;
+       int ret, ret2;
+
+       ret = twl4030_bci_read(TWL4030_BCIMFSTS3, &val);
+       if (ret)
+               goto out_norestore;
+
+       if (!(val & TWL4030_BATSTSMCHG)) {
+               dev_err(bci->dev, "missing battery, can't change charge_current\n");
+               goto out_norestore;
+       }
+
+       ret = twl_i2c_read_u8(TWL4030_MODULE_PM_MASTER, &boot_bci_prev,
+               TWL4030_PM_MASTER_BOOT_BCI);
+       if (ret)
+               goto out_norestore;
+
+       /* 
+        * Stop automatic charging here, because charge current change
+        * requires multiple register writes and CGAIN change requires
+        * automatic charge to be stopped (and CV mode disabled too).
+        */
+       ret = twl4030_clear_set_boot_bci(
+               TWL4030_CVENAC | TWL4030_BCIAUTOAC | TWL4030_BCIAUTOUSB, 0);
+       if (ret)
+               goto out;
+
+       ret = twl4030_bci_write(TWL4030_BCIMFKEY, TWL4030_KEY_IIREF);
+       if (ret)
+               goto out;
+
+       ret = twl4030_bci_write(TWL4030_BCIIREF1, new_current & 0xff);
+       if (ret)
+               goto out;
+
+       ret = twl4030_bci_write(TWL4030_BCIMFKEY, TWL4030_KEY_IIREF);
+       if (ret)
+               goto out;
+
+       ret = twl4030_bci_write(TWL4030_BCIIREF2, (new_current >> 8) & 0x1);
+       if (ret)
+               goto out;
+
+       /* Set CGAIN = 0 or 1 */
+       if (new_current > 511) {
+               cgain_set = TWL4030_CGAIN;
+               cgain_clear = 0;
+       } else {
+               cgain_set = 0;
+               cgain_clear = TWL4030_CGAIN;
+       }
+
+       ret = twl4030_clear_set(TWL4030_MODULE_MAIN_CHARGE,
+                       cgain_clear, cgain_set, TWL4030_BCICTL1);
+       if (ret)
+               goto out;
+
+       ret = twl4030_bci_read(TWL4030_BCICTL1, &val);
+       if (ret != 0 || (val & TWL4030_CGAIN) != cgain_set) {
+               dev_err(bci->dev, "CGAIN change failed\n");
+               goto out;
+       }
+
+out:
+       ret2 = twl_i2c_write_u8(TWL4030_MODULE_PM_MASTER, boot_bci_prev,
+               TWL4030_PM_MASTER_BOOT_BCI);
+       if (ret2 != 0)
+               dev_err(bci->dev, "failed boot_bci restore: %d\n", ret2);
+
+out_norestore:
+       if (ret != 0)
+               dev_err(bci->dev, "charge current change failed: %d\n", ret);
+
+       return ret;
+}
+
 /*
  * TWL4030 CHG_PRES (AC charger presence) events
  */
 static irqreturn_t twl4030_charger_interrupt(int irq, void *arg)
 {
        struct twl4030_bci *bci = arg;
+       int have_charger;
+       u8 hw_cond;
+       int ret;
+
+       ret = twl_i2c_read_u8(TWL4030_MODULE_PM_MASTER, &hw_cond,
+                             TWL4030_PM_MASTER_STS_HW_CONDITIONS);
+       if (ret < 0) {
+               dev_err(bci->dev, "HW_CONDITIONS read failed: %d\n", ret);
+               goto out;
+       }
+
+       have_charger = (hw_cond & TWL4030_STS_CHG) ? 1 : 0;
+       if (have_charger == bci->irq_had_charger)
+               goto out;
+       bci->irq_had_charger = have_charger;
+
+       dev_dbg(bci->dev, "CHG_PRES irq, hw_cond %02x\n", hw_cond);
+
+       /*
+        * deal with rare mysterious issue of CHG_PRES changing states at ~4Hz
+        * without any charger connected or anything
+        */
+       if (time_before(jiffies, bci->irq_check_count_time + IRQ_CHECK_PERIOD)) {
+               bci->irq_check_count++;
+               if (have_charger && bci->irq_check_count > IRQ_CHECK_THRESHOLD) {
+                       dev_err(bci->dev, "spurious CHG_PRES irqs detected (%d), disabling charger\n",
+                               bci->irq_check_count);
+                       twl4030_charger_enable_ac(false);
+                       bci->irq_check_ac_disabled = true;
+               }
+       } else {
+               bci->irq_check_count_time = jiffies;
+               bci->irq_check_count = 1;
+               if (have_charger && bci->irq_check_ac_disabled) {
+                       twl4030_charger_enable_ac(true);
+                       bci->irq_check_ac_disabled = false;
+               }
+       }
 
-       dev_dbg(bci->dev, "CHG_PRES irq\n");
        power_supply_changed(&bci->ac);
        power_supply_changed(&bci->usb);
 
+out:
        return IRQ_HANDLED;
 }
 
@@ -243,21 +407,30 @@ static irqreturn_t twl4030_bci_interrupt(int irq, void *arg)
        }
 
        /* various monitoring events, for now we just log them here */
-       if (irqs1 & (TWL4030_TBATOR2 | TWL4030_TBATOR1))
+       if (irqs1 & (TWL4030_TBATOR2 | TWL4030_TBATOR1) &&
+                       __ratelimit(&bci->ratelimit))
                dev_warn(bci->dev, "battery temperature out of range\n");
 
-       if (irqs1 & TWL4030_BATSTS)
+       if (irqs1 & TWL4030_BATSTS && __ratelimit(&bci->ratelimit))
                dev_crit(bci->dev, "battery disconnected\n");
 
-       if (irqs2 & TWL4030_VBATOV)
+       if (irqs2 & TWL4030_VBATOV && __ratelimit(&bci->ratelimit))
                dev_crit(bci->dev, "VBAT overvoltage\n");
 
-       if (irqs2 & TWL4030_VBUSOV)
+       if (irqs2 & TWL4030_VBUSOV && __ratelimit(&bci->ratelimit))
                dev_crit(bci->dev, "VBUS overvoltage\n");
 
-       if (irqs2 & TWL4030_ACCHGOV)
+       if (irqs2 & TWL4030_ACCHGOV && __ratelimit(&bci->ratelimit))
                dev_crit(bci->dev, "Ac charger overvoltage\n");
 
+#if 0
+       /* ack the interrupts */
+       twl_i2c_write_u8(TWL4030_MODULE_INTERRUPTS, irqs1,
+                        TWL4030_INTERRUPTS_BCIISR1A);
+       twl_i2c_write_u8(TWL4030_MODULE_INTERRUPTS, irqs2,
+                        TWL4030_INTERRUPTS_BCIISR2A);
+#endif
+
        return IRQ_HANDLED;
 }
 
@@ -318,6 +491,147 @@ static int twl4030_charger_get_current(void)
        return ret;
 }
 
+static ssize_t twl4030_bci_ac_show_enable(struct device *dev,
+                                         struct device_attribute *attr,
+                                         char *buf)
+{
+       u8 boot_bci;
+       int ret;
+
+       ret = twl_i2c_read_u8(TWL4030_MODULE_PM_MASTER, &boot_bci,
+                             TWL4030_PM_MASTER_BOOT_BCI);
+       if (ret)
+               return ret;
+
+       return sprintf(buf, "%d\n", (boot_bci & TWL4030_BCIAUTOAC) ? 1 : 0);
+}
+
+static ssize_t twl4030_bci_ac_store_enable(struct device *dev,
+                                          struct device_attribute *attr,
+                                          const char *buf, size_t count)
+{
+       struct power_supply *psy = dev_get_drvdata(dev);
+       struct twl4030_bci *bci = container_of(psy, struct twl4030_bci, ac);
+       unsigned long enable;
+       int ret;
+
+       ret = strict_strtoul(buf, 10, &enable);
+       if (ret || enable > 1)
+               return -EINVAL;
+
+       bci->ac_charge_enable = enable;
+       twl4030_charger_enable_ac(enable);
+
+       return count;
+}
+static struct device_attribute dev_attr_enable_ac =
+       __ATTR(enable, S_IRUGO | S_IWUSR, twl4030_bci_ac_show_enable,
+       twl4030_bci_ac_store_enable);
+
+static ssize_t twl4030_bci_usb_show_enable(struct device *dev,
+                                          struct device_attribute *attr,
+                                          char *buf)
+{
+       u8 boot_bci;
+       int ret;
+
+       ret = twl_i2c_read_u8(TWL4030_MODULE_PM_MASTER, &boot_bci,
+                             TWL4030_PM_MASTER_BOOT_BCI);
+       if (ret)
+               return ret;
+
+       return sprintf(buf, "%d\n", (boot_bci & TWL4030_BCIAUTOUSB) ? 1 : 0);
+}
+
+static ssize_t twl4030_bci_usb_store_enable(struct device *dev,
+                                           struct device_attribute *attr,
+                                           const char *buf, size_t count)
+{
+       struct power_supply *psy = dev_get_drvdata(dev);
+       struct twl4030_bci *bci = container_of(psy, struct twl4030_bci, usb);
+       unsigned long enable;
+       int ret;
+
+       ret = strict_strtoul(buf, 10, &enable);
+       if (ret || enable > 1)
+               return -EINVAL;
+
+       bci->usb_charge_enable = enable;
+       twl4030_charger_enable_usb(bci, enable);
+
+       return count;
+}
+static struct device_attribute dev_attr_enable_usb =
+       __ATTR(enable, S_IRUGO | S_IWUSR, twl4030_bci_usb_show_enable,
+       twl4030_bci_usb_store_enable);
+
+static ssize_t show_charge_current(struct device *dev,
+                       struct device_attribute *attr, char *buf)
+{
+       int ret, val;
+       u8 ctl;
+       
+       val = twl4030bci_read_adc_val(TWL4030_BCIIREF1);
+       if (val < 0)
+               return val;
+       ret = twl4030_bci_read(TWL4030_BCICTL1, &ctl);
+       if (ret < 0)
+               return ret;
+
+       val &= 0x1ff;
+       if (ctl & TWL4030_CGAIN)
+               val |= 0x200;
+
+       return sprintf(buf, "%d\n", val);
+}
+
+static ssize_t store_charge_current(struct device *dev,
+                       struct device_attribute *attr, const char *buf,
+                       size_t count)
+{
+       struct power_supply *psy = dev_get_drvdata(dev);
+       struct twl4030_bci *bci = dev_get_drvdata(psy->dev->parent);
+       unsigned long new_current;
+       int ret;
+
+       ret = strict_strtoul(buf, 10, &new_current);
+       if (ret)
+               return -EINVAL;
+
+       ret = set_charge_current(bci, new_current);
+       if (ret)
+               return ret;
+
+       if (psy->type == POWER_SUPPLY_TYPE_MAINS)
+               bci->ac_current = new_current;
+       else
+               bci->usb_current = new_current;
+
+       return count;
+}
+static DEVICE_ATTR(charge_current, S_IRUGO | S_IWUSR, show_charge_current,
+       store_charge_current);
+
+static struct attribute *bci_ac_attrs[] = {
+       &dev_attr_enable_ac.attr,
+       &dev_attr_charge_current.attr,
+       NULL,
+};
+
+static struct attribute *bci_usb_attrs[] = {
+       &dev_attr_enable_usb.attr,
+       &dev_attr_charge_current.attr,
+       NULL,
+};
+       
+static const struct attribute_group bci_ac_attr_group = {
+       .attrs = bci_ac_attrs,
+};
+
+static const struct attribute_group bci_usb_attr_group = {
+       .attrs = bci_usb_attrs,
+};
+
 /*
  * Returns the main charge FSM state
  * Or < 0 on failure.
@@ -355,7 +669,8 @@ static int twl4030_bci_get_property(struct power_supply *psy,
                                    union power_supply_propval *val)
 {
        struct twl4030_bci *bci = dev_get_drvdata(psy->dev->parent);
-       int is_charging;
+       int is_charging_any = 0;
+       int is_charging = 0;
        int state;
        int ret;
 
@@ -363,10 +678,29 @@ static int twl4030_bci_get_property(struct power_supply *psy,
        if (state < 0)
                return state;
 
-       if (psy->type == POWER_SUPPLY_TYPE_USB)
-               is_charging = state & TWL4030_MSTATEC_USB;
-       else
-               is_charging = state & TWL4030_MSTATEC_AC;
+       if (twl4030_bci_state_to_status(state) ==
+           POWER_SUPPLY_STATUS_CHARGING) {
+               is_charging_any =
+                       state & (TWL4030_MSTATEC_USB | TWL4030_MSTATEC_AC);
+               if (psy->type == POWER_SUPPLY_TYPE_USB)
+                       is_charging = state & TWL4030_MSTATEC_USB;
+               else
+                       is_charging = state & TWL4030_MSTATEC_AC;
+       }
+
+       if (is_charging_any != bci->was_charging_any) {
+               led_trigger_event(bci->charging_any_trig,
+                       is_charging_any ? LED_FULL : LED_OFF);
+               bci->was_charging_any = is_charging_any;
+       }
+
+       if (is_charging && psy->type != bci->current_supply) {
+               if (psy->type == POWER_SUPPLY_TYPE_USB)
+                       set_charge_current(bci, bci->usb_current);
+               else
+                       set_charge_current(bci, bci->ac_current);
+               bci->current_supply = psy->type;
+       }
 
        switch (psp) {
        case POWER_SUPPLY_PROP_STATUS:
@@ -423,10 +757,16 @@ static enum power_supply_property twl4030_charger_props[] = {
 
 static int __init twl4030_bci_probe(struct platform_device *pdev)
 {
+       const struct twl4030_bci_platform_data *pdata = pdev->dev.platform_data;
        struct twl4030_bci *bci;
        int ret;
        u32 reg;
 
+       if (pdata == NULL) {
+               dev_err(&pdev->dev, "No platform data\n");
+               return -EINVAL;
+       }
+
        bci = kzalloc(sizeof(*bci), GFP_KERNEL);
        if (bci == NULL)
                return -ENOMEM;
@@ -434,14 +774,25 @@ static int __init twl4030_bci_probe(struct platform_device *pdev)
        bci->dev = &pdev->dev;
        bci->irq_chg = platform_get_irq(pdev, 0);
        bci->irq_bci = platform_get_irq(pdev, 1);
+       bci->ac_current = 860; /* ~1.2A */
+       bci->usb_current = 330; /* ~560mA */
+       bci->irq_had_charger = -1;
+       bci->irq_check_count_time = jiffies;
 
        platform_set_drvdata(pdev, bci);
 
+       ratelimit_state_init(&bci->ratelimit, HZ, 2);
+
+       led_trigger_register_simple("twl4030_bci-charging",
+               &bci->charging_any_trig);
+
        bci->ac.name = "twl4030_ac";
        bci->ac.type = POWER_SUPPLY_TYPE_MAINS;
        bci->ac.properties = twl4030_charger_props;
        bci->ac.num_properties = ARRAY_SIZE(twl4030_charger_props);
        bci->ac.get_property = twl4030_bci_get_property;
+       bci->ac.supplied_to = pdata->supplied_to;
+       bci->ac.num_supplicants = pdata->num_supplicants;
 
        ret = power_supply_register(&pdev->dev, &bci->ac);
        if (ret) {
@@ -454,6 +805,14 @@ static int __init twl4030_bci_probe(struct platform_device *pdev)
        bci->usb.properties = twl4030_charger_props;
        bci->usb.num_properties = ARRAY_SIZE(twl4030_charger_props);
        bci->usb.get_property = twl4030_bci_get_property;
+       bci->usb.supplied_to = pdata->supplied_to;
+       bci->usb.num_supplicants = pdata->num_supplicants;
+
+       bci->usb_reg = regulator_get(bci->dev, "bci3v1");
+       if (IS_ERR(bci->usb_reg)) {
+               dev_warn(&pdev->dev, "regulator get bci3v1 failed\n");
+               bci->usb_reg = NULL;
+       }
 
        ret = power_supply_register(&pdev->dev, &bci->usb);
        if (ret) {
@@ -485,6 +844,18 @@ static int __init twl4030_bci_probe(struct platform_device *pdev)
                otg_register_notifier(bci->transceiver, &bci->otg_nb);
        }
 
+       ret = sysfs_create_group(&bci->ac.dev->kobj, &bci_ac_attr_group);
+       if (ret) {
+               dev_err(&pdev->dev, "failed to create sysfs group: %d\n", ret);
+               goto fail_sysfs1;
+       }
+
+       ret = sysfs_create_group(&bci->usb.dev->kobj, &bci_usb_attr_group);
+       if (ret) {
+               dev_err(&pdev->dev, "failed to create sysfs group: %d\n", ret);
+               goto fail_sysfs2;
+       }
+
        /* Enable interrupts now. */
        reg = ~(u32)(TWL4030_ICHGLOW | TWL4030_ICHGEOC | TWL4030_TBATOR2 |
                TWL4030_TBATOR1 | TWL4030_BATSTS);
@@ -501,12 +872,18 @@ static int __init twl4030_bci_probe(struct platform_device *pdev)
        if (ret < 0)
                dev_warn(&pdev->dev, "failed to unmask interrupts: %d\n", ret);
 
+       bci->ac_charge_enable = true;
+       bci->usb_charge_enable = true;
        twl4030_charger_enable_ac(true);
        twl4030_charger_enable_usb(bci, true);
 
        return 0;
 
 fail_unmask_interrupts:
+       sysfs_remove_group(&bci->usb.dev->kobj, &bci_usb_attr_group);
+fail_sysfs2:
+       sysfs_remove_group(&bci->ac.dev->kobj, &bci_ac_attr_group);
+fail_sysfs1:
        if (bci->transceiver != NULL) {
                otg_unregister_notifier(bci->transceiver, &bci->otg_nb);
                otg_put_transceiver(bci->transceiver);
@@ -518,7 +895,15 @@ fail_chg_irq:
        power_supply_unregister(&bci->usb);
 fail_register_usb:
        power_supply_unregister(&bci->ac);
+
+       if (bci->usb_reg) {
+               if (bci->usb_enabled)
+                       regulator_disable(bci->usb_reg);
+               regulator_put(bci->usb_reg);
+       }
+
 fail_register_ac:
+       led_trigger_unregister_simple(bci->charging_any_trig);
        platform_set_drvdata(pdev, NULL);
        kfree(bci);
 
@@ -529,6 +914,9 @@ static int __exit twl4030_bci_remove(struct platform_device *pdev)
 {
        struct twl4030_bci *bci = platform_get_drvdata(pdev);
 
+       sysfs_remove_group(&bci->usb.dev->kobj, &bci_usb_attr_group);
+       sysfs_remove_group(&bci->ac.dev->kobj, &bci_ac_attr_group);
+
        twl4030_charger_enable_ac(false);
        twl4030_charger_enable_usb(bci, false);
 
@@ -546,6 +934,14 @@ static int __exit twl4030_bci_remove(struct platform_device *pdev)
        free_irq(bci->irq_chg, bci);
        power_supply_unregister(&bci->usb);
        power_supply_unregister(&bci->ac);
+
+       if (bci->usb_reg) {
+               if (bci->usb_enabled)
+                       regulator_disable(bci->usb_reg);
+               regulator_put(bci->usb_reg);
+       }
+
+       led_trigger_unregister_simple(bci->charging_any_trig);
        platform_set_drvdata(pdev, NULL);
        kfree(bci);