From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail-oi0-f68.google.com ([209.85.218.68]:34316 "EHLO mail-oi0-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752570AbdARWCK (ORCPT ); Wed, 18 Jan 2017 17:02:10 -0500 Date: Wed, 18 Jan 2017 16:02:07 -0600 From: Rob Herring To: Matt Ranostay Cc: linux-wireless@vger.kernel.org, linux-kernel@vger.kernel.org, linux-mmc@vger.kernel.org, devicetree@vger.kernel.org, tony@atomide.com Subject: Re: [PATCH v3 1/2] devicetree: document new marvell-8xxx and pwrseq-sd8787 options Message-ID: <20170118220207.3zhbavk2uo4xui34@rob-hp-laptop> (sfid-20170118_230233_375082_4D0A2DFF) References: <20170113052937.12538-1-matt@ranostay.consulting> <20170113052937.12538-2-matt@ranostay.consulting> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <20170113052937.12538-2-matt@ranostay.consulting> Sender: linux-wireless-owner@vger.kernel.org List-ID: On Thu, Jan 12, 2017 at 09:29:36PM -0800, Matt Ranostay wrote: Changelog? > Cc: devicetree@vger.kernel.org > Signed-off-by: Matt Ranostay > --- > .../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..1b658351629b > --- /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". > +- pwndn-gpio: contains a power down GPIO specifier. powerdown-gpios > +- reset-gpio: contains a reset GPIO specifier. reset-gpios And state the active state for both. > + > +Example: > + > + wifi_pwrseq: wifi_pwrseq { > + compatible = "mmc-pwrseq-sd8787"; > + pwrdn-gpio = <&twl_gpio 0 GPIO_ACTIVE_LOW>; > + reset-gpio = <&twl_gpio 1 GPIO_ACTIVE_LOW>; > + } > diff --git a/Documentation/devicetree/bindings/net/wireless/marvell-8xxx.txt b/Documentation/devicetree/bindings/net/wireless/marvell-8xxx.txt > index 980b16df74c3..0854451ff91d 100644 > --- a/Documentation/devicetree/bindings/net/wireless/marvell-8xxx.txt > +++ b/Documentation/devicetree/bindings/net/wireless/marvell-8xxx.txt > @@ -1,4 +1,4 @@ > -Marvell 8897/8997 (sd8897/sd8997/pcie8997) SDIO/PCIE devices > +Marvell 8787/8897/8997 (sd8787/sd8897/sd8997/pcie8997) SDIO/PCIE devices > ------ > > This node provides properties for controlling the Marvell SDIO/PCIE wireless device. > @@ -8,6 +8,7 @@ connects the device to the system. > Required properties: > > - compatible : should be one of the following: > + * "marvell,sd8787" > * "marvell,sd8897" > * "marvell,sd8997" > * "pci11ab,2b42" > @@ -34,6 +35,9 @@ Optional properties: > so that the wifi chip can wakeup host platform under certain condition. > during system resume, the irq will be disabled to make sure > unnecessary interrupt is not received. > + - vmmc-supply: a phandle of a regulator, supplying VCC to the card > + - mmc-pwrseq: phandle to the MMC power sequence node. See "mmc-pwrseq-*" > + for documentation of MMC power sequence bindings. > > Example: > > @@ -46,6 +50,7 @@ so that firmware can wakeup host using this device side pin. > &mmc3 { > status = "okay"; > vmmc-supply = <&wlan_en_reg>; > + mmc-pwrseq = <&wifi_pwrseq>; > bus-width = <4>; > cap-power-off-card; > keep-power-in-suspend; > -- > 2.10.2 > > -- > To unsubscribe from this list: send the line "unsubscribe devicetree" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html