power: bq27xxx_battery: Reorganize I2C into a module
authorAndrew F. Davis <afd@ti.com>
Mon, 23 Nov 2015 16:53:51 +0000 (10:53 -0600)
committerSebastian Reichel <sre@kernel.org>
Sat, 5 Dec 2015 01:06:51 +0000 (02:06 +0100)
Separate out I2C functionality into a module. This fixes several small
issues and simplifies the driver initialization.

Signed-off-by: Andrew F. Davis <afd@ti.com>
Signed-off-by: Sebastian Reichel <sre@kernel.org>
drivers/power/Kconfig
drivers/power/Makefile
drivers/power/bq27xxx_battery.c
drivers/power/bq27xxx_battery_i2c.c [new file with mode: 0644]
include/linux/power/bq27xxx_battery.h

index 237d7aa73e8c8e15d74560c97560fb0ebdfcfead..42a5b51c3d2ecf3f576c99ffc90b284d98f56280 100644 (file)
@@ -160,22 +160,16 @@ config BATTERY_SBS
 config BATTERY_BQ27XXX
        tristate "BQ27xxx battery driver"
        help
-         Say Y here to enable support for batteries with BQ27xxx (I2C/HDQ) chips.
+         Say Y here to enable support for batteries with BQ27xxx chips.
 
 config BATTERY_BQ27XXX_I2C
-       bool "BQ27xxx I2C support"
+       tristate "BQ27xxx I2C support"
        depends on BATTERY_BQ27XXX
        depends on I2C
        default y
        help
-         Say Y here to enable support for batteries with BQ27xxx (I2C) chips.
-
-config BATTERY_BQ27XXX_PLATFORM
-       bool "BQ27xxx HDQ support"
-       depends on BATTERY_BQ27XXX
-       default y
-       help
-         Say Y here to enable support for batteries with BQ27xxx (HDQ) chips.
+         Say Y here to enable support for batteries with BQ27xxx chips
+         connected over an I2C bus.
 
 config BATTERY_DA9030
        tristate "DA9030 battery driver"
index b656638f8b3984ed41be4cb7bd26a57fb05d028c..0e4eab55f8d70a2837055d0ab875bc5988c0b9fe 100644 (file)
@@ -31,6 +31,7 @@ obj-$(CONFIG_BATTERY_IPAQ_MICRO) += ipaq_micro_battery.o
 obj-$(CONFIG_BATTERY_WM97XX)   += wm97xx_battery.o
 obj-$(CONFIG_BATTERY_SBS)      += sbs-battery.o
 obj-$(CONFIG_BATTERY_BQ27XXX)  += bq27xxx_battery.o
+obj-$(CONFIG_BATTERY_BQ27XXX_I2C) += bq27xxx_battery_i2c.o
 obj-$(CONFIG_BATTERY_DA9030)   += da9030_battery.o
 obj-$(CONFIG_BATTERY_DA9052)   += da9052-battery.o
 obj-$(CONFIG_CHARGER_DA9150)   += da9150-charger.o
index 1cad7accf339200e1fce4edfa04b1827f392cc36..8d4ef9fd9b6e8acf37d730389e161f7c5cee20d8 100644 (file)
 #include <linux/delay.h>
 #include <linux/platform_device.h>
 #include <linux/power_supply.h>
-#include <linux/idr.h>
-#include <linux/i2c.h>
 #include <linux/slab.h>
-#include <linux/interrupt.h>
-#include <asm/unaligned.h>
 
 #include <linux/power/bq27xxx_battery.h>
 
 #define BQ27XXX_POWER_CONSTANT         (29200) /* 29.2 µV^2 * 1000 */
 #define BQ27XXX_CURRENT_CONSTANT       (3570) /* 3.57 µV * 1000 */
 
-struct bq27xxx_device_info;
-struct bq27xxx_access_methods {
-       int (*read)(struct bq27xxx_device_info *di, u8 reg, bool single);
-};
-
 #define INVALID_REG_ADDR       0xff
 
 /*
@@ -110,40 +101,6 @@ enum bq27xxx_reg_index {
        BQ27XXX_REG_AP,         /* Average Power */
 };
 
