Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S376684AbdD2Oex (ORCPT ); Sat, 29 Apr 2017 10:34:53 -0400 Received: from huan1.mail.rambler.ru ([81.19.78.107]:56894 "EHLO huan1.mail.rambler.ru" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S376656AbdD2Oej (ORCPT ); Sat, 29 Apr 2017 10:34:39 -0400 From: "Alex A. Mihaylov" To: linux-kernel@vger.kernel.org Cc: zbr@ioremap.net, greg@kroah.com, "Alex A. Mihaylov" Subject: [PATCH 1/2] Add support for OneWire (W1) devices family 0x26 (MAX17211/MAX17215) Date: Sat, 29 Apr 2017 17:34:28 +0300 Message-Id: <20170429143429.5685-2-minimumlaw@rambler.ru> X-Mailer: git-send-email 2.8.4 (Apple Git-73) In-Reply-To: <20170429143429.5685-1-minimumlaw@rambler.ru> References: <20170429143429.5685-1-minimumlaw@rambler.ru> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-Rambler-User: minimumlaw@rambler.ru/178.70.112.37 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 8820 Lines: 298 Maxim Semiconductor MAX17211/MAX17215 single/multi-cell fuel gauge monitor with M5 Fuel Gauge algorithm Slave device provide software layer for access to internal registers MAX17211/MAX17215 chip. --- drivers/w1/slaves/Kconfig | 12 ++++ drivers/w1/slaves/Makefile | 1 + drivers/w1/slaves/w1_max1721x.c | 121 ++++++++++++++++++++++++++++++++++++++++ drivers/w1/slaves/w1_max1721x.h | 101 +++++++++++++++++++++++++++++++++ drivers/w1/w1_family.h | 1 + 5 files changed, 236 insertions(+) create mode 100644 drivers/w1/slaves/w1_max1721x.c create mode 100644 drivers/w1/slaves/w1_max1721x.h diff --git a/drivers/w1/slaves/Kconfig b/drivers/w1/slaves/Kconfig index cfe74d0..05467da 100644 --- a/drivers/w1/slaves/Kconfig +++ b/drivers/w1/slaves/Kconfig @@ -78,6 +78,18 @@ config W1_SLAVE_DS2433_CRC Each block has 30 bytes of data and a two byte CRC16. Full block writes are only allowed if the CRC is valid. +config W1_SLAVE_MAX1721X + tristate "Maxim MAX17211/MAX17215 battery monitor chip" + help + If you enable this you will have the MAX17211/MAX17215 battery + monitor chip support. + + The battery monitor chip is used in many batteries/devices + as the one who is responsible for charging/discharging/monitoring + Li+ batteries. + + If you are unsure, say N. + config W1_SLAVE_DS2760 tristate "Dallas 2760 battery monitor chip (HP iPAQ & others)" help diff --git a/drivers/w1/slaves/Makefile b/drivers/w1/slaves/Makefile index 1e9989a..1e9f942 100644 --- a/drivers/w1/slaves/Makefile +++ b/drivers/w1/slaves/Makefile @@ -10,6 +10,7 @@ obj-$(CONFIG_W1_SLAVE_DS2406) += w1_ds2406.o obj-$(CONFIG_W1_SLAVE_DS2423) += w1_ds2423.o obj-$(CONFIG_W1_SLAVE_DS2431) += w1_ds2431.o obj-$(CONFIG_W1_SLAVE_DS2433) += w1_ds2433.o +obj-$(CONFIG_W1_SLAVE_MAX1721X) += w1_max1721x.o obj-$(CONFIG_W1_SLAVE_DS2760) += w1_ds2760.o obj-$(CONFIG_W1_SLAVE_DS2780) += w1_ds2780.o obj-$(CONFIG_W1_SLAVE_DS2781) += w1_ds2781.o diff --git a/drivers/w1/slaves/w1_max1721x.c b/drivers/w1/slaves/w1_max1721x.c new file mode 100644 index 0000000..bb73e90 --- /dev/null +++ b/drivers/w1/slaves/w1_max1721x.c @@ -0,0 +1,121 @@ +/* + * 1-Wire implementation for the max17211 chip + * + * Copyright © 2017, Alex A. Mihaylov + * + * Use consistent with the GNU GPL is permitted, + * provided that this copyright notice is + * preserved in its entirety in all copies and derived works. + * + */ + +#include +#include +#include +#include +#include +#include +#include +#include + +#include "../w1.h" +#include "../w1_int.h" +#include "../w1_family.h" +#include "w1_max1721x.h" + +int w1_max1721x_reg_get(struct device *dev, uint16_t addr, uint16_t *val) +{ + struct w1_slave *sl = container_of(dev, struct w1_slave, dev); + int ret = -ENODEV; + + if (addr > MAX1721X_MAX_REG_NR || addr < 0 || !val || !dev) + return -EFAULT; + + mutex_lock(&sl->master->bus_mutex); + + if (!w1_reset_select_slave(sl)) { + w1_write_8(sl->master, W1_MAX1721X_READ_DATA); + w1_write_8(sl->master, addr & 0x00FF); + w1_write_8(sl->master, addr>>8 & 0x00FF); + *val = w1_read_8(sl->master); + *val |= w1_read_8(sl->master)<<8; + ret = 0; + } + + mutex_unlock(&sl->master->bus_mutex); + + return ret; +} +EXPORT_SYMBOL(w1_max1721x_reg_get); + +int w1_max1721x_reg_set(struct device *dev, uint16_t addr, uint16_t val) +{ + struct w1_slave *sl = container_of(dev, struct w1_slave, dev); + int ret = -ENODEV; + + if (addr > MAX1721X_MAX_REG_NR || addr < 0 || !dev) + return -EFAULT; + + mutex_lock(&sl->master->bus_mutex); + + if (!w1_reset_select_slave(sl)) { + w1_write_8(sl->master, W1_MAX1721X_READ_DATA); + w1_write_8(sl->master, addr & 0x00FF); + w1_write_8(sl->master, addr>>8 & 0x00FF); + w1_write_8(sl->master, val & 0x00FF); + w1_write_8(sl->master, val>>8 & 0x00FF); + ret = 0; + } + + mutex_unlock(&sl->master->bus_mutex); + + return ret; +} +EXPORT_SYMBOL(w1_max1721x_reg_set); + +static int w1_max17211_add_device(struct w1_slave *sl) +{ + int ret; + struct platform_device *pdev; + + pdev = platform_device_alloc("max1721x-battery", PLATFORM_DEVID_AUTO); + if (!pdev) + return -ENOMEM; + pdev->dev.parent = &sl->dev; + + ret = platform_device_add(pdev); + if (ret) + goto pdev_add_failed; + + dev_set_drvdata(&sl->dev, pdev); + + return 0; + +pdev_add_failed: + platform_device_put(pdev); + + return ret; +} + +static void w1_max17211_remove_device(struct w1_slave *sl) +{ + struct platform_device *pdev = dev_get_drvdata(&sl->dev); + + platform_device_unregister(pdev); +} + +static struct w1_family_ops w1_max17211_fops = { + .add_slave = w1_max17211_add_device, + .remove_slave = w1_max17211_remove_device, +}; + +static struct w1_family w1_max17211_family = { + .fid = W1_FAMILY_MAX17211, + .fops = &w1_max17211_fops, +}; +module_w1_family(w1_max17211_family); + +MODULE_LICENSE("GPL"); +MODULE_AUTHOR("Alex A. Mihaylov "); +MODULE_DESCRIPTION("1-wire Driver for MAX17211/MAX17215 battery monitor"); +MODULE_ALIAS("w1-family-" __stringify(W1_FAMILY_MAX17211)); diff --git a/drivers/w1/slaves/w1_max1721x.h b/drivers/w1/slaves/w1_max1721x.h new file mode 100644 index 0000000..1deb3d3 --- /dev/null +++ b/drivers/w1/slaves/w1_max1721x.h @@ -0,0 +1,101 @@ +/* + * 1-Wire implementation for the max17211 chip + * + * Copyright © 2017, Alex A. Mihaylov + * + * Use consistent with the GNU GPL is permitted, + * provided that this copyright notice is + * preserved in its entirety in all copies and derived works. + * + */ + +#ifndef __w1_max17211_h__ +#define __w1_max17211_h__ + +/* Known commands to the MAX1721X chip */ +#define W1_MAX1721X_READ_DATA 0x69 +#define W1_MAX1721X_WRITE_DATA 0x6C + +/* Factory settings (nonvilatile registers) (W1 specific) */ + +#define MAX1721X_REG_NRSENSE 0x1CF /* RSense in 10^-5 Ohm */ +/* Strings */ +#define MAX1721X_REG_MFG_STR 0x1CC +#define MAX1721X_REG_MFG_NUMB 3 +#define MAX1721X_REG_DEV_STR 0x1DB +#define MAX1721X_REG_DEV_NUMB 5 +/* HEX Strings */ +#define MAX1721X_REG_SER_HEX 0x1D8 + +/* Number of valid register addresses */ +#define MAX1721X_MAX_REG_NR (0x1EF) + +/* MAX1721X/MAX17215 Output Registers for I2C and W1 chips */ + +#define MAX172XX_REG_STATUS 0x000 /* status reg */ +#define MAX172XX_BAT_PRESENT (1<<4) /* battery connected bit */ +#define MAX172XX_REG_DEVNAME 0x021 /* chip config */ +#define MAX172XX_DEV_MASK 0x000F /* chip type mask */ +#define MAX172X1_DEV 0x0001 +#define MAX172X5_DEV 0x0005 +#define MAX172XX_REG_TEMP 0x008 /* Temperature */ +#define MAX172XX_REG_BATT 0x0DA /* Battery voltage */ +#define MAX172XX_REG_CURRENT 0x00A /* Actual current */ +#define MAX172XX_REG_AVGCURRENT 0x00B /* Average current */ +#define MAX172XX_REG_REPSOC 0x006 /* Percentage of charge */ +#define MAX172XX_REG_DESIGNCAP 0x018 /* Design capacity */ +#define MAX172XX_REG_REPCAP 0x005 /* Average capacity */ +#define MAX172XX_REG_TTE 0x011 /* Time to empty */ +#define MAX172XX_REG_TTF 0x020 /* Time to full */ + +/* Convert regs value to power_supply units */ + +static inline int max172xx_time_to_ps(uint16_t reg) +{ + return reg * 5625 / 1000; /* in sec. */ +} + +static inline int max172xx_percent_to_ps(uint16_t reg) +{ + return reg / 256; /* in percent from 0 to 100 */ +} + +static inline int max172xx_voltage_to_ps(uint16_t reg) +{ + return reg * 1250; /* in uV */ +} + +static inline int max172xx_capacity_to_ps(uint16_t reg) +{ + return reg * 500; /* in uAh */ +} + +/* + * Current and temperature is signed values, so unsigned regs + * value must be converted to signed type + */ + +static inline int max172xx_temperature_to_ps(uint16_t reg) +{ + return (int16_t)reg * 10 / 256; /* in tenths of deg. C */ +} + +/* + * Calculating current registers resolution: + * + * RSense stored in 10^-5 Ohm, so mesaurment voltage must be + * in 10^-11 Volts for get current in uA. + * 16 bit current reg fullscale +/-51.2mV is 102400 uV. + * So: 102400 / 65535 * 10^5 = 156252 + */ +static inline int max172xx_current_to_voltage(uint16_t reg) +{ + return (int16_t)reg * 156252; +} + +extern int w1_max1721x_reg_get(struct device *dev, uint16_t addr, + uint16_t *val); +extern int w1_max1721x_reg_set(struct device *dev, uint16_t addr, + uint16_t val); + +#endif /* !__w1_max17211_h__ */ diff --git a/drivers/w1/w1_family.h b/drivers/w1/w1_family.h index 10a7a07..797c162 100644 --- a/drivers/w1/w1_family.h +++ b/drivers/w1/w1_family.h @@ -35,6 +35,7 @@ #define W1_COUNTER_DS2423 0x1D #define W1_THERM_DS1822 0x22 #define W1_EEPROM_DS2433 0x23 +#define W1_FAMILY_MAX17211 0x26 #define W1_THERM_DS18B20 0x28 #define W1_FAMILY_DS2408 0x29 #define W1_EEPROM_DS2431 0x2D -- 2.8.4 (Apple Git-73)