* [PATCH V3 0/4] da9062: DA9062 driver submission
@ 2015-05-19 13:10 S Twiss
2015-05-19 13:10 ` [PATCH V3 3/4] watchdog: da9062: DA9062 watchdog driver S Twiss
` (2 more replies)
0 siblings, 3 replies; 18+ messages in thread
From: S Twiss @ 2015-05-19 13:10 UTC (permalink / raw)
To: Alessandro Zummo, DEVICETREE, Dmitry Torokhov, Ian Campbell,
Kumar Gala, LINUXINPUT, LINUXKERNEL, LINUXWATCHDOG, Lee Jones,
Liam Girdwood, Mark Brown, Mark Rutland, Pawel Moll, RTCLINUX,
Rob Herring, S Twiss, Samuel Ortiz, Wim Van Sebroeck
Cc: David Dajun Chen, Support Opensource
From: S Twiss <stwiss.opensource-WBD+wuPFNBhBDgjK7y7TUQ@public.gmane.org>
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
--
end-of-patch for PATCH V3
--
To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 18+ messages in thread* [PATCH V3 3/4] watchdog: da9062: DA9062 watchdog driver 2015-05-19 13:10 [PATCH V3 0/4] da9062: DA9062 driver submission S Twiss @ 2015-05-19 13:10 ` S Twiss [not found] ` <ded171f203fe1d0c72f0d7ea4c3d54257f4e970e.1432041031.git.stwiss.opensource-WBD+wuPFNBhBDgjK7y7TUQ@public.gmane.org> [not found] ` <cover.1432041031.git.stwiss.opensource-WBD+wuPFNBhBDgjK7y7TUQ@public.gmane.org> 2015-05-19 13:10 ` [PATCH V3 2/4] regulator: da9062: DA9062 regulator driver S Twiss 2 siblings, 1 reply; 18+ messages in thread From: S Twiss @ 2015-05-19 13:10 UTC (permalink / raw) To: LINUXKERNEL, LINUXWATCHDOG, Wim Van Sebroeck Cc: Alessandro Zummo, DEVICETREE, David Dajun Chen, Dmitry Torokhov, Ian Campbell, Kumar Gala, LINUXINPUT, Lee Jones, Liam Girdwood, Mark Brown, Mark Rutland, Pawel Moll, RTCLINUX, Rob Herring, S Twiss, Samuel Ortiz, Support Opensource From: S Twiss <stwiss.opensource@diasemi.com> Add watchdog driver support for DA9062 Signed-off-by: Steve Twiss <stwiss.opensource@diasemi.com> --- 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 diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig index e5e7c55..dfdb6c6 100644 --- a/drivers/watchdog/Kconfig +++ b/drivers/watchdog/Kconfig @@ -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 diff --git a/drivers/watchdog/Makefile b/drivers/watchdog/Makefile index 5c19294..57ba815 100644 --- a/drivers/watchdog/Makefile +++ b/drivers/watchdog/Makefile @@ -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 diff --git a/drivers/watchdog/da9062_wdt.c b/drivers/watchdog/da9062_wdt.c new file mode 100644 index 0000000..b3a870c --- /dev/null +++ b/drivers/watchdog/da9062_wdt.c @@ -0,0 +1,253 @@ +/* + * da9062_wdt.c - WDT device driver 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. + */ + +#include <linux/kernel.h> +#include <linux/module.h> +#include <linux/watchdog.h> +#include <linux/platform_device.h> +#include <linux/uaccess.h> +#include <linux/slab.h> +#include <linux/delay.h> +#include <linux/jiffies.h> +#include <linux/mfd/da9062/registers.h> +#include <linux/mfd/da9062/core.h> +#include <linux/regmap.h> +#include <linux/of.h> + +static const unsigned int wdt_timeout[] = { 0, 2, 4, 8, 16, 32, 65, 131 }; +#define DA9062_TWDSCALE_DISABLE 0 +#define DA9062_TWDSCALE_MIN 1 +#define DA9062_TWDSCALE_MAX (ARRAY_SIZE(wdt_timeout) - 1) +#define DA9062_WDT_MIN_TIMEOUT wdt_timeout[DA9062_TWDSCALE_MIN] +#define DA9062_WDT_MAX_TIMEOUT wdt_timeout[DA9062_TWDSCALE_MAX] +#define DA9062_WDG_DEFAULT_TIMEOUT wdt_timeout[DA9062_TWDSCALE_MAX-1] +#define DA9062_RESET_PROTECTION_MS 300 + +struct da9062_watchdog { + struct da9062 *hw; + struct watchdog_device wdtdev; + unsigned long j_time_stamp; +}; + +static void da9062_set_window_start(struct da9062_watchdog *wdt) +{ + wdt->j_time_stamp = jiffies; +} + +static void da9062_apply_window_protection(struct da9062_watchdog *wdt) +{ + unsigned long delay = msecs_to_jiffies(DA9062_RESET_PROTECTION_MS); + unsigned long timeout = wdt->j_time_stamp + delay; + unsigned long now = jiffies; + unsigned int diff_ms; + + /* if time-limit has not elapsed then wait for remainder */ + if (time_before(now, timeout)) { + diff_ms = jiffies_to_msecs(timeout-now); + dev_dbg(wdt->hw->dev, + "Kicked too quickly. Delaying %u msecs\n", diff_ms); + msleep(diff_ms); + } +} + +static unsigned int da9062_wdt_timeout_to_sel(unsigned int secs) +{ + unsigned int i; + + for (i = DA9062_TWDSCALE_MIN; i <= DA9062_TWDSCALE_MAX; i++) { + if (wdt_timeout[i] >= secs) + return i; + } + + return DA9062_TWDSCALE_MAX; +} + +static int da9062_reset_watchdog_timer(struct da9062_watchdog *wdt) +{ + int ret; + + da9062_apply_window_protection(wdt); + + ret = regmap_update_bits(wdt->hw->regmap, + DA9062AA_CONTROL_F, + DA9062AA_WATCHDOG_MASK, + DA9062AA_WATCHDOG_MASK); + + da9062_set_window_start(wdt); + + return ret; +} + +static int da9062_wdt_update_timeout_register(struct da9062_watchdog *wdt, + unsigned int regval) +{ + struct da9062 *chip = wdt->hw; + int ret; + + ret = da9062_reset_watchdog_timer(wdt); + if (ret) + return ret; + + return regmap_update_bits(chip->regmap, + DA9062AA_CONTROL_D, + DA9062AA_TWDSCALE_MASK, + regval); +} + +static int da9062_wdt_start(struct watchdog_device *wdd) +{ + struct da9062_watchdog *wdt = watchdog_get_drvdata(wdd); + unsigned int selector; + int ret; + + selector = da9062_wdt_timeout_to_sel(wdt->wdtdev.timeout); + ret = da9062_wdt_update_timeout_register(wdt, selector); + if (ret) + dev_err(wdt->hw->dev, "Watchdog failed to start (err = %d)\n", + ret); + + return ret; +} + +static int da9062_wdt_stop(struct watchdog_device *wdd) +{ + struct da9062_watchdog *wdt = watchdog_get_drvdata(wdd); + int ret; + + ret = da9062_reset_watchdog_timer(wdt); + if (ret) { + dev_err(wdt->hw->dev, "Failed to ping the watchdog (err = %d)\n", + ret); + return ret; + } + + ret = regmap_update_bits(wdt->hw->regmap, + DA9062AA_CONTROL_D, + DA9062AA_TWDSCALE_MASK, + DA9062_TWDSCALE_DISABLE); + if (ret) + dev_err(wdt->hw->dev, "Watchdog failed to stop (err = %d)\n", + ret); + + return ret; +} + +static int da9062_wdt_ping(struct watchdog_device *wdd) +{ + struct da9062_watchdog *wdt = watchdog_get_drvdata(wdd); + int ret; + + ret = da9062_reset_watchdog_timer(wdt); + if (ret) + dev_err(wdt->hw->dev, "Failed to ping the watchdog (err = %d)\n", + ret); + + return ret; +} + +static int da9062_wdt_set_timeout(struct watchdog_device *wdd, + unsigned int timeout) +{ + struct da9062_watchdog *wdt = watchdog_get_drvdata(wdd); + unsigned int selector; + int ret; + + selector = da9062_wdt_timeout_to_sel(timeout); + ret = da9062_wdt_update_timeout_register(wdt, selector); + if (ret) + dev_err(wdt->hw->dev, "Failed to set watchdog timeout (err = %d)\n", + ret); + else + wdd->timeout = wdt_timeout[selector]; + + return ret; +} + +static const struct watchdog_info da9062_watchdog_info = { + .options = WDIOF_SETTIMEOUT | WDIOF_KEEPALIVEPING, + .identity = "DA9062 WDT", +}; + +static const struct watchdog_ops da9062_watchdog_ops = { + .owner = THIS_MODULE, + .start = da9062_wdt_start, + .stop = da9062_wdt_stop, + .ping = da9062_wdt_ping, + .set_timeout = da9062_wdt_set_timeout, +}; + +static int da9062_wdt_probe(struct platform_device *pdev) +{ + int ret; + struct da9062 *chip; + struct da9062_watchdog *wdt; + + chip = dev_get_drvdata(pdev->dev.parent); + if (!chip) + return -EINVAL; + + wdt = devm_kzalloc(&pdev->dev, sizeof(*wdt), GFP_KERNEL); + if (!wdt) + return -ENOMEM; + + wdt->hw = chip; + + wdt->wdtdev.info = &da9062_watchdog_info; + wdt->wdtdev.ops = &da9062_watchdog_ops; + wdt->wdtdev.min_timeout = DA9062_WDT_MIN_TIMEOUT; + wdt->wdtdev.max_timeout = DA9062_WDT_MAX_TIMEOUT; + wdt->wdtdev.timeout = DA9062_WDG_DEFAULT_TIMEOUT; + wdt->wdtdev.status = WATCHDOG_NOWAYOUT_INIT_STATUS; + + watchdog_set_drvdata(&wdt->wdtdev, wdt); + dev_set_drvdata(&pdev->dev, wdt); + + ret = watchdog_register_device(&wdt->wdtdev); + if (ret < 0) { + dev_err(wdt->hw->dev, + "watchdog registration failed (%d)\n", ret); + return ret; + } + + da9062_set_window_start(wdt); + + ret = da9062_wdt_ping(&wdt->wdtdev); + if (ret < 0) + watchdog_unregister_device(&wdt->wdtdev); + + return ret; +} + +static int da9062_wdt_remove(struct platform_device *pdev) +{ + struct da9062_watchdog *wdt = dev_get_drvdata(&pdev->dev); + + watchdog_unregister_device(&wdt->wdtdev); + return 0; +} + +static struct platform_driver da9062_wdt_driver = { + .probe = da9062_wdt_probe, + .remove = da9062_wdt_remove, + .driver = { + .name = "da9062-watchdog", + }, +}; +module_platform_driver(da9062_wdt_driver); + +MODULE_AUTHOR("S Twiss <stwiss.opensource@diasemi.com>"); +MODULE_DESCRIPTION("WDT device driver for Dialog DA9062"); +MODULE_LICENSE("GPL"); +MODULE_ALIAS("platform:da9062-watchdog"); -- end-of-patch for PATCH V3 ^ permalink raw reply related [flat|nested] 18+ messages in thread
[parent not found: <ded171f203fe1d0c72f0d7ea4c3d54257f4e970e.1432041031.git.stwiss.opensource-WBD+wuPFNBhBDgjK7y7TUQ@public.gmane.org>]
* Re: [PATCH V3 3/4] watchdog: da9062: DA9062 watchdog driver [not found] ` <ded171f203fe1d0c72f0d7ea4c3d54257f4e970e.1432041031.git.stwiss.opensource-WBD+wuPFNBhBDgjK7y7TUQ@public.gmane.org> @ 2015-05-20 20:30 ` Guenter Roeck [not found] ` <20150520203016.GA5710-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org> 0 siblings, 1 reply; 18+ messages in thread From: Guenter Roeck @ 2015-05-20 20:30 UTC (permalink / raw) To: S Twiss Cc: LINUXKERNEL, LINUXWATCHDOG, Wim Van Sebroeck, Alessandro Zummo, DEVICETREE, David Dajun Chen, Dmitry Torokhov, Ian Campbell, Kumar Gala, LINUXINPUT, Lee Jones, Liam Girdwood, Mark Brown, Mark Rutland, Pawel Moll, RTCLINUX, Rob Herring, Samuel Ortiz, Support Opensource On Tue, May 19, 2015 at 02:10:30PM +0100, S Twiss wrote: > From: S Twiss <stwiss.opensource-WBD+wuPFNBhBDgjK7y7TUQ@public.gmane.org> > > Add watchdog driver support for DA9062 > > > Signed-off-by: Steve Twiss <stwiss.opensource-WBD+wuPFNBhBDgjK7y7TUQ@public.gmane.org> > Reviewed-by: Guenter Roeck <linux-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org> -- To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 18+ messages in thread
[parent not found: <20150520203016.GA5710-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org>]
* RE: [PATCH V3 3/4] watchdog: da9062: DA9062 watchdog driver [not found] ` <20150520203016.GA5710-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org> @ 2015-05-21 7:15 ` Opensource [Steve Twiss] 0 siblings, 0 replies; 18+ messages in thread From: Opensource [Steve Twiss] @ 2015-05-21 7:15 UTC (permalink / raw) To: Guenter Roeck Cc: LINUXKERNEL, LINUXWATCHDOG, Wim Van Sebroeck, Alessandro Zummo, DEVICETREE, David Dajun Chen, Dmitry Torokhov, Ian Campbell, Kumar Gala, LINUXINPUT, Lee Jones, Liam Girdwood, Mark Brown, Mark Rutland, Pawel Moll, RTCLINUX, Rob Herring, Samuel Ortiz, Support Opensource On 20 May 2015 21:30 Guenter Roeck wrote: > Subject: Re: [PATCH V3 3/4] watchdog: da9062: DA9062 watchdog driver > > On Tue, May 19, 2015 at 02:10:30PM +0100, S Twiss wrote: > > From: S Twiss <stwiss.opensource-WBD+wuPFNBhBDgjK7y7TUQ@public.gmane.org> > > > > Add watchdog driver support for DA9062 > > > > > > Signed-off-by: Steve Twiss <stwiss.opensource-WBD+wuPFNBhBDgjK7y7TUQ@public.gmane.org> > > > Reviewed-by: Guenter Roeck <linux-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org> Thank you Guenter Regards, Steve -- To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 18+ messages in thread
[parent not found: <cover.1432041031.git.stwiss.opensource-WBD+wuPFNBhBDgjK7y7TUQ@public.gmane.org>]
* [PATCH V3 1/4] mfd: da9062: DA9062 MFD core driver [not found] ` <cover.1432041031.git.stwiss.opensource-WBD+wuPFNBhBDgjK7y7TUQ@public.gmane.org> @ 2015-05-19 13:10 ` S Twiss 2015-05-26 16:10 ` Lee Jones 2015-05-19 13:10 ` [PATCH V3 4/4] devicetree: da9062: Add bindings for DA9062 driver S Twiss 1 sibling, 1 reply; 18+ messages in thread From: S Twiss @ 2015-05-19 13:10 UTC (permalink / raw) To: LINUXKERNEL, Lee Jones, Samuel Ortiz Cc: Alessandro Zummo, DEVICETREE, David Dajun Chen, Dmitry Torokhov, Ian Campbell, Kumar Gala, LINUXINPUT, LINUXWATCHDOG, Liam Girdwood, Mark Brown, Mark Rutland, Pawel Moll, RTCLINUX, Rob Herring, S Twiss, Support Opensource, Wim Van Sebroeck From: S Twiss <stwiss.opensource-WBD+wuPFNBhBDgjK7y7TUQ@public.gmane.org> Add MFD core driver support for DA9062 Signed-off-by: Steve Twiss <stwiss.opensource-WBD+wuPFNBhBDgjK7y7TUQ@public.gmane.org> --- Changes in V3: - Removed references to the RTC and OnKey in the mfd_cell definition. Changes in V2: - Copyright headers GPL v2 (and later) match correct 'GPL' in MODULE_LICENSE - Ensure DA9062 core is tristate in Kconfig so it can be built as a module - Move VDD_WARN code and resource into the core instead of regulator driver This patch applies against linux-next and v4.1-rc3 drivers/mfd/Kconfig | 12 + drivers/mfd/Makefile | 3 +- drivers/mfd/da9062-core.c | 611 +++++++++++++++++++ include/linux/mfd/da9062/core.h | 62 ++ include/linux/mfd/da9062/registers.h | 1108 ++++++++++++++++++++++++++++++++++ 5 files changed, 1795 insertions(+), 1 deletion(-) create mode 100644 drivers/mfd/da9062-core.c create mode 100644 include/linux/mfd/da9062/core.h create mode 100644 include/linux/mfd/da9062/registers.h diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig index d5ad04d..44fb2b9 100644 --- a/drivers/mfd/Kconfig +++ b/drivers/mfd/Kconfig @@ -183,6 +183,18 @@ config MFD_DA9055 This driver can be built as a module. If built as a module it will be called "da9055" +config MFD_DA9062 + tristate "Dialog Semiconductor DA9062 PMIC Support" + select MFD_CORE + select REGMAP_I2C + select REGMAP_IRQ + depends on I2C=y + help + Say yes here for support for the Dialog Semiconductor DA9062 PMIC. + This includes the I2C driver and core APIs. + Additional drivers must be enabled in order to use the functionality + of the device. + config MFD_DA9063 bool "Dialog Semiconductor DA9063 PMIC Support" select MFD_CORE diff --git a/drivers/mfd/Makefile b/drivers/mfd/Makefile index 0e5cfeb..d94e562 100644 --- a/drivers/mfd/Makefile +++ b/drivers/mfd/Makefile @@ -110,10 +110,11 @@ obj-$(CONFIG_MFD_LP8788) += lp8788.o lp8788-irq.o da9055-objs := da9055-core.o da9055-i2c.o obj-$(CONFIG_MFD_DA9055) += da9055.o - +obj-$(CONFIG_MFD_DA9062) += da9062-core.o da9063-objs := da9063-core.o da9063-irq.o da9063-i2c.o obj-$(CONFIG_MFD_DA9063) += da9063.o obj-$(CONFIG_MFD_DA9150) += da9150-core.o + obj-$(CONFIG_MFD_MAX14577) += max14577.o obj-$(CONFIG_MFD_MAX77686) += max77686.o obj-$(CONFIG_MFD_MAX77693) += max77693.o diff --git a/drivers/mfd/da9062-core.c b/drivers/mfd/da9062-core.c new file mode 100644 index 0000000..5aea082 --- /dev/null +++ b/drivers/mfd/da9062-core.c @@ -0,0 +1,611 @@ +/* + * da9062-core.c - CORE device driver for DA9062 + * Copyright (C) 2015 Dialog Semiconductor Ltd. + * + * This program is free software; you can redistribute it and/or + * modify it under the terms of the GNU General Public License + * as published by the Free Software Foundation; either version 2 + * of the License, or (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + */ + +#include <linux/kernel.h> +#include <linux/module.h> +#include <linux/init.h> +#include <linux/slab.h> +#include <linux/device.h> +#include <linux/delay.h> +#include <linux/interrupt.h> +#include <linux/mutex.h> +#include <linux/regmap.h> + +#include <linux/irq.h> +#include <linux/mfd/core.h> +#include <linux/i2c.h> +#include <linux/err.h> + +#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> + +/* IRQ device driver for DA9062 */ + +#define DA9062_REG_EVENT_A_OFFSET 0 +#define DA9062_REG_EVENT_B_OFFSET 1 +#define DA9062_REG_EVENT_C_OFFSET 2 + +static struct regmap_irq da9062_irqs[] = { + /* EVENT A */ + [DA9062_IRQ_ONKEY] = { + .reg_offset = DA9062_REG_EVENT_A_OFFSET, + .mask = DA9062AA_M_NONKEY_MASK, + }, + [DA9062_IRQ_ALARM] = { + .reg_offset = DA9062_REG_EVENT_A_OFFSET, + .mask = DA9062AA_M_ALARM_MASK, + }, + [DA9062_IRQ_TICK] = { + .reg_offset = DA9062_REG_EVENT_A_OFFSET, + .mask = DA9062AA_M_TICK_MASK, + }, + [DA9062_IRQ_WDG_WARN] = { + .reg_offset = DA9062_REG_EVENT_A_OFFSET, + .mask = DA9062AA_M_WDG_WARN_MASK, + }, + [DA9062_IRQ_SEQ_RDY] = { + .reg_offset = DA9062_REG_EVENT_A_OFFSET, + .mask = DA9062AA_M_SEQ_RDY_MASK, + }, + /* EVENT B */ + [DA9062_IRQ_TEMP] = { + .reg_offset = DA9062_REG_EVENT_B_OFFSET, + .mask = DA9062AA_M_TEMP_MASK, + }, + [DA9062_IRQ_LDO_LIM] = { + .reg_offset = DA9062_REG_EVENT_B_OFFSET, + .mask = DA9062AA_M_LDO_LIM_MASK, + }, + [DA9062_IRQ_DVC_RDY] = { + .reg_offset = DA9062_REG_EVENT_B_OFFSET, + .mask = DA9062AA_M_DVC_RDY_MASK, + }, + [DA9062_IRQ_VDD_WARN] = { + .reg_offset = DA9062_REG_EVENT_B_OFFSET, + .mask = DA9062AA_M_VDD_WARN_MASK, + }, + /* EVENT C */ + [DA9062_IRQ_GPI0] = { + .reg_offset = DA9062_REG_EVENT_C_OFFSET, + .mask = DA9062AA_M_GPI0_MASK, + }, + [DA9062_IRQ_GPI1] = { + .reg_offset = DA9062_REG_EVENT_C_OFFSET, + .mask = DA9062AA_M_GPI1_MASK, + }, + [DA9062_IRQ_GPI2] = { + .reg_offset = DA9062_REG_EVENT_C_OFFSET, + .mask = DA9062AA_M_GPI2_MASK, + }, + [DA9062_IRQ_GPI3] = { + .reg_offset = DA9062_REG_EVENT_C_OFFSET, + .mask = DA9062AA_M_GPI3_MASK, + }, + [DA9062_IRQ_GPI4] = { + .reg_offset = DA9062_REG_EVENT_C_OFFSET, + .mask = DA9062AA_M_GPI4_MASK, + }, +}; + +static struct regmap_irq_chip da9062_irq_chip = { + /* IRQ */ + .name = "da9062-irq", + .irqs = da9062_irqs, + .num_irqs = DA9062_NUM_IRQ, + /* STATUS and EVENT */ + .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) { + 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); + 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); +} + +/* CORE device driver for DA9062 */ + +static struct resource da9062_core_resources[] = { + { + .name = "VDD_WARN", + .start = DA9062_IRQ_VDD_WARN, + .end = DA9062_IRQ_VDD_WARN, + .flags = IORESOURCE_IRQ, + }, +}; + +static struct resource da9062_regulators_resources[] = { + { + .name = "LDO_LIM", + .start = DA9062_IRQ_LDO_LIM, + .end = DA9062_IRQ_LDO_LIM, + .flags = IORESOURCE_IRQ, + }, +}; + +static struct resource da9062_thermal_resources[] = { + { + .name = "THERMAL", + .start = DA9062_IRQ_TEMP, + .end = DA9062_IRQ_TEMP, + .flags = IORESOURCE_IRQ, + }, +}; + +static struct resource da9062_wdt_resources[] = { + { + .name = "WDG_WARN", + .start = DA9062_IRQ_WDG_WARN, + .end = DA9062_IRQ_WDG_WARN, + .flags = IORESOURCE_IRQ, + }, +}; + +static const struct mfd_cell da9062_devs[] = { + { + .name = "da9062-core", + .num_resources = ARRAY_SIZE(da9062_core_resources), + .resources = da9062_core_resources, + }, + { + .name = "da9062-regulators", + .num_resources = ARRAY_SIZE(da9062_regulators_resources), + .resources = da9062_regulators_resources, + }, + { + .name = "da9062-watchdog", + .num_resources = ARRAY_SIZE(da9062_wdt_resources), + .resources = da9062_wdt_resources, + .of_compatible = "dlg,da9062-wdt", + }, + { + .name = "da9062-thermal", + .num_resources = ARRAY_SIZE(da9062_thermal_resources), + .resources = da9062_thermal_resources, + .of_compatible = "dlg,da9062-thermal", + }, +}; + +static irqreturn_t da9062_vdd_warn_event(int irq, void *data) +{ + struct da9062 *hw = data; + + dev_err(hw->dev, "VDD warning indicator\n"); + + return IRQ_HANDLED; +} + +static int da9062_clear_fault_log(struct da9062 *chip) +{ + int ret = 0; + int fault_log = 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; + } else { + if (fault_log) { + if (fault_log & DA9062AA_TWD_ERROR_MASK) + dev_dbg(chip->dev, "Fault log entry detected: TWD_ERROR\n"); + if (fault_log & DA9062AA_POR_MASK) + dev_dbg(chip->dev, "Fault log entry detected: POR\n"); + if (fault_log & DA9062AA_VDD_FAULT_MASK) + dev_dbg(chip->dev, "Fault log entry detected: VDD_FAULT\n"); + if (fault_log & DA9062AA_VDD_START_MASK) + dev_dbg(chip->dev, "Fault log entry detected: VDD_START\n"); + if (fault_log & DA9062AA_TEMP_CRIT_MASK) + dev_dbg(chip->dev, "Fault log entry detected: TEMP_CRIT\n"); + if (fault_log & DA9062AA_KEY_RESET_MASK) + dev_dbg(chip->dev, "Fault log entry detected: KEY_RESET\n"); + if (fault_log & DA9062AA_NSHUTDOWN_MASK) + dev_dbg(chip->dev, "Fault log entry detected: NSHUTDOWN\n"); + if (fault_log & DA9062AA_WAIT_SHUT_MASK) + dev_dbg(chip->dev, "Fault log entry detected: WAIT_SHUT\n"); + } + + ret = regmap_write(chip->regmap, DA9062AA_FAULT_LOG, + fault_log); + if (ret < 0) + dev_err(chip->dev, + "Cannot reset FAULT_LOG values %d\n", ret); + } + + return ret; +} + +int da9062_device_init(struct da9062 *chip, unsigned int irq) +{ + int device_id, variant_id, variant_mrc; + int irq_vdd_warn, ret; + + ret = da9062_clear_fault_log(chip); + if (ret < 0) + dev_err(chip->dev, "Cannot clear fault log\n"); + + chip->irq_base = -1; + chip->chip_irq = irq; + + ret = regmap_read(chip->regmap, DA9062AA_DEVICE_ID, &device_id); + if (ret < 0) { + dev_err(chip->dev, "Cannot read chip ID.\n"); + return -EIO; + } + if (device_id != DA9062_PMIC_DEVICE_ID) { + dev_err(chip->dev, "Invalid device ID: 0x%02x\n", device_id); + return -ENODEV; + } + + ret = regmap_read(chip->regmap, DA9062AA_VARIANT_ID, &variant_id); + if (ret < 0) { + dev_err(chip->dev, "Cannot read chip variant id.\n"); + return -EIO; + } + + dev_info(chip->dev, + "Device detected (device-ID: 0x%02X, var-ID: 0x%02X)\n", + device_id, variant_id); + + 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; + + ret = da9062_irq_init(chip); + if (ret) { + dev_err(chip->dev, "Cannot initialize interrupts.\n"); + return ret; + } + + chip->irq_base = regmap_irq_chip_get_base(chip->regmap_irq); + + ret = mfd_add_devices(chip->dev, -1, da9062_devs, + ARRAY_SIZE(da9062_devs), NULL, chip->irq_base, + NULL); + if (ret) { + dev_err(chip->dev, "Cannot add MFD cells\n"); + da9062_irq_exit(chip); + return ret; + } + + /* VDD WARN event support */ + irq_vdd_warn = regmap_irq_get_virq(chip->regmap_irq, + DA9062_IRQ_VDD_WARN); + if (irq_vdd_warn < 0) { + dev_err(chip->dev, "Failed to get IRQ.\n"); + return irq_vdd_warn; + } + chip->irq_vdd_warn = irq_vdd_warn; + + ret = devm_request_threaded_irq(chip->dev, irq_vdd_warn, + NULL, da9062_vdd_warn_event, + IRQF_TRIGGER_LOW | IRQF_ONESHOT, + "VDD_WARN", chip); + if (ret) { + dev_warn(chip->dev, + "Failed to request VDD_WARN IRQ.\n"); + chip->irq_vdd_warn = -ENXIO; + } + + return ret; +} + +void da9062_device_exit(struct da9062 *chip) +{ + mfd_remove_devices(chip->dev); + da9062_irq_exit(chip); +} + +/* I2C device driver for DA9062 */ + +static const struct regmap_range da9062_aa_readable_ranges[] = { + { + .range_min = DA9062AA_PAGE_CON, + .range_max = DA9062AA_STATUS_B, + }, { + .range_min = DA9062AA_STATUS_D, + .range_max = DA9062AA_EVENT_C, + }, { + .range_min = DA9062AA_IRQ_MASK_A, + .range_max = DA9062AA_IRQ_MASK_C, + }, { + .range_min = DA9062AA_CONTROL_A, + .range_max = DA9062AA_GPIO_4, + }, { + .range_min = DA9062AA_GPIO_WKUP_MODE, + .range_max = DA9062AA_BUCK4_CONT, + }, { + .range_min = DA9062AA_BUCK3_CONT, + .range_max = DA9062AA_BUCK3_CONT, + }, { + .range_min = DA9062AA_LDO1_CONT, + .range_max = DA9062AA_LDO4_CONT, + }, { + .range_min = DA9062AA_DVC_1, + .range_max = DA9062AA_DVC_1, + }, { + .range_min = DA9062AA_COUNT_S, + .range_max = DA9062AA_SECOND_D, + }, { + .range_min = DA9062AA_SEQ, + .range_max = DA9062AA_ID_4_3, + }, { + .range_min = DA9062AA_ID_12_11, + .range_max = DA9062AA_ID_16_15, + }, { + .range_min = DA9062AA_ID_22_21, + .range_max = DA9062AA_ID_32_31, + }, { + .range_min = DA9062AA_SEQ_A, + .range_max = DA9062AA_BUCK3_CFG, + }, { + .range_min = DA9062AA_VBUCK2_A, + .range_max = DA9062AA_VBUCK4_A, + }, { + .range_min = DA9062AA_VBUCK3_A, + .range_max = DA9062AA_VBUCK3_A, + }, { + .range_min = DA9062AA_VLDO1_A, + .range_max = DA9062AA_VLDO4_A, + }, { + .range_min = DA9062AA_VBUCK2_B, + .range_max = DA9062AA_VBUCK4_B, + }, { + .range_min = DA9062AA_VBUCK3_B, + .range_max = DA9062AA_VBUCK3_B, + }, { + .range_min = DA9062AA_VLDO1_B, + .range_max = DA9062AA_VLDO4_B, + }, { + .range_min = DA9062AA_BBAT_CONT, + .range_max = DA9062AA_BBAT_CONT, + }, { + .range_min = DA9062AA_INTERFACE, + .range_max = DA9062AA_CONFIG_E, + }, { + .range_min = DA9062AA_CONFIG_G, + .range_max = DA9062AA_CONFIG_K, + }, { + .range_min = DA9062AA_CONFIG_M, + .range_max = DA9062AA_CONFIG_M, + }, { + .range_min = DA9062AA_TRIM_CLDR, + .range_max = DA9062AA_GP_ID_19, + }, { + .range_min = DA9062AA_DEVICE_ID, + .range_max = DA9062AA_CONFIG_ID, + }, +}; + +static const struct regmap_range da9062_aa_writeable_ranges[] = { + { + .range_min = DA9062AA_PAGE_CON, + .range_max = DA9062AA_PAGE_CON, + }, { + .range_min = DA9062AA_FAULT_LOG, + .range_max = DA9062AA_EVENT_C, + }, { + .range_min = DA9062AA_IRQ_MASK_A, + .range_max = DA9062AA_IRQ_MASK_C, + }, { + .range_min = DA9062AA_CONTROL_A, + .range_max = DA9062AA_GPIO_4, + }, { + .range_min = DA9062AA_GPIO_WKUP_MODE, + .range_max = DA9062AA_BUCK4_CONT, + }, { + .range_min = DA9062AA_BUCK3_CONT, + .range_max = DA9062AA_BUCK3_CONT, + }, { + .range_min = DA9062AA_LDO1_CONT, + .range_max = DA9062AA_LDO4_CONT, + }, { + .range_min = DA9062AA_DVC_1, + .range_max = DA9062AA_DVC_1, + }, { + .range_min = DA9062AA_COUNT_S, + .range_max = DA9062AA_ALARM_Y, + }, { + .range_min = DA9062AA_SEQ, + .range_max = DA9062AA_ID_4_3, + }, { + .range_min = DA9062AA_ID_12_11, + .range_max = DA9062AA_ID_16_15, + }, { + .range_min = DA9062AA_ID_22_21, + .range_max = DA9062AA_ID_32_31, + }, { + .range_min = DA9062AA_SEQ_A, + .range_max = DA9062AA_BUCK3_CFG, + }, { + .range_min = DA9062AA_VBUCK2_A, + .range_max = DA9062AA_VBUCK4_A, + }, { + .range_min = DA9062AA_VBUCK3_A, + .range_max = DA9062AA_VBUCK3_A, + }, { + .range_min = DA9062AA_VLDO1_A, + .range_max = DA9062AA_VLDO4_A, + }, { + .range_min = DA9062AA_VBUCK2_B, + .range_max = DA9062AA_VBUCK4_B, + }, { + .range_min = DA9062AA_VBUCK3_B, + .range_max = DA9062AA_VBUCK3_B, + }, { + .range_min = DA9062AA_VLDO1_B, + .range_max = DA9062AA_VLDO4_B, + }, { + .range_min = DA9062AA_BBAT_CONT, + .range_max = DA9062AA_BBAT_CONT, + }, { + .range_min = DA9062AA_GP_ID_0, + .range_max = DA9062AA_GP_ID_19, + }, +}; + +static const struct regmap_range da9062_aa_volatile_ranges[] = { + { + .range_min = DA9062AA_PAGE_CON, + .range_max = DA9062AA_STATUS_B, + }, { + .range_min = DA9062AA_STATUS_D, + .range_max = DA9062AA_EVENT_C, + }, { + .range_min = DA9062AA_CONTROL_F, + .range_max = DA9062AA_CONTROL_F, + }, { + .range_min = DA9062AA_COUNT_S, + .range_max = DA9062AA_SECOND_D, + }, +}; + +static const struct regmap_access_table da9062_aa_readable_table = { + .yes_ranges = da9062_aa_readable_ranges, + .n_yes_ranges = ARRAY_SIZE(da9062_aa_readable_ranges), +}; + +static const struct regmap_access_table da9062_aa_writeable_table = { + .yes_ranges = da9062_aa_writeable_ranges, + .n_yes_ranges = ARRAY_SIZE(da9062_aa_writeable_ranges), +}; + +static const struct regmap_access_table da9062_aa_volatile_table = { + .yes_ranges = da9062_aa_volatile_ranges, + .n_yes_ranges = ARRAY_SIZE(da9062_aa_volatile_ranges), +}; + +static const struct regmap_range_cfg da9062_range_cfg[] = { + { + .range_min = DA9062AA_PAGE_CON, + .range_max = DA9062AA_CONFIG_ID, + .selector_reg = DA9062AA_PAGE_CON, + .selector_mask = 1 << DA9062_I2C_PAGE_SEL_SHIFT, + .selector_shift = DA9062_I2C_PAGE_SEL_SHIFT, + .window_start = 0, + .window_len = 256, + } +}; + +static struct regmap_config da9062_regmap_config = { + .reg_bits = 8, + .val_bits = 8, + .ranges = da9062_range_cfg, + .num_ranges = ARRAY_SIZE(da9062_range_cfg), + .max_register = DA9062AA_CONFIG_ID, + .cache_type = REGCACHE_RBTREE, +}; + +static const struct of_device_id da9062_dt_ids[] = { + { .compatible = "dlg,da9062", }, + { } +}; +MODULE_DEVICE_TABLE(of, da9062_dt_ids); + +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); + if (chip == NULL) + return -ENOMEM; + + i2c_set_clientdata(i2c, chip); + chip->dev = &i2c->dev; + chip->chip_irq = i2c->irq; + + da9062_regmap_config.rd_table = &da9062_aa_readable_table; + da9062_regmap_config.wr_table = &da9062_aa_writeable_table; + da9062_regmap_config.volatile_table = &da9062_aa_volatile_table; + + chip->regmap = devm_regmap_init_i2c(i2c, &da9062_regmap_config); + if (IS_ERR(chip->regmap)) { + ret = PTR_ERR(chip->regmap); + dev_err(chip->dev, "Failed to allocate register map: %d\n", + ret); + return ret; + } + + return da9062_device_init(chip, i2c->irq); +} + +static int da9062_i2c_remove(struct i2c_client *i2c) +{ + struct da9062 *chip = i2c_get_clientdata(i2c); + + da9062_device_exit(chip); + + return 0; +} + +static const struct i2c_device_id da9062_i2c_id[] = { + {"da9062", 0}, + {}, +}; +MODULE_DEVICE_TABLE(i2c, da9062_i2c_id); + +static struct i2c_driver da9062_i2c_driver = { + .driver = { + .name = "da9062", + .of_match_table = of_match_ptr(da9062_dt_ids), + }, + .probe = da9062_i2c_probe, + .remove = da9062_i2c_remove, + .id_table = da9062_i2c_id, +}; + +module_i2c_driver(da9062_i2c_driver); + + +MODULE_DESCRIPTION("CORE device driver for Dialog DA9062"); +MODULE_AUTHOR("S Twiss <stwiss.opensource-WBD+wuPFNBhBDgjK7y7TUQ@public.gmane.org>"); +MODULE_LICENSE("GPL"); diff --git a/include/linux/mfd/da9062/core.h b/include/linux/mfd/da9062/core.h new file mode 100644 index 0000000..0b17891 --- /dev/null +++ b/include/linux/mfd/da9062/core.h @@ -0,0 +1,62 @@ +/* + * core.h - CORE H for DA9062 + * Copyright (C) 2015 Dialog Semiconductor Ltd. + * + * This program is free software; you can redistribute it and/or + * modify it under the terms of the GNU General Public License + * as published by the Free Software Foundation; either version 2 + * of the License, or (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + */ + +#ifndef __MFD_DA9062_CORE_H__ +#define __MFD_DA9062_CORE_H__ + +#include <linux/interrupt.h> +#include <linux/mfd/da9062/registers.h> + +/* Interrupts */ +enum da9062_irqs { + /* IRQ A */ + DA9062_IRQ_ONKEY, + DA9062_IRQ_ALARM, + DA9062_IRQ_TICK, + DA9062_IRQ_WDG_WARN, + DA9062_IRQ_SEQ_RDY, + /* IRQ B*/ + DA9062_IRQ_TEMP, + DA9062_IRQ_LDO_LIM, + DA9062_IRQ_DVC_RDY, + DA9062_IRQ_VDD_WARN, + /* IRQ C */ + DA9062_IRQ_GPI0, + DA9062_IRQ_GPI1, + DA9062_IRQ_GPI2, + DA9062_IRQ_GPI3, + DA9062_IRQ_GPI4, + + DA9062_NUM_IRQ, +}; + +struct da9062 { + struct device *dev; + unsigned char device_id; + unsigned char variant_mrc; + struct regmap *regmap; + int chip_irq; + unsigned int irq_base; + struct regmap_irq_chip_data *regmap_irq; + int irq_vdd_warn; +}; + +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 *); + +#endif /* __MFD_DA9062_CORE_H__ */ diff --git a/include/linux/mfd/da9062/registers.h b/include/linux/mfd/da9062/registers.h new file mode 100644 index 0000000..d07c2bc --- /dev/null +++ b/include/linux/mfd/da9062/registers.h @@ -0,0 +1,1108 @@ +/* + * registers.h - REGISTERS H for DA9062 + * Copyright (C) 2015 Dialog Semiconductor Ltd. + * + * This program is free software; you can redistribute it and/or + * modify it under the terms of the GNU General Public License + * as published by the Free Software Foundation; either version 2 + * of the License, or (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + */ + +#ifndef __DA9062_H__ +#define __DA9062_H__ + +#define DA9062_PMIC_DEVICE_ID 0x62 +#define DA9062_PMIC_VARIANT_MRC_AA 0x01 + +#define DA9062_I2C_PAGE_SEL_SHIFT 1 + +/* + * Registers + */ + +#define DA9062AA_PAGE_CON 0x000 +#define DA9062AA_STATUS_A 0x001 +#define DA9062AA_STATUS_B 0x002 +#define DA9062AA_STATUS_D 0x004 +#define DA9062AA_FAULT_LOG 0x005 +#define DA9062AA_EVENT_A 0x006 +#define DA9062AA_EVENT_B 0x007 +#define DA9062AA_EVENT_C 0x008 +#define DA9062AA_IRQ_MASK_A 0x00A +#define DA9062AA_IRQ_MASK_B 0x00B +#define DA9062AA_IRQ_MASK_C 0x00C +#define DA9062AA_CONTROL_A 0x00E +#define DA9062AA_CONTROL_B 0x00F +#define DA9062AA_CONTROL_C 0x010 +#define DA9062AA_CONTROL_D 0x011 +#define DA9062AA_CONTROL_E 0x012 +#define DA9062AA_CONTROL_F 0x013 +#define DA9062AA_PD_DIS 0x014 +#define DA9062AA_GPIO_0_1 0x015 +#define DA9062AA_GPIO_2_3 0x016 +#define DA9062AA_GPIO_4 0x017 +#define DA9062AA_GPIO_WKUP_MODE 0x01C +#define DA9062AA_GPIO_MODE0_4 0x01D +#define DA9062AA_GPIO_OUT0_2 0x01E +#define DA9062AA_GPIO_OUT3_4 0x01F +#define DA9062AA_BUCK2_CONT 0x020 +#define DA9062AA_BUCK1_CONT 0x021 +#define DA9062AA_BUCK4_CONT 0x022 +#define DA9062AA_BUCK3_CONT 0x024 +#define DA9062AA_LDO1_CONT 0x026 +#define DA9062AA_LDO2_CONT 0x027 +#define DA9062AA_LDO3_CONT 0x028 +#define DA9062AA_LDO4_CONT 0x029 +#define DA9062AA_DVC_1 0x032 +#define DA9062AA_COUNT_S 0x040 +#define DA9062AA_COUNT_MI 0x041 +#define DA9062AA_COUNT_H 0x042 +#define DA9062AA_COUNT_D 0x043 +#define DA9062AA_COUNT_MO 0x044 +#define DA9062AA_COUNT_Y 0x045 +#define DA9062AA_ALARM_S 0x046 +#define DA9062AA_ALARM_MI 0x047 +#define DA9062AA_ALARM_H 0x048 +#define DA9062AA_ALARM_D 0x049 +#define DA9062AA_ALARM_MO 0x04A +#define DA9062AA_ALARM_Y 0x04B +#define DA9062AA_SECOND_A 0x04C +#define DA9062AA_SECOND_B 0x04D +#define DA9062AA_SECOND_C 0x04E +#define DA9062AA_SECOND_D 0x04F +#define DA9062AA_SEQ 0x081 +#define DA9062AA_SEQ_TIMER 0x082 +#define DA9062AA_ID_2_1 0x083 +#define DA9062AA_ID_4_3 0x084 +#define DA9062AA_ID_12_11 0x088 +#define DA9062AA_ID_14_13 0x089 +#define DA9062AA_ID_16_15 0x08A +#define DA9062AA_ID_22_21 0x08D +#define DA9062AA_ID_24_23 0x08E +#define DA9062AA_ID_26_25 0x08F +#define DA9062AA_ID_28_27 0x090 +#define DA9062AA_ID_30_29 0x091 +#define DA9062AA_ID_32_31 0x092 +#define DA9062AA_SEQ_A 0x095 +#define DA9062AA_SEQ_B 0x096 +#define DA9062AA_WAIT 0x097 +#define DA9062AA_EN_32K 0x098 +#define DA9062AA_RESET 0x099 +#define DA9062AA_BUCK_ILIM_A 0x09A +#define DA9062AA_BUCK_ILIM_B 0x09B +#define DA9062AA_BUCK_ILIM_C 0x09C +#define DA9062AA_BUCK2_CFG 0x09D +#define DA9062AA_BUCK1_CFG 0x09E +#define DA9062AA_BUCK4_CFG 0x09F +#define DA9062AA_BUCK3_CFG 0x0A0 +#define DA9062AA_VBUCK2_A 0x0A3 +#define DA9062AA_VBUCK1_A 0x0A4 +#define DA9062AA_VBUCK4_A 0x0A5 +#define DA9062AA_VBUCK3_A 0x0A7 +#define DA9062AA_VLDO1_A 0x0A9 +#define DA9062AA_VLDO2_A 0x0AA +#define DA9062AA_VLDO3_A 0x0AB +#define DA9062AA_VLDO4_A 0x0AC +#define DA9062AA_VBUCK2_B 0x0B4 +#define DA9062AA_VBUCK1_B 0x0B5 +#define DA9062AA_VBUCK4_B 0x0B6 +#define DA9062AA_VBUCK3_B 0x0B8 +#define DA9062AA_VLDO1_B 0x0BA +#define DA9062AA_VLDO2_B 0x0BB +#define DA9062AA_VLDO3_B 0x0BC +#define DA9062AA_VLDO4_B 0x0BD +#define DA9062AA_BBAT_CONT 0x0C5 +#define DA9062AA_INTERFACE 0x105 +#define DA9062AA_CONFIG_A 0x106 +#define DA9062AA_CONFIG_B 0x107 +#define DA9062AA_CONFIG_C 0x108 +#define DA9062AA_CONFIG_D 0x109 +#define DA9062AA_CONFIG_E 0x10A +#define DA9062AA_CONFIG_G 0x10C +#define DA9062AA_CONFIG_H 0x10D +#define DA9062AA_CONFIG_I 0x10E +#define DA9062AA_CONFIG_J 0x10F +#define DA9062AA_CONFIG_K 0x110 +#define DA9062AA_CONFIG_M 0x112 +#define DA9062AA_TRIM_CLDR 0x120 +#define DA9062AA_GP_ID_0 0x121 +#define DA9062AA_GP_ID_1 0x122 +#define DA9062AA_GP_ID_2 0x123 +#define DA9062AA_GP_ID_3 0x124 +#define DA9062AA_GP_ID_4 0x125 +#define DA9062AA_GP_ID_5 0x126 +#define DA9062AA_GP_ID_6 0x127 +#define DA9062AA_GP_ID_7 0x128 +#define DA9062AA_GP_ID_8 0x129 +#define DA9062AA_GP_ID_9 0x12A +#define DA9062AA_GP_ID_10 0x12B +#define DA9062AA_GP_ID_11 0x12C +#define DA9062AA_GP_ID_12 0x12D +#define DA9062AA_GP_ID_13 0x12E +#define DA9062AA_GP_ID_14 0x12F +#define DA9062AA_GP_ID_15 0x130 +#define DA9062AA_GP_ID_16 0x131 +#define DA9062AA_GP_ID_17 0x132 +#define DA9062AA_GP_ID_18 0x133 +#define DA9062AA_GP_ID_19 0x134 +#define DA9062AA_DEVICE_ID 0x181 +#define DA9062AA_VARIANT_ID 0x182 +#define DA9062AA_CUSTOMER_ID 0x183 +#define DA9062AA_CONFIG_ID 0x184 + +/* + * Bit fields + */ + +/* DA9062AA_PAGE_CON = 0x000 */ +#define DA9062AA_PAGE_SHIFT 0 +#define DA9062AA_PAGE_MASK (0x3f << 0) +#define DA9062AA_WRITE_MODE_SHIFT 6 +#define DA9062AA_WRITE_MODE_MASK (0x01 << 6) +#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__ */ -- end-of-patch for PATCH V3 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply related [flat|nested] 18+ messages in thread
* Re: [PATCH V3 1/4] mfd: da9062: DA9062 MFD core driver 2015-05-19 13:10 ` [PATCH V3 1/4] mfd: da9062: DA9062 MFD core driver S Twiss @ 2015-05-26 16:10 ` Lee Jones 2015-05-28 12:52 ` Opensource [Steve Twiss] 2015-06-11 8:27 ` Opensource [Steve Twiss] 0 siblings, 2 replies; 18+ messages in thread From: Lee Jones @ 2015-05-26 16:10 UTC (permalink / raw) To: S Twiss Cc: LINUXKERNEL, Samuel Ortiz, Alessandro Zummo, DEVICETREE, David Dajun Chen, Dmitry Torokhov, Ian Campbell, Kumar Gala, LINUXINPUT, LINUXWATCHDOG, Liam Girdwood, Mark Brown, Mark Rutland, Pawel Moll, RTCLINUX, Rob Herring, Support Opensource, Wim Van Sebroeck On Tue, 19 May 2015, S Twiss wrote: > From: S Twiss <stwiss.opensource@diasemi.com> > > Add MFD core driver support for DA9062 > > > Signed-off-by: Steve Twiss <stwiss.opensource@diasemi.com> > > --- > Changes in V3: > - Removed references to the RTC and OnKey in the mfd_cell definition. > > Changes in V2: > - Copyright headers GPL v2 (and later) match correct 'GPL' in MODULE_LICENSE > - Ensure DA9062 core is tristate in Kconfig so it can be built as a module > - Move VDD_WARN code and resource into the core instead of regulator driver > > This patch applies against linux-next and v4.1-rc3 > > > > drivers/mfd/Kconfig | 12 + > drivers/mfd/Makefile | 3 +- > drivers/mfd/da9062-core.c | 611 +++++++++++++++++++ > include/linux/mfd/da9062/core.h | 62 ++ > include/linux/mfd/da9062/registers.h | 1108 ++++++++++++++++++++++++++++++++++ > 5 files changed, 1795 insertions(+), 1 deletion(-) > create mode 100644 drivers/mfd/da9062-core.c > create mode 100644 include/linux/mfd/da9062/core.h > create mode 100644 include/linux/mfd/da9062/registers.h > > diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig > index d5ad04d..44fb2b9 100644 > --- a/drivers/mfd/Kconfig > +++ b/drivers/mfd/Kconfig > @@ -183,6 +183,18 @@ config MFD_DA9055 > This driver can be built as a module. If built as a module it will be > called "da9055" > > +config MFD_DA9062 > + tristate "Dialog Semiconductor DA9062 PMIC Support" > + select MFD_CORE > + select REGMAP_I2C > + select REGMAP_IRQ > + depends on I2C=y > + help > + Say yes here for support for the Dialog Semiconductor DA9062 PMIC. > + This includes the I2C driver and core APIs. > + Additional drivers must be enabled in order to use the functionality > + of the device. > + > config MFD_DA9063 > bool "Dialog Semiconductor DA9063 PMIC Support" > select MFD_CORE > diff --git a/drivers/mfd/Makefile b/drivers/mfd/Makefile > index 0e5cfeb..d94e562 100644 > --- a/drivers/mfd/Makefile > +++ b/drivers/mfd/Makefile > @@ -110,10 +110,11 @@ obj-$(CONFIG_MFD_LP8788) += lp8788.o lp8788-irq.o > > da9055-objs := da9055-core.o da9055-i2c.o > obj-$(CONFIG_MFD_DA9055) += da9055.o > - > +obj-$(CONFIG_MFD_DA9062) += da9062-core.o > da9063-objs := da9063-core.o da9063-irq.o da9063-i2c.o > obj-$(CONFIG_MFD_DA9063) += da9063.o > obj-$(CONFIG_MFD_DA9150) += da9150-core.o > + > obj-$(CONFIG_MFD_MAX14577) += max14577.o > obj-$(CONFIG_MFD_MAX77686) += max77686.o > obj-$(CONFIG_MFD_MAX77693) += max77693.o > diff --git a/drivers/mfd/da9062-core.c b/drivers/mfd/da9062-core.c > new file mode 100644 > index 0000000..5aea082 > --- /dev/null > +++ b/drivers/mfd/da9062-core.c > @@ -0,0 +1,611 @@ > +/* > + * da9062-core.c - CORE device driver for DA9062 Remove the filename. They have a habit of becoming incorrect. s/CORE/Core/ Can you also mention what the DA9062 actually is? > + * Copyright (C) 2015 Dialog Semiconductor Ltd. > + * > + * This program is free software; you can redistribute it and/or > + * modify it under the terms of the GNU General Public License > + * as published by the Free Software Foundation; either version 2 > + * of the License, or (at your option) any later version. > + * > + * This program is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > + * GNU General Public License for more details. > + */ > + > +#include <linux/kernel.h> > +#include <linux/module.h> > +#include <linux/init.h> > +#include <linux/slab.h> > +#include <linux/device.h> > +#include <linux/delay.h> > +#include <linux/interrupt.h> > +#include <linux/mutex.h> > +#include <linux/regmap.h> > + Why the '\n'? > +#include <linux/irq.h> > +#include <linux/mfd/core.h> > +#include <linux/i2c.h> > +#include <linux/err.h> > + Same here? > +#include <linux/mfd/da9062/core.h> > +#include <linux/mfd/da9062/registers.h> > +#include <linux/of.h> > +#include <linux/regulator/of_regulator.h> > + > +#include <linux/proc_fs.h> > +#include <linux/kthread.h> > +#include <linux/uaccess.h> I'm a bit concerned by the number of includes here, are you sure they're all required? > +/* IRQ device driver for DA9062 */ Not sure what this means? > +#define DA9062_REG_EVENT_A_OFFSET 0 > +#define DA9062_REG_EVENT_B_OFFSET 1 > +#define DA9062_REG_EVENT_C_OFFSET 2 > + > +static struct regmap_irq da9062_irqs[] = { > + /* EVENT A */ > + [DA9062_IRQ_ONKEY] = { > + .reg_offset = DA9062_REG_EVENT_A_OFFSET, > + .mask = DA9062AA_M_NONKEY_MASK, > + }, > + [DA9062_IRQ_ALARM] = { > + .reg_offset = DA9062_REG_EVENT_A_OFFSET, > + .mask = DA9062AA_M_ALARM_MASK, > + }, > + [DA9062_IRQ_TICK] = { > + .reg_offset = DA9062_REG_EVENT_A_OFFSET, > + .mask = DA9062AA_M_TICK_MASK, > + }, > + [DA9062_IRQ_WDG_WARN] = { > + .reg_offset = DA9062_REG_EVENT_A_OFFSET, > + .mask = DA9062AA_M_WDG_WARN_MASK, > + }, > + [DA9062_IRQ_SEQ_RDY] = { > + .reg_offset = DA9062_REG_EVENT_A_OFFSET, > + .mask = DA9062AA_M_SEQ_RDY_MASK, > + }, > + /* EVENT B */ > + [DA9062_IRQ_TEMP] = { > + .reg_offset = DA9062_REG_EVENT_B_OFFSET, > + .mask = DA9062AA_M_TEMP_MASK, > + }, > + [DA9062_IRQ_LDO_LIM] = { > + .reg_offset = DA9062_REG_EVENT_B_OFFSET, > + .mask = DA9062AA_M_LDO_LIM_MASK, > + }, > + [DA9062_IRQ_DVC_RDY] = { > + .reg_offset = DA9062_REG_EVENT_B_OFFSET, > + .mask = DA9062AA_M_DVC_RDY_MASK, > + }, > + [DA9062_IRQ_VDD_WARN] = { > + .reg_offset = DA9062_REG_EVENT_B_OFFSET, > + .mask = DA9062AA_M_VDD_WARN_MASK, > + }, > + /* EVENT C */ > + [DA9062_IRQ_GPI0] = { > + .reg_offset = DA9062_REG_EVENT_C_OFFSET, > + .mask = DA9062AA_M_GPI0_MASK, > + }, > + [DA9062_IRQ_GPI1] = { > + .reg_offset = DA9062_REG_EVENT_C_OFFSET, > + .mask = DA9062AA_M_GPI1_MASK, > + }, > + [DA9062_IRQ_GPI2] = { > + .reg_offset = DA9062_REG_EVENT_C_OFFSET, > + .mask = DA9062AA_M_GPI2_MASK, > + }, > + [DA9062_IRQ_GPI3] = { > + .reg_offset = DA9062_REG_EVENT_C_OFFSET, > + .mask = DA9062AA_M_GPI3_MASK, > + }, > + [DA9062_IRQ_GPI4] = { > + .reg_offset = DA9062_REG_EVENT_C_OFFSET, > + .mask = DA9062AA_M_GPI4_MASK, > + }, > +}; > + > +static struct regmap_irq_chip da9062_irq_chip = { > + /* IRQ */ Not required I think. > + .name = "da9062-irq", > + .irqs = da9062_irqs, > + .num_irqs = DA9062_NUM_IRQ, > + /* STATUS and EVENT */ This is a standard struct, no need for additional comments. > + .num_regs = 3, > + .status_base = DA9062AA_EVENT_A, > + .mask_base = DA9062AA_IRQ_MASK_A, > + .ack_base = DA9062AA_EVENT_A, > +}; > + > +int da9062_irq_init(struct da9062 *chip) > +{ > + int ret; > + > + if (!chip->chip_irq) { Check this once, in probe(), then rid this check. > + dev_err(chip->dev, "No IRQ configured\n"); > + return -EINVAL; > + } > + > + ret = regmap_add_irq_chip(chip->regmap, chip->chip_irq, > + IRQF_TRIGGER_LOW | IRQF_ONESHOT | IRQF_SHARED, > + chip->irq_base, &da9062_irq_chip, > + &chip->regmap_irq); This is just one call. Just place that call into da9062_device_init() and rid this function. > + if (ret) { > + dev_err(chip->dev, "Failed to request IRQ %d: %d\n", > + chip->chip_irq, ret); > + return ret; > + } > + > + return 0; > +} > + > +void da9062_irq_exit(struct da9062 *chip) > +{ > + regmap_del_irq_chip(chip->chip_irq, chip->regmap_irq); Again, this is abstraction for the sake of abstraction. > +} > + > +/* CORE device driver for DA9062 */ Doesn't add anything. This whole driver is the core driver. > +static struct resource da9062_core_resources[] = { const? > + { > + .name = "VDD_WARN", > + .start = DA9062_IRQ_VDD_WARN, > + .end = DA9062_IRQ_VDD_WARN, > + .flags = IORESOURCE_IRQ, > + }, > +}; > + > +static struct resource da9062_regulators_resources[] = { > + { > + .name = "LDO_LIM", > + .start = DA9062_IRQ_LDO_LIM, > + .end = DA9062_IRQ_LDO_LIM, > + .flags = IORESOURCE_IRQ, > + }, > +}; > + > +static struct resource da9062_thermal_resources[] = { > + { > + .name = "THERMAL", > + .start = DA9062_IRQ_TEMP, > + .end = DA9062_IRQ_TEMP, > + .flags = IORESOURCE_IRQ, > + }, > +}; > + > +static struct resource da9062_wdt_resources[] = { > + { > + .name = "WDG_WARN", > + .start = DA9062_IRQ_WDG_WARN, > + .end = DA9062_IRQ_WDG_WARN, > + .flags = IORESOURCE_IRQ, > + }, > +}; Convert all of these to oneliners using DEFINE_RES_* macros. > +static const struct mfd_cell da9062_devs[] = { > + { > + .name = "da9062-core", > + .num_resources = ARRAY_SIZE(da9062_core_resources), > + .resources = da9062_core_resources, > + }, > + { > + .name = "da9062-regulators", > + .num_resources = ARRAY_SIZE(da9062_regulators_resources), > + .resources = da9062_regulators_resources, > + }, > + { > + .name = "da9062-watchdog", > + .num_resources = ARRAY_SIZE(da9062_wdt_resources), > + .resources = da9062_wdt_resources, > + .of_compatible = "dlg,da9062-wdt", > + }, > + { > + .name = "da9062-thermal", > + .num_resources = ARRAY_SIZE(da9062_thermal_resources), > + .resources = da9062_thermal_resources, > + .of_compatible = "dlg,da9062-thermal", > + }, > +}; > + > +static irqreturn_t da9062_vdd_warn_event(int irq, void *data) > +{ > + struct da9062 *hw = data; > + > + dev_err(hw->dev, "VDD warning indicator\n"); Is it an error? Doesn't look like it. > + return IRQ_HANDLED; > +} > + > +static int da9062_clear_fault_log(struct da9062 *chip) > +{ > + int ret = 0; No need to pre-initialise. > + int fault_log = 0; As above. > + ret = regmap_read(chip->regmap, DA9062AA_FAULT_LOG, &fault_log); > + if (ret < 0) { > + dev_err(chip->dev, "Cannot read FAULT_LOG.\n"); > + ret = -EIO; Just return and rid the else. > + } else { > + if (fault_log) { > + if (fault_log & DA9062AA_TWD_ERROR_MASK) > + dev_dbg(chip->dev, "Fault log entry detected: TWD_ERROR\n"); > + if (fault_log & DA9062AA_POR_MASK) > + dev_dbg(chip->dev, "Fault log entry detected: POR\n"); > + if (fault_log & DA9062AA_VDD_FAULT_MASK) > + dev_dbg(chip->dev, "Fault log entry detected: VDD_FAULT\n"); > + if (fault_log & DA9062AA_VDD_START_MASK) > + dev_dbg(chip->dev, "Fault log entry detected: VDD_START\n"); > + if (fault_log & DA9062AA_TEMP_CRIT_MASK) > + dev_dbg(chip->dev, "Fault log entry detected: TEMP_CRIT\n"); > + if (fault_log & DA9062AA_KEY_RESET_MASK) > + dev_dbg(chip->dev, "Fault log entry detected: KEY_RESET\n"); > + if (fault_log & DA9062AA_NSHUTDOWN_MASK) > + dev_dbg(chip->dev, "Fault log entry detected: NSHUTDOWN\n"); > + if (fault_log & DA9062AA_WAIT_SHUT_MASK) > + dev_dbg(chip->dev, "Fault log entry detected: WAIT_SHUT\n"); > + } > + > + ret = regmap_write(chip->regmap, DA9062AA_FAULT_LOG, > + fault_log); > + if (ret < 0) > + dev_err(chip->dev, > + "Cannot reset FAULT_LOG values %d\n", ret); > + } > + > + return ret; > +} > + > +int da9062_device_init(struct da9062 *chip, unsigned int irq) No need to pass irq, as it's part of chip. > +{ > + int device_id, variant_id, variant_mrc; > + int irq_vdd_warn, ret; Put ret on its own line. > + ret = da9062_clear_fault_log(chip); > + if (ret < 0) > + dev_err(chip->dev, "Cannot clear fault log\n"); If this is an error, you must return. If it's just a warning then use dev_warn(). > + chip->irq_base = -1; Why are you pre-initialising? > + chip->chip_irq = irq; You already assigned irq to chip_irq. > + ret = regmap_read(chip->regmap, DA9062AA_DEVICE_ID, &device_id); > + if (ret < 0) { > + dev_err(chip->dev, "Cannot read chip ID.\n"); > + return -EIO; > + } > + if (device_id != DA9062_PMIC_DEVICE_ID) { > + dev_err(chip->dev, "Invalid device ID: 0x%02x\n", device_id); > + return -ENODEV; > + } > + > + ret = regmap_read(chip->regmap, DA9062AA_VARIANT_ID, &variant_id); > + if (ret < 0) { > + dev_err(chip->dev, "Cannot read chip variant id.\n"); > + return -EIO; > + } > + > + dev_info(chip->dev, > + "Device detected (device-ID: 0x%02X, var-ID: 0x%02X)\n", > + device_id, variant_id); Probably best to put this at the end. > + variant_mrc = (variant_id & DA9062AA_MRC_MASK) >> DA9062AA_MRC_SHIFT; > + > + if (variant_mrc < DA9062_PMIC_VARIANT_MRC_AA) { > + dev_err(chip->dev, > + "Cannot support variant MRC: 0x%02X\n", variant_mrc); > + return -ENODEV; > + } > + > + chip->device_id = device_id; > + chip->variant_mrc = variant_mrc; I'd put all of the device/variant stuff it its own function, then move everything else into probe() and remove da9062_device_init(). > + ret = da9062_irq_init(chip); > + if (ret) { > + dev_err(chip->dev, "Cannot initialize interrupts.\n"); > + return ret; > + } Remove this function and call regmap_add_irq_chip() from here. > + chip->irq_base = regmap_irq_chip_get_base(chip->regmap_irq); > + > + ret = mfd_add_devices(chip->dev, -1, da9062_devs, Use PLATFORM_DEVID_NONE instead. > + ARRAY_SIZE(da9062_devs), NULL, chip->irq_base, > + NULL); Usually child devices are probed at the end i.e. the last thing in probe(). > + if (ret) { > + dev_err(chip->dev, "Cannot add MFD cells\n"); "Cannot register child devices". > + da9062_irq_exit(chip); > + return ret; > + } > + > + /* VDD WARN event support */ > + irq_vdd_warn = regmap_irq_get_virq(chip->regmap_irq, > + DA9062_IRQ_VDD_WARN); > + if (irq_vdd_warn < 0) { > + dev_err(chip->dev, "Failed to get IRQ.\n"); > + return irq_vdd_warn; > + } > + chip->irq_vdd_warn = irq_vdd_warn; > + > + ret = devm_request_threaded_irq(chip->dev, irq_vdd_warn, > + NULL, da9062_vdd_warn_event, > + IRQF_TRIGGER_LOW | IRQF_ONESHOT, > + "VDD_WARN", chip); > + if (ret) { > + dev_warn(chip->dev, > + "Failed to request VDD_WARN IRQ.\n"); > + chip->irq_vdd_warn = -ENXIO; > + } This looks like a lot of code, which doesn't really do anything. What is a VDD warning indicator anyway? > + return ret; > +} > + > +void da9062_device_exit(struct da9062 *chip) > +{ > + mfd_remove_devices(chip->dev); > + da9062_irq_exit(chip); > +} Another seemingly pointless abstraction. Why don't you do this in remove()? > +/* I2C device driver for DA9062 */ Remove these. > +static const struct regmap_range da9062_aa_readable_ranges[] = { > + { > + .range_min = DA9062AA_PAGE_CON, > + .range_max = DA9062AA_STATUS_B, > + }, { > + .range_min = DA9062AA_STATUS_D, > + .range_max = DA9062AA_EVENT_C, > + }, { > + .range_min = DA9062AA_IRQ_MASK_A, > + .range_max = DA9062AA_IRQ_MASK_C, > + }, { > + .range_min = DA9062AA_CONTROL_A, > + .range_max = DA9062AA_GPIO_4, > + }, { > + .range_min = DA9062AA_GPIO_WKUP_MODE, > + .range_max = DA9062AA_BUCK4_CONT, > + }, { > + .range_min = DA9062AA_BUCK3_CONT, > + .range_max = DA9062AA_BUCK3_CONT, > + }, { > + .range_min = DA9062AA_LDO1_CONT, > + .range_max = DA9062AA_LDO4_CONT, > + }, { > + .range_min = DA9062AA_DVC_1, > + .range_max = DA9062AA_DVC_1, > + }, { > + .range_min = DA9062AA_COUNT_S, > + .range_max = DA9062AA_SECOND_D, > + }, { > + .range_min = DA9062AA_SEQ, > + .range_max = DA9062AA_ID_4_3, > + }, { > + .range_min = DA9062AA_ID_12_11, > + .range_max = DA9062AA_ID_16_15, > + }, { > + .range_min = DA9062AA_ID_22_21, > + .range_max = DA9062AA_ID_32_31, > + }, { > + .range_min = DA9062AA_SEQ_A, > + .range_max = DA9062AA_BUCK3_CFG, > + }, { > + .range_min = DA9062AA_VBUCK2_A, > + .range_max = DA9062AA_VBUCK4_A, > + }, { > + .range_min = DA9062AA_VBUCK3_A, > + .range_max = DA9062AA_VBUCK3_A, > + }, { > + .range_min = DA9062AA_VLDO1_A, > + .range_max = DA9062AA_VLDO4_A, > + }, { > + .range_min = DA9062AA_VBUCK2_B, > + .range_max = DA9062AA_VBUCK4_B, > + }, { > + .range_min = DA9062AA_VBUCK3_B, > + .range_max = DA9062AA_VBUCK3_B, > + }, { > + .range_min = DA9062AA_VLDO1_B, > + .range_max = DA9062AA_VLDO4_B, > + }, { > + .range_min = DA9062AA_BBAT_CONT, > + .range_max = DA9062AA_BBAT_CONT, > + }, { > + .range_min = DA9062AA_INTERFACE, > + .range_max = DA9062AA_CONFIG_E, > + }, { > + .range_min = DA9062AA_CONFIG_G, > + .range_max = DA9062AA_CONFIG_K, > + }, { > + .range_min = DA9062AA_CONFIG_M, > + .range_max = DA9062AA_CONFIG_M, > + }, { > + .range_min = DA9062AA_TRIM_CLDR, > + .range_max = DA9062AA_GP_ID_19, > + }, { > + .range_min = DA9062AA_DEVICE_ID, > + .range_max = DA9062AA_CONFIG_ID, > + }, > +}; > + > +static const struct regmap_range da9062_aa_writeable_ranges[] = { > + { > + .range_min = DA9062AA_PAGE_CON, > + .range_max = DA9062AA_PAGE_CON, > + }, { > + .range_min = DA9062AA_FAULT_LOG, > + .range_max = DA9062AA_EVENT_C, > + }, { > + .range_min = DA9062AA_IRQ_MASK_A, > + .range_max = DA9062AA_IRQ_MASK_C, > + }, { > + .range_min = DA9062AA_CONTROL_A, > + .range_max = DA9062AA_GPIO_4, > + }, { > + .range_min = DA9062AA_GPIO_WKUP_MODE, > + .range_max = DA9062AA_BUCK4_CONT, > + }, { > + .range_min = DA9062AA_BUCK3_CONT, > + .range_max = DA9062AA_BUCK3_CONT, > + }, { > + .range_min = DA9062AA_LDO1_CONT, > + .range_max = DA9062AA_LDO4_CONT, > + }, { > + .range_min = DA9062AA_DVC_1, > + .range_max = DA9062AA_DVC_1, > + }, { > + .range_min = DA9062AA_COUNT_S, > + .range_max = DA9062AA_ALARM_Y, > + }, { > + .range_min = DA9062AA_SEQ, > + .range_max = DA9062AA_ID_4_3, > + }, { > + .range_min = DA9062AA_ID_12_11, > + .range_max = DA9062AA_ID_16_15, > + }, { > + .range_min = DA9062AA_ID_22_21, > + .range_max = DA9062AA_ID_32_31, > + }, { > + .range_min = DA9062AA_SEQ_A, > + .range_max = DA9062AA_BUCK3_CFG, > + }, { > + .range_min = DA9062AA_VBUCK2_A, > + .range_max = DA9062AA_VBUCK4_A, > + }, { > + .range_min = DA9062AA_VBUCK3_A, > + .range_max = DA9062AA_VBUCK3_A, > + }, { > + .range_min = DA9062AA_VLDO1_A, > + .range_max = DA9062AA_VLDO4_A, > + }, { > + .range_min = DA9062AA_VBUCK2_B, > + .range_max = DA9062AA_VBUCK4_B, > + }, { > + .range_min = DA9062AA_VBUCK3_B, > + .range_max = DA9062AA_VBUCK3_B, > + }, { > + .range_min = DA9062AA_VLDO1_B, > + .range_max = DA9062AA_VLDO4_B, > + }, { > + .range_min = DA9062AA_BBAT_CONT, > + .range_max = DA9062AA_BBAT_CONT, > + }, { > + .range_min = DA9062AA_GP_ID_0, > + .range_max = DA9062AA_GP_ID_19, > + }, > +}; > + > +static const struct regmap_range da9062_aa_volatile_ranges[] = { > + { > + .range_min = DA9062AA_PAGE_CON, > + .range_max = DA9062AA_STATUS_B, > + }, { > + .range_min = DA9062AA_STATUS_D, > + .range_max = DA9062AA_EVENT_C, > + }, { > + .range_min = DA9062AA_CONTROL_F, > + .range_max = DA9062AA_CONTROL_F, > + }, { > + .range_min = DA9062AA_COUNT_S, > + .range_max = DA9062AA_SECOND_D, > + }, > +}; > + > +static const struct regmap_access_table da9062_aa_readable_table = { > + .yes_ranges = da9062_aa_readable_ranges, > + .n_yes_ranges = ARRAY_SIZE(da9062_aa_readable_ranges), > +}; > + > +static const struct regmap_access_table da9062_aa_writeable_table = { > + .yes_ranges = da9062_aa_writeable_ranges, > + .n_yes_ranges = ARRAY_SIZE(da9062_aa_writeable_ranges), > +}; > + > +static const struct regmap_access_table da9062_aa_volatile_table = { > + .yes_ranges = da9062_aa_volatile_ranges, > + .n_yes_ranges = ARRAY_SIZE(da9062_aa_volatile_ranges), > +}; > + > +static const struct regmap_range_cfg da9062_range_cfg[] = { > + { > + .range_min = DA9062AA_PAGE_CON, > + .range_max = DA9062AA_CONFIG_ID, > + .selector_reg = DA9062AA_PAGE_CON, > + .selector_mask = 1 << DA9062_I2C_PAGE_SEL_SHIFT, > + .selector_shift = DA9062_I2C_PAGE_SEL_SHIFT, > + .window_start = 0, > + .window_len = 256, > + } > +}; > + > +static struct regmap_config da9062_regmap_config = { > + .reg_bits = 8, > + .val_bits = 8, > + .ranges = da9062_range_cfg, > + .num_ranges = ARRAY_SIZE(da9062_range_cfg), > + .max_register = DA9062AA_CONFIG_ID, > + .cache_type = REGCACHE_RBTREE, > +}; > + > +static const struct of_device_id da9062_dt_ids[] = { > + { .compatible = "dlg,da9062", }, > + { } > +}; > +MODULE_DEVICE_TABLE(of, da9062_dt_ids); Move this just above where it's to be used i.e. down to the bottom. > +static int da9062_i2c_probe(struct i2c_client *i2c, > + const struct i2c_device_id *id) > +{ > + struct da9062 *chip; > + int ret; > + > + chip = devm_kzalloc(&i2c->dev, sizeof(struct da9062), GFP_KERNEL); sizeof(*chip) > + if (chip == NULL) if (!chip) > + return -ENOMEM; > + > + i2c_set_clientdata(i2c, chip); > + chip->dev = &i2c->dev; > + chip->chip_irq = i2c->irq; Check this once here, then you don't have to worry about it again. s/chip_irq/irq/ > + da9062_regmap_config.rd_table = &da9062_aa_readable_table; > + da9062_regmap_config.wr_table = &da9062_aa_writeable_table; > + da9062_regmap_config.volatile_table = &da9062_aa_volatile_table; Why are you doing this here instead of inside 'struct regmap_config da9062_regmap_config' above? > + chip->regmap = devm_regmap_init_i2c(i2c, &da9062_regmap_config); > + if (IS_ERR(chip->regmap)) { > + ret = PTR_ERR(chip->regmap); > + dev_err(chip->dev, "Failed to allocate register map: %d\n", > + ret); > + return ret; > + } > + > + return da9062_device_init(chip, i2c->irq); > +} > + > +static int da9062_i2c_remove(struct i2c_client *i2c) > +{ > + struct da9062 *chip = i2c_get_clientdata(i2c); > + > + da9062_device_exit(chip); Remove this. > + return 0; > +} > + > +static const struct i2c_device_id da9062_i2c_id[] = { > + {"da9062", 0}, White space discipline. > + {}, > +}; > +MODULE_DEVICE_TABLE(i2c, da9062_i2c_id); > + > +static struct i2c_driver da9062_i2c_driver = { > + .driver = { > + .name = "da9062", > + .of_match_table = of_match_ptr(da9062_dt_ids), > + }, > + .probe = da9062_i2c_probe, > + .remove = da9062_i2c_remove, > + .id_table = da9062_i2c_id, > +}; > + > +module_i2c_driver(da9062_i2c_driver); > + > + Superfluous '\n'. > +MODULE_DESCRIPTION("CORE device driver for Dialog DA9062"); s/CORE/Core/ > +MODULE_AUTHOR("S Twiss <stwiss.opensource@diasemi.com>"); Full names please. > +MODULE_LICENSE("GPL"); > diff --git a/include/linux/mfd/da9062/core.h b/include/linux/mfd/da9062/core.h > new file mode 100644 > index 0000000..0b17891 > --- /dev/null > +++ b/include/linux/mfd/da9062/core.h > @@ -0,0 +1,62 @@ > +/* > + * core.h - CORE H for DA9062 What does this add? > + * Copyright (C) 2015 Dialog Semiconductor Ltd. > + * > + * This program is free software; you can redistribute it and/or > + * modify it under the terms of the GNU General Public License > + * as published by the Free Software Foundation; either version 2 > + * of the License, or (at your option) any later version. > + * > + * This program is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > + * GNU General Public License for more details. > + */ > + > +#ifndef __MFD_DA9062_CORE_H__ > +#define __MFD_DA9062_CORE_H__ > + > +#include <linux/interrupt.h> > +#include <linux/mfd/da9062/registers.h> > + > +/* Interrupts */ > +enum da9062_irqs { > + /* IRQ A */ > + DA9062_IRQ_ONKEY, > + DA9062_IRQ_ALARM, > + DA9062_IRQ_TICK, > + DA9062_IRQ_WDG_WARN, > + DA9062_IRQ_SEQ_RDY, > + /* IRQ B*/ > + DA9062_IRQ_TEMP, > + DA9062_IRQ_LDO_LIM, > + DA9062_IRQ_DVC_RDY, > + DA9062_IRQ_VDD_WARN, > + /* IRQ C */ > + DA9062_IRQ_GPI0, > + DA9062_IRQ_GPI1, > + DA9062_IRQ_GPI2, > + DA9062_IRQ_GPI3, > + DA9062_IRQ_GPI4, > + > + DA9062_NUM_IRQ, > +}; > + > +struct da9062 { > + struct device *dev; > + unsigned char device_id; > + unsigned char variant_mrc; > + struct regmap *regmap; > + int chip_irq; > + unsigned int irq_base; > + struct regmap_irq_chip_data *regmap_irq; > + int irq_vdd_warn; > +}; Are all of these used by child devices? > +int da9062_device_init(struct da9062 *, unsigned int); > +int da9062_irq_init(struct da9062 *); > + > +void da9062_device_exit(struct da9062 *); > +void da9062_irq_exit(struct da9062 *); Why are these required? > +#endif /* __MFD_DA9062_CORE_H__ */ > diff --git a/include/linux/mfd/da9062/registers.h b/include/linux/mfd/da9062/registers.h > new file mode 100644 > index 0000000..d07c2bc > --- /dev/null > +++ b/include/linux/mfd/da9062/registers.h > @@ -0,0 +1,1108 @@ > +/* > + * registers.h - REGISTERS H for DA9062 > + * Copyright (C) 2015 Dialog Semiconductor Ltd. > + * > + * This program is free software; you can redistribute it and/or > + * modify it under the terms of the GNU General Public License > + * as published by the Free Software Foundation; either version 2 > + * of the License, or (at your option) any later version. > + * > + * This program is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > + * GNU General Public License for more details. > + */ > + > +#ifndef __DA9062_H__ > +#define __DA9062_H__ > + > +#define DA9062_PMIC_DEVICE_ID 0x62 > +#define DA9062_PMIC_VARIANT_MRC_AA 0x01 > + > +#define DA9062_I2C_PAGE_SEL_SHIFT 1 > + > +/* > + * Registers > + */ Really? ;) > +#define DA9062AA_PAGE_CON 0x000 > +#define DA9062AA_STATUS_A 0x001 > +#define DA9062AA_STATUS_B 0x002 > +#define DA9062AA_STATUS_D 0x004 > +#define DA9062AA_FAULT_LOG 0x005 > +#define DA9062AA_EVENT_A 0x006 > +#define DA9062AA_EVENT_B 0x007 > +#define DA9062AA_EVENT_C 0x008 > +#define DA9062AA_IRQ_MASK_A 0x00A > +#define DA9062AA_IRQ_MASK_B 0x00B > +#define DA9062AA_IRQ_MASK_C 0x00C > +#define DA9062AA_CONTROL_A 0x00E > +#define DA9062AA_CONTROL_B 0x00F > +#define DA9062AA_CONTROL_C 0x010 > +#define DA9062AA_CONTROL_D 0x011 > +#define DA9062AA_CONTROL_E 0x012 > +#define DA9062AA_CONTROL_F 0x013 > +#define DA9062AA_PD_DIS 0x014 > +#define DA9062AA_GPIO_0_1 0x015 > +#define DA9062AA_GPIO_2_3 0x016 > +#define DA9062AA_GPIO_4 0x017 > +#define DA9062AA_GPIO_WKUP_MODE 0x01C > +#define DA9062AA_GPIO_MODE0_4 0x01D > +#define DA9062AA_GPIO_OUT0_2 0x01E > +#define DA9062AA_GPIO_OUT3_4 0x01F > +#define DA9062AA_BUCK2_CONT 0x020 > +#define DA9062AA_BUCK1_CONT 0x021 > +#define DA9062AA_BUCK4_CONT 0x022 > +#define DA9062AA_BUCK3_CONT 0x024 > +#define DA9062AA_LDO1_CONT 0x026 > +#define DA9062AA_LDO2_CONT 0x027 > +#define DA9062AA_LDO3_CONT 0x028 > +#define DA9062AA_LDO4_CONT 0x029 > +#define DA9062AA_DVC_1 0x032 > +#define DA9062AA_COUNT_S 0x040 > +#define DA9062AA_COUNT_MI 0x041 > +#define DA9062AA_COUNT_H 0x042 > +#define DA9062AA_COUNT_D 0x043 > +#define DA9062AA_COUNT_MO 0x044 > +#define DA9062AA_COUNT_Y 0x045 > +#define DA9062AA_ALARM_S 0x046 > +#define DA9062AA_ALARM_MI 0x047 > +#define DA9062AA_ALARM_H 0x048 > +#define DA9062AA_ALARM_D 0x049 > +#define DA9062AA_ALARM_MO 0x04A > +#define DA9062AA_ALARM_Y 0x04B > +#define DA9062AA_SECOND_A 0x04C > +#define DA9062AA_SECOND_B 0x04D > +#define DA9062AA_SECOND_C 0x04E > +#define DA9062AA_SECOND_D 0x04F > +#define DA9062AA_SEQ 0x081 > +#define DA9062AA_SEQ_TIMER 0x082 > +#define DA9062AA_ID_2_1 0x083 > +#define DA9062AA_ID_4_3 0x084 > +#define DA9062AA_ID_12_11 0x088 > +#define DA9062AA_ID_14_13 0x089 > +#define DA9062AA_ID_16_15 0x08A > +#define DA9062AA_ID_22_21 0x08D > +#define DA9062AA_ID_24_23 0x08E > +#define DA9062AA_ID_26_25 0x08F > +#define DA9062AA_ID_28_27 0x090 > +#define DA9062AA_ID_30_29 0x091 > +#define DA9062AA_ID_32_31 0x092 > +#define DA9062AA_SEQ_A 0x095 > +#define DA9062AA_SEQ_B 0x096 > +#define DA9062AA_WAIT 0x097 > +#define DA9062AA_EN_32K 0x098 > +#define DA9062AA_RESET 0x099 > +#define DA9062AA_BUCK_ILIM_A 0x09A > +#define DA9062AA_BUCK_ILIM_B 0x09B > +#define DA9062AA_BUCK_ILIM_C 0x09C > +#define DA9062AA_BUCK2_CFG 0x09D > +#define DA9062AA_BUCK1_CFG 0x09E > +#define DA9062AA_BUCK4_CFG 0x09F > +#define DA9062AA_BUCK3_CFG 0x0A0 > +#define DA9062AA_VBUCK2_A 0x0A3 > +#define DA9062AA_VBUCK1_A 0x0A4 > +#define DA9062AA_VBUCK4_A 0x0A5 > +#define DA9062AA_VBUCK3_A 0x0A7 > +#define DA9062AA_VLDO1_A 0x0A9 > +#define DA9062AA_VLDO2_A 0x0AA > +#define DA9062AA_VLDO3_A 0x0AB > +#define DA9062AA_VLDO4_A 0x0AC > +#define DA9062AA_VBUCK2_B 0x0B4 > +#define DA9062AA_VBUCK1_B 0x0B5 > +#define DA9062AA_VBUCK4_B 0x0B6 > +#define DA9062AA_VBUCK3_B 0x0B8 > +#define DA9062AA_VLDO1_B 0x0BA > +#define DA9062AA_VLDO2_B 0x0BB > +#define DA9062AA_VLDO3_B 0x0BC > +#define DA9062AA_VLDO4_B 0x0BD > +#define DA9062AA_BBAT_CONT 0x0C5 > +#define DA9062AA_INTERFACE 0x105 > +#define DA9062AA_CONFIG_A 0x106 > +#define DA9062AA_CONFIG_B 0x107 > +#define DA9062AA_CONFIG_C 0x108 > +#define DA9062AA_CONFIG_D 0x109 > +#define DA9062AA_CONFIG_E 0x10A > +#define DA9062AA_CONFIG_G 0x10C > +#define DA9062AA_CONFIG_H 0x10D > +#define DA9062AA_CONFIG_I 0x10E > +#define DA9062AA_CONFIG_J 0x10F > +#define DA9062AA_CONFIG_K 0x110 > +#define DA9062AA_CONFIG_M 0x112 > +#define DA9062AA_TRIM_CLDR 0x120 > +#define DA9062AA_GP_ID_0 0x121 > +#define DA9062AA_GP_ID_1 0x122 > +#define DA9062AA_GP_ID_2 0x123 > +#define DA9062AA_GP_ID_3 0x124 > +#define DA9062AA_GP_ID_4 0x125 > +#define DA9062AA_GP_ID_5 0x126 > +#define DA9062AA_GP_ID_6 0x127 > +#define DA9062AA_GP_ID_7 0x128 > +#define DA9062AA_GP_ID_8 0x129 > +#define DA9062AA_GP_ID_9 0x12A > +#define DA9062AA_GP_ID_10 0x12B > +#define DA9062AA_GP_ID_11 0x12C > +#define DA9062AA_GP_ID_12 0x12D > +#define DA9062AA_GP_ID_13 0x12E > +#define DA9062AA_GP_ID_14 0x12F > +#define DA9062AA_GP_ID_15 0x130 > +#define DA9062AA_GP_ID_16 0x131 > +#define DA9062AA_GP_ID_17 0x132 > +#define DA9062AA_GP_ID_18 0x133 > +#define DA9062AA_GP_ID_19 0x134 > +#define DA9062AA_DEVICE_ID 0x181 > +#define DA9062AA_VARIANT_ID 0x182 > +#define DA9062AA_CUSTOMER_ID 0x183 > +#define DA9062AA_CONFIG_ID 0x184 > + > +/* > + * Bit fields > + */ > + > +/* DA9062AA_PAGE_CON = 0x000 */ > +#define DA9062AA_PAGE_SHIFT 0 > +#define DA9062AA_PAGE_MASK (0x3f << 0) > +#define DA9062AA_WRITE_MODE_SHIFT 6 > +#define DA9062AA_WRITE_MODE_MASK (0x01 << 6) For 1 << X, you should use BIT(X). > +#define DA9062AA_REVERT_SHIFT 7 > +#define DA9062AA_REVERT_MASK (0x01 << 7) > + > +/* DA9062AA_STATUS_A = 0x001 */ > +#define DA9062AA_NONKEY_SHIFT 0 > +#define DA9062AA_NONKEY_MASK (0x01 << 0) > +#define DA9062AA_DVC_BUSY_SHIFT 2 > +#define DA9062AA_DVC_BUSY_MASK (0x01 << 2) > + > +/* DA9062AA_STATUS_B = 0x002 */ > +#define DA9062AA_GPI0_SHIFT 0 > +#define DA9062AA_GPI0_MASK (0x01 << 0) > +#define DA9062AA_GPI1_SHIFT 1 > +#define DA9062AA_GPI1_MASK (0x01 << 1) > +#define DA9062AA_GPI2_SHIFT 2 > +#define DA9062AA_GPI2_MASK (0x01 << 2) > +#define DA9062AA_GPI3_SHIFT 3 > +#define DA9062AA_GPI3_MASK (0x01 << 3) > +#define DA9062AA_GPI4_SHIFT 4 > +#define DA9062AA_GPI4_MASK (0x01 << 4) > + > +/* DA9062AA_STATUS_D = 0x004 */ > +#define DA9062AA_LDO1_ILIM_SHIFT 0 > +#define DA9062AA_LDO1_ILIM_MASK (0x01 << 0) > +#define DA9062AA_LDO2_ILIM_SHIFT 1 > +#define DA9062AA_LDO2_ILIM_MASK (0x01 << 1) > +#define DA9062AA_LDO3_ILIM_SHIFT 2 > +#define DA9062AA_LDO3_ILIM_MASK (0x01 << 2) > +#define DA9062AA_LDO4_ILIM_SHIFT 3 > +#define DA9062AA_LDO4_ILIM_MASK (0x01 << 3) > + > +/* DA9062AA_FAULT_LOG = 0x005 */ > +#define DA9062AA_TWD_ERROR_SHIFT 0 > +#define DA9062AA_TWD_ERROR_MASK (0x01 << 0) > +#define DA9062AA_POR_SHIFT 1 > +#define DA9062AA_POR_MASK (0x01 << 1) > +#define DA9062AA_VDD_FAULT_SHIFT 2 > +#define DA9062AA_VDD_FAULT_MASK (0x01 << 2) > +#define DA9062AA_VDD_START_SHIFT 3 > +#define DA9062AA_VDD_START_MASK (0x01 << 3) > +#define DA9062AA_TEMP_CRIT_SHIFT 4 > +#define DA9062AA_TEMP_CRIT_MASK (0x01 << 4) > +#define DA9062AA_KEY_RESET_SHIFT 5 > +#define DA9062AA_KEY_RESET_MASK (0x01 << 5) > +#define DA9062AA_NSHUTDOWN_SHIFT 6 > +#define DA9062AA_NSHUTDOWN_MASK (0x01 << 6) > +#define DA9062AA_WAIT_SHUT_SHIFT 7 > +#define DA9062AA_WAIT_SHUT_MASK (0x01 << 7) > + > +/* DA9062AA_EVENT_A = 0x006 */ > +#define DA9062AA_E_NONKEY_SHIFT 0 > +#define DA9062AA_E_NONKEY_MASK (0x01 << 0) > +#define DA9062AA_E_ALARM_SHIFT 1 > +#define DA9062AA_E_ALARM_MASK (0x01 << 1) > +#define DA9062AA_E_TICK_SHIFT 2 > +#define DA9062AA_E_TICK_MASK (0x01 << 2) > +#define DA9062AA_E_WDG_WARN_SHIFT 3 > +#define DA9062AA_E_WDG_WARN_MASK (0x01 << 3) > +#define DA9062AA_E_SEQ_RDY_SHIFT 4 > +#define DA9062AA_E_SEQ_RDY_MASK (0x01 << 4) > +#define DA9062AA_EVENTS_B_SHIFT 5 > +#define DA9062AA_EVENTS_B_MASK (0x01 << 5) > +#define DA9062AA_EVENTS_C_SHIFT 6 > +#define DA9062AA_EVENTS_C_MASK (0x01 << 6) > + > +/* DA9062AA_EVENT_B = 0x007 */ > +#define DA9062AA_E_TEMP_SHIFT 1 > +#define DA9062AA_E_TEMP_MASK (0x01 << 1) > +#define DA9062AA_E_LDO_LIM_SHIFT 3 > +#define DA9062AA_E_LDO_LIM_MASK (0x01 << 3) > +#define DA9062AA_E_DVC_RDY_SHIFT 5 > +#define DA9062AA_E_DVC_RDY_MASK (0x01 << 5) > +#define DA9062AA_E_VDD_WARN_SHIFT 7 > +#define DA9062AA_E_VDD_WARN_MASK (0x01 << 7) > + > +/* DA9062AA_EVENT_C = 0x008 */ > +#define DA9062AA_E_GPI0_SHIFT 0 > +#define DA9062AA_E_GPI0_MASK (0x01 << 0) > +#define DA9062AA_E_GPI1_SHIFT 1 > +#define DA9062AA_E_GPI1_MASK (0x01 << 1) > +#define DA9062AA_E_GPI2_SHIFT 2 > +#define DA9062AA_E_GPI2_MASK (0x01 << 2) > +#define DA9062AA_E_GPI3_SHIFT 3 > +#define DA9062AA_E_GPI3_MASK (0x01 << 3) > +#define DA9062AA_E_GPI4_SHIFT 4 > +#define DA9062AA_E_GPI4_MASK (0x01 << 4) > + > +/* DA9062AA_IRQ_MASK_A = 0x00A */ > +#define DA9062AA_M_NONKEY_SHIFT 0 > +#define DA9062AA_M_NONKEY_MASK (0x01 << 0) > +#define DA9062AA_M_ALARM_SHIFT 1 > +#define DA9062AA_M_ALARM_MASK (0x01 << 1) > +#define DA9062AA_M_TICK_SHIFT 2 > +#define DA9062AA_M_TICK_MASK (0x01 << 2) > +#define DA9062AA_M_WDG_WARN_SHIFT 3 > +#define DA9062AA_M_WDG_WARN_MASK (0x01 << 3) > +#define DA9062AA_M_SEQ_RDY_SHIFT 4 > +#define DA9062AA_M_SEQ_RDY_MASK (0x01 << 4) > + > +/* DA9062AA_IRQ_MASK_B = 0x00B */ > +#define DA9062AA_M_TEMP_SHIFT 1 > +#define DA9062AA_M_TEMP_MASK (0x01 << 1) > +#define DA9062AA_M_LDO_LIM_SHIFT 3 > +#define DA9062AA_M_LDO_LIM_MASK (0x01 << 3) > +#define DA9062AA_M_DVC_RDY_SHIFT 5 > +#define DA9062AA_M_DVC_RDY_MASK (0x01 << 5) > +#define DA9062AA_M_VDD_WARN_SHIFT 7 > +#define DA9062AA_M_VDD_WARN_MASK (0x01 << 7) > + > +/* DA9062AA_IRQ_MASK_C = 0x00C */ > +#define DA9062AA_M_GPI0_SHIFT 0 > +#define DA9062AA_M_GPI0_MASK (0x01 << 0) > +#define DA9062AA_M_GPI1_SHIFT 1 > +#define DA9062AA_M_GPI1_MASK (0x01 << 1) > +#define DA9062AA_M_GPI2_SHIFT 2 > +#define DA9062AA_M_GPI2_MASK (0x01 << 2) > +#define DA9062AA_M_GPI3_SHIFT 3 > +#define DA9062AA_M_GPI3_MASK (0x01 << 3) > +#define DA9062AA_M_GPI4_SHIFT 4 > +#define DA9062AA_M_GPI4_MASK (0x01 << 4) > + > +/* DA9062AA_CONTROL_A = 0x00E */ > +#define DA9062AA_SYSTEM_EN_SHIFT 0 > +#define DA9062AA_SYSTEM_EN_MASK (0x01 << 0) > +#define DA9062AA_POWER_EN_SHIFT 1 > +#define DA9062AA_POWER_EN_MASK (0x01 << 1) > +#define DA9062AA_POWER1_EN_SHIFT 2 > +#define DA9062AA_POWER1_EN_MASK (0x01 << 2) > +#define DA9062AA_STANDBY_SHIFT 3 > +#define DA9062AA_STANDBY_MASK (0x01 << 3) > +#define DA9062AA_M_SYSTEM_EN_SHIFT 4 > +#define DA9062AA_M_SYSTEM_EN_MASK (0x01 << 4) > +#define DA9062AA_M_POWER_EN_SHIFT 5 > +#define DA9062AA_M_POWER_EN_MASK (0x01 << 5) > +#define DA9062AA_M_POWER1_EN_SHIFT 6 > +#define DA9062AA_M_POWER1_EN_MASK (0x01 << 6) > + > +/* DA9062AA_CONTROL_B = 0x00F */ > +#define DA9062AA_WATCHDOG_PD_SHIFT 1 > +#define DA9062AA_WATCHDOG_PD_MASK (0x01 << 1) > +#define DA9062AA_FREEZE_EN_SHIFT 2 > +#define DA9062AA_FREEZE_EN_MASK (0x01 << 2) > +#define DA9062AA_NRES_MODE_SHIFT 3 > +#define DA9062AA_NRES_MODE_MASK (0x01 << 3) > +#define DA9062AA_NONKEY_LOCK_SHIFT 4 > +#define DA9062AA_NONKEY_LOCK_MASK (0x01 << 4) > +#define DA9062AA_NFREEZE_SHIFT 5 > +#define DA9062AA_NFREEZE_MASK (0x03 << 5) > +#define DA9062AA_BUCK_SLOWSTART_SHIFT 7 > +#define DA9062AA_BUCK_SLOWSTART_MASK (0x01 << 7) > + > +/* DA9062AA_CONTROL_C = 0x010 */ > +#define DA9062AA_DEBOUNCING_SHIFT 0 > +#define DA9062AA_DEBOUNCING_MASK (0x07 << 0) > +#define DA9062AA_AUTO_BOOT_SHIFT 3 > +#define DA9062AA_AUTO_BOOT_MASK (0x01 << 3) > +#define DA9062AA_OTPREAD_EN_SHIFT 4 > +#define DA9062AA_OTPREAD_EN_MASK (0x01 << 4) > +#define DA9062AA_SLEW_RATE_SHIFT 5 > +#define DA9062AA_SLEW_RATE_MASK (0x03 << 5) > +#define DA9062AA_DEF_SUPPLY_SHIFT 7 > +#define DA9062AA_DEF_SUPPLY_MASK (0x01 << 7) > + > +/* DA9062AA_CONTROL_D = 0x011 */ > +#define DA9062AA_TWDSCALE_SHIFT 0 > +#define DA9062AA_TWDSCALE_MASK (0x07 << 0) > + > +/* DA9062AA_CONTROL_E = 0x012 */ > +#define DA9062AA_RTC_MODE_PD_SHIFT 0 > +#define DA9062AA_RTC_MODE_PD_MASK (0x01 << 0) > +#define DA9062AA_RTC_MODE_SD_SHIFT 1 > +#define DA9062AA_RTC_MODE_SD_MASK (0x01 << 1) > +#define DA9062AA_RTC_EN_SHIFT 2 > +#define DA9062AA_RTC_EN_MASK (0x01 << 2) > +#define DA9062AA_V_LOCK_SHIFT 7 > +#define DA9062AA_V_LOCK_MASK (0x01 << 7) > + > +/* DA9062AA_CONTROL_F = 0x013 */ > +#define DA9062AA_WATCHDOG_SHIFT 0 > +#define DA9062AA_WATCHDOG_MASK (0x01 << 0) > +#define DA9062AA_SHUTDOWN_SHIFT 1 > +#define DA9062AA_SHUTDOWN_MASK (0x01 << 1) > +#define DA9062AA_WAKE_UP_SHIFT 2 > +#define DA9062AA_WAKE_UP_MASK (0x01 << 2) > + > +/* DA9062AA_PD_DIS = 0x014 */ > +#define DA9062AA_GPI_DIS_SHIFT 0 > +#define DA9062AA_GPI_DIS_MASK (0x01 << 0) > +#define DA9062AA_PMIF_DIS_SHIFT 2 > +#define DA9062AA_PMIF_DIS_MASK (0x01 << 2) > +#define DA9062AA_CLDR_PAUSE_SHIFT 4 > +#define DA9062AA_CLDR_PAUSE_MASK (0x01 << 4) > +#define DA9062AA_BBAT_DIS_SHIFT 5 > +#define DA9062AA_BBAT_DIS_MASK (0x01 << 5) > +#define DA9062AA_OUT32K_PAUSE_SHIFT 6 > +#define DA9062AA_OUT32K_PAUSE_MASK (0x01 << 6) > +#define DA9062AA_PMCONT_DIS_SHIFT 7 > +#define DA9062AA_PMCONT_DIS_MASK (0x01 << 7) > + > +/* DA9062AA_GPIO_0_1 = 0x015 */ > +#define DA9062AA_GPIO0_PIN_SHIFT 0 > +#define DA9062AA_GPIO0_PIN_MASK (0x03 << 0) > +#define DA9062AA_GPIO0_TYPE_SHIFT 2 > +#define DA9062AA_GPIO0_TYPE_MASK (0x01 << 2) > +#define DA9062AA_GPIO0_WEN_SHIFT 3 > +#define DA9062AA_GPIO0_WEN_MASK (0x01 << 3) > +#define DA9062AA_GPIO1_PIN_SHIFT 4 > +#define DA9062AA_GPIO1_PIN_MASK (0x03 << 4) > +#define DA9062AA_GPIO1_TYPE_SHIFT 6 > +#define DA9062AA_GPIO1_TYPE_MASK (0x01 << 6) > +#define DA9062AA_GPIO1_WEN_SHIFT 7 > +#define DA9062AA_GPIO1_WEN_MASK (0x01 << 7) > + > +/* DA9062AA_GPIO_2_3 = 0x016 */ > +#define DA9062AA_GPIO2_PIN_SHIFT 0 > +#define DA9062AA_GPIO2_PIN_MASK (0x03 << 0) > +#define DA9062AA_GPIO2_TYPE_SHIFT 2 > +#define DA9062AA_GPIO2_TYPE_MASK (0x01 << 2) > +#define DA9062AA_GPIO2_WEN_SHIFT 3 > +#define DA9062AA_GPIO2_WEN_MASK (0x01 << 3) > +#define DA9062AA_GPIO3_PIN_SHIFT 4 > +#define DA9062AA_GPIO3_PIN_MASK (0x03 << 4) > +#define DA9062AA_GPIO3_TYPE_SHIFT 6 > +#define DA9062AA_GPIO3_TYPE_MASK (0x01 << 6) > +#define DA9062AA_GPIO3_WEN_SHIFT 7 > +#define DA9062AA_GPIO3_WEN_MASK (0x01 << 7) > + > +/* DA9062AA_GPIO_4 = 0x017 */ > +#define DA9062AA_GPIO4_PIN_SHIFT 0 > +#define DA9062AA_GPIO4_PIN_MASK (0x03 << 0) > +#define DA9062AA_GPIO4_TYPE_SHIFT 2 > +#define DA9062AA_GPIO4_TYPE_MASK (0x01 << 2) > +#define DA9062AA_GPIO4_WEN_SHIFT 3 > +#define DA9062AA_GPIO4_WEN_MASK (0x01 << 3) > + > +/* DA9062AA_GPIO_WKUP_MODE = 0x01C */ > +#define DA9062AA_GPIO0_WKUP_MODE_SHIFT 0 > +#define DA9062AA_GPIO0_WKUP_MODE_MASK (0x01 << 0) > +#define DA9062AA_GPIO1_WKUP_MODE_SHIFT 1 > +#define DA9062AA_GPIO1_WKUP_MODE_MASK (0x01 << 1) > +#define DA9062AA_GPIO2_WKUP_MODE_SHIFT 2 > +#define DA9062AA_GPIO2_WKUP_MODE_MASK (0x01 << 2) > +#define DA9062AA_GPIO3_WKUP_MODE_SHIFT 3 > +#define DA9062AA_GPIO3_WKUP_MODE_MASK (0x01 << 3) > +#define DA9062AA_GPIO4_WKUP_MODE_SHIFT 4 > +#define DA9062AA_GPIO4_WKUP_MODE_MASK (0x01 << 4) > + > +/* DA9062AA_GPIO_MODE0_4 = 0x01D */ > +#define DA9062AA_GPIO0_MODE_SHIFT 0 > +#define DA9062AA_GPIO0_MODE_MASK (0x01 << 0) > +#define DA9062AA_GPIO1_MODE_SHIFT 1 > +#define DA9062AA_GPIO1_MODE_MASK (0x01 << 1) > +#define DA9062AA_GPIO2_MODE_SHIFT 2 > +#define DA9062AA_GPIO2_MODE_MASK (0x01 << 2) > +#define DA9062AA_GPIO3_MODE_SHIFT 3 > +#define DA9062AA_GPIO3_MODE_MASK (0x01 << 3) > +#define DA9062AA_GPIO4_MODE_SHIFT 4 > +#define DA9062AA_GPIO4_MODE_MASK (0x01 << 4) > + > +/* DA9062AA_GPIO_OUT0_2 = 0x01E */ > +#define DA9062AA_GPIO0_OUT_SHIFT 0 > +#define DA9062AA_GPIO0_OUT_MASK (0x07 << 0) > +#define DA9062AA_GPIO1_OUT_SHIFT 3 > +#define DA9062AA_GPIO1_OUT_MASK (0x07 << 3) > +#define DA9062AA_GPIO2_OUT_SHIFT 6 > +#define DA9062AA_GPIO2_OUT_MASK (0x03 << 6) > + > +/* DA9062AA_GPIO_OUT3_4 = 0x01F */ > +#define DA9062AA_GPIO3_OUT_SHIFT 0 > +#define DA9062AA_GPIO3_OUT_MASK (0x07 << 0) > +#define DA9062AA_GPIO4_OUT_SHIFT 3 > +#define DA9062AA_GPIO4_OUT_MASK (0x03 << 3) > + > +/* DA9062AA_BUCK2_CONT = 0x020 */ > +#define DA9062AA_BUCK2_EN_SHIFT 0 > +#define DA9062AA_BUCK2_EN_MASK (0x01 << 0) > +#define DA9062AA_BUCK2_GPI_SHIFT 1 > +#define DA9062AA_BUCK2_GPI_MASK (0x03 << 1) > +#define DA9062AA_BUCK2_CONF_SHIFT 3 > +#define DA9062AA_BUCK2_CONF_MASK (0x01 << 3) > +#define DA9062AA_VBUCK2_GPI_SHIFT 5 > +#define DA9062AA_VBUCK2_GPI_MASK (0x03 << 5) > + > +/* DA9062AA_BUCK1_CONT = 0x021 */ > +#define DA9062AA_BUCK1_EN_SHIFT 0 > +#define DA9062AA_BUCK1_EN_MASK (0x01 << 0) > +#define DA9062AA_BUCK1_GPI_SHIFT 1 > +#define DA9062AA_BUCK1_GPI_MASK (0x03 << 1) > +#define DA9062AA_BUCK1_CONF_SHIFT 3 > +#define DA9062AA_BUCK1_CONF_MASK (0x01 << 3) > +#define DA9062AA_VBUCK1_GPI_SHIFT 5 > +#define DA9062AA_VBUCK1_GPI_MASK (0x03 << 5) > + > +/* DA9062AA_BUCK4_CONT = 0x022 */ > +#define DA9062AA_BUCK4_EN_SHIFT 0 > +#define DA9062AA_BUCK4_EN_MASK (0x01 << 0) > +#define DA9062AA_BUCK4_GPI_SHIFT 1 > +#define DA9062AA_BUCK4_GPI_MASK (0x03 << 1) > +#define DA9062AA_BUCK4_CONF_SHIFT 3 > +#define DA9062AA_BUCK4_CONF_MASK (0x01 << 3) > +#define DA9062AA_VBUCK4_GPI_SHIFT 5 > +#define DA9062AA_VBUCK4_GPI_MASK (0x03 << 5) > + > +/* DA9062AA_BUCK3_CONT = 0x024 */ > +#define DA9062AA_BUCK3_EN_SHIFT 0 > +#define DA9062AA_BUCK3_EN_MASK (0x01 << 0) > +#define DA9062AA_BUCK3_GPI_SHIFT 1 > +#define DA9062AA_BUCK3_GPI_MASK (0x03 << 1) > +#define DA9062AA_BUCK3_CONF_SHIFT 3 > +#define DA9062AA_BUCK3_CONF_MASK (0x01 << 3) > +#define DA9062AA_VBUCK3_GPI_SHIFT 5 > +#define DA9062AA_VBUCK3_GPI_MASK (0x03 << 5) > + > +/* DA9062AA_LDO1_CONT = 0x026 */ > +#define DA9062AA_LDO1_EN_SHIFT 0 > +#define DA9062AA_LDO1_EN_MASK (0x01 << 0) > +#define DA9062AA_LDO1_GPI_SHIFT 1 > +#define DA9062AA_LDO1_GPI_MASK (0x03 << 1) > +#define DA9062AA_LDO1_PD_DIS_SHIFT 3 > +#define DA9062AA_LDO1_PD_DIS_MASK (0x01 << 3) > +#define DA9062AA_VLDO1_GPI_SHIFT 5 > +#define DA9062AA_VLDO1_GPI_MASK (0x03 << 5) > +#define DA9062AA_LDO1_CONF_SHIFT 7 > +#define DA9062AA_LDO1_CONF_MASK (0x01 << 7) > + > +/* DA9062AA_LDO2_CONT = 0x027 */ > +#define DA9062AA_LDO2_EN_SHIFT 0 > +#define DA9062AA_LDO2_EN_MASK (0x01 << 0) > +#define DA9062AA_LDO2_GPI_SHIFT 1 > +#define DA9062AA_LDO2_GPI_MASK (0x03 << 1) > +#define DA9062AA_LDO2_PD_DIS_SHIFT 3 > +#define DA9062AA_LDO2_PD_DIS_MASK (0x01 << 3) > +#define DA9062AA_VLDO2_GPI_SHIFT 5 > +#define DA9062AA_VLDO2_GPI_MASK (0x03 << 5) > +#define DA9062AA_LDO2_CONF_SHIFT 7 > +#define DA9062AA_LDO2_CONF_MASK (0x01 << 7) > + > +/* DA9062AA_LDO3_CONT = 0x028 */ > +#define DA9062AA_LDO3_EN_SHIFT 0 > +#define DA9062AA_LDO3_EN_MASK (0x01 << 0) > +#define DA9062AA_LDO3_GPI_SHIFT 1 > +#define DA9062AA_LDO3_GPI_MASK (0x03 << 1) > +#define DA9062AA_LDO3_PD_DIS_SHIFT 3 > +#define DA9062AA_LDO3_PD_DIS_MASK (0x01 << 3) > +#define DA9062AA_VLDO3_GPI_SHIFT 5 > +#define DA9062AA_VLDO3_GPI_MASK (0x03 << 5) > +#define DA9062AA_LDO3_CONF_SHIFT 7 > +#define DA9062AA_LDO3_CONF_MASK (0x01 << 7) > + > +/* DA9062AA_LDO4_CONT = 0x029 */ > +#define DA9062AA_LDO4_EN_SHIFT 0 > +#define DA9062AA_LDO4_EN_MASK (0x01 << 0) > +#define DA9062AA_LDO4_GPI_SHIFT 1 > +#define DA9062AA_LDO4_GPI_MASK (0x03 << 1) > +#define DA9062AA_LDO4_PD_DIS_SHIFT 3 > +#define DA9062AA_LDO4_PD_DIS_MASK (0x01 << 3) > +#define DA9062AA_VLDO4_GPI_SHIFT 5 > +#define DA9062AA_VLDO4_GPI_MASK (0x03 << 5) > +#define DA9062AA_LDO4_CONF_SHIFT 7 > +#define DA9062AA_LDO4_CONF_MASK (0x01 << 7) > + > +/* DA9062AA_DVC_1 = 0x032 */ > +#define DA9062AA_VBUCK1_SEL_SHIFT 0 > +#define DA9062AA_VBUCK1_SEL_MASK (0x01 << 0) > +#define DA9062AA_VBUCK2_SEL_SHIFT 1 > +#define DA9062AA_VBUCK2_SEL_MASK (0x01 << 1) > +#define DA9062AA_VBUCK4_SEL_SHIFT 2 > +#define DA9062AA_VBUCK4_SEL_MASK (0x01 << 2) > +#define DA9062AA_VBUCK3_SEL_SHIFT 3 > +#define DA9062AA_VBUCK3_SEL_MASK (0x01 << 3) > +#define DA9062AA_VLDO1_SEL_SHIFT 4 > +#define DA9062AA_VLDO1_SEL_MASK (0x01 << 4) > +#define DA9062AA_VLDO2_SEL_SHIFT 5 > +#define DA9062AA_VLDO2_SEL_MASK (0x01 << 5) > +#define DA9062AA_VLDO3_SEL_SHIFT 6 > +#define DA9062AA_VLDO3_SEL_MASK (0x01 << 6) > +#define DA9062AA_VLDO4_SEL_SHIFT 7 > +#define DA9062AA_VLDO4_SEL_MASK (0x01 << 7) > + > +/* DA9062AA_COUNT_S = 0x040 */ > +#define DA9062AA_COUNT_SEC_SHIFT 0 > +#define DA9062AA_COUNT_SEC_MASK (0x3f << 0) > +#define DA9062AA_RTC_READ_SHIFT 7 > +#define DA9062AA_RTC_READ_MASK (0x01 << 7) > + > +/* DA9062AA_COUNT_MI = 0x041 */ > +#define DA9062AA_COUNT_MIN_SHIFT 0 > +#define DA9062AA_COUNT_MIN_MASK (0x3f << 0) > + > +/* DA9062AA_COUNT_H = 0x042 */ > +#define DA9062AA_COUNT_HOUR_SHIFT 0 > +#define DA9062AA_COUNT_HOUR_MASK (0x1f << 0) > + > +/* DA9062AA_COUNT_D = 0x043 */ > +#define DA9062AA_COUNT_DAY_SHIFT 0 > +#define DA9062AA_COUNT_DAY_MASK (0x1f << 0) > + > +/* DA9062AA_COUNT_MO = 0x044 */ > +#define DA9062AA_COUNT_MONTH_SHIFT 0 > +#define DA9062AA_COUNT_MONTH_MASK (0x0f << 0) > + > +/* DA9062AA_COUNT_Y = 0x045 */ > +#define DA9062AA_COUNT_YEAR_SHIFT 0 > +#define DA9062AA_COUNT_YEAR_MASK (0x3f << 0) > +#define DA9062AA_MONITOR_SHIFT 6 > +#define DA9062AA_MONITOR_MASK (0x01 << 6) > + > +/* DA9062AA_ALARM_S = 0x046 */ > +#define DA9062AA_ALARM_SEC_SHIFT 0 > +#define DA9062AA_ALARM_SEC_MASK (0x3f << 0) > +#define DA9062AA_ALARM_STATUS_SHIFT 6 > +#define DA9062AA_ALARM_STATUS_MASK (0x03 << 6) > + > +/* DA9062AA_ALARM_MI = 0x047 */ > +#define DA9062AA_ALARM_MIN_SHIFT 0 > +#define DA9062AA_ALARM_MIN_MASK (0x3f << 0) > + > +/* DA9062AA_ALARM_H = 0x048 */ > +#define DA9062AA_ALARM_HOUR_SHIFT 0 > +#define DA9062AA_ALARM_HOUR_MASK (0x1f << 0) > + > +/* DA9062AA_ALARM_D = 0x049 */ > +#define DA9062AA_ALARM_DAY_SHIFT 0 > +#define DA9062AA_ALARM_DAY_MASK (0x1f << 0) > + > +/* DA9062AA_ALARM_MO = 0x04A */ > +#define DA9062AA_ALARM_MONTH_SHIFT 0 > +#define DA9062AA_ALARM_MONTH_MASK (0x0f << 0) > +#define DA9062AA_TICK_TYPE_SHIFT 4 > +#define DA9062AA_TICK_TYPE_MASK (0x01 << 4) > +#define DA9062AA_TICK_WAKE_SHIFT 5 > +#define DA9062AA_TICK_WAKE_MASK (0x01 << 5) > + > +/* DA9062AA_ALARM_Y = 0x04B */ > +#define DA9062AA_ALARM_YEAR_SHIFT 0 > +#define DA9062AA_ALARM_YEAR_MASK (0x3f << 0) > +#define DA9062AA_ALARM_ON_SHIFT 6 > +#define DA9062AA_ALARM_ON_MASK (0x01 << 6) > +#define DA9062AA_TICK_ON_SHIFT 7 > +#define DA9062AA_TICK_ON_MASK (0x01 << 7) > + > +/* DA9062AA_SECOND_A = 0x04C */ > +#define DA9062AA_SECONDS_A_SHIFT 0 > +#define DA9062AA_SECONDS_A_MASK (0xff << 0) > + > +/* DA9062AA_SECOND_B = 0x04D */ > +#define DA9062AA_SECONDS_B_SHIFT 0 > +#define DA9062AA_SECONDS_B_MASK (0xff << 0) > + > +/* DA9062AA_SECOND_C = 0x04E */ > +#define DA9062AA_SECONDS_C_SHIFT 0 > +#define DA9062AA_SECONDS_C_MASK (0xff << 0) > + > +/* DA9062AA_SECOND_D = 0x04F */ > +#define DA9062AA_SECONDS_D_SHIFT 0 > +#define DA9062AA_SECONDS_D_MASK (0xff << 0) > + > +/* DA9062AA_SEQ = 0x081 */ > +#define DA9062AA_SEQ_POINTER_SHIFT 0 > +#define DA9062AA_SEQ_POINTER_MASK (0x0f << 0) > +#define DA9062AA_NXT_SEQ_START_SHIFT 4 > +#define DA9062AA_NXT_SEQ_START_MASK (0x0f << 4) > + > +/* DA9062AA_SEQ_TIMER = 0x082 */ > +#define DA9062AA_SEQ_TIME_SHIFT 0 > +#define DA9062AA_SEQ_TIME_MASK (0x0f << 0) > +#define DA9062AA_SEQ_DUMMY_SHIFT 4 > +#define DA9062AA_SEQ_DUMMY_MASK (0x0f << 4) > + > +/* DA9062AA_ID_2_1 = 0x083 */ > +#define DA9062AA_LDO1_STEP_SHIFT 0 > +#define DA9062AA_LDO1_STEP_MASK (0x0f << 0) > +#define DA9062AA_LDO2_STEP_SHIFT 4 > +#define DA9062AA_LDO2_STEP_MASK (0x0f << 4) > + > +/* DA9062AA_ID_4_3 = 0x084 */ > +#define DA9062AA_LDO3_STEP_SHIFT 0 > +#define DA9062AA_LDO3_STEP_MASK (0x0f << 0) > +#define DA9062AA_LDO4_STEP_SHIFT 4 > +#define DA9062AA_LDO4_STEP_MASK (0x0f << 4) > + > +/* DA9062AA_ID_12_11 = 0x088 */ > +#define DA9062AA_PD_DIS_STEP_SHIFT 4 > +#define DA9062AA_PD_DIS_STEP_MASK (0x0f << 4) > + > +/* DA9062AA_ID_14_13 = 0x089 */ > +#define DA9062AA_BUCK1_STEP_SHIFT 0 > +#define DA9062AA_BUCK1_STEP_MASK (0x0f << 0) > +#define DA9062AA_BUCK2_STEP_SHIFT 4 > +#define DA9062AA_BUCK2_STEP_MASK (0x0f << 4) > + > +/* DA9062AA_ID_16_15 = 0x08A */ > +#define DA9062AA_BUCK4_STEP_SHIFT 0 > +#define DA9062AA_BUCK4_STEP_MASK (0x0f << 0) > +#define DA9062AA_BUCK3_STEP_SHIFT 4 > +#define DA9062AA_BUCK3_STEP_MASK (0x0f << 4) > + > +/* DA9062AA_ID_22_21 = 0x08D */ > +#define DA9062AA_GP_RISE1_STEP_SHIFT 0 > +#define DA9062AA_GP_RISE1_STEP_MASK (0x0f << 0) > +#define DA9062AA_GP_FALL1_STEP_SHIFT 4 > +#define DA9062AA_GP_FALL1_STEP_MASK (0x0f << 4) > + > +/* DA9062AA_ID_24_23 = 0x08E */ > +#define DA9062AA_GP_RISE2_STEP_SHIFT 0 > +#define DA9062AA_GP_RISE2_STEP_MASK (0x0f << 0) > +#define DA9062AA_GP_FALL2_STEP_SHIFT 4 > +#define DA9062AA_GP_FALL2_STEP_MASK (0x0f << 4) > + > +/* DA9062AA_ID_26_25 = 0x08F */ > +#define DA9062AA_GP_RISE3_STEP_SHIFT 0 > +#define DA9062AA_GP_RISE3_STEP_MASK (0x0f << 0) > +#define DA9062AA_GP_FALL3_STEP_SHIFT 4 > +#define DA9062AA_GP_FALL3_STEP_MASK (0x0f << 4) > + > +/* DA9062AA_ID_28_27 = 0x090 */ > +#define DA9062AA_GP_RISE4_STEP_SHIFT 0 > +#define DA9062AA_GP_RISE4_STEP_MASK (0x0f << 0) > +#define DA9062AA_GP_FALL4_STEP_SHIFT 4 > +#define DA9062AA_GP_FALL4_STEP_MASK (0x0f << 4) > + > +/* DA9062AA_ID_30_29 = 0x091 */ > +#define DA9062AA_GP_RISE5_STEP_SHIFT 0 > +#define DA9062AA_GP_RISE5_STEP_MASK (0x0f << 0) > +#define DA9062AA_GP_FALL5_STEP_SHIFT 4 > +#define DA9062AA_GP_FALL5_STEP_MASK (0x0f << 4) > + > +/* DA9062AA_ID_32_31 = 0x092 */ > +#define DA9062AA_WAIT_STEP_SHIFT 0 > +#define DA9062AA_WAIT_STEP_MASK (0x0f << 0) > +#define DA9062AA_EN32K_STEP_SHIFT 4 > +#define DA9062AA_EN32K_STEP_MASK (0x0f << 4) > + > +/* DA9062AA_SEQ_A = 0x095 */ > +#define DA9062AA_SYSTEM_END_SHIFT 0 > +#define DA9062AA_SYSTEM_END_MASK (0x0f << 0) > +#define DA9062AA_POWER_END_SHIFT 4 > +#define DA9062AA_POWER_END_MASK (0x0f << 4) > + > +/* DA9062AA_SEQ_B = 0x096 */ > +#define DA9062AA_MAX_COUNT_SHIFT 0 > +#define DA9062AA_MAX_COUNT_MASK (0x0f << 0) > +#define DA9062AA_PART_DOWN_SHIFT 4 > +#define DA9062AA_PART_DOWN_MASK (0x0f << 4) > + > +/* DA9062AA_WAIT = 0x097 */ > +#define DA9062AA_WAIT_TIME_SHIFT 0 > +#define DA9062AA_WAIT_TIME_MASK (0x0f << 0) > +#define DA9062AA_WAIT_MODE_SHIFT 4 > +#define DA9062AA_WAIT_MODE_MASK (0x01 << 4) > +#define DA9062AA_TIME_OUT_SHIFT 5 > +#define DA9062AA_TIME_OUT_MASK (0x01 << 5) > +#define DA9062AA_WAIT_DIR_SHIFT 6 > +#define DA9062AA_WAIT_DIR_MASK (0x03 << 6) > + > +/* DA9062AA_EN_32K = 0x098 */ > +#define DA9062AA_STABILISATION_TIME_SHIFT 0 > +#define DA9062AA_STABILISATION_TIME_MASK (0x07 << 0) > +#define DA9062AA_CRYSTAL_SHIFT 3 > +#define DA9062AA_CRYSTAL_MASK (0x01 << 3) > +#define DA9062AA_DELAY_MODE_SHIFT 4 > +#define DA9062AA_DELAY_MODE_MASK (0x01 << 4) > +#define DA9062AA_OUT_CLOCK_SHIFT 5 > +#define DA9062AA_OUT_CLOCK_MASK (0x01 << 5) > +#define DA9062AA_RTC_CLOCK_SHIFT 6 > +#define DA9062AA_RTC_CLOCK_MASK (0x01 << 6) > +#define DA9062AA_EN_32KOUT_SHIFT 7 > +#define DA9062AA_EN_32KOUT_MASK (0x01 << 7) > + > +/* DA9062AA_RESET = 0x099 */ > +#define DA9062AA_RESET_TIMER_SHIFT 0 > +#define DA9062AA_RESET_TIMER_MASK (0x3f << 0) > +#define DA9062AA_RESET_EVENT_SHIFT 6 > +#define DA9062AA_RESET_EVENT_MASK (0x03 << 6) > + > +/* DA9062AA_BUCK_ILIM_A = 0x09A */ > +#define DA9062AA_BUCK3_ILIM_SHIFT 0 > +#define DA9062AA_BUCK3_ILIM_MASK (0x0f << 0) > + > +/* DA9062AA_BUCK_ILIM_B = 0x09B */ > +#define DA9062AA_BUCK4_ILIM_SHIFT 0 > +#define DA9062AA_BUCK4_ILIM_MASK (0x0f << 0) > + > +/* DA9062AA_BUCK_ILIM_C = 0x09C */ > +#define DA9062AA_BUCK1_ILIM_SHIFT 0 > +#define DA9062AA_BUCK1_ILIM_MASK (0x0f << 0) > +#define DA9062AA_BUCK2_ILIM_SHIFT 4 > +#define DA9062AA_BUCK2_ILIM_MASK (0x0f << 4) > + > +/* DA9062AA_BUCK2_CFG = 0x09D */ > +#define DA9062AA_BUCK2_PD_DIS_SHIFT 5 > +#define DA9062AA_BUCK2_PD_DIS_MASK (0x01 << 5) > +#define DA9062AA_BUCK2_MODE_SHIFT 6 > +#define DA9062AA_BUCK2_MODE_MASK (0x03 << 6) > + > +/* DA9062AA_BUCK1_CFG = 0x09E */ > +#define DA9062AA_BUCK1_PD_DIS_SHIFT 5 > +#define DA9062AA_BUCK1_PD_DIS_MASK (0x01 << 5) > +#define DA9062AA_BUCK1_MODE_SHIFT 6 > +#define DA9062AA_BUCK1_MODE_MASK (0x03 << 6) > + > +/* DA9062AA_BUCK4_CFG = 0x09F */ > +#define DA9062AA_BUCK4_VTTR_EN_SHIFT 3 > +#define DA9062AA_BUCK4_VTTR_EN_MASK (0x01 << 3) > +#define DA9062AA_BUCK4_VTT_EN_SHIFT 4 > +#define DA9062AA_BUCK4_VTT_EN_MASK (0x01 << 4) > +#define DA9062AA_BUCK4_PD_DIS_SHIFT 5 > +#define DA9062AA_BUCK4_PD_DIS_MASK (0x01 << 5) > +#define DA9062AA_BUCK4_MODE_SHIFT 6 > +#define DA9062AA_BUCK4_MODE_MASK (0x03 << 6) > + > +/* DA9062AA_BUCK3_CFG = 0x0A0 */ > +#define DA9062AA_BUCK3_PD_DIS_SHIFT 5 > +#define DA9062AA_BUCK3_PD_DIS_MASK (0x01 << 5) > +#define DA9062AA_BUCK3_MODE_SHIFT 6 > +#define DA9062AA_BUCK3_MODE_MASK (0x03 << 6) > + > +/* DA9062AA_VBUCK2_A = 0x0A3 */ > +#define DA9062AA_VBUCK2_A_SHIFT 0 > +#define DA9062AA_VBUCK2_A_MASK (0x7f << 0) > +#define DA9062AA_BUCK2_SL_A_SHIFT 7 > +#define DA9062AA_BUCK2_SL_A_MASK (0x01 << 7) > + > +/* DA9062AA_VBUCK1_A = 0x0A4 */ > +#define DA9062AA_VBUCK1_A_SHIFT 0 > +#define DA9062AA_VBUCK1_A_MASK (0x7f << 0) > +#define DA9062AA_BUCK1_SL_A_SHIFT 7 > +#define DA9062AA_BUCK1_SL_A_MASK (0x01 << 7) > + > +/* DA9062AA_VBUCK4_A = 0x0A5 */ > +#define DA9062AA_VBUCK4_A_SHIFT 0 > +#define DA9062AA_VBUCK4_A_MASK (0x7f << 0) > +#define DA9062AA_BUCK4_SL_A_SHIFT 7 > +#define DA9062AA_BUCK4_SL_A_MASK (0x01 << 7) > + > +/* DA9062AA_VBUCK3_A = 0x0A7 */ > +#define DA9062AA_VBUCK3_A_SHIFT 0 > +#define DA9062AA_VBUCK3_A_MASK (0x7f << 0) > +#define DA9062AA_BUCK3_SL_A_SHIFT 7 > +#define DA9062AA_BUCK3_SL_A_MASK (0x01 << 7) > + > +/* DA9062AA_VLDO1_A = 0x0A9 */ > +#define DA9062AA_VLDO1_A_SHIFT 0 > +#define DA9062AA_VLDO1_A_MASK (0x3f << 0) > +#define DA9062AA_LDO1_SL_A_SHIFT 7 > +#define DA9062AA_LDO1_SL_A_MASK (0x01 << 7) > + > +/* DA9062AA_VLDO2_A = 0x0AA */ > +#define DA9062AA_VLDO2_A_SHIFT 0 > +#define DA9062AA_VLDO2_A_MASK (0x3f << 0) > +#define DA9062AA_LDO2_SL_A_SHIFT 7 > +#define DA9062AA_LDO2_SL_A_MASK (0x01 << 7) > + > +/* DA9062AA_VLDO3_A = 0x0AB */ > +#define DA9062AA_VLDO3_A_SHIFT 0 > +#define DA9062AA_VLDO3_A_MASK (0x3f << 0) > +#define DA9062AA_LDO3_SL_A_SHIFT 7 > +#define DA9062AA_LDO3_SL_A_MASK (0x01 << 7) > + > +/* DA9062AA_VLDO4_A = 0x0AC */ > +#define DA9062AA_VLDO4_A_SHIFT 0 > +#define DA9062AA_VLDO4_A_MASK (0x3f << 0) > +#define DA9062AA_LDO4_SL_A_SHIFT 7 > +#define DA9062AA_LDO4_SL_A_MASK (0x01 << 7) > + > +/* DA9062AA_VBUCK2_B = 0x0B4 */ > +#define DA9062AA_VBUCK2_B_SHIFT 0 > +#define DA9062AA_VBUCK2_B_MASK (0x7f << 0) > +#define DA9062AA_BUCK2_SL_B_SHIFT 7 > +#define DA9062AA_BUCK2_SL_B_MASK (0x01 << 7) > + > +/* DA9062AA_VBUCK1_B = 0x0B5 */ > +#define DA9062AA_VBUCK1_B_SHIFT 0 > +#define DA9062AA_VBUCK1_B_MASK (0x7f << 0) > +#define DA9062AA_BUCK1_SL_B_SHIFT 7 > +#define DA9062AA_BUCK1_SL_B_MASK (0x01 << 7) > + > +/* DA9062AA_VBUCK4_B = 0x0B6 */ > +#define DA9062AA_VBUCK4_B_SHIFT 0 > +#define DA9062AA_VBUCK4_B_MASK (0x7f << 0) > +#define DA9062AA_BUCK4_SL_B_SHIFT 7 > +#define DA9062AA_BUCK4_SL_B_MASK (0x01 << 7) > + > +/* DA9062AA_VBUCK3_B = 0x0B8 */ > +#define DA9062AA_VBUCK3_B_SHIFT 0 > +#define DA9062AA_VBUCK3_B_MASK (0x7f << 0) > +#define DA9062AA_BUCK3_SL_B_SHIFT 7 > +#define DA9062AA_BUCK3_SL_B_MASK (0x01 << 7) > + > +/* DA9062AA_VLDO1_B = 0x0BA */ > +#define DA9062AA_VLDO1_B_SHIFT 0 > +#define DA9062AA_VLDO1_B_MASK (0x3f << 0) > +#define DA9062AA_LDO1_SL_B_SHIFT 7 > +#define DA9062AA_LDO1_SL_B_MASK (0x01 << 7) > + > +/* DA9062AA_VLDO2_B = 0x0BB */ > +#define DA9062AA_VLDO2_B_SHIFT 0 > +#define DA9062AA_VLDO2_B_MASK (0x3f << 0) > +#define DA9062AA_LDO2_SL_B_SHIFT 7 > +#define DA9062AA_LDO2_SL_B_MASK (0x01 << 7) > + > +/* DA9062AA_VLDO3_B = 0x0BC */ > +#define DA9062AA_VLDO3_B_SHIFT 0 > +#define DA9062AA_VLDO3_B_MASK (0x3f << 0) > +#define DA9062AA_LDO3_SL_B_SHIFT 7 > +#define DA9062AA_LDO3_SL_B_MASK (0x01 << 7) > + > +/* DA9062AA_VLDO4_B = 0x0BD */ > +#define DA9062AA_VLDO4_B_SHIFT 0 > +#define DA9062AA_VLDO4_B_MASK (0x3f << 0) > +#define DA9062AA_LDO4_SL_B_SHIFT 7 > +#define DA9062AA_LDO4_SL_B_MASK (0x01 << 7) > + > +/* DA9062AA_BBAT_CONT = 0x0C5 */ > +#define DA9062AA_BCHG_VSET_SHIFT 0 > +#define DA9062AA_BCHG_VSET_MASK (0x0f << 0) > +#define DA9062AA_BCHG_ISET_SHIFT 4 > +#define DA9062AA_BCHG_ISET_MASK (0x0f << 4) > + > +/* DA9062AA_INTERFACE = 0x105 */ > +#define DA9062AA_IF_BASE_ADDR_SHIFT 4 > +#define DA9062AA_IF_BASE_ADDR_MASK (0x0f << 4) > + > +/* DA9062AA_CONFIG_A = 0x106 */ > +#define DA9062AA_PM_I_V_SHIFT 0 > +#define DA9062AA_PM_I_V_MASK (0x01 << 0) > +#define DA9062AA_PM_O_TYPE_SHIFT 2 > +#define DA9062AA_PM_O_TYPE_MASK (0x01 << 2) > +#define DA9062AA_IRQ_TYPE_SHIFT 3 > +#define DA9062AA_IRQ_TYPE_MASK (0x01 << 3) > +#define DA9062AA_PM_IF_V_SHIFT 4 > +#define DA9062AA_PM_IF_V_MASK (0x01 << 4) > +#define DA9062AA_PM_IF_FMP_SHIFT 5 > +#define DA9062AA_PM_IF_FMP_MASK (0x01 << 5) > +#define DA9062AA_PM_IF_HSM_SHIFT 6 > +#define DA9062AA_PM_IF_HSM_MASK (0x01 << 6) > + > +/* DA9062AA_CONFIG_B = 0x107 */ > +#define DA9062AA_VDD_FAULT_ADJ_SHIFT 0 > +#define DA9062AA_VDD_FAULT_ADJ_MASK (0x0f << 0) > +#define DA9062AA_VDD_HYST_ADJ_SHIFT 4 > +#define DA9062AA_VDD_HYST_ADJ_MASK (0x07 << 4) > + > +/* DA9062AA_CONFIG_C = 0x108 */ > +#define DA9062AA_BUCK_ACTV_DISCHRG_SHIFT 2 > +#define DA9062AA_BUCK_ACTV_DISCHRG_MASK (0x01 << 2) > +#define DA9062AA_BUCK1_CLK_INV_SHIFT 3 > +#define DA9062AA_BUCK1_CLK_INV_MASK (0x01 << 3) > +#define DA9062AA_BUCK4_CLK_INV_SHIFT 4 > +#define DA9062AA_BUCK4_CLK_INV_MASK (0x01 << 4) > +#define DA9062AA_BUCK3_CLK_INV_SHIFT 6 > +#define DA9062AA_BUCK3_CLK_INV_MASK (0x01 << 6) > + > +/* DA9062AA_CONFIG_D = 0x109 */ > +#define DA9062AA_GPI_V_SHIFT 0 > +#define DA9062AA_GPI_V_MASK (0x01 << 0) > +#define DA9062AA_NIRQ_MODE_SHIFT 1 > +#define DA9062AA_NIRQ_MODE_MASK (0x01 << 1) > +#define DA9062AA_SYSTEM_EN_RD_SHIFT 2 > +#define DA9062AA_SYSTEM_EN_RD_MASK (0x01 << 2) > +#define DA9062AA_FORCE_RESET_SHIFT 5 > +#define DA9062AA_FORCE_RESET_MASK (0x01 << 5) > + > +/* DA9062AA_CONFIG_E = 0x10A */ > +#define DA9062AA_BUCK1_AUTO_SHIFT 0 > +#define DA9062AA_BUCK1_AUTO_MASK (0x01 << 0) > +#define DA9062AA_BUCK2_AUTO_SHIFT 1 > +#define DA9062AA_BUCK2_AUTO_MASK (0x01 << 1) > +#define DA9062AA_BUCK4_AUTO_SHIFT 2 > +#define DA9062AA_BUCK4_AUTO_MASK (0x01 << 2) > +#define DA9062AA_BUCK3_AUTO_SHIFT 4 > +#define DA9062AA_BUCK3_AUTO_MASK (0x01 << 4) > + > +/* DA9062AA_CONFIG_G = 0x10C */ > +#define DA9062AA_LDO1_AUTO_SHIFT 0 > +#define DA9062AA_LDO1_AUTO_MASK (0x01 << 0) > +#define DA9062AA_LDO2_AUTO_SHIFT 1 > +#define DA9062AA_LDO2_AUTO_MASK (0x01 << 1) > +#define DA9062AA_LDO3_AUTO_SHIFT 2 > +#define DA9062AA_LDO3_AUTO_MASK (0x01 << 2) > +#define DA9062AA_LDO4_AUTO_SHIFT 3 > +#define DA9062AA_LDO4_AUTO_MASK (0x01 << 3) > + > +/* DA9062AA_CONFIG_H = 0x10D */ > +#define DA9062AA_BUCK1_2_MERGE_SHIFT 3 > +#define DA9062AA_BUCK1_2_MERGE_MASK (0x01 << 3) > +#define DA9062AA_BUCK2_OD_SHIFT 5 > +#define DA9062AA_BUCK2_OD_MASK (0x01 << 5) > +#define DA9062AA_BUCK1_OD_SHIFT 6 > +#define DA9062AA_BUCK1_OD_MASK (0x01 << 6) > + > +/* DA9062AA_CONFIG_I = 0x10E */ > +#define DA9062AA_NONKEY_PIN_SHIFT 0 > +#define DA9062AA_NONKEY_PIN_MASK (0x03 << 0) > +#define DA9062AA_nONKEY_SD_SHIFT 2 > +#define DA9062AA_nONKEY_SD_MASK (0x01 << 2) > +#define DA9062AA_WATCHDOG_SD_SHIFT 3 > +#define DA9062AA_WATCHDOG_SD_MASK (0x01 << 3) > +#define DA9062AA_KEY_SD_MODE_SHIFT 4 > +#define DA9062AA_KEY_SD_MODE_MASK (0x01 << 4) > +#define DA9062AA_HOST_SD_MODE_SHIFT 5 > +#define DA9062AA_HOST_SD_MODE_MASK (0x01 << 5) > +#define DA9062AA_INT_SD_MODE_SHIFT 6 > +#define DA9062AA_INT_SD_MODE_MASK (0x01 << 6) > +#define DA9062AA_LDO_SD_SHIFT 7 > +#define DA9062AA_LDO_SD_MASK (0x01 << 7) > + > +/* DA9062AA_CONFIG_J = 0x10F */ > +#define DA9062AA_KEY_DELAY_SHIFT 0 > +#define DA9062AA_KEY_DELAY_MASK (0x03 << 0) > +#define DA9062AA_SHUT_DELAY_SHIFT 2 > +#define DA9062AA_SHUT_DELAY_MASK (0x03 << 2) > +#define DA9062AA_RESET_DURATION_SHIFT 4 > +#define DA9062AA_RESET_DURATION_MASK (0x03 << 4) > +#define DA9062AA_TWOWIRE_TO_SHIFT 6 > +#define DA9062AA_TWOWIRE_TO_MASK (0x01 << 6) > +#define DA9062AA_IF_RESET_SHIFT 7 > +#define DA9062AA_IF_RESET_MASK (0x01 << 7) > + > +/* DA9062AA_CONFIG_K = 0x110 */ > +#define DA9062AA_GPIO0_PUPD_SHIFT 0 > +#define DA9062AA_GPIO0_PUPD_MASK (0x01 << 0) > +#define DA9062AA_GPIO1_PUPD_SHIFT 1 > +#define DA9062AA_GPIO1_PUPD_MASK (0x01 << 1) > +#define DA9062AA_GPIO2_PUPD_SHIFT 2 > +#define DA9062AA_GPIO2_PUPD_MASK (0x01 << 2) > +#define DA9062AA_GPIO3_PUPD_SHIFT 3 > +#define DA9062AA_GPIO3_PUPD_MASK (0x01 << 3) > +#define DA9062AA_GPIO4_PUPD_SHIFT 4 > +#define DA9062AA_GPIO4_PUPD_MASK (0x01 << 4) > + > +/* DA9062AA_CONFIG_M = 0x112 */ > +#define DA9062AA_NSHUTDOWN_PU_SHIFT 1 > +#define DA9062AA_NSHUTDOWN_PU_MASK (0x01 << 1) > +#define DA9062AA_WDG_MODE_SHIFT 3 > +#define DA9062AA_WDG_MODE_MASK (0x01 << 3) > +#define DA9062AA_OSC_FRQ_SHIFT 4 > +#define DA9062AA_OSC_FRQ_MASK (0x0f << 4) > + > +/* DA9062AA_TRIM_CLDR = 0x120 */ > +#define DA9062AA_TRIM_CLDR_SHIFT 0 > +#define DA9062AA_TRIM_CLDR_MASK (0xff << 0) > + > +/* DA9062AA_GP_ID_0 = 0x121 */ > +#define DA9062AA_GP_0_SHIFT 0 > +#define DA9062AA_GP_0_MASK (0xff << 0) > + > +/* DA9062AA_GP_ID_1 = 0x122 */ > +#define DA9062AA_GP_1_SHIFT 0 > +#define DA9062AA_GP_1_MASK (0xff << 0) > + > +/* DA9062AA_GP_ID_2 = 0x123 */ > +#define DA9062AA_GP_2_SHIFT 0 > +#define DA9062AA_GP_2_MASK (0xff << 0) > + > +/* DA9062AA_GP_ID_3 = 0x124 */ > +#define DA9062AA_GP_3_SHIFT 0 > +#define DA9062AA_GP_3_MASK (0xff << 0) > + > +/* DA9062AA_GP_ID_4 = 0x125 */ > +#define DA9062AA_GP_4_SHIFT 0 > +#define DA9062AA_GP_4_MASK (0xff << 0) > + > +/* DA9062AA_GP_ID_5 = 0x126 */ > +#define DA9062AA_GP_5_SHIFT 0 > +#define DA9062AA_GP_5_MASK (0xff << 0) > + > +/* DA9062AA_GP_ID_6 = 0x127 */ > +#define DA9062AA_GP_6_SHIFT 0 > +#define DA9062AA_GP_6_MASK (0xff << 0) > + > +/* DA9062AA_GP_ID_7 = 0x128 */ > +#define DA9062AA_GP_7_SHIFT 0 > +#define DA9062AA_GP_7_MASK (0xff << 0) > + > +/* DA9062AA_GP_ID_8 = 0x129 */ > +#define DA9062AA_GP_8_SHIFT 0 > +#define DA9062AA_GP_8_MASK (0xff << 0) > + > +/* DA9062AA_GP_ID_9 = 0x12A */ > +#define DA9062AA_GP_9_SHIFT 0 > +#define DA9062AA_GP_9_MASK (0xff << 0) > + > +/* DA9062AA_GP_ID_10 = 0x12B */ > +#define DA9062AA_GP_10_SHIFT 0 > +#define DA9062AA_GP_10_MASK (0xff << 0) > + > +/* DA9062AA_GP_ID_11 = 0x12C */ > +#define DA9062AA_GP_11_SHIFT 0 > +#define DA9062AA_GP_11_MASK (0xff << 0) > + > +/* DA9062AA_GP_ID_12 = 0x12D */ > +#define DA9062AA_GP_12_SHIFT 0 > +#define DA9062AA_GP_12_MASK (0xff << 0) > + > +/* DA9062AA_GP_ID_13 = 0x12E */ > +#define DA9062AA_GP_13_SHIFT 0 > +#define DA9062AA_GP_13_MASK (0xff << 0) > + > +/* DA9062AA_GP_ID_14 = 0x12F */ > +#define DA9062AA_GP_14_SHIFT 0 > +#define DA9062AA_GP_14_MASK (0xff << 0) > + > +/* DA9062AA_GP_ID_15 = 0x130 */ > +#define DA9062AA_GP_15_SHIFT 0 > +#define DA9062AA_GP_15_MASK (0xff << 0) > + > +/* DA9062AA_GP_ID_16 = 0x131 */ > +#define DA9062AA_GP_16_SHIFT 0 > +#define DA9062AA_GP_16_MASK (0xff << 0) > + > +/* DA9062AA_GP_ID_17 = 0x132 */ > +#define DA9062AA_GP_17_SHIFT 0 > +#define DA9062AA_GP_17_MASK (0xff << 0) > + > +/* DA9062AA_GP_ID_18 = 0x133 */ > +#define DA9062AA_GP_18_SHIFT 0 > +#define DA9062AA_GP_18_MASK (0xff << 0) > + > +/* DA9062AA_GP_ID_19 = 0x134 */ > +#define DA9062AA_GP_19_SHIFT 0 > +#define DA9062AA_GP_19_MASK (0xff << 0) > + > +/* DA9062AA_DEVICE_ID = 0x181 */ > +#define DA9062AA_DEV_ID_SHIFT 0 > +#define DA9062AA_DEV_ID_MASK (0xff << 0) > + > +/* DA9062AA_VARIANT_ID = 0x182 */ > +#define DA9062AA_VRC_SHIFT 0 > +#define DA9062AA_VRC_MASK (0x0f << 0) > +#define DA9062AA_MRC_SHIFT 4 > +#define DA9062AA_MRC_MASK (0x0f << 4) > + > +/* DA9062AA_CUSTOMER_ID = 0x183 */ > +#define DA9062AA_CUST_ID_SHIFT 0 > +#define DA9062AA_CUST_ID_MASK (0xff << 0) > + > +/* DA9062AA_CONFIG_ID = 0x184 */ > +#define DA9062AA_CONFIG_REV_SHIFT 0 > +#define DA9062AA_CONFIG_REV_MASK (0xff << 0) > + > +#endif /* __DA9062_H__ */ -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 18+ messages in thread
* RE: [PATCH V3 1/4] mfd: da9062: DA9062 MFD core driver 2015-05-26 16:10 ` Lee Jones @ 2015-05-28 12:52 ` Opensource [Steve Twiss] 2015-06-11 8:27 ` Opensource [Steve Twiss] 1 sibling, 0 replies; 18+ messages in thread From: Opensource [Steve Twiss] @ 2015-05-28 12:52 UTC (permalink / raw) To: Lee Jones Cc: LINUXKERNEL, Samuel Ortiz, Alessandro Zummo, DEVICETREE, David Dajun Chen, Dmitry Torokhov, Ian Campbell, Kumar Gala, LINUXINPUT, LINUXWATCHDOG, Liam Girdwood, Mark Brown, Mark Rutland, Pawel Moll, RTCLINUX, Rob Herring, Support Opensource, Wim Van Sebroeck [-- Warning: decoded text below may be mangled, UTF-8 assumed --] [-- Attachment #1: Type: text/plain; charset="utf-8", Size: 14236 bytes --] Hi Lee, I will refactor a lot of the driver and implement your changes as requested. I think the only major differences with your comments will be as follows: - the interrupt handler da9062_vdd_warn_event() will be erased - I would prefer the register header file to remain [mostly] untouched Please find more detailed comments below. thanks, Steve On 26 May 2015 17:10 Lee Jones wrote > To: Opensource [Steve Twiss] > Cc: LINUXKERNEL; Samuel Ortiz; Alessandro Zummo; DEVICETREE; David > Dajun Chen; Dmitry Torokhov; Ian Campbell; Kumar Gala; LINUXINPUT; > LINUXWATCHDOG; Liam Girdwood; Mark Brown; Mark Rutland; Pawel Moll; > RTCLINUX; Rob Herring; Support Opensource; Wim Van Sebroeck > Subject: Re: [PATCH V3 1/4] mfd: da9062: DA9062 MFD core driver > > On Tue, 19 May 2015, S Twiss wrote: > > > From: S Twiss <stwiss.opensource@diasemi.com> > > > > Add MFD core driver support for DA9062 > > > > Signed-off-by: Steve Twiss <stwiss.opensource@diasemi.com> > > [...] > > diff --git a/drivers/mfd/da9062-core.c b/drivers/mfd/da9062-core.c > > new file mode 100644 > > index 0000000..5aea082 > > --- /dev/null > > +++ b/drivers/mfd/da9062-core.c > > @@ -0,0 +1,611 @@ > > +/* > > + * da9062-core.c - CORE device driver for DA9062 > > Remove the filename. They have a habit of becoming incorrect. > > s/CORE/Core/ > > Can you also mention what the DA9062 actually is? > Will remove filename, and add a description "Core, IRQ and I2C driver for DA9062 PMIC" [...] > > +#include <linux/mutex.h> > > +#include <linux/regmap.h> > > + > > Why the '\n'? > > > +#include <linux/irq.h> > > +#include <linux/mfd/core.h> > > +#include <linux/i2c.h> > > +#include <linux/err.h> > > + > > Same here? > > > +#include <linux/mfd/da9062/core.h> > > +#include <linux/mfd/da9062/registers.h> > > +#include <linux/of.h> > > +#include <linux/regulator/of_regulator.h> > > + > > +#include <linux/proc_fs.h> > > +#include <linux/kthread.h> > > +#include <linux/uaccess.h> > > I'm a bit concerned by the number of includes here, are you sure > they're all required? > > > +/* IRQ device driver for DA9062 */ > > Not sure what this means? Removed unnecessary whitespace throughout the patches. And erased comments throughout that are not required [...] > > +int da9062_irq_init(struct da9062 *chip) > > +{ > > + int ret; > > + > > + if (!chip->chip_irq) { > > Check this once, in probe(), then rid this check. > > > + dev_err(chip->dev, "No IRQ configured\n"); > > + return -EINVAL; > > + } > > + > > + ret = regmap_add_irq_chip(chip->regmap, chip->chip_irq, > > + IRQF_TRIGGER_LOW | IRQF_ONESHOT | > IRQF_SHARED, > > + chip->irq_base, &da9062_irq_chip, > > + &chip->regmap_irq); > > This is just one call. Just place that call into > da9062_device_init() and rid this function. > Will refactor this part. > > + 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. > I will erase all abstractions and ... - Refactor da9062_device_init() and da9062_irq_init() into probe - Add new function get_device_type() for variant and device_id information [...] > > +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. Done. > > + > > +static irqreturn_t da9062_vdd_warn_event(int irq, void *data) > > +{ > > + struct da9062 *hw = data; > > + > > + dev_err(hw->dev, "VDD warning indicator\n"); > > Is it an error? Doesn't look like it. > > > + return IRQ_HANDLED; > > +} See later on (this is to be erased) [...] > > +static int da9062_clear_fault_log(struct da9062 *chip) > > +{ > > + int ret = 0; > > No need to pre-initialise. > > > + int fault_log = 0; > > As above. > > > + ret = regmap_read(chip->regmap, DA9062AA_FAULT_LOG, > &fault_log); > > + if (ret < 0) { > > + dev_err(chip->dev, "Cannot read FAULT_LOG.\n"); > > + ret = -EIO; > > Just return and rid the else. > > > + } else { > > + if (fault_log) { > > + if (fault_log & DA9062AA_TWD_ERROR_MASK) I will refactor da9062_clear_fault_log() to remove inconsistencies > > +int da9062_device_init(struct da9062 *chip, unsigned int irq) > > No need to pass irq, as it's part of chip. > This is going to be erased as part of a later comment (see below) [...] > > + 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(). Will use dev_warn instead > > > + chip->irq_base = -1; > > Why are you pre-initialising? > > > + chip->chip_irq = irq; > > You already assigned irq to chip_irq. > > > + ret = regmap_read(chip->regmap, DA9062AA_DEVICE_ID, > &device_id); > > + if (ret < 0) { > > + dev_err(chip->dev, "Cannot read chip ID.\n"); > > + return -EIO; > > + } > > + if (device_id != DA9062_PMIC_DEVICE_ID) { > > + dev_err(chip->dev, "Invalid device ID: 0x%02x\n", > device_id); > > + return -ENODEV; > > + } > > + > > + ret = regmap_read(chip->regmap, DA9062AA_VARIANT_ID, > &variant_id); > > + if (ret < 0) { > > + dev_err(chip->dev, "Cannot read chip variant id.\n"); > > + return -EIO; > > + } > > + > > + dev_info(chip->dev, > > + "Device detected (device-ID: 0x%02X, var-ID: 0x%02X)\n", > > + device_id, variant_id); > > Probably best to put this at the end. I have left this part in at this point. The device_id and variant_id are not passed any further than this function for the moment. > > > + 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; > > + } > > I'd put all of the device/variant stuff it its own function, then move > everything else into probe() and remove da9062_device_init(). > I will make a new function get_device_type() for variant and device_id information > > + 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. > Yes. I will do that. > > + chip->irq_base = regmap_irq_chip_get_base(chip->regmap_irq); > > + > > + ret = mfd_add_devices(chip->dev, -1, da9062_devs, > > Use PLATFORM_DEVID_NONE instead. > > > + ARRAY_SIZE(da9062_devs), NULL, chip->irq_base, > > + NULL); > > Usually child devices are probed at the end i.e. the last thing in > probe(). > Ok. I will move that to the end and use the PLATFORM_DEVID_NONE instead of the explicit -1 value. > > + if (ret) { > > + dev_err(chip->dev, "Cannot add MFD cells\n"); > > "Cannot register child devices". > Will replace that. [...] > > + /* VDD WARN event support */ > > + irq_vdd_warn = regmap_irq_get_virq(chip->regmap_irq, > > + DA9062_IRQ_VDD_WARN); > > + if (irq_vdd_warn < 0) { > > + dev_err(chip->dev, "Failed to get IRQ.\n"); > > + return irq_vdd_warn; > > + } > > + chip->irq_vdd_warn = irq_vdd_warn; > > + > > + ret = devm_request_threaded_irq(chip->dev, irq_vdd_warn, > > + NULL, da9062_vdd_warn_event, > > + IRQF_TRIGGER_LOW | IRQF_ONESHOT, > > + "VDD_WARN", chip); > > + if (ret) { > > + dev_warn(chip->dev, > > + "Failed to request VDD_WARN IRQ.\n"); > > + chip->irq_vdd_warn = -ENXIO; > > + } > > This looks like a lot of code, which doesn't really do anything. What > is a VDD warning indicator anyway? > 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_WARN was 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. 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. > > + 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()? > Will do that as part of the abstraction clean up [...] > > + > > +static struct regmap_config da9062_regmap_config = { > > + .reg_bits = 8, > > + .val_bits = 8, > > + .ranges = da9062_range_cfg, > > + .num_ranges = ARRAY_SIZE(da9062_range_cfg), > > + .max_register = DA9062AA_CONFIG_ID, > > + .cache_type = REGCACHE_RBTREE, > > +}; > > + > > +static const struct of_device_id da9062_dt_ids[] = { > > + { .compatible = "dlg,da9062", }, > > + { } > > +}; > > +MODULE_DEVICE_TABLE(of, da9062_dt_ids); > > Move this just above where it's to be used i.e. down to the bottom. > Will move that to the end of the file. > > +static int da9062_i2c_probe(struct i2c_client *i2c, > > + const struct i2c_device_id *id) > > +{ > > + struct da9062 *chip; > > + int ret; > > + > > + chip = devm_kzalloc(&i2c->dev, sizeof(struct da9062), GFP_KERNEL); > > sizeof(*chip) > > > + if (chip == NULL) > > if (!chip) Will do that. > > + 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? > Ok.. I will fix that part into the main structure. > > +static const struct i2c_device_id da9062_i2c_id[] = { > > + {"da9062", 0}, > > White space discipline. > I have put some white-spaces in the code. I think this: { "da9062", 0 }, { }, [...] > > > +MODULE_DESCRIPTION("CORE device driver for Dialog DA9062"); > > s/CORE/Core/ > Yes. > > +MODULE_AUTHOR("S Twiss <stwiss.opensource@diasemi.com>"); > > Full names please. > Ok. [...] > > diff --git a/include/linux/mfd/da9062/core.h > b/include/linux/mfd/da9062/core.h > > new file mode 100644 > > index 0000000..0b17891 > > --- /dev/null > > +++ b/include/linux/mfd/da9062/core.h > > @@ -0,0 +1,62 @@ > > +/* > > + * core.h - CORE H for DA9062 > > What does this add? > Removed the top line completely [...] > > + > > +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? > After I did the refactoring work described above, I can remove quite a few of them. unsigned char device_id; unsigned char variant_mrc; These are not currently used in any child device drivers -- because there is only one device of this type. int chip_irq; unsigned int irq_base; The chip_id is now local to the probe and can replaced with i2c->irq in that function It's only use is during clean-up and i2c->irq can be used there also. irq_base is local to probe() also, so I have made it automatic. > > + int irq_vdd_warn; I have removed the vdd_warn capability in this patch. > > +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? > Gone. [...] > > diff --git a/include/linux/mfd/da9062/registers.h > b/include/linux/mfd/da9062/registers.h > > new file mode 100644 > > index 0000000..d07c2bc > > --- /dev/null > > +++ b/include/linux/mfd/da9062/registers.h [...] > > +/* > > + * Registers > > + */ > > Really? ;) > > > +#define DA9062AA_PAGE_CON 0x000 > > +#define DA9062AA_STATUS_A 0x001 > > +#define DA9062AA_STATUS_B 0x002 [...] > > + > > +/* > > + * 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). > 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. 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 I have left anything out please let me know. regards, Steve N‹§²æìr¸›yúèšØb²X¬¶Ç§vØ^–)Þº{.nÇ+‰·¥Š{±Á«\…Ú Š{ayº\x1dʇڙë,j\a¢f£¢·hš‹»öì\x17/oSc¾™Ú³9˜uÀ¦æå‰È&jw¨®\x03(階ŠÝ¢j"ú\x1a¶^[m§ÿïêäz¹Þ–Šàþf£¢·hšˆ§~ˆmš ^ permalink raw reply [flat|nested] 18+ messages in thread
* RE: [PATCH V3 1/4] mfd: da9062: DA9062 MFD core driver 2015-05-26 16:10 ` Lee Jones 2015-05-28 12:52 ` Opensource [Steve Twiss] @ 2015-06-11 8:27 ` Opensource [Steve Twiss] 2015-06-11 8:50 ` Lee Jones 2015-06-11 8:56 ` Lee Jones 1 sibling, 2 replies; 18+ messages in thread From: Opensource [Steve Twiss] @ 2015-06-11 8:27 UTC (permalink / raw) To: 'Lee Jones' Cc: LINUXKERNEL, Samuel Ortiz, Alessandro Zummo, DEVICETREE, David Dajun Chen, Dmitry Torokhov, Ian Campbell, Kumar Gala, LINUXINPUT, LINUXWATCHDOG, Liam Girdwood, Mark Brown, Mark Rutland, Pawel Moll, RTCLINUX, Rob Herring, Support Opensource, Wim Van Sebroeck On 28 May 2015 13:53, Steve Twiss wrote: > To: 'Lee Jones' > Subject: RE: [PATCH V3 1/4] mfd: da9062: DA9062 MFD core driver > > Hi 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: > > > + /* VDD WARN event support */ > > > + irq_vdd_warn = regmap_irq_get_virq(chip->regmap_irq, > > > + DA9062_IRQ_VDD_WARN); > > > + if (irq_vdd_warn < 0) { > > > + dev_err(chip->dev, "Failed to get IRQ.\n"); > > > + return irq_vdd_warn; > > > + } > > > + chip->irq_vdd_warn = irq_vdd_warn; > > > + > > > + ret = devm_request_threaded_irq(chip->dev, irq_vdd_warn, > > > + NULL, da9062_vdd_warn_event, > > > + IRQF_TRIGGER_LOW | IRQF_ONESHOT, > > > + "VDD_WARN", chip); > > > + if (ret) { > > > + dev_warn(chip->dev, > > > + "Failed to request VDD_WARN IRQ.\n"); > > > + chip->irq_vdd_warn = -ENXIO; > > > + } > > > > This looks like a lot of code, which doesn't really do anything. What > > is a VDD warning indicator anyway? > > > > 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_WARN was 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. > > 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 > > > diff --git a/include/linux/mfd/da9062/registers.h > > b/include/linux/mfd/da9062/registers.h > > > new file mode 100644 > > > index 0000000..d07c2bc > > > --- /dev/null > > > +++ b/include/linux/mfd/da9062/registers.h > > [...] > > > > +/* > > > + * Registers > > > + */ > > > > Really? ;) > > > > > +#define DA9062AA_PAGE_CON 0x000 > > > +#define DA9062AA_STATUS_A 0x001 > > > +#define DA9062AA_STATUS_B 0x002 > > [...] > > > > + > > > +/* > > > + * 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). > > > > 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. > > 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 ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH V3 1/4] mfd: da9062: DA9062 MFD core driver 2015-06-11 8:27 ` Opensource [Steve Twiss] @ 2015-06-11 8:50 ` Lee Jones 2015-06-11 8:56 ` Lee Jones 1 sibling, 0 replies; 18+ messages in thread From: Lee Jones @ 2015-06-11 8:50 UTC (permalink / raw) To: Opensource [Steve Twiss] Cc: LINUXKERNEL, Samuel Ortiz, Alessandro Zummo, DEVICETREE, David Dajun Chen, Dmitry Torokhov, Ian Campbell, Kumar Gala, LINUXINPUT, LINUXWATCHDOG, Liam Girdwood, Mark Brown, Mark Rutland, Pawel Moll, RTCLINUX, Rob Herring, Support Opensource, Wim Van Sebroeck On Thu, 11 Jun 2015, Opensource [Steve Twiss] wrote: > On 28 May 2015 13:53, Steve Twiss wrote: > > > To: 'Lee Jones' > > Subject: RE: [PATCH V3 1/4] mfd: da9062: DA9062 MFD core driver > > > > Hi 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. 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. -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH V3 1/4] mfd: da9062: DA9062 MFD core driver 2015-06-11 8:27 ` Opensource [Steve Twiss] 2015-06-11 8:50 ` Lee Jones @ 2015-06-11 8:56 ` Lee Jones 2015-06-11 9:27 ` Opensource [Steve Twiss] 2015-06-29 8:01 ` Opensource [Steve Twiss] 1 sibling, 2 replies; 18+ messages in thread From: Lee Jones @ 2015-06-11 8:56 UTC (permalink / raw) To: Opensource [Steve Twiss] Cc: LINUXKERNEL, Samuel Ortiz, Alessandro Zummo, DEVICETREE, David Dajun Chen, Dmitry Torokhov, Ian Campbell, Kumar Gala, LINUXINPUT, LINUXWATCHDOG, Liam Girdwood, Mark Brown, Mark Rutland, Pawel Moll, RTCLINUX, Rob Herring, Support Opensource, Wim Van Sebroeck > > > > diff --git a/include/linux/mfd/da9062/registers.h > > > b/include/linux/mfd/da9062/registers.h > > > > new file mode 100644 > > > > index 0000000..d07c2bc > > > > --- /dev/null > > > > +++ b/include/linux/mfd/da9062/registers.h > > > > [...] > > > > > > +/* > > > > + * Registers > > > > + */ > > > > > > Really? ;) > > > > > > > +#define DA9062AA_PAGE_CON 0x000 > > > > +#define DA9062AA_STATUS_A 0x001 > > > > +#define DA9062AA_STATUS_B 0x002 > > > > [...] > > > > > > + > > > > +/* > > > > + * 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). > > > > > > > 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. > > > > 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. ;) -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 18+ messages in thread
* RE: [PATCH V3 1/4] mfd: da9062: DA9062 MFD core driver 2015-06-11 8:56 ` Lee Jones @ 2015-06-11 9:27 ` Opensource [Steve Twiss] 2015-06-11 21:46 ` [rtc-linux] " Alexandre Belloni 2015-06-29 8:01 ` Opensource [Steve Twiss] 1 sibling, 1 reply; 18+ messages in thread From: Opensource [Steve Twiss] @ 2015-06-11 9:27 UTC (permalink / raw) To: Lee Jones Cc: LINUXKERNEL, Samuel Ortiz, Alessandro Zummo, DEVICETREE, David Dajun Chen, Dmitry Torokhov, Ian Campbell, Kumar Gala, LINUXINPUT, LINUXWATCHDOG, Liam Girdwood, Mark Brown, Mark Rutland, Pawel Moll, RTCLINUX, Rob Herring, Support Opensource, Wim Van Sebroeck On 11 June 2015 09:57 Lee Jones wrote: > To: Opensource [Steve Twiss] > Subject: Re: [PATCH V3 1/4] mfd: da9062: DA9062 MFD core driver > > > > > > diff --git a/include/linux/mfd/da9062/registers.h [...] > > > > > +#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. > > > > > > 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. HI Lee, Thanks for replying so quickly. > 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. > 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. 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. > 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, Steve ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [rtc-linux] RE: [PATCH V3 1/4] mfd: da9062: DA9062 MFD core driver 2015-06-11 9:27 ` Opensource [Steve Twiss] @ 2015-06-11 21:46 ` Alexandre Belloni 0 siblings, 0 replies; 18+ messages in thread From: Alexandre Belloni @ 2015-06-11 21:46 UTC (permalink / raw) To: Opensource [Steve Twiss] Cc: Lee Jones, LINUXKERNEL, Samuel Ortiz, Alessandro Zummo, DEVICETREE, David Dajun Chen, Dmitry Torokhov, Ian Campbell, Kumar Gala, LINUXINPUT, LINUXWATCHDOG, Liam Girdwood, Mark Brown, Mark Rutland, Pawel Moll, RTCLINUX, Rob Herring, Support Opensource, Wim Van Sebroeck Hi, On 11/06/2015 at 09:27:45 +0000, Opensource [Steve Twiss] wrote : > > Thanks for replying so quickly. > > > 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. > > > 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. > > 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. > > 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, > Steve > > -- > -- > 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 rtc-linux+unsubscribe@googlegroups.com. > For more options, visit https://groups.google.com/d/optout. -- Alexandre Belloni, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com ^ permalink raw reply [flat|nested] 18+ messages in thread
* RE: [PATCH V3 1/4] mfd: da9062: DA9062 MFD core driver 2015-06-11 8:56 ` Lee Jones 2015-06-11 9:27 ` Opensource [Steve Twiss] @ 2015-06-29 8:01 ` Opensource [Steve Twiss] 2015-07-01 8:02 ` Lee Jones 1 sibling, 1 reply; 18+ messages in thread From: Opensource [Steve Twiss] @ 2015-06-29 8:01 UTC (permalink / raw) To: Lee Jones Cc: LINUXKERNEL, Samuel Ortiz, Alessandro Zummo, DEVICETREE, David Dajun Chen, Dmitry Torokhov, Ian Campbell, Kumar Gala, LINUXINPUT, LINUXWATCHDOG, Liam Girdwood, Mark Brown, Mark Rutland, Pawel Moll, RTCLINUX, Rob Herring, Support Opensource, Wim Van Sebroeck On 11 June 2015 09:57, Lee Jones wrote: > To: Opensource [Steve Twiss] > Subject: Re: [PATCH V3 1/4] mfd: da9062: DA9062 MFD core driver > > > > > > Steve Twiss wrote: > > > > > +#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 [...] [...] > 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 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. ;) 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 ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH V3 1/4] mfd: da9062: DA9062 MFD core driver 2015-06-29 8:01 ` Opensource [Steve Twiss] @ 2015-07-01 8:02 ` Lee Jones 0 siblings, 0 replies; 18+ messages in thread From: Lee Jones @ 2015-07-01 8:02 UTC (permalink / raw) To: Opensource [Steve Twiss] Cc: LINUXKERNEL, Samuel Ortiz, Alessandro Zummo, DEVICETREE, David Dajun Chen, Dmitry Torokhov, Ian Campbell, Kumar Gala, LINUXINPUT, LINUXWATCHDOG, Liam Girdwood, Mark Brown, Mark Rutland, Pawel Moll, RTCLINUX, Rob Herring, Support Opensource, Wim Van Sebroeck On Mon, 29 Jun 2015, Opensource [Steve Twiss] wrote: > On 11 June 2015 09:57, Lee Jones wrote: > > > To: Opensource [Steve Twiss] > > Subject: Re: [PATCH V3 1/4] mfd: da9062: DA9062 MFD core driver > > > > > > > > Steve Twiss wrote: > > > > > > +#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 [...] > > [...] > > > 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 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. ;) > > 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. -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH V3 4/4] devicetree: da9062: Add bindings for DA9062 driver [not found] ` <cover.1432041031.git.stwiss.opensource-WBD+wuPFNBhBDgjK7y7TUQ@public.gmane.org> 2015-05-19 13:10 ` [PATCH V3 1/4] mfd: da9062: DA9062 MFD core driver S Twiss @ 2015-05-19 13:10 ` S Twiss 1 sibling, 0 replies; 18+ messages in thread From: S Twiss @ 2015-05-19 13:10 UTC (permalink / raw) To: DEVICETREE, Ian Campbell, Kumar Gala, LINUXKERNEL, Mark Rutland, Pawel Moll, Rob Herring Cc: Alessandro Zummo, David Dajun Chen, Dmitry Torokhov, LINUXINPUT, LINUXWATCHDOG, Lee Jones, Liam Girdwood, Mark Brown, RTCLINUX, S Twiss, Samuel Ortiz, Support Opensource, Wim Van Sebroeck From: S Twiss <stwiss.opensource-WBD+wuPFNBhBDgjK7y7TUQ@public.gmane.org> Add device tree bindings for the DA9062 driver Signed-off-by: Steve Twiss <stwiss.opensource-WBD+wuPFNBhBDgjK7y7TUQ@public.gmane.org> --- 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 diff --git a/Documentation/devicetree/bindings/mfd/da9062.txt b/Documentation/devicetree/bindings/mfd/da9062.txt new file mode 100644 index 0000000..5765ed9 --- /dev/null +++ b/Documentation/devicetree/bindings/mfd/da9062.txt @@ -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; + }; + }; + }; + -- end-of-patch for PATCH V3 -- To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH V3 2/4] regulator: da9062: DA9062 regulator driver 2015-05-19 13:10 [PATCH V3 0/4] da9062: DA9062 driver submission S Twiss 2015-05-19 13:10 ` [PATCH V3 3/4] watchdog: da9062: DA9062 watchdog driver S Twiss [not found] ` <cover.1432041031.git.stwiss.opensource-WBD+wuPFNBhBDgjK7y7TUQ@public.gmane.org> @ 2015-05-19 13:10 ` S Twiss 2015-05-21 12:04 ` Mark Brown 2 siblings, 1 reply; 18+ messages in thread From: S Twiss @ 2015-05-19 13:10 UTC (permalink / raw) To: LINUXKERNEL, Liam Girdwood, Mark Brown Cc: Alessandro Zummo, DEVICETREE, David Dajun Chen, Dmitry Torokhov, Ian Campbell, Kumar Gala, LINUXINPUT, LINUXWATCHDOG, Lee Jones, Mark Rutland, Pawel Moll, RTCLINUX, Rob Herring, S Twiss, Samuel Ortiz, Support Opensource, Wim Van Sebroeck From: S Twiss <stwiss.opensource@diasemi.com> Add BUCK and LDO regulator driver support for DA9062 Signed-off-by: Steve Twiss <stwiss.opensource@diasemi.com> --- 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 diff --git a/drivers/regulator/Kconfig b/drivers/regulator/Kconfig index a6f116a..1fd6dc3 100644 --- a/drivers/regulator/Kconfig +++ b/drivers/regulator/Kconfig @@ -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 diff --git a/drivers/regulator/Makefile b/drivers/regulator/Makefile index 2c4da15..12ecb65 100644 --- a/drivers/regulator/Makefile +++ b/drivers/regulator/Makefile @@ -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 diff --git a/drivers/regulator/da9062-regulator.c b/drivers/regulator/da9062-regulator.c new file mode 100644 index 0000000..9805b6b --- /dev/null +++ b/drivers/regulator/da9062-regulator.c @@ -0,0 +1,843 @@ +/* + * da9062-regulator.c - REGULATOR device driver 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. + */ +#include <linux/kernel.h> +#include <linux/module.h> +#include <linux/init.h> +#include <linux/err.h> +#include <linux/slab.h> +#include <linux/of.h> +#include <linux/platform_device.h> +#include <linux/regmap.h> +#include <linux/regulator/driver.h> +#include <linux/regulator/machine.h> +#include <linux/regulator/of_regulator.h> +#include <linux/mfd/da9062/core.h> +#include <linux/mfd/da9062/registers.h> + +/* Regulator IDs */ +enum { + DA9062_ID_BUCK1, + DA9062_ID_BUCK2, + DA9062_ID_BUCK3, + DA9062_ID_BUCK4, + DA9062_ID_LDO1, + DA9062_ID_LDO2, + DA9062_ID_LDO3, + DA9062_ID_LDO4, + DA9062_MAX_REGULATORS, +}; + +/* Regulator capabilities and registers description */ +struct da9062_regulator_info { + struct regulator_desc desc; + /* Current limiting */ + unsigned int n_current_limits; + const int *current_limits; + /* Main register fields */ + struct reg_field mode; + struct reg_field suspend; + struct reg_field sleep; + struct reg_field suspend_sleep; + unsigned int suspend_vsel_reg; + struct reg_field ilimit; + /* Event detection bit */ + struct reg_field oc_event; +}; + +/* Single regulator settings */ +struct da9062_regulator { + struct regulator_desc desc; + struct regulator_dev *rdev; + struct da9062 *hw; + const struct da9062_regulator_info *info; + + struct regmap_field *mode; + struct regmap_field *suspend; + struct regmap_field *sleep; + struct regmap_field *suspend_sleep; + struct regmap_field *ilimit; +}; + +/* Encapsulates all information for the regulators driver */ +struct da9062_regulators { + int irq_ldo_lim; + unsigned n_regulators; + /* Array size to be defined during init. Keep at end. */ + struct da9062_regulator regulator[0]; +}; + +/* BUCK modes */ +enum { + BUCK_MODE_MANUAL, /* 0 */ + BUCK_MODE_SLEEP, /* 1 */ + BUCK_MODE_SYNC, /* 2 */ + BUCK_MODE_AUTO /* 3 */ +}; + +/* Regulator operations */ + +/* Current limits array (in uA) BUCK1 and BUCK3. + Entry indexes corresponds to register values. */ +static const int da9062_buck_a_limits[] = { + 500000, 600000, 700000, 800000, 900000, 1000000, 1100000, 1200000, + 1300000, 1400000, 1500000, 1600000, 1700000, 1800000, 1900000, 2000000 +}; + +/* Current limits array (in uA) for BUCK2. + Entry indexes corresponds to register values. */ +static const int da9062_buck_b_limits[] = { + 1500000, 1600000, 1700000, 1800000, 1900000, 2000000, 2100000, 2200000, + 2300000, 2400000, 2500000, 2600000, 2700000, 2800000, 2900000, 3000000 +}; + +static int da9062_set_current_limit(struct regulator_dev *rdev, + int min_ua, int max_ua) +{ + struct da9062_regulator *regl = rdev_get_drvdata(rdev); + const struct da9062_regulator_info *rinfo = regl->info; + int n, tval; + + for (n = 0; n < rinfo->n_current_limits; n++) { + tval = rinfo->current_limits[n]; + if (tval >= min_ua && tval <= max_ua) + return regmap_field_write(regl->ilimit, n); + } + + return -EINVAL; +} + +static int da9062_get_current_limit(struct regulator_dev *rdev) +{ + struct da9062_regulator *regl = rdev_get_drvdata(rdev); + const struct da9062_regulator_info *rinfo = regl->info; + unsigned int sel; + int ret; + + ret = regmap_field_read(regl->ilimit, &sel); + if (ret < 0) + return ret; + + if (sel >= rinfo->n_current_limits) + sel = rinfo->n_current_limits - 1; + + return rinfo->current_limits[sel]; +} + +static int da9062_buck_set_mode(struct regulator_dev *rdev, unsigned mode) +{ + struct da9062_regulator *regl = rdev_get_drvdata(rdev); + unsigned val; + + switch (mode) { + case REGULATOR_MODE_FAST: + val = BUCK_MODE_SYNC; + break; + case REGULATOR_MODE_NORMAL: + val = BUCK_MODE_AUTO; + break; + case REGULATOR_MODE_STANDBY: + val = BUCK_MODE_SLEEP; + break; + default: + return -EINVAL; + } + + return regmap_field_write(regl->mode, val); +} + +/* + * Bucks use single mode register field for normal operation + * and suspend state. + * There are 3 modes to map to: FAST, NORMAL, and STANDBY. + */ + +static unsigned da9062_buck_get_mode(struct regulator_dev *rdev) +{ + struct da9062_regulator *regl = rdev_get_drvdata(rdev); + struct regmap_field *field; + unsigned int val, mode = 0; + int ret; + + ret = regmap_field_read(regl->mode, &val); + if (ret < 0) + return ret; + + switch (val) { + default: + case BUCK_MODE_MANUAL: + mode = REGULATOR_MODE_FAST | REGULATOR_MODE_STANDBY; + /* Sleep flag bit decides the mode */ + break; + case BUCK_MODE_SLEEP: + return REGULATOR_MODE_STANDBY; + case BUCK_MODE_SYNC: + return REGULATOR_MODE_FAST; + case BUCK_MODE_AUTO: + return REGULATOR_MODE_NORMAL; + } + + /* Detect current regulator state */ + ret = regmap_field_read(regl->suspend, &val); + if (ret < 0) + return 0; + + /* Read regulator mode from proper register, depending on state */ + if (val) + field = regl->suspend_sleep; + else + field = regl->sleep; + + ret = regmap_field_read(field, &val); + if (ret < 0) + return 0; + + if (val) + mode &= REGULATOR_MODE_STANDBY; + else + mode &= REGULATOR_MODE_NORMAL | REGULATOR_MODE_FAST; + + return mode; +} + +/* + * LDOs use sleep flags - one for normal and one for suspend state. + * There are 2 modes to map to: NORMAL and STANDBY (sleep) for each state. + */ + +static int da9062_ldo_set_mode(struct regulator_dev *rdev, unsigned mode) +{ + struct da9062_regulator *regl = rdev_get_drvdata(rdev); + unsigned val; + + switch (mode) { + case REGULATOR_MODE_NORMAL: + val = 0; + break; + case REGULATOR_MODE_STANDBY: + val = 1; + break; + default: + return -EINVAL; + } + + return regmap_field_write(regl->sleep, val); +} + +static unsigned da9062_ldo_get_mode(struct regulator_dev *rdev) +{ + struct da9062_regulator *regl = rdev_get_drvdata(rdev); + struct regmap_field *field; + int ret, val; + + /* Detect current regulator state */ + ret = regmap_field_read(regl->suspend, &val); + if (ret < 0) + return 0; + + /* Read regulator mode from proper register, depending on state */ + if (val) + field = regl->suspend_sleep; + else + field = regl->sleep; + + ret = regmap_field_read(field, &val); + if (ret < 0) + return 0; + + if (val) + return REGULATOR_MODE_STANDBY; + else + return REGULATOR_MODE_NORMAL; +} + +static int da9062_buck_get_status(struct regulator_dev *rdev) +{ + int ret = regulator_is_enabled_regmap(rdev); + + if (ret == 0) { + ret = REGULATOR_STATUS_OFF; + } else if (ret > 0) { + ret = da9062_buck_get_mode(rdev); + if (ret > 0) + ret = regulator_mode_to_status(ret); + else if (ret == 0) + ret = -EIO; + } + + return ret; +} + +static int da9062_ldo_get_status(struct regulator_dev *rdev) +{ + int ret = regulator_is_enabled_regmap(rdev); + + if (ret == 0) { + ret = REGULATOR_STATUS_OFF; + } else if (ret > 0) { + ret = da9062_ldo_get_mode(rdev); + if (ret > 0) + ret = regulator_mode_to_status(ret); + else if (ret == 0) + ret = -EIO; + } + + return ret; +} + +static int da9062_set_suspend_voltage(struct regulator_dev *rdev, int uv) +{ + struct da9062_regulator *regl = rdev_get_drvdata(rdev); + const struct da9062_regulator_info *rinfo = regl->info; + int ret, sel; + + sel = regulator_map_voltage_linear(rdev, uv, uv); + if (sel < 0) + return sel; + + sel <<= ffs(rdev->desc->vsel_mask) - 1; + + ret = regmap_update_bits(regl->hw->regmap, rinfo->suspend_vsel_reg, + rdev->desc->vsel_mask, sel); + + return ret; +} + +static int da9062_suspend_enable(struct regulator_dev *rdev) +{ + struct da9062_regulator *regl = rdev_get_drvdata(rdev); + + return regmap_field_write(regl->suspend, 1); +} + +static int da9062_suspend_disable(struct regulator_dev *rdev) +{ + struct da9062_regulator *regl = rdev_get_drvdata(rdev); + + return regmap_field_write(regl->suspend, 0); +} + +static int da9062_buck_set_suspend_mode(struct regulator_dev *rdev, + unsigned mode) +{ + struct da9062_regulator *regl = rdev_get_drvdata(rdev); + int val; + + switch (mode) { + case REGULATOR_MODE_FAST: + val = BUCK_MODE_SYNC; + break; + case REGULATOR_MODE_NORMAL: + val = BUCK_MODE_AUTO; + break; + case REGULATOR_MODE_STANDBY: + val = BUCK_MODE_SLEEP; + break; + default: + return -EINVAL; + } + + return regmap_field_write(regl->mode, val); +} + +static int da9062_ldo_set_suspend_mode(struct regulator_dev *rdev, + unsigned mode) +{ + struct da9062_regulator *regl = rdev_get_drvdata(rdev); + unsigned val; + + switch (mode) { + case REGULATOR_MODE_NORMAL: + val = 0; + break; + case REGULATOR_MODE_STANDBY: + val = 1; + break; + default: + return -EINVAL; + } + + return regmap_field_write(regl->suspend_sleep, val); +} + +static struct regulator_ops da9062_buck_ops = { + .enable = regulator_enable_regmap, + .disable = regulator_disable_regmap, + .is_enabled = regulator_is_enabled_regmap, + .get_voltage_sel = regulator_get_voltage_sel_regmap, + .set_voltage_sel = regulator_set_voltage_sel_regmap, + .list_voltage = regulator_list_voltage_linear, + .set_current_limit = da9062_set_current_limit, + .get_current_limit = da9062_get_current_limit, + .set_mode = da9062_buck_set_mode, + .get_mode = da9062_buck_get_mode, + .get_status = da9062_buck_get_status, + .set_suspend_voltage = da9062_set_suspend_voltage, + .set_suspend_enable = da9062_suspend_enable, + .set_suspend_disable = da9062_suspend_disable, + .set_suspend_mode = da9062_buck_set_suspend_mode, +}; + +static struct regulator_ops da9062_ldo_ops = { + .enable = regulator_enable_regmap, + .disable = regulator_disable_regmap, + .is_enabled = regulator_is_enabled_regmap, + .get_voltage_sel = regulator_get_voltage_sel_regmap, + .set_voltage_sel = regulator_set_voltage_sel_regmap, + .list_voltage = regulator_list_voltage_linear, + .set_mode = da9062_ldo_set_mode, + .get_mode = da9062_ldo_get_mode, + .get_status = da9062_ldo_get_status, + .set_suspend_voltage = da9062_set_suspend_voltage, + .set_suspend_enable = da9062_suspend_enable, + .set_suspend_disable = da9062_suspend_disable, + .set_suspend_mode = da9062_ldo_set_suspend_mode, +}; + +/* Regulator information */ +static const struct da9062_regulator_info local_regulator_info[] = { + { + .desc.id = DA9062_ID_BUCK1, + .desc.name = "DA9062 BUCK1", + .desc.of_match = of_match_ptr("buck1"), + .desc.regulators_node = of_match_ptr("regulators"), + .desc.ops = &da9062_buck_ops, + .desc.min_uV = (300) * 1000, + .desc.uV_step = (10) * 1000, + .desc.n_voltages = ((1570) - (300))/(10) + 1, + .current_limits = da9062_buck_a_limits, + .n_current_limits = ARRAY_SIZE(da9062_buck_a_limits), + .desc.enable_reg = DA9062AA_BUCK1_CONT, + .desc.enable_mask = DA9062AA_BUCK1_EN_MASK, + .desc.vsel_reg = DA9062AA_VBUCK1_A, + .desc.vsel_mask = DA9062AA_VBUCK1_A_MASK, + .desc.linear_min_sel = 0, + .sleep = REG_FIELD(DA9062AA_VBUCK1_A, + __builtin_ffs((int)DA9062AA_BUCK1_SL_A_MASK) - 1, + sizeof(unsigned int) * 8 - + __builtin_clz((DA9062AA_BUCK1_SL_A_MASK)) - 1), + .suspend_sleep = REG_FIELD(DA9062AA_VBUCK1_B, + __builtin_ffs((int)DA9062AA_BUCK1_SL_B_MASK) - 1, + sizeof(unsigned int) * 8 - + __builtin_clz((DA9062AA_BUCK1_SL_B_MASK)) - 1), + .suspend_vsel_reg = DA9062AA_VBUCK1_B, + .mode = REG_FIELD(DA9062AA_BUCK1_CFG, + __builtin_ffs((int)DA9062AA_BUCK1_MODE_MASK) - 1, + sizeof(unsigned int) * 8 - + __builtin_clz((DA9062AA_BUCK1_MODE_MASK)) - 1), + .suspend = REG_FIELD(DA9062AA_DVC_1, + __builtin_ffs((int)DA9062AA_VBUCK1_SEL_MASK) - 1, + sizeof(unsigned int) * 8 - + __builtin_clz((DA9062AA_VBUCK1_SEL_MASK)) - 1), + .ilimit = REG_FIELD(DA9062AA_BUCK_ILIM_C, + __builtin_ffs((int)DA9062AA_BUCK1_ILIM_MASK) - 1, + sizeof(unsigned int) * 8 - + __builtin_clz((DA9062AA_BUCK1_ILIM_MASK)) - 1), + }, + { + .desc.id = DA9062_ID_BUCK2, + .desc.name = "DA9062 BUCK2", + .desc.of_match = of_match_ptr("buck2"), + .desc.regulators_node = of_match_ptr("regulators"), + .desc.ops = &da9062_buck_ops, + .desc.min_uV = (300) * 1000, + .desc.uV_step = (10) * 1000, + .desc.n_voltages = ((1570) - (300))/(10) + 1, + .current_limits = da9062_buck_a_limits, + .n_current_limits = ARRAY_SIZE(da9062_buck_a_limits), + .desc.enable_reg = DA9062AA_BUCK2_CONT, + .desc.enable_mask = DA9062AA_BUCK2_EN_MASK, + .desc.vsel_reg = DA9062AA_VBUCK2_A, + .desc.vsel_mask = DA9062AA_VBUCK2_A_MASK, + .desc.linear_min_sel = 0, + .sleep = REG_FIELD(DA9062AA_VBUCK2_A, + __builtin_ffs((int)DA9062AA_BUCK2_SL_A_MASK) - 1, + sizeof(unsigned int) * 8 - + __builtin_clz((DA9062AA_BUCK2_SL_A_MASK)) - 1), + .suspend_sleep = REG_FIELD(DA9062AA_VBUCK2_B, + __builtin_ffs((int)DA9062AA_BUCK2_SL_B_MASK) - 1, + sizeof(unsigned int) * 8 - + __builtin_clz((DA9062AA_BUCK2_SL_B_MASK)) - 1), + .suspend_vsel_reg = DA9062AA_VBUCK2_B, + .mode = REG_FIELD(DA9062AA_BUCK2_CFG, + __builtin_ffs((int)DA9062AA_BUCK2_MODE_MASK) - 1, + sizeof(unsigned int) * 8 - + __builtin_clz((DA9062AA_BUCK2_MODE_MASK)) - 1), + .suspend = REG_FIELD(DA9062AA_DVC_1, + __builtin_ffs((int)DA9062AA_VBUCK2_SEL_MASK) - 1, + sizeof(unsigned int) * 8 - + __builtin_clz((DA9062AA_VBUCK2_SEL_MASK)) - 1), + .ilimit = REG_FIELD(DA9062AA_BUCK_ILIM_C, + __builtin_ffs((int)DA9062AA_BUCK2_ILIM_MASK) - 1, + sizeof(unsigned int) * 8 - + __builtin_clz((DA9062AA_BUCK2_ILIM_MASK)) - 1), + }, + { + .desc.id = DA9062_ID_BUCK3, + .desc.name = "DA9062 BUCK3", + .desc.of_match = of_match_ptr("buck3"), + .desc.regulators_node = of_match_ptr("regulators"), + .desc.ops = &da9062_buck_ops, + .desc.min_uV = (800) * 1000, + .desc.uV_step = (20) * 1000, + .desc.n_voltages = ((3340) - (800))/(20) + 1, + .current_limits = da9062_buck_b_limits, + .n_current_limits = ARRAY_SIZE(da9062_buck_b_limits), + .desc.enable_reg = DA9062AA_BUCK3_CONT, + .desc.enable_mask = DA9062AA_BUCK3_EN_MASK, + .desc.vsel_reg = DA9062AA_VBUCK3_A, + .desc.vsel_mask = DA9062AA_VBUCK3_A_MASK, + .desc.linear_min_sel = 0, + .sleep = REG_FIELD(DA9062AA_VBUCK3_A, + __builtin_ffs((int)DA9062AA_BUCK3_SL_A_MASK) - 1, + sizeof(unsigned int) * 8 - + __builtin_clz((DA9062AA_BUCK3_SL_A_MASK)) - 1), + .suspend_sleep = REG_FIELD(DA9062AA_VBUCK3_B, + __builtin_ffs((int)DA9062AA_BUCK3_SL_B_MASK) - 1, + sizeof(unsigned int) * 8 - + __builtin_clz((DA9062AA_BUCK3_SL_B_MASK)) - 1), + .suspend_vsel_reg = DA9062AA_VBUCK3_B, + .mode = REG_FIELD(DA9062AA_BUCK3_CFG, + __builtin_ffs((int)DA9062AA_BUCK3_MODE_MASK) - 1, + sizeof(unsigned int) * 8 - + __builtin_clz((DA9062AA_BUCK3_MODE_MASK)) - 1), + .suspend = REG_FIELD(DA9062AA_DVC_1, + __builtin_ffs((int)DA9062AA_VBUCK3_SEL_MASK) - 1, + sizeof(unsigned int) * 8 - + __builtin_clz((DA9062AA_VBUCK3_SEL_MASK)) - 1), + .ilimit = REG_FIELD(DA9062AA_BUCK_ILIM_A, + __builtin_ffs((int)DA9062AA_BUCK3_ILIM_MASK) - 1, + sizeof(unsigned int) * 8 - + __builtin_clz((DA9062AA_BUCK3_ILIM_MASK)) - 1), + }, + { + .desc.id = DA9062_ID_BUCK4, + .desc.name = "DA9062 BUCK4", + .desc.of_match = of_match_ptr("buck4"), + .desc.regulators_node = of_match_ptr("regulators"), + .desc.ops = &da9062_buck_ops, + .desc.min_uV = (530) * 1000, + .desc.uV_step = (10) * 1000, + .desc.n_voltages = ((1800) - (530))/(10) + 1, + .current_limits = da9062_buck_a_limits, + .n_current_limits = ARRAY_SIZE(da9062_buck_a_limits), + .desc.enable_reg = DA9062AA_BUCK4_CONT, + .desc.enable_mask = DA9062AA_BUCK4_EN_MASK, + .desc.vsel_reg = DA9062AA_VBUCK4_A, + .desc.vsel_mask = DA9062AA_VBUCK4_A_MASK, + .desc.linear_min_sel = 0, + .sleep = REG_FIELD(DA9062AA_VBUCK4_A, + __builtin_ffs((int)DA9062AA_BUCK4_SL_A_MASK) - 1, + sizeof(unsigned int) * 8 - + __builtin_clz((DA9062AA_BUCK4_SL_A_MASK)) - 1), + .suspend_sleep = REG_FIELD(DA9062AA_VBUCK4_B, + __builtin_ffs((int)DA9062AA_BUCK4_SL_B_MASK) - 1, + sizeof(unsigned int) * 8 - + __builtin_clz((DA9062AA_BUCK4_SL_B_MASK)) - 1), + .suspend_vsel_reg = DA9062AA_VBUCK4_B, + .mode = REG_FIELD(DA9062AA_BUCK4_CFG, + __builtin_ffs((int)DA9062AA_BUCK4_MODE_MASK) - 1, + sizeof(unsigned int) * 8 - + __builtin_clz((DA9062AA_BUCK4_MODE_MASK)) - 1), + .suspend = REG_FIELD(DA9062AA_DVC_1, + __builtin_ffs((int)DA9062AA_VBUCK4_SEL_MASK) - 1, + sizeof(unsigned int) * 8 - + __builtin_clz((DA9062AA_VBUCK4_SEL_MASK)) - 1), + .ilimit = REG_FIELD(DA9062AA_BUCK_ILIM_B, + __builtin_ffs((int)DA9062AA_BUCK4_ILIM_MASK) - 1, + sizeof(unsigned int) * 8 - + __builtin_clz((DA9062AA_BUCK4_ILIM_MASK)) - 1), + }, + { + .desc.id = DA9062_ID_LDO1, + .desc.name = "DA9062 LDO1", + .desc.of_match = of_match_ptr("ldo1"), + .desc.regulators_node = of_match_ptr("regulators"), + .desc.ops = &da9062_ldo_ops, + .desc.min_uV = (900) * 1000, + .desc.uV_step = (50) * 1000, + .desc.n_voltages = ((3600) - (900))/(50) + 1, + .desc.enable_reg = DA9062AA_LDO1_CONT, + .desc.enable_mask = DA9062AA_LDO1_EN_MASK, + .desc.vsel_reg = DA9062AA_VLDO1_A, + .desc.vsel_mask = DA9062AA_VLDO1_A_MASK, + .desc.linear_min_sel = 0, + .sleep = REG_FIELD(DA9062AA_VLDO1_A, + __builtin_ffs((int)DA9062AA_LDO1_SL_A_MASK) - 1, + sizeof(unsigned int) * 8 - + __builtin_clz((DA9062AA_LDO1_SL_A_MASK)) - 1), + .suspend_sleep = REG_FIELD(DA9062AA_VLDO1_B, + __builtin_ffs((int)DA9062AA_LDO1_SL_B_MASK) - 1, + sizeof(unsigned int) * 8 - + __builtin_clz((DA9062AA_LDO1_SL_B_MASK)) - 1), + .suspend_vsel_reg = DA9062AA_VLDO1_B, + .suspend = REG_FIELD(DA9062AA_DVC_1, + __builtin_ffs((int)DA9062AA_VLDO1_SEL_MASK) - 1, + sizeof(unsigned int) * 8 - + __builtin_clz((DA9062AA_VLDO1_SEL_MASK)) - 1), + .oc_event = REG_FIELD(DA9062AA_STATUS_D, + __builtin_ffs((int)DA9062AA_LDO1_ILIM_MASK) - 1, + sizeof(unsigned int) * 8 - + __builtin_clz((DA9062AA_LDO1_ILIM_MASK)) - 1), + }, + { + .desc.id = DA9062_ID_LDO2, + .desc.name = "DA9062 LDO2", + .desc.of_match = of_match_ptr("ldo2"), + .desc.regulators_node = of_match_ptr("regulators"), + .desc.ops = &da9062_ldo_ops, + .desc.min_uV = (900) * 1000, + .desc.uV_step = (50) * 1000, + .desc.n_voltages = ((3600) - (600))/(50) + 1, + .desc.enable_reg = DA9062AA_LDO2_CONT, + .desc.enable_mask = DA9062AA_LDO2_EN_MASK, + .desc.vsel_reg = DA9062AA_VLDO2_A, + .desc.vsel_mask = DA9062AA_VLDO2_A_MASK, + .desc.linear_min_sel = 0, + .sleep = REG_FIELD(DA9062AA_VLDO2_A, + __builtin_ffs((int)DA9062AA_LDO2_SL_A_MASK) - 1, + sizeof(unsigned int) * 8 - + __builtin_clz((DA9062AA_LDO2_SL_A_MASK)) - 1), + .suspend_sleep = REG_FIELD(DA9062AA_VLDO2_B, + __builtin_ffs((int)DA9062AA_LDO2_SL_B_MASK) - 1, + sizeof(unsigned int) * 8 - + __builtin_clz((DA9062AA_LDO2_SL_B_MASK)) - 1), + .suspend_vsel_reg = DA9062AA_VLDO2_B, + .suspend = REG_FIELD(DA9062AA_DVC_1, + __builtin_ffs((int)DA9062AA_VLDO2_SEL_MASK) - 1, + sizeof(unsigned int) * 8 - + __builtin_clz((DA9062AA_VLDO2_SEL_MASK)) - 1), + .oc_event = REG_FIELD(DA9062AA_STATUS_D, + __builtin_ffs((int)DA9062AA_LDO2_ILIM_MASK) - 1, + sizeof(unsigned int) * 8 - + __builtin_clz((DA9062AA_LDO2_ILIM_MASK)) - 1), + }, + { + .desc.id = DA9062_ID_LDO3, + .desc.name = "DA9062 LDO3", + .desc.of_match = of_match_ptr("ldo3"), + .desc.regulators_node = of_match_ptr("regulators"), + .desc.ops = &da9062_ldo_ops, + .desc.min_uV = (900) * 1000, + .desc.uV_step = (50) * 1000, + .desc.n_voltages = ((3600) - (900))/(50) + 1, + .desc.enable_reg = DA9062AA_LDO3_CONT, + .desc.enable_mask = DA9062AA_LDO3_EN_MASK, + .desc.vsel_reg = DA9062AA_VLDO3_A, + .desc.vsel_mask = DA9062AA_VLDO3_A_MASK, + .desc.linear_min_sel = 0, + .sleep = REG_FIELD(DA9062AA_VLDO3_A, + __builtin_ffs((int)DA9062AA_LDO3_SL_A_MASK) - 1, + sizeof(unsigned int) * 8 - + __builtin_clz((DA9062AA_LDO3_SL_A_MASK)) - 1), + .suspend_sleep = REG_FIELD(DA9062AA_VLDO3_B, + __builtin_ffs((int)DA9062AA_LDO3_SL_B_MASK) - 1, + sizeof(unsigned int) * 8 - + __builtin_clz((DA9062AA_LDO3_SL_B_MASK)) - 1), + .suspend_vsel_reg = DA9062AA_VLDO3_B, + .suspend = REG_FIELD(DA9062AA_DVC_1, + __builtin_ffs((int)DA9062AA_VLDO3_SEL_MASK) - 1, + sizeof(unsigned int) * 8 - + __builtin_clz((DA9062AA_VLDO3_SEL_MASK)) - 1), + .oc_event = REG_FIELD(DA9062AA_STATUS_D, + __builtin_ffs((int)DA9062AA_LDO3_ILIM_MASK) - 1, + sizeof(unsigned int) * 8 - + __builtin_clz((DA9062AA_LDO3_ILIM_MASK)) - 1), + }, + { + .desc.id = DA9062_ID_LDO4, + .desc.name = "DA9062 LDO4", + .desc.of_match = of_match_ptr("ldo4"), + .desc.regulators_node = of_match_ptr("regulators"), + .desc.ops = &da9062_ldo_ops, + .desc.min_uV = (900) * 1000, + .desc.uV_step = (50) * 1000, + .desc.n_voltages = ((3600) - (900))/(50) + 1, + .desc.enable_reg = DA9062AA_LDO4_CONT, + .desc.enable_mask = DA9062AA_LDO4_EN_MASK, + .desc.vsel_reg = DA9062AA_VLDO4_A, + .desc.vsel_mask = DA9062AA_VLDO4_A_MASK, + .desc.linear_min_sel = 0, + .sleep = REG_FIELD(DA9062AA_VLDO4_A, + __builtin_ffs((int)DA9062AA_LDO4_SL_A_MASK) - 1, + sizeof(unsigned int) * 8 - + __builtin_clz((DA9062AA_LDO4_SL_A_MASK)) - 1), + .suspend_sleep = REG_FIELD(DA9062AA_VLDO4_B, + __builtin_ffs((int)DA9062AA_LDO4_SL_B_MASK) - 1, + sizeof(unsigned int) * 8 - + __builtin_clz((DA9062AA_LDO4_SL_B_MASK)) - 1), + .suspend_vsel_reg = DA9062AA_VLDO4_B, + .suspend = REG_FIELD(DA9062AA_DVC_1, + __builtin_ffs((int)DA9062AA_VLDO4_SEL_MASK) - 1, + sizeof(unsigned int) * 8 - + __builtin_clz((DA9062AA_VLDO4_SEL_MASK)) - 1), + .oc_event = REG_FIELD(DA9062AA_STATUS_D, + __builtin_ffs((int)DA9062AA_LDO4_ILIM_MASK) - 1, + sizeof(unsigned int) * 8 - + __builtin_clz((DA9062AA_LDO4_ILIM_MASK)) - 1), + }, +}; + +/* Regulator interrupt handlers */ +static irqreturn_t da9062_ldo_lim_event(int irq, void *data) +{ + struct da9062_regulators *regulators = data; + struct da9062 *hw = regulators->regulator[0].hw; + struct da9062_regulator *regl; + int handled = IRQ_NONE; + int bits, i, ret; + + ret = regmap_read(hw->regmap, DA9062AA_STATUS_D, &bits); + if (ret < 0) { + dev_err(hw->dev, + "Failed to read LDO overcurrent indicator\n"); + goto ldo_lim_error; + } + + for (i = regulators->n_regulators - 1; i >= 0; i--) { + regl = ®ulators->regulator[i]; + if (regl->info->oc_event.reg != DA9062AA_STATUS_D) + continue; + + if (BIT(regl->info->oc_event.lsb) & bits) { + regulator_notifier_call_chain(regl->rdev, + REGULATOR_EVENT_OVER_CURRENT, NULL); + handled = IRQ_HANDLED; + } + } + +ldo_lim_error: + return handled; +} + +static int da9062_regulator_probe(struct platform_device *pdev) +{ + struct da9062 *chip = dev_get_drvdata(pdev->dev.parent); + struct of_regulator_match *reg_matches = NULL; + struct da9062_regulators *regulators; + struct da9062_regulator *regl; + struct regulator_config config = { }; + int irq, n, ret; + size_t size; + + /* Allocate memory required by usable regulators */ + size = sizeof(struct da9062_regulators) + + DA9062_MAX_REGULATORS * sizeof(struct da9062_regulator); + regulators = devm_kzalloc(&pdev->dev, size, GFP_KERNEL); + if (!regulators) + return -ENOMEM; + + regulators->n_regulators = DA9062_MAX_REGULATORS; + platform_set_drvdata(pdev, regulators); + + n = 0; + while (n < regulators->n_regulators) { + /* Initialise regulator structure */ + regl = ®ulators->regulator[n]; + regl->hw = chip; + regl->info = &local_regulator_info[n]; + regl->desc = regl->info->desc; + regl->desc.type = REGULATOR_VOLTAGE; + regl->desc.owner = THIS_MODULE; + + if (regl->info->mode.reg) + regl->mode = devm_regmap_field_alloc( + &pdev->dev, + chip->regmap, + regl->info->mode); + if (regl->info->suspend.reg) + regl->suspend = devm_regmap_field_alloc( + &pdev->dev, + chip->regmap, + regl->info->suspend); + if (regl->info->sleep.reg) + regl->sleep = devm_regmap_field_alloc( + &pdev->dev, + chip->regmap, + regl->info->sleep); + if (regl->info->suspend_sleep.reg) + regl->suspend_sleep = devm_regmap_field_alloc( + &pdev->dev, + chip->regmap, + regl->info->suspend_sleep); + if (regl->info->ilimit.reg) + regl->ilimit = devm_regmap_field_alloc( + &pdev->dev, + chip->regmap, + regl->info->ilimit); + + /* Register regulator */ + memset(&config, 0, sizeof(config)); + config.dev = chip->dev; + config.driver_data = regl; + config.regmap = chip->regmap; + + regl->rdev = devm_regulator_register(&pdev->dev, ®l->desc, + &config); + if (IS_ERR(regl->rdev)) { + dev_err(&pdev->dev, + "Failed to register %s regulator\n", + regl->desc.name); + return PTR_ERR(regl->rdev); + } + + n++; + } + + /* LDOs overcurrent event support */ + irq = platform_get_irq_byname(pdev, "LDO_LIM"); + if (irq < 0) { + dev_err(&pdev->dev, "Failed to get IRQ.\n"); + return irq; + } + regulators->irq_ldo_lim = irq; + + ret = devm_request_threaded_irq(&pdev->dev, irq, + NULL, da9062_ldo_lim_event, + IRQF_TRIGGER_LOW | IRQF_ONESHOT, + "LDO_LIM", regulators); + if (ret) { + dev_warn(&pdev->dev, + "Failed to request LDO_LIM IRQ.\n"); + regulators->irq_ldo_lim = -ENXIO; + } + + return 0; +} + +static struct platform_driver da9062_regulator_driver = { + .driver = { + .name = "da9062-regulators", + .owner = THIS_MODULE, + }, + .probe = da9062_regulator_probe, +}; + +static int __init da9062_regulator_init(void) +{ + return platform_driver_register(&da9062_regulator_driver); +} +subsys_initcall(da9062_regulator_init); + +static void __exit da9062_regulator_cleanup(void) +{ + platform_driver_unregister(&da9062_regulator_driver); +} +module_exit(da9062_regulator_cleanup); + +/* Module information */ +MODULE_AUTHOR("S Twiss <stwiss.opensource@diasemi.com>"); +MODULE_DESCRIPTION("REGULATOR device driver for Dialog DA9062"); +MODULE_LICENSE("GPL"); +MODULE_ALIAS("platform: da9062-regulators"); -- end-of-patch for PATCH V3 ^ permalink raw reply related [flat|nested] 18+ messages in thread
* Re: [PATCH V3 2/4] regulator: da9062: DA9062 regulator driver 2015-05-19 13:10 ` [PATCH V3 2/4] regulator: da9062: DA9062 regulator driver S Twiss @ 2015-05-21 12:04 ` Mark Brown 2015-05-21 13:48 ` Opensource [Steve Twiss] 0 siblings, 1 reply; 18+ messages in thread From: Mark Brown @ 2015-05-21 12:04 UTC (permalink / raw) To: S Twiss Cc: LINUXKERNEL, Liam Girdwood, Alessandro Zummo, DEVICETREE, David Dajun Chen, Dmitry Torokhov, Ian Campbell, Kumar Gala, LINUXINPUT, LINUXWATCHDOG, Lee Jones, Mark Rutland, Pawel Moll, RTCLINUX, Rob Herring, Samuel Ortiz, Support Opensource, Wim Van Sebroeck [-- Attachment #1: Type: text/plain, Size: 187 bytes --] On Tue, May 19, 2015 at 02:10:30PM +0100, S Twiss wrote: > From: S Twiss <stwiss.opensource@diasemi.com> > > Add BUCK and LDO regulator driver support for DA9062 Applied, thanks. [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 473 bytes --] ^ permalink raw reply [flat|nested] 18+ messages in thread
* RE: [PATCH V3 2/4] regulator: da9062: DA9062 regulator driver 2015-05-21 12:04 ` Mark Brown @ 2015-05-21 13:48 ` Opensource [Steve Twiss] 0 siblings, 0 replies; 18+ messages in thread From: Opensource [Steve Twiss] @ 2015-05-21 13:48 UTC (permalink / raw) To: Mark Brown Cc: LINUXKERNEL, Liam Girdwood, Alessandro Zummo, DEVICETREE, David Dajun Chen, Dmitry Torokhov, Ian Campbell, Kumar Gala, LINUXINPUT, LINUXWATCHDOG, Lee Jones, Mark Rutland, Pawel Moll, RTCLINUX, Rob Herring, Samuel Ortiz, Support Opensource, Wim Van Sebroeck On 21 May 2015 13:04 Mark Brown wrote: > Subject: Re: [PATCH V3 2/4] regulator: da9062: DA9062 regulator driver > > On Tue, May 19, 2015 at 02:10:30PM +0100, S Twiss wrote: > > From: S Twiss <stwiss.opensource@diasemi.com> > > > > Add BUCK and LDO regulator driver support for DA9062 > > Applied, thanks. Thank you Regards, Steve ^ permalink raw reply [flat|nested] 18+ messages in thread
end of thread, other threads:[~2015-07-01 8:02 UTC | newest]
Thread overview: 18+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-05-19 13:10 [PATCH V3 0/4] da9062: DA9062 driver submission S Twiss
2015-05-19 13:10 ` [PATCH V3 3/4] watchdog: da9062: DA9062 watchdog driver S Twiss
[not found] ` <ded171f203fe1d0c72f0d7ea4c3d54257f4e970e.1432041031.git.stwiss.opensource-WBD+wuPFNBhBDgjK7y7TUQ@public.gmane.org>
2015-05-20 20:30 ` Guenter Roeck
[not found] ` <20150520203016.GA5710-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org>
2015-05-21 7:15 ` Opensource [Steve Twiss]
[not found] ` <cover.1432041031.git.stwiss.opensource-WBD+wuPFNBhBDgjK7y7TUQ@public.gmane.org>
2015-05-19 13:10 ` [PATCH V3 1/4] mfd: da9062: DA9062 MFD core driver S Twiss
2015-05-26 16:10 ` Lee Jones
2015-05-28 12:52 ` Opensource [Steve Twiss]
2015-06-11 8:27 ` Opensource [Steve Twiss]
2015-06-11 8:50 ` Lee Jones
2015-06-11 8:56 ` Lee Jones
2015-06-11 9:27 ` Opensource [Steve Twiss]
2015-06-11 21:46 ` [rtc-linux] " Alexandre Belloni
2015-06-29 8:01 ` Opensource [Steve Twiss]
2015-07-01 8:02 ` Lee Jones
2015-05-19 13:10 ` [PATCH V3 4/4] devicetree: da9062: Add bindings for DA9062 driver S Twiss
2015-05-19 13:10 ` [PATCH V3 2/4] regulator: da9062: DA9062 regulator driver S Twiss
2015-05-21 12:04 ` Mark Brown
2015-05-21 13:48 ` Opensource [Steve Twiss]
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).