Re: [PATCH v4 3/3] mwifiex: parse host wakeup configuration from device
From: Rob Herring <robh@kernel.org>
Date: 2016-02-18 14:38:03
Also in:
linux-bluetooth, lkml
On Tue, Feb 16, 2016 at 01:14:39AM -0800, Amitkumar Karwar wrote:
quoted hunk ↗ jump to hunk
From: Xinming Hu <redacted> This patch implement a framework for board specific wakeup. driver parse irq/gpio number from device tree, corresponding resources will be allocated, and used for host suspend/resume. driver private binding file is also updated in the patch. Signed-off-by: Xinming Hu <redacted> Signed-off-by: Amitkumar Karwar <redacted> --- v4: Replaced #ifdef with __maybe_unused(Arnd Bergmann) --- .../bindings/net/wireless/marvell-sd8xxx.txt | 10 +++ .../net/wireless/marvell/mwifiex/platform_drv.c | 91 ++++++++++++++++++++++ 2 files changed, 101 insertions(+)diff --git a/Documentation/devicetree/bindings/net/wireless/marvell-sd8xxx.txt b/Documentation/devicetree/bindings/net/wireless/marvell-sd8xxx.txt index c337fab..50138fd 100644 --- a/Documentation/devicetree/bindings/net/wireless/marvell-sd8xxx.txt +++ b/Documentation/devicetree/bindings/net/wireless/marvell-sd8xxx.txt@@ -14,6 +14,11 @@ Optional properties: - marvell,wakeup-gpios : Chip's wakeup gpio pin number. This needs to be downloaded to to firmware. Chip notifies wifi wakeup signal to SOC through this pin. + - interrupt-parent: phandle of the parent interrupt controller + - interrupts : interrupt number to the cpu + - gpios: specify SOC's wakeup GPIO
If you define the GPIO line as an interrupt, you shouldn't need it to be as GPIO too. Same on the BT side.
quoted hunk ↗ jump to hunk
+ - pinctrl-names : a pinctrl state named "default" must be defined + - pinctrl-0 : pin control group to be used for this controller Example:@@ -27,5 +32,10 @@ wlan { marvell,caldata_00_txpwrlimit_2g_cfg_set = /bits/ 8 < 0x01 0x00 0x06 0x00 0x08 0x02 0x89 0x01 ...>; marvell,wakeup-gpios = <3>; + interrupt-parent = <&pio>; + interrupts = <38 IRQ_TYPE_LEVEL_LOW>; + gpios = <&pio 38 GPIO_ACTIVE_HIGH>; + pinctrl-names = "default"; + pinctrl-0 = <&wake_pins>; };