Hello Eliad, On Wed, Mar 11, 2015 at 1:28 AM, Javier Martinez Canillas <javier@xxxxxxxxxxxx> wrote: > On Mon, Mar 9, 2015 at 4:36 PM, Eliad Peller <eliad@xxxxxxxxxx> wrote: >> --- a/arch/arm/boot/dts/omap3-igep0020-rev-f.dts >> +++ b/arch/arm/boot/dts/omap3-igep0020-rev-f.dts >> @@ -42,4 +42,13 @@ >> vmmc-supply = <&lbep5clwmc_wlen>; Something I forgot to mention is that this vmmc-supply is a DT hack since the WLAN SDIO chip needs a WL_EN signal to be asserted as a part of the chip power sequencing. But there wasn't a DT binding to describe that so most boards use the same hack which is to define a fake fixed-regulator with an enable GPIO just to toggle that pin. But now the MMC subsystem has support for power sequence providers and the pwrseq-simple driver (Documentation/devicetree/bindings/mmc/mmc-pwrseq-simple.txt) has support for reset GPIOs so you may want to change that as well. In fact, the pwrseq-simple also has support for an external clock (and can be extended to support more than one) so if the clocks are not internal to the wl12xx, maybe these should be defined in the pwrseq-simple dev node assuming that there is a clock driver for them. Best regards, Javier -- 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
