Received-SPF: pass (google.com: domain of linux-kernel+bounces-162981-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
From: Nathan Morrisson <nmorrisson@phytec.com>
To: lee@kernel.org,
	robh@kernel.org,
	krzk+dt@kernel.org,
	conor+dt@kernel.org,
	nm@ti.com,
	vigneshr@ti.com,
	kristo@kernel.org,
	j-keerthy@ti.com
Cc: devicetree@vger.kernel.org,
	linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	upstream@lists.phytec.de,
	w.egorov@phytec.de
Subject: [PATCH v2 1/2] dt-bindings: mfd: Convert lp873x.txt to json-schema
Date: Mon, 29 Apr 2024 12:58:29 -0700
Message-Id: <20240429195830.4027250-1-nmorrisson@phytec.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?us-ascii?Q?rN3UNMBTno4P23L/2xcFRMVPnFO1X7d9gnfQcX1QX5Y0H1G2Gu+cJazHcp4s?=
 =?us-ascii?Q?Uz7TEf8UC5jcm6KaC9vm/3ErHOafVVAaHeXo1S0RixQ4RjKZl8gTjgiUdwQM?=
 =?us-ascii?Q?eS/V+FT8FoDnnmhmFsQcONv9dXg4X43QEx8uw61P9/BWrcf9hNbb4MNCoQGZ?=
 =?us-ascii?Q?MfnEx6s6PeqLP9/1PRZ1isw1ugriOTvWghzYnInUiApSwLv7Ma/Cs7qRZNUZ?=
 =?us-ascii?Q?Ao3F6rTmvj6xOI0B7Kec80S4efGvJAEPnnU5lqlXjngXYxAzaSFetwb8plit?=
 =?us-ascii?Q?St10cNM1PmjSnzuMX7Uj16eILmk1hfxAGNCfrOs6PqyupY6RCG1rbHxrVAr3?=
 =?us-ascii?Q?aIJGwbjoGQN3FoWpep1Ynq9nw61vh2WjgdnAfQPmhDUR2CqCldEXdfx1kKcc?=
 =?us-ascii?Q?OqUxhhplDwXScsUOT8nmRjD1422hTbHAJJfoH9eZgXUcRcidjO/jCDuP9dP3?=
 =?us-ascii?Q?sADW4DlAytdefATJQmwmcRXhc1ZjV2o4ijc/NzWYJUYbHlt80ots9X54f7o/?=
 =?us-ascii?Q?LbRwNDcJvpmCD+CR/WriNDlw0wSNgv61c3sGOaa3xLktuYy3Eoqb1yMIhivI?=
 =?us-ascii?Q?q+NNbhnZAAFOkBWv77ZNd2dJlSfcg84s1qvomtMF9d4qfHTV4tewIef2WkwE?=
 =?us-ascii?Q?DndcdILtyuiqnUZEPde/whehd3eq/r4GQPPP9aqnMdYZrLrap4GRvftEij75?=
 =?us-ascii?Q?Z3Qax0WozTimKF0T3pWN/8MzBv+aK7q59OEjLuXAjIoyORL7M3nMCjr+KxxC?=
 =?us-ascii?Q?fRcWM0Ifr6auIj5vweqqDEZjCbGqNI0EVahru7TzFGqSGMO0x5HfkktuCRdM?=
 =?us-ascii?Q?/FDTFvGNG6ISDIfvEOog4mVsGr1B7G/zB3CtT+SXr02i1G0MX5muiLFyJQfA?=
 =?us-ascii?Q?mEbfAYJYcg6oFxrnBkjf11pSQT3Lza7LpruWovo6WYujE6/wN+J+NYuwym2H?=
 =?us-ascii?Q?UDyuaWEY5FqBVU3nXMZsyt6pINEFcRW75How3VirmI68/4lve2gi4aOAnGRo?=
 =?us-ascii?Q?upkU0qHchGoyTGZeKIrmEI/tb6IW/PG+d1DTNPcHDcawKlBzAjwI/E/3qrCY?=
 =?us-ascii?Q?9aQM8DKVF550BWKB7VRqsNxn3WO9hwbZeiG2xdZ+LGIpgThobqtIuzGYZok4?=
 =?us-ascii?Q?Xx6K7GmBGADbMARl7J/p//xvvqzKIr9xDf5HHRu8ZJ2wA1sGwHp8heLCTrS+?=
 =?us-ascii?Q?sD9lp/sEqz6n7m98Mpa+DmuOh39Cg0SGeOI6A2u00dQ+gd9UIK5N2ulEkz8x?=
 =?us-ascii?Q?TYlkNWXkubpmJSgt01lpVKPoP2HWXOSS/T8wZ4NDIWzibf3cTylLw7flYkwk?=
 =?us-ascii?Q?y2VMdDAe9keWCrJdJyRJok6PzNS6dPkbY3/QtLLO+X+twnwnWRtWHnPSFYY5?=
 =?us-ascii?Q?caZWy+XoNXmlNDTFwwbcZAw7ayUcgp3dnm7Q7mhnAyDCd4TV2hcBOaeEzleW?=
 =?us-ascii?Q?oZ3UHxiH8ewsxtv5eFIXGHFJ5zFEjslJh0Va9c7C0ueFNdbYzKKPbboJk2SR?=
 =?us-ascii?Q?J7ZYLVf7emHuGfwyV1pYOGVxZlB5MDxMFglLjklsL4LIYQEndAwHPtT/EsAe?=
 =?us-ascii?Q?2aLslQEYF+7d20kdi7HjyDYYCG1RquwNyuu1v9Bh?=
X-OriginatorOrg: phytec.com
X-MS-Exchange-CrossTenant-Network-Message-Id: de1431a7-3ec0-465f-e9bd-08dc6886c5ab
X-MS-Exchange-CrossTenant-AuthSource: SA1PR22MB5636.namprd22.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 29 Apr 2024 19:58:43.2987
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 67bcab1a-5db0-4ee8-86f4-1533d0b4b5c7
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: X+GtzQvgYGS5xpJ4bpQcBCT7y0y8I7yL8gmmhvRJRP7DpNTlk1+kDSCEQPoYBPOBY9DqJXoMcuTH86zFo4I4hg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH7PR22MB3962

Convert the lp873x binding to DT schema format. The gpio-controller
and #gpio-cells properties were removed from required because using
the device as a GPIO controller is optional.

Signed-off-by: Nathan Morrisson <nmorrisson@phytec.com>
---
v2:
  - Added "mfd:" to subject prefix
  - Added vendor TI to file name
  - Updated title to match hardware
  - Removed | where we do not need to preserve formatting
  - Added *-in-supply back to required properties, they should not have been removed
  - Added explanation for why gpio properties were removed from required
  - Updated the example to be more generic and got rid of unnecessary i2c node

 .../devicetree/bindings/mfd/lp873x.txt        |  67 -----------
 .../devicetree/bindings/mfd/ti,lp8732.yaml    | 112 ++++++++++++++++++
 2 files changed, 112 insertions(+), 67 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/mfd/lp873x.txt
 create mode 100644 Documentation/devicetree/bindings/mfd/ti,lp8732.yaml

diff --git a/Documentation/devicetree/bindings/mfd/lp873x.txt b/Documentation/devicetree/bindings/mfd/lp873x.txt
deleted file mode 100644
index ae9cf39bd101..000000000000
--- a/Documentation/devicetree/bindings/mfd/lp873x.txt
+++ /dev/null
@@ -1,67 +0,0 @@
-TI LP873X PMIC MFD driver
-
-Required properties:
-  - compatible:	"ti,lp8732", "ti,lp8733"
-  - reg:		I2C slave address.
-  - gpio-controller:	Marks the device node as a GPIO Controller.
-  - #gpio-cells:	Should be two.  The first cell is the pin number and
-			the second cell is used to specify flags.
-			See ../gpio/gpio.txt for more information.
-  - xxx-in-supply:	Phandle to parent supply node of each regulator
-			populated under regulators node. xxx can be
-			buck0, buck1, ldo0 or ldo1.
-  - regulators:	List of child nodes that specify the regulator
-			initialization data.
-Example:
-
-pmic: lp8733@60 {
-	compatible = "ti,lp8733";
-	reg = <0x60>;
-	gpio-controller;
-	#gpio-cells = <2>;
-
-	buck0-in-supply = <&vsys_3v3>;
-	buck1-in-supply = <&vsys_3v3>;
-	ldo0-in-supply = <&vsys_3v3>;
-	ldo1-in-supply = <&vsys_3v3>;
-
-	regulators {
-		lp8733_buck0: buck0 {
-			regulator-name = "lp8733-buck0";
-			regulator-min-microvolt = <800000>;
-			regulator-max-microvolt = <1400000>;
-			regulator-min-microamp = <1500000>;
-			regulator-max-microamp = <4000000>;
-			regulator-ramp-delay = <10000>;
-			regulator-always-on;
-			regulator-boot-on;
-		};
-
-		lp8733_buck1: buck1 {
-			regulator-name = "lp8733-buck1";
-			regulator-min-microvolt = <800000>;
-			regulator-max-microvolt = <1400000>;
-			regulator-min-microamp = <1500000>;
-			regulator-max-microamp = <4000000>;
-			regulator-ramp-delay = <10000>;
-			regulator-boot-on;
-			regulator-always-on;
-		};
-
-		lp8733_ldo0: ldo0 {
-			regulator-name = "lp8733-ldo0";
-			regulator-min-microvolt = <800000>;
-			regulator-max-microvolt = <3000000>;
-			regulator-boot-on;
-			regulator-always-on;
-		};
-
-		lp8733_ldo1: ldo1 {
-			regulator-name = "lp8733-ldo1";
-			regulator-min-microvolt = <800000>;
-			regulator-max-microvolt = <3000000>;
-			regulator-always-on;
-			regulator-boot-on;
-		};
-	};
-};
diff --git a/Documentation/devicetree/bindings/mfd/ti,lp8732.yaml b/Documentation/devicetree/bindings/mfd/ti,lp8732.yaml
new file mode 100644
index 000000000000..9a90cee2b545
--- /dev/null
+++ b/Documentation/devicetree/bindings/mfd/ti,lp8732.yaml
@@ -0,0 +1,112 @@
+# SPDX-License-Identifier: GPL-2.0 OR BSD-2-Clause
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/mfd/ti,lp8732.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: TI LP873X Power Management Integrated Circuit
+
+maintainers:
+  - J Keerthy <j-keerthy@ti.com>
+
+description:
+  PMIC with two high-current buck converters and two linear regulators.
+
+properties:
+  compatible:
+    enum:
+      - ti,lp8732
+      - ti,lp8733
+
+  reg:
+    maxItems: 1
+
+  gpio-controller: true
+
+  '#gpio-cells':
+    const: 2
+
+  regulators:
+    description:
+      List of child nodes that specify the regulator initialization data.
+    type: object
+    patternProperties:
+      "^buck[01]|ldo[01]$":
+        type: object
+        $ref: /schemas/regulator/regulator.yaml#
+        unevaluatedProperties: false
+    additionalProperties: false
+
+patternProperties:
+  '^(buck[01]|ldo[01])-in-supply$':
+    description: Phandle to parent supply of each regulator populated under regulators node.
+
+required:
+  - compatible
+  - reg
+  - regulators
+  - buck0-in-supply
+  - buck1-in-supply
+  - ldo0-in-supply
+  - ldo1-in-supply
+
+additionalProperties: false
+
+examples:
+  - |
+    i2c {
+        #address-cells = <1>;
+        #size-cells = <0>;
+
+        pmic: pmic@60 {
+            compatible = "ti,lp8733";
+            reg = <0x60>;
+            gpio-controller;
+            #gpio-cells = <2>;
+
+            buck0-in-supply = <&vsys_3v3>;
+            buck1-in-supply = <&vsys_3v3>;
+            ldo0-in-supply = <&vsys_3v3>;
+            ldo1-in-supply = <&vsys_3v3>;
+
+            regulators {
+                buck0: buck0 {
+                    regulator-name = "buck0";
+                    regulator-min-microvolt = <800000>;
+                    regulator-max-microvolt = <1400000>;
+                    regulator-min-microamp = <1500000>;
+                    regulator-max-microamp = <4000000>;
+                    regulator-ramp-delay = <10000>;
+                    regulator-always-on;
+                    regulator-boot-on;
+                };
+
+                buck1: buck1 {
+                    regulator-name = "buck1";
+                    regulator-min-microvolt = <800000>;
+                    regulator-max-microvolt = <1400000>;
+                    regulator-min-microamp = <1500000>;
+                    regulator-max-microamp = <4000000>;
+                    regulator-ramp-delay = <10000>;
+                    regulator-boot-on;
+                    regulator-always-on;
+                };
+
+                ldo0: ldo0 {
+                    regulator-name = "ldo0";
+                    regulator-min-microvolt = <800000>;
+                    regulator-max-microvolt = <3000000>;
+                    regulator-boot-on;
+                    regulator-always-on;
+                };
+
+                ldo1: ldo1 {
+                    regulator-name = "ldo1";
+                    regulator-min-microvolt = <800000>;
+                    regulator-max-microvolt = <3000000>;
+                    regulator-always-on;
+                    regulator-boot-on;
+                };
+            };
+        };
+    };
-- 
2.25.1