regulator: 88pg86x: new i2c dual regulator chip

This chip is found on Google Chromecast and Valve Steam Link devices.
It provides two DC regulators with I2C voltage control, separate GPIO
enable pins and one sleep mode pin.

This driver does not expose GPIO functionality, but supports voltage
control in 1.0-2.2V range, based on I2C register information given in
Chromecast kernel driver by Jisheng Zhang.

Cc: Jisheng Zhang <Jisheng.Zhang@synaptics.com>
Signed-off-by: Alexander Monakov <amonakov@ispras.ru>
Signed-off-by: Mark Brown <broonie@kernel.org>

diff --git a/drivers/regulator/88pg86x.c b/drivers/regulator/88pg86x.c
new file mode 100644 (file)
index 0000000..d5ef55c
--- /dev/null
+++ b/drivers/regulator/88pg86x.c
@@ -0,0 +1,114 @@
+// SPDX-License-Identifier: GPL-2.0
+#include <linux/module.h>
+#include <linux/i2c.h>
+#include <linux/of.h>
+#include <linux/regulator/driver.h>
+#include <linux/regmap.h>
+
+static const struct regulator_ops pg86x_ops = {
+       .set_voltage_sel = regulator_set_voltage_sel_regmap,
+       .get_voltage_sel = regulator_get_voltage_sel_regmap,
+       .list_voltage = regulator_list_voltage_linear_range,
+};
+
+static const struct regulator_linear_range pg86x_buck1_ranges[] = {
+       REGULATOR_LINEAR_RANGE(      0,  0, 10,     0),
+       REGULATOR_LINEAR_RANGE(1000000, 11, 34, 25000),
+       REGULATOR_LINEAR_RANGE(1600000, 35, 47, 50000),
+};
+
+static const struct regulator_linear_range pg86x_buck2_ranges[] = {
+       REGULATOR_LINEAR_RANGE(      0,  0, 15,     0),
+       REGULATOR_LINEAR_RANGE(1000000, 16, 39, 25000),
+       REGULATOR_LINEAR_RANGE(1600000, 40, 52, 50000),
+};
+
+static const struct regulator_desc pg86x_regulators[] = {
+       {
+               .id = 0,
+               .type = REGULATOR_VOLTAGE,
+               .name = "buck1",
+               .of_match = of_match_ptr("buck1"),
+               .n_voltages = 11 + 24 + 13,
+               .linear_ranges = pg86x_buck1_ranges,
+               .n_linear_ranges = 3,
+               .vsel_reg  = 0x24,
+               .vsel_mask = 0xff,
+               .ops = &pg86x_ops,
+               .owner = THIS_MODULE
+       },
+       {
+               .id = 1,
+               .type = REGULATOR_VOLTAGE,
+               .name = "buck2",
+               .of_match = of_match_ptr("buck2"),
+               .n_voltages = 16 + 24 + 13,
+               .linear_ranges = pg86x_buck2_ranges,
+               .n_linear_ranges = 3,
+               .vsel_reg  = 0x13,
+               .vsel_mask = 0xff,
+               .ops = &pg86x_ops,
+               .owner = THIS_MODULE
+       },
+};
+
+static const struct regmap_config pg86x_regmap = {
+       .reg_bits = 8,
+       .val_bits = 8,
+};
+
+static int pg86x_i2c_probe(struct i2c_client *i2c)
+{
+       int id, ret;
+       struct regulator_config config = {.dev = &i2c->dev};
+       struct regmap *regmap = devm_regmap_init_i2c(i2c, &pg86x_regmap);
+
+       if (IS_ERR(regmap)) {
+               ret = PTR_ERR(regmap);
+               dev_err(&i2c->dev, "regmap init failed: %d\n", ret);
+               return ret;
+       }
+
+       for (id = 0; id < ARRAY_SIZE(pg86x_regulators); id++) {
+               struct regulator_dev *rdev;
+               rdev = devm_regulator_register(&i2c->dev,
+                                              &pg86x_regulators[id],
+                                              &config);
+               if (IS_ERR(rdev)) {
+                       ret = PTR_ERR(rdev);
+                       dev_err(&i2c->dev, "failed to register %s: %d\n",
+                               pg86x_regulators[id].name, ret);
+                       return ret;
+               }
+       }
+       return 0;
+}
+
+static const struct of_device_id pg86x_dt_ids [] = {
+       { .compatible = "marvell,88pg867" },
+       { .compatible = "marvell,88pg868" },
+       { }
+};
+MODULE_DEVICE_TABLE(of, pg86x_dt_ids);
+
+static const struct i2c_device_id pg86x_i2c_id[] = {
+       { "88pg867", },
+       { "88pg868", },
+       { }
+};
+MODULE_DEVICE_TABLE(i2c, pg86x_i2c_id);
+
+static struct i2c_driver pg86x_regulator_driver = {
+       .driver = {
+               .name = "88pg86x",
+               .of_match_table = of_match_ptr(pg86x_dt_ids),
+       },
+       .probe_new = pg86x_i2c_probe,
+       .id_table = pg86x_i2c_id,
+};
+
+module_i2c_driver(pg86x_regulator_driver);
+
+MODULE_DESCRIPTION("Marvell 88PG86X voltage regulator");
+MODULE_AUTHOR("Alexander Monakov <amonakov@gmail.com>");
+MODULE_LICENSE("GPL");

diff --git a/drivers/regulator/Kconfig b/drivers/regulator/Kconfig
index b27417ca188a0acd510899b11fcd6617115e4e5f..097f61784a7d9ee66298846d630522ff3a2c3836 100644 (file)
--- a/drivers/regulator/Kconfig
+++ b/drivers/regulator/Kconfig
@@ -54,6 +54,15 @@ config REGULATOR_USERSPACE_CONSUMER
 
          If unsure, say no.
 
+config REGULATOR_88PG86X
+       tristate "Marvell 88PG86X voltage regulators"
+       depends on I2C
+       select REGMAP_I2C
+       help
+         This driver supports Marvell 88PG867 and 88PG868 voltage regulators.
+         They provide two I2C-controlled DC/DC step-down converters with
+         sleep mode and separate enable pins.
+
 config REGULATOR_88PM800
        tristate "Marvell 88PM800 Power regulators"
        depends on MFD_88PM800

diff --git a/drivers/regulator/Makefile b/drivers/regulator/Makefile
index 19fea09ba10a2be775d205a9aeaf3ef5d1b60719..590674fbecd7a25c050d0a1fb7c2ca9887caa742 100644 (file)
--- a/drivers/regulator/Makefile
+++ b/drivers/regulator/Makefile
@@ -10,6 +10,7 @@ obj-$(CONFIG_REGULATOR_FIXED_VOLTAGE) += fixed.o
 obj-$(CONFIG_REGULATOR_VIRTUAL_CONSUMER) += virtual.o
 obj-$(CONFIG_REGULATOR_USERSPACE_CONSUMER) += userspace-consumer.o
 
+obj-$(CONFIG_REGULATOR_88PG86X) += 88pg86x.o
 obj-$(CONFIG_REGULATOR_88PM800) += 88pm800.o
 obj-$(CONFIG_REGULATOR_88PM8607) += 88pm8607.o
 obj-$(CONFIG_REGULATOR_CPCAP) += cpcap-regulator.o