linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Sebastian Reichel <sebastian.reichel@collabora.com>
To: Sebastian Reichel <sre@kernel.org>, Rob Herring <robh+dt@kernel.org>
Cc: linux-pm@vger.kernel.org, devicetree@vger.kernel.org,
	linux-kernel@vger.kernel.org,
	Sebastian Reichel <sebastian.reichel@collabora.com>,
	kernel@collabora.com, Tony Lindgren <tony@atomide.com>
Subject: [PATCHv2 03/38] dt-bindings: power: supply: cpcap-charger: Convert to DT schema format
Date: Wed, 17 Mar 2021 14:48:29 +0100	[thread overview]
Message-ID: <20210317134904.80737-4-sebastian.reichel@collabora.com> (raw)
In-Reply-To: <20210317134904.80737-1-sebastian.reichel@collabora.com>

Convert the binding to DT schema format. I also added the missing bits
used by the only in-tree user and implemented in the driver.

Cc: Tony Lindgren <tony@atomide.com>
Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.com>
---
 .../bindings/power/supply/cpcap-charger.txt   |  40 -------
 .../bindings/power/supply/cpcap-charger.yaml  | 106 ++++++++++++++++++
 2 files changed, 106 insertions(+), 40 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/power/supply/cpcap-charger.txt
 create mode 100644 Documentation/devicetree/bindings/power/supply/cpcap-charger.yaml

diff --git a/Documentation/devicetree/bindings/power/supply/cpcap-charger.txt b/Documentation/devicetree/bindings/power/supply/cpcap-charger.txt
deleted file mode 100644
index 6048f636783f..000000000000
--- a/Documentation/devicetree/bindings/power/supply/cpcap-charger.txt
+++ /dev/null
@@ -1,40 +0,0 @@
-Motorola CPCAP PMIC battery charger binding
-
-Required properties:
-- compatible: Shall be "motorola,mapphone-cpcap-charger"
-- interrupts: Interrupt specifier for each name in interrupt-names
-- interrupt-names: Should contain the following entries:
-		   "chrg_det", "rvrs_chrg", "chrg_se1b", "se0conn",
-		   "rvrs_mode", "chrgcurr2", "chrgcurr1", "vbusvld",
-		   "battdetb"
-- io-channels: IIO ADC channel specifier for each name in io-channel-names
-- io-channel-names: Should contain the following entries:
-		    "battdetb", "battp", "vbus", "chg_isense", "batti"
-
-Optional properties:
-- mode-gpios: Optionally CPCAP charger can have a companion wireless
-	      charge controller that is controlled with two GPIOs
-	      that are active low.
-
-Example:
-
-cpcap_charger: charger {
-	compatible = "motorola,mapphone-cpcap-charger";
-	interrupts-extended = <
-		&cpcap 13 0 &cpcap 12 0 &cpcap 29 0 &cpcap 28 0
-		&cpcap 22 0 &cpcap 21 0 &cpcap 20 0 &cpcap 19 0
-		&cpcap 54 0
-	>;
-	interrupt-names =
-		"chrg_det", "rvrs_chrg", "chrg_se1b", "se0conn",
-		"rvrs_mode", "chrgcurr2", "chrgcurr1", "vbusvld",
-		"battdetb";
-	mode-gpios = <&gpio3 29 GPIO_ACTIVE_LOW
-		      &gpio3 23 GPIO_ACTIVE_LOW>;
-	io-channels = <&cpcap_adc 0 &cpcap_adc 1
-		       &cpcap_adc 2 &cpcap_adc 5
-		       &cpcap_adc 6>;
-	io-channel-names = "battdetb", "battp",
-			   "vbus", "chg_isense",
-			   "batti";
-};
diff --git a/Documentation/devicetree/bindings/power/supply/cpcap-charger.yaml b/Documentation/devicetree/bindings/power/supply/cpcap-charger.yaml
new file mode 100644
index 000000000000..cb6353683d7b
--- /dev/null
+++ b/Documentation/devicetree/bindings/power/supply/cpcap-charger.yaml
@@ -0,0 +1,106 @@
+# SPDX-License-Identifier: GPL-2.0
+# Copyright (C) 2021 Sebastian Reichel
+%YAML 1.2
+---
+$id: "http://devicetree.org/schemas/power/supply/cpcap-charger.yaml#"
+$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+
+title: Motorola CPCAP PMIC charger
+
+maintainers:
+  - Tony Lindgren <tony@atomide.com>
+  - Sebastian Reichel <sre@kernel.org>
+
+description: |
+  Motorola CPCAP is a PMIC found in some mobile phones, e.g.
+  the Droid 4. This binding describes its battery charger
+  sub-function.
+
+allOf:
+  - $ref: power-supply.yaml#
+
+properties:
+  compatible:
+    const: motorola,mapphone-cpcap-charger
+
+  interrupts:
+    items:
+      - description: charger detection interrupt
+      - description: reverse charge interrupt
+      - description: SE1 charger detection interrupt
+      - description: SE0 charger detection interrupt
+      - description: reverse mode interrupt
+      - description: charge current 2 interrupt
+      - description: charge current 1 interrupt
+      - description: VBUS valid interrupt
+      - description: battery detect interrupt
+
+  interrupt-names:
+    items:
+      - const: chrg_det
+      - const: rvrs_chrg
+      - const: chrg_se1b
+      - const: se0conn
+      - const: rvrs_mode
+      - const: chrgcurr2
+      - const: chrgcurr1
+      - const: vbusvld
+      - const: battdetb
+
+  io-channels:
+    items:
+      - description: battery temperature
+      - description: battery voltage
+      - description: VBUS voltage
+      - description: battery charge current
+      - description: battery current
+
+  io-channel-names:
+    items:
+      - const: battdetb
+      - const: battp
+      - const: vbus
+      - const: chg_isense
+      - const: batti
+
+  mode-gpios:
+    description: |
+      Optionally CPCAP charger can have a companion wireless
+      charge controller that is controlled with two GPIOs
+      that are active low.
+    minItems: 2
+    maxItems: 2
+
+required:
+  - compatible
+  - interrupts
+  - interrupt-names
+  - io-channels
+  - io-channel-names
+
+additionalProperties: false
+
+examples:
+  - |
+    #include <dt-bindings/gpio/gpio.h>
+    cpcap {
+      charger {
+        compatible = "motorola,mapphone-cpcap-charger";
+        interrupts-extended =
+                <&cpcap 13 0>, <&cpcap 12 0>, <&cpcap 29 0>, <&cpcap 28 0>,
+                <&cpcap 22 0>, <&cpcap 21 0>, <&cpcap 20 0>, <&cpcap 19 0>,
+                <&cpcap 54 0>;
+        interrupt-names =
+                "chrg_det", "rvrs_chrg", "chrg_se1b", "se0conn",
+                "rvrs_mode", "chrgcurr2", "chrgcurr1", "vbusvld",
+                "battdetb";
+        mode-gpios = <&gpio3 29 GPIO_ACTIVE_LOW>,
+                     <&gpio3 23 GPIO_ACTIVE_LOW>;
+        io-channels = <&cpcap_adc 0>, <&cpcap_adc 1>,
+                      <&cpcap_adc 2>, <&cpcap_adc 5>,
+                      <&cpcap_adc 6>;
+        io-channel-names = "battdetb", "battp",
+                           "vbus", "chg_isense",
+                           "batti";
+      };
+    };
-- 
2.30.2


  parent reply	other threads:[~2021-03-17 13:50 UTC|newest]

