On Tue, Sep 16, 2014 at 02:53:58PM +0200, Loic Poulain wrote: > Clock is disabled when the device is blocked. > So, clock_enabled is the logical negation of "blocked". > > Signed-off-by: Loic Poulain <loic.poulain@xxxxxxxxx> > --- > net/rfkill/rfkill-gpio.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/net/rfkill/rfkill-gpio.c b/net/rfkill/rfkill-gpio.c > index 14c98e4..408e51f 100644 > --- a/net/rfkill/rfkill-gpio.c > +++ b/net/rfkill/rfkill-gpio.c > @@ -54,7 +54,7 @@ static int rfkill_gpio_set_power(void *data, bool blocked) > if (blocked && !IS_ERR(rfkill->clk) && rfkill->clk_enabled) > clk_disable(rfkill->clk); > > - rfkill->clk_enabled = blocked; > + rfkill->clk_enabled = !blocked; > > return 0; > } This looks like the right fix, but...the code has been that way for a long time. If this patch is correct, how has this gone undetected for so long? John -- John W. Linville Someday the world will need a hero, and you linville@xxxxxxxxxxxxx might be all we have. Be ready. -- 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
