On Fri, 2007-07-06 at 12:59 +0530, Satyam Sharma wrote:
> Hi Guillaume,
>
> [ Added relevant folks to Cc: ]
Botched backport, head is correct. It's worth noting that WPA didn't
work before the original commit at _all_, and after the commit it would
actually work, but you probably couldn't turn it off :)
Thanks for the catch, I'll fix and push to linville.
dan
> On 7/6/07, Guillaume LECERF <[email protected]> wrote:
> > [...]
> >
> > --- a/drivers/net/wireless/libertas/cmd.c
> > +++ b/drivers/net/wireless/libertas/cmd.c
> > @@ -241,7 +241,7 @@
> > if (*enable)
> > penableRSN->enable = cpu_to_le16(cmd_enable_rsn);
> > else
> > - penableRSN->enable = cpu_to_le16(cmd_enable_rsn);
> > + penableRSN->enable = cpu_to_le16(cmd_disable_rsn);
> > }
> >
> > lbs_deb_leave(LBS_DEB_CMD);
> >
>
> Thanks for the patch (looks reasonable to me), but:
>
> 1. No point sending a patch without a "Signed-off-by:" line.
> 2. Changelog (a few lines of explanation before signed-off-by) must
> specify what was wrong, and why the patch is correct.
> 3. Always Cc: relevant maintainers or at least _somebody_ who
> _will_ pick it up.
> 4. Gmail has done horrible things to the patch as usual, use some
> other mailer or script.
>
> And in general, please read: Documentation/SubmittingPatches
> and http://www.zip.com.au/~akpm/linux/patches/stuff/tpp.txt
>
> Satyam
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]