From: Svyatoslav Ryhel <clamor95@gmail.com>
To: Liam Girdwood <lgirdwood@gmail.com>,
Mark Brown <broonie@kernel.org>, Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
Svyatoslav Ryhel <clamor95@gmail.com>
Cc: linux-kernel@vger.kernel.org, devicetree@vger.kernel.org
Subject: [PATCH v1 1/7] regulator: dt-bindings: ti,lp872x: Convert to DT schema
Date: Tue, 6 Oct 2026 18:44:25 +0300 [thread overview]
Message-ID: <20261006154431.318936-2-clamor95@gmail.com> (raw)
In-Reply-To: <20261006154431.318936-1-clamor95@gmail.com>
Convert TI LP872x regulator binding from text to DT schema.
Signed-off-by: Svyatoslav Ryhel <clamor95@gmail.com>
---
.../devicetree/bindings/regulator/lp872x.txt | 161 ------------
.../bindings/regulator/ti,lp8720.yaml | 232 ++++++++++++++++++
2 files changed, 232 insertions(+), 161 deletions(-)
delete mode 100644 Documentation/devicetree/bindings/regulator/lp872x.txt
create mode 100644 Documentation/devicetree/bindings/regulator/ti,lp8720.yaml
diff --git a/Documentation/devicetree/bindings/regulator/lp872x.txt b/Documentation/devicetree/bindings/regulator/lp872x.txt
deleted file mode 100644
index ab895cd1cac1a..0000000000000
--- a/Documentation/devicetree/bindings/regulator/lp872x.txt
+++ /dev/null
@@ -1,161 +0,0 @@
-Binding for TI/National Semiconductor LP872x Driver
-
-Required properties:
- - compatible: "ti,lp8720" or "ti,lp8725"
- - reg: I2C slave address. 0x7d = LP8720, 0x7a = LP8725
-
-Optional properties:
- - ti,general-config: the value of LP872X_GENERAL_CFG register (u8)
- (LP8720)
- bit[2]: BUCK output voltage control by external DVS pin or register
- 1 = external pin, 0 = bit7 of register 08h
- bit[1]: sleep control by external DVS pin or register
- 1 = external pin, 0 = bit6 of register 08h
- bit[0]: time step unit(usec). 1 = 25, 0 = 50
-
- (LP8725)
- bit[7:6]: time step unit(usec). 00 = 32, 01 = 64, 10 = 128, 11 = 256
- bit[4]: BUCK2 enable control. 1 = enable, 0 = disable
- bit[3]: BUCK2 output voltage register address. 1 = 0Ah, 0 = 0Bh
- bit[2]: BUCK1 output voltage control by external DVS pin or register
- 1 = register 08h, 0 = DVS
- bit[1]: LDO sleep control. 1 = sleep mode, 0 = normal
- bit[0]: BUCK1 enable control, 1 = enable, 0 = disable
-
- For more details, please see the datasheet.
-
- - ti,update-config: define it when LP872X_GENERAL_CFG register should be set
- - ti,dvs-gpio: GPIO specifier for external DVS pin control of LP872x devices.
- - ti,dvs-vsel: DVS selector. 0 = SEL_V1, 1 = SEL_V2.
- - ti,dvs-state: initial DVS pin state. 0 = DVS_LOW, 1 = DVS_HIGH.
- - enable-gpios: GPIO specifier for EN pin control of LP872x devices.
-
- Sub nodes for regulator_init_data
- LP8720 has maximum 6 nodes. (child name: ldo1 ~ 5 and buck)
- LP8725 has maximum 9 nodes. (child name: ldo1 ~ 5, lilo1,2 and buck1,2)
- For more details, please see the following binding document.
- (Documentation/devicetree/bindings/regulator/regulator.txt)
-
-Datasheet
- - LP8720: https://www.ti.com/lit/ds/symlink/lp8720.pdf
- - LP8725: https://www.ti.com/lit/ds/symlink/lp8725.pdf
-
-Example 1) LP8720
-
-lp8720@7d {
- compatible = "ti,lp8720";
- reg = <0x7d>;
-
- /* external DVS pin used, timestep is 25usec */
- ti,general-config = /bits/ 8 <0x03>;
- ti,update-config;
-
- /*
- * The dvs-gpio depends on the processor environment.
- * For example, following GPIO specifier means GPIO134 in OMAP4.
- */
- ti,dvs-gpio = <&gpio5 6 0>;
- ti,dvs-vsel = /bits/ 8 <1>; /* SEL_V2 */
- ti,dvs-state = /bits/ 8 <1>; /* DVS_HIGH */
-
- vaf: ldo1 {
- regulator-min-microvolt = <1200000>;
- regulator-max-microvolt = <3300000>;
- };
-
- vmmc: ldo2 {
- regulator-min-microvolt = <1200000>;
- regulator-max-microvolt = <3300000>;
- };
-
- vcam_io: ldo3 {
- regulator-min-microvolt = <1200000>;
- regulator-max-microvolt = <3300000>;
- regulator-boot-on;
- };
-
- vcam_core: ldo4 {
- regulator-min-microvolt = <800000>;
- regulator-max-microvolt = <2850000>;
- regulator-boot-on;
- };
-
- vcam: ldo5 {
- regulator-min-microvolt = <1200000>;
- regulator-max-microvolt = <3300000>;
- };
-
- vcc: buck {
- regulator-name = "VBUCK";
- regulator-min-microvolt = <800000>;
- regulator-max-microvolt = <2300000>;
- };
-};
-
-Example 2) LP8725
-
-lp8725@7a {
- compatible = "ti,lp8725";
- reg = <0x7a>;
-
- /* Enable BUCK1,2, no DVS, normal LDO mode, timestep is 256usec */
- ti,general-config = /bits/ 8 <0xdd>;
- ti,update-config;
-
- vcam_io: ldo1 {
- regulator-min-microvolt = <1200000>;
- regulator-max-microvolt = <3300000>;
- };
-
- vcam_core: ldo2 {
- regulator-min-microvolt = <1200000>;
- regulator-max-microvolt = <3300000>;
- };
-
- vcam: ldo3 {
- regulator-min-microvolt = <1200000>;
- regulator-max-microvolt = <3300000>;
- };
-
- vcmmb_io: ldo4 {
- regulator-min-microvolt = <1200000>;
- regulator-max-microvolt = <3300000>;
- regulator-boot-on;
- };
-
- vcmmb_core: ldo5 {
- regulator-min-microvolt = <1200000>;
- regulator-max-microvolt = <3300000>;
- regulator-boot-on;
- };
-
- vaux1: lilo1 {
- regulator-name = "VAUX1";
- regulator-min-microvolt = <800000>;
- regulator-max-microvolt = <3300000>;
- };
-
- vaux2: lilo2 {
- regulator-name = "VAUX2";
- regulator-min-microvolt = <800000>;
- regulator-max-microvolt = <3300000>;
- };
-
- vcc1: buck1 {
- regulator-name = "VBUCK1";
- regulator-min-microvolt = <800000>;
- regulator-max-microvolt = <3000000>;
- regulator-min-microamp = <460000>;
- regulator-max-microamp = <1370000>;
- regulator-boot-on;
- };
-
- vcc2: buck2 {
- regulator-name = "VBUCK2";
- regulator-min-microvolt = <800000>;
- regulator-max-microvolt = <3000000>;
- regulator-min-microamp = <460000>;
- regulator-max-microamp = <1370000>;
- regulator-boot-on;
- };
-};
diff --git a/Documentation/devicetree/bindings/regulator/ti,lp8720.yaml b/Documentation/devicetree/bindings/regulator/ti,lp8720.yaml
new file mode 100644
index 0000000000000..f2384749314f1
--- /dev/null
+++ b/Documentation/devicetree/bindings/regulator/ti,lp8720.yaml
@@ -0,0 +1,232 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/regulator/ti,lp8720.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: TI/National Semiconductor LP872x PMIC
+
+description: >
+ The LP8720 is a multi-function, programmable Power Management Unit,
+ optimized for sub block power requirement solutions. This device
+ integrates one highly efficient 400 mA step-down DC-DC converter with
+ Dynamic Voltage Scale (DVS), five low-noise low dropout (LDO) voltage
+ regulators, and a 400 KHz i2c-compatible interface to allow a host
+ controller access to the internal control registers of the LP8720.
+ Additionally, the LP8720 features programmable power-on sequencing.
+
+ LP8720: https://www.ti.com/lit/ds/symlink/lp8720.pdf
+ LP8725: https://www.ti.com/lit/ds/symlink/lp8725.pdf
+
+maintainers:
+ - Svyatoslav Ryhel <clamor95@gmail.com>
+
+properties:
+ compatible:
+ enum:
+ - ti,lp8720
+ - ti,lp8725
+
+ reg:
+ maxItems: 1
+
+ enable-gpios:
+ description: GPIO specifier for EN pin control.
+ maxItems: 1
+
+ dvs-gpios:
+ description: GPIO specifier for external DVS pin control.
+ maxItems: 1
+
+ ti,general-config:
+ $ref: /schemas/types.yaml#/definitions/uint8
+ description: >
+ The value of LP872X_GENERAL_CFG register
+
+ (LP8720)
+ bit[2]: BUCK output voltage control by external DVS pin or register
+ 1 = external pin, 0 = bit7 of register 08h
+ bit[1]: sleep control by external DVS pin or register
+ 1 = external pin, 0 = bit6 of register 08h
+ bit[0]: time step unit(usec). 1 = 25, 0 = 50
+
+ (LP8725)
+ bit[7:6]: time step unit(usec). 00 = 32, 01 = 64, 10 = 128, 11 = 256
+ bit[4]: BUCK2 enable control. 1 = enable, 0 = disable
+ bit[3]: BUCK2 output voltage register address. 1 = 0Ah, 0 = 0Bh
+ bit[2]: BUCK1 output voltage control by external DVS pin or register
+ 1 = register 08h, 0 = DVS
+ bit[1]: LDO sleep control. 1 = sleep mode, 0 = normal
+ bit[0]: BUCK1 enable control, 1 = enable, 0 = disable
+
+ For more details, please see the datasheet.
+
+ ti,update-config:
+ type: boolean
+ description: Define it when LP872X_GENERAL_CFG register should be set
+
+ ti,dvs-vsel:
+ $ref: /schemas/types.yaml#/definitions/uint8
+ description: DVS selector. 0 = SEL_V1, 1 = SEL_V2.
+
+ ti,dvs-state:
+ $ref: /schemas/types.yaml#/definitions/uint8
+ description: Initial DVS pin state. 0 = DVS_LOW, 1 = DVS_HIGH.
+
+patternProperties:
+ "^(buck|buck[1-2]|ldo[1-5]|lilo[1-2])$":
+ $ref: /schemas/regulator/regulator.yaml#
+ unevaluatedProperties: false
+
+allOf:
+ - if:
+ properties:
+ compatible:
+ contains:
+ const: ti,lp8720
+ then:
+ patternProperties:
+ "^(buck[1-2]|lilo[1-2])$": false
+
+ - if:
+ properties:
+ compatible:
+ contains:
+ const: ti,lp8725
+ then:
+ properties:
+ buck: false
+
+required:
+ - compatible
+ - reg
+
+additionalProperties: false
+
+examples:
+ - |
+ i2c {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ pmic@7d {
+ compatible = "ti,lp8720";
+ reg = <0x7d>;
+
+ /* external DVS pin used, timestep is 25usec */
+ ti,general-config = /bits/ 8 <0x03>;
+ ti,update-config;
+
+ /*
+ * The dvs-gpio depends on the processor environment.
+ * For example, following GPIO specifier means GPIO134 in OMAP4.
+ */
+ dvs-gpios = <&gpio5 6 0>;
+ ti,dvs-vsel = /bits/ 8 <1>; /* SEL_V2 */
+ ti,dvs-state = /bits/ 8 <1>; /* DVS_HIGH */
+
+ ldo1 {
+ regulator-min-microvolt = <1200000>;
+ regulator-max-microvolt = <3300000>;
+ };
+
+ ldo2 {
+ regulator-min-microvolt = <1200000>;
+ regulator-max-microvolt = <3300000>;
+ };
+
+ ldo3 {
+ regulator-min-microvolt = <1200000>;
+ regulator-max-microvolt = <3300000>;
+ regulator-boot-on;
+ };
+
+ ldo4 {
+ regulator-min-microvolt = <800000>;
+ regulator-max-microvolt = <2850000>;
+ regulator-boot-on;
+ };
+
+ ldo5 {
+ regulator-min-microvolt = <1200000>;
+ regulator-max-microvolt = <3300000>;
+ };
+
+ buck {
+ regulator-name = "VBUCK";
+ regulator-min-microvolt = <800000>;
+ regulator-max-microvolt = <2300000>;
+ };
+ };
+ };
+ - |
+ i2c {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ pmic@7a {
+ compatible = "ti,lp8725";
+ reg = <0x7a>;
+
+ /* Enable BUCK1,2, no DVS, normal LDO mode, timestep is 256usec */
+ ti,general-config = /bits/ 8 <0xdd>;
+ ti,update-config;
+
+ ldo1 {
+ regulator-min-microvolt = <1200000>;
+ regulator-max-microvolt = <3300000>;
+ };
+
+ ldo2 {
+ regulator-min-microvolt = <1200000>;
+ regulator-max-microvolt = <3300000>;
+ };
+
+ ldo3 {
+ regulator-min-microvolt = <1200000>;
+ regulator-max-microvolt = <3300000>;
+ };
+
+ ldo4 {
+ regulator-min-microvolt = <1200000>;
+ regulator-max-microvolt = <3300000>;
+ regulator-boot-on;
+ };
+
+ ldo5 {
+ regulator-min-microvolt = <1200000>;
+ regulator-max-microvolt = <3300000>;
+ regulator-boot-on;
+ };
+
+ lilo1 {
+ regulator-name = "VAUX1";
+ regulator-min-microvolt = <800000>;
+ regulator-max-microvolt = <3300000>;
+ };
+
+ lilo2 {
+ regulator-name = "VAUX2";
+ regulator-min-microvolt = <800000>;
+ regulator-max-microvolt = <3300000>;
+ };
+
+ buck1 {
+ regulator-name = "VBUCK1";
+ regulator-min-microvolt = <800000>;
+ regulator-max-microvolt = <3000000>;
+ regulator-min-microamp = <460000>;
+ regulator-max-microamp = <1370000>;
+ regulator-boot-on;
+ };
+
+ buck2 {
+ regulator-name = "VBUCK2";
+ regulator-min-microvolt = <800000>;
+ regulator-max-microvolt = <3000000>;
+ regulator-min-microamp = <460000>;
+ regulator-max-microamp = <1370000>;
+ regulator-boot-on;
+ };
+ };
+ };
--
2.53.0
next prev parent reply other threads:[~2026-10-06 15:44 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-10-06 15:44 [PATCH v1 0/7] lp872x: remove platform_data based probing and adjustments Svyatoslav Ryhel
2026-10-06 15:44 ` Svyatoslav Ryhel [this message]
2026-10-06 15:44 ` [PATCH v1 2/7] regulator: dt-bindings: ti,lp872x: Document input supplies Svyatoslav Ryhel
2026-10-06 15:44 ` [PATCH v1 3/7] regulator: lp872x: Fold driver header into driver file Svyatoslav Ryhel
2026-10-06 15:44 ` [PATCH v1 4/7] regulator: lp872x: Remove redundant regmap wrappers Svyatoslav Ryhel
2026-10-06 15:44 ` [PATCH v1 5/7] regulator: lp872x: Remove platform_data based probing Svyatoslav Ryhel
2026-10-07 8:39 ` Mark Brown
2026-10-07 8:59 ` Svyatoslav Ryhel
2026-10-06 15:44 ` [PATCH v1 6/7] regulator: lp872x: Add support for input and regulator power supplies Svyatoslav Ryhel
2026-10-06 15:44 ` [PATCH v1 7/7] regulator: lp872x: Use macro to fill regulator descriptor Svyatoslav Ryhel
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20261006154431.318936-2-clamor95@gmail.com \
--to=clamor95@gmail.com \
--cc=broonie@kernel.org \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=krzk+dt@kernel.org \
--cc=lgirdwood@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=robh@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®