linux-kernel - [PATCH v2 1/3] dt-bindings: pfuze100: add optional disable switch-regulators binding

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <20180713125002.24331-2-m.felsch@pengutronix.de>
Date:   Fri, 13 Jul 2018 14:50:01 +0200
From:   Marco Felsch <m.felsch@...gutronix.de>
To:     lgirdwood@...il.com, broonie@...nel.org, robh+dt@...nel.org,
        mark.rutland@....com
Cc:     fabio.estevam@....com, Anson.Huang@....com, kernel@...gutronix.de,
        devicetree@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: [PATCH v2 1/3] dt-bindings: pfuze100: add optional disable switch-regulators binding

This binding is used to keep the backward compatibility with the current
dtb's [1]. The binding informs the driver that the unused switch regulators
can be disabled.
If it is not specified, the driver doesn't disable the switch regulators.

[1] https://patchwork.kernel.org/patch/10490381/

Signed-off-by: Marco Felsch <m.felsch@...gutronix.de>

---
Changes in V2:
  - add more information about the binding
  - rename binding and add vendor prefix

 .../devicetree/bindings/regulator/pfuze100.txt        | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/Documentation/devicetree/bindings/regulator/pfuze100.txt b/Documentation/devicetree/bindings/regulator/pfuze100.txt
index 672c939045ff..2c46b8d368db 100644
--- a/Documentation/devicetree/bindings/regulator/pfuze100.txt
+++ b/Documentation/devicetree/bindings/regulator/pfuze100.txt
@@ -4,6 +4,17 @@ Required properties:
 - compatible: "fsl,pfuze100", "fsl,pfuze200", "fsl,pfuze3000", "fsl,pfuze3001"
 - reg: I2C slave address
 
+Optional properties:
+- fsl,pfuze-support-disable: Boolean, if present disable all unused switch
+  regulators to save power consumption. Attention, till 4.18 these regulators
+  were always on without specifying "regulator-always-on". So be sure to mark
+  import regulators as "regulator-always-on" (e.g. DDR ref, DDR supply). If
+  not present, the switched regualtors are always on and can't be disabled.
+  This binding is a workaround to keep backward compatibility with old dtb's
+  which rely on the fact that the switched regulators are always on and don't
+  mark them explicit as "regulator-always-on". On new dtbs this property should
+  always be present.
+
 Required child node:
 - regulators: This is the list of child nodes that specify the regulator
   initialization data for defined regulators. Please refer to below doc
-- 
2.18.0