On Mon, 2013-07-01 at 23:46 +0300, Felipe Balbi wrote: > Hi, > > On Mon, Jul 01, 2013 at 10:34:10PM +0300, Luciano Coelho wrote: > > diff --git a/arch/arm/boot/dts/omap4-panda-common.dtsi b/arch/arm/boot/dts/omap4-panda-common.dtsi > > index 670c3ce..7f061b8 100644 > > --- a/arch/arm/boot/dts/omap4-panda-common.dtsi > > +++ b/arch/arm/boot/dts/omap4-panda-common.dtsi > > @@ -65,11 +65,19 @@ > > enable-active-high; > > }; > > > > + > > wlan { > > - compatible = "ti,wilink6"; > > - interrupt-parent = <&gpio2>; > > - interrupts = <21 0x4>; /* gpio line 53, high level triggered */ > > - refclock = <2>; /* 38.4 MHz */ > > + compatible = "ti,wilink6"; > > + interrupt-parent = <&gpio2>; > > + interrupts = <21 0x4>; /* gpio line 53, high level triggered */ > > + clocks = <&refclock>; > > + clock-names = "refclock"; > > hmmm, shouldn't you provide both clocks (refclock and tcx0clock) > explicitly here ? No, not needed for Panda. Panda uses WiLink6 and only the refclock needs to be provided. > Also, you should probably make it clear that the WiLink module is fed by > the 32K sync clock just to make sure clock usecounts are correctly > incremented ? Hmmm, yes, that is probably a good idea. At least to make sure everything is initialized properly before the WiLink module is up and running. I'll look into it and eventually add in a separate patch. Thanks for your comments! -- Luca. -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html