On Tue, 19 May 2015, S Twiss wrote: > From: S Twiss <stwiss.opensource@xxxxxxxxxxx> > > Add MFD core driver support for DA9062 > > > Signed-off-by: Steve Twiss <stwiss.opensource@xxxxxxxxxxx> > > --- > Changes in V3: > - Removed references to the RTC and OnKey in the mfd_cell definition. > > Changes in V2: > - Copyright headers GPL v2 (and later) match correct 'GPL' in MODULE_LICENSE > - Ensure DA9062 core is tristate in Kconfig so it can be built as a module > - Move VDD_WARN code and resource into the core instead of regulator driver > > This patch applies against linux-next and v4.1-rc3 > > > > drivers/mfd/Kconfig | 12 + > drivers/mfd/Makefile | 3 +- > drivers/mfd/da9062-core.c | 611 +++++++++++++++++++ > include/linux/mfd/da9062/core.h | 62 ++ > include/linux/mfd/da9062/registers.h | 1108 ++++++++++++++++++++++++++++++++++ > 5 files changed, 1795 insertions(+), 1 deletion(-) > create mode 100644 drivers/mfd/da9062-core.c > create mode 100644 include/linux/mfd/da9062/core.h > create mode 100644 include/linux/mfd/da9062/registers.h > > diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig > index d5ad04d..44fb2b9 100644 > --- a/drivers/mfd/Kconfig > +++ b/drivers/mfd/Kconfig > @@ -183,6 +183,18 @@ config MFD_DA9055 > This driver can be built as a module. If built as a module it will be > called "da9055" > > +config MFD_DA9062 > + tristate "Dialog Semiconductor DA9062 PMIC Support" > + select MFD_CORE > + select REGMAP_I2C > + select REGMAP_IRQ > + depends on I2C=y > + help > + Say yes here for support for the Dialog Semiconductor DA9062 PMIC. > + This includes the I2C driver and core APIs. > + Additional drivers must be enabled in order to use the functionality > + of the device. > + > config MFD_DA9063 > bool "Dialog Semiconductor DA9063 PMIC Support" > select MFD_CORE > diff --git a/drivers/mfd/Makefile b/drivers/mfd/Makefile > index 0e5cfeb..d94e562 100644 > --- a/drivers/mfd/Makefile > +++ b/drivers/mfd/Makefile > @@ -110,10 +110,11 @@ obj-$(CONFIG_MFD_LP8788) += lp8788.o lp8788-irq.o > > da9055-objs := da9055-core.o da9055-i2c.o > obj-$(CONFIG_MFD_DA9055) += da9055.o > - > +obj-$(CONFIG_MFD_DA9062) += da9062-core.o > da9063-objs := da9063-core.o da9063-irq.o da9063-i2c.o > obj-$(CONFIG_MFD_DA9063) += da9063.o > obj-$(CONFIG_MFD_DA9150) += da9150-core.o > + > obj-$(CONFIG_MFD_MAX14577) += max14577.o > obj-$(CONFIG_MFD_MAX77686) += max77686.o > obj-$(CONFIG_MFD_MAX77693) += max77693.o > diff --git a/drivers/mfd/da9062-core.c b/drivers/mfd/da9062-core.c > new file mode 100644 > index 0000000..5aea082 > --- /dev/null > +++ b/drivers/mfd/da9062-core.c > @@ -0,0 +1,611 @@ > +/* > + * da9062-core.c - CORE device driver for DA9062 Remove the filename. They have a habit of becoming incorrect. s/CORE/Core/ Can you also mention what the DA9062 actually is? > + * Copyright (C) 2015 Dialog Semiconductor Ltd. > + * > + * This program is free software; you can redistribute it and/or > + * modify it under the terms of the GNU General Public License > + * as published by the Free Software Foundation; either version 2 > + * of the License, or (at your option) any later version. > + * > + * This program is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > + * GNU General Public License for more details. > + */ > + > +#include <linux/kernel.h> > +#include <linux/module.h> > +#include <linux/init.h> > +#include <linux/slab.h> > +#include <linux/device.h> > +#include <linux/delay.h> > +#include <linux/interrupt.h> > +#include <linux/mutex.h> > +#include <linux/regmap.h> > + Why the '\n'? > +#include <linux/irq.h> > +#include <linux/mfd/core.h> > +#include <linux/i2c.h> > +#include <linux/err.h> > + Same here? > +#include <linux/mfd/da9062/core.h> > +#include <linux/mfd/da9062/registers.h> > +#include <linux/of.h> > +#include <linux/regulator/of_regulator.h> > + > +#include <linux/proc_fs.h> > +#include <linux/kthread.h> > +#include <linux/uaccess.h> I'm a bit concerned by the number of includes here, are you sure they're all required? > +/* IRQ device driver for DA9062 */ Not sure what this means? > +#define DA9062_REG_EVENT_A_OFFSET 0 > +#define DA9062_REG_EVENT_B_OFFSET 1 > +#define DA9062_REG_EVENT_C_OFFSET 2 > + > +static struct regmap_irq da9062_irqs[] = { > + /* EVENT A */ > + [DA9062_IRQ_ONKEY] = { > + .reg_offset = DA9062_REG_EVENT_A_OFFSET, > + .mask = DA9062AA_M_NONKEY_MASK, > + }, > + [DA9062_IRQ_ALARM] = { > + .reg_offset = DA9062_REG_EVENT_A_OFFSET, > + .mask = DA9062AA_M_ALARM_MASK, > + }, > + [DA9062_IRQ_TICK] = { > + .reg_offset = DA9062_REG_EVENT_A_OFFSET, > + .mask = DA9062AA_M_TICK_MASK, > + }, > + [DA9062_IRQ_WDG_WARN] = { > + .reg_offset = DA9062_REG_EVENT_A_OFFSET, > + .mask = DA9062AA_M_WDG_WARN_MASK, > + }, > + [DA9062_IRQ_SEQ_RDY] = { > + .reg_offset = DA9062_REG_EVENT_A_OFFSET, > + .mask = DA9062AA_M_SEQ_RDY_MASK, > + }, > + /* EVENT B */ > + [DA9062_IRQ_TEMP] = { > + .reg_offset = DA9062_REG_EVENT_B_OFFSET, > + .mask = DA9062AA_M_TEMP_MASK, > + }, > + [DA9062_IRQ_LDO_LIM] = { > + .reg_offset = DA9062_REG_EVENT_B_OFFSET, > + .mask = DA9062AA_M_LDO_LIM_MASK, > + }, > + [DA9062_IRQ_DVC_RDY] = { > + .reg_offset = DA9062_REG_EVENT_B_OFFSET, > + .mask = DA9062AA_M_DVC_RDY_MASK, > + }, > + [DA9062_IRQ_VDD_WARN] = { > + .reg_offset = DA9062_REG_EVENT_B_OFFSET, > + .mask = DA9062AA_M_VDD_WARN_MASK, > + }, > + /* EVENT C */ > + [DA9062_IRQ_GPI0] = { > + .reg_offset = DA9062_REG_EVENT_C_OFFSET, > + .mask = DA9062AA_M_GPI0_MASK, > + }, > + [DA9062_IRQ_GPI1] = { > + .reg_offset = DA9062_REG_EVENT_C_OFFSET, > + .mask = DA9062AA_M_GPI1_MASK, > + }, > + [DA9062_IRQ_GPI2] = { > + .reg_offset = DA9062_REG_EVENT_C_OFFSET, > + .mask = DA9062AA_M_GPI2_MASK, > + }, > + [DA9062_IRQ_GPI3] = { > + .reg_offset = DA9062_REG_EVENT_C_OFFSET, > + .mask = DA9062AA_M_GPI3_MASK, > + }, > + [DA9062_IRQ_GPI4] = { > + .reg_offset = DA9062_REG_EVENT_C_OFFSET, > + .mask = DA9062AA_M_GPI4_MASK, > + }, > +}; > + > +static struct regmap_irq_chip da9062_irq_chip = { > + /* IRQ */ Not required I think. > + .name = "da9062-irq", > + .irqs = da9062_irqs, > + .num_irqs = DA9062_NUM_IRQ, > + /* STATUS and EVENT */ This is a standard struct, no need for additional comments. > + .num_regs = 3, > + .status_base = DA9062AA_EVENT_A, > + .mask_base = DA9062AA_IRQ_MASK_A, > + .ack_base = DA9062AA_EVENT_A, > +}; > + > +int da9062_irq_init(struct da9062 *chip) > +{ > + int ret; > + > + if (!chip->chip_irq) { Check this once, in probe(), then rid this check. > + dev_err(chip->dev, "No IRQ configured\n"); > + return -EINVAL; > + } > + > + ret = regmap_add_irq_chip(chip->regmap, chip->chip_irq, > + IRQF_TRIGGER_LOW | IRQF_ONESHOT | IRQF_SHARED, > + chip->irq_base, &da9062_irq_chip, > + &chip->regmap_irq); This is just one call. Just place that call into da9062_device_init() and rid this function. > + if (ret) { > + dev_err(chip->dev, "Failed to request IRQ %d: %d\n", > + chip->chip_irq, ret); > + return ret; > + } > + > + return 0; > +} > + > +void da9062_irq_exit(struct da9062 *chip) > +{ > + regmap_del_irq_chip(chip->chip_irq, chip->regmap_irq); Again, this is abstraction for the sake of abstraction. > +} > + > +/* CORE device driver for DA9062 */ Doesn't add anything. This whole driver is the core driver. > +static struct resource da9062_core_resources[] = { const? > + { > + .name = "VDD_WARN", > + .start = DA9062_IRQ_VDD_WARN, > + .end = DA9062_IRQ_VDD_WARN, > + .flags = IORESOURCE_IRQ, > + }, > +}; > + > +static struct resource da9062_regulators_resources[] = { > + { > + .name = "LDO_LIM", > + .start = DA9062_IRQ_LDO_LIM, > + .end = DA9062_IRQ_LDO_LIM, > + .flags = IORESOURCE_IRQ, > + }, > +}; > + > +static struct resource da9062_thermal_resources[] = { > + { > + .name = "THERMAL", > + .start = DA9062_IRQ_TEMP, > + .end = DA9062_IRQ_TEMP, > + .flags = IORESOURCE_IRQ, > + }, > +}; > + > +static struct resource da9062_wdt_resources[] = { > + { > + .name = "WDG_WARN", > + .start = DA9062_IRQ_WDG_WARN, > + .end = DA9062_IRQ_WDG_WARN, > + .flags = IORESOURCE_IRQ, > + }, > +}; Convert all of these to oneliners using DEFINE_RES_* macros. > +static const struct mfd_cell da9062_devs[] = { > + { > + .name = "da9062-core", > + .num_resources = ARRAY_SIZE(da9062_core_resources), > + .resources = da9062_core_resources, > + }, > + { > + .name = "da9062-regulators", > + .num_resources = ARRAY_SIZE(da9062_regulators_resources), > + .resources = da9062_regulators_resources, > + }, > + { > + .name = "da9062-watchdog", > + .num_resources = ARRAY_SIZE(da9062_wdt_resources), > + .resources = da9062_wdt_resources, > + .of_compatible = "dlg,da9062-wdt", > + }, > + { > + .name = "da9062-thermal", > + .num_resources = ARRAY_SIZE(da9062_thermal_resources), > + .resources = da9062_thermal_resources, > + .of_compatible = "dlg,da9062-thermal", > + }, > +}; > + > +static irqreturn_t da9062_vdd_warn_event(int irq, void *data) > +{ > + struct da9062 *hw = data; > + > + dev_err(hw->dev, "VDD warning indicator\n"); Is it an error? Doesn't look like it. > + return IRQ_HANDLED; > +} > + > +static int da9062_clear_fault_log(struct da9062 *chip) > +{ > + int ret = 0; No need to pre-initialise. > + int fault_log = 0; As above. > + ret = regmap_read(chip->regmap, DA9062AA_FAULT_LOG, &fault_log); > + if (ret < 0) { > + dev_err(chip->dev, "Cannot read FAULT_LOG.\n"); > + ret = -EIO; Just return and rid the else. > + } else { > + if (fault_log) { > + if (fault_log & DA9062AA_TWD_ERROR_MASK) > + dev_dbg(chip->dev, "Fault log entry detected: TWD_ERROR\n"); > + if (fault_log & DA9062AA_POR_MASK) > + dev_dbg(chip->dev, "Fault log entry detected: POR\n"); > + if (fault_log & DA9062AA_VDD_FAULT_MASK) > + dev_dbg(chip->dev, "Fault log entry detected: VDD_FAULT\n"); > + if (fault_log & DA9062AA_VDD_START_MASK) > + dev_dbg(chip->dev, "Fault log entry detected: VDD_START\n"); > + if (fault_log & DA9062AA_TEMP_CRIT_MASK) > + dev_dbg(chip->dev, "Fault log entry detected: TEMP_CRIT\n"); > + if (fault_log & DA9062AA_KEY_RESET_MASK) > + dev_dbg(chip->dev, "Fault log entry detected: KEY_RESET\n"); > + if (fault_log & DA9062AA_NSHUTDOWN_MASK) > + dev_dbg(chip->dev, "Fault log entry detected: NSHUTDOWN\n"); > + if (fault_log & DA9062AA_WAIT_SHUT_MASK) > + dev_dbg(chip->dev, "Fault log entry detected: WAIT_SHUT\n"); > + } > + > + ret = regmap_write(chip->regmap, DA9062AA_FAULT_LOG, > + fault_log); > + if (ret < 0) > + dev_err(chip->dev, > + "Cannot reset FAULT_LOG values %d\n", ret); > + } > + > + return ret; > +} > + > +int da9062_device_init(struct da9062 *chip, unsigned int irq) No need to pass irq, as it's part of chip. > +{ > + int device_id, variant_id, variant_mrc; > + int irq_vdd_warn, ret; Put ret on its own line. > + ret = da9062_clear_fault_log(chip); > + if (ret < 0) > + dev_err(chip->dev, "Cannot clear fault log\n"); If this is an error, you must return. If it's just a warning then use dev_warn(). > + chip->irq_base = -1; Why are you pre-initialising? > + chip->chip_irq = irq; You already assigned irq to chip_irq. > + ret = regmap_read(chip->regmap, DA9062AA_DEVICE_ID, &device_id); > + if (ret < 0) { > + dev_err(chip->dev, "Cannot read chip ID.\n"); > + return -EIO; > + } > + if (device_id != DA9062_PMIC_DEVICE_ID) { > + dev_err(chip->dev, "Invalid device ID: 0x%02x\n", device_id); > + return -ENODEV; > + } > + > + ret = regmap_read(chip->regmap, DA9062AA_VARIANT_ID, &variant_id); > + if (ret < 0) { > + dev_err(chip->dev, "Cannot read chip variant id.\n"); > + return -EIO; > + } > + > + dev_info(chip->dev, > + "Device detected (device-ID: 0x%02X, var-ID: 0x%02X)\n", > + device_id, variant_id); Probably best to put this at the end. > + variant_mrc = (variant_id & DA9062AA_MRC_MASK) >> DA9062AA_MRC_SHIFT; > + > + if (variant_mrc < DA9062_PMIC_VARIANT_MRC_AA) { > + dev_err(chip->dev, > + "Cannot support variant MRC: 0x%02X\n", variant_mrc); > + return -ENODEV; > + } > + > + chip->device_id = device_id; > + chip->variant_mrc = variant_mrc; I'd put all of the device/variant stuff it its own function, then move everything else into probe() and remove da9062_device_init(). > + ret = da9062_irq_init(chip); > + if (ret) { > + dev_err(chip->dev, "Cannot initialize interrupts.\n"); > + return ret; > + } Remove this function and call regmap_add_irq_chip() from here. > + chip->irq_base = regmap_irq_chip_get_base(chip->regmap_irq); > + > + ret = mfd_add_devices(chip->dev, -1, da9062_devs, Use PLATFORM_DEVID_NONE instead. > + ARRAY_SIZE(da9062_devs), NULL, chip->irq_base, > + NULL); Usually child devices are probed at the end i.e. the last thing in probe(). > + if (ret) { > + dev_err(chip->dev, "Cannot add MFD cells\n"); "Cannot register child devices". > + da9062_irq_exit(chip); > + return ret; > + } > + > + /* VDD WARN event support */ > + irq_vdd_warn = regmap_irq_get_virq(chip->regmap_irq, > + DA9062_IRQ_VDD_WARN); > + if (irq_vdd_warn < 0) { > + dev_err(chip->dev, "Failed to get IRQ.\n"); > + return irq_vdd_warn; > + } > + chip->irq_vdd_warn = irq_vdd_warn; > + > + ret = devm_request_threaded_irq(chip->dev, irq_vdd_warn, > + NULL, da9062_vdd_warn_event, > + IRQF_TRIGGER_LOW | IRQF_ONESHOT, > + "VDD_WARN", chip); > + if (ret) { > + dev_warn(chip->dev, > + "Failed to request VDD_WARN IRQ.\n"); > + chip->irq_vdd_warn = -ENXIO; > + } This looks like a lot of code, which doesn't really do anything. What is a VDD warning indicator anyway? > + return ret; > +} > + > +void da9062_device_exit(struct da9062 *chip) > +{ > + mfd_remove_devices(chip->dev); > + da9062_irq_exit(chip); > +} Another seemingly pointless abstraction. Why don't you do this in remove()? > +/* I2C device driver for DA9062 */ Remove these. > +static const struct regmap_range da9062_aa_readable_ranges[] = { > + { > + .range_min = DA9062AA_PAGE_CON, > + .range_max = DA9062AA_STATUS_B, > + }, { > + .range_min = DA9062AA_STATUS_D, > + .range_max = DA9062AA_EVENT_C, > + }, { > + .range_min = DA9062AA_IRQ_MASK_A, > + .range_max = DA9062AA_IRQ_MASK_C, > + }, { > + .range_min = DA9062AA_CONTROL_A, > + .range_max = DA9062AA_GPIO_4, > + }, { > + .range_min = DA9062AA_GPIO_WKUP_MODE, > + .range_max = DA9062AA_BUCK4_CONT, > + }, { > + .range_min = DA9062AA_BUCK3_CONT, > + .range_max = DA9062AA_BUCK3_CONT, > + }, { > + .range_min = DA9062AA_LDO1_CONT, > + .range_max = DA9062AA_LDO4_CONT, > + }, { > + .range_min = DA9062AA_DVC_1, > + .range_max = DA9062AA_DVC_1, > + }, { > + .range_min = DA9062AA_COUNT_S, > + .range_max = DA9062AA_SECOND_D, > + }, { > + .range_min = DA9062AA_SEQ, > + .range_max = DA9062AA_ID_4_3, > + }, { > + .range_min = DA9062AA_ID_12_11, > + .range_max = DA9062AA_ID_16_15, > + }, { > + .range_min = DA9062AA_ID_22_21, > + .range_max = DA9062AA_ID_32_31, > + }, { > + .range_min = DA9062AA_SEQ_A, > + .range_max = DA9062AA_BUCK3_CFG, > + }, { > + .range_min = DA9062AA_VBUCK2_A, > + .range_max = DA9062AA_VBUCK4_A, > + }, { > + .range_min = DA9062AA_VBUCK3_A, > + .range_max = DA9062AA_VBUCK3_A, > + }, { > + .range_min = DA9062AA_VLDO1_A, > + .range_max = DA9062AA_VLDO4_A, > + }, { > + .range_min = DA9062AA_VBUCK2_B, > + .range_max = DA9062AA_VBUCK4_B, > + }, { > + .range_min = DA9062AA_VBUCK3_B, > + .range_max = DA9062AA_VBUCK3_B, > + }, { > + .range_min = DA9062AA_VLDO1_B, > + .range_max = DA9062AA_VLDO4_B, > + }, { > + .range_min = DA9062AA_BBAT_CONT, > + .range_max = DA9062AA_BBAT_CONT, > + }, { > + .range_min = DA9062AA_INTERFACE, > + .range_max = DA9062AA_CONFIG_E, > + }, { > + .range_min = DA9062AA_CONFIG_G, > + .range_max = DA9062AA_CONFIG_K, > + }, { > + .range_min = DA9062AA_CONFIG_M, > + .range_max = DA9062AA_CONFIG_M, > + }, { > + .range_min = DA9062AA_TRIM_CLDR, > + .range_max = DA9062AA_GP_ID_19, > + }, { > + .range_min = DA9062AA_DEVICE_ID, > + .range_max = DA9062AA_CONFIG_ID, > + }, > +}; > + > +static const struct regmap_range da9062_aa_writeable_ranges[] = { > + { > + .range_min = DA9062AA_PAGE_CON, > + .range_max = DA9062AA_PAGE_CON, > + }, { > + .range_min = DA9062AA_FAULT_LOG, > + .range_max = DA9062AA_EVENT_C, > + }, { > + .range_min = DA9062AA_IRQ_MASK_A, > + .range_max = DA9062AA_IRQ_MASK_C, > + }, { > + .range_min = DA9062AA_CONTROL_A, > + .range_max = DA9062AA_GPIO_4, > + }, { > + .range_min = DA9062AA_GPIO_WKUP_MODE, > + .range_max = DA9062AA_BUCK4_CONT, > + }, { > + .range_min = DA9062AA_BUCK3_CONT, > + .range_max = DA9062AA_BUCK3_CONT, > + }, { > + .range_min = DA9062AA_LDO1_CONT, > + .range_max = DA9062AA_LDO4_CONT, > + }, { > + .range_min = DA9062AA_DVC_1, > + .range_max = DA9062AA_DVC_1, > + }, { > + .range_min = DA9062AA_COUNT_S, > + .range_max = DA9062AA_ALARM_Y, > + }, { > + .range_min = DA9062AA_SEQ, > + .range_max = DA9062AA_ID_4_3, > + }, { > + .range_min = DA9062AA_ID_12_11, > + .range_max = DA9062AA_ID_16_15, > + }, { > + .range_min = DA9062AA_ID_22_21, > + .range_max = DA9062AA_ID_32_31, > + }, { > + .range_min = DA9062AA_SEQ_A, > + .range_max = DA9062AA_BUCK3_CFG, > + }, { > + .range_min = DA9062AA_VBUCK2_A, > + .range_max = DA9062AA_VBUCK4_A, > + }, { > + .range_min = DA9062AA_VBUCK3_A, > + .range_max = DA9062AA_VBUCK3_A, > + }, { > + .range_min = DA9062AA_VLDO1_A, > + .range_max = DA9062AA_VLDO4_A, > + }, { > + .range_min = DA9062AA_VBUCK2_B, > + .range_max = DA9062AA_VBUCK4_B, > + }, { > + .range_min = DA9062AA_VBUCK3_B, > + .range_max = DA9062AA_VBUCK3_B, > + }, { > + .range_min = DA9062AA_VLDO1_B, > + .range_max = DA9062AA_VLDO4_B, > + }, { > + .range_min = DA9062AA_BBAT_CONT, > + .range_max = DA9062AA_BBAT_CONT, > + }, { > + .range_min = DA9062AA_GP_ID_0, > + .range_max = DA9062AA_GP_ID_19, > + }, > +}; > + > +static const struct regmap_range da9062_aa_volatile_ranges[] = { > + { > + .range_min = DA9062AA_PAGE_CON, > + .range_max = DA9062AA_STATUS_B, > + }, { > + .range_min = DA9062AA_STATUS_D, > + .range_max = DA9062AA_EVENT_C, > + }, { > + .range_min = DA9062AA_CONTROL_F, > + .range_max = DA9062AA_CONTROL_F, > + }, { > + .range_min = DA9062AA_COUNT_S, > + .range_max = DA9062AA_SECOND_D, > + }, > +}; > + > +static const struct regmap_access_table da9062_aa_readable_table = { > + .yes_ranges = da9062_aa_readable_ranges, > + .n_yes_ranges = ARRAY_SIZE(da9062_aa_readable_ranges), > +}; > + > +static const struct regmap_access_table da9062_aa_writeable_table = { > + .yes_ranges = da9062_aa_writeable_ranges, > + .n_yes_ranges = ARRAY_SIZE(da9062_aa_writeable_ranges), > +}; > + > +static const struct regmap_access_table da9062_aa_volatile_table = { > + .yes_ranges = da9062_aa_volatile_ranges, > + .n_yes_ranges = ARRAY_SIZE(da9062_aa_volatile_ranges), > +}; > + > +static const struct regmap_range_cfg da9062_range_cfg[] = { > + { > + .range_min = DA9062AA_PAGE_CON, > + .range_max = DA9062AA_CONFIG_ID, > + .selector_reg = DA9062AA_PAGE_CON, > + .selector_mask = 1 << DA9062_I2C_PAGE_SEL_SHIFT, > + .selector_shift = DA9062_I2C_PAGE_SEL_SHIFT, > + .window_start = 0, > + .window_len = 256, > + } > +}; > + > +static struct regmap_config da9062_regmap_config = { > + .reg_bits = 8, > + .val_bits = 8, > + .ranges = da9062_range_cfg, > + .num_ranges = ARRAY_SIZE(da9062_range_cfg), > + .max_register = DA9062AA_CONFIG_ID, > + .cache_type = REGCACHE_RBTREE, > +}; > + > +static const struct of_device_id da9062_dt_ids[] = { > + { .compatible = "dlg,da9062", }, > + { } > +}; > +MODULE_DEVICE_TABLE(of, da9062_dt_ids); Move this just above where it's to be used i.e. down to the bottom. > +static int da9062_i2c_probe(struct i2c_client *i2c, > + const struct i2c_device_id *id) > +{ > + struct da9062 *chip; > + int ret; > + > + chip = devm_kzalloc(&i2c->dev, sizeof(struct da9062), GFP_KERNEL); sizeof(*chip) > + if (chip == NULL) if (!chip) > + return -ENOMEM; > + > + i2c_set_clientdata(i2c, chip); > + chip->dev = &i2c->dev; > + chip->chip_irq = i2c->irq; Check this once here, then you don't have to worry about it again. s/chip_irq/irq/ > + da9062_regmap_config.rd_table = &da9062_aa_readable_table; > + da9062_regmap_config.wr_table = &da9062_aa_writeable_table; > + da9062_regmap_config.volatile_table = &da9062_aa_volatile_table; Why are you doing this here instead of inside 'struct regmap_config da9062_regmap_config' above? > + chip->regmap = devm_regmap_init_i2c(i2c, &da9062_regmap_config); > + if (IS_ERR(chip->regmap)) { > + ret = PTR_ERR(chip->regmap); > + dev_err(chip->dev, "Failed to allocate register map: %d\n", > + ret); > + return ret; > + } > + > + return da9062_device_init(chip, i2c->irq); > +} > + > +static int da9062_i2c_remove(struct i2c_client *i2c) > +{ > + struct da9062 *chip = i2c_get_clientdata(i2c); > + > + da9062_device_exit(chip); Remove this. > + return 0; > +} > + > +static const struct i2c_device_id da9062_i2c_id[] = { > + {"da9062", 0}, White space discipline. > + {}, > +}; > +MODULE_DEVICE_TABLE(i2c, da9062_i2c_id); > + > +static struct i2c_driver da9062_i2c_driver = { > + .driver = { > + .name = "da9062", > + .of_match_table = of_match_ptr(da9062_dt_ids), > + }, > + .probe = da9062_i2c_probe, > + .remove = da9062_i2c_remove, > + .id_table = da9062_i2c_id, > +}; > + > +module_i2c_driver(da9062_i2c_driver); > + > + Superfluous '\n'. > +MODULE_DESCRIPTION("CORE device driver for Dialog DA9062"); s/CORE/Core/ > +MODULE_AUTHOR("S Twiss <stwiss.opensource@xxxxxxxxxxx>"); Full names please. > +MODULE_LICENSE("GPL"); > diff --git a/include/linux/mfd/da9062/core.h b/include/linux/mfd/da9062/core.h > new file mode 100644 > index 0000000..0b17891 > --- /dev/null > +++ b/include/linux/mfd/da9062/core.h > @@ -0,0 +1,62 @@ > +/* > + * core.h - CORE H for DA9062 What does this add? > + * Copyright (C) 2015 Dialog Semiconductor Ltd. > + * > + * This program is free software; you can redistribute it and/or > + * modify it under the terms of the GNU General Public License > + * as published by the Free Software Foundation; either version 2 > + * of the License, or (at your option) any later version. > + * > + * This program is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > + * GNU General Public License for more details. > + */ > + > +#ifndef __MFD_DA9062_CORE_H__ > +#define __MFD_DA9062_CORE_H__ > + > +#include <linux/interrupt.h> > +#include <linux/mfd/da9062/registers.h> > + > +/* Interrupts */ > +enum da9062_irqs { > + /* IRQ A */ > + DA9062_IRQ_ONKEY, > + DA9062_IRQ_ALARM, > + DA9062_IRQ_TICK, > + DA9062_IRQ_WDG_WARN, > + DA9062_IRQ_SEQ_RDY, > + /* IRQ B*/ > + DA9062_IRQ_TEMP, > + DA9062_IRQ_LDO_LIM, > + DA9062_IRQ_DVC_RDY, > + DA9062_IRQ_VDD_WARN, > + /* IRQ C */ > + DA9062_IRQ_GPI0, > + DA9062_IRQ_GPI1, > + DA9062_IRQ_GPI2, > + DA9062_IRQ_GPI3, > + DA9062_IRQ_GPI4, > + > + DA9062_NUM_IRQ, > +}; > + > +struct da9062 { > + struct device *dev; > + unsigned char device_id; > + unsigned char variant_mrc; > + struct regmap *regmap; > + int chip_irq; > + unsigned int irq_base; > + struct regmap_irq_chip_data *regmap_irq; > + int irq_vdd_warn; > +}; Are all of these used by child devices? > +int da9062_device_init(struct da9062 *, unsigned int); > +int da9062_irq_init(struct da9062 *); > + > +void da9062_device_exit(struct da9062 *); > +void da9062_irq_exit(struct da9062 *); Why are these required? > +#endif /* __MFD_DA9062_CORE_H__ */ > diff --git a/include/linux/mfd/da9062/registers.h b/include/linux/mfd/da9062/registers.h > new file mode 100644 > index 0000000..d07c2bc > --- /dev/null > +++ b/include/linux/mfd/da9062/registers.h > @@ -0,0 +1,1108 @@ > +/* > + * registers.h - REGISTERS H for DA9062 > + * Copyright (C) 2015 Dialog Semiconductor Ltd. > + * > + * This program is free software; you can redistribute it and/or > + * modify it under the terms of the GNU General Public License > + * as published by the Free Software Foundation; either version 2 > + * of the License, or (at your option) any later version. > + * > + * This program is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > + * GNU General Public License for more details. > + */ > + > +#ifndef __DA9062_H__ > +#define __DA9062_H__ > + > +#define DA9062_PMIC_DEVICE_ID 0x62 > +#define DA9062_PMIC_VARIANT_MRC_AA 0x01 > + > +#define DA9062_I2C_PAGE_SEL_SHIFT 1 > + > +/* > + * Registers > + */ Really? ;) > +#define DA9062AA_PAGE_CON 0x000 > +#define DA9062AA_STATUS_A 0x001 > +#define DA9062AA_STATUS_B 0x002 > +#define DA9062AA_STATUS_D 0x004 > +#define DA9062AA_FAULT_LOG 0x005 > +#define DA9062AA_EVENT_A 0x006 > +#define DA9062AA_EVENT_B 0x007 > +#define DA9062AA_EVENT_C 0x008 > +#define DA9062AA_IRQ_MASK_A 0x00A > +#define DA9062AA_IRQ_MASK_B 0x00B > +#define DA9062AA_IRQ_MASK_C 0x00C > +#define DA9062AA_CONTROL_A 0x00E > +#define DA9062AA_CONTROL_B 0x00F > +#define DA9062AA_CONTROL_C 0x010 > +#define DA9062AA_CONTROL_D 0x011 > +#define DA9062AA_CONTROL_E 0x012 > +#define DA9062AA_CONTROL_F 0x013 > +#define DA9062AA_PD_DIS 0x014 > +#define DA9062AA_GPIO_0_1 0x015 > +#define DA9062AA_GPIO_2_3 0x016 > +#define DA9062AA_GPIO_4 0x017 > +#define DA9062AA_GPIO_WKUP_MODE 0x01C > +#define DA9062AA_GPIO_MODE0_4 0x01D > +#define DA9062AA_GPIO_OUT0_2 0x01E > +#define DA9062AA_GPIO_OUT3_4 0x01F > +#define DA9062AA_BUCK2_CONT 0x020 > +#define DA9062AA_BUCK1_CONT 0x021 > +#define DA9062AA_BUCK4_CONT 0x022 > +#define DA9062AA_BUCK3_CONT 0x024 > +#define DA9062AA_LDO1_CONT 0x026 > +#define DA9062AA_LDO2_CONT 0x027 > +#define DA9062AA_LDO3_CONT 0x028 > +#define DA9062AA_LDO4_CONT 0x029 > +#define DA9062AA_DVC_1 0x032 > +#define DA9062AA_COUNT_S 0x040 > +#define DA9062AA_COUNT_MI 0x041 > +#define DA9062AA_COUNT_H 0x042 > +#define DA9062AA_COUNT_D 0x043 > +#define DA9062AA_COUNT_MO 0x044 > +#define DA9062AA_COUNT_Y 0x045 > +#define DA9062AA_ALARM_S 0x046 > +#define DA9062AA_ALARM_MI 0x047 > +#define DA9062AA_ALARM_H 0x048 > +#define DA9062AA_ALARM_D 0x049 > +#define DA9062AA_ALARM_MO 0x04A > +#define DA9062AA_ALARM_Y 0x04B > +#define DA9062AA_SECOND_A 0x04C > +#define DA9062AA_SECOND_B 0x04D > +#define DA9062AA_SECOND_C 0x04E > +#define DA9062AA_SECOND_D 0x04F > +#define DA9062AA_SEQ 0x081 > +#define DA9062AA_SEQ_TIMER 0x082 > +#define DA9062AA_ID_2_1 0x083 > +#define DA9062AA_ID_4_3 0x084 > +#define DA9062AA_ID_12_11 0x088 > +#define DA9062AA_ID_14_13 0x089 > +#define DA9062AA_ID_16_15 0x08A > +#define DA9062AA_ID_22_21 0x08D > +#define DA9062AA_ID_24_23 0x08E > +#define DA9062AA_ID_26_25 0x08F > +#define DA9062AA_ID_28_27 0x090 > +#define DA9062AA_ID_30_29 0x091 > +#define DA9062AA_ID_32_31 0x092 > +#define DA9062AA_SEQ_A 0x095 > +#define DA9062AA_SEQ_B 0x096 > +#define DA9062AA_WAIT 0x097 > +#define DA9062AA_EN_32K 0x098 > +#define DA9062AA_RESET 0x099 > +#define DA9062AA_BUCK_ILIM_A 0x09A > +#define DA9062AA_BUCK_ILIM_B 0x09B > +#define DA9062AA_BUCK_ILIM_C 0x09C > +#define DA9062AA_BUCK2_CFG 0x09D > +#define DA9062AA_BUCK1_CFG 0x09E > +#define DA9062AA_BUCK4_CFG 0x09F > +#define DA9062AA_BUCK3_CFG 0x0A0 > +#define DA9062AA_VBUCK2_A 0x0A3 > +#define DA9062AA_VBUCK1_A 0x0A4 > +#define DA9062AA_VBUCK4_A 0x0A5 > +#define DA9062AA_VBUCK3_A 0x0A7 > +#define DA9062AA_VLDO1_A 0x0A9 > +#define DA9062AA_VLDO2_A 0x0AA > +#define DA9062AA_VLDO3_A 0x0AB > +#define DA9062AA_VLDO4_A 0x0AC > +#define DA9062AA_VBUCK2_B 0x0B4 > +#define DA9062AA_VBUCK1_B 0x0B5 > +#define DA9062AA_VBUCK4_B 0x0B6 > +#define DA9062AA_VBUCK3_B 0x0B8 > +#define DA9062AA_VLDO1_B 0x0BA > +#define DA9062AA_VLDO2_B 0x0BB > +#define DA9062AA_VLDO3_B 0x0BC > +#define DA9062AA_VLDO4_B 0x0BD > +#define DA9062AA_BBAT_CONT 0x0C5 > +#define DA9062AA_INTERFACE 0x105 > +#define DA9062AA_CONFIG_A 0x106 > +#define DA9062AA_CONFIG_B 0x107 > +#define DA9062AA_CONFIG_C 0x108 > +#define DA9062AA_CONFIG_D 0x109 > +#define DA9062AA_CONFIG_E 0x10A > +#define DA9062AA_CONFIG_G 0x10C > +#define DA9062AA_CONFIG_H 0x10D > +#define DA9062AA_CONFIG_I 0x10E > +#define DA9062AA_CONFIG_J 0x10F > +#define DA9062AA_CONFIG_K 0x110 > +#define DA9062AA_CONFIG_M 0x112 > +#define DA9062AA_TRIM_CLDR 0x120 > +#define DA9062AA_GP_ID_0 0x121 > +#define DA9062AA_GP_ID_1 0x122 > +#define DA9062AA_GP_ID_2 0x123 > +#define DA9062AA_GP_ID_3 0x124 > +#define DA9062AA_GP_ID_4 0x125 > +#define DA9062AA_GP_ID_5 0x126 > +#define DA9062AA_GP_ID_6 0x127 > +#define DA9062AA_GP_ID_7 0x128 > +#define DA9062AA_GP_ID_8 0x129 > +#define DA9062AA_GP_ID_9 0x12A > +#define DA9062AA_GP_ID_10 0x12B > +#define DA9062AA_GP_ID_11 0x12C > +#define DA9062AA_GP_ID_12 0x12D > +#define DA9062AA_GP_ID_13 0x12E > +#define DA9062AA_GP_ID_14 0x12F > +#define DA9062AA_GP_ID_15 0x130 > +#define DA9062AA_GP_ID_16 0x131 > +#define DA9062AA_GP_ID_17 0x132 > +#define DA9062AA_GP_ID_18 0x133 > +#define DA9062AA_GP_ID_19 0x134 > +#define DA9062AA_DEVICE_ID 0x181 > +#define DA9062AA_VARIANT_ID 0x182 > +#define DA9062AA_CUSTOMER_ID 0x183 > +#define DA9062AA_CONFIG_ID 0x184 > + > +/* > + * Bit fields > + */ > + > +/* DA9062AA_PAGE_CON = 0x000 */ > +#define DA9062AA_PAGE_SHIFT 0 > +#define DA9062AA_PAGE_MASK (0x3f << 0) > +#define DA9062AA_WRITE_MODE_SHIFT 6 > +#define DA9062AA_WRITE_MODE_MASK (0x01 << 6) For 1 << X, you should use BIT(X). > +#define DA9062AA_REVERT_SHIFT 7 > +#define DA9062AA_REVERT_MASK (0x01 << 7) > + > +/* DA9062AA_STATUS_A = 0x001 */ > +#define DA9062AA_NONKEY_SHIFT 0 > +#define DA9062AA_NONKEY_MASK (0x01 << 0) > +#define DA9062AA_DVC_BUSY_SHIFT 2 > +#define DA9062AA_DVC_BUSY_MASK (0x01 << 2) > + > +/* DA9062AA_STATUS_B = 0x002 */ > +#define DA9062AA_GPI0_SHIFT 0 > +#define DA9062AA_GPI0_MASK (0x01 << 0) > +#define DA9062AA_GPI1_SHIFT 1 > +#define DA9062AA_GPI1_MASK (0x01 << 1) > +#define DA9062AA_GPI2_SHIFT 2 > +#define DA9062AA_GPI2_MASK (0x01 << 2) > +#define DA9062AA_GPI3_SHIFT 3 > +#define DA9062AA_GPI3_MASK (0x01 << 3) > +#define DA9062AA_GPI4_SHIFT 4 > +#define DA9062AA_GPI4_MASK (0x01 << 4) > + > +/* DA9062AA_STATUS_D = 0x004 */ > +#define DA9062AA_LDO1_ILIM_SHIFT 0 > +#define DA9062AA_LDO1_ILIM_MASK (0x01 << 0) > +#define DA9062AA_LDO2_ILIM_SHIFT 1 > +#define DA9062AA_LDO2_ILIM_MASK (0x01 << 1) > +#define DA9062AA_LDO3_ILIM_SHIFT 2 > +#define DA9062AA_LDO3_ILIM_MASK (0x01 << 2) > +#define DA9062AA_LDO4_ILIM_SHIFT 3 > +#define DA9062AA_LDO4_ILIM_MASK (0x01 << 3) > + > +/* DA9062AA_FAULT_LOG = 0x005 */ > +#define DA9062AA_TWD_ERROR_SHIFT 0 > +#define DA9062AA_TWD_ERROR_MASK (0x01 << 0) > +#define DA9062AA_POR_SHIFT 1 > +#define DA9062AA_POR_MASK (0x01 << 1) > +#define DA9062AA_VDD_FAULT_SHIFT 2 > +#define DA9062AA_VDD_FAULT_MASK (0x01 << 2) > +#define DA9062AA_VDD_START_SHIFT 3 > +#define DA9062AA_VDD_START_MASK (0x01 << 3) > +#define DA9062AA_TEMP_CRIT_SHIFT 4 > +#define DA9062AA_TEMP_CRIT_MASK (0x01 << 4) > +#define DA9062AA_KEY_RESET_SHIFT 5 > +#define DA9062AA_KEY_RESET_MASK (0x01 << 5) > +#define DA9062AA_NSHUTDOWN_SHIFT 6 > +#define DA9062AA_NSHUTDOWN_MASK (0x01 << 6) > +#define DA9062AA_WAIT_SHUT_SHIFT 7 > +#define DA9062AA_WAIT_SHUT_MASK (0x01 << 7) > + > +/* DA9062AA_EVENT_A = 0x006 */ > +#define DA9062AA_E_NONKEY_SHIFT 0 > +#define DA9062AA_E_NONKEY_MASK (0x01 << 0) > +#define DA9062AA_E_ALARM_SHIFT 1 > +#define DA9062AA_E_ALARM_MASK (0x01 << 1) > +#define DA9062AA_E_TICK_SHIFT 2 > +#define DA9062AA_E_TICK_MASK (0x01 << 2) > +#define DA9062AA_E_WDG_WARN_SHIFT 3 > +#define DA9062AA_E_WDG_WARN_MASK (0x01 << 3) > +#define DA9062AA_E_SEQ_RDY_SHIFT 4 > +#define DA9062AA_E_SEQ_RDY_MASK (0x01 << 4) > +#define DA9062AA_EVENTS_B_SHIFT 5 > +#define DA9062AA_EVENTS_B_MASK (0x01 << 5) > +#define DA9062AA_EVENTS_C_SHIFT 6 > +#define DA9062AA_EVENTS_C_MASK (0x01 << 6) > + > +/* DA9062AA_EVENT_B = 0x007 */ > +#define DA9062AA_E_TEMP_SHIFT 1 > +#define DA9062AA_E_TEMP_MASK (0x01 << 1) > +#define DA9062AA_E_LDO_LIM_SHIFT 3 > +#define DA9062AA_E_LDO_LIM_MASK (0x01 << 3) > +#define DA9062AA_E_DVC_RDY_SHIFT 5 > +#define DA9062AA_E_DVC_RDY_MASK (0x01 << 5) > +#define DA9062AA_E_VDD_WARN_SHIFT 7 > +#define DA9062AA_E_VDD_WARN_MASK (0x01 << 7) > + > +/* DA9062AA_EVENT_C = 0x008 */ > +#define DA9062AA_E_GPI0_SHIFT 0 > +#define DA9062AA_E_GPI0_MASK (0x01 << 0) > +#define DA9062AA_E_GPI1_SHIFT 1 > +#define DA9062AA_E_GPI1_MASK (0x01 << 1) > +#define DA9062AA_E_GPI2_SHIFT 2 > +#define DA9062AA_E_GPI2_MASK (0x01 << 2) > +#define DA9062AA_E_GPI3_SHIFT 3 > +#define DA9062AA_E_GPI3_MASK (0x01 << 3) > +#define DA9062AA_E_GPI4_SHIFT 4 > +#define DA9062AA_E_GPI4_MASK (0x01 << 4) > + > +/* DA9062AA_IRQ_MASK_A = 0x00A */ > +#define DA9062AA_M_NONKEY_SHIFT 0 > +#define DA9062AA_M_NONKEY_MASK (0x01 << 0) > +#define DA9062AA_M_ALARM_SHIFT 1 > +#define DA9062AA_M_ALARM_MASK (0x01 << 1) > +#define DA9062AA_M_TICK_SHIFT 2 > +#define DA9062AA_M_TICK_MASK (0x01 << 2) > +#define DA9062AA_M_WDG_WARN_SHIFT 3 > +#define DA9062AA_M_WDG_WARN_MASK (0x01 << 3) > +#define DA9062AA_M_SEQ_RDY_SHIFT 4 > +#define DA9062AA_M_SEQ_RDY_MASK (0x01 << 4) > + > +/* DA9062AA_IRQ_MASK_B = 0x00B */ > +#define DA9062AA_M_TEMP_SHIFT 1 > +#define DA9062AA_M_TEMP_MASK (0x01 << 1) > +#define DA9062AA_M_LDO_LIM_SHIFT 3 > +#define DA9062AA_M_LDO_LIM_MASK (0x01 << 3) > +#define DA9062AA_M_DVC_RDY_SHIFT 5 > +#define DA9062AA_M_DVC_RDY_MASK (0x01 << 5) > +#define DA9062AA_M_VDD_WARN_SHIFT 7 > +#define DA9062AA_M_VDD_WARN_MASK (0x01 << 7) > + > +/* DA9062AA_IRQ_MASK_C = 0x00C */ > +#define DA9062AA_M_GPI0_SHIFT 0 > +#define DA9062AA_M_GPI0_MASK (0x01 << 0) > +#define DA9062AA_M_GPI1_SHIFT 1 > +#define DA9062AA_M_GPI1_MASK (0x01 << 1) > +#define DA9062AA_M_GPI2_SHIFT 2 > +#define DA9062AA_M_GPI2_MASK (0x01 << 2) > +#define DA9062AA_M_GPI3_SHIFT 3 > +#define DA9062AA_M_GPI3_MASK (0x01 << 3) > +#define DA9062AA_M_GPI4_SHIFT 4 > +#define DA9062AA_M_GPI4_MASK (0x01 << 4) > + > +/* DA9062AA_CONTROL_A = 0x00E */ > +#define DA9062AA_SYSTEM_EN_SHIFT 0 > +#define DA9062AA_SYSTEM_EN_MASK (0x01 << 0) > +#define DA9062AA_POWER_EN_SHIFT 1 > +#define DA9062AA_POWER_EN_MASK (0x01 << 1) > +#define DA9062AA_POWER1_EN_SHIFT 2 > +#define DA9062AA_POWER1_EN_MASK (0x01 << 2) > +#define DA9062AA_STANDBY_SHIFT 3 > +#define DA9062AA_STANDBY_MASK (0x01 << 3) > +#define DA9062AA_M_SYSTEM_EN_SHIFT 4 > +#define DA9062AA_M_SYSTEM_EN_MASK (0x01 << 4) > +#define DA9062AA_M_POWER_EN_SHIFT 5 > +#define DA9062AA_M_POWER_EN_MASK (0x01 << 5) > +#define DA9062AA_M_POWER1_EN_SHIFT 6 > +#define DA9062AA_M_POWER1_EN_MASK (0x01 << 6) > + > +/* DA9062AA_CONTROL_B = 0x00F */ > +#define DA9062AA_WATCHDOG_PD_SHIFT 1 > +#define DA9062AA_WATCHDOG_PD_MASK (0x01 << 1) > +#define DA9062AA_FREEZE_EN_SHIFT 2 > +#define DA9062AA_FREEZE_EN_MASK (0x01 << 2) > +#define DA9062AA_NRES_MODE_SHIFT 3 > +#define DA9062AA_NRES_MODE_MASK (0x01 << 3) > +#define DA9062AA_NONKEY_LOCK_SHIFT 4 > +#define DA9062AA_NONKEY_LOCK_MASK (0x01 << 4) > +#define DA9062AA_NFREEZE_SHIFT 5 > +#define DA9062AA_NFREEZE_MASK (0x03 << 5) > +#define DA9062AA_BUCK_SLOWSTART_SHIFT 7 > +#define DA9062AA_BUCK_SLOWSTART_MASK (0x01 << 7) > + > +/* DA9062AA_CONTROL_C = 0x010 */ > +#define DA9062AA_DEBOUNCING_SHIFT 0 > +#define DA9062AA_DEBOUNCING_MASK (0x07 << 0) > +#define DA9062AA_AUTO_BOOT_SHIFT 3 > +#define DA9062AA_AUTO_BOOT_MASK (0x01 << 3) > +#define DA9062AA_OTPREAD_EN_SHIFT 4 > +#define DA9062AA_OTPREAD_EN_MASK (0x01 << 4) > +#define DA9062AA_SLEW_RATE_SHIFT 5 > +#define DA9062AA_SLEW_RATE_MASK (0x03 << 5) > +#define DA9062AA_DEF_SUPPLY_SHIFT 7 > +#define DA9062AA_DEF_SUPPLY_MASK (0x01 << 7) > + > +/* DA9062AA_CONTROL_D = 0x011 */ > +#define DA9062AA_TWDSCALE_SHIFT 0 > +#define DA9062AA_TWDSCALE_MASK (0x07 << 0) > + > +/* DA9062AA_CONTROL_E = 0x012 */ > +#define DA9062AA_RTC_MODE_PD_SHIFT 0 > +#define DA9062AA_RTC_MODE_PD_MASK (0x01 << 0) > +#define DA9062AA_RTC_MODE_SD_SHIFT 1 > +#define DA9062AA_RTC_MODE_SD_MASK (0x01 << 1) > +#define DA9062AA_RTC_EN_SHIFT 2 > +#define DA9062AA_RTC_EN_MASK (0x01 << 2) > +#define DA9062AA_V_LOCK_SHIFT 7 > +#define DA9062AA_V_LOCK_MASK (0x01 << 7) > + > +/* DA9062AA_CONTROL_F = 0x013 */ > +#define DA9062AA_WATCHDOG_SHIFT 0 > +#define DA9062AA_WATCHDOG_MASK (0x01 << 0) > +#define DA9062AA_SHUTDOWN_SHIFT 1 > +#define DA9062AA_SHUTDOWN_MASK (0x01 << 1) > +#define DA9062AA_WAKE_UP_SHIFT 2 > +#define DA9062AA_WAKE_UP_MASK (0x01 << 2) > + > +/* DA9062AA_PD_DIS = 0x014 */ > +#define DA9062AA_GPI_DIS_SHIFT 0 > +#define DA9062AA_GPI_DIS_MASK (0x01 << 0) > +#define DA9062AA_PMIF_DIS_SHIFT 2 > +#define DA9062AA_PMIF_DIS_MASK (0x01 << 2) > +#define DA9062AA_CLDR_PAUSE_SHIFT 4 > +#define DA9062AA_CLDR_PAUSE_MASK (0x01 << 4) > +#define DA9062AA_BBAT_DIS_SHIFT 5 > +#define DA9062AA_BBAT_DIS_MASK (0x01 << 5) > +#define DA9062AA_OUT32K_PAUSE_SHIFT 6 > +#define DA9062AA_OUT32K_PAUSE_MASK (0x01 << 6) > +#define DA9062AA_PMCONT_DIS_SHIFT 7 > +#define DA9062AA_PMCONT_DIS_MASK (0x01 << 7) > + > +/* DA9062AA_GPIO_0_1 = 0x015 */ > +#define DA9062AA_GPIO0_PIN_SHIFT 0 > +#define DA9062AA_GPIO0_PIN_MASK (0x03 << 0) > +#define DA9062AA_GPIO0_TYPE_SHIFT 2 > +#define DA9062AA_GPIO0_TYPE_MASK (0x01 << 2) > +#define DA9062AA_GPIO0_WEN_SHIFT 3 > +#define DA9062AA_GPIO0_WEN_MASK (0x01 << 3) > +#define DA9062AA_GPIO1_PIN_SHIFT 4 > +#define DA9062AA_GPIO1_PIN_MASK (0x03 << 4) > +#define DA9062AA_GPIO1_TYPE_SHIFT 6 > +#define DA9062AA_GPIO1_TYPE_MASK (0x01 << 6) > +#define DA9062AA_GPIO1_WEN_SHIFT 7 > +#define DA9062AA_GPIO1_WEN_MASK (0x01 << 7) > + > +/* DA9062AA_GPIO_2_3 = 0x016 */ > +#define DA9062AA_GPIO2_PIN_SHIFT 0 > +#define DA9062AA_GPIO2_PIN_MASK (0x03 << 0) > +#define DA9062AA_GPIO2_TYPE_SHIFT 2 > +#define DA9062AA_GPIO2_TYPE_MASK (0x01 << 2) > +#define DA9062AA_GPIO2_WEN_SHIFT 3 > +#define DA9062AA_GPIO2_WEN_MASK (0x01 << 3) > +#define DA9062AA_GPIO3_PIN_SHIFT 4 > +#define DA9062AA_GPIO3_PIN_MASK (0x03 << 4) > +#define DA9062AA_GPIO3_TYPE_SHIFT 6 > +#define DA9062AA_GPIO3_TYPE_MASK (0x01 << 6) > +#define DA9062AA_GPIO3_WEN_SHIFT 7 > +#define DA9062AA_GPIO3_WEN_MASK (0x01 << 7) > + > +/* DA9062AA_GPIO_4 = 0x017 */ > +#define DA9062AA_GPIO4_PIN_SHIFT 0 > +#define DA9062AA_GPIO4_PIN_MASK (0x03 << 0) > +#define DA9062AA_GPIO4_TYPE_SHIFT 2 > +#define DA9062AA_GPIO4_TYPE_MASK (0x01 << 2) > +#define DA9062AA_GPIO4_WEN_SHIFT 3 > +#define DA9062AA_GPIO4_WEN_MASK (0x01 << 3) > + > +/* DA9062AA_GPIO_WKUP_MODE = 0x01C */ > +#define DA9062AA_GPIO0_WKUP_MODE_SHIFT 0 > +#define DA9062AA_GPIO0_WKUP_MODE_MASK (0x01 << 0) > +#define DA9062AA_GPIO1_WKUP_MODE_SHIFT 1 > +#define DA9062AA_GPIO1_WKUP_MODE_MASK (0x01 << 1) > +#define DA9062AA_GPIO2_WKUP_MODE_SHIFT 2 > +#define DA9062AA_GPIO2_WKUP_MODE_MASK (0x01 << 2) > +#define DA9062AA_GPIO3_WKUP_MODE_SHIFT 3 > +#define DA9062AA_GPIO3_WKUP_MODE_MASK (0x01 << 3) > +#define DA9062AA_GPIO4_WKUP_MODE_SHIFT 4 > +#define DA9062AA_GPIO4_WKUP_MODE_MASK (0x01 << 4) > + > +/* DA9062AA_GPIO_MODE0_4 = 0x01D */ > +#define DA9062AA_GPIO0_MODE_SHIFT 0 > +#define DA9062AA_GPIO0_MODE_MASK (0x01 << 0) > +#define DA9062AA_GPIO1_MODE_SHIFT 1 > +#define DA9062AA_GPIO1_MODE_MASK (0x01 << 1) > +#define DA9062AA_GPIO2_MODE_SHIFT 2 > +#define DA9062AA_GPIO2_MODE_MASK (0x01 << 2) > +#define DA9062AA_GPIO3_MODE_SHIFT 3 > +#define DA9062AA_GPIO3_MODE_MASK (0x01 << 3) > +#define DA9062AA_GPIO4_MODE_SHIFT 4 > +#define DA9062AA_GPIO4_MODE_MASK (0x01 << 4) > + > +/* DA9062AA_GPIO_OUT0_2 = 0x01E */ > +#define DA9062AA_GPIO0_OUT_SHIFT 0 > +#define DA9062AA_GPIO0_OUT_MASK (0x07 << 0) > +#define DA9062AA_GPIO1_OUT_SHIFT 3 > +#define DA9062AA_GPIO1_OUT_MASK (0x07 << 3) > +#define DA9062AA_GPIO2_OUT_SHIFT 6 > +#define DA9062AA_GPIO2_OUT_MASK (0x03 << 6) > + > +/* DA9062AA_GPIO_OUT3_4 = 0x01F */ > +#define DA9062AA_GPIO3_OUT_SHIFT 0 > +#define DA9062AA_GPIO3_OUT_MASK (0x07 << 0) > +#define DA9062AA_GPIO4_OUT_SHIFT 3 > +#define DA9062AA_GPIO4_OUT_MASK (0x03 << 3) > + > +/* DA9062AA_BUCK2_CONT = 0x020 */ > +#define DA9062AA_BUCK2_EN_SHIFT 0 > +#define DA9062AA_BUCK2_EN_MASK (0x01 << 0) > +#define DA9062AA_BUCK2_GPI_SHIFT 1 > +#define DA9062AA_BUCK2_GPI_MASK (0x03 << 1) > +#define DA9062AA_BUCK2_CONF_SHIFT 3 > +#define DA9062AA_BUCK2_CONF_MASK (0x01 << 3) > +#define DA9062AA_VBUCK2_GPI_SHIFT 5 > +#define DA9062AA_VBUCK2_GPI_MASK (0x03 << 5) > + > +/* DA9062AA_BUCK1_CONT = 0x021 */ > +#define DA9062AA_BUCK1_EN_SHIFT 0 > +#define DA9062AA_BUCK1_EN_MASK (0x01 << 0) > +#define DA9062AA_BUCK1_GPI_SHIFT 1 > +#define DA9062AA_BUCK1_GPI_MASK (0x03 << 1) > +#define DA9062AA_BUCK1_CONF_SHIFT 3 > +#define DA9062AA_BUCK1_CONF_MASK (0x01 << 3) > +#define DA9062AA_VBUCK1_GPI_SHIFT 5 > +#define DA9062AA_VBUCK1_GPI_MASK (0x03 << 5) > + > +/* DA9062AA_BUCK4_CONT = 0x022 */ > +#define DA9062AA_BUCK4_EN_SHIFT 0 > +#define DA9062AA_BUCK4_EN_MASK (0x01 << 0) > +#define DA9062AA_BUCK4_GPI_SHIFT 1 > +#define DA9062AA_BUCK4_GPI_MASK (0x03 << 1) > +#define DA9062AA_BUCK4_CONF_SHIFT 3 > +#define DA9062AA_BUCK4_CONF_MASK (0x01 << 3) > +#define DA9062AA_VBUCK4_GPI_SHIFT 5 > +#define DA9062AA_VBUCK4_GPI_MASK (0x03 << 5) > + > +/* DA9062AA_BUCK3_CONT = 0x024 */ > +#define DA9062AA_BUCK3_EN_SHIFT 0 > +#define DA9062AA_BUCK3_EN_MASK (0x01 << 0) > +#define DA9062AA_BUCK3_GPI_SHIFT 1 > +#define DA9062AA_BUCK3_GPI_MASK (0x03 << 1) > +#define DA9062AA_BUCK3_CONF_SHIFT 3 > +#define DA9062AA_BUCK3_CONF_MASK (0x01 << 3) > +#define DA9062AA_VBUCK3_GPI_SHIFT 5 > +#define DA9062AA_VBUCK3_GPI_MASK (0x03 << 5) > + > +/* DA9062AA_LDO1_CONT = 0x026 */ > +#define DA9062AA_LDO1_EN_SHIFT 0 > +#define DA9062AA_LDO1_EN_MASK (0x01 << 0) > +#define DA9062AA_LDO1_GPI_SHIFT 1 > +#define DA9062AA_LDO1_GPI_MASK (0x03 << 1) > +#define DA9062AA_LDO1_PD_DIS_SHIFT 3 > +#define DA9062AA_LDO1_PD_DIS_MASK (0x01 << 3) > +#define DA9062AA_VLDO1_GPI_SHIFT 5 > +#define DA9062AA_VLDO1_GPI_MASK (0x03 << 5) > +#define DA9062AA_LDO1_CONF_SHIFT 7 > +#define DA9062AA_LDO1_CONF_MASK (0x01 << 7) > + > +/* DA9062AA_LDO2_CONT = 0x027 */ > +#define DA9062AA_LDO2_EN_SHIFT 0 > +#define DA9062AA_LDO2_EN_MASK (0x01 << 0) > +#define DA9062AA_LDO2_GPI_SHIFT 1 > +#define DA9062AA_LDO2_GPI_MASK (0x03 << 1) > +#define DA9062AA_LDO2_PD_DIS_SHIFT 3 > +#define DA9062AA_LDO2_PD_DIS_MASK (0x01 << 3) > +#define DA9062AA_VLDO2_GPI_SHIFT 5 > +#define DA9062AA_VLDO2_GPI_MASK (0x03 << 5) > +#define DA9062AA_LDO2_CONF_SHIFT 7 > +#define DA9062AA_LDO2_CONF_MASK (0x01 << 7) > + > +/* DA9062AA_LDO3_CONT = 0x028 */ > +#define DA9062AA_LDO3_EN_SHIFT 0 > +#define DA9062AA_LDO3_EN_MASK (0x01 << 0) > +#define DA9062AA_LDO3_GPI_SHIFT 1 > +#define DA9062AA_LDO3_GPI_MASK (0x03 << 1) > +#define DA9062AA_LDO3_PD_DIS_SHIFT 3 > +#define DA9062AA_LDO3_PD_DIS_MASK (0x01 << 3) > +#define DA9062AA_VLDO3_GPI_SHIFT 5 > +#define DA9062AA_VLDO3_GPI_MASK (0x03 << 5) > +#define DA9062AA_LDO3_CONF_SHIFT 7 > +#define DA9062AA_LDO3_CONF_MASK (0x01 << 7) > + > +/* DA9062AA_LDO4_CONT = 0x029 */ > +#define DA9062AA_LDO4_EN_SHIFT 0 > +#define DA9062AA_LDO4_EN_MASK (0x01 << 0) > +#define DA9062AA_LDO4_GPI_SHIFT 1 > +#define DA9062AA_LDO4_GPI_MASK (0x03 << 1) > +#define DA9062AA_LDO4_PD_DIS_SHIFT 3 > +#define DA9062AA_LDO4_PD_DIS_MASK (0x01 << 3) > +#define DA9062AA_VLDO4_GPI_SHIFT 5 > +#define DA9062AA_VLDO4_GPI_MASK (0x03 << 5) > +#define DA9062AA_LDO4_CONF_SHIFT 7 > +#define DA9062AA_LDO4_CONF_MASK (0x01 << 7) > + > +/* DA9062AA_DVC_1 = 0x032 */ > +#define DA9062AA_VBUCK1_SEL_SHIFT 0 > +#define DA9062AA_VBUCK1_SEL_MASK (0x01 << 0) > +#define DA9062AA_VBUCK2_SEL_SHIFT 1 > +#define DA9062AA_VBUCK2_SEL_MASK (0x01 << 1) > +#define DA9062AA_VBUCK4_SEL_SHIFT 2 > +#define DA9062AA_VBUCK4_SEL_MASK (0x01 << 2) > +#define DA9062AA_VBUCK3_SEL_SHIFT 3 > +#define DA9062AA_VBUCK3_SEL_MASK (0x01 << 3) > +#define DA9062AA_VLDO1_SEL_SHIFT 4 > +#define DA9062AA_VLDO1_SEL_MASK (0x01 << 4) > +#define DA9062AA_VLDO2_SEL_SHIFT 5 > +#define DA9062AA_VLDO2_SEL_MASK (0x01 << 5) > +#define DA9062AA_VLDO3_SEL_SHIFT 6 > +#define DA9062AA_VLDO3_SEL_MASK (0x01 << 6) > +#define DA9062AA_VLDO4_SEL_SHIFT 7 > +#define DA9062AA_VLDO4_SEL_MASK (0x01 << 7) > + > +/* DA9062AA_COUNT_S = 0x040 */ > +#define DA9062AA_COUNT_SEC_SHIFT 0 > +#define DA9062AA_COUNT_SEC_MASK (0x3f << 0) > +#define DA9062AA_RTC_READ_SHIFT 7 > +#define DA9062AA_RTC_READ_MASK (0x01 << 7) > + > +/* DA9062AA_COUNT_MI = 0x041 */ > +#define DA9062AA_COUNT_MIN_SHIFT 0 > +#define DA9062AA_COUNT_MIN_MASK (0x3f << 0) > + > +/* DA9062AA_COUNT_H = 0x042 */ > +#define DA9062AA_COUNT_HOUR_SHIFT 0 > +#define DA9062AA_COUNT_HOUR_MASK (0x1f << 0) > + > +/* DA9062AA_COUNT_D = 0x043 */ > +#define DA9062AA_COUNT_DAY_SHIFT 0 > +#define DA9062AA_COUNT_DAY_MASK (0x1f << 0) > + > +/* DA9062AA_COUNT_MO = 0x044 */ > +#define DA9062AA_COUNT_MONTH_SHIFT 0 > +#define DA9062AA_COUNT_MONTH_MASK (0x0f << 0) > + > +/* DA9062AA_COUNT_Y = 0x045 */ > +#define DA9062AA_COUNT_YEAR_SHIFT 0 > +#define DA9062AA_COUNT_YEAR_MASK (0x3f << 0) > +#define DA9062AA_MONITOR_SHIFT 6 > +#define DA9062AA_MONITOR_MASK (0x01 << 6) > + > +/* DA9062AA_ALARM_S = 0x046 */ > +#define DA9062AA_ALARM_SEC_SHIFT 0 > +#define DA9062AA_ALARM_SEC_MASK (0x3f << 0) > +#define DA9062AA_ALARM_STATUS_SHIFT 6 > +#define DA9062AA_ALARM_STATUS_MASK (0x03 << 6) > + > +/* DA9062AA_ALARM_MI = 0x047 */ > +#define DA9062AA_ALARM_MIN_SHIFT 0 > +#define DA9062AA_ALARM_MIN_MASK (0x3f << 0) > + > +/* DA9062AA_ALARM_H = 0x048 */ > +#define DA9062AA_ALARM_HOUR_SHIFT 0 > +#define DA9062AA_ALARM_HOUR_MASK (0x1f << 0) > + > +/* DA9062AA_ALARM_D = 0x049 */ > +#define DA9062AA_ALARM_DAY_SHIFT 0 > +#define DA9062AA_ALARM_DAY_MASK (0x1f << 0) > + > +/* DA9062AA_ALARM_MO = 0x04A */ > +#define DA9062AA_ALARM_MONTH_SHIFT 0 > +#define DA9062AA_ALARM_MONTH_MASK (0x0f << 0) > +#define DA9062AA_TICK_TYPE_SHIFT 4 > +#define DA9062AA_TICK_TYPE_MASK (0x01 << 4) > +#define DA9062AA_TICK_WAKE_SHIFT 5 > +#define DA9062AA_TICK_WAKE_MASK (0x01 << 5) > + > +/* DA9062AA_ALARM_Y = 0x04B */ > +#define DA9062AA_ALARM_YEAR_SHIFT 0 > +#define DA9062AA_ALARM_YEAR_MASK (0x3f << 0) > +#define DA9062AA_ALARM_ON_SHIFT 6 > +#define DA9062AA_ALARM_ON_MASK (0x01 << 6) > +#define DA9062AA_TICK_ON_SHIFT 7 > +#define DA9062AA_TICK_ON_MASK (0x01 << 7) > + > +/* DA9062AA_SECOND_A = 0x04C */ > +#define DA9062AA_SECONDS_A_SHIFT 0 > +#define DA9062AA_SECONDS_A_MASK (0xff << 0) > + > +/* DA9062AA_SECOND_B = 0x04D */ > +#define DA9062AA_SECONDS_B_SHIFT 0 > +#define DA9062AA_SECONDS_B_MASK (0xff << 0) > + > +/* DA9062AA_SECOND_C = 0x04E */ > +#define DA9062AA_SECONDS_C_SHIFT 0 > +#define DA9062AA_SECONDS_C_MASK (0xff << 0) > + > +/* DA9062AA_SECOND_D = 0x04F */ > +#define DA9062AA_SECONDS_D_SHIFT 0 > +#define DA9062AA_SECONDS_D_MASK (0xff << 0) > + > +/* DA9062AA_SEQ = 0x081 */ > +#define DA9062AA_SEQ_POINTER_SHIFT 0 > +#define DA9062AA_SEQ_POINTER_MASK (0x0f << 0) > +#define DA9062AA_NXT_SEQ_START_SHIFT 4 > +#define DA9062AA_NXT_SEQ_START_MASK (0x0f << 4) > + > +/* DA9062AA_SEQ_TIMER = 0x082 */ > +#define DA9062AA_SEQ_TIME_SHIFT 0 > +#define DA9062AA_SEQ_TIME_MASK (0x0f << 0) > +#define DA9062AA_SEQ_DUMMY_SHIFT 4 > +#define DA9062AA_SEQ_DUMMY_MASK (0x0f << 4) > + > +/* DA9062AA_ID_2_1 = 0x083 */ > +#define DA9062AA_LDO1_STEP_SHIFT 0 > +#define DA9062AA_LDO1_STEP_MASK (0x0f << 0) > +#define DA9062AA_LDO2_STEP_SHIFT 4 > +#define DA9062AA_LDO2_STEP_MASK (0x0f << 4) > + > +/* DA9062AA_ID_4_3 = 0x084 */ > +#define DA9062AA_LDO3_STEP_SHIFT 0 > +#define DA9062AA_LDO3_STEP_MASK (0x0f << 0) > +#define DA9062AA_LDO4_STEP_SHIFT 4 > +#define DA9062AA_LDO4_STEP_MASK (0x0f << 4) > + > +/* DA9062AA_ID_12_11 = 0x088 */ > +#define DA9062AA_PD_DIS_STEP_SHIFT 4 > +#define DA9062AA_PD_DIS_STEP_MASK (0x0f << 4) > + > +/* DA9062AA_ID_14_13 = 0x089 */ > +#define DA9062AA_BUCK1_STEP_SHIFT 0 > +#define DA9062AA_BUCK1_STEP_MASK (0x0f << 0) > +#define DA9062AA_BUCK2_STEP_SHIFT 4 > +#define DA9062AA_BUCK2_STEP_MASK (0x0f << 4) > + > +/* DA9062AA_ID_16_15 = 0x08A */ > +#define DA9062AA_BUCK4_STEP_SHIFT 0 > +#define DA9062AA_BUCK4_STEP_MASK (0x0f << 0) > +#define DA9062AA_BUCK3_STEP_SHIFT 4 > +#define DA9062AA_BUCK3_STEP_MASK (0x0f << 4) > + > +/* DA9062AA_ID_22_21 = 0x08D */ > +#define DA9062AA_GP_RISE1_STEP_SHIFT 0 > +#define DA9062AA_GP_RISE1_STEP_MASK (0x0f << 0) > +#define DA9062AA_GP_FALL1_STEP_SHIFT 4 > +#define DA9062AA_GP_FALL1_STEP_MASK (0x0f << 4) > + > +/* DA9062AA_ID_24_23 = 0x08E */ > +#define DA9062AA_GP_RISE2_STEP_SHIFT 0 > +#define DA9062AA_GP_RISE2_STEP_MASK (0x0f << 0) > +#define DA9062AA_GP_FALL2_STEP_SHIFT 4 > +#define DA9062AA_GP_FALL2_STEP_MASK (0x0f << 4) > + > +/* DA9062AA_ID_26_25 = 0x08F */ > +#define DA9062AA_GP_RISE3_STEP_SHIFT 0 > +#define DA9062AA_GP_RISE3_STEP_MASK (0x0f << 0) > +#define DA9062AA_GP_FALL3_STEP_SHIFT 4 > +#define DA9062AA_GP_FALL3_STEP_MASK (0x0f << 4) > + > +/* DA9062AA_ID_28_27 = 0x090 */ > +#define DA9062AA_GP_RISE4_STEP_SHIFT 0 > +#define DA9062AA_GP_RISE4_STEP_MASK (0x0f << 0) > +#define DA9062AA_GP_FALL4_STEP_SHIFT 4 > +#define DA9062AA_GP_FALL4_STEP_MASK (0x0f << 4) > + > +/* DA9062AA_ID_30_29 = 0x091 */ > +#define DA9062AA_GP_RISE5_STEP_SHIFT 0 > +#define DA9062AA_GP_RISE5_STEP_MASK (0x0f << 0) > +#define DA9062AA_GP_FALL5_STEP_SHIFT 4 > +#define DA9062AA_GP_FALL5_STEP_MASK (0x0f << 4) > + > +/* DA9062AA_ID_32_31 = 0x092 */ > +#define DA9062AA_WAIT_STEP_SHIFT 0 > +#define DA9062AA_WAIT_STEP_MASK (0x0f << 0) > +#define DA9062AA_EN32K_STEP_SHIFT 4 > +#define DA9062AA_EN32K_STEP_MASK (0x0f << 4) > + > +/* DA9062AA_SEQ_A = 0x095 */ > +#define DA9062AA_SYSTEM_END_SHIFT 0 > +#define DA9062AA_SYSTEM_END_MASK (0x0f << 0) > +#define DA9062AA_POWER_END_SHIFT 4 > +#define DA9062AA_POWER_END_MASK (0x0f << 4) > + > +/* DA9062AA_SEQ_B = 0x096 */ > +#define DA9062AA_MAX_COUNT_SHIFT 0 > +#define DA9062AA_MAX_COUNT_MASK (0x0f << 0) > +#define DA9062AA_PART_DOWN_SHIFT 4 > +#define DA9062AA_PART_DOWN_MASK (0x0f << 4) > + > +/* DA9062AA_WAIT = 0x097 */ > +#define DA9062AA_WAIT_TIME_SHIFT 0 > +#define DA9062AA_WAIT_TIME_MASK (0x0f << 0) > +#define DA9062AA_WAIT_MODE_SHIFT 4 > +#define DA9062AA_WAIT_MODE_MASK (0x01 << 4) > +#define DA9062AA_TIME_OUT_SHIFT 5 > +#define DA9062AA_TIME_OUT_MASK (0x01 << 5) > +#define DA9062AA_WAIT_DIR_SHIFT 6 > +#define DA9062AA_WAIT_DIR_MASK (0x03 << 6) > + > +/* DA9062AA_EN_32K = 0x098 */ > +#define DA9062AA_STABILISATION_TIME_SHIFT 0 > +#define DA9062AA_STABILISATION_TIME_MASK (0x07 << 0) > +#define DA9062AA_CRYSTAL_SHIFT 3 > +#define DA9062AA_CRYSTAL_MASK (0x01 << 3) > +#define DA9062AA_DELAY_MODE_SHIFT 4 > +#define DA9062AA_DELAY_MODE_MASK (0x01 << 4) > +#define DA9062AA_OUT_CLOCK_SHIFT 5 > +#define DA9062AA_OUT_CLOCK_MASK (0x01 << 5) > +#define DA9062AA_RTC_CLOCK_SHIFT 6 > +#define DA9062AA_RTC_CLOCK_MASK (0x01 << 6) > +#define DA9062AA_EN_32KOUT_SHIFT 7 > +#define DA9062AA_EN_32KOUT_MASK (0x01 << 7) > + > +/* DA9062AA_RESET = 0x099 */ > +#define DA9062AA_RESET_TIMER_SHIFT 0 > +#define DA9062AA_RESET_TIMER_MASK (0x3f << 0) > +#define DA9062AA_RESET_EVENT_SHIFT 6 > +#define DA9062AA_RESET_EVENT_MASK (0x03 << 6) > + > +/* DA9062AA_BUCK_ILIM_A = 0x09A */ > +#define DA9062AA_BUCK3_ILIM_SHIFT 0 > +#define DA9062AA_BUCK3_ILIM_MASK (0x0f << 0) > + > +/* DA9062AA_BUCK_ILIM_B = 0x09B */ > +#define DA9062AA_BUCK4_ILIM_SHIFT 0 > +#define DA9062AA_BUCK4_ILIM_MASK (0x0f << 0) > + > +/* DA9062AA_BUCK_ILIM_C = 0x09C */ > +#define DA9062AA_BUCK1_ILIM_SHIFT 0 > +#define DA9062AA_BUCK1_ILIM_MASK (0x0f << 0) > +#define DA9062AA_BUCK2_ILIM_SHIFT 4 > +#define DA9062AA_BUCK2_ILIM_MASK (0x0f << 4) > + > +/* DA9062AA_BUCK2_CFG = 0x09D */ > +#define DA9062AA_BUCK2_PD_DIS_SHIFT 5 > +#define DA9062AA_BUCK2_PD_DIS_MASK (0x01 << 5) > +#define DA9062AA_BUCK2_MODE_SHIFT 6 > +#define DA9062AA_BUCK2_MODE_MASK (0x03 << 6) > + > +/* DA9062AA_BUCK1_CFG = 0x09E */ > +#define DA9062AA_BUCK1_PD_DIS_SHIFT 5 > +#define DA9062AA_BUCK1_PD_DIS_MASK (0x01 << 5) > +#define DA9062AA_BUCK1_MODE_SHIFT 6 > +#define DA9062AA_BUCK1_MODE_MASK (0x03 << 6) > + > +/* DA9062AA_BUCK4_CFG = 0x09F */ > +#define DA9062AA_BUCK4_VTTR_EN_SHIFT 3 > +#define DA9062AA_BUCK4_VTTR_EN_MASK (0x01 << 3) > +#define DA9062AA_BUCK4_VTT_EN_SHIFT 4 > +#define DA9062AA_BUCK4_VTT_EN_MASK (0x01 << 4) > +#define DA9062AA_BUCK4_PD_DIS_SHIFT 5 > +#define DA9062AA_BUCK4_PD_DIS_MASK (0x01 << 5) > +#define DA9062AA_BUCK4_MODE_SHIFT 6 > +#define DA9062AA_BUCK4_MODE_MASK (0x03 << 6) > + > +/* DA9062AA_BUCK3_CFG = 0x0A0 */ > +#define DA9062AA_BUCK3_PD_DIS_SHIFT 5 > +#define DA9062AA_BUCK3_PD_DIS_MASK (0x01 << 5) > +#define DA9062AA_BUCK3_MODE_SHIFT 6 > +#define DA9062AA_BUCK3_MODE_MASK (0x03 << 6) > + > +/* DA9062AA_VBUCK2_A = 0x0A3 */ > +#define DA9062AA_VBUCK2_A_SHIFT 0 > +#define DA9062AA_VBUCK2_A_MASK (0x7f << 0) > +#define DA9062AA_BUCK2_SL_A_SHIFT 7 > +#define DA9062AA_BUCK2_SL_A_MASK (0x01 << 7) > + > +/* DA9062AA_VBUCK1_A = 0x0A4 */ > +#define DA9062AA_VBUCK1_A_SHIFT 0 > +#define DA9062AA_VBUCK1_A_MASK (0x7f << 0) > +#define DA9062AA_BUCK1_SL_A_SHIFT 7 > +#define DA9062AA_BUCK1_SL_A_MASK (0x01 << 7) > + > +/* DA9062AA_VBUCK4_A = 0x0A5 */ > +#define DA9062AA_VBUCK4_A_SHIFT 0 > +#define DA9062AA_VBUCK4_A_MASK (0x7f << 0) > +#define DA9062AA_BUCK4_SL_A_SHIFT 7 > +#define DA9062AA_BUCK4_SL_A_MASK (0x01 << 7) > + > +/* DA9062AA_VBUCK3_A = 0x0A7 */ > +#define DA9062AA_VBUCK3_A_SHIFT 0 > +#define DA9062AA_VBUCK3_A_MASK (0x7f << 0) > +#define DA9062AA_BUCK3_SL_A_SHIFT 7 > +#define DA9062AA_BUCK3_SL_A_MASK (0x01 << 7) > + > +/* DA9062AA_VLDO1_A = 0x0A9 */ > +#define DA9062AA_VLDO1_A_SHIFT 0 > +#define DA9062AA_VLDO1_A_MASK (0x3f << 0) > +#define DA9062AA_LDO1_SL_A_SHIFT 7 > +#define DA9062AA_LDO1_SL_A_MASK (0x01 << 7) > + > +/* DA9062AA_VLDO2_A = 0x0AA */ > +#define DA9062AA_VLDO2_A_SHIFT 0 > +#define DA9062AA_VLDO2_A_MASK (0x3f << 0) > +#define DA9062AA_LDO2_SL_A_SHIFT 7 > +#define DA9062AA_LDO2_SL_A_MASK (0x01 << 7) > + > +/* DA9062AA_VLDO3_A = 0x0AB */ > +#define DA9062AA_VLDO3_A_SHIFT 0 > +#define DA9062AA_VLDO3_A_MASK (0x3f << 0) > +#define DA9062AA_LDO3_SL_A_SHIFT 7 > +#define DA9062AA_LDO3_SL_A_MASK (0x01 << 7) > + > +/* DA9062AA_VLDO4_A = 0x0AC */ > +#define DA9062AA_VLDO4_A_SHIFT 0 > +#define DA9062AA_VLDO4_A_MASK (0x3f << 0) > +#define DA9062AA_LDO4_SL_A_SHIFT 7 > +#define DA9062AA_LDO4_SL_A_MASK (0x01 << 7) > + > +/* DA9062AA_VBUCK2_B = 0x0B4 */ > +#define DA9062AA_VBUCK2_B_SHIFT 0 > +#define DA9062AA_VBUCK2_B_MASK (0x7f << 0) > +#define DA9062AA_BUCK2_SL_B_SHIFT 7 > +#define DA9062AA_BUCK2_SL_B_MASK (0x01 << 7) > + > +/* DA9062AA_VBUCK1_B = 0x0B5 */ > +#define DA9062AA_VBUCK1_B_SHIFT 0 > +#define DA9062AA_VBUCK1_B_MASK (0x7f << 0) > +#define DA9062AA_BUCK1_SL_B_SHIFT 7 > +#define DA9062AA_BUCK1_SL_B_MASK (0x01 << 7) > + > +/* DA9062AA_VBUCK4_B = 0x0B6 */ > +#define DA9062AA_VBUCK4_B_SHIFT 0 > +#define DA9062AA_VBUCK4_B_MASK (0x7f << 0) > +#define DA9062AA_BUCK4_SL_B_SHIFT 7 > +#define DA9062AA_BUCK4_SL_B_MASK (0x01 << 7) > + > +/* DA9062AA_VBUCK3_B = 0x0B8 */ > +#define DA9062AA_VBUCK3_B_SHIFT 0 > +#define DA9062AA_VBUCK3_B_MASK (0x7f << 0) > +#define DA9062AA_BUCK3_SL_B_SHIFT 7 > +#define DA9062AA_BUCK3_SL_B_MASK (0x01 << 7) > + > +/* DA9062AA_VLDO1_B = 0x0BA */ > +#define DA9062AA_VLDO1_B_SHIFT 0 > +#define DA9062AA_VLDO1_B_MASK (0x3f << 0) > +#define DA9062AA_LDO1_SL_B_SHIFT 7 > +#define DA9062AA_LDO1_SL_B_MASK (0x01 << 7) > + > +/* DA9062AA_VLDO2_B = 0x0BB */ > +#define DA9062AA_VLDO2_B_SHIFT 0 > +#define DA9062AA_VLDO2_B_MASK (0x3f << 0) > +#define DA9062AA_LDO2_SL_B_SHIFT 7 > +#define DA9062AA_LDO2_SL_B_MASK (0x01 << 7) > + > +/* DA9062AA_VLDO3_B = 0x0BC */ > +#define DA9062AA_VLDO3_B_SHIFT 0 > +#define DA9062AA_VLDO3_B_MASK (0x3f << 0) > +#define DA9062AA_LDO3_SL_B_SHIFT 7 > +#define DA9062AA_LDO3_SL_B_MASK (0x01 << 7) > + > +/* DA9062AA_VLDO4_B = 0x0BD */ > +#define DA9062AA_VLDO4_B_SHIFT 0 > +#define DA9062AA_VLDO4_B_MASK (0x3f << 0) > +#define DA9062AA_LDO4_SL_B_SHIFT 7 > +#define DA9062AA_LDO4_SL_B_MASK (0x01 << 7) > + > +/* DA9062AA_BBAT_CONT = 0x0C5 */ > +#define DA9062AA_BCHG_VSET_SHIFT 0 > +#define DA9062AA_BCHG_VSET_MASK (0x0f << 0) > +#define DA9062AA_BCHG_ISET_SHIFT 4 > +#define DA9062AA_BCHG_ISET_MASK (0x0f << 4) > + > +/* DA9062AA_INTERFACE = 0x105 */ > +#define DA9062AA_IF_BASE_ADDR_SHIFT 4 > +#define DA9062AA_IF_BASE_ADDR_MASK (0x0f << 4) > + > +/* DA9062AA_CONFIG_A = 0x106 */ > +#define DA9062AA_PM_I_V_SHIFT 0 > +#define DA9062AA_PM_I_V_MASK (0x01 << 0) > +#define DA9062AA_PM_O_TYPE_SHIFT 2 > +#define DA9062AA_PM_O_TYPE_MASK (0x01 << 2) > +#define DA9062AA_IRQ_TYPE_SHIFT 3 > +#define DA9062AA_IRQ_TYPE_MASK (0x01 << 3) > +#define DA9062AA_PM_IF_V_SHIFT 4 > +#define DA9062AA_PM_IF_V_MASK (0x01 << 4) > +#define DA9062AA_PM_IF_FMP_SHIFT 5 > +#define DA9062AA_PM_IF_FMP_MASK (0x01 << 5) > +#define DA9062AA_PM_IF_HSM_SHIFT 6 > +#define DA9062AA_PM_IF_HSM_MASK (0x01 << 6) > + > +/* DA9062AA_CONFIG_B = 0x107 */ > +#define DA9062AA_VDD_FAULT_ADJ_SHIFT 0 > +#define DA9062AA_VDD_FAULT_ADJ_MASK (0x0f << 0) > +#define DA9062AA_VDD_HYST_ADJ_SHIFT 4 > +#define DA9062AA_VDD_HYST_ADJ_MASK (0x07 << 4) > + > +/* DA9062AA_CONFIG_C = 0x108 */ > +#define DA9062AA_BUCK_ACTV_DISCHRG_SHIFT 2 > +#define DA9062AA_BUCK_ACTV_DISCHRG_MASK (0x01 << 2) > +#define DA9062AA_BUCK1_CLK_INV_SHIFT 3 > +#define DA9062AA_BUCK1_CLK_INV_MASK (0x01 << 3) > +#define DA9062AA_BUCK4_CLK_INV_SHIFT 4 > +#define DA9062AA_BUCK4_CLK_INV_MASK (0x01 << 4) > +#define DA9062AA_BUCK3_CLK_INV_SHIFT 6 > +#define DA9062AA_BUCK3_CLK_INV_MASK (0x01 << 6) > + > +/* DA9062AA_CONFIG_D = 0x109 */ > +#define DA9062AA_GPI_V_SHIFT 0 > +#define DA9062AA_GPI_V_MASK (0x01 << 0) > +#define DA9062AA_NIRQ_MODE_SHIFT 1 > +#define DA9062AA_NIRQ_MODE_MASK (0x01 << 1) > +#define DA9062AA_SYSTEM_EN_RD_SHIFT 2 > +#define DA9062AA_SYSTEM_EN_RD_MASK (0x01 << 2) > +#define DA9062AA_FORCE_RESET_SHIFT 5 > +#define DA9062AA_FORCE_RESET_MASK (0x01 << 5) > + > +/* DA9062AA_CONFIG_E = 0x10A */ > +#define DA9062AA_BUCK1_AUTO_SHIFT 0 > +#define DA9062AA_BUCK1_AUTO_MASK (0x01 << 0) > +#define DA9062AA_BUCK2_AUTO_SHIFT 1 > +#define DA9062AA_BUCK2_AUTO_MASK (0x01 << 1) > +#define DA9062AA_BUCK4_AUTO_SHIFT 2 > +#define DA9062AA_BUCK4_AUTO_MASK (0x01 << 2) > +#define DA9062AA_BUCK3_AUTO_SHIFT 4 > +#define DA9062AA_BUCK3_AUTO_MASK (0x01 << 4) > + > +/* DA9062AA_CONFIG_G = 0x10C */ > +#define DA9062AA_LDO1_AUTO_SHIFT 0 > +#define DA9062AA_LDO1_AUTO_MASK (0x01 << 0) > +#define DA9062AA_LDO2_AUTO_SHIFT 1 > +#define DA9062AA_LDO2_AUTO_MASK (0x01 << 1) > +#define DA9062AA_LDO3_AUTO_SHIFT 2 > +#define DA9062AA_LDO3_AUTO_MASK (0x01 << 2) > +#define DA9062AA_LDO4_AUTO_SHIFT 3 > +#define DA9062AA_LDO4_AUTO_MASK (0x01 << 3) > + > +/* DA9062AA_CONFIG_H = 0x10D */ > +#define DA9062AA_BUCK1_2_MERGE_SHIFT 3 > +#define DA9062AA_BUCK1_2_MERGE_MASK (0x01 << 3) > +#define DA9062AA_BUCK2_OD_SHIFT 5 > +#define DA9062AA_BUCK2_OD_MASK (0x01 << 5) > +#define DA9062AA_BUCK1_OD_SHIFT 6 > +#define DA9062AA_BUCK1_OD_MASK (0x01 << 6) > + > +/* DA9062AA_CONFIG_I = 0x10E */ > +#define DA9062AA_NONKEY_PIN_SHIFT 0 > +#define DA9062AA_NONKEY_PIN_MASK (0x03 << 0) > +#define DA9062AA_nONKEY_SD_SHIFT 2 > +#define DA9062AA_nONKEY_SD_MASK (0x01 << 2) > +#define DA9062AA_WATCHDOG_SD_SHIFT 3 > +#define DA9062AA_WATCHDOG_SD_MASK (0x01 << 3) > +#define DA9062AA_KEY_SD_MODE_SHIFT 4 > +#define DA9062AA_KEY_SD_MODE_MASK (0x01 << 4) > +#define DA9062AA_HOST_SD_MODE_SHIFT 5 > +#define DA9062AA_HOST_SD_MODE_MASK (0x01 << 5) > +#define DA9062AA_INT_SD_MODE_SHIFT 6 > +#define DA9062AA_INT_SD_MODE_MASK (0x01 << 6) > +#define DA9062AA_LDO_SD_SHIFT 7 > +#define DA9062AA_LDO_SD_MASK (0x01 << 7) > + > +/* DA9062AA_CONFIG_J = 0x10F */ > +#define DA9062AA_KEY_DELAY_SHIFT 0 > +#define DA9062AA_KEY_DELAY_MASK (0x03 << 0) > +#define DA9062AA_SHUT_DELAY_SHIFT 2 > +#define DA9062AA_SHUT_DELAY_MASK (0x03 << 2) > +#define DA9062AA_RESET_DURATION_SHIFT 4 > +#define DA9062AA_RESET_DURATION_MASK (0x03 << 4) > +#define DA9062AA_TWOWIRE_TO_SHIFT 6 > +#define DA9062AA_TWOWIRE_TO_MASK (0x01 << 6) > +#define DA9062AA_IF_RESET_SHIFT 7 > +#define DA9062AA_IF_RESET_MASK (0x01 << 7) > + > +/* DA9062AA_CONFIG_K = 0x110 */ > +#define DA9062AA_GPIO0_PUPD_SHIFT 0 > +#define DA9062AA_GPIO0_PUPD_MASK (0x01 << 0) > +#define DA9062AA_GPIO1_PUPD_SHIFT 1 > +#define DA9062AA_GPIO1_PUPD_MASK (0x01 << 1) > +#define DA9062AA_GPIO2_PUPD_SHIFT 2 > +#define DA9062AA_GPIO2_PUPD_MASK (0x01 << 2) > +#define DA9062AA_GPIO3_PUPD_SHIFT 3 > +#define DA9062AA_GPIO3_PUPD_MASK (0x01 << 3) > +#define DA9062AA_GPIO4_PUPD_SHIFT 4 > +#define DA9062AA_GPIO4_PUPD_MASK (0x01 << 4) > + > +/* DA9062AA_CONFIG_M = 0x112 */ > +#define DA9062AA_NSHUTDOWN_PU_SHIFT 1 > +#define DA9062AA_NSHUTDOWN_PU_MASK (0x01 << 1) > +#define DA9062AA_WDG_MODE_SHIFT 3 > +#define DA9062AA_WDG_MODE_MASK (0x01 << 3) > +#define DA9062AA_OSC_FRQ_SHIFT 4 > +#define DA9062AA_OSC_FRQ_MASK (0x0f << 4) > + > +/* DA9062AA_TRIM_CLDR = 0x120 */ > +#define DA9062AA_TRIM_CLDR_SHIFT 0 > +#define DA9062AA_TRIM_CLDR_MASK (0xff << 0) > + > +/* DA9062AA_GP_ID_0 = 0x121 */ > +#define DA9062AA_GP_0_SHIFT 0 > +#define DA9062AA_GP_0_MASK (0xff << 0) > + > +/* DA9062AA_GP_ID_1 = 0x122 */ > +#define DA9062AA_GP_1_SHIFT 0 > +#define DA9062AA_GP_1_MASK (0xff << 0) > + > +/* DA9062AA_GP_ID_2 = 0x123 */ > +#define DA9062AA_GP_2_SHIFT 0 > +#define DA9062AA_GP_2_MASK (0xff << 0) > + > +/* DA9062AA_GP_ID_3 = 0x124 */ > +#define DA9062AA_GP_3_SHIFT 0 > +#define DA9062AA_GP_3_MASK (0xff << 0) > + > +/* DA9062AA_GP_ID_4 = 0x125 */ > +#define DA9062AA_GP_4_SHIFT 0 > +#define DA9062AA_GP_4_MASK (0xff << 0) > + > +/* DA9062AA_GP_ID_5 = 0x126 */ > +#define DA9062AA_GP_5_SHIFT 0 > +#define DA9062AA_GP_5_MASK (0xff << 0) > + > +/* DA9062AA_GP_ID_6 = 0x127 */ > +#define DA9062AA_GP_6_SHIFT 0 > +#define DA9062AA_GP_6_MASK (0xff << 0) > + > +/* DA9062AA_GP_ID_7 = 0x128 */ > +#define DA9062AA_GP_7_SHIFT 0 > +#define DA9062AA_GP_7_MASK (0xff << 0) > + > +/* DA9062AA_GP_ID_8 = 0x129 */ > +#define DA9062AA_GP_8_SHIFT 0 > +#define DA9062AA_GP_8_MASK (0xff << 0) > + > +/* DA9062AA_GP_ID_9 = 0x12A */ > +#define DA9062AA_GP_9_SHIFT 0 > +#define DA9062AA_GP_9_MASK (0xff << 0) > + > +/* DA9062AA_GP_ID_10 = 0x12B */ > +#define DA9062AA_GP_10_SHIFT 0 > +#define DA9062AA_GP_10_MASK (0xff << 0) > + > +/* DA9062AA_GP_ID_11 = 0x12C */ > +#define DA9062AA_GP_11_SHIFT 0 > +#define DA9062AA_GP_11_MASK (0xff << 0) > + > +/* DA9062AA_GP_ID_12 = 0x12D */ > +#define DA9062AA_GP_12_SHIFT 0 > +#define DA9062AA_GP_12_MASK (0xff << 0) > + > +/* DA9062AA_GP_ID_13 = 0x12E */ > +#define DA9062AA_GP_13_SHIFT 0 > +#define DA9062AA_GP_13_MASK (0xff << 0) > + > +/* DA9062AA_GP_ID_14 = 0x12F */ > +#define DA9062AA_GP_14_SHIFT 0 > +#define DA9062AA_GP_14_MASK (0xff << 0) > + > +/* DA9062AA_GP_ID_15 = 0x130 */ > +#define DA9062AA_GP_15_SHIFT 0 > +#define DA9062AA_GP_15_MASK (0xff << 0) > + > +/* DA9062AA_GP_ID_16 = 0x131 */ > +#define DA9062AA_GP_16_SHIFT 0 > +#define DA9062AA_GP_16_MASK (0xff << 0) > + > +/* DA9062AA_GP_ID_17 = 0x132 */ > +#define DA9062AA_GP_17_SHIFT 0 > +#define DA9062AA_GP_17_MASK (0xff << 0) > + > +/* DA9062AA_GP_ID_18 = 0x133 */ > +#define DA9062AA_GP_18_SHIFT 0 > +#define DA9062AA_GP_18_MASK (0xff << 0) > + > +/* DA9062AA_GP_ID_19 = 0x134 */ > +#define DA9062AA_GP_19_SHIFT 0 > +#define DA9062AA_GP_19_MASK (0xff << 0) > + > +/* DA9062AA_DEVICE_ID = 0x181 */ > +#define DA9062AA_DEV_ID_SHIFT 0 > +#define DA9062AA_DEV_ID_MASK (0xff << 0) > + > +/* DA9062AA_VARIANT_ID = 0x182 */ > +#define DA9062AA_VRC_SHIFT 0 > +#define DA9062AA_VRC_MASK (0x0f << 0) > +#define DA9062AA_MRC_SHIFT 4 > +#define DA9062AA_MRC_MASK (0x0f << 4) > + > +/* DA9062AA_CUSTOMER_ID = 0x183 */ > +#define DA9062AA_CUST_ID_SHIFT 0 > +#define DA9062AA_CUST_ID_MASK (0xff << 0) > + > +/* DA9062AA_CONFIG_ID = 0x184 */ > +#define DA9062AA_CONFIG_REV_SHIFT 0 > +#define DA9062AA_CONFIG_REV_MASK (0xff << 0) > + > +#endif /* __DA9062_H__ */ -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html