Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Unverified Commit 36fd679f authored by Mark Brown's avatar Mark Brown
Browse files

Merge remote-tracking branches 'regulator/topic/88pg86x',... 

Merge remote-tracking branches 'regulator/topic/88pg86x', 'regulator/topic/dt', 'regulator/topic/formatting' and 'regulator/topic/gpio' into regulator-next
parents bca334e3 a265b03b 4b752916 d3e4eccb 37ad490b

Documentation/devicetree/bindings/regulator/88pg86x.txt

0 → 100644
+22 −0
Original line number Diff line number Diff line 
Marvell 88PG867/88PG868 voltage regulators



Required properties:

- compatible: one of "marvell,88pg867", "marvell,88pg868";

- reg: I2C slave address.



Optional subnodes for regulators: "buck1", "buck2", using common regulator

bindings given in <Documentation/devicetree/bindings/regulator/regulator.txt>.



Example:



	pg868@19 {

		compatible = "marvell,88pg868";

		reg = <0x19>;



		vcpu: buck1 {

			regulator-boot-on;

			regulator-always-on;

			regulator-min-microvolt = <1000000>;

			regulator-max-microvolt = <1350000>;

		};

	};

Documentation/devicetree/bindings/regulator/fixed-regulator.txt

+1 −0
Original line number Diff line number Diff line
@@ -2,6 +2,7 @@ Fixed Voltage regulators 


Required properties:

- compatible: Must be "regulator-fixed";

- regulator-name: Defined in regulator.txt as optional, but required here.



Optional properties:

- gpio: gpio to use for enable control

Documentation/devicetree/bindings/regulator/gpio-regulator.txt

+2 −0
Original line number Diff line number Diff line
@@ -2,6 +2,8 @@ GPIO controlled regulators 


Required properties:

- compatible		: Must be "regulator-gpio".

- regulator-name	: Defined in regulator.txt as optional, but required

			  here.

- states		: Selection of available voltages and GPIO configs.

                          if there are no states, then use a fixed regulator

drivers/regulator/88pg86x.c

0 → 100644
+114 −0
Original line number Diff line number Diff line 
// 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");

drivers/regulator/Kconfig

+9 −0
Original line number Diff line number Diff line
@@ -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