Thread overview: 51+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-03-17 13:48 [PATCHv2 00/38] Convert power-supply DT bindings to YAML Sebastian Reichel
2021-03-17 13:48 ` [PATCHv2 01/38] ARM: dts: motorola-cpcap-mapphone: Prepare for dtbs_check parsing Sebastian Reichel
2021-03-17 14:29   ` Tony Lindgren
2021-03-23 12:51     ` Sebastian Reichel
2021-03-23 12:56       ` Tony Lindgren
2021-03-17 13:48 ` [PATCHv2 02/38] dt-bindings: power: supply: cpcap-battery: Convert to DT schema format Sebastian Reichel
2021-03-19 10:18   ` Tony Lindgren
2021-03-17 13:48 ` Sebastian Reichel [this message]
2021-03-19 10:18   ` [PATCHv2 03/38] dt-bindings: power: supply: cpcap-charger: " Tony Lindgren
2021-03-17 13:48 ` [PATCHv2 04/38] dt-bindings: power: supply: bq25890: " Sebastian Reichel
2021-03-17 13:48 ` [PATCHv2 05/38] dt-bindings: power: supply: bq24257: " Sebastian Reichel
2021-03-17 13:48 ` [PATCHv2 06/38] dt-bindings: power: supply: bq24190: " Sebastian Reichel
2021-03-17 13:48 ` [PATCHv2 07/38] dt-bindings: power: supply: bq2415x: " Sebastian Reichel
2021-03-17 13:48 ` [PATCHv2 08/38] dt-bindings: power: supply: bq24735: " Sebastian Reichel
2021-03-17 13:48 ` [PATCHv2 09/38] dt-bindings: power: supply: isp1704: " Sebastian Reichel
2021-03-17 13:48 ` [PATCHv2 10/38] dt-bindings: power: supply: sbs-charger: " Sebastian Reichel
2021-03-17 13:48 ` [PATCHv2 11/38] dt-bindings: power: supply: sbs-manager: " Sebastian Reichel
2021-03-17 13:48 ` [PATCHv2 12/38] dt-bindings: power: supply: ds2760: " Sebastian Reichel
2021-03-17 13:48 ` [PATCHv2 13/38] dt-bindings: power: supply: sc27xx-fg: " Sebastian Reichel
2021-03-17 13:48 ` [PATCHv2 14/38] dt-bindings: power: supply: sc2731-charger: " Sebastian Reichel
2021-03-17 13:48 ` [PATCHv2 15/38] dt-bindings: power: supply: tps65090: " Sebastian Reichel
2021-03-17 13:48 ` [PATCHv2 16/38] dt-bindings: power: supply: tps65217: " Sebastian Reichel
2021-03-17 13:48 ` [PATCHv2 17/38] dt-bindings: power: supply: twl4030: " Sebastian Reichel
2021-03-17 13:48 ` [PATCHv2 18/38] dt-bindings: power: supply: n900-battery: " Sebastian Reichel
2021-03-17 17:01   ` Pali Rohár
2021-03-17 13:48 ` [PATCHv2 19/38] dt-bindings: power: supply: lego-ev3-battery: " Sebastian Reichel
2021-03-23 16:57   ` David Lechner
2021-03-17 13:48 ` [PATCHv2 20/38] dt-bindings: power: supply: max17040: " Sebastian Reichel
2021-03-17 13:48 ` [PATCHv2 21/38] dt-bindings: power: supply: max14656: " Sebastian Reichel
2021-03-17 13:48 ` [PATCHv2 22/38] dt-bindings: power: supply: max17042: " Sebastian Reichel
2021-03-17 13:48 ` [PATCHv2 23/38] dt-bindings: power: supply: max8903: " Sebastian Reichel
2021-03-17 13:48 ` [PATCHv2 24/38] dt-bindings: power: supply: ucs1002: " Sebastian Reichel
2021-03-17 13:48 ` [PATCHv2 25/38] dt-bindings: power: supply: pm8941-charger: " Sebastian Reichel
2021-03-17 13:48 ` [PATCHv2 26/38] dt-bindings: power: supply: pm8941-coincell: " Sebastian Reichel
2021-03-17 13:48 ` [PATCHv2 27/38] dt-bindings: power: supply: act8945a: " Sebastian Reichel
2021-03-17 13:48 ` [PATCHv2 28/38] dt-bindings: power: supply: axp20x: " Sebastian Reichel
2021-03-17 13:48 ` [PATCHv2 29/38] dt-bindings: power: supply: da9150: " Sebastian Reichel
2021-03-17 13:48 ` [PATCHv2 30/38] dt-bindings: power: supply: lp8727: " Sebastian Reichel
2021-03-17 13:48 ` [PATCHv2 31/38] dt-bindings: power: supply: lt3651: " Sebastian Reichel
2021-03-17 13:48 ` [PATCHv2 32/38] dt-bindings: power: supply: ltc294x: " Sebastian Reichel
2021-03-17 13:48 ` [PATCHv2 33/38] dt-bindings: power: supply: rt9455: " Sebastian Reichel
2021-03-17 13:49 ` [PATCHv2 34/38] dt-bindings: power: supply: olpc-battery: " Sebastian Reichel
2021-03-17 13:49 ` [PATCHv2 35/38] dt-bindings: power: supply: ab8500: " Sebastian Reichel
2021-04-08 23:28   ` Linus Walleij
2021-04-09 13:43     ` Sebastian Reichel
2021-03-17 13:49 ` [PATCHv2 36/38] dt-bindings: power: supply: sbs-battery: Fix dtbs_check Sebastian Reichel
2021-03-17 13:49 ` [PATCHv2 37/38] dt-bindings: power: supply: Fix remaining battery.txt links Sebastian Reichel
2021-03-17 13:49 ` [PATCHv2 38/38] dt-bindings: power: supply: Drop power_supply.txt Sebastian Reichel
2021-03-22 16:25 ` [PATCHv2 00/38] Convert power-supply DT bindings to YAML Rob Herring
2021-03-22 21:42   ` Sebastian Reichel
2021-03-23  1:05     ` Rob Herring

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=20210317134904.80737-4-sebastian.reichel@collabora.com \
    --to=sebastian.reichel@collabora.com \
    --cc=devicetree@vger.kernel.org \
    --cc=kernel@collabora.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pm@vger.kernel.org \
    --cc=robh+dt@kernel.org \
    --cc=sre@kernel.org \
    --cc=tony@atomide.com \
    /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;
as well as URLs for NNTP newsgroup(s).