On 17 September 2016 at 11:01, Heiko Stuebner [off-list ref] wrote:

Hi,

Am Samstag, 17. September 2016, 10:32:49 CEST schrieb Ziyuan Xu:

quoted

Add power domain as a optional property for sdhci-of-arasan, which can
be turned off in the so-called unused condition, such as suspend and
remove. Aim to lower power requirements.

Signed-off-by: Ziyuan Xu <redacted>

Acked-by: Rob Herring <robh@kernel.org>

Reviewed-by: Heiko Stuebner <heiko@sntech.de>

@Ulf: I guess you'll take this binding change through the mmc tree and I'll
pick up the devicetree change.

Ideally the documentation and DT change should go together, so I think
it's better for you to pick up both.

I intend to ack it, although it would be nice with some more
information as I requested, in the change-log first.

Kind regards
Uffe

Heiko

quoted

---

Changes in v3:
- fix a typo
- add Rob's ack-tag

Changes in v2:
- fix a typo
- add an example code for power-domain

 Documentation/devicetree/bindings/mmc/arasan,sdhci.txt | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/mmc/arasan,sdhci.txt

b/Documentation/devicetree/bindings/mmc/arasan,sdhci.txt index
3404afa..eb84d29 100644

--- a/Documentation/devicetree/bindings/mmc/arasan,sdhci.txt
+++ b/Documentation/devicetree/bindings/mmc/arasan,sdhci.txt

@@ -1,12 +1,14 @@
 Device Tree Bindings for the Arasan SDHCI Controller

-  The bindings follow the mmc[1], clock[2], interrupt[3] and phy[4]

bindings. +  The bindings follow the mmc[1], clock[2], interrupt[3], phy[4]
and power +  domain[5] bindings.
   Only deviations are documented here.

   [1] Documentation/devicetree/bindings/mmc/mmc.txt
   [2] Documentation/devicetree/bindings/clock/clock-bindings.txt
   [3] Documentation/devicetree/bindings/interrupt-controller/interrupts.txt
[4] Documentation/devicetree/bindings/phy/phy-bindings.txt
+  [5] Documentation/devicetree/bindings/power/power_domain.txt

 Required Properties:
   - compatible: Compatibility string.  One of:

@@ -36,6 +38,8 @@ Optional Properties:
   - #clock-cells: If specified this should be the value <0>.  With this

property in place we will export a clock representing the Card Clock.  This
clock is expected to be consumed by our PHY.  You must also specify +  -
power-domains: A phandle and PM domain as specifier defined by bindings +
 of the power controller specified by phandle.

 Example:
      sdhci@e0100000 {

@@ -71,5 +75,6 @@ Example:
              phys = <&emmc_phy>;
              phy-names = "phy_arasan";
              #clock-cells = <0>;
+             power-domains = <&power RK3399_PD_EMMC>;
              status = "disabled";
      };