diff options
author | Sebastian Reichel <sre@kernel.org> | 2016-06-17 13:54:32 +0200 |
---|---|---|
committer | Sebastian Reichel <sre@kernel.org> | 2016-08-11 01:11:03 +0200 |
commit | 8c0984e5a75337df513047ec92a6c09d78e3e5cd (patch) | |
tree | a2fc0a0dd7c7c40d9379bdd1c2b279c0959d0e1f /drivers/power/ds2781_battery.c | |
parent | 29b4817d4018df78086157ea3a55c1d9424a7cfc (diff) |
power: move power supply drivers to power/supply
This moves all power supply drivers from drivers/power/
to drivers/power/supply/. The intention is a cleaner
source tree, since drivers/power/ also contains frameworks
unrelated to power supply, like adaptive voltage scaling.
Signed-off-by: Sebastian Reichel <sre@kernel.org>
Diffstat (limited to 'drivers/power/ds2781_battery.c')
-rw-r--r-- | drivers/power/ds2781_battery.c | 839 |
1 files changed, 0 insertions, 839 deletions
diff --git a/drivers/power/ds2781_battery.c b/drivers/power/ds2781_battery.c deleted file mode 100644 index c3680024f399..000000000000 --- a/drivers/power/ds2781_battery.c +++ /dev/null @@ -1,839 +0,0 @@ -/* - * 1-wire client/driver for the Maxim/Dallas DS2781 Stand-Alone Fuel Gauge IC - * - * Author: Renata Sayakhova <renata@oktetlabs.ru> - * - * Based on ds2780_battery drivers - * - * 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. - * - */ - -#include <linux/module.h> -#include <linux/slab.h> -#include <linux/param.h> -#include <linux/pm.h> -#include <linux/platform_device.h> -#include <linux/power_supply.h> -#include <linux/idr.h> - -#include "../w1/w1.h" -#include "../w1/slaves/w1_ds2781.h" - -/* Current unit measurement in uA for a 1 milli-ohm sense resistor */ -#define DS2781_CURRENT_UNITS 1563 -/* Charge unit measurement in uAh for a 1 milli-ohm sense resistor */ -#define DS2781_CHARGE_UNITS 6250 -/* Number of bytes in user EEPROM space */ -#define DS2781_USER_EEPROM_SIZE (DS2781_EEPROM_BLOCK0_END - \ - DS2781_EEPROM_BLOCK0_START + 1) -/* Number of bytes in parameter EEPROM space */ -#define DS2781_PARAM_EEPROM_SIZE (DS2781_EEPROM_BLOCK1_END - \ - DS2781_EEPROM_BLOCK1_START + 1) - -struct ds2781_device_info { - struct device *dev; - struct power_supply *bat; - struct power_supply_desc bat_desc; - struct device *w1_dev; -}; - -enum current_types { - CURRENT_NOW, - CURRENT_AVG, -}; - -static const char model[] = "DS2781"; -static const char manufacturer[] = "Maxim/Dallas"; - -static inline struct ds2781_device_info * -to_ds2781_device_info(struct power_supply *psy) -{ - return power_supply_get_drvdata(psy); -} - -static inline struct power_supply *to_power_supply(struct device *dev) -{ - return dev_get_drvdata(dev); -} - -static inline int ds2781_battery_io(struct ds2781_device_info *dev_info, - char *buf, int addr, size_t count, int io) -{ - return w1_ds2781_io(dev_info->w1_dev, buf, addr, count, io); -} - -static int w1_ds2781_read(struct ds2781_device_info *dev_info, char *buf, - int addr, size_t count) -{ - return ds2781_battery_io(dev_info, buf, addr, count, 0); -} - -static inline int ds2781_read8(struct ds2781_device_info *dev_info, u8 *val, - int addr) -{ - return ds2781_battery_io(dev_info, val, addr, sizeof(u8), 0); -} - -static int ds2781_read16(struct ds2781_device_info *dev_info, s16 *val, - int addr) -{ - int ret; - u8 raw[2]; - - ret = ds2781_battery_io(dev_info, raw, addr, sizeof(raw), 0); - if (ret < 0) - return ret; - - *val = (raw[0] << 8) | raw[1]; - - return 0; -} - -static inline int ds2781_read_block(struct ds2781_device_info *dev_info, - u8 *val, int addr, size_t count) -{ - return ds2781_battery_io(dev_info, val, addr, count, 0); -} - -static inline int ds2781_write(struct ds2781_device_info *dev_info, u8 *val, - int addr, size_t count) -{ - return ds2781_battery_io(dev_info, val, addr, count, 1); -} - -static inline int ds2781_store_eeprom(struct device *dev, int addr) -{ - return w1_ds2781_eeprom_cmd(dev, addr, W1_DS2781_COPY_DATA); -} - -static inline int ds2781_recall_eeprom(struct device *dev, int addr) -{ - return w1_ds2781_eeprom_cmd(dev, addr, W1_DS2781_RECALL_DATA); -} - -static int ds2781_save_eeprom(struct ds2781_device_info *dev_info, int reg) -{ - int ret; - - ret = ds2781_store_eeprom(dev_info->w1_dev, reg); - if (ret < 0) - return ret; - - ret = ds2781_recall_eeprom(dev_info->w1_dev, reg); - if (ret < 0) - return ret; - - return 0; -} - -/* Set sense resistor value in mhos */ -static int ds2781_set_sense_register(struct ds2781_device_info *dev_info, - u8 conductance) -{ - int ret; - - ret = ds2781_write(dev_info, &conductance, - DS2781_RSNSP, sizeof(u8)); - if (ret < 0) - return ret; - - return ds2781_save_eeprom(dev_info, DS2781_RSNSP); -} - -/* Get RSGAIN value from 0 to 1.999 in steps of 0.001 */ -static int ds2781_get_rsgain_register(struct ds2781_device_info *dev_info, - u16 *rsgain) -{ - return ds2781_read16(dev_info, rsgain, DS2781_RSGAIN_MSB); -} - -/* Set RSGAIN value from 0 to 1.999 in steps of 0.001 */ -static int ds2781_set_rsgain_register(struct ds2781_device_info *dev_info, - u16 rsgain) -{ - int ret; - u8 raw[] = {rsgain >> 8, rsgain & 0xFF}; - - ret = ds2781_write(dev_info, raw, - DS2781_RSGAIN_MSB, sizeof(raw)); - if (ret < 0) - return ret; - - return ds2781_save_eeprom(dev_info, DS2781_RSGAIN_MSB); -} - -static int ds2781_get_voltage(struct ds2781_device_info *dev_info, - int *voltage_uV) -{ - int ret; - char val[2]; - int voltage_raw; - - ret = w1_ds2781_read(dev_info, val, DS2781_VOLT_MSB, 2 * sizeof(u8)); - if (ret < 0) - return ret; - /* - * The voltage value is located in 10 bits across the voltage MSB - * and LSB registers in two's compliment form - * Sign bit of the voltage value is in bit 7 of the voltage MSB register - * Bits 9 - 3 of the voltage value are in bits 6 - 0 of the - * voltage MSB register - * Bits 2 - 0 of the voltage value are in bits 7 - 5 of the - * voltage LSB register - */ - voltage_raw = (val[0] << 3) | - (val[1] >> 5); - - /* DS2781 reports voltage in units of 9.76mV, but the battery class - * reports in units of uV, so convert by multiplying by 9760. */ - *voltage_uV = voltage_raw * 9760; - - return 0; -} - -static int ds2781_get_temperature(struct ds2781_device_info *dev_info, - int *temp) -{ - int ret; - char val[2]; - int temp_raw; - - ret = w1_ds2781_read(dev_info, val, DS2781_TEMP_MSB, 2 * sizeof(u8)); - if (ret < 0) - return ret; - /* - * The temperature value is located in 10 bits across the temperature - * MSB and LSB registers in two's compliment form - * Sign bit of the temperature value is in bit 7 of the temperature - * MSB register - * Bits 9 - 3 of the temperature value are in bits 6 - 0 of the - * temperature MSB register - * Bits 2 - 0 of the temperature value are in bits 7 - 5 of the - * temperature LSB register - */ - temp_raw = ((val[0]) << 3) | - (val[1] >> 5); - *temp = temp_raw + (temp_raw / 4); - - return 0; -} - -static int ds2781_get_current(struct ds2781_device_info *dev_info, - enum current_types type, int *current_uA) -{ - int ret, sense_res; - s16 current_raw; - u8 sense_res_raw, reg_msb; - - /* - * The units of measurement for current are dependent on the value of - * the sense resistor. - */ - ret = ds2781_read8(dev_info, &sense_res_raw, DS2781_RSNSP); - if (ret < 0) - return ret; - - if (sense_res_raw == 0) { - dev_err(dev_info->dev, "sense resistor value is 0\n"); - return -EINVAL; - } - sense_res = 1000 / sense_res_raw; - - if (type == CURRENT_NOW) - reg_msb = DS2781_CURRENT_MSB; - else if (type == CURRENT_AVG) - reg_msb = DS2781_IAVG_MSB; - else - return -EINVAL; - - /* - * The current value is located in 16 bits across the current MSB - * and LSB registers in two's compliment form - * Sign bit of the current value is in bit 7 of the current MSB register - * Bits 14 - 8 of the current value are in bits 6 - 0 of the current - * MSB register - * Bits 7 - 0 of the current value are in bits 7 - 0 of the current - * LSB register - */ - ret = ds2781_read16(dev_info, ¤t_raw, reg_msb); - if (ret < 0) - return ret; - - *current_uA = current_raw * (DS2781_CURRENT_UNITS / sense_res); - return 0; -} - -static int ds2781_get_accumulated_current(struct ds2781_device_info *dev_info, - int *accumulated_current) -{ - int ret, sense_res; - s16 current_raw; - u8 sense_res_raw; - - /* - * The units of measurement for accumulated current are dependent on - * the value of the sense resistor. - */ - ret = ds2781_read8(dev_info, &sense_res_raw, DS2781_RSNSP); - if (ret < 0) - return ret; - - if (sense_res_raw == 0) { - dev_err(dev_info->dev, "sense resistor value is 0\n"); - return -EINVAL; - } - sense_res = 1000 / sense_res_raw; - - /* - * The ACR value is located in 16 bits across the ACR MSB and - * LSB registers - * Bits 15 - 8 of the ACR value are in bits 7 - 0 of the ACR - * MSB register - * Bits 7 - 0 of the ACR value are in bits 7 - 0 of the ACR - * LSB register - */ - ret = ds2781_read16(dev_info, ¤t_raw, DS2781_ACR_MSB); - if (ret < 0) - return ret; - - *accumulated_current = current_raw * (DS2781_CHARGE_UNITS / sense_res); - return 0; -} - -static int ds2781_get_capacity(struct ds2781_device_info *dev_info, - int *capacity) -{ - int ret; - u8 raw; - - ret = ds2781_read8(dev_info, &raw, DS2781_RARC); - if (ret < 0) - return ret; - - *capacity = raw; - return 0; -} - -static int ds2781_get_status(struct ds2781_device_info *dev_info, int *status) -{ - int ret, current_uA, capacity; - - ret = ds2781_get_current(dev_info, CURRENT_NOW, ¤t_uA); - if (ret < 0) - return ret; - - ret = ds2781_get_capacity(dev_info, &capacity); - if (ret < 0) - return ret; - - if (power_supply_am_i_supplied(dev_info->bat)) { - if (capacity == 100) - *status = POWER_SUPPLY_STATUS_FULL; - else if (current_uA > 50000) - *status = POWER_SUPPLY_STATUS_CHARGING; - else - *status = POWER_SUPPLY_STATUS_NOT_CHARGING; - } else { - *status = POWER_SUPPLY_STATUS_DISCHARGING; - } - return 0; -} - -static int ds2781_get_charge_now(struct ds2781_device_info *dev_info, - int *charge_now) -{ - int ret; - u16 charge_raw; - - /* - * The RAAC value is located in 16 bits across the RAAC MSB and - * LSB registers - * Bits 15 - 8 of the RAAC value are in bits 7 - 0 of the RAAC - * MSB register - * Bits 7 - 0 of the RAAC value are in bits 7 - 0 of the RAAC - * LSB register - */ - ret = ds2781_read16(dev_info, &charge_raw, DS2781_RAAC_MSB); - if (ret < 0) - return ret; - - *charge_now = charge_raw * 1600; - return 0; -} - -static int ds2781_get_control_register(struct ds2781_device_info *dev_info, - u8 *control_reg) -{ - return ds2781_read8(dev_info, control_reg, DS2781_CONTROL); -} - -static int ds2781_set_control_register(struct ds2781_device_info *dev_info, - u8 control_reg) -{ - int ret; - - ret = ds2781_write(dev_info, &control_reg, - DS2781_CONTROL, sizeof(u8)); - if (ret < 0) - return ret; - - return ds2781_save_eeprom(dev_info, DS2781_CONTROL); -} - -static int ds2781_battery_get_property(struct power_supply *psy, - enum power_supply_property psp, - union power_supply_propval *val) -{ - int ret = 0; - struct ds2781_device_info *dev_info = to_ds2781_device_info(psy); - - switch (psp) { - case POWER_SUPPLY_PROP_VOLTAGE_NOW: - ret = ds2781_get_voltage(dev_info, &val->intval); - break; - - case POWER_SUPPLY_PROP_TEMP: - ret = ds2781_get_temperature(dev_info, &val->intval); - break; - - case POWER_SUPPLY_PROP_MODEL_NAME: - val->strval = model; - break; - - case POWER_SUPPLY_PROP_MANUFACTURER: - val->strval = manufacturer; - break; - - case POWER_SUPPLY_PROP_CURRENT_NOW: - ret = ds2781_get_current(dev_info, CURRENT_NOW, &val->intval); - break; - - case POWER_SUPPLY_PROP_CURRENT_AVG: - ret = ds2781_get_current(dev_info, CURRENT_AVG, &val->intval); - break; - - case POWER_SUPPLY_PROP_STATUS: - ret = ds2781_get_status(dev_info, &val->intval); - break; - - case POWER_SUPPLY_PROP_CAPACITY: - ret = ds2781_get_capacity(dev_info, &val->intval); - break; - - case POWER_SUPPLY_PROP_CHARGE_COUNTER: - ret = ds2781_get_accumulated_current(dev_info, &val->intval); - break; - - case POWER_SUPPLY_PROP_CHARGE_NOW: - ret = ds2781_get_charge_now(dev_info, &val->intval); - break; - - default: - ret = -EINVAL; - } - - return ret; -} - -static enum power_supply_property ds2781_battery_props[] = { - POWER_SUPPLY_PROP_STATUS, - POWER_SUPPLY_PROP_VOLTAGE_NOW, - POWER_SUPPLY_PROP_TEMP, - POWER_SUPPLY_PROP_MODEL_NAME, - POWER_SUPPLY_PROP_MANUFACTURER, - POWER_SUPPLY_PROP_CURRENT_NOW, - POWER_SUPPLY_PROP_CURRENT_AVG, - POWER_SUPPLY_PROP_CAPACITY, - POWER_SUPPLY_PROP_CHARGE_COUNTER, - POWER_SUPPLY_PROP_CHARGE_NOW, -}; - -static ssize_t ds2781_get_pmod_enabled(struct device *dev, - struct device_attribute *attr, - char *buf) -{ - int ret; - u8 control_reg; - struct power_supply *psy = to_power_supply(dev); - struct ds2781_device_info *dev_info = to_ds2781_device_info(psy); - - /* Get power mode */ - ret = ds2781_get_control_register(dev_info, &control_reg); - if (ret < 0) - return ret; - - return sprintf(buf, "%d\n", - !!(control_reg & DS2781_CONTROL_PMOD)); -} - -static ssize_t ds2781_set_pmod_enabled(struct device *dev, - struct device_attribute *attr, - const char *buf, - size_t count) -{ - int ret; - u8 control_reg, new_setting; - struct power_supply *psy = to_power_supply(dev); - struct ds2781_device_info *dev_info = to_ds2781_device_info(psy); - - /* Set power mode */ - ret = ds2781_get_control_register(dev_info, &control_reg); - if (ret < 0) - return ret; - - ret = kstrtou8(buf, 0, &new_setting); - if (ret < 0) - return ret; - - if ((new_setting != 0) && (new_setting != 1)) { - dev_err(dev_info->dev, "Invalid pmod setting (0 or 1)\n"); - return -EINVAL; - } - - if (new_setting) - control_reg |= DS2781_CONTROL_PMOD; - else - control_reg &= ~DS2781_CONTROL_PMOD; - - ret = ds2781_set_control_register(dev_info, control_reg); - if (ret < 0) - return ret; - - return count; -} - -static ssize_t ds2781_get_sense_resistor_value(struct device *dev, - struct device_attribute *attr, - char *buf) -{ - int ret; - u8 sense_resistor; - struct power_supply *psy = to_power_supply(dev); - struct ds2781_device_info *dev_info = to_ds2781_device_info(psy); - - ret = ds2781_read8(dev_info, &sense_resistor, DS2781_RSNSP); - if (ret < 0) - return ret; - - ret = sprintf(buf, "%d\n", sense_resistor); - return ret; -} - -static ssize_t ds2781_set_sense_resistor_value(struct device *dev, - struct device_attribute *attr, - const char *buf, - size_t count) -{ - int ret; - u8 new_setting; - struct power_supply *psy = to_power_supply(dev); - struct ds2781_device_info *dev_info = to_ds2781_device_info(psy); - - ret = kstrtou8(buf, 0, &new_setting); - if (ret < 0) - return ret; - - ret = ds2781_set_sense_register(dev_info, new_setting); - if (ret < 0) - return ret; - - return count; -} - -static ssize_t ds2781_get_rsgain_setting(struct device *dev, - struct device_attribute *attr, - char *buf) -{ - int ret; - u16 rsgain; - struct power_supply *psy = to_power_supply(dev); - struct ds2781_device_info *dev_info = to_ds2781_device_info(psy); - - ret = ds2781_get_rsgain_register(dev_info, &rsgain); - if (ret < 0) - return ret; - - return sprintf(buf, "%d\n", rsgain); -} - -static ssize_t ds2781_set_rsgain_setting(struct device *dev, - struct device_attribute *attr, - const char *buf, - size_t count) -{ - int ret; - u16 new_setting; - struct power_supply *psy = to_power_supply(dev); - struct ds2781_device_info *dev_info = to_ds2781_device_info(psy); - - ret = kstrtou16(buf, 0, &new_setting); - if (ret < 0) - return ret; - - /* Gain can only be from 0 to 1.999 in steps of .001 */ - if (new_setting > 1999) { - dev_err(dev_info->dev, "Invalid rsgain setting (0 - 1999)\n"); - return -EINVAL; - } - - ret = ds2781_set_rsgain_register(dev_info, new_setting); - if (ret < 0) - return ret; - - return count; -} - -static ssize_t ds2781_get_pio_pin(struct device *dev, - struct device_attribute *attr, - char *buf) -{ - int ret; - u8 sfr; - struct power_supply *psy = to_power_supply(dev); - struct ds2781_device_info *dev_info = to_ds2781_device_info(psy); - - ret = ds2781_read8(dev_info, &sfr, DS2781_SFR); - if (ret < 0) - return ret; - - ret = sprintf(buf, "%d\n", sfr & DS2781_SFR_PIOSC); - return ret; -} - -static ssize_t ds2781_set_pio_pin(struct device *dev, - struct device_attribute *attr, - const char *buf, - size_t count) -{ - int ret; - u8 new_setting; - struct power_supply *psy = to_power_supply(dev); - struct ds2781_device_info *dev_info = to_ds2781_device_info(psy); - - ret = kstrtou8(buf, 0, &new_setting); - if (ret < 0) - return ret; - - if ((new_setting != 0) && (new_setting != 1)) { - dev_err(dev_info->dev, "Invalid pio_pin setting (0 or 1)\n"); - return -EINVAL; - } - - ret = ds2781_write(dev_info, &new_setting, - DS2781_SFR, sizeof(u8)); - if (ret < 0) - return ret; - - return count; -} - -static ssize_t ds2781_read_param_eeprom_bin(struct file *filp, - struct kobject *kobj, - struct bin_attribute *bin_attr, - char *buf, loff_t off, size_t count) -{ - struct device *dev = container_of(kobj, struct device, kobj); - struct power_supply *psy = to_power_supply(dev); - struct ds2781_device_info *dev_info = to_ds2781_device_info(psy); - - return ds2781_read_block(dev_info, buf, - DS2781_EEPROM_BLOCK1_START + off, count); -} - -static ssize_t ds2781_write_param_eeprom_bin(struct file *filp, - struct kobject *kobj, - struct bin_attribute *bin_attr, - char *buf, loff_t off, size_t count) -{ - struct device *dev = container_of(kobj, struct device, kobj); - struct power_supply *psy = to_power_supply(dev); - struct ds2781_device_info *dev_info = to_ds2781_device_info(psy); - int ret; - - ret = ds2781_write(dev_info, buf, - DS2781_EEPROM_BLOCK1_START + off, count); - if (ret < 0) - return ret; - - ret = ds2781_save_eeprom(dev_info, DS2781_EEPROM_BLOCK1_START); - if (ret < 0) - return ret; - - return count; -} - -static struct bin_attribute ds2781_param_eeprom_bin_attr = { - .attr = { - .name = "param_eeprom", - .mode = S_IRUGO | S_IWUSR, - }, - .size = DS2781_PARAM_EEPROM_SIZE, - .read = ds2781_read_param_eeprom_bin, - .write = ds2781_write_param_eeprom_bin, -}; - -static ssize_t ds2781_read_user_eeprom_bin(struct file *filp, - struct kobject *kobj, - struct bin_attribute *bin_attr, - char *buf, loff_t off, size_t count) -{ - struct device *dev = container_of(kobj, struct device, kobj); - struct power_supply *psy = to_power_supply(dev); - struct ds2781_device_info *dev_info = to_ds2781_device_info(psy); - - return ds2781_read_block(dev_info, buf, - DS2781_EEPROM_BLOCK0_START + off, count); - -} - -static ssize_t ds2781_write_user_eeprom_bin(struct file *filp, - struct kobject *kobj, - struct bin_attribute *bin_attr, - char *buf, loff_t off, size_t count) -{ - struct device *dev = container_of(kobj, struct device, kobj); - struct power_supply *psy = to_power_supply(dev); - struct ds2781_device_info *dev_info = to_ds2781_device_info(psy); - int ret; - - ret = ds2781_write(dev_info, buf, - DS2781_EEPROM_BLOCK0_START + off, count); - if (ret < 0) - return ret; - - ret = ds2781_save_eeprom(dev_info, DS2781_EEPROM_BLOCK0_START); - if (ret < 0) - return ret; - - return count; -} - -static struct bin_attribute ds2781_user_eeprom_bin_attr = { - .attr = { - .name = "user_eeprom", - .mode = S_IRUGO | S_IWUSR, - }, - .size = DS2781_USER_EEPROM_SIZE, - .read = ds2781_read_user_eeprom_bin, - .write = ds2781_write_user_eeprom_bin, -}; - -static DEVICE_ATTR(pmod_enabled, S_IRUGO | S_IWUSR, ds2781_get_pmod_enabled, - ds2781_set_pmod_enabled); -static DEVICE_ATTR(sense_resistor_value, S_IRUGO | S_IWUSR, - ds2781_get_sense_resistor_value, ds2781_set_sense_resistor_value); -static DEVICE_ATTR(rsgain_setting, S_IRUGO | S_IWUSR, ds2781_get_rsgain_setting, - ds2781_set_rsgain_setting); -static DEVICE_ATTR(pio_pin, S_IRUGO | S_IWUSR, ds2781_get_pio_pin, - ds2781_set_pio_pin); - - -static struct attribute *ds2781_attributes[] = { - &dev_attr_pmod_enabled.attr, - &dev_attr_sense_resistor_value.attr, - &dev_attr_rsgain_setting.attr, - &dev_attr_pio_pin.attr, - NULL -}; - -static const struct attribute_group ds2781_attr_group = { - .attrs = ds2781_attributes, -}; - -static int ds2781_battery_probe(struct platform_device *pdev) -{ - struct power_supply_config psy_cfg = {}; - int ret = 0; - struct ds2781_device_info *dev_info; - - dev_info = devm_kzalloc(&pdev->dev, sizeof(*dev_info), GFP_KERNEL); - if (!dev_info) - return -ENOMEM; - - platform_set_drvdata(pdev, dev_info); - - dev_info->dev = &pdev->dev; - dev_info->w1_dev = pdev->dev.parent; - dev_info->bat_desc.name = dev_name(&pdev->dev); - dev_info->bat_desc.type = POWER_SUPPLY_TYPE_BATTERY; - dev_info->bat_desc.properties = ds2781_battery_props; - dev_info->bat_desc.num_properties = ARRAY_SIZE(ds2781_battery_props); - dev_info->bat_desc.get_property = ds2781_battery_get_property; - - psy_cfg.drv_data = dev_info; - - dev_info->bat = power_supply_register(&pdev->dev, &dev_info->bat_desc, - &psy_cfg); - if (IS_ERR(dev_info->bat)) { - dev_err(dev_info->dev, "failed to register battery\n"); - ret = PTR_ERR(dev_info->bat); - goto fail; - } - - ret = sysfs_create_group(&dev_info->bat->dev.kobj, &ds2781_attr_group); - if (ret) { - dev_err(dev_info->dev, "failed to create sysfs group\n"); - goto fail_unregister; - } - - ret = sysfs_create_bin_file(&dev_info->bat->dev.kobj, - &ds2781_param_eeprom_bin_attr); - if (ret) { - dev_err(dev_info->dev, - "failed to create param eeprom bin file"); - goto fail_remove_group; - } - - ret = sysfs_create_bin_file(&dev_info->bat->dev.kobj, - &ds2781_user_eeprom_bin_attr); - if (ret) { - dev_err(dev_info->dev, - "failed to create user eeprom bin file"); - goto fail_remove_bin_file; - } - - return 0; - -fail_remove_bin_file: - sysfs_remove_bin_file(&dev_info->bat->dev.kobj, - &ds2781_param_eeprom_bin_attr); -fail_remove_group: - sysfs_remove_group(&dev_info->bat->dev.kobj, &ds2781_attr_group); -fail_unregister: - power_supply_unregister(dev_info->bat); -fail: - return ret; -} - -static int ds2781_battery_remove(struct platform_device *pdev) -{ - struct ds2781_device_info *dev_info = platform_get_drvdata(pdev); - - /* - * Remove attributes before unregistering power supply - * because 'bat' will be freed on power_supply_unregister() call. - */ - sysfs_remove_group(&dev_info->bat->dev.kobj, &ds2781_attr_group); - - power_supply_unregister(dev_info->bat); - - return 0; -} - -static struct platform_driver ds2781_battery_driver = { - .driver = { - .name = "ds2781-battery", - }, - .probe = ds2781_battery_probe, - .remove = ds2781_battery_remove, -}; -module_platform_driver(ds2781_battery_driver); - -MODULE_LICENSE("GPL"); -MODULE_AUTHOR("Renata Sayakhova <renata@oktetlabs.ru>"); -MODULE_DESCRIPTION("Maxim/Dallas DS2781 Stand-Alone Fuel Gauage IC driver"); -MODULE_ALIAS("platform:ds2781-battery"); - |