Skip to contentSkip to conversation

regulator: rtq6752: Add binding document for Richtek RTQ6752

PATCH In review cy_huang 4 participants 5 messages Jul 9, 2021 – Jul 14, 2021

Subsystems (2)
lore
Share & tools
lore

cy_huang

author
#0 lore raw

PATCH 1/2 regulator: rtq6752: Add binding document for Richtek RTQ6752

From: ChiYuan Huang <[email protected]>

Add binding document for Richtek RTQ6752.

---
 .../regulator/richtek,rtq6752-regulator.yaml       | 78 ++++++++++++++++++++++
 1 file changed, 78 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/regulator/richtek,rtq6752-regulator.yaml
Documentation/devicetree/bindings/regulator/richtek,rtq6752-regulator.yaml new file +78 -0
new file mode 100644
index 00000000..641840e
@@ -0,0 +1,78 @@
+# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/regulator/richtek,rtq6752-regulator.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Richtek RTQ6752 TFT LCD Voltage Regulator
+
+maintainers:
+  - ChiYuan Huang <[email protected]>
+
+description: |
+  The RTQ6752 is an I2C interface pgorammable power management IC. It includes
+  two synchronous boost converter for PAVDD, and one synchronous NAVDD
+  buck-boost. The device is suitable for automotive TFT-LCD panel.
+
+properties:
+  compatible:
+    enum:
+      - richtek,rtq6752
+
+  reg:
+    maxItems: 1
+
+  enable-gpios:
+    description: |
+      A connection of the chip 'enable' gpio line. If not provided, treat it as
+      external pull up.
+    maxItems: 1
+
+  regulators:
+    type: object
+    $ref: regulator.yaml#
+
+    patternProperties:
+      "^(p|n)avdd$":
+        type: object
+        $ref: regulator.yaml#
+        description: |
+          regulator description for pavdd and navdd.
+
+    additionalProperties: false
+
+required:
+  - compatible
+  - reg
+  - regulators
+
+additionalProperties: false
+
+examples:
+  - |
+    i2c {
+      #address-cells = <1>;
+      #size-cells = <0>;
+
+      rtq6752@6b {
+        compatible = "richtek,rtq6752";
+        status = "okay";
+        reg = <0x6b>;
+        enable-gpios = <&gpio26 2 0>;
+
+        regulators {
+          pavdd {
+            regulator-name = "rtq6752-pavdd";
+            regulator-min-microvolt = <5000000>;
+            regulator-max-microvolt = <7300000>;
+            regulator-boot-on;
+          };
+          navdd {
+            regulator-name = "rtq6752-navdd";
+            regulator-min-microvolt = <5000000>;
+            regulator-max-microvolt = <7300000>;
+            regulator-boot-on;
+          };
+        };
+      };
+    };
Signature · 1 line
2.7.4
author
#1 lore raw

PATCH 2/2 regulator: rt6752: Add support for Richtek RTQ6752

From: ChiYuan Huang <[email protected]>

Add suport for Richtek RTQ6752.

---
 drivers/regulator/Kconfig             |   9 ++
 drivers/regulator/Makefile            |   1 +
 drivers/regulator/rtq6752-regulator.c | 285 ++++++++++++++++++++++++++++++++++
 3 files changed, 295 insertions(+)
 create mode 100644 drivers/regulator/rtq6752-regulator.c
drivers/regulator/Kconfig +9 -0
index 24ce9a1..6562d4c 100644
@@ -1066,6 +1066,15 @@ config REGULATOR_RTMV20
 	  the Richtek RTMV20. It can support the load current up to 6A and
 	  integrate strobe/vsync/fsin signal to synchronize the IR camera.
 
+config REGULATOR_RTQ6752
+	tristate "Richtek RTQ6752 TFT LCD voltage regulator"
+	depends on I2C
+	select REGMAP_I2C
+	help
+	  This driver adds support for Richtek RTQ6752. RTQ6752 includes two
+	  synchronous boost converters for PAVDD, and one synchronous NAVDD
+	  buck-boost. This device is suitable for automotive TFT-LCD panel.
+
 config REGULATOR_S2MPA01
 	tristate "Samsung S2MPA01 voltage regulator"
 	depends on MFD_SEC_CORE || COMPILE_TEST
drivers/regulator/Makefile +1 -0
index 8c2f822..102b2a0 100644
@@ -128,6 +128,7 @@ obj-$(CONFIG_REGULATOR_RT5033)	+= rt5033-regulator.o
 obj-$(CONFIG_REGULATOR_RT6160)	+= rt6160-regulator.o
 obj-$(CONFIG_REGULATOR_RT6245)	+= rt6245-regulator.o
 obj-$(CONFIG_REGULATOR_RTMV20)	+= rtmv20-regulator.o