-struct bq27xxx_reg_cache {
-       int temperature;
-       int time_to_empty;
-       int time_to_empty_avg;
-       int time_to_full;
-       int charge_full;
-       int cycle_count;
-       int capacity;
-       int energy;
-       int flags;
-       int power_avg;
-       int health;
-};
-
-struct bq27xxx_device_info {
-       struct device           *dev;
-       int                     id;
-       enum bq27xxx_chip       chip;
-
-       struct bq27xxx_reg_cache cache;
-       int charge_design_full;
-
-       unsigned long last_update;
-       struct delayed_work work;
-
-       struct power_supply     *bat;
-
-       struct bq27xxx_access_methods bus;
-
-       struct mutex lock;
-
-       u8 *regs;
-};
-
 /* Register mappings */
 static u8 bq27000_regs[] = {
        0x00,   /* CONTROL      */
@@ -710,7 +667,7 @@ static int bq27xxx_battery_read_health(struct bq27xxx_device_info *di)
        return POWER_SUPPLY_HEALTH_GOOD;
 }
 
-static void bq27xxx_battery_update(struct bq27xxx_device_info *di)
+void bq27xxx_battery_update(struct bq27xxx_device_info *di)
 {
        struct bq27xxx_reg_cache cache = {0, };
        bool has_ci_flag = di->chip == BQ27000 || di->chip == BQ27010;
@@ -761,6 +718,7 @@ static void bq27xxx_battery_update(struct bq27xxx_device_info *di)
 
        di->last_update = jiffies;
 }
+EXPORT_SYMBOL_GPL(bq27xxx_battery_update);
 
 static void bq27xxx_battery_poll(struct work_struct *work)
 {
@@ -991,32 +949,30 @@ static void bq27xxx_external_power_changed(struct power_supply *psy)
        schedule_delayed_work(&di->work, 0);
 }
 
