On Wed, 2012-05-30 at 17:45 +0300, Eliad Peller wrote: > > +#ifdef CONFIG_PM > > ret = enable_irq_wake(wl->irq); > > if (!ret) { > > wl->irq_wake_enabled = true; > > @@ -5315,6 +5316,7 @@ int __devinit wlcore_probe(struct wl1271 *wl, struct platform_device *pdev) > > } > > } > > disable_irq(wl->irq); > > +#endif > > i think disable_irq should be outside the ifdef. Well, ok, I can do that if you want, but you may still want to fix this for 3.5 since it'll otherwise cause a runtime error (registration failure) without CONFIG_PM and I don't think this patch will go to 3.5. johannes -- 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