+obj-$(CONFIG_REGULATOR_RTQ6752)	+= rtq6752-regulator.o
 obj-$(CONFIG_REGULATOR_S2MPA01) += s2mpa01.o
 obj-$(CONFIG_REGULATOR_S2MPS11) += s2mps11.o
 obj-$(CONFIG_REGULATOR_S5M8767) += s5m8767.o
drivers/regulator/rtq6752-regulator.c new file +285 -0

285 lines not on this page · raw message

Signature · 1 line
2.7.4
maintainer
#2 lore raw
In reply to cy_huang
Quoted 1 line · cy_huang

Add binding document for Richtek RTQ6752.

Applied to

https://git.kernel.org/pub/scm/linux/kernel/git/broonie/regulator.git for-next

Thanks!

[1/2] regulator: rtq6752: Add binding document for Richtek RTQ6752

      commit: 526e99cf43fe3884014f9e9f2ca5d04d56007287
[2/2] regulator: rt6752: Add support for Richtek RTQ6752
      commit: f40f9409719fa092924803723b9445be13fac8fb

All being well this means that it will be integrated into the linux-next tree (usually sometime in the next 24 hours) and sent to Linus during the next merge window (or sooner if it is a bug fix), however if problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing and review of the tree, please engage with people reporting problems and send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they should be sent as incremental updates against current git, existing patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying to this mail.

Thanks,
Mark

maintainer
#3 lore raw
In reply to cy_huang
Quoted 39 lines · cy_huang

From: ChiYuan Huang <[email protected]>

Add binding document for Richtek RTQ6752.

---
 .../regulator/richtek,rtq6752-regulator.yaml       | 78 ++++++++++++++++++++++
 1 file changed, 78 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/regulator/richtek,rtq6752-regulator.yaml
Documentation/devicetree/bindings/regulator/richtek,rtq6752-regulator.yaml new file +23 -0
new file mode 100644
index 00000000..641840e
@@ -0,0 +1,78 @@
+# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/regulator/richtek,rtq6752-regulator.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Richtek RTQ6752 TFT LCD Voltage Regulator
+
+maintainers:
+  - ChiYuan Huang <[email protected]>
+
+description: |
+  The RTQ6752 is an I2C interface pgorammable power management IC. It includes
+  two synchronous boost converter for PAVDD, and one synchronous NAVDD
+  buck-boost. The device is suitable for automotive TFT-LCD panel.
+
+properties:
+  compatible:
+    enum:
+      - richtek,rtq6752
+
+  reg:
+    maxItems: 1

Documentation/devicetree/bindings/regulator/richtek,rtq6752-regulator.yaml line 33

Quoted 10 lines
Documentation/devicetree/bindings/regulator/richtek,rtq6752-regulator.yaml new file +10 -0
+
+  enable-gpios:
+    description: |
+      A connection of the chip 'enable' gpio line. If not provided, treat it as
+      external pull up.
+    maxItems: 1
+
+  regulators:
+    type: object
+    $ref: regulator.yaml#

This node is not a regulator, just a container. Drop.

Quoted 16 lines
Documentation/devicetree/bindings/regulator/richtek,rtq6752-regulator.yaml new file +16 -0
+
+    patternProperties:
+      "^(p|n)avdd$":
+        type: object
+        $ref: regulator.yaml#
+        description: |
+          regulator description for pavdd and navdd.
+
+    additionalProperties: false
+
+required:
+  - compatible
+  - reg
+  - regulators
+
+additionalProperties: false

Documentation/devicetree/bindings/regulator/richtek,rtq6752-regulator.yaml line 59

Quoted 10 lines
Documentation/devicetree/bindings/regulator/richtek,rtq6752-regulator.yaml new file +10 -0
+
+examples:
+  - |
+    i2c {
+      #address-cells = <1>;
+      #size-cells = <0>;
+
+      rtq6752@6b {
+        compatible = "richtek,rtq6752";
+        status = "okay";

Don't show status in examples.

Quoted 23 lines
Documentation/devicetree/bindings/regulator/richtek,rtq6752-regulator.yaml new file +19 -0
+        reg = <0x6b>;
+        enable-gpios = <&gpio26 2 0>;
+
+        regulators {
+          pavdd {
+            regulator-name = "rtq6752-pavdd";
+            regulator-min-microvolt = <5000000>;
+            regulator-max-microvolt = <7300000>;
+            regulator-boot-on;
+          };
+          navdd {
+            regulator-name = "rtq6752-navdd";
+            regulator-min-microvolt = <5000000>;
+            regulator-max-microvolt = <7300000>;
+            regulator-boot-on;
+          };
+        };
+      };
+    };
Signature · 1 line
2.7.4
#4 lore raw
Quoted 52 lines · cy_huang
Quoted 51 lines

