* [PATCH V3 2/2] mfd: Add ROHM BD9571MWV-M MFD PMIC driver
[not found] <20170502121816.10572-1-marek.vasut+renesas@gmail.com>
@ 2017-05-02 12:18 ` Marek Vasut
2017-06-27 9:54 ` Marek Vasut
2017-07-10 15:33 ` Geert Uytterhoeven
0 siblings, 2 replies; 9+ messages in thread
From: Marek Vasut @ 2017-05-02 12:18 UTC (permalink / raw)
To: linux-renesas-soc
Cc: lee.jones, Marek Vasut, linux-kernel, Geert Uytterhoeven
Add the MFD part of the ROHM BD9571MWV-M PMIC driver and MAINTAINERS
entry. The MFD part only specifies the regmap bits for the PMIC and
binds the subdevs together.
Signed-off-by: Marek Vasut <marek.vasut+renesas@gmail.com>
Cc: linux-kernel@vger.kernel.org
Cc: Geert Uytterhoeven <geert+renesas@glider.be>
Cc: Lee Jones <lee.jones@linaro.org>
---
V2: - Change BD9571MWV_AVS_VD09_VID0,1,2,3 to BD9571MWV_AVS_VD09_VID(n)
- Change BD9571MWV_AVS_DVFS_VID0,1,2,3 to BD9571MWV_AVS_DVFS_VID(n)
- Make the AVS_VD09 range RW, so it can be used by the regulator
driver for the VD09 regulator
- Report the regmap read return values when attempting to read ID
registers fails
V3: No change
---
MAINTAINERS | 11 ++
drivers/mfd/Kconfig | 13 +++
drivers/mfd/Makefile | 1 +
drivers/mfd/bd9571mwv.c | 230 ++++++++++++++++++++++++++++++++++++++++++
include/linux/mfd/bd9571mwv.h | 115 +++++++++++++++++++++
5 files changed, 370 insertions(+)
create mode 100644 drivers/mfd/bd9571mwv.c
create mode 100644 include/linux/mfd/bd9571mwv.h
diff --git a/MAINTAINERS b/MAINTAINERS
index 2a9aa4be5846..9c30e6b00358 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -10942,6 +10942,17 @@ L: linux-serial@vger.kernel.org
S: Odd Fixes
F: drivers/tty/serial/rp2.*
+ROHM MULTIFUNCTION BD9571MWV-M PMIC DEVICE DRIVERS
+M: Marek Vasut <marek.vasut+renesas@gmail.com>
+L: linux-kernel@vger.kernel.org
+L: linux-renesas-soc@vger.kernel.org
+S: Supported
+F: drivers/mfd/bd9571mwv.c
+F: drivers/regulator/bd9571mwv-regulator.c
+F: drivers/gpio/gpio-bd9571mwv.c
+F: include/linux/mfd/bd9571mwv.h
+F: Documentation/devicetree/bindings/mfd/bd9571mwv.txt
+
ROSE NETWORK LAYER
M: Ralf Baechle <ralf@linux-mips.org>
L: linux-hams@vger.kernel.org
diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig
index 3eb5c93595f6..6f4668ae28cc 100644
--- a/drivers/mfd/Kconfig
+++ b/drivers/mfd/Kconfig
@@ -133,6 +133,19 @@ config MFD_BCM590XX
help
Support for the BCM590xx PMUs from Broadcom
+config MFD_BD9571MWV
+ tristate "ROHM BD9571MWV PMIC"
+ select MFD_CORE
+ select REGMAP_I2C
+ depends on I2C
+ help
+ Support for the ROHM BD9571MWV PMIC, which contains single
+ voltage regulator, voltage sampling units, GPIO block and
+ watchdog block.
+
+ This driver can also be built as a module. If so, the module
+ will be called bd9571mwv.
+
config MFD_AC100
tristate "X-Powers AC100"
select MFD_CORE
diff --git a/drivers/mfd/Makefile b/drivers/mfd/Makefile
index c16bf1ea0ea9..3cacc9748e13 100644
--- a/drivers/mfd/Makefile
+++ b/drivers/mfd/Makefile
@@ -10,6 +10,7 @@ obj-$(CONFIG_MFD_ACT8945A) += act8945a.o
obj-$(CONFIG_MFD_SM501) += sm501.o
obj-$(CONFIG_MFD_ASIC3) += asic3.o tmio_core.o
obj-$(CONFIG_MFD_BCM590XX) += bcm590xx.o
+obj-$(CONFIG_MFD_BD9571MWV) += bd9571mwv.o
cros_ec_core-objs := cros_ec.o
cros_ec_core-$(CONFIG_ACPI) += cros_ec_acpi_gpe.o
obj-$(CONFIG_MFD_CROS_EC) += cros_ec_core.o
diff --git a/drivers/mfd/bd9571mwv.c b/drivers/mfd/bd9571mwv.c
new file mode 100644
index 000000000000..64e088dfe7b0
--- /dev/null
+++ b/drivers/mfd/bd9571mwv.c
@@ -0,0 +1,230 @@
+/*
+ * ROHM BD9571MWV-M MFD driver
+ *
+ * Copyright (C) 2017 Marek Vasut <marek.vasut+renesas@gmail.com>
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License version 2 as
+ * published by the Free Software Foundation.
+ *
+ * This program is distributed "as is" WITHOUT ANY WARRANTY of any
+ * kind, whether expressed or implied; without even the implied warranty
+ * of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License version 2 for more details.
+ *
+ * Based on the TPS65086 driver
+ */
+
+#include <linux/i2c.h>
+#include <linux/interrupt.h>
+#include <linux/mfd/core.h>
+#include <linux/module.h>
+
+#include <linux/mfd/bd9571mwv.h>
+
+static const struct mfd_cell bd9571mwv_cells[] = {
+ { .name = "bd9571mwv-regulator", },
+ { .name = "bd9571mwv-gpio", },
+};
+
+static const struct regmap_range bd9571mwv_readable_yes_ranges[] = {
+ regmap_reg_range(BD9571MWV_VENDOR_CODE, BD9571MWV_PRODUCT_REVISION),
+ regmap_reg_range(BD9571MWV_AVS_SET_MONI, BD9571MWV_AVS_DVFS_VID(3)),
+ regmap_reg_range(BD9571MWV_VD18_VID, BD9571MWV_VD33_VID),
+ regmap_reg_range(BD9571MWV_DVFS_VINIT, BD9571MWV_DVFS_VINIT),
+ regmap_reg_range(BD9571MWV_DVFS_SETVMAX, BD9571MWV_DVFS_MONIVDAC),
+ regmap_reg_range(BD9571MWV_GPIO_IN, BD9571MWV_GPIO_IN),
+ regmap_reg_range(BD9571MWV_GPIO_INT, BD9571MWV_GPIO_INTMASK),
+ regmap_reg_range(BD9571MWV_INT_INTREQ, BD9571MWV_INT_INTMASK),
+};
+
+static const struct regmap_access_table bd9571mwv_readable_table = {
+ .yes_ranges = bd9571mwv_readable_yes_ranges,
+ .n_yes_ranges = ARRAY_SIZE(bd9571mwv_readable_yes_ranges),
+};
+
+static const struct regmap_range bd9571mwv_writable_yes_ranges[] = {
+ regmap_reg_range(BD9571MWV_AVS_VD09_VID(0), BD9571MWV_AVS_VD09_VID(3)),
+ regmap_reg_range(BD9571MWV_DVFS_SETVID, BD9571MWV_DVFS_SETVID),
+ regmap_reg_range(BD9571MWV_GPIO_DIR, BD9571MWV_GPIO_OUT),
+ regmap_reg_range(BD9571MWV_GPIO_INT_SET, BD9571MWV_GPIO_INTMASK),
+ regmap_reg_range(BD9571MWV_INT_INTREQ, BD9571MWV_INT_INTMASK),
+};
+
+static const struct regmap_access_table bd9571mwv_writable_table = {
+ .yes_ranges = bd9571mwv_writable_yes_ranges,
+ .n_yes_ranges = ARRAY_SIZE(bd9571mwv_writable_yes_ranges),
+};
+
+static const struct regmap_range bd9571mwv_volatile_yes_ranges[] = {
+ regmap_reg_range(BD9571MWV_GPIO_IN, BD9571MWV_GPIO_IN),
+ regmap_reg_range(BD9571MWV_GPIO_INT, BD9571MWV_GPIO_INT),
+ regmap_reg_range(BD9571MWV_INT_INTREQ, BD9571MWV_INT_INTREQ),
+};
+
+static const struct regmap_access_table bd9571mwv_volatile_table = {
+ .yes_ranges = bd9571mwv_volatile_yes_ranges,
+ .n_yes_ranges = ARRAY_SIZE(bd9571mwv_volatile_yes_ranges),
+};
+
+static const struct regmap_config bd9571mwv_regmap_config = {
+ .reg_bits = 8,
+ .val_bits = 8,
+ .cache_type = REGCACHE_RBTREE,
+ .rd_table = &bd9571mwv_readable_table,
+ .wr_table = &bd9571mwv_writable_table,
+ .volatile_table = &bd9571mwv_volatile_table,
+ .max_register = 0xff,
+};
+
+static const struct regmap_irq bd9571mwv_irqs[] = {
+ REGMAP_IRQ_REG(BD9571MWV_IRQ_MD1, 0,
+ BD9571MWV_INT_INTREQ_MD1_INT),
+ REGMAP_IRQ_REG(BD9571MWV_IRQ_MD2_E1, 0,
+ BD9571MWV_INT_INTREQ_MD2_E1_INT),
+ REGMAP_IRQ_REG(BD9571MWV_IRQ_MD2_E2, 0,
+ BD9571MWV_INT_INTREQ_MD2_E2_INT),
+ REGMAP_IRQ_REG(BD9571MWV_IRQ_PROT_ERR, 0,
+ BD9571MWV_INT_INTREQ_PROT_ERR_INT),
+ REGMAP_IRQ_REG(BD9571MWV_IRQ_GP, 0,
+ BD9571MWV_INT_INTREQ_GP_INT),
+ REGMAP_IRQ_REG(BD9571MWV_IRQ_128H_OF, 0,
+ BD9571MWV_INT_INTREQ_128H_OF_INT),
+ REGMAP_IRQ_REG(BD9571MWV_IRQ_WDT_OF, 0,
+ BD9571MWV_INT_INTREQ_WDT_OF_INT),
+ REGMAP_IRQ_REG(BD9571MWV_IRQ_BKUP_TRG, 0,
+ BD9571MWV_INT_INTREQ_BKUP_TRG_INT),
+};
+
+static struct regmap_irq_chip bd9571mwv_irq_chip = {
+ .name = "bd9571mwv",
+ .status_base = BD9571MWV_INT_INTREQ,
+ .mask_base = BD9571MWV_INT_INTMASK,
+ .ack_base = BD9571MWV_INT_INTREQ,
+ .init_ack_masked = true,
+ .num_regs = 1,
+ .irqs = bd9571mwv_irqs,
+ .num_irqs = ARRAY_SIZE(bd9571mwv_irqs),
+};
+
+static int bd9571mwv_identify(struct bd9571mwv *bd)
+{
+ struct device *dev = bd->dev;
+ unsigned int value;
+ int ret;
+
+ ret = regmap_read(bd->regmap, BD9571MWV_VENDOR_CODE, &value);
+ if (ret) {
+ dev_err(dev, "Failed to read vendor code register (ret=%i)\n",
+ ret);
+ return ret;
+ }
+
+ if (value != BD9571MWV_VENDOR_CODE_VAL) {
+ dev_err(dev, "Invalid vendor code ID %02x (expected %02x)\n",
+ value, BD9571MWV_VENDOR_CODE_VAL);
+ return -EINVAL;
+ }
+
+ ret = regmap_read(bd->regmap, BD9571MWV_PRODUCT_CODE, &value);
+ if (ret) {
+ dev_err(dev, "Failed to read product code register (ret=%i)\n",
+ ret);
+ return ret;
+ }
+
+ if (value != BD9571MWV_PRODUCT_CODE_VAL) {
+ dev_err(dev, "Invalid product code ID %02x (expected %02x)\n",
+ value, BD9571MWV_PRODUCT_CODE_VAL);
+ return -EINVAL;
+ }
+
+ ret = regmap_read(bd->regmap, BD9571MWV_PRODUCT_REVISION, &value);
+ if (ret) {
+ dev_err(dev, "Failed to read revision register (ret=%i)\n",
+ ret);
+ return ret;
+ }
+
+ dev_info(dev, "Device: BD9571MWV rev. %d\n", value & 0xff);
+
+ return 0;
+}
+
+static int bd9571mwv_probe(struct i2c_client *client,
+ const struct i2c_device_id *ids)
+{
+ struct bd9571mwv *bd;
+ int ret;
+
+ bd = devm_kzalloc(&client->dev, sizeof(*bd), GFP_KERNEL);
+ if (!bd)
+ return -ENOMEM;
+
+ i2c_set_clientdata(client, bd);
+ bd->dev = &client->dev;
+ bd->irq = client->irq;
+
+ bd->regmap = devm_regmap_init_i2c(client, &bd9571mwv_regmap_config);
+ if (IS_ERR(bd->regmap)) {
+ dev_err(bd->dev, "Failed to initialize register map\n");
+ return PTR_ERR(bd->regmap);
+ }
+
+ ret = bd9571mwv_identify(bd);
+ if (ret)
+ return ret;
+
+ ret = regmap_add_irq_chip(bd->regmap, bd->irq, IRQF_ONESHOT, 0,
+ &bd9571mwv_irq_chip, &bd->irq_data);
+ if (ret) {
+ dev_err(bd->dev, "Failed to register IRQ chip\n");
+ return ret;
+ }
+
+ ret = mfd_add_devices(bd->dev, PLATFORM_DEVID_AUTO, bd9571mwv_cells,
+ ARRAY_SIZE(bd9571mwv_cells), NULL, 0,
+ regmap_irq_get_domain(bd->irq_data));
+ if (ret) {
+ regmap_del_irq_chip(bd->irq, bd->irq_data);
+ return ret;
+ }
+
+ return 0;
+}
+
+static int bd9571mwv_remove(struct i2c_client *client)
+{
+ struct bd9571mwv *bd = i2c_get_clientdata(client);
+
+ regmap_del_irq_chip(bd->irq, bd->irq_data);
+
+ return 0;
+}
+
+static const struct of_device_id bd9571mwv_of_match_table[] = {
+ { .compatible = "rohm,bd9571mwv", },
+ { /* sentinel */ }
+};
+MODULE_DEVICE_TABLE(of, bd9571mwv_of_match_table);
+
+static const struct i2c_device_id bd9571mwv_id_table[] = {
+ { "bd9571mwv", 0 },
+ { /* sentinel */ }
+};
+MODULE_DEVICE_TABLE(i2c, bd9571mwv_id_table);
+
+static struct i2c_driver bd9571mwv_driver = {
+ .driver = {
+ .name = "bd9571mwv",
+ .of_match_table = bd9571mwv_of_match_table,
+ },
+ .probe = bd9571mwv_probe,
+ .remove = bd9571mwv_remove,
+ .id_table = bd9571mwv_id_table,
+};
+module_i2c_driver(bd9571mwv_driver);
+
+MODULE_AUTHOR("Marek Vasut <marek.vasut+renesas@gmail.com>");
+MODULE_DESCRIPTION("BD9571MWV PMIC Driver");
+MODULE_LICENSE("GPL v2");
diff --git a/include/linux/mfd/bd9571mwv.h b/include/linux/mfd/bd9571mwv.h
new file mode 100644
index 000000000000..f0708ba4cbba
--- /dev/null
+++ b/include/linux/mfd/bd9571mwv.h
@@ -0,0 +1,115 @@
+/*
+ * ROHM BD9571MWV-M driver
+ *
+ * Copyright (C) 2017 Marek Vasut <marek.vasut+renesas@gmail.com>
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License version 2 as
+ * published by the Free Software Foundation.
+ *
+ * This program is distributed "as is" WITHOUT ANY WARRANTY of any
+ * kind, whether expressed or implied; without even the implied warranty
+ * of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License version 2 for more details.
+ *
+ * Based on the TPS65086 driver
+ */
+
+#ifndef __LINUX_MFD_BD9571MWV_H
+#define __LINUX_MFD_BD9571MWV_H
+
+#include <linux/device.h>
+#include <linux/regmap.h>
+
+/* List of registers for BD9571MWV */
+#define BD9571MWV_VENDOR_CODE 0x00
+#define BD9571MWV_VENDOR_CODE_VAL 0xdb
+#define BD9571MWV_PRODUCT_CODE 0x01
+#define BD9571MWV_PRODUCT_CODE_VAL 0x60
+#define BD9571MWV_PRODUCT_REVISION 0x02
+
+#define BD9571MWV_I2C_FUSA_MODE 0x10
+#define BD9571MWV_I2C_MD2_E1_BIT_1 0x11
+#define BD9571MWV_I2C_MD2_E1_BIT_2 0x12
+
+#define BD9571MWV_BKUP_MODE_CNT 0x20
+#define BD9571MWV_BKUP_MODE_STATUS 0x21
+#define BD9571MWV_BKUP_RECOVERY_CNT 0x22
+#define BD9571MWV_BKUP_CTRL_TIM_CNT 0x23
+#define BD9571MWV_WAITBKUP_WDT_CNT 0x24
+#define BD9571MWV_128H_TIM_CNT 0x26
+#define BD9571MWV_QLLM_CNT 0x27
+
+#define BD9571MWV_AVS_SET_MONI 0x31
+#define BD9571MWV_AVS_SET_MONI_MASK 0x3
+#define BD9571MWV_AVS_VD09_VID(n) (0x32 + (n))
+#define BD9571MWV_AVS_DVFS_VID(n) (0x36 + (n))
+
+#define BD9571MWV_VD18_VID 0x42
+#define BD9571MWV_VD25_VID 0x43
+#define BD9571MWV_VD33_VID 0x44
+
+#define BD9571MWV_DVFS_VINIT 0x50
+#define BD9571MWV_DVFS_SETVMAX 0x52
+#define BD9571MWV_DVFS_BOOSTVID 0x53
+#define BD9571MWV_DVFS_SETVID 0x54
+#define BD9571MWV_DVFS_MONIVDAC 0x55
+#define BD9571MWV_DVFS_PGD_CNT 0x56
+
+#define BD9571MWV_GPIO_DIR 0x60
+#define BD9571MWV_GPIO_OUT 0x61
+#define BD9571MWV_GPIO_IN 0x62
+#define BD9571MWV_GPIO_DEB 0x63
+#define BD9571MWV_GPIO_INT_SET 0x64
+#define BD9571MWV_GPIO_INT 0x65
+#define BD9571MWV_GPIO_INTMASK 0x66
+
+#define BD9571MWV_REG_KEEP(n) (0x70 + (n))
+
+#define BD9571MWV_PMIC_INTERNAL_STATUS 0x80
+#define BD9571MWV_PROT_ERROR_STATUS0 0x81
+#define BD9571MWV_PROT_ERROR_STATUS1 0x82
+#define BD9571MWV_PROT_ERROR_STATUS2 0x83
+#define BD9571MWV_PROT_ERROR_STATUS3 0x84
+#define BD9571MWV_PROT_ERROR_STATUS4 0x85
+
+#define BD9571MWV_INT_INTREQ 0x90
+#define BD9571MWV_INT_INTREQ_MD1_INT BIT(0)
+#define BD9571MWV_INT_INTREQ_MD2_E1_INT BIT(1)
+#define BD9571MWV_INT_INTREQ_MD2_E2_INT BIT(2)
+#define BD9571MWV_INT_INTREQ_PROT_ERR_INT BIT(3)
+#define BD9571MWV_INT_INTREQ_GP_INT BIT(4)
+#define BD9571MWV_INT_INTREQ_128H_OF_INT BIT(5)
+#define BD9571MWV_INT_INTREQ_WDT_OF_INT BIT(6)
+#define BD9571MWV_INT_INTREQ_BKUP_TRG_INT BIT(7)
+#define BD9571MWV_INT_INTMASK 0x91
+
+#define BD9571MWV_ACCESS_KEY 0xff
+
+/* Define the BD9571MWV IRQ numbers */
+enum bd9571mwv_irqs {
+ BD9571MWV_IRQ_MD1,
+ BD9571MWV_IRQ_MD2_E1,
+ BD9571MWV_IRQ_MD2_E2,
+ BD9571MWV_IRQ_PROT_ERR,
+ BD9571MWV_IRQ_GP,
+ BD9571MWV_IRQ_128H_OF,
+ BD9571MWV_IRQ_WDT_OF,
+ BD9571MWV_IRQ_BKUP_TRG,
+};
+
+/**
+ * struct bd9571mwv - state holder for the bd9571mwv driver
+ *
+ * Device data may be used to access the BD9571MWV chip
+ */
+struct bd9571mwv {
+ struct device *dev;
+ struct regmap *regmap;
+
+ /* IRQ Data */
+ int irq;
+ struct regmap_irq_chip_data *irq_data;
+};
+
+#endif /* __LINUX_MFD_BD9571MWV_H */
--
2.11.0
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH V3 2/2] mfd: Add ROHM BD9571MWV-M MFD PMIC driver
2017-05-02 12:18 ` [PATCH V3 2/2] mfd: Add ROHM BD9571MWV-M MFD PMIC driver Marek Vasut
@ 2017-06-27 9:54 ` Marek Vasut
2017-07-03 11:55 ` Lee Jones
2017-07-10 15:33 ` Geert Uytterhoeven
1 sibling, 1 reply; 9+ messages in thread
From: Marek Vasut @ 2017-06-27 9:54 UTC (permalink / raw)
To: linux-renesas-soc
Cc: lee.jones, Marek Vasut, linux-kernel, Geert Uytterhoeven
On 05/02/2017 02:18 PM, Marek Vasut wrote:
> Add the MFD part of the ROHM BD9571MWV-M PMIC driver and MAINTAINERS
> entry. The MFD part only specifies the regmap bits for the PMIC and
> binds the subdevs together.
>
> Signed-off-by: Marek Vasut <marek.vasut+renesas@gmail.com>
> Cc: linux-kernel@vger.kernel.org
> Cc: Geert Uytterhoeven <geert+renesas@glider.be>
> Cc: Lee Jones <lee.jones@linaro.org>
Lee, bump, do you plan to apply these patches anytime soon ?
> ---
> V2: - Change BD9571MWV_AVS_VD09_VID0,1,2,3 to BD9571MWV_AVS_VD09_VID(n)
> - Change BD9571MWV_AVS_DVFS_VID0,1,2,3 to BD9571MWV_AVS_DVFS_VID(n)
> - Make the AVS_VD09 range RW, so it can be used by the regulator
> driver for the VD09 regulator
> - Report the regmap read return values when attempting to read ID
> registers fails
> V3: No change
> ---
> MAINTAINERS | 11 ++
> drivers/mfd/Kconfig | 13 +++
> drivers/mfd/Makefile | 1 +
> drivers/mfd/bd9571mwv.c | 230 ++++++++++++++++++++++++++++++++++++++++++
> include/linux/mfd/bd9571mwv.h | 115 +++++++++++++++++++++
> 5 files changed, 370 insertions(+)
> create mode 100644 drivers/mfd/bd9571mwv.c
> create mode 100644 include/linux/mfd/bd9571mwv.h
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 2a9aa4be5846..9c30e6b00358 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -10942,6 +10942,17 @@ L: linux-serial@vger.kernel.org
> S: Odd Fixes
> F: drivers/tty/serial/rp2.*
>
> +ROHM MULTIFUNCTION BD9571MWV-M PMIC DEVICE DRIVERS
> +M: Marek Vasut <marek.vasut+renesas@gmail.com>
> +L: linux-kernel@vger.kernel.org
> +L: linux-renesas-soc@vger.kernel.org
> +S: Supported
> +F: drivers/mfd/bd9571mwv.c
> +F: drivers/regulator/bd9571mwv-regulator.c
> +F: drivers/gpio/gpio-bd9571mwv.c
> +F: include/linux/mfd/bd9571mwv.h
> +F: Documentation/devicetree/bindings/mfd/bd9571mwv.txt
> +
> ROSE NETWORK LAYER
> M: Ralf Baechle <ralf@linux-mips.org>
> L: linux-hams@vger.kernel.org
> diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig
> index 3eb5c93595f6..6f4668ae28cc 100644
> --- a/drivers/mfd/Kconfig
> +++ b/drivers/mfd/Kconfig
> @@ -133,6 +133,19 @@ config MFD_BCM590XX
> help
> Support for the BCM590xx PMUs from Broadcom
>
> +config MFD_BD9571MWV
> + tristate "ROHM BD9571MWV PMIC"
> + select MFD_CORE
> + select REGMAP_I2C
> + depends on I2C
> + help
> + Support for the ROHM BD9571MWV PMIC, which contains single
> + voltage regulator, voltage sampling units, GPIO block and
> + watchdog block.
> +
> + This driver can also be built as a module. If so, the module
> + will be called bd9571mwv.
> +
> config MFD_AC100
> tristate "X-Powers AC100"
> select MFD_CORE
> diff --git a/drivers/mfd/Makefile b/drivers/mfd/Makefile
> index c16bf1ea0ea9..3cacc9748e13 100644
> --- a/drivers/mfd/Makefile
> +++ b/drivers/mfd/Makefile
> @@ -10,6 +10,7 @@ obj-$(CONFIG_MFD_ACT8945A) += act8945a.o
> obj-$(CONFIG_MFD_SM501) += sm501.o
> obj-$(CONFIG_MFD_ASIC3) += asic3.o tmio_core.o
> obj-$(CONFIG_MFD_BCM590XX) += bcm590xx.o
> +obj-$(CONFIG_MFD_BD9571MWV) += bd9571mwv.o
> cros_ec_core-objs := cros_ec.o
> cros_ec_core-$(CONFIG_ACPI) += cros_ec_acpi_gpe.o
> obj-$(CONFIG_MFD_CROS_EC) += cros_ec_core.o
> diff --git a/drivers/mfd/bd9571mwv.c b/drivers/mfd/bd9571mwv.c
> new file mode 100644
> index 000000000000..64e088dfe7b0
> --- /dev/null
> +++ b/drivers/mfd/bd9571mwv.c
> @@ -0,0 +1,230 @@
> +/*
> + * ROHM BD9571MWV-M MFD driver
> + *
> + * Copyright (C) 2017 Marek Vasut <marek.vasut+renesas@gmail.com>
> + *
> + * This program is free software; you can redistribute it and/or
> + * modify it under the terms of the GNU General Public License version 2 as
> + * published by the Free Software Foundation.
> + *
> + * This program is distributed "as is" WITHOUT ANY WARRANTY of any
> + * kind, whether expressed or implied; without even the implied warranty
> + * of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> + * GNU General Public License version 2 for more details.
> + *
> + * Based on the TPS65086 driver
> + */
> +
> +#include <linux/i2c.h>
> +#include <linux/interrupt.h>
> +#include <linux/mfd/core.h>
> +#include <linux/module.h>
> +
> +#include <linux/mfd/bd9571mwv.h>
> +
> +static const struct mfd_cell bd9571mwv_cells[] = {
> + { .name = "bd9571mwv-regulator", },
> + { .name = "bd9571mwv-gpio", },
> +};
> +
> +static const struct regmap_range bd9571mwv_readable_yes_ranges[] = {
> + regmap_reg_range(BD9571MWV_VENDOR_CODE, BD9571MWV_PRODUCT_REVISION),
> + regmap_reg_range(BD9571MWV_AVS_SET_MONI, BD9571MWV_AVS_DVFS_VID(3)),
> + regmap_reg_range(BD9571MWV_VD18_VID, BD9571MWV_VD33_VID),
> + regmap_reg_range(BD9571MWV_DVFS_VINIT, BD9571MWV_DVFS_VINIT),
> + regmap_reg_range(BD9571MWV_DVFS_SETVMAX, BD9571MWV_DVFS_MONIVDAC),
> + regmap_reg_range(BD9571MWV_GPIO_IN, BD9571MWV_GPIO_IN),
> + regmap_reg_range(BD9571MWV_GPIO_INT, BD9571MWV_GPIO_INTMASK),
> + regmap_reg_range(BD9571MWV_INT_INTREQ, BD9571MWV_INT_INTMASK),
> +};
> +
> +static const struct regmap_access_table bd9571mwv_readable_table = {
> + .yes_ranges = bd9571mwv_readable_yes_ranges,
> + .n_yes_ranges = ARRAY_SIZE(bd9571mwv_readable_yes_ranges),
> +};
> +
> +static const struct regmap_range bd9571mwv_writable_yes_ranges[] = {
> + regmap_reg_range(BD9571MWV_AVS_VD09_VID(0), BD9571MWV_AVS_VD09_VID(3)),
> + regmap_reg_range(BD9571MWV_DVFS_SETVID, BD9571MWV_DVFS_SETVID),
> + regmap_reg_range(BD9571MWV_GPIO_DIR, BD9571MWV_GPIO_OUT),
> + regmap_reg_range(BD9571MWV_GPIO_INT_SET, BD9571MWV_GPIO_INTMASK),
> + regmap_reg_range(BD9571MWV_INT_INTREQ, BD9571MWV_INT_INTMASK),
> +};
> +
> +static const struct regmap_access_table bd9571mwv_writable_table = {
> + .yes_ranges = bd9571mwv_writable_yes_ranges,
> + .n_yes_ranges = ARRAY_SIZE(bd9571mwv_writable_yes_ranges),
> +};
> +
> +static const struct regmap_range bd9571mwv_volatile_yes_ranges[] = {
> + regmap_reg_range(BD9571MWV_GPIO_IN, BD9571MWV_GPIO_IN),
> + regmap_reg_range(BD9571MWV_GPIO_INT, BD9571MWV_GPIO_INT),
> + regmap_reg_range(BD9571MWV_INT_INTREQ, BD9571MWV_INT_INTREQ),
> +};
> +
> +static const struct regmap_access_table bd9571mwv_volatile_table = {
> + .yes_ranges = bd9571mwv_volatile_yes_ranges,
> + .n_yes_ranges = ARRAY_SIZE(bd9571mwv_volatile_yes_ranges),
> +};
> +
> +static const struct regmap_config bd9571mwv_regmap_config = {
> + .reg_bits = 8,
> + .val_bits = 8,
> + .cache_type = REGCACHE_RBTREE,
> + .rd_table = &bd9571mwv_readable_table,
> + .wr_table = &bd9571mwv_writable_table,
> + .volatile_table = &bd9571mwv_volatile_table,
> + .max_register = 0xff,
> +};
> +
> +static const struct regmap_irq bd9571mwv_irqs[] = {
> + REGMAP_IRQ_REG(BD9571MWV_IRQ_MD1, 0,
> + BD9571MWV_INT_INTREQ_MD1_INT),
> + REGMAP_IRQ_REG(BD9571MWV_IRQ_MD2_E1, 0,
> + BD9571MWV_INT_INTREQ_MD2_E1_INT),
> + REGMAP_IRQ_REG(BD9571MWV_IRQ_MD2_E2, 0,
> + BD9571MWV_INT_INTREQ_MD2_E2_INT),
> + REGMAP_IRQ_REG(BD9571MWV_IRQ_PROT_ERR, 0,
> + BD9571MWV_INT_INTREQ_PROT_ERR_INT),
> + REGMAP_IRQ_REG(BD9571MWV_IRQ_GP, 0,
> + BD9571MWV_INT_INTREQ_GP_INT),
> + REGMAP_IRQ_REG(BD9571MWV_IRQ_128H_OF, 0,
> + BD9571MWV_INT_INTREQ_128H_OF_INT),
> + REGMAP_IRQ_REG(BD9571MWV_IRQ_WDT_OF, 0,
> + BD9571MWV_INT_INTREQ_WDT_OF_INT),
> + REGMAP_IRQ_REG(BD9571MWV_IRQ_BKUP_TRG, 0,
> + BD9571MWV_INT_INTREQ_BKUP_TRG_INT),
> +};
> +
> +static struct regmap_irq_chip bd9571mwv_irq_chip = {
> + .name = "bd9571mwv",
> + .status_base = BD9571MWV_INT_INTREQ,
> + .mask_base = BD9571MWV_INT_INTMASK,
> + .ack_base = BD9571MWV_INT_INTREQ,
> + .init_ack_masked = true,
> + .num_regs = 1,
> + .irqs = bd9571mwv_irqs,
> + .num_irqs = ARRAY_SIZE(bd9571mwv_irqs),
> +};
> +
> +static int bd9571mwv_identify(struct bd9571mwv *bd)
> +{
> + struct device *dev = bd->dev;
> + unsigned int value;
> + int ret;
> +
> + ret = regmap_read(bd->regmap, BD9571MWV_VENDOR_CODE, &value);
> + if (ret) {
> + dev_err(dev, "Failed to read vendor code register (ret=%i)\n",
> + ret);
> + return ret;
> + }
> +
> + if (value != BD9571MWV_VENDOR_CODE_VAL) {
> + dev_err(dev, "Invalid vendor code ID %02x (expected %02x)\n",
> + value, BD9571MWV_VENDOR_CODE_VAL);
> + return -EINVAL;
> + }
> +
> + ret = regmap_read(bd->regmap, BD9571MWV_PRODUCT_CODE, &value);
> + if (ret) {
> + dev_err(dev, "Failed to read product code register (ret=%i)\n",
> + ret);
> + return ret;
> + }
> +
> + if (value != BD9571MWV_PRODUCT_CODE_VAL) {
> + dev_err(dev, "Invalid product code ID %02x (expected %02x)\n",
> + value, BD9571MWV_PRODUCT_CODE_VAL);
> + return -EINVAL;
> + }
> +
> + ret = regmap_read(bd->regmap, BD9571MWV_PRODUCT_REVISION, &value);
> + if (ret) {
> + dev_err(dev, "Failed to read revision register (ret=%i)\n",
> + ret);
> + return ret;
> + }
> +
> + dev_info(dev, "Device: BD9571MWV rev. %d\n", value & 0xff);
> +
> + return 0;
> +}
> +
> +static int bd9571mwv_probe(struct i2c_client *client,
> + const struct i2c_device_id *ids)
> +{
> + struct bd9571mwv *bd;
> + int ret;
> +
> + bd = devm_kzalloc(&client->dev, sizeof(*bd), GFP_KERNEL);
> + if (!bd)
> + return -ENOMEM;
> +
> + i2c_set_clientdata(client, bd);
> + bd->dev = &client->dev;
> + bd->irq = client->irq;
> +
> + bd->regmap = devm_regmap_init_i2c(client, &bd9571mwv_regmap_config);
> + if (IS_ERR(bd->regmap)) {
> + dev_err(bd->dev, "Failed to initialize register map\n");
> + return PTR_ERR(bd->regmap);
> + }
> +
> + ret = bd9571mwv_identify(bd);
> + if (ret)
> + return ret;
> +
> + ret = regmap_add_irq_chip(bd->regmap, bd->irq, IRQF_ONESHOT, 0,
> + &bd9571mwv_irq_chip, &bd->irq_data);
> + if (ret) {
> + dev_err(bd->dev, "Failed to register IRQ chip\n");
> + return ret;
> + }
> +
> + ret = mfd_add_devices(bd->dev, PLATFORM_DEVID_AUTO, bd9571mwv_cells,
> + ARRAY_SIZE(bd9571mwv_cells), NULL, 0,
> + regmap_irq_get_domain(bd->irq_data));
> + if (ret) {
> + regmap_del_irq_chip(bd->irq, bd->irq_data);
> + return ret;
> + }
> +
> + return 0;
> +}
> +
> +static int bd9571mwv_remove(struct i2c_client *client)
> +{
> + struct bd9571mwv *bd = i2c_get_clientdata(client);
> +
> + regmap_del_irq_chip(bd->irq, bd->irq_data);
> +
> + return 0;
> +}
> +
> +static const struct of_device_id bd9571mwv_of_match_table[] = {
> + { .compatible = "rohm,bd9571mwv", },
> + { /* sentinel */ }
> +};
> +MODULE_DEVICE_TABLE(of, bd9571mwv_of_match_table);
> +
> +static const struct i2c_device_id bd9571mwv_id_table[] = {
> + { "bd9571mwv", 0 },
> + { /* sentinel */ }
> +};
> +MODULE_DEVICE_TABLE(i2c, bd9571mwv_id_table);
> +
> +static struct i2c_driver bd9571mwv_driver = {
> + .driver = {
> + .name = "bd9571mwv",
> + .of_match_table = bd9571mwv_of_match_table,
> + },
> + .probe = bd9571mwv_probe,
> + .remove = bd9571mwv_remove,
> + .id_table = bd9571mwv_id_table,
> +};
> +module_i2c_driver(bd9571mwv_driver);
> +
> +MODULE_AUTHOR("Marek Vasut <marek.vasut+renesas@gmail.com>");
> +MODULE_DESCRIPTION("BD9571MWV PMIC Driver");
> +MODULE_LICENSE("GPL v2");
> diff --git a/include/linux/mfd/bd9571mwv.h b/include/linux/mfd/bd9571mwv.h
> new file mode 100644
> index 000000000000..f0708ba4cbba
> --- /dev/null
> +++ b/include/linux/mfd/bd9571mwv.h
> @@ -0,0 +1,115 @@
> +/*
> + * ROHM BD9571MWV-M driver
> + *
> + * Copyright (C) 2017 Marek Vasut <marek.vasut+renesas@gmail.com>
> + *
> + * This program is free software; you can redistribute it and/or
> + * modify it under the terms of the GNU General Public License version 2 as
> + * published by the Free Software Foundation.
> + *
> + * This program is distributed "as is" WITHOUT ANY WARRANTY of any
> + * kind, whether expressed or implied; without even the implied warranty
> + * of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> + * GNU General Public License version 2 for more details.
> + *
> + * Based on the TPS65086 driver
> + */
> +
> +#ifndef __LINUX_MFD_BD9571MWV_H
> +#define __LINUX_MFD_BD9571MWV_H
> +
> +#include <linux/device.h>
> +#include <linux/regmap.h>
> +
> +/* List of registers for BD9571MWV */
> +#define BD9571MWV_VENDOR_CODE 0x00
> +#define BD9571MWV_VENDOR_CODE_VAL 0xdb
> +#define BD9571MWV_PRODUCT_CODE 0x01
> +#define BD9571MWV_PRODUCT_CODE_VAL 0x60
> +#define BD9571MWV_PRODUCT_REVISION 0x02
> +
> +#define BD9571MWV_I2C_FUSA_MODE 0x10
> +#define BD9571MWV_I2C_MD2_E1_BIT_1 0x11
> +#define BD9571MWV_I2C_MD2_E1_BIT_2 0x12
> +
> +#define BD9571MWV_BKUP_MODE_CNT 0x20
> +#define BD9571MWV_BKUP_MODE_STATUS 0x21
> +#define BD9571MWV_BKUP_RECOVERY_CNT 0x22
> +#define BD9571MWV_BKUP_CTRL_TIM_CNT 0x23
> +#define BD9571MWV_WAITBKUP_WDT_CNT 0x24
> +#define BD9571MWV_128H_TIM_CNT 0x26
> +#define BD9571MWV_QLLM_CNT 0x27
> +
> +#define BD9571MWV_AVS_SET_MONI 0x31
> +#define BD9571MWV_AVS_SET_MONI_MASK 0x3
> +#define BD9571MWV_AVS_VD09_VID(n) (0x32 + (n))
> +#define BD9571MWV_AVS_DVFS_VID(n) (0x36 + (n))
> +
> +#define BD9571MWV_VD18_VID 0x42
> +#define BD9571MWV_VD25_VID 0x43
> +#define BD9571MWV_VD33_VID 0x44
> +
> +#define BD9571MWV_DVFS_VINIT 0x50
> +#define BD9571MWV_DVFS_SETVMAX 0x52
> +#define BD9571MWV_DVFS_BOOSTVID 0x53
> +#define BD9571MWV_DVFS_SETVID 0x54
> +#define BD9571MWV_DVFS_MONIVDAC 0x55
> +#define BD9571MWV_DVFS_PGD_CNT 0x56
> +
> +#define BD9571MWV_GPIO_DIR 0x60
> +#define BD9571MWV_GPIO_OUT 0x61
> +#define BD9571MWV_GPIO_IN 0x62
> +#define BD9571MWV_GPIO_DEB 0x63
> +#define BD9571MWV_GPIO_INT_SET 0x64
> +#define BD9571MWV_GPIO_INT 0x65
> +#define BD9571MWV_GPIO_INTMASK 0x66
> +
> +#define BD9571MWV_REG_KEEP(n) (0x70 + (n))
> +
> +#define BD9571MWV_PMIC_INTERNAL_STATUS 0x80
> +#define BD9571MWV_PROT_ERROR_STATUS0 0x81
> +#define BD9571MWV_PROT_ERROR_STATUS1 0x82
> +#define BD9571MWV_PROT_ERROR_STATUS2 0x83
> +#define BD9571MWV_PROT_ERROR_STATUS3 0x84
> +#define BD9571MWV_PROT_ERROR_STATUS4 0x85
> +
> +#define BD9571MWV_INT_INTREQ 0x90
> +#define BD9571MWV_INT_INTREQ_MD1_INT BIT(0)
> +#define BD9571MWV_INT_INTREQ_MD2_E1_INT BIT(1)
> +#define BD9571MWV_INT_INTREQ_MD2_E2_INT BIT(2)
> +#define BD9571MWV_INT_INTREQ_PROT_ERR_INT BIT(3)
> +#define BD9571MWV_INT_INTREQ_GP_INT BIT(4)
> +#define BD9571MWV_INT_INTREQ_128H_OF_INT BIT(5)
> +#define BD9571MWV_INT_INTREQ_WDT_OF_INT BIT(6)
> +#define BD9571MWV_INT_INTREQ_BKUP_TRG_INT BIT(7)
> +#define BD9571MWV_INT_INTMASK 0x91
> +
> +#define BD9571MWV_ACCESS_KEY 0xff
> +
> +/* Define the BD9571MWV IRQ numbers */
> +enum bd9571mwv_irqs {
> + BD9571MWV_IRQ_MD1,
> + BD9571MWV_IRQ_MD2_E1,
> + BD9571MWV_IRQ_MD2_E2,
> + BD9571MWV_IRQ_PROT_ERR,
> + BD9571MWV_IRQ_GP,
> + BD9571MWV_IRQ_128H_OF,
> + BD9571MWV_IRQ_WDT_OF,
> + BD9571MWV_IRQ_BKUP_TRG,
> +};
> +
> +/**
> + * struct bd9571mwv - state holder for the bd9571mwv driver
> + *
> + * Device data may be used to access the BD9571MWV chip
> + */
> +struct bd9571mwv {
> + struct device *dev;
> + struct regmap *regmap;
> +
> + /* IRQ Data */
> + int irq;
> + struct regmap_irq_chip_data *irq_data;
> +};
> +
> +#endif /* __LINUX_MFD_BD9571MWV_H */
>
--
Best regards,
Marek Vasut
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH V3 2/2] mfd: Add ROHM BD9571MWV-M MFD PMIC driver
2017-06-27 9:54 ` Marek Vasut
@ 2017-07-03 11:55 ` Lee Jones
2017-07-03 12:03 ` Marek Vasut
0 siblings, 1 reply; 9+ messages in thread
From: Lee Jones @ 2017-07-03 11:55 UTC (permalink / raw)
To: Marek Vasut
Cc: linux-renesas-soc, Marek Vasut, linux-kernel, Geert Uytterhoeven
On Tue, 27 Jun 2017, Marek Vasut wrote:
> On 05/02/2017 02:18 PM, Marek Vasut wrote:
> > Add the MFD part of the ROHM BD9571MWV-M PMIC driver and MAINTAINERS
> > entry. The MFD part only specifies the regmap bits for the PMIC and
> > binds the subdevs together.
> >
> > Signed-off-by: Marek Vasut <marek.vasut+renesas@gmail.com>
> > Cc: linux-kernel@vger.kernel.org
> > Cc: Geert Uytterhoeven <geert+renesas@glider.be>
> > Cc: Lee Jones <lee.jones@linaro.org>
>
> Lee, bump, do you plan to apply these patches anytime soon ?
Looks like these were missed for some reason.
I'll put them back on the pile for review.
Please note that the merge window is currently open, so you are likely
to experience a period of inactivity until -rc1 is released.
> > ---
> > V2: - Change BD9571MWV_AVS_VD09_VID0,1,2,3 to BD9571MWV_AVS_VD09_VID(n)
> > - Change BD9571MWV_AVS_DVFS_VID0,1,2,3 to BD9571MWV_AVS_DVFS_VID(n)
> > - Make the AVS_VD09 range RW, so it can be used by the regulator
> > driver for the VD09 regulator
> > - Report the regmap read return values when attempting to read ID
> > registers fails
> > V3: No change
> > ---
> > MAINTAINERS | 11 ++
> > drivers/mfd/Kconfig | 13 +++
> > drivers/mfd/Makefile | 1 +
> > drivers/mfd/bd9571mwv.c | 230 ++++++++++++++++++++++++++++++++++++++++++
> > include/linux/mfd/bd9571mwv.h | 115 +++++++++++++++++++++
> > 5 files changed, 370 insertions(+)
> > create mode 100644 drivers/mfd/bd9571mwv.c
> > create mode 100644 include/linux/mfd/bd9571mwv.h
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 2a9aa4be5846..9c30e6b00358 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -10942,6 +10942,17 @@ L: linux-serial@vger.kernel.org
> > S: Odd Fixes
> > F: drivers/tty/serial/rp2.*
> >
> > +ROHM MULTIFUNCTION BD9571MWV-M PMIC DEVICE DRIVERS
> > +M: Marek Vasut <marek.vasut+renesas@gmail.com>
> > +L: linux-kernel@vger.kernel.org
> > +L: linux-renesas-soc@vger.kernel.org
> > +S: Supported
> > +F: drivers/mfd/bd9571mwv.c
> > +F: drivers/regulator/bd9571mwv-regulator.c
> > +F: drivers/gpio/gpio-bd9571mwv.c
> > +F: include/linux/mfd/bd9571mwv.h
> > +F: Documentation/devicetree/bindings/mfd/bd9571mwv.txt
> > +
> > ROSE NETWORK LAYER
> > M: Ralf Baechle <ralf@linux-mips.org>
> > L: linux-hams@vger.kernel.org
> > diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig
> > index 3eb5c93595f6..6f4668ae28cc 100644
> > --- a/drivers/mfd/Kconfig
> > +++ b/drivers/mfd/Kconfig
> > @@ -133,6 +133,19 @@ config MFD_BCM590XX
> > help
> > Support for the BCM590xx PMUs from Broadcom
> >
> > +config MFD_BD9571MWV
> > + tristate "ROHM BD9571MWV PMIC"
> > + select MFD_CORE
> > + select REGMAP_I2C
> > + depends on I2C
> > + help
> > + Support for the ROHM BD9571MWV PMIC, which contains single
> > + voltage regulator, voltage sampling units, GPIO block and
> > + watchdog block.
> > +
> > + This driver can also be built as a module. If so, the module
> > + will be called bd9571mwv.
> > +
> > config MFD_AC100
> > tristate "X-Powers AC100"
> > select MFD_CORE
> > diff --git a/drivers/mfd/Makefile b/drivers/mfd/Makefile
> > index c16bf1ea0ea9..3cacc9748e13 100644
> > --- a/drivers/mfd/Makefile
> > +++ b/drivers/mfd/Makefile
> > @@ -10,6 +10,7 @@ obj-$(CONFIG_MFD_ACT8945A) += act8945a.o
> > obj-$(CONFIG_MFD_SM501) += sm501.o
> > obj-$(CONFIG_MFD_ASIC3) += asic3.o tmio_core.o
> > obj-$(CONFIG_MFD_BCM590XX) += bcm590xx.o
> > +obj-$(CONFIG_MFD_BD9571MWV) += bd9571mwv.o
> > cros_ec_core-objs := cros_ec.o
> > cros_ec_core-$(CONFIG_ACPI) += cros_ec_acpi_gpe.o
> > obj-$(CONFIG_MFD_CROS_EC) += cros_ec_core.o
> > diff --git a/drivers/mfd/bd9571mwv.c b/drivers/mfd/bd9571mwv.c
> > new file mode 100644
> > index 000000000000..64e088dfe7b0
> > --- /dev/null
> > +++ b/drivers/mfd/bd9571mwv.c
> > @@ -0,0 +1,230 @@
> > +/*
> > + * ROHM BD9571MWV-M MFD driver
> > + *
> > + * Copyright (C) 2017 Marek Vasut <marek.vasut+renesas@gmail.com>
> > + *
> > + * This program is free software; you can redistribute it and/or
> > + * modify it under the terms of the GNU General Public License version 2 as
> > + * published by the Free Software Foundation.
> > + *
> > + * This program is distributed "as is" WITHOUT ANY WARRANTY of any
> > + * kind, whether expressed or implied; without even the implied warranty
> > + * of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> > + * GNU General Public License version 2 for more details.
> > + *
> > + * Based on the TPS65086 driver
> > + */
> > +
> > +#include <linux/i2c.h>
> > +#include <linux/interrupt.h>
> > +#include <linux/mfd/core.h>
> > +#include <linux/module.h>
> > +
> > +#include <linux/mfd/bd9571mwv.h>
> > +
> > +static const struct mfd_cell bd9571mwv_cells[] = {
> > + { .name = "bd9571mwv-regulator", },
> > + { .name = "bd9571mwv-gpio", },
> > +};
> > +
> > +static const struct regmap_range bd9571mwv_readable_yes_ranges[] = {
> > + regmap_reg_range(BD9571MWV_VENDOR_CODE, BD9571MWV_PRODUCT_REVISION),
> > + regmap_reg_range(BD9571MWV_AVS_SET_MONI, BD9571MWV_AVS_DVFS_VID(3)),
> > + regmap_reg_range(BD9571MWV_VD18_VID, BD9571MWV_VD33_VID),
> > + regmap_reg_range(BD9571MWV_DVFS_VINIT, BD9571MWV_DVFS_VINIT),
> > + regmap_reg_range(BD9571MWV_DVFS_SETVMAX, BD9571MWV_DVFS_MONIVDAC),
> > + regmap_reg_range(BD9571MWV_GPIO_IN, BD9571MWV_GPIO_IN),
> > + regmap_reg_range(BD9571MWV_GPIO_INT, BD9571MWV_GPIO_INTMASK),
> > + regmap_reg_range(BD9571MWV_INT_INTREQ, BD9571MWV_INT_INTMASK),
> > +};
> > +
> > +static const struct regmap_access_table bd9571mwv_readable_table = {
> > + .yes_ranges = bd9571mwv_readable_yes_ranges,
> > + .n_yes_ranges = ARRAY_SIZE(bd9571mwv_readable_yes_ranges),
> > +};
> > +
> > +static const struct regmap_range bd9571mwv_writable_yes_ranges[] = {
> > + regmap_reg_range(BD9571MWV_AVS_VD09_VID(0), BD9571MWV_AVS_VD09_VID(3)),
> > + regmap_reg_range(BD9571MWV_DVFS_SETVID, BD9571MWV_DVFS_SETVID),
> > + regmap_reg_range(BD9571MWV_GPIO_DIR, BD9571MWV_GPIO_OUT),
> > + regmap_reg_range(BD9571MWV_GPIO_INT_SET, BD9571MWV_GPIO_INTMASK),
> > + regmap_reg_range(BD9571MWV_INT_INTREQ, BD9571MWV_INT_INTMASK),
> > +};
> > +
> > +static const struct regmap_access_table bd9571mwv_writable_table = {
> > + .yes_ranges = bd9571mwv_writable_yes_ranges,
> > + .n_yes_ranges = ARRAY_SIZE(bd9571mwv_writable_yes_ranges),
> > +};
> > +
> > +static const struct regmap_range bd9571mwv_volatile_yes_ranges[] = {
> > + regmap_reg_range(BD9571MWV_GPIO_IN, BD9571MWV_GPIO_IN),
> > + regmap_reg_range(BD9571MWV_GPIO_INT, BD9571MWV_GPIO_INT),
> > + regmap_reg_range(BD9571MWV_INT_INTREQ, BD9571MWV_INT_INTREQ),
> > +};
> > +
> > +static const struct regmap_access_table bd9571mwv_volatile_table = {
> > + .yes_ranges = bd9571mwv_volatile_yes_ranges,
> > + .n_yes_ranges = ARRAY_SIZE(bd9571mwv_volatile_yes_ranges),
> > +};
> > +
> > +static const struct regmap_config bd9571mwv_regmap_config = {
> > + .reg_bits = 8,
> > + .val_bits = 8,
> > + .cache_type = REGCACHE_RBTREE,
> > + .rd_table = &bd9571mwv_readable_table,
> > + .wr_table = &bd9571mwv_writable_table,
> > + .volatile_table = &bd9571mwv_volatile_table,
> > + .max_register = 0xff,
> > +};
> > +
> > +static const struct regmap_irq bd9571mwv_irqs[] = {
> > + REGMAP_IRQ_REG(BD9571MWV_IRQ_MD1, 0,
> > + BD9571MWV_INT_INTREQ_MD1_INT),
> > + REGMAP_IRQ_REG(BD9571MWV_IRQ_MD2_E1, 0,
> > + BD9571MWV_INT_INTREQ_MD2_E1_INT),
> > + REGMAP_IRQ_REG(BD9571MWV_IRQ_MD2_E2, 0,
> > + BD9571MWV_INT_INTREQ_MD2_E2_INT),
> > + REGMAP_IRQ_REG(BD9571MWV_IRQ_PROT_ERR, 0,
> > + BD9571MWV_INT_INTREQ_PROT_ERR_INT),
> > + REGMAP_IRQ_REG(BD9571MWV_IRQ_GP, 0,
> > + BD9571MWV_INT_INTREQ_GP_INT),
> > + REGMAP_IRQ_REG(BD9571MWV_IRQ_128H_OF, 0,
> > + BD9571MWV_INT_INTREQ_128H_OF_INT),
> > + REGMAP_IRQ_REG(BD9571MWV_IRQ_WDT_OF, 0,
> > + BD9571MWV_INT_INTREQ_WDT_OF_INT),
> > + REGMAP_IRQ_REG(BD9571MWV_IRQ_BKUP_TRG, 0,
> > + BD9571MWV_INT_INTREQ_BKUP_TRG_INT),
> > +};
> > +
> > +static struct regmap_irq_chip bd9571mwv_irq_chip = {
> > + .name = "bd9571mwv",
> > + .status_base = BD9571MWV_INT_INTREQ,
> > + .mask_base = BD9571MWV_INT_INTMASK,
> > + .ack_base = BD9571MWV_INT_INTREQ,
> > + .init_ack_masked = true,
> > + .num_regs = 1,
> > + .irqs = bd9571mwv_irqs,
> > + .num_irqs = ARRAY_SIZE(bd9571mwv_irqs),
> > +};
> > +
> > +static int bd9571mwv_identify(struct bd9571mwv *bd)
> > +{
> > + struct device *dev = bd->dev;
> > + unsigned int value;
> > + int ret;
> > +
> > + ret = regmap_read(bd->regmap, BD9571MWV_VENDOR_CODE, &value);
> > + if (ret) {
> > + dev_err(dev, "Failed to read vendor code register (ret=%i)\n",
> > + ret);
> > + return ret;
> > + }
> > +
> > + if (value != BD9571MWV_VENDOR_CODE_VAL) {
> > + dev_err(dev, "Invalid vendor code ID %02x (expected %02x)\n",
> > + value, BD9571MWV_VENDOR_CODE_VAL);
> > + return -EINVAL;
> > + }
> > +
> > + ret = regmap_read(bd->regmap, BD9571MWV_PRODUCT_CODE, &value);
> > + if (ret) {
> > + dev_err(dev, "Failed to read product code register (ret=%i)\n",
> > + ret);
> > + return ret;
> > + }
> > +
> > + if (value != BD9571MWV_PRODUCT_CODE_VAL) {
> > + dev_err(dev, "Invalid product code ID %02x (expected %02x)\n",
> > + value, BD9571MWV_PRODUCT_CODE_VAL);
> > + return -EINVAL;
> > + }
> > +
> > + ret = regmap_read(bd->regmap, BD9571MWV_PRODUCT_REVISION, &value);
> > + if (ret) {
> > + dev_err(dev, "Failed to read revision register (ret=%i)\n",
> > + ret);
> > + return ret;
> > + }
> > +
> > + dev_info(dev, "Device: BD9571MWV rev. %d\n", value & 0xff);
> > +
> > + return 0;
> > +}
> > +
> > +static int bd9571mwv_probe(struct i2c_client *client,
> > + const struct i2c_device_id *ids)
> > +{
> > + struct bd9571mwv *bd;
> > + int ret;
> > +
> > + bd = devm_kzalloc(&client->dev, sizeof(*bd), GFP_KERNEL);
> > + if (!bd)
> > + return -ENOMEM;
> > +
> > + i2c_set_clientdata(client, bd);
> > + bd->dev = &client->dev;
> > + bd->irq = client->irq;
> > +
> > + bd->regmap = devm_regmap_init_i2c(client, &bd9571mwv_regmap_config);
> > + if (IS_ERR(bd->regmap)) {
> > + dev_err(bd->dev, "Failed to initialize register map\n");
> > + return PTR_ERR(bd->regmap);
> > + }
> > +
> > + ret = bd9571mwv_identify(bd);
> > + if (ret)
> > + return ret;
> > +
> > + ret = regmap_add_irq_chip(bd->regmap, bd->irq, IRQF_ONESHOT, 0,
> > + &bd9571mwv_irq_chip, &bd->irq_data);
> > + if (ret) {
> > + dev_err(bd->dev, "Failed to register IRQ chip\n");
> > + return ret;
> > + }
> > +
> > + ret = mfd_add_devices(bd->dev, PLATFORM_DEVID_AUTO, bd9571mwv_cells,
> > + ARRAY_SIZE(bd9571mwv_cells), NULL, 0,
> > + regmap_irq_get_domain(bd->irq_data));
> > + if (ret) {
> > + regmap_del_irq_chip(bd->irq, bd->irq_data);
> > + return ret;
> > + }
> > +
> > + return 0;
> > +}
> > +
> > +static int bd9571mwv_remove(struct i2c_client *client)
> > +{
> > + struct bd9571mwv *bd = i2c_get_clientdata(client);
> > +
> > + regmap_del_irq_chip(bd->irq, bd->irq_data);
> > +
> > + return 0;
> > +}
> > +
> > +static const struct of_device_id bd9571mwv_of_match_table[] = {
> > + { .compatible = "rohm,bd9571mwv", },
> > + { /* sentinel */ }
> > +};
> > +MODULE_DEVICE_TABLE(of, bd9571mwv_of_match_table);
> > +
> > +static const struct i2c_device_id bd9571mwv_id_table[] = {
> > + { "bd9571mwv", 0 },
> > + { /* sentinel */ }
> > +};
> > +MODULE_DEVICE_TABLE(i2c, bd9571mwv_id_table);
> > +
> > +static struct i2c_driver bd9571mwv_driver = {
> > + .driver = {
> > + .name = "bd9571mwv",
> > + .of_match_table = bd9571mwv_of_match_table,
> > + },
> > + .probe = bd9571mwv_probe,
> > + .remove = bd9571mwv_remove,
> > + .id_table = bd9571mwv_id_table,
> > +};
> > +module_i2c_driver(bd9571mwv_driver);
> > +
> > +MODULE_AUTHOR("Marek Vasut <marek.vasut+renesas@gmail.com>");
> > +MODULE_DESCRIPTION("BD9571MWV PMIC Driver");
> > +MODULE_LICENSE("GPL v2");
> > diff --git a/include/linux/mfd/bd9571mwv.h b/include/linux/mfd/bd9571mwv.h
> > new file mode 100644
> > index 000000000000..f0708ba4cbba
> > --- /dev/null
> > +++ b/include/linux/mfd/bd9571mwv.h
> > @@ -0,0 +1,115 @@
> > +/*
> > + * ROHM BD9571MWV-M driver
> > + *
> > + * Copyright (C) 2017 Marek Vasut <marek.vasut+renesas@gmail.com>
> > + *
> > + * This program is free software; you can redistribute it and/or
> > + * modify it under the terms of the GNU General Public License version 2 as
> > + * published by the Free Software Foundation.
> > + *
> > + * This program is distributed "as is" WITHOUT ANY WARRANTY of any
> > + * kind, whether expressed or implied; without even the implied warranty
> > + * of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> > + * GNU General Public License version 2 for more details.
> > + *
> > + * Based on the TPS65086 driver
> > + */
> > +
> > +#ifndef __LINUX_MFD_BD9571MWV_H
> > +#define __LINUX_MFD_BD9571MWV_H
> > +
> > +#include <linux/device.h>
> > +#include <linux/regmap.h>
> > +
> > +/* List of registers for BD9571MWV */
> > +#define BD9571MWV_VENDOR_CODE 0x00
> > +#define BD9571MWV_VENDOR_CODE_VAL 0xdb
> > +#define BD9571MWV_PRODUCT_CODE 0x01
> > +#define BD9571MWV_PRODUCT_CODE_VAL 0x60
> > +#define BD9571MWV_PRODUCT_REVISION 0x02
> > +
> > +#define BD9571MWV_I2C_FUSA_MODE 0x10
> > +#define BD9571MWV_I2C_MD2_E1_BIT_1 0x11
> > +#define BD9571MWV_I2C_MD2_E1_BIT_2 0x12
> > +
> > +#define BD9571MWV_BKUP_MODE_CNT 0x20
> > +#define BD9571MWV_BKUP_MODE_STATUS 0x21
> > +#define BD9571MWV_BKUP_RECOVERY_CNT 0x22
> > +#define BD9571MWV_BKUP_CTRL_TIM_CNT 0x23
> > +#define BD9571MWV_WAITBKUP_WDT_CNT 0x24
> > +#define BD9571MWV_128H_TIM_CNT 0x26
> > +#define BD9571MWV_QLLM_CNT 0x27
> > +
> > +#define BD9571MWV_AVS_SET_MONI 0x31
> > +#define BD9571MWV_AVS_SET_MONI_MASK 0x3
> > +#define BD9571MWV_AVS_VD09_VID(n) (0x32 + (n))
> > +#define BD9571MWV_AVS_DVFS_VID(n) (0x36 + (n))
> > +
> > +#define BD9571MWV_VD18_VID 0x42
> > +#define BD9571MWV_VD25_VID 0x43
> > +#define BD9571MWV_VD33_VID 0x44
> > +
> > +#define BD9571MWV_DVFS_VINIT 0x50
> > +#define BD9571MWV_DVFS_SETVMAX 0x52
> > +#define BD9571MWV_DVFS_BOOSTVID 0x53
> > +#define BD9571MWV_DVFS_SETVID 0x54
> > +#define BD9571MWV_DVFS_MONIVDAC 0x55
> > +#define BD9571MWV_DVFS_PGD_CNT 0x56
> > +
> > +#define BD9571MWV_GPIO_DIR 0x60
> > +#define BD9571MWV_GPIO_OUT 0x61
> > +#define BD9571MWV_GPIO_IN 0x62
> > +#define BD9571MWV_GPIO_DEB 0x63
> > +#define BD9571MWV_GPIO_INT_SET 0x64
> > +#define BD9571MWV_GPIO_INT 0x65
> > +#define BD9571MWV_GPIO_INTMASK 0x66
> > +
> > +#define BD9571MWV_REG_KEEP(n) (0x70 + (n))
> > +
> > +#define BD9571MWV_PMIC_INTERNAL_STATUS 0x80
> > +#define BD9571MWV_PROT_ERROR_STATUS0 0x81
> > +#define BD9571MWV_PROT_ERROR_STATUS1 0x82
> > +#define BD9571MWV_PROT_ERROR_STATUS2 0x83
> > +#define BD9571MWV_PROT_ERROR_STATUS3 0x84
> > +#define BD9571MWV_PROT_ERROR_STATUS4 0x85
> > +
> > +#define BD9571MWV_INT_INTREQ 0x90
> > +#define BD9571MWV_INT_INTREQ_MD1_INT BIT(0)
> > +#define BD9571MWV_INT_INTREQ_MD2_E1_INT BIT(1)
> > +#define BD9571MWV_INT_INTREQ_MD2_E2_INT BIT(2)
> > +#define BD9571MWV_INT_INTREQ_PROT_ERR_INT BIT(3)
> > +#define BD9571MWV_INT_INTREQ_GP_INT BIT(4)
> > +#define BD9571MWV_INT_INTREQ_128H_OF_INT BIT(5)
> > +#define BD9571MWV_INT_INTREQ_WDT_OF_INT BIT(6)
> > +#define BD9571MWV_INT_INTREQ_BKUP_TRG_INT BIT(7)
> > +#define BD9571MWV_INT_INTMASK 0x91
> > +
> > +#define BD9571MWV_ACCESS_KEY 0xff
> > +
> > +/* Define the BD9571MWV IRQ numbers */
> > +enum bd9571mwv_irqs {
> > + BD9571MWV_IRQ_MD1,
> > + BD9571MWV_IRQ_MD2_E1,
> > + BD9571MWV_IRQ_MD2_E2,
> > + BD9571MWV_IRQ_PROT_ERR,
> > + BD9571MWV_IRQ_GP,
> > + BD9571MWV_IRQ_128H_OF,
> > + BD9571MWV_IRQ_WDT_OF,
> > + BD9571MWV_IRQ_BKUP_TRG,
> > +};
> > +
> > +/**
> > + * struct bd9571mwv - state holder for the bd9571mwv driver
> > + *
> > + * Device data may be used to access the BD9571MWV chip
> > + */
> > +struct bd9571mwv {
> > + struct device *dev;
> > + struct regmap *regmap;
> > +
> > + /* IRQ Data */
> > + int irq;
> > + struct regmap_irq_chip_data *irq_data;
> > +};
> > +
> > +#endif /* __LINUX_MFD_BD9571MWV_H */
> >
>
>
--
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH V3 2/2] mfd: Add ROHM BD9571MWV-M MFD PMIC driver
2017-07-03 11:55 ` Lee Jones
@ 2017-07-03 12:03 ` Marek Vasut
2017-07-03 13:48 ` Lee Jones
0 siblings, 1 reply; 9+ messages in thread
From: Marek Vasut @ 2017-07-03 12:03 UTC (permalink / raw)
To: Lee Jones
Cc: linux-renesas-soc, Marek Vasut, linux-kernel, Geert Uytterhoeven
On 07/03/2017 01:55 PM, Lee Jones wrote:
> On Tue, 27 Jun 2017, Marek Vasut wrote:
>
>> On 05/02/2017 02:18 PM, Marek Vasut wrote:
>>> Add the MFD part of the ROHM BD9571MWV-M PMIC driver and MAINTAINERS
>>> entry. The MFD part only specifies the regmap bits for the PMIC and
>>> binds the subdevs together.
>>>
>>> Signed-off-by: Marek Vasut <marek.vasut+renesas@gmail.com>
>>> Cc: linux-kernel@vger.kernel.org
>>> Cc: Geert Uytterhoeven <geert+renesas@glider.be>
>>> Cc: Lee Jones <lee.jones@linaro.org>
>>
>> Lee, bump, do you plan to apply these patches anytime soon ?
>
> Looks like these were missed for some reason.
Yes, they missed the previous merge window for some reason too, while
the rest of the patches from the set made it in. It'd be real nice if
they made it into the 4.13 ...
> I'll put them back on the pile for review.
>
> Please note that the merge window is currently open, so you are likely
> to experience a period of inactivity until -rc1 is released.
I'd really appreciate it if these patches made it into the 4.13 release,
see above.
>>> ---
>>> V2: - Change BD9571MWV_AVS_VD09_VID0,1,2,3 to BD9571MWV_AVS_VD09_VID(n)
>>> - Change BD9571MWV_AVS_DVFS_VID0,1,2,3 to BD9571MWV_AVS_DVFS_VID(n)
>>> - Make the AVS_VD09 range RW, so it can be used by the regulator
>>> driver for the VD09 regulator
>>> - Report the regmap read return values when attempting to read ID
>>> registers fails
>>> V3: No change
>>> ---
>>> MAINTAINERS | 11 ++
>>> drivers/mfd/Kconfig | 13 +++
>>> drivers/mfd/Makefile | 1 +
>>> drivers/mfd/bd9571mwv.c | 230 ++++++++++++++++++++++++++++++++++++++++++
>>> include/linux/mfd/bd9571mwv.h | 115 +++++++++++++++++++++
>>> 5 files changed, 370 insertions(+)
>>> create mode 100644 drivers/mfd/bd9571mwv.c
>>> create mode 100644 include/linux/mfd/bd9571mwv.h
>>>
>>> diff --git a/MAINTAINERS b/MAINTAINERS
>>> index 2a9aa4be5846..9c30e6b00358 100644
>>> --- a/MAINTAINERS
>>> +++ b/MAINTAINERS
>>> @@ -10942,6 +10942,17 @@ L: linux-serial@vger.kernel.org
>>> S: Odd Fixes
>>> F: drivers/tty/serial/rp2.*
>>>
>>> +ROHM MULTIFUNCTION BD9571MWV-M PMIC DEVICE DRIVERS
>>> +M: Marek Vasut <marek.vasut+renesas@gmail.com>
>>> +L: linux-kernel@vger.kernel.org
>>> +L: linux-renesas-soc@vger.kernel.org
>>> +S: Supported
>>> +F: drivers/mfd/bd9571mwv.c
>>> +F: drivers/regulator/bd9571mwv-regulator.c
>>> +F: drivers/gpio/gpio-bd9571mwv.c
>>> +F: include/linux/mfd/bd9571mwv.h
>>> +F: Documentation/devicetree/bindings/mfd/bd9571mwv.txt
>>> +
>>> ROSE NETWORK LAYER
>>> M: Ralf Baechle <ralf@linux-mips.org>
>>> L: linux-hams@vger.kernel.org
>>> diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig
>>> index 3eb5c93595f6..6f4668ae28cc 100644
>>> --- a/drivers/mfd/Kconfig
>>> +++ b/drivers/mfd/Kconfig
>>> @@ -133,6 +133,19 @@ config MFD_BCM590XX
>>> help
>>> Support for the BCM590xx PMUs from Broadcom
>>>
>>> +config MFD_BD9571MWV
>>> + tristate "ROHM BD9571MWV PMIC"
>>> + select MFD_CORE
>>> + select REGMAP_I2C
>>> + depends on I2C
>>> + help
>>> + Support for the ROHM BD9571MWV PMIC, which contains single
>>> + voltage regulator, voltage sampling units, GPIO block and
>>> + watchdog block.
>>> +
>>> + This driver can also be built as a module. If so, the module
>>> + will be called bd9571mwv.
>>> +
>>> config MFD_AC100
>>> tristate "X-Powers AC100"
>>> select MFD_CORE
>>> diff --git a/drivers/mfd/Makefile b/drivers/mfd/Makefile
>>> index c16bf1ea0ea9..3cacc9748e13 100644
>>> --- a/drivers/mfd/Makefile
>>> +++ b/drivers/mfd/Makefile
>>> @@ -10,6 +10,7 @@ obj-$(CONFIG_MFD_ACT8945A) += act8945a.o
>>> obj-$(CONFIG_MFD_SM501) += sm501.o
>>> obj-$(CONFIG_MFD_ASIC3) += asic3.o tmio_core.o
>>> obj-$(CONFIG_MFD_BCM590XX) += bcm590xx.o
>>> +obj-$(CONFIG_MFD_BD9571MWV) += bd9571mwv.o
>>> cros_ec_core-objs := cros_ec.o
>>> cros_ec_core-$(CONFIG_ACPI) += cros_ec_acpi_gpe.o
>>> obj-$(CONFIG_MFD_CROS_EC) += cros_ec_core.o
>>> diff --git a/drivers/mfd/bd9571mwv.c b/drivers/mfd/bd9571mwv.c
>>> new file mode 100644
>>> index 000000000000..64e088dfe7b0
>>> --- /dev/null
>>> +++ b/drivers/mfd/bd9571mwv.c
>>> @@ -0,0 +1,230 @@
>>> +/*
>>> + * ROHM BD9571MWV-M MFD driver
>>> + *
>>> + * Copyright (C) 2017 Marek Vasut <marek.vasut+renesas@gmail.com>
>>> + *
>>> + * This program is free software; you can redistribute it and/or
>>> + * modify it under the terms of the GNU General Public License version 2 as
>>> + * published by the Free Software Foundation.
>>> + *
>>> + * This program is distributed "as is" WITHOUT ANY WARRANTY of any
>>> + * kind, whether expressed or implied; without even the implied warranty
>>> + * of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
>>> + * GNU General Public License version 2 for more details.
>>> + *
>>> + * Based on the TPS65086 driver
>>> + */
>>> +
>>> +#include <linux/i2c.h>
>>> +#include <linux/interrupt.h>
>>> +#include <linux/mfd/core.h>
>>> +#include <linux/module.h>
>>> +
>>> +#include <linux/mfd/bd9571mwv.h>
>>> +
>>> +static const struct mfd_cell bd9571mwv_cells[] = {
>>> + { .name = "bd9571mwv-regulator", },
>>> + { .name = "bd9571mwv-gpio", },
>>> +};
>>> +
>>> +static const struct regmap_range bd9571mwv_readable_yes_ranges[] = {
>>> + regmap_reg_range(BD9571MWV_VENDOR_CODE, BD9571MWV_PRODUCT_REVISION),
>>> + regmap_reg_range(BD9571MWV_AVS_SET_MONI, BD9571MWV_AVS_DVFS_VID(3)),
>>> + regmap_reg_range(BD9571MWV_VD18_VID, BD9571MWV_VD33_VID),
>>> + regmap_reg_range(BD9571MWV_DVFS_VINIT, BD9571MWV_DVFS_VINIT),
>>> + regmap_reg_range(BD9571MWV_DVFS_SETVMAX, BD9571MWV_DVFS_MONIVDAC),
>>> + regmap_reg_range(BD9571MWV_GPIO_IN, BD9571MWV_GPIO_IN),
>>> + regmap_reg_range(BD9571MWV_GPIO_INT, BD9571MWV_GPIO_INTMASK),
>>> + regmap_reg_range(BD9571MWV_INT_INTREQ, BD9571MWV_INT_INTMASK),
>>> +};
>>> +
>>> +static const struct regmap_access_table bd9571mwv_readable_table = {
>>> + .yes_ranges = bd9571mwv_readable_yes_ranges,
>>> + .n_yes_ranges = ARRAY_SIZE(bd9571mwv_readable_yes_ranges),
>>> +};
>>> +
>>> +static const struct regmap_range bd9571mwv_writable_yes_ranges[] = {
>>> + regmap_reg_range(BD9571MWV_AVS_VD09_VID(0), BD9571MWV_AVS_VD09_VID(3)),
>>> + regmap_reg_range(BD9571MWV_DVFS_SETVID, BD9571MWV_DVFS_SETVID),
>>> + regmap_reg_range(BD9571MWV_GPIO_DIR, BD9571MWV_GPIO_OUT),
>>> + regmap_reg_range(BD9571MWV_GPIO_INT_SET, BD9571MWV_GPIO_INTMASK),
>>> + regmap_reg_range(BD9571MWV_INT_INTREQ, BD9571MWV_INT_INTMASK),
>>> +};
>>> +
>>> +static const struct regmap_access_table bd9571mwv_writable_table = {
>>> + .yes_ranges = bd9571mwv_writable_yes_ranges,
>>> + .n_yes_ranges = ARRAY_SIZE(bd9571mwv_writable_yes_ranges),
>>> +};
>>> +
>>> +static const struct regmap_range bd9571mwv_volatile_yes_ranges[] = {
>>> + regmap_reg_range(BD9571MWV_GPIO_IN, BD9571MWV_GPIO_IN),
>>> + regmap_reg_range(BD9571MWV_GPIO_INT, BD9571MWV_GPIO_INT),
>>> + regmap_reg_range(BD9571MWV_INT_INTREQ, BD9571MWV_INT_INTREQ),
>>> +};
>>> +
>>> +static const struct regmap_access_table bd9571mwv_volatile_table = {
>>> + .yes_ranges = bd9571mwv_volatile_yes_ranges,
>>> + .n_yes_ranges = ARRAY_SIZE(bd9571mwv_volatile_yes_ranges),
>>> +};
>>> +
>>> +static const struct regmap_config bd9571mwv_regmap_config = {
>>> + .reg_bits = 8,
>>> + .val_bits = 8,
>>> + .cache_type = REGCACHE_RBTREE,
>>> + .rd_table = &bd9571mwv_readable_table,
>>> + .wr_table = &bd9571mwv_writable_table,
>>> + .volatile_table = &bd9571mwv_volatile_table,
>>> + .max_register = 0xff,
>>> +};
>>> +
>>> +static const struct regmap_irq bd9571mwv_irqs[] = {
>>> + REGMAP_IRQ_REG(BD9571MWV_IRQ_MD1, 0,
>>> + BD9571MWV_INT_INTREQ_MD1_INT),
>>> + REGMAP_IRQ_REG(BD9571MWV_IRQ_MD2_E1, 0,
>>> + BD9571MWV_INT_INTREQ_MD2_E1_INT),
>>> + REGMAP_IRQ_REG(BD9571MWV_IRQ_MD2_E2, 0,
>>> + BD9571MWV_INT_INTREQ_MD2_E2_INT),
>>> + REGMAP_IRQ_REG(BD9571MWV_IRQ_PROT_ERR, 0,
>>> + BD9571MWV_INT_INTREQ_PROT_ERR_INT),
>>> + REGMAP_IRQ_REG(BD9571MWV_IRQ_GP, 0,
>>> + BD9571MWV_INT_INTREQ_GP_INT),
>>> + REGMAP_IRQ_REG(BD9571MWV_IRQ_128H_OF, 0,
>>> + BD9571MWV_INT_INTREQ_128H_OF_INT),
>>> + REGMAP_IRQ_REG(BD9571MWV_IRQ_WDT_OF, 0,
>>> + BD9571MWV_INT_INTREQ_WDT_OF_INT),
>>> + REGMAP_IRQ_REG(BD9571MWV_IRQ_BKUP_TRG, 0,
>>> + BD9571MWV_INT_INTREQ_BKUP_TRG_INT),
>>> +};
>>> +
>>> +static struct regmap_irq_chip bd9571mwv_irq_chip = {
>>> + .name = "bd9571mwv",
>>> + .status_base = BD9571MWV_INT_INTREQ,
>>> + .mask_base = BD9571MWV_INT_INTMASK,
>>> + .ack_base = BD9571MWV_INT_INTREQ,
>>> + .init_ack_masked = true,
>>> + .num_regs = 1,
>>> + .irqs = bd9571mwv_irqs,
>>> + .num_irqs = ARRAY_SIZE(bd9571mwv_irqs),
>>> +};
>>> +
>>> +static int bd9571mwv_identify(struct bd9571mwv *bd)
>>> +{
>>> + struct device *dev = bd->dev;
>>> + unsigned int value;
>>> + int ret;
>>> +
>>> + ret = regmap_read(bd->regmap, BD9571MWV_VENDOR_CODE, &value);
>>> + if (ret) {
>>> + dev_err(dev, "Failed to read vendor code register (ret=%i)\n",
>>> + ret);
>>> + return ret;
>>> + }
>>> +
>>> + if (value != BD9571MWV_VENDOR_CODE_VAL) {
>>> + dev_err(dev, "Invalid vendor code ID %02x (expected %02x)\n",
>>> + value, BD9571MWV_VENDOR_CODE_VAL);
>>> + return -EINVAL;
>>> + }
>>> +
>>> + ret = regmap_read(bd->regmap, BD9571MWV_PRODUCT_CODE, &value);
>>> + if (ret) {
>>> + dev_err(dev, "Failed to read product code register (ret=%i)\n",
>>> + ret);
>>> + return ret;
>>> + }
>>> +
>>> + if (value != BD9571MWV_PRODUCT_CODE_VAL) {
>>> + dev_err(dev, "Invalid product code ID %02x (expected %02x)\n",
>>> + value, BD9571MWV_PRODUCT_CODE_VAL);
>>> + return -EINVAL;
>>> + }
>>> +
>>> + ret = regmap_read(bd->regmap, BD9571MWV_PRODUCT_REVISION, &value);
>>> + if (ret) {
>>> + dev_err(dev, "Failed to read revision register (ret=%i)\n",
>>> + ret);
>>> + return ret;
>>> + }
>>> +
>>> + dev_info(dev, "Device: BD9571MWV rev. %d\n", value & 0xff);
>>> +
>>> + return 0;
>>> +}
>>> +
>>> +static int bd9571mwv_probe(struct i2c_client *client,
>>> + const struct i2c_device_id *ids)
>>> +{
>>> + struct bd9571mwv *bd;
>>> + int ret;
>>> +
>>> + bd = devm_kzalloc(&client->dev, sizeof(*bd), GFP_KERNEL);
>>> + if (!bd)
>>> + return -ENOMEM;
>>> +
>>> + i2c_set_clientdata(client, bd);
>>> + bd->dev = &client->dev;
>>> + bd->irq = client->irq;
>>> +
>>> + bd->regmap = devm_regmap_init_i2c(client, &bd9571mwv_regmap_config);
>>> + if (IS_ERR(bd->regmap)) {
>>> + dev_err(bd->dev, "Failed to initialize register map\n");
>>> + return PTR_ERR(bd->regmap);
>>> + }
>>> +
>>> + ret = bd9571mwv_identify(bd);
>>> + if (ret)
>>> + return ret;
>>> +
>>> + ret = regmap_add_irq_chip(bd->regmap, bd->irq, IRQF_ONESHOT, 0,
>>> + &bd9571mwv_irq_chip, &bd->irq_data);
>>> + if (ret) {
>>> + dev_err(bd->dev, "Failed to register IRQ chip\n");
>>> + return ret;
>>> + }
>>> +
>>> + ret = mfd_add_devices(bd->dev, PLATFORM_DEVID_AUTO, bd9571mwv_cells,
>>> + ARRAY_SIZE(bd9571mwv_cells), NULL, 0,
>>> + regmap_irq_get_domain(bd->irq_data));
>>> + if (ret) {
>>> + regmap_del_irq_chip(bd->irq, bd->irq_data);
>>> + return ret;
>>> + }
>>> +
>>> + return 0;
>>> +}
>>> +
>>> +static int bd9571mwv_remove(struct i2c_client *client)
>>> +{
>>> + struct bd9571mwv *bd = i2c_get_clientdata(client);
>>> +
>>> + regmap_del_irq_chip(bd->irq, bd->irq_data);
>>> +
>>> + return 0;
>>> +}
>>> +
>>> +static const struct of_device_id bd9571mwv_of_match_table[] = {
>>> + { .compatible = "rohm,bd9571mwv", },
>>> + { /* sentinel */ }
>>> +};
>>> +MODULE_DEVICE_TABLE(of, bd9571mwv_of_match_table);
>>> +
>>> +static const struct i2c_device_id bd9571mwv_id_table[] = {
>>> + { "bd9571mwv", 0 },
>>> + { /* sentinel */ }
>>> +};
>>> +MODULE_DEVICE_TABLE(i2c, bd9571mwv_id_table);
>>> +
>>> +static struct i2c_driver bd9571mwv_driver = {
>>> + .driver = {
>>> + .name = "bd9571mwv",
>>> + .of_match_table = bd9571mwv_of_match_table,
>>> + },
>>> + .probe = bd9571mwv_probe,
>>> + .remove = bd9571mwv_remove,
>>> + .id_table = bd9571mwv_id_table,
>>> +};
>>> +module_i2c_driver(bd9571mwv_driver);
>>> +
>>> +MODULE_AUTHOR("Marek Vasut <marek.vasut+renesas@gmail.com>");
>>> +MODULE_DESCRIPTION("BD9571MWV PMIC Driver");
>>> +MODULE_LICENSE("GPL v2");
>>> diff --git a/include/linux/mfd/bd9571mwv.h b/include/linux/mfd/bd9571mwv.h
>>> new file mode 100644
>>> index 000000000000..f0708ba4cbba
>>> --- /dev/null
>>> +++ b/include/linux/mfd/bd9571mwv.h
>>> @@ -0,0 +1,115 @@
>>> +/*
>>> + * ROHM BD9571MWV-M driver
>>> + *
>>> + * Copyright (C) 2017 Marek Vasut <marek.vasut+renesas@gmail.com>
>>> + *
>>> + * This program is free software; you can redistribute it and/or
>>> + * modify it under the terms of the GNU General Public License version 2 as
>>> + * published by the Free Software Foundation.
>>> + *
>>> + * This program is distributed "as is" WITHOUT ANY WARRANTY of any
>>> + * kind, whether expressed or implied; without even the implied warranty
>>> + * of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
>>> + * GNU General Public License version 2 for more details.
>>> + *
>>> + * Based on the TPS65086 driver
>>> + */
>>> +
>>> +#ifndef __LINUX_MFD_BD9571MWV_H
>>> +#define __LINUX_MFD_BD9571MWV_H
>>> +
>>> +#include <linux/device.h>
>>> +#include <linux/regmap.h>
>>> +
>>> +/* List of registers for BD9571MWV */
>>> +#define BD9571MWV_VENDOR_CODE 0x00
>>> +#define BD9571MWV_VENDOR_CODE_VAL 0xdb
>>> +#define BD9571MWV_PRODUCT_CODE 0x01
>>> +#define BD9571MWV_PRODUCT_CODE_VAL 0x60
>>> +#define BD9571MWV_PRODUCT_REVISION 0x02
>>> +
>>> +#define BD9571MWV_I2C_FUSA_MODE 0x10
>>> +#define BD9571MWV_I2C_MD2_E1_BIT_1 0x11
>>> +#define BD9571MWV_I2C_MD2_E1_BIT_2 0x12
>>> +
>>> +#define BD9571MWV_BKUP_MODE_CNT 0x20
>>> +#define BD9571MWV_BKUP_MODE_STATUS 0x21
>>> +#define BD9571MWV_BKUP_RECOVERY_CNT 0x22
>>> +#define BD9571MWV_BKUP_CTRL_TIM_CNT 0x23
>>> +#define BD9571MWV_WAITBKUP_WDT_CNT 0x24
>>> +#define BD9571MWV_128H_TIM_CNT 0x26
>>> +#define BD9571MWV_QLLM_CNT 0x27
>>> +
>>> +#define BD9571MWV_AVS_SET_MONI 0x31
>>> +#define BD9571MWV_AVS_SET_MONI_MASK 0x3
>>> +#define BD9571MWV_AVS_VD09_VID(n) (0x32 + (n))
>>> +#define BD9571MWV_AVS_DVFS_VID(n) (0x36 + (n))
>>> +
>>> +#define BD9571MWV_VD18_VID 0x42
>>> +#define BD9571MWV_VD25_VID 0x43
>>> +#define BD9571MWV_VD33_VID 0x44
>>> +
>>> +#define BD9571MWV_DVFS_VINIT 0x50
>>> +#define BD9571MWV_DVFS_SETVMAX 0x52
>>> +#define BD9571MWV_DVFS_BOOSTVID 0x53
>>> +#define BD9571MWV_DVFS_SETVID 0x54
>>> +#define BD9571MWV_DVFS_MONIVDAC 0x55
>>> +#define BD9571MWV_DVFS_PGD_CNT 0x56
>>> +
>>> +#define BD9571MWV_GPIO_DIR 0x60
>>> +#define BD9571MWV_GPIO_OUT 0x61
>>> +#define BD9571MWV_GPIO_IN 0x62
>>> +#define BD9571MWV_GPIO_DEB 0x63
>>> +#define BD9571MWV_GPIO_INT_SET 0x64
>>> +#define BD9571MWV_GPIO_INT 0x65
>>> +#define BD9571MWV_GPIO_INTMASK 0x66
>>> +
>>> +#define BD9571MWV_REG_KEEP(n) (0x70 + (n))
>>> +
>>> +#define BD9571MWV_PMIC_INTERNAL_STATUS 0x80
>>> +#define BD9571MWV_PROT_ERROR_STATUS0 0x81
>>> +#define BD9571MWV_PROT_ERROR_STATUS1 0x82
>>> +#define BD9571MWV_PROT_ERROR_STATUS2 0x83
>>> +#define BD9571MWV_PROT_ERROR_STATUS3 0x84
>>> +#define BD9571MWV_PROT_ERROR_STATUS4 0x85
>>> +
>>> +#define BD9571MWV_INT_INTREQ 0x90
>>> +#define BD9571MWV_INT_INTREQ_MD1_INT BIT(0)
>>> +#define BD9571MWV_INT_INTREQ_MD2_E1_INT BIT(1)
>>> +#define BD9571MWV_INT_INTREQ_MD2_E2_INT BIT(2)
>>> +#define BD9571MWV_INT_INTREQ_PROT_ERR_INT BIT(3)
>>> +#define BD9571MWV_INT_INTREQ_GP_INT BIT(4)
>>> +#define BD9571MWV_INT_INTREQ_128H_OF_INT BIT(5)
>>> +#define BD9571MWV_INT_INTREQ_WDT_OF_INT BIT(6)
>>> +#define BD9571MWV_INT_INTREQ_BKUP_TRG_INT BIT(7)
>>> +#define BD9571MWV_INT_INTMASK 0x91
>>> +
>>> +#define BD9571MWV_ACCESS_KEY 0xff
>>> +
>>> +/* Define the BD9571MWV IRQ numbers */
>>> +enum bd9571mwv_irqs {
>>> + BD9571MWV_IRQ_MD1,
>>> + BD9571MWV_IRQ_MD2_E1,
>>> + BD9571MWV_IRQ_MD2_E2,
>>> + BD9571MWV_IRQ_PROT_ERR,
>>> + BD9571MWV_IRQ_GP,
>>> + BD9571MWV_IRQ_128H_OF,
>>> + BD9571MWV_IRQ_WDT_OF,
>>> + BD9571MWV_IRQ_BKUP_TRG,
>>> +};
>>> +
>>> +/**
>>> + * struct bd9571mwv - state holder for the bd9571mwv driver
>>> + *
>>> + * Device data may be used to access the BD9571MWV chip
>>> + */
>>> +struct bd9571mwv {
>>> + struct device *dev;
>>> + struct regmap *regmap;
>>> +
>>> + /* IRQ Data */
>>> + int irq;
>>> + struct regmap_irq_chip_data *irq_data;
>>> +};
>>> +
>>> +#endif /* __LINUX_MFD_BD9571MWV_H */
>>>
>>
>>
>
--
Best regards,
Marek Vasut
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH V3 2/2] mfd: Add ROHM BD9571MWV-M MFD PMIC driver
2017-07-03 12:03 ` Marek Vasut
@ 2017-07-03 13:48 ` Lee Jones
2017-07-03 14:16 ` Marek Vasut
0 siblings, 1 reply; 9+ messages in thread
From: Lee Jones @ 2017-07-03 13:48 UTC (permalink / raw)
To: Marek Vasut
Cc: linux-renesas-soc, Marek Vasut, linux-kernel, Geert Uytterhoeven
On Mon, 03 Jul 2017, Marek Vasut wrote:
> On 07/03/2017 01:55 PM, Lee Jones wrote:
> > On Tue, 27 Jun 2017, Marek Vasut wrote:
> >
> >> On 05/02/2017 02:18 PM, Marek Vasut wrote:
> >>> Add the MFD part of the ROHM BD9571MWV-M PMIC driver and MAINTAINERS
> >>> entry. The MFD part only specifies the regmap bits for the PMIC and
> >>> binds the subdevs together.
> >>>
> >>> Signed-off-by: Marek Vasut <marek.vasut+renesas@gmail.com>
> >>> Cc: linux-kernel@vger.kernel.org
> >>> Cc: Geert Uytterhoeven <geert+renesas@glider.be>
> >>> Cc: Lee Jones <lee.jones@linaro.org>
> >>
> >> Lee, bump, do you plan to apply these patches anytime soon ?
> >
> > Looks like these were missed for some reason.
>
> Yes, they missed the previous merge window for some reason too, while
> the rest of the patches from the set made it in. It'd be real nice if
> they made it into the 4.13 ...
They won't. The merge window is already open.
You're looking at v4.14 now.
> > I'll put them back on the pile for review.
> >
> > Please note that the merge window is currently open, so you are likely
> > to experience a period of inactivity until -rc1 is released.
>
> I'd really appreciate it if these patches made it into the 4.13 release,
> see above.
>
> >>> ---
> >>> V2: - Change BD9571MWV_AVS_VD09_VID0,1,2,3 to BD9571MWV_AVS_VD09_VID(n)
> >>> - Change BD9571MWV_AVS_DVFS_VID0,1,2,3 to BD9571MWV_AVS_DVFS_VID(n)
> >>> - Make the AVS_VD09 range RW, so it can be used by the regulator
> >>> driver for the VD09 regulator
> >>> - Report the regmap read return values when attempting to read ID
> >>> registers fails
> >>> V3: No change
> >>> ---
> >>> MAINTAINERS | 11 ++
> >>> drivers/mfd/Kconfig | 13 +++
> >>> drivers/mfd/Makefile | 1 +
> >>> drivers/mfd/bd9571mwv.c | 230 ++++++++++++++++++++++++++++++++++++++++++
> >>> include/linux/mfd/bd9571mwv.h | 115 +++++++++++++++++++++
> >>> 5 files changed, 370 insertions(+)
> >>> create mode 100644 drivers/mfd/bd9571mwv.c
> >>> create mode 100644 include/linux/mfd/bd9571mwv.h
> >>>
> >>> diff --git a/MAINTAINERS b/MAINTAINERS
> >>> index 2a9aa4be5846..9c30e6b00358 100644
> >>> --- a/MAINTAINERS
> >>> +++ b/MAINTAINERS
> >>> @@ -10942,6 +10942,17 @@ L: linux-serial@vger.kernel.org
> >>> S: Odd Fixes
> >>> F: drivers/tty/serial/rp2.*
> >>>
> >>> +ROHM MULTIFUNCTION BD9571MWV-M PMIC DEVICE DRIVERS
> >>> +M: Marek Vasut <marek.vasut+renesas@gmail.com>
> >>> +L: linux-kernel@vger.kernel.org
> >>> +L: linux-renesas-soc@vger.kernel.org
> >>> +S: Supported
> >>> +F: drivers/mfd/bd9571mwv.c
> >>> +F: drivers/regulator/bd9571mwv-regulator.c
> >>> +F: drivers/gpio/gpio-bd9571mwv.c
> >>> +F: include/linux/mfd/bd9571mwv.h
> >>> +F: Documentation/devicetree/bindings/mfd/bd9571mwv.txt
> >>> +
> >>> ROSE NETWORK LAYER
> >>> M: Ralf Baechle <ralf@linux-mips.org>
> >>> L: linux-hams@vger.kernel.org
> >>> diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig
> >>> index 3eb5c93595f6..6f4668ae28cc 100644
> >>> --- a/drivers/mfd/Kconfig
> >>> +++ b/drivers/mfd/Kconfig
> >>> @@ -133,6 +133,19 @@ config MFD_BCM590XX
> >>> help
> >>> Support for the BCM590xx PMUs from Broadcom
> >>>
> >>> +config MFD_BD9571MWV
> >>> + tristate "ROHM BD9571MWV PMIC"
> >>> + select MFD_CORE
> >>> + select REGMAP_I2C
> >>> + depends on I2C
> >>> + help
> >>> + Support for the ROHM BD9571MWV PMIC, which contains single
> >>> + voltage regulator, voltage sampling units, GPIO block and
> >>> + watchdog block.
> >>> +
> >>> + This driver can also be built as a module. If so, the module
> >>> + will be called bd9571mwv.
> >>> +
> >>> config MFD_AC100
> >>> tristate "X-Powers AC100"
> >>> select MFD_CORE
> >>> diff --git a/drivers/mfd/Makefile b/drivers/mfd/Makefile
> >>> index c16bf1ea0ea9..3cacc9748e13 100644
> >>> --- a/drivers/mfd/Makefile
> >>> +++ b/drivers/mfd/Makefile
> >>> @@ -10,6 +10,7 @@ obj-$(CONFIG_MFD_ACT8945A) += act8945a.o
> >>> obj-$(CONFIG_MFD_SM501) += sm501.o
> >>> obj-$(CONFIG_MFD_ASIC3) += asic3.o tmio_core.o
> >>> obj-$(CONFIG_MFD_BCM590XX) += bcm590xx.o
> >>> +obj-$(CONFIG_MFD_BD9571MWV) += bd9571mwv.o
> >>> cros_ec_core-objs := cros_ec.o
> >>> cros_ec_core-$(CONFIG_ACPI) += cros_ec_acpi_gpe.o
> >>> obj-$(CONFIG_MFD_CROS_EC) += cros_ec_core.o
> >>> diff --git a/drivers/mfd/bd9571mwv.c b/drivers/mfd/bd9571mwv.c
> >>> new file mode 100644
> >>> index 000000000000..64e088dfe7b0
> >>> --- /dev/null
> >>> +++ b/drivers/mfd/bd9571mwv.c
> >>> @@ -0,0 +1,230 @@
> >>> +/*
> >>> + * ROHM BD9571MWV-M MFD driver
> >>> + *
> >>> + * Copyright (C) 2017 Marek Vasut <marek.vasut+renesas@gmail.com>
> >>> + *
> >>> + * This program is free software; you can redistribute it and/or
> >>> + * modify it under the terms of the GNU General Public License version 2 as
> >>> + * published by the Free Software Foundation.
> >>> + *
> >>> + * This program is distributed "as is" WITHOUT ANY WARRANTY of any
> >>> + * kind, whether expressed or implied; without even the implied warranty
> >>> + * of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> >>> + * GNU General Public License version 2 for more details.
> >>> + *
> >>> + * Based on the TPS65086 driver
> >>> + */
> >>> +
> >>> +#include <linux/i2c.h>
> >>> +#include <linux/interrupt.h>
> >>> +#include <linux/mfd/core.h>
> >>> +#include <linux/module.h>
> >>> +
> >>> +#include <linux/mfd/bd9571mwv.h>
> >>> +
> >>> +static const struct mfd_cell bd9571mwv_cells[] = {
> >>> + { .name = "bd9571mwv-regulator", },
> >>> + { .name = "bd9571mwv-gpio", },
> >>> +};
> >>> +
> >>> +static const struct regmap_range bd9571mwv_readable_yes_ranges[] = {
> >>> + regmap_reg_range(BD9571MWV_VENDOR_CODE, BD9571MWV_PRODUCT_REVISION),
> >>> + regmap_reg_range(BD9571MWV_AVS_SET_MONI, BD9571MWV_AVS_DVFS_VID(3)),
> >>> + regmap_reg_range(BD9571MWV_VD18_VID, BD9571MWV_VD33_VID),
> >>> + regmap_reg_range(BD9571MWV_DVFS_VINIT, BD9571MWV_DVFS_VINIT),
> >>> + regmap_reg_range(BD9571MWV_DVFS_SETVMAX, BD9571MWV_DVFS_MONIVDAC),
> >>> + regmap_reg_range(BD9571MWV_GPIO_IN, BD9571MWV_GPIO_IN),
> >>> + regmap_reg_range(BD9571MWV_GPIO_INT, BD9571MWV_GPIO_INTMASK),
> >>> + regmap_reg_range(BD9571MWV_INT_INTREQ, BD9571MWV_INT_INTMASK),
> >>> +};
> >>> +
> >>> +static const struct regmap_access_table bd9571mwv_readable_table = {
> >>> + .yes_ranges = bd9571mwv_readable_yes_ranges,
> >>> + .n_yes_ranges = ARRAY_SIZE(bd9571mwv_readable_yes_ranges),
> >>> +};
> >>> +
> >>> +static const struct regmap_range bd9571mwv_writable_yes_ranges[] = {
> >>> + regmap_reg_range(BD9571MWV_AVS_VD09_VID(0), BD9571MWV_AVS_VD09_VID(3)),
> >>> + regmap_reg_range(BD9571MWV_DVFS_SETVID, BD9571MWV_DVFS_SETVID),
> >>> + regmap_reg_range(BD9571MWV_GPIO_DIR, BD9571MWV_GPIO_OUT),
> >>> + regmap_reg_range(BD9571MWV_GPIO_INT_SET, BD9571MWV_GPIO_INTMASK),
> >>> + regmap_reg_range(BD9571MWV_INT_INTREQ, BD9571MWV_INT_INTMASK),
> >>> +};
> >>> +
> >>> +static const struct regmap_access_table bd9571mwv_writable_table = {
> >>> + .yes_ranges = bd9571mwv_writable_yes_ranges,
> >>> + .n_yes_ranges = ARRAY_SIZE(bd9571mwv_writable_yes_ranges),
> >>> +};
> >>> +
> >>> +static const struct regmap_range bd9571mwv_volatile_yes_ranges[] = {
> >>> + regmap_reg_range(BD9571MWV_GPIO_IN, BD9571MWV_GPIO_IN),
> >>> + regmap_reg_range(BD9571MWV_GPIO_INT, BD9571MWV_GPIO_INT),
> >>> + regmap_reg_range(BD9571MWV_INT_INTREQ, BD9571MWV_INT_INTREQ),
> >>> +};
> >>> +
> >>> +static const struct regmap_access_table bd9571mwv_volatile_table = {
> >>> + .yes_ranges = bd9571mwv_volatile_yes_ranges,
> >>> + .n_yes_ranges = ARRAY_SIZE(bd9571mwv_volatile_yes_ranges),
> >>> +};
> >>> +
> >>> +static const struct regmap_config bd9571mwv_regmap_config = {
> >>> + .reg_bits = 8,
> >>> + .val_bits = 8,
> >>> + .cache_type = REGCACHE_RBTREE,
> >>> + .rd_table = &bd9571mwv_readable_table,
> >>> + .wr_table = &bd9571mwv_writable_table,
> >>> + .volatile_table = &bd9571mwv_volatile_table,
> >>> + .max_register = 0xff,
> >>> +};
> >>> +
> >>> +static const struct regmap_irq bd9571mwv_irqs[] = {
> >>> + REGMAP_IRQ_REG(BD9571MWV_IRQ_MD1, 0,
> >>> + BD9571MWV_INT_INTREQ_MD1_INT),
> >>> + REGMAP_IRQ_REG(BD9571MWV_IRQ_MD2_E1, 0,
> >>> + BD9571MWV_INT_INTREQ_MD2_E1_INT),
> >>> + REGMAP_IRQ_REG(BD9571MWV_IRQ_MD2_E2, 0,
> >>> + BD9571MWV_INT_INTREQ_MD2_E2_INT),
> >>> + REGMAP_IRQ_REG(BD9571MWV_IRQ_PROT_ERR, 0,
> >>> + BD9571MWV_INT_INTREQ_PROT_ERR_INT),
> >>> + REGMAP_IRQ_REG(BD9571MWV_IRQ_GP, 0,
> >>> + BD9571MWV_INT_INTREQ_GP_INT),
> >>> + REGMAP_IRQ_REG(BD9571MWV_IRQ_128H_OF, 0,
> >>> + BD9571MWV_INT_INTREQ_128H_OF_INT),
> >>> + REGMAP_IRQ_REG(BD9571MWV_IRQ_WDT_OF, 0,
> >>> + BD9571MWV_INT_INTREQ_WDT_OF_INT),
> >>> + REGMAP_IRQ_REG(BD9571MWV_IRQ_BKUP_TRG, 0,
> >>> + BD9571MWV_INT_INTREQ_BKUP_TRG_INT),
> >>> +};
> >>> +
> >>> +static struct regmap_irq_chip bd9571mwv_irq_chip = {
> >>> + .name = "bd9571mwv",
> >>> + .status_base = BD9571MWV_INT_INTREQ,
> >>> + .mask_base = BD9571MWV_INT_INTMASK,
> >>> + .ack_base = BD9571MWV_INT_INTREQ,
> >>> + .init_ack_masked = true,
> >>> + .num_regs = 1,
> >>> + .irqs = bd9571mwv_irqs,
> >>> + .num_irqs = ARRAY_SIZE(bd9571mwv_irqs),
> >>> +};
> >>> +
> >>> +static int bd9571mwv_identify(struct bd9571mwv *bd)
> >>> +{
> >>> + struct device *dev = bd->dev;
> >>> + unsigned int value;
> >>> + int ret;
> >>> +
> >>> + ret = regmap_read(bd->regmap, BD9571MWV_VENDOR_CODE, &value);
> >>> + if (ret) {
> >>> + dev_err(dev, "Failed to read vendor code register (ret=%i)\n",
> >>> + ret);
> >>> + return ret;
> >>> + }
> >>> +
> >>> + if (value != BD9571MWV_VENDOR_CODE_VAL) {
> >>> + dev_err(dev, "Invalid vendor code ID %02x (expected %02x)\n",
> >>> + value, BD9571MWV_VENDOR_CODE_VAL);
> >>> + return -EINVAL;
> >>> + }
> >>> +
> >>> + ret = regmap_read(bd->regmap, BD9571MWV_PRODUCT_CODE, &value);
> >>> + if (ret) {
> >>> + dev_err(dev, "Failed to read product code register (ret=%i)\n",
> >>> + ret);
> >>> + return ret;
> >>> + }
> >>> +
> >>> + if (value != BD9571MWV_PRODUCT_CODE_VAL) {
> >>> + dev_err(dev, "Invalid product code ID %02x (expected %02x)\n",
> >>> + value, BD9571MWV_PRODUCT_CODE_VAL);
> >>> + return -EINVAL;
> >>> + }
> >>> +
> >>> + ret = regmap_read(bd->regmap, BD9571MWV_PRODUCT_REVISION, &value);
> >>> + if (ret) {
> >>> + dev_err(dev, "Failed to read revision register (ret=%i)\n",
> >>> + ret);
> >>> + return ret;
> >>> + }
> >>> +
> >>> + dev_info(dev, "Device: BD9571MWV rev. %d\n", value & 0xff);
> >>> +
> >>> + return 0;
> >>> +}
> >>> +
> >>> +static int bd9571mwv_probe(struct i2c_client *client,
> >>> + const struct i2c_device_id *ids)
> >>> +{
> >>> + struct bd9571mwv *bd;
> >>> + int ret;
> >>> +
> >>> + bd = devm_kzalloc(&client->dev, sizeof(*bd), GFP_KERNEL);
> >>> + if (!bd)
> >>> + return -ENOMEM;
> >>> +
> >>> + i2c_set_clientdata(client, bd);
> >>> + bd->dev = &client->dev;
> >>> + bd->irq = client->irq;
> >>> +
> >>> + bd->regmap = devm_regmap_init_i2c(client, &bd9571mwv_regmap_config);
> >>> + if (IS_ERR(bd->regmap)) {
> >>> + dev_err(bd->dev, "Failed to initialize register map\n");
> >>> + return PTR_ERR(bd->regmap);
> >>> + }
> >>> +
> >>> + ret = bd9571mwv_identify(bd);
> >>> + if (ret)
> >>> + return ret;
> >>> +
> >>> + ret = regmap_add_irq_chip(bd->regmap, bd->irq, IRQF_ONESHOT, 0,
> >>> + &bd9571mwv_irq_chip, &bd->irq_data);
> >>> + if (ret) {
> >>> + dev_err(bd->dev, "Failed to register IRQ chip\n");
> >>> + return ret;
> >>> + }
> >>> +
> >>> + ret = mfd_add_devices(bd->dev, PLATFORM_DEVID_AUTO, bd9571mwv_cells,
> >>> + ARRAY_SIZE(bd9571mwv_cells), NULL, 0,
> >>> + regmap_irq_get_domain(bd->irq_data));
> >>> + if (ret) {
> >>> + regmap_del_irq_chip(bd->irq, bd->irq_data);
> >>> + return ret;
> >>> + }
> >>> +
> >>> + return 0;
> >>> +}
> >>> +
> >>> +static int bd9571mwv_remove(struct i2c_client *client)
> >>> +{
> >>> + struct bd9571mwv *bd = i2c_get_clientdata(client);
> >>> +
> >>> + regmap_del_irq_chip(bd->irq, bd->irq_data);
> >>> +
> >>> + return 0;
> >>> +}
> >>> +
> >>> +static const struct of_device_id bd9571mwv_of_match_table[] = {
> >>> + { .compatible = "rohm,bd9571mwv", },
> >>> + { /* sentinel */ }
> >>> +};
> >>> +MODULE_DEVICE_TABLE(of, bd9571mwv_of_match_table);
> >>> +
> >>> +static const struct i2c_device_id bd9571mwv_id_table[] = {
> >>> + { "bd9571mwv", 0 },
> >>> + { /* sentinel */ }
> >>> +};
> >>> +MODULE_DEVICE_TABLE(i2c, bd9571mwv_id_table);
> >>> +
> >>> +static struct i2c_driver bd9571mwv_driver = {
> >>> + .driver = {
> >>> + .name = "bd9571mwv",
> >>> + .of_match_table = bd9571mwv_of_match_table,
> >>> + },
> >>> + .probe = bd9571mwv_probe,
> >>> + .remove = bd9571mwv_remove,
> >>> + .id_table = bd9571mwv_id_table,
> >>> +};
> >>> +module_i2c_driver(bd9571mwv_driver);
> >>> +
> >>> +MODULE_AUTHOR("Marek Vasut <marek.vasut+renesas@gmail.com>");
> >>> +MODULE_DESCRIPTION("BD9571MWV PMIC Driver");
> >>> +MODULE_LICENSE("GPL v2");
> >>> diff --git a/include/linux/mfd/bd9571mwv.h b/include/linux/mfd/bd9571mwv.h
> >>> new file mode 100644
> >>> index 000000000000..f0708ba4cbba
> >>> --- /dev/null
> >>> +++ b/include/linux/mfd/bd9571mwv.h
> >>> @@ -0,0 +1,115 @@
> >>> +/*
> >>> + * ROHM BD9571MWV-M driver
> >>> + *
> >>> + * Copyright (C) 2017 Marek Vasut <marek.vasut+renesas@gmail.com>
> >>> + *
> >>> + * This program is free software; you can redistribute it and/or
> >>> + * modify it under the terms of the GNU General Public License version 2 as
> >>> + * published by the Free Software Foundation.
> >>> + *
> >>> + * This program is distributed "as is" WITHOUT ANY WARRANTY of any
> >>> + * kind, whether expressed or implied; without even the implied warranty
> >>> + * of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> >>> + * GNU General Public License version 2 for more details.
> >>> + *
> >>> + * Based on the TPS65086 driver
> >>> + */
> >>> +
> >>> +#ifndef __LINUX_MFD_BD9571MWV_H
> >>> +#define __LINUX_MFD_BD9571MWV_H
> >>> +
> >>> +#include <linux/device.h>
> >>> +#include <linux/regmap.h>
> >>> +
> >>> +/* List of registers for BD9571MWV */
> >>> +#define BD9571MWV_VENDOR_CODE 0x00
> >>> +#define BD9571MWV_VENDOR_CODE_VAL 0xdb
> >>> +#define BD9571MWV_PRODUCT_CODE 0x01
> >>> +#define BD9571MWV_PRODUCT_CODE_VAL 0x60
> >>> +#define BD9571MWV_PRODUCT_REVISION 0x02
> >>> +
> >>> +#define BD9571MWV_I2C_FUSA_MODE 0x10
> >>> +#define BD9571MWV_I2C_MD2_E1_BIT_1 0x11
> >>> +#define BD9571MWV_I2C_MD2_E1_BIT_2 0x12
> >>> +
> >>> +#define BD9571MWV_BKUP_MODE_CNT 0x20
> >>> +#define BD9571MWV_BKUP_MODE_STATUS 0x21
> >>> +#define BD9571MWV_BKUP_RECOVERY_CNT 0x22
> >>> +#define BD9571MWV_BKUP_CTRL_TIM_CNT 0x23
> >>> +#define BD9571MWV_WAITBKUP_WDT_CNT 0x24
> >>> +#define BD9571MWV_128H_TIM_CNT 0x26
> >>> +#define BD9571MWV_QLLM_CNT 0x27
> >>> +
> >>> +#define BD9571MWV_AVS_SET_MONI 0x31
> >>> +#define BD9571MWV_AVS_SET_MONI_MASK 0x3
> >>> +#define BD9571MWV_AVS_VD09_VID(n) (0x32 + (n))
> >>> +#define BD9571MWV_AVS_DVFS_VID(n) (0x36 + (n))
> >>> +
> >>> +#define BD9571MWV_VD18_VID 0x42
> >>> +#define BD9571MWV_VD25_VID 0x43
> >>> +#define BD9571MWV_VD33_VID 0x44
> >>> +
> >>> +#define BD9571MWV_DVFS_VINIT 0x50
> >>> +#define BD9571MWV_DVFS_SETVMAX 0x52
> >>> +#define BD9571MWV_DVFS_BOOSTVID 0x53
> >>> +#define BD9571MWV_DVFS_SETVID 0x54
> >>> +#define BD9571MWV_DVFS_MONIVDAC 0x55
> >>> +#define BD9571MWV_DVFS_PGD_CNT 0x56
> >>> +
> >>> +#define BD9571MWV_GPIO_DIR 0x60
> >>> +#define BD9571MWV_GPIO_OUT 0x61
> >>> +#define BD9571MWV_GPIO_IN 0x62
> >>> +#define BD9571MWV_GPIO_DEB 0x63
> >>> +#define BD9571MWV_GPIO_INT_SET 0x64
> >>> +#define BD9571MWV_GPIO_INT 0x65
> >>> +#define BD9571MWV_GPIO_INTMASK 0x66
> >>> +
> >>> +#define BD9571MWV_REG_KEEP(n) (0x70 + (n))
> >>> +
> >>> +#define BD9571MWV_PMIC_INTERNAL_STATUS 0x80
> >>> +#define BD9571MWV_PROT_ERROR_STATUS0 0x81
> >>> +#define BD9571MWV_PROT_ERROR_STATUS1 0x82
> >>> +#define BD9571MWV_PROT_ERROR_STATUS2 0x83
> >>> +#define BD9571MWV_PROT_ERROR_STATUS3 0x84
> >>> +#define BD9571MWV_PROT_ERROR_STATUS4 0x85
> >>> +
> >>> +#define BD9571MWV_INT_INTREQ 0x90
> >>> +#define BD9571MWV_INT_INTREQ_MD1_INT BIT(0)
> >>> +#define BD9571MWV_INT_INTREQ_MD2_E1_INT BIT(1)
> >>> +#define BD9571MWV_INT_INTREQ_MD2_E2_INT BIT(2)
> >>> +#define BD9571MWV_INT_INTREQ_PROT_ERR_INT BIT(3)
> >>> +#define BD9571MWV_INT_INTREQ_GP_INT BIT(4)
> >>> +#define BD9571MWV_INT_INTREQ_128H_OF_INT BIT(5)
> >>> +#define BD9571MWV_INT_INTREQ_WDT_OF_INT BIT(6)
> >>> +#define BD9571MWV_INT_INTREQ_BKUP_TRG_INT BIT(7)
> >>> +#define BD9571MWV_INT_INTMASK 0x91
> >>> +
> >>> +#define BD9571MWV_ACCESS_KEY 0xff
> >>> +
> >>> +/* Define the BD9571MWV IRQ numbers */
> >>> +enum bd9571mwv_irqs {
> >>> + BD9571MWV_IRQ_MD1,
> >>> + BD9571MWV_IRQ_MD2_E1,
> >>> + BD9571MWV_IRQ_MD2_E2,
> >>> + BD9571MWV_IRQ_PROT_ERR,
> >>> + BD9571MWV_IRQ_GP,
> >>> + BD9571MWV_IRQ_128H_OF,
> >>> + BD9571MWV_IRQ_WDT_OF,
> >>> + BD9571MWV_IRQ_BKUP_TRG,
> >>> +};
> >>> +
> >>> +/**
> >>> + * struct bd9571mwv - state holder for the bd9571mwv driver
> >>> + *
> >>> + * Device data may be used to access the BD9571MWV chip
> >>> + */
> >>> +struct bd9571mwv {
> >>> + struct device *dev;
> >>> + struct regmap *regmap;
> >>> +
> >>> + /* IRQ Data */
> >>> + int irq;
> >>> + struct regmap_irq_chip_data *irq_data;
> >>> +};
> >>> +
> >>> +#endif /* __LINUX_MFD_BD9571MWV_H */
> >>>
> >>
> >>
> >
>
>
--
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH V3 2/2] mfd: Add ROHM BD9571MWV-M MFD PMIC driver
2017-07-03 13:48 ` Lee Jones
@ 2017-07-03 14:16 ` Marek Vasut
2017-07-03 15:21 ` Lee Jones
0 siblings, 1 reply; 9+ messages in thread
From: Marek Vasut @ 2017-07-03 14:16 UTC (permalink / raw)
To: Lee Jones
Cc: linux-renesas-soc, Marek Vasut, linux-kernel, Geert Uytterhoeven
On 07/03/2017 03:48 PM, Lee Jones wrote:
> On Mon, 03 Jul 2017, Marek Vasut wrote:
>
>> On 07/03/2017 01:55 PM, Lee Jones wrote:
>>> On Tue, 27 Jun 2017, Marek Vasut wrote:
>>>
>>>> On 05/02/2017 02:18 PM, Marek Vasut wrote:
>>>>> Add the MFD part of the ROHM BD9571MWV-M PMIC driver and MAINTAINERS
>>>>> entry. The MFD part only specifies the regmap bits for the PMIC and
>>>>> binds the subdevs together.
>>>>>
>>>>> Signed-off-by: Marek Vasut <marek.vasut+renesas@gmail.com>
>>>>> Cc: linux-kernel@vger.kernel.org
>>>>> Cc: Geert Uytterhoeven <geert+renesas@glider.be>
>>>>> Cc: Lee Jones <lee.jones@linaro.org>
>>>>
>>>> Lee, bump, do you plan to apply these patches anytime soon ?
>>>
>>> Looks like these were missed for some reason.
>>
>> Yes, they missed the previous merge window for some reason too, while
>> the rest of the patches from the set made it in. It'd be real nice if
>> they made it into the 4.13 ...
>
> They won't. The merge window is already open.
>
> You're looking at v4.14 now.
So the rest of the patches made it into 4.12 and the MFD bit will make
it into 4.14 ... why precisely ?
>>> I'll put them back on the pile for review.
>>>
>>> Please note that the merge window is currently open, so you are likely
>>> to experience a period of inactivity until -rc1 is released.
>>
>> I'd really appreciate it if these patches made it into the 4.13 release,
>> see above.
>>
>>>>> ---
>>>>> V2: - Change BD9571MWV_AVS_VD09_VID0,1,2,3 to BD9571MWV_AVS_VD09_VID(n)
>>>>> - Change BD9571MWV_AVS_DVFS_VID0,1,2,3 to BD9571MWV_AVS_DVFS_VID(n)
>>>>> - Make the AVS_VD09 range RW, so it can be used by the regulator
>>>>> driver for the VD09 regulator
>>>>> - Report the regmap read return values when attempting to read ID
>>>>> registers fails
>>>>> V3: No change
>>>>> ---
>>>>> MAINTAINERS | 11 ++
>>>>> drivers/mfd/Kconfig | 13 +++
>>>>> drivers/mfd/Makefile | 1 +
>>>>> drivers/mfd/bd9571mwv.c | 230 ++++++++++++++++++++++++++++++++++++++++++
>>>>> include/linux/mfd/bd9571mwv.h | 115 +++++++++++++++++++++
>>>>> 5 files changed, 370 insertions(+)
>>>>> create mode 100644 drivers/mfd/bd9571mwv.c
>>>>> create mode 100644 include/linux/mfd/bd9571mwv.h
>>>>>
>>>>> diff --git a/MAINTAINERS b/MAINTAINERS
>>>>> index 2a9aa4be5846..9c30e6b00358 100644
>>>>> --- a/MAINTAINERS
>>>>> +++ b/MAINTAINERS
>>>>> @@ -10942,6 +10942,17 @@ L: linux-serial@vger.kernel.org
>>>>> S: Odd Fixes
>>>>> F: drivers/tty/serial/rp2.*
>>>>>
>>>>> +ROHM MULTIFUNCTION BD9571MWV-M PMIC DEVICE DRIVERS
>>>>> +M: Marek Vasut <marek.vasut+renesas@gmail.com>
>>>>> +L: linux-kernel@vger.kernel.org
>>>>> +L: linux-renesas-soc@vger.kernel.org
>>>>> +S: Supported
>>>>> +F: drivers/mfd/bd9571mwv.c
>>>>> +F: drivers/regulator/bd9571mwv-regulator.c
>>>>> +F: drivers/gpio/gpio-bd9571mwv.c
>>>>> +F: include/linux/mfd/bd9571mwv.h
>>>>> +F: Documentation/devicetree/bindings/mfd/bd9571mwv.txt
>>>>> +
>>>>> ROSE NETWORK LAYER
>>>>> M: Ralf Baechle <ralf@linux-mips.org>
>>>>> L: linux-hams@vger.kernel.org
>>>>> diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig
>>>>> index 3eb5c93595f6..6f4668ae28cc 100644
>>>>> --- a/drivers/mfd/Kconfig
>>>>> +++ b/drivers/mfd/Kconfig
>>>>> @@ -133,6 +133,19 @@ config MFD_BCM590XX
>>>>> help
>>>>> Support for the BCM590xx PMUs from Broadcom
>>>>>
>>>>> +config MFD_BD9571MWV
>>>>> + tristate "ROHM BD9571MWV PMIC"
>>>>> + select MFD_CORE
>>>>> + select REGMAP_I2C
>>>>> + depends on I2C
>>>>> + help
>>>>> + Support for the ROHM BD9571MWV PMIC, which contains single
>>>>> + voltage regulator, voltage sampling units, GPIO block and
>>>>> + watchdog block.
>>>>> +
>>>>> + This driver can also be built as a module. If so, the module
>>>>> + will be called bd9571mwv.
>>>>> +
>>>>> config MFD_AC100
>>>>> tristate "X-Powers AC100"
>>>>> select MFD_CORE
>>>>> diff --git a/drivers/mfd/Makefile b/drivers/mfd/Makefile
>>>>> index c16bf1ea0ea9..3cacc9748e13 100644
>>>>> --- a/drivers/mfd/Makefile
>>>>> +++ b/drivers/mfd/Makefile
>>>>> @@ -10,6 +10,7 @@ obj-$(CONFIG_MFD_ACT8945A) += act8945a.o
>>>>> obj-$(CONFIG_MFD_SM501) += sm501.o
>>>>> obj-$(CONFIG_MFD_ASIC3) += asic3.o tmio_core.o
>>>>> obj-$(CONFIG_MFD_BCM590XX) += bcm590xx.o
>>>>> +obj-$(CONFIG_MFD_BD9571MWV) += bd9571mwv.o
>>>>> cros_ec_core-objs := cros_ec.o
>>>>> cros_ec_core-$(CONFIG_ACPI) += cros_ec_acpi_gpe.o
>>>>> obj-$(CONFIG_MFD_CROS_EC) += cros_ec_core.o
>>>>> diff --git a/drivers/mfd/bd9571mwv.c b/drivers/mfd/bd9571mwv.c
>>>>> new file mode 100644
>>>>> index 000000000000..64e088dfe7b0
>>>>> --- /dev/null
>>>>> +++ b/drivers/mfd/bd9571mwv.c
>>>>> @@ -0,0 +1,230 @@
>>>>> +/*
>>>>> + * ROHM BD9571MWV-M MFD driver
>>>>> + *
>>>>> + * Copyright (C) 2017 Marek Vasut <marek.vasut+renesas@gmail.com>
>>>>> + *
>>>>> + * This program is free software; you can redistribute it and/or
>>>>> + * modify it under the terms of the GNU General Public License version 2 as
>>>>> + * published by the Free Software Foundation.
>>>>> + *
>>>>> + * This program is distributed "as is" WITHOUT ANY WARRANTY of any
>>>>> + * kind, whether expressed or implied; without even the implied warranty
>>>>> + * of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
>>>>> + * GNU General Public License version 2 for more details.
>>>>> + *
>>>>> + * Based on the TPS65086 driver
>>>>> + */
>>>>> +
>>>>> +#include <linux/i2c.h>
>>>>> +#include <linux/interrupt.h>
>>>>> +#include <linux/mfd/core.h>
>>>>> +#include <linux/module.h>
>>>>> +
>>>>> +#include <linux/mfd/bd9571mwv.h>
>>>>> +
>>>>> +static const struct mfd_cell bd9571mwv_cells[] = {
>>>>> + { .name = "bd9571mwv-regulator", },
>>>>> + { .name = "bd9571mwv-gpio", },
>>>>> +};
>>>>> +
>>>>> +static const struct regmap_range bd9571mwv_readable_yes_ranges[] = {
>>>>> + regmap_reg_range(BD9571MWV_VENDOR_CODE, BD9571MWV_PRODUCT_REVISION),
>>>>> + regmap_reg_range(BD9571MWV_AVS_SET_MONI, BD9571MWV_AVS_DVFS_VID(3)),
>>>>> + regmap_reg_range(BD9571MWV_VD18_VID, BD9571MWV_VD33_VID),
>>>>> + regmap_reg_range(BD9571MWV_DVFS_VINIT, BD9571MWV_DVFS_VINIT),
>>>>> + regmap_reg_range(BD9571MWV_DVFS_SETVMAX, BD9571MWV_DVFS_MONIVDAC),
>>>>> + regmap_reg_range(BD9571MWV_GPIO_IN, BD9571MWV_GPIO_IN),
>>>>> + regmap_reg_range(BD9571MWV_GPIO_INT, BD9571MWV_GPIO_INTMASK),
>>>>> + regmap_reg_range(BD9571MWV_INT_INTREQ, BD9571MWV_INT_INTMASK),
>>>>> +};
>>>>> +
>>>>> +static const struct regmap_access_table bd9571mwv_readable_table = {
>>>>> + .yes_ranges = bd9571mwv_readable_yes_ranges,
>>>>> + .n_yes_ranges = ARRAY_SIZE(bd9571mwv_readable_yes_ranges),
>>>>> +};
>>>>> +
>>>>> +static const struct regmap_range bd9571mwv_writable_yes_ranges[] = {
>>>>> + regmap_reg_range(BD9571MWV_AVS_VD09_VID(0), BD9571MWV_AVS_VD09_VID(3)),
>>>>> + regmap_reg_range(BD9571MWV_DVFS_SETVID, BD9571MWV_DVFS_SETVID),
>>>>> + regmap_reg_range(BD9571MWV_GPIO_DIR, BD9571MWV_GPIO_OUT),
>>>>> + regmap_reg_range(BD9571MWV_GPIO_INT_SET, BD9571MWV_GPIO_INTMASK),
>>>>> + regmap_reg_range(BD9571MWV_INT_INTREQ, BD9571MWV_INT_INTMASK),
>>>>> +};
>>>>> +
>>>>> +static const struct regmap_access_table bd9571mwv_writable_table = {
>>>>> + .yes_ranges = bd9571mwv_writable_yes_ranges,
>>>>> + .n_yes_ranges = ARRAY_SIZE(bd9571mwv_writable_yes_ranges),
>>>>> +};
>>>>> +
>>>>> +static const struct regmap_range bd9571mwv_volatile_yes_ranges[] = {
>>>>> + regmap_reg_range(BD9571MWV_GPIO_IN, BD9571MWV_GPIO_IN),
>>>>> + regmap_reg_range(BD9571MWV_GPIO_INT, BD9571MWV_GPIO_INT),
>>>>> + regmap_reg_range(BD9571MWV_INT_INTREQ, BD9571MWV_INT_INTREQ),
>>>>> +};
>>>>> +
>>>>> +static const struct regmap_access_table bd9571mwv_volatile_table = {
>>>>> + .yes_ranges = bd9571mwv_volatile_yes_ranges,
>>>>> + .n_yes_ranges = ARRAY_SIZE(bd9571mwv_volatile_yes_ranges),
>>>>> +};
>>>>> +
>>>>> +static const struct regmap_config bd9571mwv_regmap_config = {
>>>>> + .reg_bits = 8,
>>>>> + .val_bits = 8,
>>>>> + .cache_type = REGCACHE_RBTREE,
>>>>> + .rd_table = &bd9571mwv_readable_table,
>>>>> + .wr_table = &bd9571mwv_writable_table,
>>>>> + .volatile_table = &bd9571mwv_volatile_table,
>>>>> + .max_register = 0xff,
>>>>> +};
>>>>> +
>>>>> +static const struct regmap_irq bd9571mwv_irqs[] = {
>>>>> + REGMAP_IRQ_REG(BD9571MWV_IRQ_MD1, 0,
>>>>> + BD9571MWV_INT_INTREQ_MD1_INT),
>>>>> + REGMAP_IRQ_REG(BD9571MWV_IRQ_MD2_E1, 0,
>>>>> + BD9571MWV_INT_INTREQ_MD2_E1_INT),
>>>>> + REGMAP_IRQ_REG(BD9571MWV_IRQ_MD2_E2, 0,
>>>>> + BD9571MWV_INT_INTREQ_MD2_E2_INT),
>>>>> + REGMAP_IRQ_REG(BD9571MWV_IRQ_PROT_ERR, 0,
>>>>> + BD9571MWV_INT_INTREQ_PROT_ERR_INT),
>>>>> + REGMAP_IRQ_REG(BD9571MWV_IRQ_GP, 0,
>>>>> + BD9571MWV_INT_INTREQ_GP_INT),
>>>>> + REGMAP_IRQ_REG(BD9571MWV_IRQ_128H_OF, 0,
>>>>> + BD9571MWV_INT_INTREQ_128H_OF_INT),
>>>>> + REGMAP_IRQ_REG(BD9571MWV_IRQ_WDT_OF, 0,
>>>>> + BD9571MWV_INT_INTREQ_WDT_OF_INT),
>>>>> + REGMAP_IRQ_REG(BD9571MWV_IRQ_BKUP_TRG, 0,
>>>>> + BD9571MWV_INT_INTREQ_BKUP_TRG_INT),
>>>>> +};
>>>>> +
>>>>> +static struct regmap_irq_chip bd9571mwv_irq_chip = {
>>>>> + .name = "bd9571mwv",
>>>>> + .status_base = BD9571MWV_INT_INTREQ,
>>>>> + .mask_base = BD9571MWV_INT_INTMASK,
>>>>> + .ack_base = BD9571MWV_INT_INTREQ,
>>>>> + .init_ack_masked = true,
>>>>> + .num_regs = 1,
>>>>> + .irqs = bd9571mwv_irqs,
>>>>> + .num_irqs = ARRAY_SIZE(bd9571mwv_irqs),
>>>>> +};
>>>>> +
>>>>> +static int bd9571mwv_identify(struct bd9571mwv *bd)
>>>>> +{
>>>>> + struct device *dev = bd->dev;
>>>>> + unsigned int value;
>>>>> + int ret;
>>>>> +
>>>>> + ret = regmap_read(bd->regmap, BD9571MWV_VENDOR_CODE, &value);
>>>>> + if (ret) {
>>>>> + dev_err(dev, "Failed to read vendor code register (ret=%i)\n",
>>>>> + ret);
>>>>> + return ret;
>>>>> + }
>>>>> +
>>>>> + if (value != BD9571MWV_VENDOR_CODE_VAL) {
>>>>> + dev_err(dev, "Invalid vendor code ID %02x (expected %02x)\n",
>>>>> + value, BD9571MWV_VENDOR_CODE_VAL);
>>>>> + return -EINVAL;
>>>>> + }
>>>>> +
>>>>> + ret = regmap_read(bd->regmap, BD9571MWV_PRODUCT_CODE, &value);
>>>>> + if (ret) {
>>>>> + dev_err(dev, "Failed to read product code register (ret=%i)\n",
>>>>> + ret);
>>>>> + return ret;
>>>>> + }
>>>>> +
>>>>> + if (value != BD9571MWV_PRODUCT_CODE_VAL) {
>>>>> + dev_err(dev, "Invalid product code ID %02x (expected %02x)\n",
>>>>> + value, BD9571MWV_PRODUCT_CODE_VAL);
>>>>> + return -EINVAL;
>>>>> + }
>>>>> +
>>>>> + ret = regmap_read(bd->regmap, BD9571MWV_PRODUCT_REVISION, &value);
>>>>> + if (ret) {
>>>>> + dev_err(dev, "Failed to read revision register (ret=%i)\n",
>>>>> + ret);
>>>>> + return ret;
>>>>> + }
>>>>> +
>>>>> + dev_info(dev, "Device: BD9571MWV rev. %d\n", value & 0xff);
>>>>> +
>>>>> + return 0;
>>>>> +}
>>>>> +
>>>>> +static int bd9571mwv_probe(struct i2c_client *client,
>>>>> + const struct i2c_device_id *ids)
>>>>> +{
>>>>> + struct bd9571mwv *bd;
>>>>> + int ret;
>>>>> +
>>>>> + bd = devm_kzalloc(&client->dev, sizeof(*bd), GFP_KERNEL);
>>>>> + if (!bd)
>>>>> + return -ENOMEM;
>>>>> +
>>>>> + i2c_set_clientdata(client, bd);
>>>>> + bd->dev = &client->dev;
>>>>> + bd->irq = client->irq;
>>>>> +
>>>>> + bd->regmap = devm_regmap_init_i2c(client, &bd9571mwv_regmap_config);
>>>>> + if (IS_ERR(bd->regmap)) {
>>>>> + dev_err(bd->dev, "Failed to initialize register map\n");
>>>>> + return PTR_ERR(bd->regmap);
>>>>> + }
>>>>> +
>>>>> + ret = bd9571mwv_identify(bd);
>>>>> + if (ret)
>>>>> + return ret;
>>>>> +
>>>>> + ret = regmap_add_irq_chip(bd->regmap, bd->irq, IRQF_ONESHOT, 0,
>>>>> + &bd9571mwv_irq_chip, &bd->irq_data);
>>>>> + if (ret) {
>>>>> + dev_err(bd->dev, "Failed to register IRQ chip\n");
>>>>> + return ret;
>>>>> + }
>>>>> +
>>>>> + ret = mfd_add_devices(bd->dev, PLATFORM_DEVID_AUTO, bd9571mwv_cells,
>>>>> + ARRAY_SIZE(bd9571mwv_cells), NULL, 0,
>>>>> + regmap_irq_get_domain(bd->irq_data));
>>>>> + if (ret) {
>>>>> + regmap_del_irq_chip(bd->irq, bd->irq_data);
>>>>> + return ret;
>>>>> + }
>>>>> +
>>>>> + return 0;
>>>>> +}
>>>>> +
>>>>> +static int bd9571mwv_remove(struct i2c_client *client)
>>>>> +{
>>>>> + struct bd9571mwv *bd = i2c_get_clientdata(client);
>>>>> +
>>>>> + regmap_del_irq_chip(bd->irq, bd->irq_data);
>>>>> +
>>>>> + return 0;
>>>>> +}
>>>>> +
>>>>> +static const struct of_device_id bd9571mwv_of_match_table[] = {
>>>>> + { .compatible = "rohm,bd9571mwv", },
>>>>> + { /* sentinel */ }
>>>>> +};
>>>>> +MODULE_DEVICE_TABLE(of, bd9571mwv_of_match_table);
>>>>> +
>>>>> +static const struct i2c_device_id bd9571mwv_id_table[] = {
>>>>> + { "bd9571mwv", 0 },
>>>>> + { /* sentinel */ }
>>>>> +};
>>>>> +MODULE_DEVICE_TABLE(i2c, bd9571mwv_id_table);
>>>>> +
>>>>> +static struct i2c_driver bd9571mwv_driver = {
>>>>> + .driver = {
>>>>> + .name = "bd9571mwv",
>>>>> + .of_match_table = bd9571mwv_of_match_table,
>>>>> + },
>>>>> + .probe = bd9571mwv_probe,
>>>>> + .remove = bd9571mwv_remove,
>>>>> + .id_table = bd9571mwv_id_table,
>>>>> +};
>>>>> +module_i2c_driver(bd9571mwv_driver);
>>>>> +
>>>>> +MODULE_AUTHOR("Marek Vasut <marek.vasut+renesas@gmail.com>");
>>>>> +MODULE_DESCRIPTION("BD9571MWV PMIC Driver");
>>>>> +MODULE_LICENSE("GPL v2");
>>>>> diff --git a/include/linux/mfd/bd9571mwv.h b/include/linux/mfd/bd9571mwv.h
>>>>> new file mode 100644
>>>>> index 000000000000..f0708ba4cbba
>>>>> --- /dev/null
>>>>> +++ b/include/linux/mfd/bd9571mwv.h
>>>>> @@ -0,0 +1,115 @@
>>>>> +/*
>>>>> + * ROHM BD9571MWV-M driver
>>>>> + *
>>>>> + * Copyright (C) 2017 Marek Vasut <marek.vasut+renesas@gmail.com>
>>>>> + *
>>>>> + * This program is free software; you can redistribute it and/or
>>>>> + * modify it under the terms of the GNU General Public License version 2 as
>>>>> + * published by the Free Software Foundation.
>>>>> + *
>>>>> + * This program is distributed "as is" WITHOUT ANY WARRANTY of any
>>>>> + * kind, whether expressed or implied; without even the implied warranty
>>>>> + * of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
>>>>> + * GNU General Public License version 2 for more details.
>>>>> + *
>>>>> + * Based on the TPS65086 driver
>>>>> + */
>>>>> +
>>>>> +#ifndef __LINUX_MFD_BD9571MWV_H
>>>>> +#define __LINUX_MFD_BD9571MWV_H
>>>>> +
>>>>> +#include <linux/device.h>
>>>>> +#include <linux/regmap.h>
>>>>> +
>>>>> +/* List of registers for BD9571MWV */
>>>>> +#define BD9571MWV_VENDOR_CODE 0x00
>>>>> +#define BD9571MWV_VENDOR_CODE_VAL 0xdb
>>>>> +#define BD9571MWV_PRODUCT_CODE 0x01
>>>>> +#define BD9571MWV_PRODUCT_CODE_VAL 0x60
>>>>> +#define BD9571MWV_PRODUCT_REVISION 0x02
>>>>> +
>>>>> +#define BD9571MWV_I2C_FUSA_MODE 0x10
>>>>> +#define BD9571MWV_I2C_MD2_E1_BIT_1 0x11
>>>>> +#define BD9571MWV_I2C_MD2_E1_BIT_2 0x12
>>>>> +
>>>>> +#define BD9571MWV_BKUP_MODE_CNT 0x20
>>>>> +#define BD9571MWV_BKUP_MODE_STATUS 0x21
>>>>> +#define BD9571MWV_BKUP_RECOVERY_CNT 0x22
>>>>> +#define BD9571MWV_BKUP_CTRL_TIM_CNT 0x23
>>>>> +#define BD9571MWV_WAITBKUP_WDT_CNT 0x24
>>>>> +#define BD9571MWV_128H_TIM_CNT 0x26
>>>>> +#define BD9571MWV_QLLM_CNT 0x27
>>>>> +
>>>>> +#define BD9571MWV_AVS_SET_MONI 0x31
>>>>> +#define BD9571MWV_AVS_SET_MONI_MASK 0x3
>>>>> +#define BD9571MWV_AVS_VD09_VID(n) (0x32 + (n))
>>>>> +#define BD9571MWV_AVS_DVFS_VID(n) (0x36 + (n))
>>>>> +
>>>>> +#define BD9571MWV_VD18_VID 0x42
>>>>> +#define BD9571MWV_VD25_VID 0x43
>>>>> +#define BD9571MWV_VD33_VID 0x44
>>>>> +
>>>>> +#define BD9571MWV_DVFS_VINIT 0x50
>>>>> +#define BD9571MWV_DVFS_SETVMAX 0x52
>>>>> +#define BD9571MWV_DVFS_BOOSTVID 0x53
>>>>> +#define BD9571MWV_DVFS_SETVID 0x54
>>>>> +#define BD9571MWV_DVFS_MONIVDAC 0x55
>>>>> +#define BD9571MWV_DVFS_PGD_CNT 0x56
>>>>> +
>>>>> +#define BD9571MWV_GPIO_DIR 0x60
>>>>> +#define BD9571MWV_GPIO_OUT 0x61
>>>>> +#define BD9571MWV_GPIO_IN 0x62
>>>>> +#define BD9571MWV_GPIO_DEB 0x63
>>>>> +#define BD9571MWV_GPIO_INT_SET 0x64
>>>>> +#define BD9571MWV_GPIO_INT 0x65
>>>>> +#define BD9571MWV_GPIO_INTMASK 0x66
>>>>> +
>>>>> +#define BD9571MWV_REG_KEEP(n) (0x70 + (n))
>>>>> +
>>>>> +#define BD9571MWV_PMIC_INTERNAL_STATUS 0x80
>>>>> +#define BD9571MWV_PROT_ERROR_STATUS0 0x81
>>>>> +#define BD9571MWV_PROT_ERROR_STATUS1 0x82
>>>>> +#define BD9571MWV_PROT_ERROR_STATUS2 0x83
>>>>> +#define BD9571MWV_PROT_ERROR_STATUS3 0x84
>>>>> +#define BD9571MWV_PROT_ERROR_STATUS4 0x85
>>>>> +
>>>>> +#define BD9571MWV_INT_INTREQ 0x90
>>>>> +#define BD9571MWV_INT_INTREQ_MD1_INT BIT(0)
>>>>> +#define BD9571MWV_INT_INTREQ_MD2_E1_INT BIT(1)
>>>>> +#define BD9571MWV_INT_INTREQ_MD2_E2_INT BIT(2)
>>>>> +#define BD9571MWV_INT_INTREQ_PROT_ERR_INT BIT(3)
>>>>> +#define BD9571MWV_INT_INTREQ_GP_INT BIT(4)
>>>>> +#define BD9571MWV_INT_INTREQ_128H_OF_INT BIT(5)
>>>>> +#define BD9571MWV_INT_INTREQ_WDT_OF_INT BIT(6)
>>>>> +#define BD9571MWV_INT_INTREQ_BKUP_TRG_INT BIT(7)
>>>>> +#define BD9571MWV_INT_INTMASK 0x91
>>>>> +
>>>>> +#define BD9571MWV_ACCESS_KEY 0xff
>>>>> +
>>>>> +/* Define the BD9571MWV IRQ numbers */
>>>>> +enum bd9571mwv_irqs {
>>>>> + BD9571MWV_IRQ_MD1,
>>>>> + BD9571MWV_IRQ_MD2_E1,
>>>>> + BD9571MWV_IRQ_MD2_E2,
>>>>> + BD9571MWV_IRQ_PROT_ERR,
>>>>> + BD9571MWV_IRQ_GP,
>>>>> + BD9571MWV_IRQ_128H_OF,
>>>>> + BD9571MWV_IRQ_WDT_OF,
>>>>> + BD9571MWV_IRQ_BKUP_TRG,
>>>>> +};
>>>>> +
>>>>> +/**
>>>>> + * struct bd9571mwv - state holder for the bd9571mwv driver
>>>>> + *
>>>>> + * Device data may be used to access the BD9571MWV chip
>>>>> + */
>>>>> +struct bd9571mwv {
>>>>> + struct device *dev;
>>>>> + struct regmap *regmap;
>>>>> +
>>>>> + /* IRQ Data */
>>>>> + int irq;
>>>>> + struct regmap_irq_chip_data *irq_data;
>>>>> +};
>>>>> +
>>>>> +#endif /* __LINUX_MFD_BD9571MWV_H */
>>>>>
>>>>
>>>>
>>>
>>
>>
>
--
Best regards,
Marek Vasut
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH V3 2/2] mfd: Add ROHM BD9571MWV-M MFD PMIC driver
2017-07-03 14:16 ` Marek Vasut
@ 2017-07-03 15:21 ` Lee Jones
0 siblings, 0 replies; 9+ messages in thread
From: Lee Jones @ 2017-07-03 15:21 UTC (permalink / raw)
To: Marek Vasut
Cc: linux-renesas-soc, Marek Vasut, linux-kernel, Geert Uytterhoeven
On Mon, 03 Jul 2017, Marek Vasut wrote:
> On 07/03/2017 03:48 PM, Lee Jones wrote:
> > On Mon, 03 Jul 2017, Marek Vasut wrote:
> >
> >> On 07/03/2017 01:55 PM, Lee Jones wrote:
> >>> On Tue, 27 Jun 2017, Marek Vasut wrote:
> >>>
> >>>> On 05/02/2017 02:18 PM, Marek Vasut wrote:
> >>>>> Add the MFD part of the ROHM BD9571MWV-M PMIC driver and MAINTAINERS
> >>>>> entry. The MFD part only specifies the regmap bits for the PMIC and
> >>>>> binds the subdevs together.
> >>>>>
> >>>>> Signed-off-by: Marek Vasut <marek.vasut+renesas@gmail.com>
> >>>>> Cc: linux-kernel@vger.kernel.org
> >>>>> Cc: Geert Uytterhoeven <geert+renesas@glider.be>
> >>>>> Cc: Lee Jones <lee.jones@linaro.org>
> >>>>
> >>>> Lee, bump, do you plan to apply these patches anytime soon ?
> >>>
> >>> Looks like these were missed for some reason.
> >>
> >> Yes, they missed the previous merge window for some reason too, while
> >> the rest of the patches from the set made it in. It'd be real nice if
> >> they made it into the 4.13 ...
> >
> > They won't. The merge window is already open.
> >
> > You're looking at v4.14 now.
>
> So the rest of the patches made it into 4.12 and the MFD bit will make
> it into 4.14 ... why precisely ?
Mark applied the regulator patch before I was even aware the patch-set
existed. The first patch you sent to me was on the 24th of April. I
guess you forgot to Cc me on patches originating before that date?
The v4.12 merge-window opened on the 30th of April, so your patch
missed the deadline.
Linus applied the GPIO patch to the v4.12 -rcs. He shouldn't have
done that really, but justified it well enough. I could not apply the
MFD patch at this time because RobH still had review comments
outstanding.
You sent v3 on the 2nd May whilst we were at -rc5. Since this is
late in the cycle, there is a chance for patches to be applied, but
that is not guaranteed. Unfortunately for you this also coincided
with my vacation. You sent a bump 5 days before the merge window was
about it open, which is way too late in the cycle to be applying
patches.
Unfortunately on this occasion, you were a victim of circumstance and
poor timing.
1.5 cycles to have your patches applied is still perfectly
acceptable. Especially after 3 revisions.
> >>> I'll put them back on the pile for review.
> >>>
> >>> Please note that the merge window is currently open, so you are likely
> >>> to experience a period of inactivity until -rc1 is released.
> >>
> >> I'd really appreciate it if these patches made it into the 4.13 release,
> >> see above.
> >>
> >>>>> ---
> >>>>> V2: - Change BD9571MWV_AVS_VD09_VID0,1,2,3 to BD9571MWV_AVS_VD09_VID(n)
> >>>>> - Change BD9571MWV_AVS_DVFS_VID0,1,2,3 to BD9571MWV_AVS_DVFS_VID(n)
> >>>>> - Make the AVS_VD09 range RW, so it can be used by the regulator
> >>>>> driver for the VD09 regulator
> >>>>> - Report the regmap read return values when attempting to read ID
> >>>>> registers fails
> >>>>> V3: No change
> >>>>> ---
> >>>>> MAINTAINERS | 11 ++
> >>>>> drivers/mfd/Kconfig | 13 +++
> >>>>> drivers/mfd/Makefile | 1 +
> >>>>> drivers/mfd/bd9571mwv.c | 230 ++++++++++++++++++++++++++++++++++++++++++
> >>>>> include/linux/mfd/bd9571mwv.h | 115 +++++++++++++++++++++
> >>>>> 5 files changed, 370 insertions(+)
> >>>>> create mode 100644 drivers/mfd/bd9571mwv.c
> >>>>> create mode 100644 include/linux/mfd/bd9571mwv.h
> >>>>>
> >>>>> diff --git a/MAINTAINERS b/MAINTAINERS
> >>>>> index 2a9aa4be5846..9c30e6b00358 100644
> >>>>> --- a/MAINTAINERS
> >>>>> +++ b/MAINTAINERS
> >>>>> @@ -10942,6 +10942,17 @@ L: linux-serial@vger.kernel.org
> >>>>> S: Odd Fixes
> >>>>> F: drivers/tty/serial/rp2.*
> >>>>>
> >>>>> +ROHM MULTIFUNCTION BD9571MWV-M PMIC DEVICE DRIVERS
> >>>>> +M: Marek Vasut <marek.vasut+renesas@gmail.com>
> >>>>> +L: linux-kernel@vger.kernel.org
> >>>>> +L: linux-renesas-soc@vger.kernel.org
> >>>>> +S: Supported
> >>>>> +F: drivers/mfd/bd9571mwv.c
> >>>>> +F: drivers/regulator/bd9571mwv-regulator.c
> >>>>> +F: drivers/gpio/gpio-bd9571mwv.c
> >>>>> +F: include/linux/mfd/bd9571mwv.h
> >>>>> +F: Documentation/devicetree/bindings/mfd/bd9571mwv.txt
> >>>>> +
> >>>>> ROSE NETWORK LAYER
> >>>>> M: Ralf Baechle <ralf@linux-mips.org>
> >>>>> L: linux-hams@vger.kernel.org
> >>>>> diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig
> >>>>> index 3eb5c93595f6..6f4668ae28cc 100644
> >>>>> --- a/drivers/mfd/Kconfig
> >>>>> +++ b/drivers/mfd/Kconfig
> >>>>> @@ -133,6 +133,19 @@ config MFD_BCM590XX
> >>>>> help
> >>>>> Support for the BCM590xx PMUs from Broadcom
> >>>>>
> >>>>> +config MFD_BD9571MWV
> >>>>> + tristate "ROHM BD9571MWV PMIC"
> >>>>> + select MFD_CORE
> >>>>> + select REGMAP_I2C
> >>>>> + depends on I2C
> >>>>> + help
> >>>>> + Support for the ROHM BD9571MWV PMIC, which contains single
> >>>>> + voltage regulator, voltage sampling units, GPIO block and
> >>>>> + watchdog block.
> >>>>> +
> >>>>> + This driver can also be built as a module. If so, the module
> >>>>> + will be called bd9571mwv.
> >>>>> +
> >>>>> config MFD_AC100
> >>>>> tristate "X-Powers AC100"
> >>>>> select MFD_CORE
> >>>>> diff --git a/drivers/mfd/Makefile b/drivers/mfd/Makefile
> >>>>> index c16bf1ea0ea9..3cacc9748e13 100644
> >>>>> --- a/drivers/mfd/Makefile
> >>>>> +++ b/drivers/mfd/Makefile
> >>>>> @@ -10,6 +10,7 @@ obj-$(CONFIG_MFD_ACT8945A) += act8945a.o
> >>>>> obj-$(CONFIG_MFD_SM501) += sm501.o
> >>>>> obj-$(CONFIG_MFD_ASIC3) += asic3.o tmio_core.o
> >>>>> obj-$(CONFIG_MFD_BCM590XX) += bcm590xx.o
> >>>>> +obj-$(CONFIG_MFD_BD9571MWV) += bd9571mwv.o
> >>>>> cros_ec_core-objs := cros_ec.o
> >>>>> cros_ec_core-$(CONFIG_ACPI) += cros_ec_acpi_gpe.o
> >>>>> obj-$(CONFIG_MFD_CROS_EC) += cros_ec_core.o
> >>>>> diff --git a/drivers/mfd/bd9571mwv.c b/drivers/mfd/bd9571mwv.c
> >>>>> new file mode 100644
> >>>>> index 000000000000..64e088dfe7b0
> >>>>> --- /dev/null
> >>>>> +++ b/drivers/mfd/bd9571mwv.c
> >>>>> @@ -0,0 +1,230 @@
> >>>>> +/*
> >>>>> + * ROHM BD9571MWV-M MFD driver
> >>>>> + *
> >>>>> + * Copyright (C) 2017 Marek Vasut <marek.vasut+renesas@gmail.com>
> >>>>> + *
> >>>>> + * This program is free software; you can redistribute it and/or
> >>>>> + * modify it under the terms of the GNU General Public License version 2 as
> >>>>> + * published by the Free Software Foundation.
> >>>>> + *
> >>>>> + * This program is distributed "as is" WITHOUT ANY WARRANTY of any
> >>>>> + * kind, whether expressed or implied; without even the implied warranty
> >>>>> + * of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> >>>>> + * GNU General Public License version 2 for more details.
> >>>>> + *
> >>>>> + * Based on the TPS65086 driver
> >>>>> + */
> >>>>> +
> >>>>> +#include <linux/i2c.h>
> >>>>> +#include <linux/interrupt.h>
> >>>>> +#include <linux/mfd/core.h>
> >>>>> +#include <linux/module.h>
> >>>>> +
> >>>>> +#include <linux/mfd/bd9571mwv.h>
> >>>>> +
> >>>>> +static const struct mfd_cell bd9571mwv_cells[] = {
> >>>>> + { .name = "bd9571mwv-regulator", },
> >>>>> + { .name = "bd9571mwv-gpio", },
> >>>>> +};
> >>>>> +
> >>>>> +static const struct regmap_range bd9571mwv_readable_yes_ranges[] = {
> >>>>> + regmap_reg_range(BD9571MWV_VENDOR_CODE, BD9571MWV_PRODUCT_REVISION),
> >>>>> + regmap_reg_range(BD9571MWV_AVS_SET_MONI, BD9571MWV_AVS_DVFS_VID(3)),
> >>>>> + regmap_reg_range(BD9571MWV_VD18_VID, BD9571MWV_VD33_VID),
> >>>>> + regmap_reg_range(BD9571MWV_DVFS_VINIT, BD9571MWV_DVFS_VINIT),
> >>>>> + regmap_reg_range(BD9571MWV_DVFS_SETVMAX, BD9571MWV_DVFS_MONIVDAC),
> >>>>> + regmap_reg_range(BD9571MWV_GPIO_IN, BD9571MWV_GPIO_IN),
> >>>>> + regmap_reg_range(BD9571MWV_GPIO_INT, BD9571MWV_GPIO_INTMASK),
> >>>>> + regmap_reg_range(BD9571MWV_INT_INTREQ, BD9571MWV_INT_INTMASK),
> >>>>> +};
> >>>>> +
> >>>>> +static const struct regmap_access_table bd9571mwv_readable_table = {
> >>>>> + .yes_ranges = bd9571mwv_readable_yes_ranges,
> >>>>> + .n_yes_ranges = ARRAY_SIZE(bd9571mwv_readable_yes_ranges),
> >>>>> +};
> >>>>> +
> >>>>> +static const struct regmap_range bd9571mwv_writable_yes_ranges[] = {
> >>>>> + regmap_reg_range(BD9571MWV_AVS_VD09_VID(0), BD9571MWV_AVS_VD09_VID(3)),
> >>>>> + regmap_reg_range(BD9571MWV_DVFS_SETVID, BD9571MWV_DVFS_SETVID),
> >>>>> + regmap_reg_range(BD9571MWV_GPIO_DIR, BD9571MWV_GPIO_OUT),
> >>>>> + regmap_reg_range(BD9571MWV_GPIO_INT_SET, BD9571MWV_GPIO_INTMASK),
> >>>>> + regmap_reg_range(BD9571MWV_INT_INTREQ, BD9571MWV_INT_INTMASK),
> >>>>> +};
> >>>>> +
> >>>>> +static const struct regmap_access_table bd9571mwv_writable_table = {
> >>>>> + .yes_ranges = bd9571mwv_writable_yes_ranges,
> >>>>> + .n_yes_ranges = ARRAY_SIZE(bd9571mwv_writable_yes_ranges),
> >>>>> +};
> >>>>> +
> >>>>> +static const struct regmap_range bd9571mwv_volatile_yes_ranges[] = {
> >>>>> + regmap_reg_range(BD9571MWV_GPIO_IN, BD9571MWV_GPIO_IN),
> >>>>> + regmap_reg_range(BD9571MWV_GPIO_INT, BD9571MWV_GPIO_INT),
> >>>>> + regmap_reg_range(BD9571MWV_INT_INTREQ, BD9571MWV_INT_INTREQ),
> >>>>> +};
> >>>>> +
> >>>>> +static const struct regmap_access_table bd9571mwv_volatile_table = {
> >>>>> + .yes_ranges = bd9571mwv_volatile_yes_ranges,
> >>>>> + .n_yes_ranges = ARRAY_SIZE(bd9571mwv_volatile_yes_ranges),
> >>>>> +};
> >>>>> +
> >>>>> +static const struct regmap_config bd9571mwv_regmap_config = {
> >>>>> + .reg_bits = 8,
> >>>>> + .val_bits = 8,
> >>>>> + .cache_type = REGCACHE_RBTREE,
> >>>>> + .rd_table = &bd9571mwv_readable_table,
> >>>>> + .wr_table = &bd9571mwv_writable_table,
> >>>>> + .volatile_table = &bd9571mwv_volatile_table,
> >>>>> + .max_register = 0xff,
> >>>>> +};
> >>>>> +
> >>>>> +static const struct regmap_irq bd9571mwv_irqs[] = {
> >>>>> + REGMAP_IRQ_REG(BD9571MWV_IRQ_MD1, 0,
> >>>>> + BD9571MWV_INT_INTREQ_MD1_INT),
> >>>>> + REGMAP_IRQ_REG(BD9571MWV_IRQ_MD2_E1, 0,
> >>>>> + BD9571MWV_INT_INTREQ_MD2_E1_INT),
> >>>>> + REGMAP_IRQ_REG(BD9571MWV_IRQ_MD2_E2, 0,
> >>>>> + BD9571MWV_INT_INTREQ_MD2_E2_INT),
> >>>>> + REGMAP_IRQ_REG(BD9571MWV_IRQ_PROT_ERR, 0,
> >>>>> + BD9571MWV_INT_INTREQ_PROT_ERR_INT),
> >>>>> + REGMAP_IRQ_REG(BD9571MWV_IRQ_GP, 0,
> >>>>> + BD9571MWV_INT_INTREQ_GP_INT),
> >>>>> + REGMAP_IRQ_REG(BD9571MWV_IRQ_128H_OF, 0,
> >>>>> + BD9571MWV_INT_INTREQ_128H_OF_INT),
> >>>>> + REGMAP_IRQ_REG(BD9571MWV_IRQ_WDT_OF, 0,
> >>>>> + BD9571MWV_INT_INTREQ_WDT_OF_INT),
> >>>>> + REGMAP_IRQ_REG(BD9571MWV_IRQ_BKUP_TRG, 0,
> >>>>> + BD9571MWV_INT_INTREQ_BKUP_TRG_INT),
> >>>>> +};
> >>>>> +
> >>>>> +static struct regmap_irq_chip bd9571mwv_irq_chip = {
> >>>>> + .name = "bd9571mwv",
> >>>>> + .status_base = BD9571MWV_INT_INTREQ,
> >>>>> + .mask_base = BD9571MWV_INT_INTMASK,
> >>>>> + .ack_base = BD9571MWV_INT_INTREQ,
> >>>>> + .init_ack_masked = true,
> >>>>> + .num_regs = 1,
> >>>>> + .irqs = bd9571mwv_irqs,
> >>>>> + .num_irqs = ARRAY_SIZE(bd9571mwv_irqs),
> >>>>> +};
> >>>>> +
> >>>>> +static int bd9571mwv_identify(struct bd9571mwv *bd)
> >>>>> +{
> >>>>> + struct device *dev = bd->dev;
> >>>>> + unsigned int value;
> >>>>> + int ret;
> >>>>> +
> >>>>> + ret = regmap_read(bd->regmap, BD9571MWV_VENDOR_CODE, &value);
> >>>>> + if (ret) {
> >>>>> + dev_err(dev, "Failed to read vendor code register (ret=%i)\n",
> >>>>> + ret);
> >>>>> + return ret;
> >>>>> + }
> >>>>> +
> >>>>> + if (value != BD9571MWV_VENDOR_CODE_VAL) {
> >>>>> + dev_err(dev, "Invalid vendor code ID %02x (expected %02x)\n",
> >>>>> + value, BD9571MWV_VENDOR_CODE_VAL);
> >>>>> + return -EINVAL;
> >>>>> + }
> >>>>> +
> >>>>> + ret = regmap_read(bd->regmap, BD9571MWV_PRODUCT_CODE, &value);
> >>>>> + if (ret) {
> >>>>> + dev_err(dev, "Failed to read product code register (ret=%i)\n",
> >>>>> + ret);
> >>>>> + return ret;
> >>>>> + }
> >>>>> +
> >>>>> + if (value != BD9571MWV_PRODUCT_CODE_VAL) {
> >>>>> + dev_err(dev, "Invalid product code ID %02x (expected %02x)\n",
> >>>>> + value, BD9571MWV_PRODUCT_CODE_VAL);
> >>>>> + return -EINVAL;
> >>>>> + }
> >>>>> +
> >>>>> + ret = regmap_read(bd->regmap, BD9571MWV_PRODUCT_REVISION, &value);
> >>>>> + if (ret) {
> >>>>> + dev_err(dev, "Failed to read revision register (ret=%i)\n",
> >>>>> + ret);
> >>>>> + return ret;
> >>>>> + }
> >>>>> +
> >>>>> + dev_info(dev, "Device: BD9571MWV rev. %d\n", value & 0xff);
> >>>>> +
> >>>>> + return 0;
> >>>>> +}
> >>>>> +
> >>>>> +static int bd9571mwv_probe(struct i2c_client *client,
> >>>>> + const struct i2c_device_id *ids)
> >>>>> +{
> >>>>> + struct bd9571mwv *bd;
> >>>>> + int ret;
> >>>>> +
> >>>>> + bd = devm_kzalloc(&client->dev, sizeof(*bd), GFP_KERNEL);
> >>>>> + if (!bd)
> >>>>> + return -ENOMEM;
> >>>>> +
> >>>>> + i2c_set_clientdata(client, bd);
> >>>>> + bd->dev = &client->dev;
> >>>>> + bd->irq = client->irq;
> >>>>> +
> >>>>> + bd->regmap = devm_regmap_init_i2c(client, &bd9571mwv_regmap_config);
> >>>>> + if (IS_ERR(bd->regmap)) {
> >>>>> + dev_err(bd->dev, "Failed to initialize register map\n");
> >>>>> + return PTR_ERR(bd->regmap);
> >>>>> + }
> >>>>> +
> >>>>> + ret = bd9571mwv_identify(bd);
> >>>>> + if (ret)
> >>>>> + return ret;
> >>>>> +
> >>>>> + ret = regmap_add_irq_chip(bd->regmap, bd->irq, IRQF_ONESHOT, 0,
> >>>>> + &bd9571mwv_irq_chip, &bd->irq_data);
> >>>>> + if (ret) {
> >>>>> + dev_err(bd->dev, "Failed to register IRQ chip\n");
> >>>>> + return ret;
> >>>>> + }
> >>>>> +
> >>>>> + ret = mfd_add_devices(bd->dev, PLATFORM_DEVID_AUTO, bd9571mwv_cells,
> >>>>> + ARRAY_SIZE(bd9571mwv_cells), NULL, 0,
> >>>>> + regmap_irq_get_domain(bd->irq_data));
> >>>>> + if (ret) {
> >>>>> + regmap_del_irq_chip(bd->irq, bd->irq_data);
> >>>>> + return ret;
> >>>>> + }
> >>>>> +
> >>>>> + return 0;
> >>>>> +}
> >>>>> +
> >>>>> +static int bd9571mwv_remove(struct i2c_client *client)
> >>>>> +{
> >>>>> + struct bd9571mwv *bd = i2c_get_clientdata(client);
> >>>>> +
> >>>>> + regmap_del_irq_chip(bd->irq, bd->irq_data);
> >>>>> +
> >>>>> + return 0;
> >>>>> +}
> >>>>> +
> >>>>> +static const struct of_device_id bd9571mwv_of_match_table[] = {
> >>>>> + { .compatible = "rohm,bd9571mwv", },
> >>>>> + { /* sentinel */ }
> >>>>> +};
> >>>>> +MODULE_DEVICE_TABLE(of, bd9571mwv_of_match_table);
> >>>>> +
> >>>>> +static const struct i2c_device_id bd9571mwv_id_table[] = {
> >>>>> + { "bd9571mwv", 0 },
> >>>>> + { /* sentinel */ }
> >>>>> +};
> >>>>> +MODULE_DEVICE_TABLE(i2c, bd9571mwv_id_table);
> >>>>> +
> >>>>> +static struct i2c_driver bd9571mwv_driver = {
> >>>>> + .driver = {
> >>>>> + .name = "bd9571mwv",
> >>>>> + .of_match_table = bd9571mwv_of_match_table,
> >>>>> + },
> >>>>> + .probe = bd9571mwv_probe,
> >>>>> + .remove = bd9571mwv_remove,
> >>>>> + .id_table = bd9571mwv_id_table,
> >>>>> +};
> >>>>> +module_i2c_driver(bd9571mwv_driver);
> >>>>> +
> >>>>> +MODULE_AUTHOR("Marek Vasut <marek.vasut+renesas@gmail.com>");
> >>>>> +MODULE_DESCRIPTION("BD9571MWV PMIC Driver");
> >>>>> +MODULE_LICENSE("GPL v2");
> >>>>> diff --git a/include/linux/mfd/bd9571mwv.h b/include/linux/mfd/bd9571mwv.h
> >>>>> new file mode 100644
> >>>>> index 000000000000..f0708ba4cbba
> >>>>> --- /dev/null
> >>>>> +++ b/include/linux/mfd/bd9571mwv.h
> >>>>> @@ -0,0 +1,115 @@
> >>>>> +/*
> >>>>> + * ROHM BD9571MWV-M driver
> >>>>> + *
> >>>>> + * Copyright (C) 2017 Marek Vasut <marek.vasut+renesas@gmail.com>
> >>>>> + *
> >>>>> + * This program is free software; you can redistribute it and/or
> >>>>> + * modify it under the terms of the GNU General Public License version 2 as
> >>>>> + * published by the Free Software Foundation.
> >>>>> + *
> >>>>> + * This program is distributed "as is" WITHOUT ANY WARRANTY of any
> >>>>> + * kind, whether expressed or implied; without even the implied warranty
> >>>>> + * of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> >>>>> + * GNU General Public License version 2 for more details.
> >>>>> + *
> >>>>> + * Based on the TPS65086 driver
> >>>>> + */
> >>>>> +
> >>>>> +#ifndef __LINUX_MFD_BD9571MWV_H
> >>>>> +#define __LINUX_MFD_BD9571MWV_H
> >>>>> +
> >>>>> +#include <linux/device.h>
> >>>>> +#include <linux/regmap.h>
> >>>>> +
> >>>>> +/* List of registers for BD9571MWV */
> >>>>> +#define BD9571MWV_VENDOR_CODE 0x00
> >>>>> +#define BD9571MWV_VENDOR_CODE_VAL 0xdb
> >>>>> +#define BD9571MWV_PRODUCT_CODE 0x01
> >>>>> +#define BD9571MWV_PRODUCT_CODE_VAL 0x60
> >>>>> +#define BD9571MWV_PRODUCT_REVISION 0x02
> >>>>> +
> >>>>> +#define BD9571MWV_I2C_FUSA_MODE 0x10
> >>>>> +#define BD9571MWV_I2C_MD2_E1_BIT_1 0x11
> >>>>> +#define BD9571MWV_I2C_MD2_E1_BIT_2 0x12
> >>>>> +
> >>>>> +#define BD9571MWV_BKUP_MODE_CNT 0x20
> >>>>> +#define BD9571MWV_BKUP_MODE_STATUS 0x21
> >>>>> +#define BD9571MWV_BKUP_RECOVERY_CNT 0x22
> >>>>> +#define BD9571MWV_BKUP_CTRL_TIM_CNT 0x23
> >>>>> +#define BD9571MWV_WAITBKUP_WDT_CNT 0x24
> >>>>> +#define BD9571MWV_128H_TIM_CNT 0x26
> >>>>> +#define BD9571MWV_QLLM_CNT 0x27
> >>>>> +
> >>>>> +#define BD9571MWV_AVS_SET_MONI 0x31
> >>>>> +#define BD9571MWV_AVS_SET_MONI_MASK 0x3
> >>>>> +#define BD9571MWV_AVS_VD09_VID(n) (0x32 + (n))
> >>>>> +#define BD9571MWV_AVS_DVFS_VID(n) (0x36 + (n))
> >>>>> +
> >>>>> +#define BD9571MWV_VD18_VID 0x42
> >>>>> +#define BD9571MWV_VD25_VID 0x43
> >>>>> +#define BD9571MWV_VD33_VID 0x44
> >>>>> +
> >>>>> +#define BD9571MWV_DVFS_VINIT 0x50
> >>>>> +#define BD9571MWV_DVFS_SETVMAX 0x52
> >>>>> +#define BD9571MWV_DVFS_BOOSTVID 0x53
> >>>>> +#define BD9571MWV_DVFS_SETVID 0x54
> >>>>> +#define BD9571MWV_DVFS_MONIVDAC 0x55
> >>>>> +#define BD9571MWV_DVFS_PGD_CNT 0x56
> >>>>> +
> >>>>> +#define BD9571MWV_GPIO_DIR 0x60
> >>>>> +#define BD9571MWV_GPIO_OUT 0x61
> >>>>> +#define BD9571MWV_GPIO_IN 0x62
> >>>>> +#define BD9571MWV_GPIO_DEB 0x63
> >>>>> +#define BD9571MWV_GPIO_INT_SET 0x64
> >>>>> +#define BD9571MWV_GPIO_INT 0x65
> >>>>> +#define BD9571MWV_GPIO_INTMASK 0x66
> >>>>> +
> >>>>> +#define BD9571MWV_REG_KEEP(n) (0x70 + (n))
> >>>>> +
> >>>>> +#define BD9571MWV_PMIC_INTERNAL_STATUS 0x80
> >>>>> +#define BD9571MWV_PROT_ERROR_STATUS0 0x81
> >>>>> +#define BD9571MWV_PROT_ERROR_STATUS1 0x82
> >>>>> +#define BD9571MWV_PROT_ERROR_STATUS2 0x83
> >>>>> +#define BD9571MWV_PROT_ERROR_STATUS3 0x84
> >>>>> +#define BD9571MWV_PROT_ERROR_STATUS4 0x85
> >>>>> +
> >>>>> +#define BD9571MWV_INT_INTREQ 0x90
> >>>>> +#define BD9571MWV_INT_INTREQ_MD1_INT BIT(0)
> >>>>> +#define BD9571MWV_INT_INTREQ_MD2_E1_INT BIT(1)
> >>>>> +#define BD9571MWV_INT_INTREQ_MD2_E2_INT BIT(2)
> >>>>> +#define BD9571MWV_INT_INTREQ_PROT_ERR_INT BIT(3)
> >>>>> +#define BD9571MWV_INT_INTREQ_GP_INT BIT(4)
> >>>>> +#define BD9571MWV_INT_INTREQ_128H_OF_INT BIT(5)
> >>>>> +#define BD9571MWV_INT_INTREQ_WDT_OF_INT BIT(6)
> >>>>> +#define BD9571MWV_INT_INTREQ_BKUP_TRG_INT BIT(7)
> >>>>> +#define BD9571MWV_INT_INTMASK 0x91
> >>>>> +
> >>>>> +#define BD9571MWV_ACCESS_KEY 0xff
> >>>>> +
> >>>>> +/* Define the BD9571MWV IRQ numbers */
> >>>>> +enum bd9571mwv_irqs {
> >>>>> + BD9571MWV_IRQ_MD1,
> >>>>> + BD9571MWV_IRQ_MD2_E1,
> >>>>> + BD9571MWV_IRQ_MD2_E2,
> >>>>> + BD9571MWV_IRQ_PROT_ERR,
> >>>>> + BD9571MWV_IRQ_GP,
> >>>>> + BD9571MWV_IRQ_128H_OF,
> >>>>> + BD9571MWV_IRQ_WDT_OF,
> >>>>> + BD9571MWV_IRQ_BKUP_TRG,
> >>>>> +};
> >>>>> +
> >>>>> +/**
> >>>>> + * struct bd9571mwv - state holder for the bd9571mwv driver
> >>>>> + *
> >>>>> + * Device data may be used to access the BD9571MWV chip
> >>>>> + */
> >>>>> +struct bd9571mwv {
> >>>>> + struct device *dev;
> >>>>> + struct regmap *regmap;
> >>>>> +
> >>>>> + /* IRQ Data */
> >>>>> + int irq;
> >>>>> + struct regmap_irq_chip_data *irq_data;
> >>>>> +};
> >>>>> +
> >>>>> +#endif /* __LINUX_MFD_BD9571MWV_H */
> >>>>>
> >>>>
> >>>>
> >>>
> >>
> >>
> >
>
>
--
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH V3 2/2] mfd: Add ROHM BD9571MWV-M MFD PMIC driver
2017-05-02 12:18 ` [PATCH V3 2/2] mfd: Add ROHM BD9571MWV-M MFD PMIC driver Marek Vasut
2017-06-27 9:54 ` Marek Vasut
@ 2017-07-10 15:33 ` Geert Uytterhoeven
2017-07-17 20:46 ` Marek Vasut
1 sibling, 1 reply; 9+ messages in thread
From: Geert Uytterhoeven @ 2017-07-10 15:33 UTC (permalink / raw)
To: Marek Vasut
Cc: Linux-Renesas, Lee Jones, Marek Vasut, linux-kernel, Geert Uytterhoeven
Hi Marek,
On Tue, May 2, 2017 at 2:18 PM, Marek Vasut <marek.vasut@gmail.com> wrote:
> Add the MFD part of the ROHM BD9571MWV-M PMIC driver and MAINTAINERS
> entry. The MFD part only specifies the regmap bits for the PMIC and
> binds the subdevs together.
>
> Signed-off-by: Marek Vasut <marek.vasut+renesas@gmail.com>
> Cc: linux-kernel@vger.kernel.org
> Cc: Geert Uytterhoeven <geert+renesas@glider.be>
> Cc: Lee Jones <lee.jones@linaro.org>
> --- a/drivers/mfd/Kconfig
> +++ b/drivers/mfd/Kconfig
> @@ -133,6 +133,19 @@ config MFD_BCM590XX
> help
> Support for the BCM590xx PMUs from Broadcom
>
> +config MFD_BD9571MWV
> + tristate "ROHM BD9571MWV PMIC"
> + select MFD_CORE
> + select REGMAP_I2C
Missing "select REGMAP_IRQ".
Else it fails with:
drivers/built-in.o: In function `bd9571mwv_remove':
:(.text+0x103190): undefined reference to `regmap_del_irq_chip'
drivers/built-in.o: In function `bd9571mwv_probe':
:(.text+0x10332c): undefined reference to `regmap_add_irq_chip'
:(.text+0x103358): undefined reference to `regmap_irq_get_domain'
:(.text+0x103394): undefined reference to `regmap_del_irq_chip'
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH V3 2/2] mfd: Add ROHM BD9571MWV-M MFD PMIC driver
2017-07-10 15:33 ` Geert Uytterhoeven
@ 2017-07-17 20:46 ` Marek Vasut
0 siblings, 0 replies; 9+ messages in thread
From: Marek Vasut @ 2017-07-17 20:46 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Linux-Renesas, Lee Jones, Marek Vasut, linux-kernel, Geert Uytterhoeven
On 07/10/2017 05:33 PM, Geert Uytterhoeven wrote:
> Hi Marek,
>
> On Tue, May 2, 2017 at 2:18 PM, Marek Vasut <marek.vasut@gmail.com> wrote:
>> Add the MFD part of the ROHM BD9571MWV-M PMIC driver and MAINTAINERS
>> entry. The MFD part only specifies the regmap bits for the PMIC and
>> binds the subdevs together.
>>
>> Signed-off-by: Marek Vasut <marek.vasut+renesas@gmail.com>
>> Cc: linux-kernel@vger.kernel.org
>> Cc: Geert Uytterhoeven <geert+renesas@glider.be>
>> Cc: Lee Jones <lee.jones@linaro.org>
>
>> --- a/drivers/mfd/Kconfig
>> +++ b/drivers/mfd/Kconfig
>> @@ -133,6 +133,19 @@ config MFD_BCM590XX
>> help
>> Support for the BCM590xx PMUs from Broadcom
>>
>> +config MFD_BD9571MWV
>> + tristate "ROHM BD9571MWV PMIC"
>> + select MFD_CORE
>> + select REGMAP_I2C
>
> Missing "select REGMAP_IRQ".
>
> Else it fails with:
Added
--
Best regards,
Marek Vasut
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2017-07-17 20:46 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <20170502121816.10572-1-marek.vasut+renesas@gmail.com>
2017-05-02 12:18 ` [PATCH V3 2/2] mfd: Add ROHM BD9571MWV-M MFD PMIC driver Marek Vasut
2017-06-27 9:54 ` Marek Vasut
2017-07-03 11:55 ` Lee Jones
2017-07-03 12:03 ` Marek Vasut
2017-07-03 13:48 ` Lee Jones
2017-07-03 14:16 ` Marek Vasut
2017-07-03 15:21 ` Lee Jones
2017-07-10 15:33 ` Geert Uytterhoeven
2017-07-17 20:46 ` Marek Vasut
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).