Thread (1 message) 1 message, 1 author, 2017-05-25

Re: [PATCH v2 1/2] ARM: dts: orange-pi-zero: specify wifi interrupts

From: Chen-Yu Tsai <hidden>
Date: 2017-05-25 02:57:39
Also in: linux-arm-kernel

On Thu, May 25, 2017 at 5:24 AM, Sergey Matyukevich [off-list ref] wrote:
quoted hunk
The orange-pi-zero board has Allwinner XR819 SDIO wifi chip. The board
dts file provides a node enabling mmc1 controller. Together with an
out-of-tree driver https://github.com/fifteenhex/xradio this node
enables using the wifi chip.

This commit specifies XR819 interrupt configuration for the driver.

Signed-off-by: Sergey Matyukevich <redacted>
---
 arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts | 3 +++
 1 file changed, 3 insertions(+)
diff --git a/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts b/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts
index 9e8b082c134f..e97dfd1775d1 100644
--- a/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts
+++ b/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts
@@ -128,6 +128,9 @@
         */
        xr819: sdio_wifi@1 {
                reg = <1>;
+               compatible = "xradio,xr819";
You should submit device tree bindings first. See:

https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/tree/Documentation/devicetree/bindings/net/wireless

for examples.
+               interrupt-parent = <&pio>;
+               interrupts = <6 10 IRQ_TYPE_EDGE_RISING>;
Are these optional or required properties? AFAIK SDIO supports in-band
interrupts with a bit of overhead, so they should be optional.

ChenYu
        };
 };

--
2.11.0
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help