* [PATCH v3 0/7] ARM: dts: AM43x: Add devcice tree nodes
@ 2014-07-09 5:36 Keerthy
2014-07-09 5:36 ` [PATCH v3 1/7] regulator: tps65218: Add fixed_uV fields for dcdc5 and dcdc6 Keerthy
` (7 more replies)
0 siblings, 8 replies; 19+ messages in thread
From: Keerthy @ 2014-07-09 5:36 UTC (permalink / raw)
To: devicetree, linux-kernel, linux-omap
Cc: mark.rutland, galak, robh+dt, linux, balbi, tony, kishon, broonie,
Keerthy
The patch series adds the device tree nodes and the corresponding
documentation. The series also enabled tps65218 config options
in the omap2plus_defconfig.
The series is boot tested on both AM43x-epos-evm and AM437x-gp-evm.
Keerthy (7):
regulator: tps65218: Add fixed_uV fields for dcdc5 and dcdc6
mfd: Add DT bindings for tps65218 PMIC
regulator: Add DT bindings for tps65218 PMIC regulators.
ARM: dts: AM43x: Add TPS65218 device tree nodes
ARM: dts: AM437x: Fix i2c nodes indentation
ARM: dts: AM437x: Add TPS65218 device tree nodes
ARM: configs: omap2plus_defconfig: enable TPS65218 configs
Documentation/devicetree/bindings/mfd/tps65218.txt | 89 ++++++++++++++++++++
.../devicetree/bindings/regulator/tps65218.txt | 23 +++++
arch/arm/boot/dts/am437x-gp-evm.dts | 71 ++++++++++++++--
arch/arm/boot/dts/am43x-epos-evm.dts | 59 +++++++++++++
arch/arm/configs/omap2plus_defconfig | 2 +
drivers/regulator/tps65218-regulator.c | 17 ++--
6 files changed, 246 insertions(+), 15 deletions(-)
create mode 100644 Documentation/devicetree/bindings/mfd/tps65218.txt
create mode 100644 Documentation/devicetree/bindings/regulator/tps65218.txt
--
1.7.9.5
^ permalink raw reply [flat|nested] 19+ messages in thread* [PATCH v3 1/7] regulator: tps65218: Add fixed_uV fields for dcdc5 and dcdc6 2014-07-09 5:36 [PATCH v3 0/7] ARM: dts: AM43x: Add devcice tree nodes Keerthy @ 2014-07-09 5:36 ` Keerthy 2014-07-09 9:14 ` Mark Brown 2014-07-09 10:22 ` [PATCH v4] " Keerthy 2014-07-09 5:36 ` [PATCH v3 2/7] mfd: Add DT bindings for tps65218 PMIC Keerthy ` (6 subsequent siblings) 7 siblings, 2 replies; 19+ messages in thread From: Keerthy @ 2014-07-09 5:36 UTC (permalink / raw) To: devicetree, linux-kernel, linux-omap Cc: mark.rutland, galak, robh+dt, linux, balbi, tony, kishon, broonie, Keerthy Add fixed_uV fields for dcdc5 and dcdc6. Signed-off-by: Keerthy <j-keerthy@ti.com> --- Changes from V2: * Added fixed_uV fields for the regulator_desc structures. drivers/regulator/tps65218-regulator.c | 17 +++++++++-------- 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/drivers/regulator/tps65218-regulator.c b/drivers/regulator/tps65218-regulator.c index 9effe48..27dc70d 100644 --- a/drivers/regulator/tps65218-regulator.c +++ b/drivers/regulator/tps65218-regulator.c @@ -30,7 +30,7 @@ enum tps65218_regulators { DCDC1, DCDC2, DCDC3, DCDC4, DCDC5, DCDC6, LDO1 }; #define TPS65218_REGULATOR(_name, _id, _ops, _n, _vr, _vm, _er, _em, _t, \ - _lr, _nlr, _delay) \ + _lr, _nlr, _delay, _fuv) \ { \ .name = _name, \ .id = _id, \ @@ -46,6 +46,7 @@ enum tps65218_regulators { DCDC1, DCDC2, DCDC3, DCDC4, DCDC5, DCDC6, LDO1 }; .linear_ranges = _lr, \ .n_linear_ranges = _nlr, \ .ramp_delay = _delay, \ + .fixed_uV = _fuv \ } \ #define TPS65218_INFO(_id, _nm, _min, _max) \ @@ -186,33 +187,33 @@ static const struct regulator_desc regulators[] = { TPS65218_REG_CONTROL_DCDC1, TPS65218_CONTROL_DCDC1_MASK, TPS65218_REG_ENABLE1, TPS65218_ENABLE1_DC1_EN, NULL, - dcdc1_dcdc2_ranges, 2, 4000), + dcdc1_dcdc2_ranges, 2, 4000, 0), TPS65218_REGULATOR("DCDC2", TPS65218_DCDC_2, tps65218_dcdc12_ops, 64, TPS65218_REG_CONTROL_DCDC2, TPS65218_CONTROL_DCDC2_MASK, TPS65218_REG_ENABLE1, TPS65218_ENABLE1_DC2_EN, NULL, - dcdc1_dcdc2_ranges, 2, 4000), + dcdc1_dcdc2_ranges, 2, 4000, 0), TPS65218_REGULATOR("DCDC3", TPS65218_DCDC_3, tps65218_ldo1_dcdc34_ops, 64, TPS65218_REG_CONTROL_DCDC3, TPS65218_CONTROL_DCDC3_MASK, TPS65218_REG_ENABLE1, TPS65218_ENABLE1_DC3_EN, NULL, - ldo1_dcdc3_ranges, 2, 0), + ldo1_dcdc3_ranges, 2, 0, 0), TPS65218_REGULATOR("DCDC4", TPS65218_DCDC_4, tps65218_ldo1_dcdc34_ops, 53, TPS65218_REG_CONTROL_DCDC4, TPS65218_CONTROL_DCDC4_MASK, TPS65218_REG_ENABLE1, TPS65218_ENABLE1_DC4_EN, NULL, - dcdc4_ranges, 2, 0), + dcdc4_ranges, 2, 0, 0), TPS65218_REGULATOR("DCDC5", TPS65218_DCDC_5, tps65218_dcdc56_pmic_ops, 1, -1, -1, TPS65218_REG_ENABLE1, - TPS65218_ENABLE1_DC5_EN, NULL, NULL, 0, 0), + TPS65218_ENABLE1_DC5_EN, NULL, NULL, 0, 0, 1000000), TPS65218_REGULATOR("DCDC6", TPS65218_DCDC_6, tps65218_dcdc56_pmic_ops, 1, -1, -1, TPS65218_REG_ENABLE1, - TPS65218_ENABLE1_DC6_EN, NULL, NULL, 0, 0), + TPS65218_ENABLE1_DC6_EN, NULL, NULL, 0, 0, 1800000), TPS65218_REGULATOR("LDO1", TPS65218_LDO_1, tps65218_ldo1_dcdc34_ops, 64, TPS65218_REG_CONTROL_LDO1, TPS65218_CONTROL_LDO1_MASK, TPS65218_REG_ENABLE2, TPS65218_ENABLE2_LDO1_EN, NULL, ldo1_dcdc3_ranges, - 2, 0), + 2, 0, 0), }; static int tps65218_regulator_probe(struct platform_device *pdev) -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 19+ messages in thread
* Re: [PATCH v3 1/7] regulator: tps65218: Add fixed_uV fields for dcdc5 and dcdc6 2014-07-09 5:36 ` [PATCH v3 1/7] regulator: tps65218: Add fixed_uV fields for dcdc5 and dcdc6 Keerthy @ 2014-07-09 9:14 ` Mark Brown 2014-07-09 9:36 ` Keerthy 2014-07-09 10:22 ` [PATCH v4] " Keerthy 1 sibling, 1 reply; 19+ messages in thread From: Mark Brown @ 2014-07-09 9:14 UTC (permalink / raw) To: Keerthy Cc: devicetree, linux-kernel, linux-omap, mark.rutland, galak, robh+dt, linux, balbi, tony, kishon [-- Attachment #1: Type: text/plain, Size: 269 bytes --] On Wed, Jul 09, 2014 at 11:06:27AM +0530, Keerthy wrote: > Add fixed_uV fields for dcdc5 and dcdc6. This doesn't apply against current code, can you please check and resend? There were some cleanups from Felipe that just went in, it looks like a collision with them. [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 836 bytes --] ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH v3 1/7] regulator: tps65218: Add fixed_uV fields for dcdc5 and dcdc6 2014-07-09 9:14 ` Mark Brown @ 2014-07-09 9:36 ` Keerthy 0 siblings, 0 replies; 19+ messages in thread From: Keerthy @ 2014-07-09 9:36 UTC (permalink / raw) To: Mark Brown Cc: Keerthy, devicetree, linux-kernel, linux-omap, mark.rutland, galak, robh+dt, linux, balbi, tony, kishon On Wednesday 09 July 2014 02:44 PM, Mark Brown wrote: > On Wed, Jul 09, 2014 at 11:06:27AM +0530, Keerthy wrote: >> Add fixed_uV fields for dcdc5 and dcdc6. > This doesn't apply against current code, can you please check and > resend? There were some cleanups from Felipe that just went in, it > looks like a collision with them. Yup. They touch the same lines. Those patches were not yet taken before i posted. I will rebase and post. Thanks, Keerthy ^ permalink raw reply [flat|nested] 19+ messages in thread
* [PATCH v4] regulator: tps65218: Add fixed_uV fields for dcdc5 and dcdc6 2014-07-09 5:36 ` [PATCH v3 1/7] regulator: tps65218: Add fixed_uV fields for dcdc5 and dcdc6 Keerthy 2014-07-09 9:14 ` Mark Brown @ 2014-07-09 10:22 ` Keerthy 2014-07-10 12:12 ` Mark Brown 1 sibling, 1 reply; 19+ messages in thread From: Keerthy @ 2014-07-09 10:22 UTC (permalink / raw) To: devicetree, linux-kernel, linux-omap Cc: mark.rutland, galak, robh+dt, linux, balbi, tony, kishon, broonie, Keerthy Add fixed_uV fields for dcdc5 and dcdc6. Signed-off-by: Keerthy <j-keerthy@ti.com> --- Changes from V3: * Rebased to Latest for/tps65218 branch. drivers/regulator/tps65218-regulator.c | 17 +++++++++-------- 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/drivers/regulator/tps65218-regulator.c b/drivers/regulator/tps65218-regulator.c index e610dad..f0a4028 100644 --- a/drivers/regulator/tps65218-regulator.c +++ b/drivers/regulator/tps65218-regulator.c @@ -30,7 +30,7 @@ enum tps65218_regulators { DCDC1, DCDC2, DCDC3, DCDC4, DCDC5, DCDC6, LDO1 }; #define TPS65218_REGULATOR(_name, _id, _ops, _n, _vr, _vm, _er, _em, \ - _lr, _nlr, _delay) \ + _lr, _nlr, _delay, _fuv) \ { \ .name = _name, \ .id = _id, \ @@ -46,6 +46,7 @@ enum tps65218_regulators { DCDC1, DCDC2, DCDC3, DCDC4, DCDC5, DCDC6, LDO1 }; .linear_ranges = _lr, \ .n_linear_ranges = _nlr, \ .ramp_delay = _delay, \ + .fixed_uV = _fuv \ } \ #define TPS65218_INFO(_id, _nm, _min, _max) \ @@ -186,32 +187,32 @@ static const struct regulator_desc regulators[] = { TPS65218_REG_CONTROL_DCDC1, TPS65218_CONTROL_DCDC1_MASK, TPS65218_REG_ENABLE1, TPS65218_ENABLE1_DC1_EN, - dcdc1_dcdc2_ranges, 2, 4000), + dcdc1_dcdc2_ranges, 2, 4000, 0), TPS65218_REGULATOR("DCDC2", TPS65218_DCDC_2, tps65218_dcdc12_ops, 64, TPS65218_REG_CONTROL_DCDC2, TPS65218_CONTROL_DCDC2_MASK, TPS65218_REG_ENABLE1, TPS65218_ENABLE1_DC2_EN, - dcdc1_dcdc2_ranges, 2, 4000), + dcdc1_dcdc2_ranges, 2, 4000, 0), TPS65218_REGULATOR("DCDC3", TPS65218_DCDC_3, tps65218_ldo1_dcdc34_ops, 64, TPS65218_REG_CONTROL_DCDC3, TPS65218_CONTROL_DCDC3_MASK, TPS65218_REG_ENABLE1, - TPS65218_ENABLE1_DC3_EN, ldo1_dcdc3_ranges, 2, 0), + TPS65218_ENABLE1_DC3_EN, ldo1_dcdc3_ranges, 2, 0, 0), TPS65218_REGULATOR("DCDC4", TPS65218_DCDC_4, tps65218_ldo1_dcdc34_ops, 53, TPS65218_REG_CONTROL_DCDC4, TPS65218_CONTROL_DCDC4_MASK, TPS65218_REG_ENABLE1, TPS65218_ENABLE1_DC4_EN, - dcdc4_ranges, 2, 0), + dcdc4_ranges, 2, 0, 0), TPS65218_REGULATOR("DCDC5", TPS65218_DCDC_5, tps65218_dcdc56_pmic_ops, 1, -1, -1, TPS65218_REG_ENABLE1, - TPS65218_ENABLE1_DC5_EN, NULL, 0, 0), + TPS65218_ENABLE1_DC5_EN, NULL, 0, 0, 1000000), TPS65218_REGULATOR("DCDC6", TPS65218_DCDC_6, tps65218_dcdc56_pmic_ops, 1, -1, -1, TPS65218_REG_ENABLE1, - TPS65218_ENABLE1_DC6_EN, NULL, 0, 0), + TPS65218_ENABLE1_DC6_EN, NULL, 0, 0, 1800000), TPS65218_REGULATOR("LDO1", TPS65218_LDO_1, tps65218_ldo1_dcdc34_ops, 64, TPS65218_REG_CONTROL_LDO1, TPS65218_CONTROL_LDO1_MASK, TPS65218_REG_ENABLE2, TPS65218_ENABLE2_LDO1_EN, ldo1_dcdc3_ranges, - 2, 0), + 2, 0, 0), }; static int tps65218_regulator_probe(struct platform_device *pdev) -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 19+ messages in thread
* Re: [PATCH v4] regulator: tps65218: Add fixed_uV fields for dcdc5 and dcdc6 2014-07-09 10:22 ` [PATCH v4] " Keerthy @ 2014-07-10 12:12 ` Mark Brown 2014-07-10 13:10 ` Keerthy 0 siblings, 1 reply; 19+ messages in thread From: Mark Brown @ 2014-07-10 12:12 UTC (permalink / raw) To: Keerthy Cc: devicetree, linux-kernel, linux-omap, mark.rutland, galak, robh+dt, linux, balbi, tony, kishon [-- Attachment #1: Type: text/plain, Size: 213 bytes --] On Wed, Jul 09, 2014 at 03:52:59PM +0530, Keerthy wrote: > Add fixed_uV fields for dcdc5 and dcdc6. Applied, but don't bury new patches into the middle of old threads - it's a good way to get your patch ignored. [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 836 bytes --] ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH v4] regulator: tps65218: Add fixed_uV fields for dcdc5 and dcdc6 2014-07-10 12:12 ` Mark Brown @ 2014-07-10 13:10 ` Keerthy 2014-07-10 14:50 ` Mark Brown 0 siblings, 1 reply; 19+ messages in thread From: Keerthy @ 2014-07-10 13:10 UTC (permalink / raw) To: Mark Brown Cc: Keerthy, devicetree, linux-kernel, linux-omap, mark.rutland, galak, robh+dt, linux, balbi, tony, kishon On Thursday 10 July 2014 05:42 PM, Mark Brown wrote: > On Wed, Jul 09, 2014 at 03:52:59PM +0530, Keerthy wrote: >> Add fixed_uV fields for dcdc5 and dcdc6. > Applied, but don't bury new patches into the middle of old threads - > it's a good way to get your patch ignored. Okay. I thought only one patch had to be redone on top of the other branch so did it this way. I will resend a new series next time onwards. Thanks. Regards, Keerthy ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH v4] regulator: tps65218: Add fixed_uV fields for dcdc5 and dcdc6 2014-07-10 13:10 ` Keerthy @ 2014-07-10 14:50 ` Mark Brown 0 siblings, 0 replies; 19+ messages in thread From: Mark Brown @ 2014-07-10 14:50 UTC (permalink / raw) To: Keerthy Cc: Keerthy, devicetree, linux-kernel, linux-omap, mark.rutland, galak, robh+dt, linux, balbi, tony, kishon [-- Attachment #1: Type: text/plain, Size: 500 bytes --] On Thu, Jul 10, 2014 at 06:40:39PM +0530, Keerthy wrote: > On Thursday 10 July 2014 05:42 PM, Mark Brown wrote: > >Applied, but don't bury new patches into the middle of old threads - > >it's a good way to get your patch ignored. > Okay. I thought only one patch had to be redone on top of the other branch > so did it this way. I will resend a new series next time onwards. Thanks. For something like this where there's actually no dependencies it's OK to resend the patch as a standalone thing. [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 836 bytes --] ^ permalink raw reply [flat|nested] 19+ messages in thread
* [PATCH v3 2/7] mfd: Add DT bindings for tps65218 PMIC 2014-07-09 5:36 [PATCH v3 0/7] ARM: dts: AM43x: Add devcice tree nodes Keerthy 2014-07-09 5:36 ` [PATCH v3 1/7] regulator: tps65218: Add fixed_uV fields for dcdc5 and dcdc6 Keerthy @ 2014-07-09 5:36 ` Keerthy 2014-07-09 5:36 ` [PATCH v3 3/7] regulator: Add DT bindings for tps65218 PMIC regulators Keerthy ` (5 subsequent siblings) 7 siblings, 0 replies; 19+ messages in thread From: Keerthy @ 2014-07-09 5:36 UTC (permalink / raw) To: devicetree, linux-kernel, linux-omap Cc: mark.rutland, galak, robh+dt, linux, balbi, tony, kishon, broonie, Keerthy Add DT bindings for tps65218 PMIC Signed-off-by: Keerthy <j-keerthy@ti.com> Reviewed-by: Felipe Balbi <balbi@ti.com> --- Documentation/devicetree/bindings/mfd/tps65218.txt | 89 ++++++++++++++++++++ 1 file changed, 89 insertions(+) create mode 100644 Documentation/devicetree/bindings/mfd/tps65218.txt diff --git a/Documentation/devicetree/bindings/mfd/tps65218.txt b/Documentation/devicetree/bindings/mfd/tps65218.txt new file mode 100644 index 0000000..01ac973 --- /dev/null +++ b/Documentation/devicetree/bindings/mfd/tps65218.txt @@ -0,0 +1,89 @@ +Texas Instruments TPS65218 family + +The TPS65218 are Integrated Power Management Chips. + +Required properties: +- compatible : Must be "ti,tps65218"; + For Integrated power-management used in AM437x based boards +- interrupts : This i2c device has an IRQ line connected to the main SoC +- interrupt-controller : Since the tps65218 supports several interrupts + internally, it is considered as an interrupt controller cascaded to the + SoC one. +- #interrupt-cells = <2>; +- interrupt-parent : The parent interrupt controller which is gic. + +Optional node: +- TPS65218 PMIC has a number of child nodes. Mainly the regularors and SMPSs. + +Example: +/* + * Integrated Power Management Chip + */ +tps65218: tps65218@24 { + reg = <0x24>; + compatible = "ti,tps65218"; + interrupts = <GIC_SPI 7 IRQ_TYPE_NONE>; /* NMIn */ + interrupt-parent = <&gic>; + interrupt-controller; + #interrupt-cells = <2>; + + dcdc1: regulator-dcdc1 { + compatible = "ti,tps65218-dcdc1"; + /* VDD_CORE voltage limits min of OPP50 and max of OPP100 */ + regulator-name = "vdd_core"; + regulator-min-microvolt = <912000>; + regulator-max-microvolt = <1144000>; + regulator-boot-on; + regulator-always-on; + }; + + dcdc2: regulator-dcdc2 { + compatible = "ti,tps65218-dcdc2"; + /* VDD_MPU voltage limits min of OPP50 and max of OPP_NITRO */ + regulator-name = "vdd_mpu"; + regulator-min-microvolt = <912000>; + regulator-max-microvolt = <1378000>; + regulator-boot-on; + regulator-always-on; + }; + + dcdc3: regulator-dcdc3 { + compatible = "ti,tps65218-dcdc3"; + regulator-name = "vdcdc3"; + regulator-min-microvolt = <1350000>; + regulator-max-microvolt = <1350000>; + regulator-boot-on; + regulator-always-on; + }; + + dcdc4: regulator-dcdc4 { + compatible = "ti,tps65218-dcdc4"; + regulator-name = "v3_3d"; + regulator-min-microvolt = <3300000>; + regulator-max-microvolt = <3300000>; + regulator-boot-on; + regulator-always-on; + }; + + dcdc5: regulator-dcdc5 { + compatible = "ti,tps65218-dcdc5"; + regulator-name = "v1_0bat"; + regulator-min-microvolt = <1000000>; + regulator-max-microvolt = <1000000>; + }; + + dcdc6: regulator-dcdc6 { + compatible = "ti,tps65218-dcdc6"; + regulator-name = "v1_8vat"; + regulator-min-microvolt = <1800000>; + regulator-max-microvolt = <1800000>; + }; + + ldo1: regulator-ldo1 { + compatible = "ti,tps65218-ldo1"; + regulator-min-microvolt = <1800000>; + regulator-max-microvolt = <1800000>; + regulator-boot-on; + regulator-always-on; + }; +}; -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH v3 3/7] regulator: Add DT bindings for tps65218 PMIC regulators. 2014-07-09 5:36 [PATCH v3 0/7] ARM: dts: AM43x: Add devcice tree nodes Keerthy 2014-07-09 5:36 ` [PATCH v3 1/7] regulator: tps65218: Add fixed_uV fields for dcdc5 and dcdc6 Keerthy 2014-07-09 5:36 ` [PATCH v3 2/7] mfd: Add DT bindings for tps65218 PMIC Keerthy @ 2014-07-09 5:36 ` Keerthy 2014-07-09 5:36 ` [PATCH v3 4/7] ARM: dts: AM43x: Add TPS65218 device tree nodes Keerthy ` (4 subsequent siblings) 7 siblings, 0 replies; 19+ messages in thread From: Keerthy @ 2014-07-09 5:36 UTC (permalink / raw) To: devicetree, linux-kernel, linux-omap Cc: mark.rutland, galak, robh+dt, linux, balbi, tony, kishon, broonie, Keerthy Add DT bindings for tps65218 PMIC regulators. Signed-off-by: Keerthy <j-keerthy@ti.com> Reviewed-by: Felipe Balbi <balbi@ti.com> --- .../devicetree/bindings/regulator/tps65218.txt | 23 ++++++++++++++++++++ 1 file changed, 23 insertions(+) create mode 100644 Documentation/devicetree/bindings/regulator/tps65218.txt diff --git a/Documentation/devicetree/bindings/regulator/tps65218.txt b/Documentation/devicetree/bindings/regulator/tps65218.txt new file mode 100644 index 0000000..fccc1d2 --- /dev/null +++ b/Documentation/devicetree/bindings/regulator/tps65218.txt @@ -0,0 +1,23 @@ +TPS65218 family of regulators + +Required properties: +For tps65218 regulators/LDOs +- compatible: + - "ti,tps65218-dcdc1" for DCDC1 + - "ti,tps65218-dcdc2" for DCDC2 + - "ti,tps65218-dcdc3" for DCDC3 + - "ti,tps65218-dcdc4" for DCDC4 + - "ti,tps65218-dcdc5" for DCDC5 + - "ti,tps65218-dcdc6" for DCDC6 + - "ti,tps65218-ldo1" for LDO1 + +Optional properties: +- Any optional property defined in bindings/regulator/regulator.txt + +Example: + + xyz: regulator@0 { + compatible = "ti,tps65218-dcdc1"; + regulator-min-microvolt = <1000000>; + regulator-max-microvolt = <3000000>; + }; -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH v3 4/7] ARM: dts: AM43x: Add TPS65218 device tree nodes 2014-07-09 5:36 [PATCH v3 0/7] ARM: dts: AM43x: Add devcice tree nodes Keerthy ` (2 preceding siblings ...) 2014-07-09 5:36 ` [PATCH v3 3/7] regulator: Add DT bindings for tps65218 PMIC regulators Keerthy @ 2014-07-09 5:36 ` Keerthy 2014-07-09 5:36 ` [PATCH v3 5/7] ARM: dts: AM437x: Fix i2c nodes indentation Keerthy ` (3 subsequent siblings) 7 siblings, 0 replies; 19+ messages in thread From: Keerthy @ 2014-07-09 5:36 UTC (permalink / raw) To: devicetree, linux-kernel, linux-omap Cc: mark.rutland, galak, robh+dt, linux, balbi, tony, kishon, broonie, Keerthy Add TPS65218 device tree nodes. i2c clock frequency setting also added as part of tps65218 nodes addition. As i2c clock enabling is required. Signed-off-by: Keerthy <j-keerthy@ti.com> Reviewed-by: Felipe Balbi <balbi@ti.com> --- Changes from V1: * Added dcdc3, dcdc5, dcdc6 nodes. * dcdc4 is not added because of a potential PMIC bug under investigation. arch/arm/boot/dts/am43x-epos-evm.dts | 59 ++++++++++++++++++++++++++++++++++ 1 file changed, 59 insertions(+) diff --git a/arch/arm/boot/dts/am43x-epos-evm.dts b/arch/arm/boot/dts/am43x-epos-evm.dts index 90098f9..f1ee7495 100644 --- a/arch/arm/boot/dts/am43x-epos-evm.dts +++ b/arch/arm/boot/dts/am43x-epos-evm.dts @@ -327,6 +327,65 @@ status = "okay"; pinctrl-names = "default"; pinctrl-0 = <&i2c0_pins>; + clock-frequency = <400000>; + + tps65218: tps65218@24 { + reg = <0x24>; + compatible = "ti,tps65218"; + interrupts = <GIC_SPI 7 IRQ_TYPE_NONE>; /* NMIn */ + interrupt-parent = <&gic>; + interrupt-controller; + #interrupt-cells = <2>; + + dcdc1: regulator-dcdc1 { + compatible = "ti,tps65218-dcdc1"; + regulator-name = "vdd_core"; + regulator-min-microvolt = <912000>; + regulator-max-microvolt = <1144000>; + regulator-boot-on; + regulator-always-on; + }; + + dcdc2: regulator-dcdc2 { + compatible = "ti,tps65218-dcdc2"; + regulator-name = "vdd_mpu"; + regulator-min-microvolt = <912000>; + regulator-max-microvolt = <1378000>; + regulator-boot-on; + regulator-always-on; + }; + + dcdc3: regulator-dcdc3 { + compatible = "ti,tps65218-dcdc3"; + regulator-name = "vdcdc3"; + regulator-min-microvolt = <1350000>; + regulator-max-microvolt = <1350000>; + regulator-boot-on; + regulator-always-on; + }; + + dcdc5: regulator-dcdc5 { + compatible = "ti,tps65218-dcdc5"; + regulator-name = "v1_0bat"; + regulator-min-microvolt = <1000000>; + regulator-max-microvolt = <1000000>; + }; + + dcdc6: regulator-dcdc6 { + compatible = "ti,tps65218-dcdc6"; + regulator-name = "v1_8bat"; + regulator-min-microvolt = <1800000>; + regulator-max-microvolt = <1800000>; + }; + + ldo1: regulator-ldo1 { + compatible = "ti,tps65218-ldo1"; + regulator-min-microvolt = <1800000>; + regulator-max-microvolt = <1800000>; + regulator-boot-on; + regulator-always-on; + }; + }; at24@50 { compatible = "at24,24c256"; -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH v3 5/7] ARM: dts: AM437x: Fix i2c nodes indentation 2014-07-09 5:36 [PATCH v3 0/7] ARM: dts: AM43x: Add devcice tree nodes Keerthy ` (3 preceding siblings ...) 2014-07-09 5:36 ` [PATCH v3 4/7] ARM: dts: AM43x: Add TPS65218 device tree nodes Keerthy @ 2014-07-09 5:36 ` Keerthy 2014-07-09 5:36 ` [PATCH v3 6/7] ARM: dts: AM437x: Add TPS65218 device tree nodes Keerthy ` (2 subsequent siblings) 7 siblings, 0 replies; 19+ messages in thread From: Keerthy @ 2014-07-09 5:36 UTC (permalink / raw) To: devicetree, linux-kernel, linux-omap Cc: mark.rutland, galak, robh+dt, linux, balbi, tony, kishon, broonie, Keerthy Fix i2c nodes indentation. Signed-off-by: Keerthy <j-keerthy@ti.com> Reviewed-by: Felipe Balbi <balbi@ti.com> --- arch/arm/boot/dts/am437x-gp-evm.dts | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/arch/arm/boot/dts/am437x-gp-evm.dts b/arch/arm/boot/dts/am437x-gp-evm.dts index 003766c..85ca430 100644 --- a/arch/arm/boot/dts/am437x-gp-evm.dts +++ b/arch/arm/boot/dts/am437x-gp-evm.dts @@ -257,16 +257,15 @@ }; &i2c0 { - status = "okay"; - pinctrl-names = "default"; - pinctrl-0 = <&i2c0_pins>; + status = "okay"; + pinctrl-names = "default"; + pinctrl-0 = <&i2c0_pins>; }; &i2c1 { - status = "okay"; - pinctrl-names = "default"; - pinctrl-0 = <&i2c1_pins>; - + status = "okay"; + pinctrl-names = "default"; + pinctrl-0 = <&i2c1_pins>; pixcir_ts@5c { compatible = "pixcir,pixcir_tangoc"; pinctrl-names = "default"; -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH v3 6/7] ARM: dts: AM437x: Add TPS65218 device tree nodes 2014-07-09 5:36 [PATCH v3 0/7] ARM: dts: AM43x: Add devcice tree nodes Keerthy ` (4 preceding siblings ...) 2014-07-09 5:36 ` [PATCH v3 5/7] ARM: dts: AM437x: Fix i2c nodes indentation Keerthy @ 2014-07-09 5:36 ` Keerthy 2014-07-09 5:36 ` [PATCH v3 7/7] ARM: configs: omap2plus_defconfig: enable TPS65218 configs Keerthy 2014-07-09 8:31 ` [PATCH v3 0/7] ARM: dts: AM43x: Add devcice tree nodes Tony Lindgren 7 siblings, 0 replies; 19+ messages in thread From: Keerthy @ 2014-07-09 5:36 UTC (permalink / raw) To: devicetree, linux-kernel, linux-omap Cc: mark.rutland, galak, robh+dt, linux, balbi, tony, kishon, broonie, Keerthy Add TPS65218 device tree nodes. i2c clock frequency setting also added as part of tps65218 nodes addition. As i2c clock enabling is required. Signed-off-by: Keerthy <j-keerthy@ti.com> Reviewed-by: Felipe Balbi <balbi@ti.com> --- Changes from V1: * Added dcdc3, dcdc5, dcdc6 nodes. * dcdc4 is not added because of a potential PMIC bug under investigation. arch/arm/boot/dts/am437x-gp-evm.dts | 58 +++++++++++++++++++++++++++++++++++ 1 file changed, 58 insertions(+) diff --git a/arch/arm/boot/dts/am437x-gp-evm.dts b/arch/arm/boot/dts/am437x-gp-evm.dts index 85ca430..f0422c2 100644 --- a/arch/arm/boot/dts/am437x-gp-evm.dts +++ b/arch/arm/boot/dts/am437x-gp-evm.dts @@ -260,6 +260,64 @@ status = "okay"; pinctrl-names = "default"; pinctrl-0 = <&i2c0_pins>; + clock-frequency = <400000>; + + tps65218: tps65218@24 { + reg = <0x24>; + compatible = "ti,tps65218"; + interrupts = <GIC_SPI 7 IRQ_TYPE_NONE>; /* NMIn */ + interrupt-parent = <&gic>; + interrupt-controller; + #interrupt-cells = <2>; + + dcdc1: regulator-dcdc1 { + compatible = "ti,tps65218-dcdc1"; + regulator-name = "vdd_core"; + regulator-min-microvolt = <912000>; + regulator-max-microvolt = <1144000>; + regulator-boot-on; + regulator-always-on; + }; + + dcdc2: regulator-dcdc2 { + compatible = "ti,tps65218-dcdc2"; + regulator-name = "vdd_mpu"; + regulator-min-microvolt = <912000>; + regulator-max-microvolt = <1378000>; + regulator-boot-on; + regulator-always-on; + }; + + dcdc3: regulator-dcdc3 { + compatible = "ti,tps65218-dcdc3"; + regulator-name = "vdcdc3"; + regulator-min-microvolt = <1350000>; + regulator-max-microvolt = <1350000>; + regulator-boot-on; + regulator-always-on; + }; + dcdc5: regulator-dcdc5 { + compatible = "ti,tps65218-dcdc5"; + regulator-name = "v1_0bat"; + regulator-min-microvolt = <1000000>; + regulator-max-microvolt = <1000000>; + }; + + dcdc6: regulator-dcdc6 { + compatible = "ti,tps65218-dcdc6"; + regulator-name = "v1_8bat"; + regulator-min-microvolt = <1800000>; + regulator-max-microvolt = <1800000>; + }; + + ldo1: regulator-ldo1 { + compatible = "ti,tps65218-ldo1"; + regulator-min-microvolt = <1800000>; + regulator-max-microvolt = <1800000>; + regulator-boot-on; + regulator-always-on; + }; + }; }; &i2c1 { -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH v3 7/7] ARM: configs: omap2plus_defconfig: enable TPS65218 configs 2014-07-09 5:36 [PATCH v3 0/7] ARM: dts: AM43x: Add devcice tree nodes Keerthy ` (5 preceding siblings ...) 2014-07-09 5:36 ` [PATCH v3 6/7] ARM: dts: AM437x: Add TPS65218 device tree nodes Keerthy @ 2014-07-09 5:36 ` Keerthy 2014-07-09 8:31 ` [PATCH v3 0/7] ARM: dts: AM43x: Add devcice tree nodes Tony Lindgren 7 siblings, 0 replies; 19+ messages in thread From: Keerthy @ 2014-07-09 5:36 UTC (permalink / raw) To: devicetree, linux-kernel, linux-omap Cc: mark.rutland, galak, robh+dt, linux, balbi, tony, kishon, broonie, Keerthy Enable TPS65218 config options. Signed-off-by: Keerthy <j-keerthy@ti.com> Acked-by: Felipe Balbi <balbi@ti.com> --- arch/arm/configs/omap2plus_defconfig | 2 ++ 1 file changed, 2 insertions(+) diff --git a/arch/arm/configs/omap2plus_defconfig b/arch/arm/configs/omap2plus_defconfig index 536a137..f650f00 100644 --- a/arch/arm/configs/omap2plus_defconfig +++ b/arch/arm/configs/omap2plus_defconfig @@ -180,6 +180,7 @@ CONFIG_TWL4030_WATCHDOG=y CONFIG_MFD_SYSCON=y CONFIG_MFD_PALMAS=y CONFIG_MFD_TPS65217=y +CONFIG_MFD_TPS65218=y CONFIG_MFD_TPS65910=y CONFIG_TWL6040_CORE=y CONFIG_REGULATOR_FIXED_VOLTAGE=y @@ -188,6 +189,7 @@ CONFIG_REGULATOR_TI_ABB=y CONFIG_REGULATOR_TPS65023=y CONFIG_REGULATOR_TPS6507X=y CONFIG_REGULATOR_TPS65217=y +CONFIG_REGULATOR_TPS65218=y CONFIG_REGULATOR_TPS65910=y CONFIG_REGULATOR_TWL4030=y CONFIG_REGULATOR_PBIAS=y -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 19+ messages in thread
* Re: [PATCH v3 0/7] ARM: dts: AM43x: Add devcice tree nodes 2014-07-09 5:36 [PATCH v3 0/7] ARM: dts: AM43x: Add devcice tree nodes Keerthy ` (6 preceding siblings ...) 2014-07-09 5:36 ` [PATCH v3 7/7] ARM: configs: omap2plus_defconfig: enable TPS65218 configs Keerthy @ 2014-07-09 8:31 ` Tony Lindgren 2014-07-09 10:59 ` Keerthy 7 siblings, 1 reply; 19+ messages in thread From: Tony Lindgren @ 2014-07-09 8:31 UTC (permalink / raw) To: Keerthy Cc: devicetree, linux-kernel, linux-omap, mark.rutland, galak, robh+dt, linux, balbi, kishon, broonie * Keerthy <j-keerthy@ti.com> [140708 22:40]: > The patch series adds the device tree nodes and the corresponding > documentation. The series also enabled tps65218 config options > in the omap2plus_defconfig. > > The series is boot tested on both AM43x-epos-evm and AM437x-gp-evm. Are the .dts changes safe to apply separately without breaking anything? Regards, Tony ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH v3 0/7] ARM: dts: AM43x: Add devcice tree nodes 2014-07-09 8:31 ` [PATCH v3 0/7] ARM: dts: AM43x: Add devcice tree nodes Tony Lindgren @ 2014-07-09 10:59 ` Keerthy 2014-07-09 11:51 ` Tony Lindgren 0 siblings, 1 reply; 19+ messages in thread From: Keerthy @ 2014-07-09 10:59 UTC (permalink / raw) To: Tony Lindgren Cc: Keerthy, devicetree, linux-kernel, linux-omap, mark.rutland, galak, robh+dt, linux, balbi, kishon, broonie On Wednesday 09 July 2014 02:01 PM, Tony Lindgren wrote: > * Keerthy <j-keerthy@ti.com> [140708 22:40]: >> The patch series adds the device tree nodes and the corresponding >> documentation. The series also enabled tps65218 config options >> in the omap2plus_defconfig. >> >> The series is boot tested on both AM43x-epos-evm and AM437x-gp-evm. > Are the .dts changes safe to apply separately without breaking > anything? Tony, Patch 1/7 had to be re-based on Mark's latest branch for tps65218. Other patches can be taken as is. These patches are boot tested on both am437x-gp-evm and am43x-epos-evm. > Regards, > > Tony Regards, Keerthy ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH v3 0/7] ARM: dts: AM43x: Add devcice tree nodes 2014-07-09 10:59 ` Keerthy @ 2014-07-09 11:51 ` Tony Lindgren 2014-07-10 13:25 ` Keerthy 0 siblings, 1 reply; 19+ messages in thread From: Tony Lindgren @ 2014-07-09 11:51 UTC (permalink / raw) To: Keerthy Cc: Keerthy, devicetree, linux-kernel, linux-omap, mark.rutland, galak, robh+dt, linux, balbi, kishon, broonie * Keerthy <a0393675@ti.com> [140709 04:03]: > On Wednesday 09 July 2014 02:01 PM, Tony Lindgren wrote: > >* Keerthy <j-keerthy@ti.com> [140708 22:40]: > >>The patch series adds the device tree nodes and the corresponding > >>documentation. The series also enabled tps65218 config options > >>in the omap2plus_defconfig. > >> > >>The series is boot tested on both AM43x-epos-evm and AM437x-gp-evm. > >Are the .dts changes safe to apply separately without breaking > >anything? > Tony, > > Patch 1/7 had to be re-based on Mark's latest branch for tps65218. > Other patches can be taken as is. These patches are boot tested on > both am437x-gp-evm and am43x-epos-evm. OK thanks applying patches 4 - 7 into omap-for-v3.17/dt. Tony ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH v3 0/7] ARM: dts: AM43x: Add devcice tree nodes 2014-07-09 11:51 ` Tony Lindgren @ 2014-07-10 13:25 ` Keerthy 2014-07-10 13:50 ` Keerthy 0 siblings, 1 reply; 19+ messages in thread From: Keerthy @ 2014-07-10 13:25 UTC (permalink / raw) To: Tony Lindgren, broonie Cc: Keerthy, devicetree, linux-kernel, linux-omap, mark.rutland, galak, robh+dt, linux, balbi, kishon On Wednesday 09 July 2014 05:21 PM, Tony Lindgren wrote: > * Keerthy <a0393675@ti.com> [140709 04:03]: >> On Wednesday 09 July 2014 02:01 PM, Tony Lindgren wrote: >>> * Keerthy <j-keerthy@ti.com> [140708 22:40]: >>>> The patch series adds the device tree nodes and the corresponding >>>> documentation. The series also enabled tps65218 config options >>>> in the omap2plus_defconfig. >>>> >>>> The series is boot tested on both AM43x-epos-evm and AM437x-gp-evm. >>> Are the .dts changes safe to apply separately without breaking >>> anything? >> Tony, >> >> Patch 1/7 had to be re-based on Mark's latest branch for tps65218. >> Other patches can be taken as is. These patches are boot tested on >> both am437x-gp-evm and am43x-epos-evm. > OK thanks applying patches 4 - 7 into omap-for-v3.17/dt. > > Tony Thanks Tony. Mark, Could you please pull patch 2 and patch 3? The documentation is missing. Kind Regards, Keerthy ^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH v3 0/7] ARM: dts: AM43x: Add devcice tree nodes 2014-07-10 13:25 ` Keerthy @ 2014-07-10 13:50 ` Keerthy 0 siblings, 0 replies; 19+ messages in thread From: Keerthy @ 2014-07-10 13:50 UTC (permalink / raw) To: Tony Lindgren, broonie Cc: Keerthy, devicetree, linux-kernel, linux-omap, mark.rutland, galak, robh+dt, linux, balbi, kishon On Thursday 10 July 2014 06:55 PM, Keerthy wrote: > On Wednesday 09 July 2014 05:21 PM, Tony Lindgren wrote: >> * Keerthy <a0393675@ti.com> [140709 04:03]: >>> On Wednesday 09 July 2014 02:01 PM, Tony Lindgren wrote: >>>> * Keerthy <j-keerthy@ti.com> [140708 22:40]: >>>>> The patch series adds the device tree nodes and the corresponding >>>>> documentation. The series also enabled tps65218 config options >>>>> in the omap2plus_defconfig. >>>>> >>>>> The series is boot tested on both AM43x-epos-evm and AM437x-gp-evm. >>>> Are the .dts changes safe to apply separately without breaking >>>> anything? >>> Tony, >>> >>> Patch 1/7 had to be re-based on Mark's latest branch for tps65218. >>> Other patches can be taken as is. These patches are boot tested on >>> both am437x-gp-evm and am43x-epos-evm. >> OK thanks applying patches 4 - 7 into omap-for-v3.17/dt. >> >> Tony > > Thanks Tony. > > Mark, > > Could you please pull patch 2 and patch 3? The documentation is missing. > https://patchwork.kernel.org/patch/4511721/ and https://patchwork.kernel.org/patch/4511751/ Regards, Keerthy > Kind Regards, > Keerthy ^ permalink raw reply [flat|nested] 19+ messages in thread
end of thread, other threads:[~2014-07-10 14:52 UTC | newest] Thread overview: 19+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2014-07-09 5:36 [PATCH v3 0/7] ARM: dts: AM43x: Add devcice tree nodes Keerthy 2014-07-09 5:36 ` [PATCH v3 1/7] regulator: tps65218: Add fixed_uV fields for dcdc5 and dcdc6 Keerthy 2014-07-09 9:14 ` Mark Brown 2014-07-09 9:36 ` Keerthy 2014-07-09 10:22 ` [PATCH v4] " Keerthy 2014-07-10 12:12 ` Mark Brown 2014-07-10 13:10 ` Keerthy 2014-07-10 14:50 ` Mark Brown 2014-07-09 5:36 ` [PATCH v3 2/7] mfd: Add DT bindings for tps65218 PMIC Keerthy 2014-07-09 5:36 ` [PATCH v3 3/7] regulator: Add DT bindings for tps65218 PMIC regulators Keerthy 2014-07-09 5:36 ` [PATCH v3 4/7] ARM: dts: AM43x: Add TPS65218 device tree nodes Keerthy 2014-07-09 5:36 ` [PATCH v3 5/7] ARM: dts: AM437x: Fix i2c nodes indentation Keerthy 2014-07-09 5:36 ` [PATCH v3 6/7] ARM: dts: AM437x: Add TPS65218 device tree nodes Keerthy 2014-07-09 5:36 ` [PATCH v3 7/7] ARM: configs: omap2plus_defconfig: enable TPS65218 configs Keerthy 2014-07-09 8:31 ` [PATCH v3 0/7] ARM: dts: AM43x: Add devcice tree nodes Tony Lindgren 2014-07-09 10:59 ` Keerthy 2014-07-09 11:51 ` Tony Lindgren 2014-07-10 13:25 ` Keerthy 2014-07-10 13:50 ` Keerthy
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox