> > 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. Does it make sense for the out-of-tree driver ? > > + 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. Those properties are optional. Driver works anyway, though it complains about missing interrupts. IIUC xradio driver falls back to polling mode only if xr819 interrupt configuraton is not specified. Regards, Sergey -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html