devicetree.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: Lee Jones <lee.jones@linaro.org>,
	Linus Walleij <linus.walleij@linaro.org>,
	linux-pm@vger.kernel.org, devicetree@vger.kernel.org,
	linux-kernel@vger.kernel.org,
	Sebastian Reichel <sebastian.reichel@collabora.com>,
	kernel@collabora.com
Subject: [PATCH 0/6] Convert remaining power-supply DT bindings to YAML
Date: Tue, 13 Apr 2021 01:03:14 +0200	[thread overview]
Message-ID: <20210412230320.382885-1-sebastian.reichel@collabora.com> (raw)

Hi,

Let's get rid of the remaining three non-YAML power-supply binding files.

1. charger-manager
   I initially skipped this, since the binding is quite complex.
   It turned out a basic yaml binding is not that complex.
   This does not check for supplies matching cm-regulator-name,
   but that should be fine considering this is a legacy binding.

2. max8925_battery.txt
   I initially skipped this, since it does not have a compatible
   node. This converts the MFD binding file
   compatible value. It needs to be described in the PMIC binding
   file, which has not yet been converted.

3. stericsson,ab8500-battery.txt
   This also describes a node below the PMIC, which does not have
   its own compatible value. The same applies.

I tested the series using dt-validate 2021.3.dev6+g38efe3f and nothing
is reported when running any of the following

for file in Documentation/devicetree/bindings/mfd/stericsson,ab8500.yaml \
            Documentation/devicetree/bindings/mfd/maxim,max8925.yaml \
	    Documentation/devicetree/bindings/power/supply/charger-manager.yaml ;
do
	make -s -j4 dtbs_check DT_SCHEMA_FILES=$file
	make -s -j4 dt_binding_check DT_SCHEMA_FILES=$file
done

Merging notes:
 * The ARM DTS changes can be merged independently. The DTS changes should be
   compatible with existing drivers and existing binding documenting.
 * The last patch depends on eb917497734c ("dt-bindings: power: supply: ab8500:
   Convert to DT schema format") queued into power-supply's for-next branch for
   v5.13. The change is not critical, so it can wait for 5.14.
    
Thanks for reviewing it,

-- Sebastian

Sebastian Reichel (6):
  ARM: dts: mmp2-brownstone: Fix max8925 vendor prefix
  ARM: dts: ux500: Fix interrupt cells
  ARM: dts: ux500: Rename gpio-controller node
  dt-bindings: power: supply: charger-manager: Convert to DT schema
    format
  dt-bindings: power: mfd: max8925: Convert to DT schema format
  dt-bindings: power: mfd: ab8500: Convert to DT schema format

 .../leds/backlight/max8925-backlight.txt      |  10 -
 .../devicetree/bindings/mfd/ab8500.txt        | 280 -------
 .../devicetree/bindings/mfd/max8925.txt       |  64 --
 .../bindings/mfd/maxim,max8925.yaml           | 139 ++++
 .../bindings/mfd/stericsson,ab8500.yaml       | 696 ++++++++++++++++++
 .../bindings/pinctrl/ste,abx500.txt           | 318 --------
 .../bindings/power/supply/charger-manager.txt |  91 ---
 .../power/supply/charger-manager.yaml         | 215 ++++++
 .../bindings/power/supply/max8925_battery.txt |  18 -
 .../supply/stericsson,ab8500-battery.txt      |  34 -
 arch/arm/boot/dts/mmp2-brownstone.dts         |   2 +-
 arch/arm/boot/dts/ste-ab8500.dtsi             |  28 +-
 arch/arm/boot/dts/ste-ab8505.dtsi             |  24 +-
 arch/arm/boot/dts/ste-href-ab8500.dtsi        |   2 +-
 arch/arm/boot/dts/ste-href.dtsi               |   2 +-
 arch/arm/boot/dts/ste-snowball.dts            |   2 +-
 16 files changed, 1080 insertions(+), 845 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/leds/backlight/max8925-backlight.txt
 delete mode 100644 Documentation/devicetree/bindings/mfd/ab8500.txt
 delete mode 100644 Documentation/devicetree/bindings/mfd/max8925.txt
 create mode 100644 Documentation/devicetree/bindings/mfd/maxim,max8925.yaml
 create mode 100644 Documentation/devicetree/bindings/mfd/stericsson,ab8500.yaml
 delete mode 100644 Documentation/devicetree/bindings/pinctrl/ste,abx500.txt
 delete mode 100644 Documentation/devicetree/bindings/power/supply/charger-manager.txt
 create mode 100644 Documentation/devicetree/bindings/power/supply/charger-manager.yaml
 delete mode 100644 Documentation/devicetree/bindings/power/supply/max8925_battery.txt
 delete mode 100644 Documentation/devicetree/bindings/power/supply/stericsson,ab8500-battery.txt

-- 
2.30.2


             reply	other threads:[~2021-04-12 23:03 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-04-12 23:03 Sebastian Reichel [this message]
2021-04-12 23:03 ` [PATCH 1/6] ARM: dts: mmp2-brownstone: Fix max8925 vendor prefix Sebastian Reichel
2021-04-12 23:03 ` [PATCH 2/6] ARM: dts: ux500: Fix interrupt cells Sebastian Reichel
2021-05-18  0:11   ` Linus Walleij
2021-04-12 23:03 ` [PATCH 3/6] ARM: dts: ux500: Rename gpio-controller node Sebastian Reichel
2021-05-18  0:15   ` Linus Walleij
2021-04-12 23:03 ` [PATCH 4/6] dt-bindings: power: supply: charger-manager: Convert to DT schema format Sebastian Reichel
2021-04-13 15:26   ` Rob Herring
2021-05-13 15:51     ` Sebastian Reichel
2021-04-12 23:03 ` [PATCH 5/6] dt-bindings: power: mfd: max8925: " Sebastian Reichel
2021-04-13 15:34   ` Rob Herring
2021-04-12 23:03 ` [PATCH 6/6] dt-bindings: power: mfd: ab8500: " Sebastian Reichel
2021-04-13 15:57   ` 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=20210412230320.382885-1-sebastian.reichel@collabora.com \
    --to=sebastian.reichel@collabora.com \
    --cc=devicetree@vger.kernel.org \
    --cc=kernel@collabora.com \
    --cc=lee.jones@linaro.org \
    --cc=linus.walleij@linaro.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pm@vger.kernel.org \
    --cc=robh+dt@kernel.org \
    --cc=sre@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;
as well as URLs for NNTP newsgroup(s).