On Sun, Sep 7, 2008 at 1:30 AM, Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote: > On Sun, 2008-09-07 at 01:14 +0300, Tomas Winkler wrote: > >> - ieee80211_led_assoc(local, assoc); >> + ieee80211_led_assoc(local, 1); >> >> - sdata->bss_conf.assoc = assoc; >> + sdata->bss_conf.assoc = 1; > > I think you should use "true" at least for the latter "1" > >> + sdata->bss_conf.assoc = 0; > > and "false" here, and if the LED interface uses "bool" then for that > too. > > The actual patch looks good to me though. Will do Thanks Tomas -- 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
