[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20150526161024.GQ11677@x1>
Date: Tue, 26 May 2015 17:10:24 +0100
From: Lee Jones <lee.jones@...aro.org>
To: S Twiss <stwiss.opensource@...semi.com>
Cc: LINUXKERNEL <linux-kernel@...r.kernel.org>,
Samuel Ortiz <sameo@...ux.intel.com>,
Alessandro Zummo <a.zummo@...ertech.it>,
DEVICETREE <devicetree@...r.kernel.org>,
David Dajun Chen <david.chen@...semi.com>,
Dmitry Torokhov <dmitry.torokhov@...il.com>,
Ian Campbell <ijc+devicetree@...lion.org.uk>,
Kumar Gala <galak@...eaurora.org>,
LINUXINPUT <linux-input@...r.kernel.org>,
LINUXWATCHDOG <linux-watchdog@...r.kernel.org>,
Liam Girdwood <lgirdwood@...il.com>,
Mark Brown <broonie@...nel.org>,
Mark Rutland <mark.rutland@....com>,
Pawel Moll <pawel.moll@....com>,
RTCLINUX <rtc-linux@...glegroups.com>,
Rob Herring <robh+dt@...nel.org>,
Support Opensource <support.opensource@...semi.com>,
Wim Van Sebroeck <wim@...ana.be>
Subject: Re: [PATCH V3 1/4] mfd: da9062: DA9062 MFD core driver
On Tue, 19 May 2015, S Twiss wrote:
> From: S Twiss <stwiss.opensource@...semi.com>
>
> Add MFD core driver support for DA9062
>
>
> Signed-off-by: Steve Twiss <stwiss.opensource@...semi.com>
>
> ---
> 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@...semi.com>");
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-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists