Re: [PATCH] ath5k: added initial RFKILL support

From: Tobias Doerffel
Date: Fri May 29 2009 - 18:33:34 EST


Hi,

Am Dienstag, 26. Mai 2009 01:32:46 schrieb Johannes Berg:
> On Tue, 2009-05-26 at 01:24 +0200, Tobias Doerffel wrote:
> > + err = rfkill_register(sc->rf_kill.rfkill);
> > + if (err) {
> > + sc->rf_kill.rfkill = NULL;
> > + return -1;
> > + }
>
> Missing rfkill_free, and why -EPERM?
Thanks for this remark. I fixed the code and will post an updated patch soon.

Any other comments on the patch? If not, any chance to get it merged into
wireless-testing branch?

> Could you try to work against v11
> of my rfkill patch, or better even against the cfg80211 rfkill instead?
I didn't look at both of them yet. However I think we first should try to get
current rfkill support into mainline as fast as possible. Improvements and
adaptations to reworked rfkill framework can follow.

Regards,

Toby

Attachment: signature.asc
Description: This is a digitally signed message part.