From: ChiYuan Huang <[email protected]>

Add binding document for Richtek RTQ6752.

---
 .../regulator/richtek,rtq6752-regulator.yaml       | 78
++++++++++++++++++++++
 1 file changed, 78 insertions(+)
 create mode 100644

Documentation/devicetree/bindings/regulator/richtek,rtq6752-regulator.yaml

(quoted diff) +0 -0

regulator.yaml b/Documentation/devicetree/bindings/regulator/richtek,rtq6752-regulator.yaml new file mode 100644
index 00000000..641840e

Documentation/devicetree/bindings/regulator/richtek,rtq6752- new file +0 -0

regulator.yaml

Documentation/devicetree/bindings/regulator/richtek,rtq6752- new file +4 -0
@@ -0,0 +1,78 @@
+# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/regulator/richtek,rtq6752-regulator.yaml

#
+$schema: http://devicetree.org/meta-schemas/core.yaml#

Documentation/devicetree/bindings/regulator/richtek,rtq6752- new file +8 -0
+
+title: Richtek RTQ6752 TFT LCD Voltage Regulator
+
+maintainers:
+  - ChiYuan Huang <[email protected]>
+
+description: |
+  The RTQ6752 is an I2C interface pgorammable power management IC. It

includes

Documentation/devicetree/bindings/regulator/richtek,rtq6752- new file +14 -0
+  two synchronous boost converter for PAVDD, and one synchronous NAVDD
+  buck-boost. The device is suitable for automotive TFT-LCD panel.
+
+properties:
+  compatible:
+    enum:
+      - richtek,rtq6752
+
+  reg:
+    maxItems: 1
+
+  enable-gpios:
+    description: |
+      A connection of the chip 'enable' gpio line. If not provided, treat
Quoted 9 lines

Documentation/devicetree/bindings/regulator/richtek,rtq6752-

Quoted 7 lines

it as

Documentation/devicetree/bindings/regulator/richtek,rtq6752- new file +6 -0
+      external pull up.
+    maxItems: 1
+
+  regulators:
+    type: object
+    $ref: regulator.yaml#

This node is not a regulator, just a container. Drop.

Yes.

Quoted 20 lines
Quoted 20 lines
Documentation/devicetree/bindings/regulator/richtek,rtq6752- new file +19 -0
+
+    patternProperties:
+      "^(p|n)avdd$":
+        type: object
+        $ref: regulator.yaml#
+        description: |
+          regulator description for pavdd and navdd.
+
+    additionalProperties: false
+
+required:
+  - compatible
+  - reg
+  - regulators
+
+additionalProperties: false
+
+examples:
+  - |
Quoted 9 lines

Documentation/devicetree/bindings/regulator/richtek,rtq6752-

Quoted 7 lines
Documentation/devicetree/bindings/regulator/richtek,rtq6752- new file +7 -0
+    i2c {
+      #address-cells = <1>;
+      #size-cells = <0>;
+
+      rtq6752@6b {
+        compatible = "richtek,rtq6752";
+        status = "okay";

Don't show status in examples.

Okay.

I'll use one patch to update the above comment.
Thanks.

Quoted 12 lines
Quoted 12 lines
Documentation/devicetree/bindings/regulator/richtek,rtq6752- new file +11 -0
+        reg = <0x6b>;
+        enable-gpios = <&gpio26 2 0>;
+
+        regulators {
+          pavdd {
+            regulator-name = "rtq6752-pavdd";
+            regulator-min-microvolt = <5000000>;
+            regulator-max-microvolt = <7300000>;
+            regulator-boot-on;
+          };
+          navdd {
Quoted 8 lines

Documentation/devicetree/bindings/regulator/richtek,rtq6752-

Quoted 8 lines
Documentation/devicetree/bindings/regulator/richtek,rtq6752- new file +8 -0
+            regulator-name = "rtq6752-navdd";
+            regulator-min-microvolt = <5000000>;
+            regulator-max-microvolt = <7300000>;
+            regulator-boot-on;
+          };
+        };
+      };
+    };

************* Email Confidentiality Notice ********************

The information contained in this e-mail message (including any attachments) may be confidential, proprietary, privileged, or otherwise exempt from disclosure under applicable laws. It is intended to be conveyed only to the designated recipient(s). Any use, dissemination, distribution, printing, retaining or copying of this e-mail (including its attachments) by unintended recipient(s) is strictly prohibited and may be unlawful. If you are not an intended recipient of this e-mail, or believe that you have received this e-mail in error, please notify the sender immediately (by replying to this e-mail), delete any and all copies of this e-mail (including any attachments) from your system, and do not disclose the content of this e-mail to any other person. Thank you!