-static int bq27xxx_powersupply_init(struct bq27xxx_device_info *di,
-                                   const char *name)
+int bq27xxx_battery_setup(struct bq27xxx_device_info *di)
 {
-       int ret;
        struct power_supply_desc *psy_desc;
        struct power_supply_config psy_cfg = { .drv_data = di, };
 
+       INIT_DELAYED_WORK(&di->work, bq27xxx_battery_poll);
+       mutex_init(&di->lock);
+       di->regs = bq27xxx_regs[di->chip];
+
        psy_desc = devm_kzalloc(di->dev, sizeof(*psy_desc), GFP_KERNEL);
        if (!psy_desc)
                return -ENOMEM;
 
-       psy_desc->name = name;
+       psy_desc->name = di->name;
        psy_desc->type = POWER_SUPPLY_TYPE_BATTERY;
        psy_desc->properties = bq27xxx_battery_props[di->chip].props;
        psy_desc->num_properties = bq27xxx_battery_props[di->chip].size;
        psy_desc->get_property = bq27xxx_battery_get_property;
        psy_desc->external_power_changed = bq27xxx_external_power_changed;
 
-       INIT_DELAYED_WORK(&di->work, bq27xxx_battery_poll);
-       mutex_init(&di->lock);
-
        di->bat = power_supply_register_no_ws(di->dev, psy_desc, &psy_cfg);
        if (IS_ERR(di->bat)) {
-               ret = PTR_ERR(di->bat);
-               dev_err(di->dev, "failed to register battery: %d\n", ret);
-               return ret;
+               dev_err(di->dev, "failed to register battery\n");
+               return PTR_ERR(di->bat);
        }
 
        dev_info(di->dev, "support ver. %s enabled\n", DRIVER_VERSION);
@@ -1025,8 +981,9 @@ static int bq27xxx_powersupply_init(struct bq27xxx_device_info *di,
 
        return 0;
 }
+EXPORT_SYMBOL_GPL(bq27xxx_battery_setup);
 
-static void bq27xxx_powersupply_unregister(struct bq27xxx_device_info *di)
+void bq27xxx_battery_teardown(struct bq27xxx_device_info *di)
 {
        /*
         * power_supply_unregister call bq27xxx_battery_get_property which
@@ -1042,192 +999,7 @@ static void bq27xxx_powersupply_unregister(struct bq27xxx_device_info *di)
 
        mutex_destroy(&di->lock);
 }
-
-/* i2c specific code */
-#ifdef CONFIG_BATTERY_BQ27XXX_I2C
-
-/* If the system has several batteries we need a different name for each
- * of them...
- */
-static DEFINE_IDR(battery_id);
-static DEFINE_MUTEX(battery_mutex);
-
-static irqreturn_t bq27xxx_battery_irq_handler_thread(int irq, void *data)
-{
-       struct bq27xxx_device_info *di = data;
-
-       bq27xxx_battery_update(di);
-
-       return IRQ_HANDLED;
-}
-
-static int bq27xxx_battery_i2c_read(struct bq27xxx_device_info *di, u8 reg,
-                                   bool single)
-{
-       struct i2c_client *client = to_i2c_client(di->dev);
-       struct i2c_msg msg[2];
-       unsigned char data[2];
-       int ret;
-
-       if (!client->adapter)
-               return -ENODEV;
-
-       msg[0].addr = client->addr;
-       msg[0].flags = 0;
-       msg[0].buf = &reg;
-       msg[0].len = sizeof(reg);
-       msg[1].addr = client->addr;
-       msg[1].flags = I2C_M_RD;
-       msg[1].buf = data;
-       if (single)
-               msg[1].len = 1;
-       else
-               msg[1].len = 2;
-
-       ret = i2c_transfer(client->adapter, msg, ARRAY_SIZE(msg));
-       if (ret < 0)
-               return ret;
-
-       if (!single)
-               ret = get_unaligned_le16(data);
-       else
-               ret = data[0];
-
-       return ret;
-}
-
-static int bq27xxx_battery_i2c_probe(struct i2c_client *client,
-                                    const struct i2c_device_id *id)
-{
-       char *name;
-       struct bq27xxx_device_info *di;
-       int num;
-       int retval = 0;
-
-       /* Get new ID for the new battery device */
-       mutex_lock(&battery_mutex);
-       num = idr_alloc(&battery_id, client, 0, 0, GFP_KERNEL);
-       mutex_unlock(&battery_mutex);
-       if (num < 0)
-               return num;
-
-       name = devm_kasprintf(&client->dev, GFP_KERNEL, "%s-%d", id->name, num);
-       if (!name) {
-               retval = -ENOMEM;
-               goto batt_failed;
-       }
-
-       di = devm_kzalloc(&client->dev, sizeof(*di), GFP_KERNEL);
-       if (!di) {
-               retval = -ENOMEM;
-               goto batt_failed;
-       }
-
-       di->id = num;
-       di->dev = &client->dev;
-       di->chip = id->driver_data;
-       di->bus.read = &bq27xxx_battery_i2c_read;
-       di->regs = bq27xxx_regs[di->chip];
-
-       retval = bq27xxx_powersupply_init(di, name);
-       if (retval)
-               goto batt_failed;
-
-       /* Schedule a polling after about 1 min */
-       schedule_delayed_work(&di->work, 60 * HZ);
-
-       i2c_set_clientdata(client, di);
-
-       if (client->irq) {
-               retval = devm_request_threaded_irq(&client->dev, client->irq,
-                               NULL, bq27xxx_battery_irq_handler_thread,
-                               IRQF_ONESHOT,
-                               name, di);
-               if (retval) {
-                       dev_err(&client->dev,
-                               "Unable to register IRQ %d error %d\n",
-                               client->irq, retval);
-                       return retval;
-               }
-       }
-
-       return 0;
-
-batt_failed:
-       mutex_lock(&battery_mutex);
-       idr_remove(&battery_id, num);
-       mutex_unlock(&battery_mutex);
-
-       return retval;
-}
-
-static int bq27xxx_battery_i2c_remove(struct i2c_client *client)
-{
-       struct bq27xxx_device_info *di = i2c_get_clientdata(client);
-
-       bq27xxx_powersupply_unregister(di);
-
-       mutex_lock(&battery_mutex);
-       idr_remove(&battery_id, di->id);
-       mutex_unlock(&battery_mutex);
-
-       return 0;
-}
-
-static const struct i2c_device_id bq27xxx_id[] = {
-       { "bq27200", BQ27000 },
-       { "bq27210", BQ27010 },
-       { "bq27500", BQ27500 },
-       { "bq27510", BQ27500 },
-       { "bq27520", BQ27500 },
-       { "bq27530", BQ27530 },
-       { "bq27531", BQ27530 },
-       { "bq27541", BQ27541 },
-       { "bq27542", BQ27541 },
-       { "bq27546", BQ27541 },
-       { "bq27742", BQ27541 },
-       { "bq27545", BQ27545 },
-       { "bq27421", BQ27421 },
-       { "bq27425", BQ27421 },
-       { "bq27441", BQ27421 },
-       { "bq27621", BQ27421 },
-       {},
-};
-MODULE_DEVICE_TABLE(i2c, bq27xxx_id);
-
-static struct i2c_driver bq27xxx_battery_i2c_driver = {
-       .driver = {
-               .name = "bq27xxx-battery",
-       },
-       .probe = bq27xxx_battery_i2c_probe,
-       .remove = bq27xxx_battery_i2c_remove,
-       .id_table = bq27xxx_id,
-};
-
-static inline int bq27xxx_battery_i2c_init(void)
-{
-       int ret = i2c_add_driver(&bq27xxx_battery_i2c_driver);
-
-       if (ret)
-               pr_err("Unable to register BQ27xxx i2c driver\n");
-
-       return ret;
-}
-
-static inline void bq27xxx_battery_i2c_exit(void)
-{
-       i2c_del_driver(&bq27xxx_battery_i2c_driver);
-}
-
-#else
-
-static inline int bq27xxx_battery_i2c_init(void) { return 0; }
-static inline void bq27xxx_battery_i2c_exit(void) {};
-
-#endif
-
-/* platform specific code */
-#ifdef CONFIG_BATTERY_BQ27XXX_PLATFORM
+EXPORT_SYMBOL_GPL(bq27xxx_battery_teardown);
 
 static int bq27xxx_battery_platform_read(struct bq27xxx_device_info *di, u8 reg,
                                         bool single)
@@ -1267,7 +1039,6 @@ static int bq27xxx_battery_platform_probe(struct platform_device *pdev)
 {
        struct bq27xxx_device_info *di;
        struct bq27xxx_platform_data *pdata = pdev->dev.platform_data;
-       const char *name;
 
        if (!pdata) {
                dev_err(&pdev->dev, "no platform_data supplied\n");
@@ -1292,83 +1063,36 @@ static int bq27xxx_battery_platform_probe(struct platform_device *pdev)
 
        di->dev = &pdev->dev;
        di->chip = pdata->chip;
-       di->regs = bq27xxx_regs[di->chip];
-
-       name = pdata->name ?: dev_name(&pdev->dev);
-       di->bus.read = &bq27xxx_battery_platform_read;
+       di->name = pdata->name ?: dev_name(&pdev->dev);
+       di->bus.read = bq27xxx_battery_platform_read;
 
-       return bq27xxx_powersupply_init(di, name);
+       return bq27xxx_battery_setup(di);
 }
 
 static int bq27xxx_battery_platform_remove(struct platform_device *pdev)
 {
        struct bq27xxx_device_info *di = platform_get_drvdata(pdev);
 
-       bq27xxx_powersupply_unregister(di);
+       bq27xxx_battery_teardown(di);
 
        return 0;
 }
 
+static const struct platform_device_id bq27xxx_battery_platform_id_table[] = {
+       { "bq27000-battery", },
+       { /* sentinel */ }
+};
+MODULE_DEVICE_TABLE(platform, bq27xxx_battery_platform_id_table);
+
 static struct platform_driver bq27xxx_battery_platform_driver = {
        .probe  = bq27xxx_battery_platform_probe,
        .remove = bq27xxx_battery_platform_remove,
        .driver = {
                .name = "bq27000-battery",
        },
+       .id_table = bq27xxx_battery_platform_id_table,
 };
-
-static inline int bq27xxx_battery_platform_init(void)
-{
-       int ret = platform_driver_register(&bq27xxx_battery_platform_driver);
-
-       if (ret)
-               pr_err("Unable to register BQ27xxx platform driver\n");
-
-       return ret;
-}
-
-static inline void bq27xxx_battery_platform_exit(void)
-{
-       platform_driver_unregister(&bq27xxx_battery_platform_driver);
-}
-
-#else
-
-static inline int bq27xxx_battery_platform_init(void) { return 0; }
-static inline void bq27xxx_battery_platform_exit(void) {};
-
-#endif
-
-/*
- * Module stuff
- */
-
-static int __init bq27xxx_battery_init(void)
-{
-       int ret;
-
-       ret = bq27xxx_battery_i2c_init();
-       if (ret)
-               return ret;
-
-       ret = bq27xxx_battery_platform_init();
-       if (ret)
-               bq27xxx_battery_i2c_exit();
-
-       return ret;
-}
-module_init(bq27xxx_battery_init);
-
-static void __exit bq27xxx_battery_exit(void)
-{
-       bq27xxx_battery_platform_exit();
-       bq27xxx_battery_i2c_exit();
-}
-module_exit(bq27xxx_battery_exit);
-
-#ifdef CONFIG_BATTERY_BQ27XXX_PLATFORM
-MODULE_ALIAS("platform:bq27000-battery");
-#endif
+module_platform_driver(bq27xxx_battery_platform_driver);
 
 MODULE_AUTHOR("Rodolfo Giometti <giometti@linux.it>");
 MODULE_DESCRIPTION("BQ27xxx battery monitor driver");
diff --git a/drivers/power/bq27xxx_battery_i2c.c b/drivers/power/bq27xxx_battery_i2c.c
new file mode 100644 (file)
index 0000000..9429e66
--- /dev/null
@@ -0,0 +1,150 @@
+/*
+ * SCI Reset driver for Keystone based devices
+ *
+ * Copyright (C) 2015 Texas Instruments Incorporated - http://www.ti.com/
+ *     Andrew F. Davis <afd@ti.com>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 as
+ * published by the Free Software Foundation.
+ *
+ * This program is distributed "as is" WITHOUT ANY WARRANTY of any
+ * kind, whether express or implied; without even the implied warranty
+ * of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ */
+
+#include <linux/i2c.h>
+#include <linux/interrupt.h>
+#include <linux/module.h>
+#include <asm/unaligned.h>
+
+#include <linux/power/bq27xxx_battery.h>
+
+static irqreturn_t bq27xxx_battery_irq_handler_thread(int irq, void *data)
+{
+       struct bq27xxx_device_info *di = data;
+
+       bq27xxx_battery_update(di);
+
+       return IRQ_HANDLED;
+}
+
+static int bq27xxx_battery_i2c_read(struct bq27xxx_device_info *di, u8 reg,
+                                   bool single)
+{
+       struct i2c_client *client = to_i2c_client(di->dev);
+       struct i2c_msg msg[2];
+       unsigned char data[2];
+       int ret;
+
+       if (!client->adapter)
+               return -ENODEV;
+
+       msg[0].addr = client->addr;
+       msg[0].flags = 0;
+       msg[0].buf = &reg;
+       msg[0].len = sizeof(reg);
+       msg[1].addr = client->addr;
+       msg[1].flags = I2C_M_RD;
+       msg[1].buf = data;
+       if (single)
+               msg[1].len = 1;
+       else
+               msg[1].len = 2;
+
+       ret = i2c_transfer(client->adapter, msg, ARRAY_SIZE(msg));
+       if (ret < 0)
+               return ret;
+
+       if (!single)
+               ret = get_unaligned_le16(data);
+       else
+               ret = data[0];
+
+       return ret;
+}
+
+static int bq27xxx_battery_i2c_probe(struct i2c_client *client,
+                                    const struct i2c_device_id *id)
+{
+       struct bq27xxx_device_info *di;
+       int ret;
+
+       di = devm_kzalloc(&client->dev, sizeof(*di), GFP_KERNEL);
+       if (!di)
+               return -ENOMEM;
+
+       di->dev = &client->dev;
+       di->chip = id->driver_data;
+       di->name = id->name;
+       di->bus.read = bq27xxx_battery_i2c_read;
+
+       ret = bq27xxx_battery_setup(di);
+       if (ret)
+               return ret;
+
+       /* Schedule a polling after about 1 min */
+       schedule_delayed_work(&di->work, 60 * HZ);
+
+       i2c_set_clientdata(client, di);
+
+       if (client->irq) {
+               ret = devm_request_threaded_irq(&client->dev, client->irq,
+                               NULL, bq27xxx_battery_irq_handler_thread,
+                               IRQF_ONESHOT,
+                               di->name, di);
+               if (ret) {
+                       dev_err(&client->dev,
+                               "Unable to register IRQ %d error %d\n",
+                               client->irq, ret);
+                       return ret;
+               }
+       }
+
+       return 0;
+}
+
+static int bq27xxx_battery_i2c_remove(struct i2c_client *client)
+{
+       struct bq27xxx_device_info *di = i2c_get_clientdata(client);
+
+       bq27xxx_battery_teardown(di);
+
+       return 0;
+}
+
+static const struct i2c_device_id bq27xxx_i2c_id_table[] = {
+       { "bq27200", BQ27000 },
+       { "bq27210", BQ27010 },
+       { "bq27500", BQ27500 },
+       { "bq27510", BQ27500 },
+       { "bq27520", BQ27500 },
+       { "bq27530", BQ27530 },
+       { "bq27531", BQ27530 },
+       { "bq27541", BQ27541 },
+       { "bq27542", BQ27541 },
+       { "bq27546", BQ27541 },
+       { "bq27742", BQ27541 },
+       { "bq27545", BQ27545 },
+       { "bq27421", BQ27421 },
+       { "bq27425", BQ27421 },
+       { "bq27441", BQ27421 },
+       { "bq27621", BQ27421 },
+       {},
+};
+MODULE_DEVICE_TABLE(i2c, bq27xxx_i2c_id_table);
+
+static struct i2c_driver bq27xxx_battery_i2c_driver = {
+       .driver = {
+               .name = "bq27xxx-battery",
+       },
+       .probe = bq27xxx_battery_i2c_probe,
+       .remove = bq27xxx_battery_i2c_remove,
+       .id_table = bq27xxx_i2c_id_table,
+};
+module_i2c_driver(bq27xxx_battery_i2c_driver);
+
+MODULE_AUTHOR("Andrew F. Davis <afd@ti.com>");
+MODULE_DESCRIPTION("BQ27xxx battery monitor i2c driver");
+MODULE_LICENSE("GPL");
index 45f6a7b5b3cb6ce7cf11b411190e0d9d07ca4d57..998d8f1c3c9172037ef185963f415dbb0fad999f 100644 (file)
@@ -1,6 +1,16 @@
 #ifndef __LINUX_BQ27X00_BATTERY_H__
 #define __LINUX_BQ27X00_BATTERY_H__
 
+enum bq27xxx_chip {
+       BQ27000 = 1, /* bq27000, bq27200 */
+       BQ27010, /* bq27010, bq27210 */
+       BQ27500, /* bq27500, bq27510, bq27520 */
+       BQ27530, /* bq27530, bq27531 */
+       BQ27541, /* bq27541, bq27542, bq27546, bq27742 */
+       BQ27545, /* bq27545 */
+       BQ27421, /* bq27421, bq27425, bq27441, bq27621 */
+};
+
 /**
  * struct bq27xxx_plaform_data - Platform data for bq27xxx devices
  * @name: Name of the battery.
  *     register to be read. The return value should either be the content of
  *     the passed register or an error value.
  */
-enum bq27xxx_chip {
-       BQ27000 = 1, /* bq27000, bq27200 */
-       BQ27010, /* bq27010, bq27210 */
-       BQ27500, /* bq27500, bq27510, bq27520 */
-       BQ27530, /* bq27530, bq27531 */
-       BQ27541, /* bq27541, bq27542, bq27546, bq27742 */
-       BQ27545, /* bq27545 */
-       BQ27421, /* bq27421, bq27425, bq27441, bq27621 */
-};
-
 struct bq27xxx_platform_data {
        const char *name;
        enum bq27xxx_chip chip;
        int (*read)(struct device *dev, unsigned int);
 };
 
+struct bq27xxx_device_info;
+struct bq27xxx_access_methods {
+       int (*read)(struct bq27xxx_device_info *di, u8 reg, bool single);
+};
+
+struct bq27xxx_reg_cache {
+       int temperature;
+       int time_to_empty;
+       int time_to_empty_avg;
+       int time_to_full;
+       int charge_full;
+       int cycle_count;
+       int capacity;
+       int energy;
+       int flags;
+       int power_avg;
+       int health;
+};
+
+struct bq27xxx_device_info {
+       struct device *dev;
+       enum bq27xxx_chip chip;
+       const char *name;
+       struct bq27xxx_access_methods bus;
+       struct bq27xxx_reg_cache cache;
+       int charge_design_full;
+       unsigned long last_update;
+       struct delayed_work work;
+       struct power_supply *bat;
+       struct mutex lock;
+       u8 *regs;
+};
+
+void bq27xxx_battery_update(struct bq27xxx_device_info *di);
+int bq27xxx_battery_setup(struct bq27xxx_device_info *di);
+void bq27xxx_battery_teardown(struct bq27xxx_device_info *di);
+
 #endif