On Mon, Jan 23, 2017 at 9:04 AM, Rob Herring [off-list ref] wrote:

On Fri, Jan 20, 2017 at 09:50:38PM -0800, Matt Ranostay wrote:

quoted

Cc: devicetree@vger.kernel.org
Signed-off-by: Matt Ranostay <redacted>
---
 .../devicetree/bindings/mmc/mmc-pwrseq-sd8787.txt          | 14 ++++++++++++++
 .../devicetree/bindings/net/wireless/marvell-8xxx.txt      |  7 ++++++-
 2 files changed, 20 insertions(+), 1 deletion(-)
 create mode 100644 Documentation/devicetree/bindings/mmc/mmc-pwrseq-sd8787.txt

diff --git a/Documentation/devicetree/bindings/mmc/mmc-pwrseq-sd8787.txt b/Documentation/devicetree/bindings/mmc/mmc-pwrseq-sd8787.txt
new file mode 100644
index 000000000000..4bb56766982e
--- /dev/null
+++ b/Documentation/devicetree/bindings/mmc/mmc-pwrseq-sd8787.txt

@@ -0,0 +1,14 @@
+* Marvell SD8787 power sequence provider
+
+Required properties:
+- compatible: must be "mmc-pwrseq-sd8787".
+- powerdown-gpio: contains a power down GPIO specifier with active state
+- reset-gpio: contains a reset GPIO specifier with active state

My comments on the previous v3 still apply.

Ah yikes.. forgot to increment the patchset.  And noticed the gpio
isn't gpios... will fixed in "v4"

Thanks,

Matt

quoted

+
+Example:
+
+     wifi_pwrseq: wifi_pwrseq {
+             compatible = "mmc-pwrseq-sd8787";
+             powerdown-gpio = <&twl_gpio 0 GPIO_ACTIVE_LOW>;
+             reset-gpio = <&twl_gpio 1 GPIO_ACTIVE_LOW>;
+     }