da9062: DA9062 driver submission
Subsystems (7)
v3 cover da9062: DA9062 driver submission
From: S Twiss <[email protected]>
This patch set adds support for the Dialog DA9062 Power Management IC.
In this patch set the following is provided:
- [PATCH V3 1/4]: MFD core support
- [PATCH V3 2/4]: BUCK and LDO regulator driver
- [PATCH V3 3/4]: Watchdog driver
- [PATCH V3 4/4]: Add bindings for all DA9062 components
Changes in V2:
- Alter subject title in 'cover letter'
- Reordered the patch numbering: now 4/4 patches instead of 6/6
- Dropped PATCH V1 3/6 RTC driver
- Dropped PATCH V1 4/6 OnKey driver
This patch applies against linux-next and v4.1-rc3
Thank you,
Steve Twiss, Dialog Semiconductor Ltd.
S Twiss (4):
mfd: da9062: DA9062 MFD core driver
regulator: da9062: DA9062 regulator driver
watchdog: da9062: DA9062 watchdog driver
devicetree: da9062: Add bindings for DA9062 driver
Documentation/devicetree/bindings/mfd/da9062.txt | 79 ++ drivers/mfd/Kconfig | 12 + drivers/mfd/Makefile | 3 +- drivers/mfd/da9062-core.c | 611 ++++++++++++ drivers/regulator/Kconfig | 10 + drivers/regulator/Makefile | 1 + drivers/regulator/da9062-regulator.c | 843 ++++++++++++++++ drivers/watchdog/Kconfig | 9 + drivers/watchdog/Makefile | 1 + drivers/watchdog/da9062_wdt.c | 253 +++++ include/linux/mfd/da9062/core.h | 62 ++ include/linux/mfd/da9062/registers.h | 1108 ++++++++++++++++++++++ 12 files changed, 2991 insertions(+), 1 deletion(-) create mode 100644 Documentation/devicetree/bindings/mfd/da9062.txt create mode 100644 drivers/mfd/da9062-core.c create mode 100644 drivers/regulator/da9062-regulator.c create mode 100644 drivers/watchdog/da9062_wdt.c create mode 100644 include/linux/mfd/da9062/core.h create mode 100644 include/linux/mfd/da9062/registers.h
Signature · 6 lines
end-of-patch for PATCH V3 -- To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
PATCH v3 1/4 mfd: da9062: DA9062 MFD core driver
From: S Twiss <[email protected]>
Add MFD core driver support for DA9062
---
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
drivers/mfd/Kconfig +12 -0
@@ -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
drivers/mfd/Makefile +2 -1
@@ -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
drivers/mfd/da9062-core.c new file +611 -0
611 lines not on this page · raw message
include/linux/mfd/da9062/core.h new file +62 -0
62 lines not on this page · raw message
include/linux/mfd/da9062/registers.h new file +1108 -0
1,108 lines not on this page · raw message
Signature · 6 lines
end-of-patch for PATCH V3 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH V3 1/4] mfd: da9062: DA9062 MFD core driver
Quoted 76 lines · S Twiss
From: S Twiss <[email protected]>
Add MFD core driver support for DA9062
Signed-off-by Steve Twiss <[email protected]>---
Changes in V3:
- Removed references to the RTC and OnKey in themfd_celldefinition.Changes in V2:
- Copyright headers GPL v2 (and later) match correct 'GPL' inMODULE_LICENSE
- Ensure DA9062 core is tristate inKconfigso it can be built as a module
- MoveVDD_WARNcode and resource into the core instead of regulator driverThis 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.hdrivers/mfd/Kconfig +12
-0@@ -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_COREdrivers/mfd/Makefile +2
-1@@ -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.odrivers/mfd/da9062-core.c new file +0
-0
drivers/mfd/da9062-core.c line 2
Quoted 2 lines
drivers/mfd/da9062-core.c new file +2
-0@@ -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?
Quoted 13 lines
drivers/mfd/da9062-core.c new file +13
-0+* 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. +*/ +
drivers/mfd/da9062-core.c line 25
Quoted 10 lines
drivers/mfd/da9062-core.c new file +10
-0+#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'?
drivers/mfd/da9062-core.c line 30
Quoted 5 lines
drivers/mfd/da9062-core.c new file +5
-0+#include <linux/irq.h> +#include <linux/mfd/core.h> +#include <linux/i2c.h> +#include <linux/err.h> +
Same here?
drivers/mfd/da9062-core.c line 38
Quoted 8 lines
drivers/mfd/da9062-core.c new file +8
-0+#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?
drivers/mfd/da9062-core.c line 40
Quoted 1 line
drivers/mfd/da9062-core.c new file +1
-0+/* IRQ device driver for DA9062 */
Not sure what this means?
Quoted 58 lines
drivers/mfd/da9062-core.c new file +58
-0+#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 ,
drivers/mfd/da9062-core.c line 109
Quoted 10 lines
drivers/mfd/da9062-core.c new file +10
-0+ .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.
drivers/mfd/da9062-core.c line 113
Quoted 4 lines
drivers/mfd/da9062-core.c new file +4
-0+ .name = "da9062-irq", + .irqs = da9062_irqs, + .num_irqs = DA9062_NUM_IRQ, +/* STATUS and EVENT */
This is a standard struct, no need for additional comments.
drivers/mfd/da9062-core.c line 124
Quoted 11 lines
drivers/mfd/da9062-core.c new file +11
-0+ .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.
drivers/mfd/da9062-core.c line 132
Quoted 8 lines
drivers/mfd/da9062-core.c new file +8
-0+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 intoda9062_device_init() and rid this function.
drivers/mfd/da9062-core.c line 144
Quoted 12 lines
drivers/mfd/da9062-core.c new file +12
-0+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.
drivers/mfd/da9062-core.c line 147
Quoted 3 lines
drivers/mfd/da9062-core.c new file +3
-0+} + +/* CORE device driver for DA9062 */
Doesn't add anything. This whole driver is the core driver.
drivers/mfd/da9062-core.c line 149
Quoted 1 line
drivers/mfd/da9062-core.c new file +1
-0+static struct resource da9062_core_resources[] = {
const?
Quoted 24 lines
drivers/mfd/da9062-core.c new file +24
-0+ { + .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 , + },
drivers/mfd/da9062-core.c line 183
Quoted 10 lines
drivers/mfd/da9062-core.c new file +10
-0+}; + +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.
Quoted 20 lines
drivers/mfd/da9062-core.c new file +20
-0+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),
drivers/mfd/da9062-core.c line 214
Quoted 10 lines
drivers/mfd/da9062-core.c new file +10
-0+ .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.
drivers/mfd/da9062-core.c line 221
Quoted 6 lines
drivers/mfd/da9062-core.c new file +6
-0+return IRQ_HANDLED ; +} + +static int da9062_clear_fault_log (struct da9062 *chip) +{ +int ret =0 ;
No need to pre-initialise.
drivers/mfd/da9062-core.c line 222
Quoted 1 line
drivers/mfd/da9062-core.c new file +1
-0+int fault_log =0 ;
As above.
drivers/mfd/da9062-core.c line 227
Quoted 4 lines
drivers/mfd/da9062-core.c new file +4
-0+ 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.
Quoted 21 lines
drivers/mfd/da9062-core.c new file +21
-0+ }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,
drivers/mfd/da9062-core.c line 258
Quoted 10 lines
drivers/mfd/da9062-core.c new file +10
-0+ 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.
drivers/mfd/da9062-core.c line 261
Quoted 3 lines
drivers/mfd/da9062-core.c new file +3
-0+{ +int device_id, variant_id, variant_mrc; +int irq_vdd_warn, ret;
Put ret on its own line.
drivers/mfd/da9062-core.c line 265
Quoted 3 lines
drivers/mfd/da9062-core.c new file +3
-0+ 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().
drivers/mfd/da9062-core.c line 267
Quoted 1 line
drivers/mfd/da9062-core.c new file +1
-0+ chip->irq_base =-1 ;
Why are you pre-initialising?
drivers/mfd/da9062-core.c line 268
Quoted 1 line
drivers/mfd/da9062-core.c new file +1
-0+ chip->chip_irq = irq;
You already assigned irq to chip_irq.
Quoted 9 lines
drivers/mfd/da9062-core.c new file +9
-0+ 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 ; + }
drivers/mfd/da9062-core.c line 288
Quoted 10 lines
drivers/mfd/da9062-core.c new file +10
-0+ + 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.
drivers/mfd/da9062-core.c line 299
Quoted 10 lines
drivers/mfd/da9062-core.c new file +10
-0+ 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().
drivers/mfd/da9062-core.c line 305
Quoted 5 lines
drivers/mfd/da9062-core.c new file +5
-0+ 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.
drivers/mfd/da9062-core.c line 309
Quoted 3 lines
drivers/mfd/da9062-core.c new file +3
-0+ 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.
drivers/mfd/da9062-core.c line 311
Quoted 2 lines
drivers/mfd/da9062-core.c new file +2
-0+ARRAY_SIZE (da9062_devs),NULL , chip->irq_base, + NULL);
Usually child devices are probed at the end i.e. the last thing in probe().
drivers/mfd/da9062-core.c line 301
Quoted 2 lines
drivers/mfd/da9062-core.c new file +2
-0+if (ret) { +dev_err (chip->dev,"Cannot add MFD cells\n" );
"Cannot register child devices".
Quoted 12 lines
drivers/mfd/da9062-core.c new file +12
-0+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;
drivers/mfd/da9062-core.c line 335
Quoted 10 lines
drivers/mfd/da9062-core.c new file +10
-0+ + 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?
drivers/mfd/da9062-core.c line 344
Quoted 8 lines
drivers/mfd/da9062-core.c new file +8
-0+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()?
drivers/mfd/da9062-core.c line 346
Quoted 1 line
drivers/mfd/da9062-core.c new file +1
-0+/* I2C device driver for DA9062 */
Remove these.
Quoted 193 lines
drivers/mfd/da9062-core.c new file +193
-0+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,
drivers/mfd/da9062-core.c line 550
Quoted 10 lines
drivers/mfd/da9062-core.c new file +10
-0+ .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.
drivers/mfd/da9062-core.c line 558
Quoted 7 lines
drivers/mfd/da9062-core.c new file +7
-0+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)
drivers/mfd/da9062-core.c line 559
Quoted 1 line
drivers/mfd/da9062-core.c new file +1
-0+if (chip ==NULL )
if (!chip)
drivers/mfd/da9062-core.c line 564
Quoted 5 lines
drivers/mfd/da9062-core.c new file +5
-0+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/
drivers/mfd/da9062-core.c line 568
Quoted 3 lines
drivers/mfd/da9062-core.c new file +3
-0+ 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?
Quoted 6 lines
drivers/mfd/da9062-core.c new file +6
-0+ 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;
drivers/mfd/da9062-core.c line 585
Quoted 10 lines
drivers/mfd/da9062-core.c new file +10
-0+ } + +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.
drivers/mfd/da9062-core.c line 591
Quoted 5 lines
drivers/mfd/da9062-core.c new file +5
-0+return 0 ; +} + +static const struct i2c_device_id da9062_i2c_id[] = { + {"da9062" ,0 },
White space discipline.
Quoted 7 lines
drivers/mfd/da9062-core.c new file +7
-0+ {}, +}; +MODULE_DEVICE_TABLE (i2c, da9062_i2c_id); + +static struct i2c_driver da9062_i2c_driver = { + .driver = { + .name ="da9062" ,
drivers/mfd/da9062-core.c line 608
Quoted 10 lines
drivers/mfd/da9062-core.c new file +10
-0+ .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'.
drivers/mfd/da9062-core.c line 609
Quoted 1 line
drivers/mfd/da9062-core.c new file +1
-0+MODULE_DESCRIPTION ("CORE device driver for Dialog DA9062" );
s/CORE/Core/
drivers/mfd/da9062-core.c line 591
Quoted 1 line
drivers/mfd/da9062-core.c new file +1
-0+MODULE_AUTHOR ("S Twiss <[email protected]>" );
Full names please.
include/linux/mfd/da9062/core.h line 2
Quoted 9 lines
drivers/mfd/da9062-core.c new file +1
-0+MODULE_LICENSE ("GPL" );include/linux/mfd/da9062/core.h new file +2
-0@@ -0,0 +1,62 @@ +/* +* core.h - CORE H for DA9062
What does this add?
Quoted 42 lines
include/linux/mfd/da9062/core.h new file +42
-0+* 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 , +}; +
include/linux/mfd/da9062/core.h line 54
Quoted 10 lines
include/linux/mfd/da9062/core.h new file +10
-0+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?
include/linux/mfd/da9062/core.h line 60
Quoted 5 lines
include/linux/mfd/da9062/core.h new file +5
-0+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?
Quoted 23 lines
include/linux/mfd/da9062/core.h new file +1
-0+#endif /* __MFD_DA9062_CORE_H__ */ include/linux/mfd/da9062/registers.h new file +16
-0@@ -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__
include/linux/mfd/da9062/registers.h line 26
Quoted 10 lines
include/linux/mfd/da9062/registers.h new file +10
-0+#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? ;)
Quoted 129 lines
include/linux/mfd/da9062/registers.h new file +129
-0+#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
include/linux/mfd/da9062/registers.h line 166
Quoted 10 lines
include/linux/mfd/da9062/registers.h new file +10
-0+ +/* +* 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).
Quoted 942 lines
include/linux/mfd/da9062/registers.h new file +942
-0+#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__ */
Signature · 8 lines
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 [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
RE: [PATCH V3 1/4] mfd: da9062: DA9062 MFD core driver
SGkgTGVlLCANCg0KSSB3aWxsIHJlZmFjdG9yIGEgbG90IG9mIHRoZSBkcml2ZXIgYW5kIGltcGxl bWVudCB5b3VyIGNoYW5nZXMgYXMgcmVxdWVzdGVkLg0KSSB0aGluayB0aGUgb25seSBtYWpvciBk aWZmZXJlbmNlcyB3aXRoIHlvdXIgY29tbWVudHMgd2lsbCBiZSBhcyBmb2xsb3dzOg0KDQotIHRo ZSBpbnRlcnJ1cHQgaGFuZGxlciBkYTkwNjJfdmRkX3dhcm5fZXZlbnQoKSB3aWxsIGJlIGVyYXNl ZA0KLSBJIHdvdWxkIHByZWZlciB0aGUgcmVnaXN0ZXIgaGVhZGVyIGZpbGUgdG8gcmVtYWluICBb bW9zdGx5XSB1bnRvdWNoZWQgDQoNClBsZWFzZSBmaW5kIG1vcmUgZGV0YWlsZWQgY29tbWVudHMg YmVsb3cuDQoNCnRoYW5rcywNClN0ZXZlDQoNCk9uIDI2IE1heSAyMDE1IDE3OjEwIExlZSBKb25l cyB3cm90ZQ0KDQo+IFRvOiBPcGVuc291cmNlIFtTdGV2ZSBUd2lzc10NCj4gQ2M6IExJTlVYS0VS TkVMOyBTYW11ZWwgT3J0aXo7IEFsZXNzYW5kcm8gWnVtbW87IERFVklDRVRSRUU7IERhdmlkDQo+ IERhanVuIENoZW47IERtaXRyeSBUb3Jva2hvdjsgSWFuIENhbXBiZWxsOyBLdW1hciBHYWxhOyBM SU5VWElOUFVUOw0KPiBMSU5VWFdBVENIRE9HOyBMaWFtIEdpcmR3b29kOyBNYXJrIEJyb3duOyBN YXJrIFJ1dGxhbmQ7IFBhd2VsIE1vbGw7DQo+IFJUQ0xJTlVYOyBSb2IgSGVycmluZzsgU3VwcG9y dCBPcGVuc291cmNlOyBXaW0gVmFuIFNlYnJvZWNrDQo+IFN1YmplY3Q6IFJlOiBbUEFUQ0ggVjMg MS80XSBtZmQ6IGRhOTA2MjogREE5MDYyIE1GRCBjb3JlIGRyaXZlcg0KPiANCj4gT24gVHVlLCAx OSBNYXkgMjAxNSwgUyBUd2lzcyB3cm90ZToNCj4gDQo+ID4gRnJvbTogUyBUd2lzcyA8c3R3aXNz Lm9wZW5zb3VyY2VAZGlhc2VtaS5jb20+DQo+ID4NCj4gPiBBZGQgTUZEIGNvcmUgZHJpdmVyIHN1 cHBvcnQgZm9yIERBOTA2Mg0KPiA+DQo+ID4gU2lnbmVkLW9mZi1ieTogU3RldmUgVHdpc3MgPHN0 d2lzcy5vcGVuc291cmNlQGRpYXNlbWkuY29tPg0KPiA+DQoNClsuLi5dDQoNCj4gPiBkaWZmIC0t Z2l0IGEvZHJpdmVycy9tZmQvZGE5MDYyLWNvcmUuYyBiL2RyaXZlcnMvbWZkL2RhOTA2Mi1jb3Jl LmMNCj4gPiBuZXcgZmlsZSBtb2RlIDEwMDY0NA0KPiA+IGluZGV4IDAwMDAwMDAuLjVhZWEwODIN Cj4gPiAtLS0gL2Rldi9udWxsDQo+ID4gKysrIGIvZHJpdmVycy9tZmQvZGE5MDYyLWNvcmUuYw0K PiA+IEBAIC0wLDAgKzEsNjExIEBADQo+ID4gKy8qDQo+ID4gKyAqIGRhOTA2Mi1jb3JlLmMgLSBD T1JFIGRldmljZSBkcml2ZXIgZm9yIERBOTA2Mg0KPiANCj4gUmVtb3ZlIHRoZSBmaWxlbmFtZS4g IFRoZXkgaGF2ZSBhIGhhYml0IG9mIGJlY29taW5nIGluY29ycmVjdC4NCj4gDQo+IHMvQ09SRS9D b3JlLw0KPiANCj4gQ2FuIHlvdSBhbHNvIG1lbnRpb24gd2hhdCB0aGUgREE5MDYyIGFjdHVhbGx5 IGlzPw0KPiANCg0KV2lsbCByZW1vdmUgZmlsZW5hbWUsIGFuZCBhZGQgYSBkZXNjcmlwdGlvbg0K IkNvcmUsIElSUSBhbmQgSTJDIGRyaXZlciBmb3IgREE5MDYyIFBNSUMiIA0KDQpbLi4uXQ0KDQo+ ID4gKyNpbmNsdWRlIDxsaW51eC9tdXRleC5oPg0KPiA+ICsjaW5jbHVkZSA8bGludXgvcmVnbWFw Lmg+DQo+ID4gKw0KPiANCj4gV2h5IHRoZSAnXG4nPw0KPiANCj4gPiArI2luY2x1ZGUgPGxpbnV4 L2lycS5oPg0KPiA+ICsjaW5jbHVkZSA8bGludXgvbWZkL2NvcmUuaD4NCj4gPiArI2luY2x1ZGUg PGxpbnV4L2kyYy5oPg0KPiA+ICsjaW5jbHVkZSA8bGludXgvZXJyLmg+DQo+ID4gKw0KPiANCj4g U2FtZSBoZXJlPw0KPiANCj4gPiArI2luY2x1ZGUgPGxpbnV4L21mZC9kYTkwNjIvY29yZS5oPg0K PiA+ICsjaW5jbHVkZSA8bGludXgvbWZkL2RhOTA2Mi9yZWdpc3RlcnMuaD4NCj4gPiArI2luY2x1 ZGUgPGxpbnV4L29mLmg+DQo+ID4gKyNpbmNsdWRlIDxsaW51eC9yZWd1bGF0b3Ivb2ZfcmVndWxh dG9yLmg+DQo+ID4gKw0KPiA+ICsjaW5jbHVkZSA8bGludXgvcHJvY19mcy5oPg0KPiA+ICsjaW5j bHVkZSA8bGludXgva3RocmVhZC5oPg0KPiA+ICsjaW5jbHVkZSA8bGludXgvdWFjY2Vzcy5oPg0K PiANCj4gSSdtIGEgYml0IGNvbmNlcm5lZCBieSB0aGUgbnVtYmVyIG9mIGluY2x1ZGVzIGhlcmUs IGFyZSB5b3Ugc3VyZQ0KPiB0aGV5J3JlIGFsbCByZXF1aXJlZD8NCj4gDQo+ID4gKy8qIElSUSBk ZXZpY2UgZHJpdmVyIGZvciBEQTkwNjIgKi8NCj4gDQo+IE5vdCBzdXJlIHdoYXQgdGhpcyBtZWFu cz8NCg0KUmVtb3ZlZCB1bm5lY2Vzc2FyeSB3aGl0ZXNwYWNlIHRocm91Z2hvdXQgdGhlIHBhdGNo ZXMuDQpBbmQgZXJhc2VkIGNvbW1lbnRzIHRocm91Z2hvdXQgIHRoYXQgYXJlIG5vdCByZXF1aXJl ZA0KDQpbLi4uXQ0KDQo+ID4gK2ludCBkYTkwNjJfaXJxX2luaXQoc3RydWN0IGRhOTA2MiAqY2hp cCkNCj4gPiArew0KPiA+ICsJaW50IHJldDsNCj4gPiArDQo+ID4gKwlpZiAoIWNoaXAtPmNoaXBf aXJxKSB7DQo+IA0KPiBDaGVjayB0aGlzIG9uY2UsIGluIHByb2JlKCksIHRoZW4gcmlkIHRoaXMg Y2hlY2suDQo+IA0KPiA+ICsJCWRldl9lcnIoY2hpcC0+ZGV2LCAiTm8gSVJRIGNvbmZpZ3VyZWRc biIpOw0KPiA+ICsJCXJldHVybiAtRUlOVkFMOw0KPiA+ICsJfQ0KPiA+ICsNCj4gPiArCXJldCA9 IHJlZ21hcF9hZGRfaXJxX2NoaXAoY2hpcC0+cmVnbWFwLCBjaGlwLT5jaGlwX2lycSwNCj4gPiAr CQkJSVJRRl9UUklHR0VSX0xPVyB8IElSUUZfT05FU0hPVCB8DQo+IElSUUZfU0hBUkVELA0KPiA+ ICsJCQljaGlwLT5pcnFfYmFzZSwgJmRhOTA2Ml9pcnFfY2hpcCwNCj4gPiArCQkJJmNoaXAtPnJl Z21hcF9pcnEpOw0KPiANCj4gVGhpcyBpcyBqdXN0IG9uZSBjYWxsLiAgSnVzdCBwbGFjZSB0aGF0 IGNhbGwgaW50bw0KPiBkYTkwNjJfZGV2aWNlX2luaXQoKSBhbmQgcmlkIHRoaXMgZnVuY3Rpb24u DQo+IA0KDQpXaWxsIHJlZmFjdG9yIHRoaXMgcGFydC4NCg0KPiA+ICsJaWYgKHJldCkgew0KPiA+ ICsJCWRldl9lcnIoY2hpcC0+ZGV2LCAiRmFpbGVkIHRvIHJlcXVlc3QgSVJRICVkOiAlZFxuIiwN Cj4gPiArCQkJY2hpcC0+Y2hpcF9pcnEsIHJldCk7DQo+ID4gKwkJcmV0dXJuIHJldDsNCj4gPiAr CX0NCj4gPiArDQo+ID4gKwlyZXR1cm4gMDsNCj4gPiArfQ0KPiA+ICsNCj4gPiArdm9pZCBkYTkw NjJfaXJxX2V4aXQoc3RydWN0IGRhOTA2MiAqY2hpcCkNCj4gPiArew0KPiA+ICsJcmVnbWFwX2Rl bF9pcnFfY2hpcChjaGlwLT5jaGlwX2lycSwgY2hpcC0+cmVnbWFwX2lycSk7DQo+IA0KPiBBZ2Fp biwgdGhpcyBpcyBhYnN0cmFjdGlvbiBmb3IgdGhlIHNha2Ugb2YgYWJzdHJhY3Rpb24uDQo+IA0K DQpJIHdpbGwgZXJhc2UgYWxsIGFic3RyYWN0aW9ucyBhbmQgLi4uDQogIC0gUmVmYWN0b3IgZGE5 MDYyX2RldmljZV9pbml0KCkgYW5kIGRhOTA2Ml9pcnFfaW5pdCgpIGludG8gcHJvYmUNCiAgLSBB ZGQgbmV3IGZ1bmN0aW9uIGdldF9kZXZpY2VfdHlwZSgpIGZvciB2YXJpYW50IGFuZCBkZXZpY2Vf aWQgaW5mb3JtYXRpb24NCg0KWy4uLl0NCg0KPiA+ICtzdGF0aWMgc3RydWN0IHJlc291cmNlIGRh OTA2Ml93ZHRfcmVzb3VyY2VzW10gPSB7DQo+ID4gKwl7DQo+ID4gKwkJLm5hbWUJPSAiV0RHX1dB Uk4iLA0KPiA+ICsJCS5zdGFydAk9IERBOTA2Ml9JUlFfV0RHX1dBUk4sDQo+ID4gKwkJLmVuZAk9 IERBOTA2Ml9JUlFfV0RHX1dBUk4sDQo+ID4gKwkJLmZsYWdzCT0gSU9SRVNPVVJDRV9JUlEsDQo+ ID4gKwl9LA0KPiA+ICt9Ow0KPiANCj4gQ29udmVydCBhbGwgb2YgdGhlc2UgdG8gb25lbGluZXJz IHVzaW5nIERFRklORV9SRVNfKiBtYWNyb3MuDQoNCkRvbmUuDQoNCj4gPiArDQo+ID4gK3N0YXRp YyBpcnFyZXR1cm5fdCBkYTkwNjJfdmRkX3dhcm5fZXZlbnQoaW50IGlycSwgdm9pZCAqZGF0YSkN Cj4gPiArew0KPiA+ICsJc3RydWN0IGRhOTA2MiAqaHcgPSBkYXRhOw0KPiA+ICsNCj4gPiArCWRl dl9lcnIoaHctPmRldiwgIlZERCB3YXJuaW5nIGluZGljYXRvclxuIik7DQo+IA0KPiBJcyBpdCBh biBlcnJvcj8gIERvZXNuJ3QgbG9vayBsaWtlIGl0Lg0KPiANCj4gPiArCXJldHVybiBJUlFfSEFO RExFRDsNCj4gPiArfQ0KDQpTZWUgbGF0ZXIgb24gKHRoaXMgaXMgdG8gYmUgZXJhc2VkKQ0KDQpb Li4uXQ0KDQo+ID4gK3N0YXRpYyBpbnQgZGE5MDYyX2NsZWFyX2ZhdWx0X2xvZyhzdHJ1Y3QgZGE5 MDYyICpjaGlwKQ0KPiA+ICt7DQo+ID4gKwlpbnQgcmV0ID0gMDsNCj4gDQo+IE5vIG5lZWQgdG8g cHJlLWluaXRpYWxpc2UuDQo+IA0KPiA+ICsJaW50IGZhdWx0X2xvZyA9IDA7DQo+IA0KPiBBcyBh Ym92ZS4NCj4gDQo+ID4gKwlyZXQgPSByZWdtYXBfcmVhZChjaGlwLT5yZWdtYXAsIERBOTA2MkFB X0ZBVUxUX0xPRywNCj4gJmZhdWx0X2xvZyk7DQo+ID4gKwlpZiAocmV0IDwgMCkgew0KPiA+ICsJ CWRldl9lcnIoY2hpcC0+ZGV2LCAiQ2Fubm90IHJlYWQgRkFVTFRfTE9HLlxuIik7DQo+ID4gKwkJ cmV0ID0gLUVJTzsNCj4gDQo+IEp1c3QgcmV0dXJuIGFuZCByaWQgdGhlIGVsc2UuDQo+IA0KPiA+ ICsJfSBlbHNlIHsNCj4gPiArCQlpZiAoZmF1bHRfbG9nKSB7DQo+ID4gKwkJCWlmIChmYXVsdF9s b2cgJiBEQTkwNjJBQV9UV0RfRVJST1JfTUFTSykNCg0KSSB3aWxsIHJlZmFjdG9yIGRhOTA2Ml9j bGVhcl9mYXVsdF9sb2coKSB0byByZW1vdmUgaW5jb25zaXN0ZW5jaWVzDQoNCj4gPiAraW50IGRh OTA2Ml9kZXZpY2VfaW5pdChzdHJ1Y3QgZGE5MDYyICpjaGlwLCB1bnNpZ25lZCBpbnQgaXJxKQ0K PiANCj4gTm8gbmVlZCB0byBwYXNzIGlycSwgYXMgaXQncyBwYXJ0IG9mIGNoaXAuDQo+IA0KDQpU aGlzIGlzIGdvaW5nIHRvIGJlIGVyYXNlZCBhcyBwYXJ0IG9mIGEgbGF0ZXIgY29tbWVudCAoc2Vl IGJlbG93KQ0KDQpbLi4uXQ0KDQo+ID4gKwlyZXQgPSBkYTkwNjJfY2xlYXJfZmF1bHRfbG9nKGNo aXApOw0KPiA+ICsJaWYgKHJldCA8IDApDQo+ID4gKwkJZGV2X2VycihjaGlwLT5kZXYsICJDYW5u b3QgY2xlYXIgZmF1bHQgbG9nXG4iKTsNCj4gDQo+IElmIHRoaXMgaXMgYW4gZXJyb3IsIHlvdSBt dXN0IHJldHVybi4gIElmIGl0J3MganVzdCBhIHdhcm5pbmcgdGhlbiB1c2UNCj4gZGV2X3dhcm4o KS4NCg0KV2lsbCB1c2UgZGV2X3dhcm4gaW5zdGVhZA0KDQo+IA0KPiA+ICsJY2hpcC0+aXJxX2Jh c2UgPSAtMTsNCj4gDQo+IFdoeSBhcmUgeW91IHByZS1pbml0aWFsaXNpbmc/DQo+IA0KPiA+ICsJ Y2hpcC0+Y2hpcF9pcnEgPSBpcnE7DQo+IA0KPiBZb3UgYWxyZWFkeSBhc3NpZ25lZCBpcnEgdG8g Y2hpcF9pcnEuDQo+IA0KPiA+ICsJcmV0ID0gcmVnbWFwX3JlYWQoY2hpcC0+cmVnbWFwLCBEQTkw NjJBQV9ERVZJQ0VfSUQsDQo+ICZkZXZpY2VfaWQpOw0KPiA+ICsJaWYgKHJldCA8IDApIHsNCj4g PiArCQlkZXZfZXJyKGNoaXAtPmRldiwgIkNhbm5vdCByZWFkIGNoaXAgSUQuXG4iKTsNCj4gPiAr CQlyZXR1cm4gLUVJTzsNCj4gPiArCX0NCj4gPiArCWlmIChkZXZpY2VfaWQgIT0gREE5MDYyX1BN SUNfREVWSUNFX0lEKSB7DQo+ID4gKwkJZGV2X2VycihjaGlwLT5kZXYsICJJbnZhbGlkIGRldmlj ZSBJRDogMHglMDJ4XG4iLA0KPiBkZXZpY2VfaWQpOw0KPiA+ICsJCXJldHVybiAtRU5PREVWOw0K PiA+ICsJfQ0KPiA+ICsNCj4gPiArCXJldCA9IHJlZ21hcF9yZWFkKGNoaXAtPnJlZ21hcCwgREE5 MDYyQUFfVkFSSUFOVF9JRCwNCj4gJnZhcmlhbnRfaWQpOw0KPiA+ICsJaWYgKHJldCA8IDApIHsN Cj4gPiArCQlkZXZfZXJyKGNoaXAtPmRldiwgIkNhbm5vdCByZWFkIGNoaXAgdmFyaWFudCBpZC5c biIpOw0KPiA+ICsJCXJldHVybiAtRUlPOw0KPiA+ICsJfQ0KPiA+ICsNCj4gPiArCWRldl9pbmZv KGNoaXAtPmRldiwNCj4gPiArCQkgIkRldmljZSBkZXRlY3RlZCAoZGV2aWNlLUlEOiAweCUwMlgs IHZhci1JRDogMHglMDJYKVxuIiwNCj4gPiArCQkgZGV2aWNlX2lkLCB2YXJpYW50X2lkKTsNCj4g DQo+IFByb2JhYmx5IGJlc3QgdG8gcHV0IHRoaXMgYXQgdGhlIGVuZC4NCg0KSSBoYXZlIGxlZnQg dGhpcyBwYXJ0IGluIGF0IHRoaXMgcG9pbnQuIFRoZSBkZXZpY2VfaWQgYW5kIHZhcmlhbnRfaWQg YXJlIG5vdA0KcGFzc2VkIGFueSBmdXJ0aGVyIHRoYW4gdGhpcyBmdW5jdGlvbiBmb3IgdGhlIG1v bWVudC4NCg0KPiANCj4gPiArCXZhcmlhbnRfbXJjID0gKHZhcmlhbnRfaWQgJiBEQTkwNjJBQV9N UkNfTUFTSykgPj4NCj4gREE5MDYyQUFfTVJDX1NISUZUOw0KPiA+ICsNCj4gPiArCWlmICh2YXJp YW50X21yYyA8IERBOTA2Ml9QTUlDX1ZBUklBTlRfTVJDX0FBKSB7DQo+ID4gKwkJZGV2X2Vycihj aGlwLT5kZXYsDQo+ID4gKwkJCSJDYW5ub3Qgc3VwcG9ydCB2YXJpYW50IE1SQzogMHglMDJYXG4i LA0KPiB2YXJpYW50X21yYyk7DQo+ID4gKwkJcmV0dXJuIC1FTk9ERVY7DQo+ID4gKwl9DQo+IA0K PiBJJ2QgcHV0IGFsbCBvZiB0aGUgZGV2aWNlL3ZhcmlhbnQgc3R1ZmYgaXQgaXRzIG93biBmdW5j dGlvbiwgdGhlbiBtb3ZlDQo+IGV2ZXJ5dGhpbmcgZWxzZSBpbnRvIHByb2JlKCkgYW5kIHJlbW92 ZSBkYTkwNjJfZGV2aWNlX2luaXQoKS4NCj4gDQoNCkkgd2lsbCBtYWtlIGEgbmV3IGZ1bmN0aW9u IGdldF9kZXZpY2VfdHlwZSgpIGZvciB2YXJpYW50IGFuZCBkZXZpY2VfaWQgaW5mb3JtYXRpb24N Cg0KPiA+ICsJcmV0ID0gZGE5MDYyX2lycV9pbml0KGNoaXApOw0KPiA+ICsJaWYgKHJldCkgew0K PiA+ICsJCWRldl9lcnIoY2hpcC0+ZGV2LCAiQ2Fubm90IGluaXRpYWxpemUgaW50ZXJydXB0cy5c biIpOw0KPiA+ICsJCXJldHVybiByZXQ7DQo+ID4gKwl9DQo+IA0KPiBSZW1vdmUgdGhpcyBmdW5j dGlvbiBhbmQgY2FsbCByZWdtYXBfYWRkX2lycV9jaGlwKCkgZnJvbSBoZXJlLg0KPiANCg0KWWVz LiBJIHdpbGwgZG8gdGhhdC4NCg0KPiA+ICsJY2hpcC0+aXJxX2Jhc2UgPSByZWdtYXBfaXJxX2No aXBfZ2V0X2Jhc2UoY2hpcC0+cmVnbWFwX2lycSk7DQo+ID4gKw0KPiA+ICsJcmV0ID0gbWZkX2Fk ZF9kZXZpY2VzKGNoaXAtPmRldiwgLTEsIGRhOTA2Ml9kZXZzLA0KPiANCj4gVXNlIFBMQVRGT1JN X0RFVklEX05PTkUgaW5zdGVhZC4NCj4gDQo+ID4gKwkJCSAgICAgIEFSUkFZX1NJWkUoZGE5MDYy X2RldnMpLCBOVUxMLCBjaGlwLT5pcnFfYmFzZSwNCj4gPiArCQkJICAgICAgTlVMTCk7DQo+IA0K PiBVc3VhbGx5IGNoaWxkIGRldmljZXMgYXJlIHByb2JlZCBhdCB0aGUgZW5kIGkuZS4gdGhlIGxh c3QgdGhpbmcgaW4NCj4gcHJvYmUoKS4NCj4gDQoNCk9rLiBJIHdpbGwgbW92ZSB0aGF0IHRvIHRo ZSBlbmQgYW5kIHVzZSB0aGUgUExBVEZPUk1fREVWSURfTk9ORSBpbnN0ZWFkDQpvZiB0aGUgZXhw bGljaXQgLTEgdmFsdWUuDQoNCj4gPiArCWlmIChyZXQpIHsNCj4gPiArCQlkZXZfZXJyKGNoaXAt PmRldiwgIkNhbm5vdCBhZGQgTUZEIGNlbGxzXG4iKTsNCj4gDQo+ICJDYW5ub3QgcmVnaXN0ZXIg Y2hpbGQgZGV2aWNlcyIuDQo+IA0KDQpXaWxsIHJlcGxhY2UgdGhhdC4NCg0KWy4uLl0NCg0KPiA+ ICsJLyogVkREIFdBUk4gZXZlbnQgc3VwcG9ydCAqLw0KPiA+ICsJaXJxX3ZkZF93YXJuID0gcmVn bWFwX2lycV9nZXRfdmlycShjaGlwLT5yZWdtYXBfaXJxLA0KPiA+ICsJCQkJCSAgIERBOTA2Ml9J UlFfVkREX1dBUk4pOw0KPiA+ICsJaWYgKGlycV92ZGRfd2FybiA8IDApIHsNCj4gPiArCQlkZXZf ZXJyKGNoaXAtPmRldiwgIkZhaWxlZCB0byBnZXQgSVJRLlxuIik7DQo+ID4gKwkJcmV0dXJuIGly cV92ZGRfd2FybjsNCj4gPiArCX0NCj4gPiArCWNoaXAtPmlycV92ZGRfd2FybiA9IGlycV92ZGRf d2FybjsNCj4gPiArDQo+ID4gKwlyZXQgPSBkZXZtX3JlcXVlc3RfdGhyZWFkZWRfaXJxKGNoaXAt PmRldiwgaXJxX3ZkZF93YXJuLA0KPiA+ICsJCQkJCU5VTEwsIGRhOTA2Ml92ZGRfd2Fybl9ldmVu dCwNCj4gPiArCQkJCQlJUlFGX1RSSUdHRVJfTE9XIHwgSVJRRl9PTkVTSE9ULA0KPiA+ICsJCQkJ CSJWRERfV0FSTiIsIGNoaXApOw0KPiA+ICsJaWYgKHJldCkgew0KPiA+ICsJCWRldl93YXJuKGNo aXAtPmRldiwNCj4gPiArCQkJICJGYWlsZWQgdG8gcmVxdWVzdCBWRERfV0FSTiBJUlEuXG4iKTsN Cj4gPiArCQljaGlwLT5pcnFfdmRkX3dhcm4gPSAtRU5YSU87DQo+ID4gKwl9DQo+IA0KPiBUaGlz IGxvb2tzIGxpa2UgYSBsb3Qgb2YgY29kZSwgd2hpY2ggZG9lc24ndCByZWFsbHkgZG8gYW55dGhp bmcuICBXaGF0DQo+IGlzIGEgVkREIHdhcm5pbmcgaW5kaWNhdG9yIGFueXdheT8NCj4gDQoNCkkg d2lsbCByZW1vdmUgdGhpcy4NCg0KVGhlIElSUSBoYW5kbGVyIGRhOTA2Ml92ZGRfd2Fybl9ldmVu dCgpIC0tIHNlZSBlYXJsaWVyIGFib3ZlIC0tIGRvZXMNCm5vdCBjdXJyZW50bHkgZG8gYW55dGhp bmcgYXBhcnQgZnJvbSBoYW5kbGUgdGhlIElSUSB0aGF0IHdhcyByZXF1ZXN0ZWQNCmhlcmUuIEl0 IHByaW50cyBhIHN0YXRlbWVudCB0byBzYXkgdGhlIG1haW4gUE1JQyB2b2x0YWdlIHN1cHBseSBk cm9wcGVkDQpiZWxvdyBhIGRlZmluZWQgdHJpZ2dlciBwb2ludCwgYnV0IGRvZXNuJ3QgYWN0dWFs bHkgZG8gYW55dGhpbmcgdG8gbWl0aWdhdGUNCnRoaXMgcHJvYmxlbS4NCg0KUHJldmlvdXNseSB0 aGlzIFZERF9XQVJOIHdhcyBpbiB0aGUgcmVndWxhdG9yIGRyaXZlciwgaG93ZXZlciBpdCBzaG91 bGQNCmJlIG1hZGUgYXZhaWxhYmxlIGV2ZW4gaWYgdGhlIHJlZ3VsYXRvciBkcml2ZXIgaXMgbm90 IGluc3RhbGxlZCAtLSBzbyBJIGFkZGVkIGl0DQp0byB0aGUgY29yZSBpbnN0ZWFkLg0KDQpJbiBh IHByZXZpb3VzIGRyaXZlciBzdWJtaXNzaW9uIEkgaGFkIGEgc2ltaWxhciBwcm9ibGVtLCBhIHdh cm5pbmcgSVJRIHdhcw0KanVzdCBwcmludGluZyB0byB0aGUgY29uc29sZSB0byBzYXkgdGhlcmUg d2FzIGFuIGVycm9yIC0tIHRoZSBoYW5kbGVyIGFuZA0KSVJRIGNvZGUgd2FzIHB1dCBpbiBieSBt ZSBzbyBpdCBjb3VsZCBiZSB1c2VkIGlmIHRoZSBkcml2ZXIgd2FzIHRha2VuIGFuZA0KaW50ZWdy YXRlZCBpbnRvIGEgZnVsbHkgd29ya2luZyBzeXN0ZW0uDQoNCkkgd2FzIGFza2VkIHRvIHJlbW92 ZSBpdCBpbiB0aGUgb3RoZXIgZHJpdmVyIC0tIGFuZCBJIGhhdmUgZG9uZSB0aGUgc2FtZQ0KaGVy ZSBmb3Igbm93LiBJIGNhbiBhbHdheXMgYWRkIGl0IGJhY2sgbGF0ZXIuDQoNCj4gPiArCXJldHVy biByZXQ7DQo+ID4gK30NCj4gPiArDQo+ID4gK3ZvaWQgZGE5MDYyX2RldmljZV9leGl0KHN0cnVj dCBkYTkwNjIgKmNoaXApDQo+ID4gK3sNCj4gPiArCW1mZF9yZW1vdmVfZGV2aWNlcyhjaGlwLT5k ZXYpOw0KPiA+ICsJZGE5MDYyX2lycV9leGl0KGNoaXApOw0KPiA+ICt9DQo+IA0KPiBBbm90aGVy IHNlZW1pbmdseSBwb2ludGxlc3MgYWJzdHJhY3Rpb24uICBXaHkgZG9uJ3QgeW91IGRvIHRoaXMg aW4NCj4gcmVtb3ZlKCk/DQo+IA0KDQpXaWxsIGRvIHRoYXQgYXMgcGFydCBvZiB0aGUgYWJzdHJh Y3Rpb24gY2xlYW4gdXANCg0KWy4uLl0NCg0KPiA+ICsNCj4gPiArc3RhdGljIHN0cnVjdCByZWdt YXBfY29uZmlnIGRhOTA2Ml9yZWdtYXBfY29uZmlnID0gew0KPiA+ICsJLnJlZ19iaXRzID0gOCwN Cj4gPiArCS52YWxfYml0cyA9IDgsDQo+ID4gKwkucmFuZ2VzID0gZGE5MDYyX3JhbmdlX2NmZywN Cj4gPiArCS5udW1fcmFuZ2VzID0gQVJSQVlfU0laRShkYTkwNjJfcmFuZ2VfY2ZnKSwNCj4gPiAr CS5tYXhfcmVnaXN0ZXIgPSBEQTkwNjJBQV9DT05GSUdfSUQsDQo+ID4gKwkuY2FjaGVfdHlwZSA9 IFJFR0NBQ0hFX1JCVFJFRSwNCj4gPiArfTsNCj4gPiArDQo+ID4gK3N0YXRpYyBjb25zdCBzdHJ1 Y3Qgb2ZfZGV2aWNlX2lkIGRhOTA2Ml9kdF9pZHNbXSA9IHsNCj4gPiArCXsgLmNvbXBhdGlibGUg PSAiZGxnLGRhOTA2MiIsIH0sDQo+ID4gKwl7IH0NCj4gPiArfTsNCj4gPiArTU9EVUxFX0RFVklD RV9UQUJMRShvZiwgZGE5MDYyX2R0X2lkcyk7DQo+IA0KPiBNb3ZlIHRoaXMganVzdCBhYm92ZSB3 aGVyZSBpdCdzIHRvIGJlIHVzZWQgaS5lLiBkb3duIHRvIHRoZSBib3R0b20uDQo+IA0KDQpXaWxs IG1vdmUgdGhhdCB0byB0aGUgZW5kIG9mIHRoZSBmaWxlLg0KDQo+ID4gK3N0YXRpYyBpbnQgZGE5 MDYyX2kyY19wcm9iZShzdHJ1Y3QgaTJjX2NsaWVudCAqaTJjLA0KPiA+ICsJY29uc3Qgc3RydWN0 IGkyY19kZXZpY2VfaWQgKmlkKQ0KPiA+ICt7DQo+ID4gKwlzdHJ1Y3QgZGE5MDYyICpjaGlwOw0K PiA+ICsJaW50IHJldDsNCj4gPiArDQo+ID4gKwljaGlwID0gZGV2bV9remFsbG9jKCZpMmMtPmRl diwgc2l6ZW9mKHN0cnVjdCBkYTkwNjIpLCBHRlBfS0VSTkVMKTsNCj4gDQo+IHNpemVvZigqY2hp cCkNCj4gDQo+ID4gKwlpZiAoY2hpcCA9PSBOVUxMKQ0KPiANCj4gaWYgKCFjaGlwKQ0KDQpXaWxs IGRvIHRoYXQuDQoNCj4gPiArCWRhOTA2Ml9yZWdtYXBfY29uZmlnLnJkX3RhYmxlID0gJmRhOTA2 Ml9hYV9yZWFkYWJsZV90YWJsZTsNCj4gPiArCWRhOTA2Ml9yZWdtYXBfY29uZmlnLndyX3RhYmxl ID0gJmRhOTA2Ml9hYV93cml0ZWFibGVfdGFibGU7DQo+ID4gKwlkYTkwNjJfcmVnbWFwX2NvbmZp Zy52b2xhdGlsZV90YWJsZSA9ICZkYTkwNjJfYWFfdm9sYXRpbGVfdGFibGU7DQo+IA0KPiBXaHkg YXJlIHlvdSBkb2luZyB0aGlzIGhlcmUgaW5zdGVhZCBvZiBpbnNpZGUNCj4gJ3N0cnVjdCByZWdt YXBfY29uZmlnIGRhOTA2Ml9yZWdtYXBfY29uZmlnJyBhYm92ZT8NCj4gDQoNCk9rLi4gSSB3aWxs IGZpeCB0aGF0IHBhcnQgaW50byB0aGUgbWFpbiBzdHJ1Y3R1cmUuDQoNCg0KPiA+ICtzdGF0aWMg Y29uc3Qgc3RydWN0IGkyY19kZXZpY2VfaWQgZGE5MDYyX2kyY19pZFtdID0gew0KPiA+ICsJeyJk YTkwNjIiLCAwfSwNCj4gDQo+IFdoaXRlIHNwYWNlIGRpc2NpcGxpbmUuDQo+IA0KDQpJIGhhdmUg cHV0IHNvbWUgd2hpdGUtc3BhY2VzIGluIHRoZSBjb2RlLg0KSSB0aGluayB0aGlzOg0KDQp7ICJk YTkwNjIiLCAwIH0sDQp7IH0sDQoNClsuLi5dDQoNCj4gDQo+ID4gK01PRFVMRV9ERVNDUklQVElP TigiQ09SRSBkZXZpY2UgZHJpdmVyIGZvciBEaWFsb2cgREE5MDYyIik7DQo+IA0KPiBzL0NPUkUv Q29yZS8NCj4gDQoNClllcy4NCg0KPiA+ICtNT0RVTEVfQVVUSE9SKCJTIFR3aXNzIDxzdHdpc3Mu b3BlbnNvdXJjZUBkaWFzZW1pLmNvbT4iKTsNCj4gDQo+IEZ1bGwgbmFtZXMgcGxlYXNlLg0KPiAN Cg0KT2suIA0KDQpbLi4uXQ0KDQo+ID4gZGlmZiAtLWdpdCBhL2luY2x1ZGUvbGludXgvbWZkL2Rh OTA2Mi9jb3JlLmgNCj4gYi9pbmNsdWRlL2xpbnV4L21mZC9kYTkwNjIvY29yZS5oDQo+ID4gbmV3 IGZpbGUgbW9kZSAxMDA2NDQNCj4gPiBpbmRleCAwMDAwMDAwLi4wYjE3ODkxDQo+ID4gLS0tIC9k ZXYvbnVsbA0KPiA+ICsrKyBiL2luY2x1ZGUvbGludXgvbWZkL2RhOTA2Mi9jb3JlLmgNCj4gPiBA QCAtMCwwICsxLDYyIEBADQo+ID4gKy8qDQo+ID4gKyAqIGNvcmUuaCAtIENPUkUgSCBmb3IgREE5 MDYyDQo+IA0KPiBXaGF0IGRvZXMgdGhpcyBhZGQ/DQo+IA0KDQpSZW1vdmVkIHRoZSB0b3AgbGlu ZSBjb21wbGV0ZWx5DQoNClsuLi5dDQoNCj4gPiArDQo+ID4gK3N0cnVjdCBkYTkwNjIgew0KPiA+ ICsJc3RydWN0IGRldmljZSAqZGV2Ow0KPiA+ICsJdW5zaWduZWQgY2hhciBkZXZpY2VfaWQ7DQo+ ID4gKwl1bnNpZ25lZCBjaGFyIHZhcmlhbnRfbXJjOw0KPiA+ICsJc3RydWN0IHJlZ21hcCAqcmVn bWFwOw0KPiA+ICsJaW50IGNoaXBfaXJxOw0KPiA+ICsJdW5zaWduZWQgaW50IGlycV9iYXNlOw0K PiA+ICsJc3RydWN0IHJlZ21hcF9pcnFfY2hpcF9kYXRhICpyZWdtYXBfaXJxOw0KPiA+ICsJaW50 IGlycV92ZGRfd2FybjsNCj4gPiArfTsNCj4gDQo+IEFyZSBhbGwgb2YgdGhlc2UgdXNlZCBieSBj aGlsZCBkZXZpY2VzPw0KPiANCg0KQWZ0ZXIgSSBkaWQgdGhlIHJlZmFjdG9yaW5nIHdvcmsgZGVz Y3JpYmVkIGFib3ZlLCBJIGNhbiByZW1vdmUgcXVpdGUgYSBmZXcgb2YgdGhlbS4NCg0KdW5zaWdu ZWQgY2hhciBkZXZpY2VfaWQ7DQp1bnNpZ25lZCBjaGFyIHZhcmlhbnRfbXJjOw0KDQpUaGVzZSBh cmUgbm90IGN1cnJlbnRseSB1c2VkIGluIGFueSBjaGlsZCBkZXZpY2UgZHJpdmVycyAtLSBiZWNh dXNlIHRoZXJlIGlzIG9ubHkNCm9uZSBkZXZpY2Ugb2YgdGhpcyB0eXBlLg0KDQppbnQgY2hpcF9p cnE7DQp1bnNpZ25lZCBpbnQgaXJxX2Jhc2U7DQoNClRoZSBjaGlwX2lkIGlzIG5vdyBsb2NhbCB0 byB0aGUgcHJvYmUgYW5kIGNhbiByZXBsYWNlZCB3aXRoIGkyYy0+aXJxIGluIHRoYXQgZnVuY3Rp b24NCkl0J3Mgb25seSB1c2UgaXMgZHVyaW5nIGNsZWFuLXVwIGFuZCBpMmMtPmlycSBjYW4gYmUg dXNlZCB0aGVyZSBhbHNvLg0KaXJxX2Jhc2UgaXMgbG9jYWwgdG8gcHJvYmUoKSBhbHNvLCBzbyBJ IGhhdmUgbWFkZSBpdCBhdXRvbWF0aWMuDQoNCj4gPiArCWludCBpcnFfdmRkX3dhcm47DQoNCkkg aGF2ZSByZW1vdmVkIHRoZSB2ZGRfd2FybiBjYXBhYmlsaXR5IGluIHRoaXMgcGF0Y2guDQoNCj4g PiAraW50IGRhOTA2Ml9kZXZpY2VfaW5pdChzdHJ1Y3QgZGE5MDYyICosIHVuc2lnbmVkIGludCk7 DQo+ID4gK2ludCBkYTkwNjJfaXJxX2luaXQoc3RydWN0IGRhOTA2MiAqKTsNCj4gPiArDQo+ID4g K3ZvaWQgZGE5MDYyX2RldmljZV9leGl0KHN0cnVjdCBkYTkwNjIgKik7DQo+ID4gK3ZvaWQgZGE5 MDYyX2lycV9leGl0KHN0cnVjdCBkYTkwNjIgKik7DQo+IA0KPiBXaHkgYXJlIHRoZXNlIHJlcXVp cmVkPw0KPiANCg0KR29uZS4NCg0KWy4uLl0NCg0KPiA+IGRpZmYgLS1naXQgYS9pbmNsdWRlL2xp bnV4L21mZC9kYTkwNjIvcmVnaXN0ZXJzLmgNCj4gYi9pbmNsdWRlL2xpbnV4L21mZC9kYTkwNjIv cmVnaXN0ZXJzLmgNCj4gPiBuZXcgZmlsZSBtb2RlIDEwMDY0NA0KPiA+IGluZGV4IDAwMDAwMDAu LmQwN2MyYmMNCj4gPiAtLS0gL2Rldi9udWxsDQo+ID4gKysrIGIvaW5jbHVkZS9saW51eC9tZmQv ZGE5MDYyL3JlZ2lzdGVycy5oDQoNClsuLi5dDQoNCj4gPiArLyoNCj4gPiArICogUmVnaXN0ZXJz DQo+ID4gKyAqLw0KPiANCj4gUmVhbGx5PyA7KQ0KPiANCj4gPiArI2RlZmluZSBEQTkwNjJBQV9Q QUdFX0NPTgkJMHgwMDANCj4gPiArI2RlZmluZSBEQTkwNjJBQV9TVEFUVVNfQQkJMHgwMDENCj4g PiArI2RlZmluZSBEQTkwNjJBQV9TVEFUVVNfQgkJMHgwMDINCg0KWy4uLl0NCg0KPiA+ICsNCj4g PiArLyoNCj4gPiArICogQml0IGZpZWxkcw0KPiA+ICsgKi8NCj4gPiArDQo+ID4gKy8qIERBOTA2 MkFBX1BBR0VfQ09OID0gMHgwMDAgKi8NCj4gPiArI2RlZmluZSBEQTkwNjJBQV9QQUdFX1NISUZU CQkwDQo+ID4gKyNkZWZpbmUgREE5MDYyQUFfUEFHRV9NQVNLCQkoMHgzZiA8PCAwKQ0KPiA+ICsj ZGVmaW5lIERBOTA2MkFBX1dSSVRFX01PREVfU0hJRlQJNg0KPiA+ICsjZGVmaW5lIERBOTA2MkFB X1dSSVRFX01PREVfTUFTSwkoMHgwMSA8PCA2KQ0KPiANCj4gRm9yIDEgPDwgWCwgeW91IHNob3Vs ZCB1c2UgQklUKFgpLg0KPiANCg0KRm9yIHRoZSB0d28gY29tbWVudHMgYWJvdmUgIlJlZ2lzdGVy cyIgYW5kICJCaXQgZmllbGRzIiBhbmQgdGhlICgxPDx4KQ0KZGVmaW5pdGlvbnMgLi4uDQoNClRo ZSB3aG9sZSBvZiB0aGlzIGZpbGUgaXMgYXV0b21hdGljYWxseSBnZW5lcmF0ZWQgYnkgb3VyIGhh cmR3YXJlIGRlc2lnbmVycw0KSSB3b3VsZCBwcmVmZXIgaXQgaWYgdGhlIHJlZ2lzdGVyIGRlZmlu aXRpb25zIGFuZCBiaXQgZmllbGRzIGFyZSBub3QgYWx0ZXJlZCB1c2luZw0KdGhlICNkZWZpbmUg QklUKG5yKSAoMVVMPDwobnIpKSBtYWNybyBhbmQgdGhlIGNvbW1lbnRzIHJlbW92ZWQgYmVjYXVz ZQ0Kd2UgaGF2ZSBzY3JpcHRzIHRoYXQgY2FuIGJlIHVzZWQgdG8gY2hlY2sgdGhpcyBmaWxlIGF1 dG9tYXRpY2FsbHkuDQoNCkFsc28gaWYgdGhlIHJlZ2lzdGVyIG1hcCBpcyBldmVyIHVwZGF0ZWQs IHRoZW4gaXQgd2lsbCBiZSBlYXNpZXIgZm9yIG1lIHRvIGRpZmYNCnRoZSBuZXcgZGVsaXZlcmVk IHJlZ2lzdGVyIGFuZCBiaXQgZmllbGQgZGVmaW5pdGlvbnMgd2l0aCB0aGUgb2xkIG9uZS4NCg0K TXkgcHJlZmVyZW5jZSB3b3VsZCBiZSBub3QgdG8gY2hhbmdlIHRoaXMgaGVhZGVyIGZpbGUuDQoN ClsuLi5dDQoNCklmIEkgaGF2ZSBsZWZ0IGFueXRoaW5nIG91dCBwbGVhc2UgbGV0IG1lIGtub3cu DQoNCnJlZ2FyZHMsDQpTdGV2ZQ0K
--
To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
RE: [PATCH V3 1/4] mfd: da9062: DA9062 MFD core driver
In reply to Lee JonesQuoted 6 lines · Steve Twiss
To: 'Lee Jones'
Subject: RE: [PATCH V3 1/4] mfd: da9062: DA9062 MFD core driverHi Lee,
I will refactor a lot of the driver and implement your changes as requested.
Hi Lee,
I realise this is a busy kernel time for you, as ever, so this is just to see if I am missing anything with the reply I sent about the requested alterations a couple of weeks ago.
It relates to the addition of support for the Dialog DA9062 Power Management IC
- https://lkml.org/lkml/2015/5/28/358
The main changes requested (for the core MFD) can be found in here:
- https://lkml.org/lkml/2015/5/28/359
I believe that the only two major differences with your previous comments were those relating to the interrupt handler da9062_vdd_warn_event() -- which has been erased, and the header file -- which I would prefer the to remain [mostly] untouched if possible.
The reasons for these two differences are described below:
Quoted 35 lines
Quoted 13 lines
Quoted 13 lines
drivers/mfd/da9062-core.c new file +13
-0+/* 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);Quoted 9 lines
drivers/mfd/da9062-core.c line 335
Quoted 5 lines
drivers/mfd/da9062-core.c new file +5
-0+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?
I will remove this.
The IRQ handler
da9062_vdd_warn_event()-- see earlier above -- does not currently do anything apart from handle the IRQ that was requested here. It prints a statement to say the main PMIC voltage supply dropped below a defined trigger point, but doesn't actually do anything to mitigate this problem.Previously this
VDD_WARNwas in the regulator driver, however it should be made available even if the regulator driver is not installed -- so I added it to the core instead.
Quoted 8 lines
In a previous driver submission I had a similar problem, a warning IRQ was just printing to the console to say there was an error -- the handler and IRQ code was put in by me so it could be used if the driver was taken and integrated into a fully working system.
I was asked to remove it in the other driver -- and I have done the same here for now. I can always add it back later.
And
Quoted 43 lines
Quoted 6 lines
Review
Quoted 1 line
(quoted diff) +0
-0
b/include/linux/mfd/da9062/registers.hQuoted 4 lines
new file mode 100644
index 0000000..d07c2bcinclude/linux/mfd/da9062/registers.h new file +0
-0[...]
Quoted 9 lines
Quoted 3 lines
+/*
+ * Registers
+ */Really? ;)
Quoted 3 lines
include/linux/mfd/da9062/registers.h new file +3
-0+#define DA9062AA_PAGE_CON 0x000 +#define DA9062AA_STATUS_A 0x001 +#define DA9062AA_STATUS_B 0x002[...]
Quoted 4 lines
Quoted 4 lines
include/linux/mfd/da9062/registers.h new file +4
-0+ +/* +* Bit fields +*/ Quoted 9 lines
include/linux/mfd/da9062/registers.h line 166
Quoted 6 lines
include/linux/mfd/da9062/registers.h new file +6
-0+ +/* 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).
For the two comments above "Registers" and "Bit fields" and the (1<<x) definitions ...
The whole of this file is automatically generated by our hardware designers I would prefer it if the register definitions and bit fields are not altered using the #define BIT(nr) (1UL<<(nr)) macro and the comments removed because we have scripts that can be used to check this file automatically.
Quoted 6 lines
Also if the register map is ever updated, then it will be easier for me to diff the new delivered register and bit field definitions with the old one.
My preference would be not to change this header file.
[...]
If these last two things are a problem can you please let me know.
regards,
Steve
Re: [PATCH V3 1/4] mfd: da9062: DA9062 MFD core driver
Quoted 7 lines · Opensource [Steve Twiss]
Quoted 5 lines · Steve Twiss
To: 'Lee Jones'
Subject: RE: [PATCH V3 1/4] mfd: da9062: DA9062 MFD core driverHi Lee,
Quoted 7 lines
Quoted 1 line
I will refactor a lot of the driver and implement your changes as requested.
Hi Lee,
I realise this is a busy kernel time for you, as ever, so this is just to see if I am missing anything with the reply I sent about the requested alterations a couple of weeks ago.
Looks like I missed v4. It's laying in my inbox 'read' and not marked as 'important', goes against my usual process. I'll take a look now.
Signature · 8 lines
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 [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH V3 1/4] mfd: da9062: DA9062 MFD core driver
In reply to Opensource [Steve Twiss]Quoted 43 lines
Quoted 43 lines
Quoted 6 lines
Review
Quoted 1 line
(quoted diff) +0
-0
b/include/linux/mfd/da9062/registers.hQuoted 4 lines
new file mode 100644
index 0000000..d07c2bcinclude/linux/mfd/da9062/registers.h new file +0
-0[...]
Quoted 9 lines
Quoted 3 lines
+/*
+ * Registers
+ */Really? ;)
Quoted 3 lines
include/linux/mfd/da9062/registers.h new file +3
-0+#define DA9062AA_PAGE_CON 0x000 +#define DA9062AA_STATUS_A 0x001 +#define DA9062AA_STATUS_B 0x002[...]
Quoted 4 lines
Quoted 4 lines
include/linux/mfd/da9062/registers.h new file +4
-0+ +/* +* Bit fields +*/ Quoted 9 lines
include/linux/mfd/da9062/registers.h line 166
Quoted 6 lines
include/linux/mfd/da9062/registers.h new file +6
-0+ +/* 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).
For the two comments above "Registers" and "Bit fields" and the (1<<x) definitions ...
The whole of this file is automatically generated by our hardware designers I would prefer it if the register definitions and bit fields are not altered using the #define BIT(nr) (1UL<<(nr)) macro and the comments removed because we have scripts that can be used to check this file automatically.
Quoted 8 lines
Quoted 6 lines
Also if the register map is ever updated, then it will be easier for me to diff the new delivered register and bit field definitions with the old one.
My preference would be not to change this header file.
[...]
If these last two things are a problem can you please let me know.
I'm still not particularly happy with this. Can yo speak to your H/W guys and get them to change their scripts to output sensible header files?
To be honest, it's probably not a blocker for acceptance, but if someone writes a patch next week to change all of the (0x01 << X) lines to start using the BIT() macro, I will accept it. Better to influenced your guys so you are not overly inconvenienced.
FWIW, when upstreaming code, the excuse "someone else wrote it", has
never been a good one to use on the lists. Believe me, I've
tried. ;)
Signature · 8 lines
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 [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
RE: [PATCH V3 1/4] mfd: da9062: DA9062 MFD core driver
T24gMTEgSnVuZSAyMDE1IDA5OjU3IExlZSBKb25lcyB3cm90ZToNCg0KPiBUbzogT3BlbnNvdXJj ZSBbU3RldmUgVHdpc3NdDQo+IFN1YmplY3Q6IFJlOiBbUEFUQ0ggVjMgMS80XSBtZmQ6IGRhOTA2 MjogREE5MDYyIE1GRCBjb3JlIGRyaXZlcg0KPiANCj4gPiA+ID4gPiBkaWZmIC0tZ2l0IGEvaW5j bHVkZS9saW51eC9tZmQvZGE5MDYyL3JlZ2lzdGVycy5oDQoNClsuLi5dDQoNCj4gPiA+ID4gPiAr I2RlZmluZSBEQTkwNjJBQV9XUklURV9NT0RFX1NISUZUCTYNCj4gPiA+ID4gPiArI2RlZmluZSBE QTkwNjJBQV9XUklURV9NT0RFX01BU0sJKDB4MDEgPDwgNikNCj4gPiA+ID4NCj4gPiA+ID4gRm9y IDEgPDwgWCwgeW91IHNob3VsZCB1c2UgQklUKFgpLg0KPiA+ID4gPg0KPiA+ID4NCj4gPiA+IEZv ciB0aGUgdHdvIGNvbW1lbnRzIGFib3ZlICJSZWdpc3RlcnMiIGFuZCAiQml0IGZpZWxkcyIgYW5k IHRoZSAoMTw8eCkNCj4gPiA+IGRlZmluaXRpb25zIC4uLg0KPiA+ID4NCj4gPiA+IFRoZSB3aG9s ZSBvZiB0aGlzIGZpbGUgaXMgYXV0b21hdGljYWxseSBnZW5lcmF0ZWQgYnkgb3VyIGhhcmR3YXJl IGRlc2lnbmVycw0KPiA+ID4gSSB3b3VsZCBwcmVmZXIgaXQgaWYgdGhlIHJlZ2lzdGVyIGRlZmlu aXRpb25zIGFuZCBiaXQgZmllbGRzIGFyZSBub3QgYWx0ZXJlZCB1c2luZw0KPiA+ID4gdGhlICNk ZWZpbmUgQklUKG5yKSAoMVVMPDwobnIpKSBtYWNybyBhbmQgdGhlIGNvbW1lbnRzIHJlbW92ZWQg YmVjYXVzZQ0KPiA+ID4gd2UgaGF2ZSBzY3JpcHRzIHRoYXQgY2FuIGJlIHVzZWQgdG8gY2hlY2sg dGhpcyBmaWxlIGF1dG9tYXRpY2FsbHkuDQo+ID4gPg0KPiA+ID4gQWxzbyBpZiB0aGUgcmVnaXN0 ZXIgbWFwIGlzIGV2ZXIgdXBkYXRlZCwgdGhlbiBpdCB3aWxsIGJlIGVhc2llciBmb3IgbWUgdG8g ZGlmZg0KPiA+ID4gdGhlIG5ldyBkZWxpdmVyZWQgcmVnaXN0ZXIgYW5kIGJpdCBmaWVsZCBkZWZp bml0aW9ucyB3aXRoIHRoZSBvbGQgb25lLg0KPiA+ID4NCj4gPiA+IE15IHByZWZlcmVuY2Ugd291 bGQgYmUgbm90IHRvIGNoYW5nZSB0aGlzIGhlYWRlciBmaWxlLg0KPiA+ID4NCj4gPiA+IFsuLi5d DQo+ID4NCj4gPiBJZiB0aGVzZSBsYXN0IHR3byB0aGluZ3MgYXJlIGEgcHJvYmxlbSBjYW4geW91 IHBsZWFzZSBsZXQgbWUga25vdy4NCg0KSEkgTGVlLA0KDQpUaGFua3MgZm9yIHJlcGx5aW5nIHNv IHF1aWNrbHkuDQoNCj4gSSdtIHN0aWxsIG5vdCBwYXJ0aWN1bGFybHkgaGFwcHkgd2l0aCB0aGlz LiAgQ2FuIHlvIHNwZWFrIHRvIHlvdXIgSC9XDQo+IGd1eXMgYW5kIGdldCB0aGVtIHRvIGNoYW5n ZSB0aGVpciBzY3JpcHRzIHRvIG91dHB1dCBzZW5zaWJsZSBoZWFkZXINCj4gZmlsZXM/DQoNCkFo LiBPay4gDQoNCkZvciBvdXIgc2lkZSwgdGhlIGdlbmVyYXRlZCBoZWFkZXJzIG1pZ2h0IG5vdCBq dXN0IGJlIHVzZWQgZm9yIExpbnV4LiBJJ3ZlDQpqdXN0IGRpc2N1c3NlZCB0aGlzIHdpdGggbXkg Y29sbGVhZ3VlcyBhbmQgdGhleSB3aWxsIG5lZWQgaXQgdG8gcmVtYWluLg0KU28gSSBndWVzcyBp bnRlcm5hbGx5IHdlIHdpbGwga2VlcCB0aGUgaGVhZGVycyBsaWtlIHRoaXMsIGJ1dCBhcyBpdCBl bnRlcnMNCm15IHN1Ym1pc3Npb24gcHJvY2VzcyBJIGNhbiBjaGFuZ2UgaXQgZm9yIHRoZSBMaW51 eCBjb21tdW5pdHkuDQogDQo+IFRvIGJlIGhvbmVzdCwgaXQncyBwcm9iYWJseSBub3QgYSBibG9j a2VyIGZvciBhY2NlcHRhbmNlLCBidXQgaWYgc29tZW9uZQ0KPiB3cml0ZXMgYSBwYXRjaCBuZXh0 IHdlZWsgdG8gY2hhbmdlIGFsbCBvZiB0aGUgKDB4MDEgPDwgWCkgbGluZXMgdG8NCj4gc3RhcnQg dXNpbmcgdGhlIEJJVCgpIG1hY3JvLCBJIHdpbGwgYWNjZXB0IGl0LiAgQmV0dGVyIHRvIGluZmx1 ZW5jZWQNCj4geW91ciBndXlzIHNvIHlvdSBhcmUgbm90IG92ZXJseSBpbmNvbnZlbmllbmNlZC4N Cg0KWWVwOiBJIHdpbGwgY2hhbmdlIHRoaXMgQklUKCkgbWFjcm8gZm9yIHRoZSBzdWJtaXNzaW9u cyBpbiBmdXR1cmUuDQpEZXBlbmRpbmcgb24gdGhlIG5leHQgc3RlcCwgSSB3aWxsIHNlbmQgYSBw YXRjaCB0byB0aGlzIG9yIHVwZGF0ZSB0aGUgc3VibWlzc2lvbg0KaWYgdGhlcmUgYXJlIGZ1cnRo ZXIgY29tbWVudHMgb24gdGhpcyBwYXRjaCBzZXQuDQoNCj4gRldJVywgd2hlbiB1cHN0cmVhbWlu ZyBjb2RlLCB0aGUgZXhjdXNlICJzb21lb25lIGVsc2Ugd3JvdGUgaXQiLCBoYXMNCj4gbmV2ZXIg YmVlbiBhIGdvb2Qgb25lIHRvIHVzZSBvbiB0aGUgbGlzdHMuICAgQmVsaWV2ZSBtZSwgSSd2ZQ0K PiB0cmllZC4gOykNCg0KaGVoIG9rYXkgOikNCg0KUmVnYXJkcywNClN0ZXZlDQoNCg== --
To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [rtc-linux] RE: [PATCH V3 1/4] mfd: da9062: DA9062 MFD core driver
Hi,
Quoted 14 lines · Opensource [Steve Twiss]
Thanks for replying so quickly.
Quoted 3 lines
I'm still not particularly happy with this. Can yo speak to your H/W guys and get them to change their scripts to output sensible header files?
Ah. Ok.
For our side, the generated headers might not just be used for Linux. I've just discussed this with my colleagues and they will need it to remain. So I guess internally we will keep the headers like this, but as it enters my submission process I can change it for the Linux community.
Quoted 9 lines
Quoted 4 lines
To be honest, it's probably not a blocker for acceptance, but if someone writes a patch next week to change all of the (
0x01<< X) lines to start using theBIT()macro, I will accept it. Better to influenced your guys so you are not overly inconvenienced.Yep: I will change this
BIT()macro for the submissions in future. Depending on the next step, I will send a patch to this or update the submission if there are further comments on this patch set.
Maybe a good solution would be to define BIT() at the top of your generated header so that you simply have to remove it or replace it with the proper include before mainline submission.
Quoted 19 lines
Quoted 3 lines
FWIW, when upstreaming code, the excuse "someone else wrote it", has
never been a good one to use on the lists. Believe me, I've
tried. ;)heh okay :)
Regards,
SteveSignature · 9 lines
-- You received this message because you are subscribed to "rtc-linux". Membership options at http://groups.google.com/group/rtc-linux . Please read http://groups.google.com/group/rtc-linux/web/checklist before submitting a driver. --- You received this message because you are subscribed to the Google Groups "rtc-linux" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. For more options, visit https://groups.google.com/d/optout.
Signature · 3 lines
Alexandre Belloni, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com
RE: [PATCH V3 1/4] mfd: da9062: DA9062 MFD core driver
In reply to Lee JonesQuoted 15 lines · Lee Jones
To: Opensource [Steve Twiss]
Subject: Re: [PATCH V3 1/4] mfd: da9062: DA9062 MFD core driverQuoted 12 lines
Quoted 12 lines
Quoted 8 lines
include/linux/mfd/da9062/registers.h line 166
Quoted 5 lines
Steve Twiss wrote:
include/linux/mfd/da9062/registers.h new file +4
-0+#define DA9062AA_PAGE_SHIFT 0 +#define DA9062AA_PAGE_MASK (0x3f << 0) +#define DA9062AA_WRITE_MODE_SHIFT 6 +#define DA9062AA_WRITE_MODE_MASK (0x01 << 6)Lee Jones wrote:
For 1 << X, you should use BIT(X).Steve Twiss wrote:
The whole of this file is automatically generated by our hardware designers I would prefer it if the register definitions and bit fields are not altered [...]
[...]
Quoted 8 lines
Lee Jones wrote:
To be honest, it's probably not a blocker for acceptance, but if someone writes a patch next week to change all of the (0x01<< X) lines to start using theBIT()macro, I will accept it. Better to influenced your guys so you are not overly inconvenienced.
FWIW, when upstreaming code, the excuse "someone else wrote it", has
never been a good one to use on the lists. Believe me, I've
tried. ;)
Hi Lee,
Depending on your next step ..
I can either send a patch to this now or wait until you have reviewed the MFD
core: if there are any further comments on this, I can update the submission retrospectively.
Regards,
Steve
Re: [PATCH V3 1/4] mfd: da9062: DA9062 MFD core driver
Quoted 29 lines · Opensource [Steve Twiss]
Quoted 15 lines · Lee Jones
To: Opensource [Steve Twiss]
Subject: Re: [PATCH V3 1/4] mfd: da9062: DA9062 MFD core driverQuoted 12 lines
Quoted 12 lines
Quoted 8 lines
include/linux/mfd/da9062/registers.h line 166
Quoted 5 lines
Steve Twiss wrote:
include/linux/mfd/da9062/registers.h new file +4
-0+#define DA9062AA_PAGE_SHIFT 0 +#define DA9062AA_PAGE_MASK (0x3f << 0) +#define DA9062AA_WRITE_MODE_SHIFT 6 +#define DA9062AA_WRITE_MODE_MASK (0x01 << 6)Lee Jones wrote:
For 1 << X, you should use BIT(X).Steve Twiss wrote:
The whole of this file is automatically generated by our hardware designers I would prefer it if the register definitions and bit fields are not altered [...][...]
Quoted 8 lines
Lee Jones wrote:
To be honest, it's probably not a blocker for acceptance, but if someone writes a patch next week to change all of the (0x01<< X) lines to start using theBIT()macro, I will accept it. Better to influenced your guys so you are not overly inconvenienced.
FWIW, when upstreaming code, the excuse "someone else wrote it", has
never been a good one to use on the lists. Believe me, I've
tried. ;)
Quoted 6 lines
Hi Lee,
Depending on your next step ..
I can either send a patch to this now or wait until you have reviewed the MFD
core: if there are any further comments on this, I can update the submission retrospectively.
Let's start over. Please re-submit the set.
Signature · 8 lines
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 [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
PATCH v3 2/4 regulator: da9062: DA9062 regulator driver
In reply to S TwissFrom: S Twiss <[email protected]>
Add BUCK and LDO regulator driver support for DA9062
---
Changes in V3:
- No change
Changes in V2:
- Copyright headers GPL v2 (and later) match correct 'GPL' in MODULE_LICENSE
- Log errors for any issues in interrupt handler da9062_ldo_lim_event()
- Return IRQ_NONE when no regulators flag an event in da9062_ldo_lim_event()
- Remove VDD_WARN code and resource. Put it into the core instead of regulator driver. This IRQ handler does not belong in the regulator code.
- Usage of core functions for 'simplified DT parsing method' during probe and removal of platform data structures and unnecessary DT searches
This patch applies against linux-next and v4.1-rc3
drivers/regulator/Kconfig | 10 + drivers/regulator/Makefile | 1 + drivers/regulator/da9062-regulator.c | 843 +++++++++++++++++++++++++++++++++++ 3 files changed, 854 insertions(+) create mode 100644 drivers/regulator/da9062-regulator.c
drivers/regulator/Kconfig +10 -0
@@ -178,6 +178,16 @@ config REGULATOR_DA9055 This driver can also be built as a module. If so, the module will be called da9055-regulator. +config REGULATOR_DA9062 + tristate "Dialog Semiconductor DA9062 regulators" + depends on MFD_DA9062 + help + Say y here to support the BUCKs and LDOs regulators found on + DA9062 PMICs. + + This driver can also be built as a module. If so, the module + will be called da9062-regulator. + config REGULATOR_DA9063 tristate "Dialog Semiconductor DA9063 regulators" depends on MFD_DA9063
drivers/regulator/Makefile +1 -0
@@ -25,6 +25,7 @@ obj-$(CONFIG_REGULATOR_BCM590XX) += bcm590xx-regulator.o obj-$(CONFIG_REGULATOR_DA903X) += da903x.o obj-$(CONFIG_REGULATOR_DA9052) += da9052-regulator.o obj-$(CONFIG_REGULATOR_DA9055) += da9055-regulator.o +obj-$(CONFIG_REGULATOR_DA9062) += da9062-regulator.o obj-$(CONFIG_REGULATOR_DA9063) += da9063-regulator.o obj-$(CONFIG_REGULATOR_DA9210) += da9210-regulator.o obj-$(CONFIG_REGULATOR_DA9211) += da9211-regulator.o
drivers/regulator/da9062-regulator.c new file +843 -0
843 lines not on this page · raw message
Signature · 1 line
end-of-patch for PATCH V3
Re: [PATCH V3 2/4] regulator: da9062: DA9062 regulator driver
Quoted 3 lines · S Twiss
From: S Twiss <[email protected]>
Add BUCK and LDO regulator driver support for DA9062
Applied, thanks.
RE: [PATCH V3 2/4] regulator: da9062: DA9062 regulator driver
Quoted 8 lines · On 21 May 2015 13:04 Mark Brown wrote:
Subject: Re: [PATCH V3 2/4] regulator: da9062: DA9062 regulator driver
Quoted 3 lines · S Twiss
From: S Twiss <[email protected]>
Add BUCK and LDO regulator driver support for DA9062
Applied, thanks.
Thank you
Regards,
Steve
PATCH v3 3/4 watchdog: da9062: DA9062 watchdog driver
In reply to S TwissFrom: S Twiss <[email protected]>
Add watchdog driver support for DA9062
---
Changes in V3:
- Removed unnecessary return statement in da9062_apply_window_protection()
- Deleted dev_err() kernel noise in da9062_wdt_update_timeout_register()
- Changed dev_alert() to be a dev_err() in da9062_wdt_stop()
- Deleted the "watchdog ping" debug message in da9062_wdt_ping()
- Refactored the error path inthe probe() to remove redundant error gotos
- Altered dev_err() string to notify failures more specifically in probe()
- Removed the space in MODULE_ALIAS string "platform: da9062-watchdog"
- Ensure request for devm_request_threaded_irq() removal is completed
Changes in V2:
- Removed informational dev_info() 'installed watchdog' message
- Copyright headers GPL v2 (and later) match correct 'GPL' in MODULE_LICENSE
- Removed the explicit 300 msecs delay from the reset_watchdog_timer() function and replaced it with a variable delay (depending on the difference since the last ping). A debug message is used to catch the multiple pings trying to break the 300 msecs protection barrier.
- Fix error paths for the functions da9062_wdt_update_timeout_register() and da9062_wdt_stop()
- Add error paths in the probe() and correctly clean-up the registered device if there is a problem after registration.
This patch applies against linux-next and v4.1-rc3
drivers/watchdog/Kconfig | 9 ++ drivers/watchdog/Makefile | 1 + drivers/watchdog/da9062_wdt.c | 253 ++++++++++++++++++++++++++++++++++++++++++ 3 files changed, 263 insertions(+) create mode 100644 drivers/watchdog/da9062_wdt.c
drivers/watchdog/Kconfig +9 -0
@@ -96,6 +96,15 @@ config DA9063_WATCHDOG This driver can be built as a module. The module name is da9063_wdt. +config DA9062_WATCHDOG + tristate "Dialog DA9062 Watchdog" + depends on MFD_DA9062 + select WATCHDOG_CORE + help + Support for the watchdog in the DA9062 PMIC. + + This driver can be built as a module. The module name is da9062_wdt. + config GPIO_WATCHDOG tristate "Watchdog device controlled through GPIO-line" depends on OF_GPIO
drivers/watchdog/Makefile +1 -0
@@ -179,6 +179,7 @@ obj-$(CONFIG_XEN_WDT) += xen_wdt.o # Architecture Independent obj-$(CONFIG_DA9052_WATCHDOG) += da9052_wdt.o obj-$(CONFIG_DA9055_WATCHDOG) += da9055_wdt.o +obj-$(CONFIG_DA9062_WATCHDOG) += da9062_wdt.o obj-$(CONFIG_DA9063_WATCHDOG) += da9063_wdt.o obj-$(CONFIG_GPIO_WATCHDOG) += gpio_wdt.o obj-$(CONFIG_WM831X_WATCHDOG) += wm831x_wdt.o
drivers/watchdog/da9062_wdt.c new file +253 -0
253 lines not on this page · raw message
Signature · 1 line
end-of-patch for PATCH V3
Re: [PATCH V3 3/4] watchdog: da9062: DA9062 watchdog driver
Quoted 7 lines · S Twiss
From: S Twiss <[email protected]>
Add watchdog driver support for DA9062
Signed-off-by Steve Twiss <[email protected]>
--
To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
RE: [PATCH V3 3/4] watchdog: da9062: DA9062 watchdog driver
Quoted 11 lines · On 20 May 2015 21:30 Guenter Roeck wrote:
Subject: Re: [PATCH V3 3/4] watchdog: da9062: DA9062 watchdog driver
Quoted 7 lines · S Twiss
From: S Twiss <[email protected]>
Add watchdog driver support for DA9062
Signed-off-by Steve Twiss <[email protected]>Reviewed-by Guenter Roeck <[email protected]>
Thank you Guenter
Regards,
Steve
--
To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
PATCH v3 4/4 devicetree: da9062: Add bindings for DA9062 driver
In reply to S TwissFrom: S Twiss <[email protected]>
Add device tree bindings for the DA9062 driver
---
Changes in V3:
- No change
Changes in V2:
- Dropped the RTC and Onkey binding information in this patch-set Those drivers have been dropped from this patch set and the binding information has been removed accordingly.
This patch applies against linux-next and v4.1-rc3
Documentation/devicetree/bindings/mfd/da9062.txt | 79 ++++++++++++++++++++++++ 1 file changed, 79 insertions(+) create mode 100644 Documentation/devicetree/bindings/mfd/da9062.txt
Documentation/devicetree/bindings/mfd/da9062.txt new file +79 -0
@@ -0,0 +1,79 @@ +* Dialog DA9062 Power Management Integrated Circuit (PMIC) + +DA9062 consists of a large and varied group of sub-devices: + +Device Supply Names Description +------ ------------ ----------- +da9062-regulator : : LDOs & BUCKs +da9062-watchdog : : Watchdog Timer + +====== + +Required properties: + +- compatible : Should be "dlg,da9062". +- reg : Specifies the I2C slave address (this defaults to 0x58 but it can be + modified to match the chip's OTP settings). +- interrupt-parent : Specifies the reference to the interrupt controller for + the DA9062. +- interrupts : IRQ line information. +- interrupt-controller + +See Documentation/devicetree/bindings/interrupt-controller/interrupts.txt for +further information on IRQ bindings. + +Sub-nodes: + +- regulators : This node defines the settings for the LDOs and BUCKs. The + DA9062 regulators are bound using their names listed below: + + buck1 : BUCK_1 + buck2 : BUCK_2 + buck3 : BUCK_3 + buck4 : BUCK_4 + ldo1 : LDO_1 + ldo2 : LDO_2 + ldo3 : LDO_3 + ldo4 : LDO_4 + + The component follows the standard regulator framework and the bindings + details of individual regulator device can be found in: + Documentation/devicetree/bindings/regulator/regulator.txt + + +- watchdog: This node defines the settings for the watchdog driver associated + with the DA9062 PMIC. The compatible = "dlg,da9062-watchdog" should be added + if a node is created. + + +Example: + + pmic0: da9062@58 { + compatible = "dlg,da9062"; + reg = <0x58>; + interrupt-parent = <&gpio6>; + interrupts = <11 IRQ_TYPE_LEVEL_LOW>; + interrupt-controller; + + watchdog { + compatible = "dlg,da9062-watchdog"; + }; + + regulators { + DA9062_BUCK1: buck1 { + regulator-name = "BUCK1"; + regulator-min-microvolt = <300000>; + regulator-max-microvolt = <1570000>; + regulator-min-microamp = <500000>; + regulator-max-microamp = <2000000>; + regulator-boot-on; + }; + DA9062_LDO1: ldo1 { + regulator-name = "LDO_1"; + regulator-min-microvolt = <900000>; + regulator-max-microvolt = <3600000>; + regulator-boot-on; + }; + }; + }; +
Signature · 6 lines
end-of-patch for PATCH V3 -- To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html