Re: [PATCH] at76_usb: update drivers/staging/at76_usb w/ mac80211port

[John W. Linville]

On Mon, Oct 27, 2008 at 11:56:34AM -0700, Greg KH wrote:
> On Fri, Oct 24, 2008 at 03:48:59PM -0400, John W. Linville wrote:
> > This updates drivers/staging/at76_usb to correspond with the latest
> > version previously available in wireless-testing.
> > 
> > Signed-off-by: John W. Linville <linville@xxxxxxxxxxxxx>
> 
> I've applied this, but it does add the following warnings, which I don't
> think you want to have:
> 	drivers/staging/at76_usb/at76_usb.c:885: warning: âat76_set_associdâ defined but not used
> 	drivers/staging/at76_usb/at76_usb.c:903: warning: âat76_set_listen_intervalâ defined but not used
> 	drivers/staging/at76_usb/at76_usb.c:989: warning: âat76_add_mac_addressâ defined but not used
> 
> 
> I'll remove these unused functions now from the in-kernel version.

Those were still there because Kalle had said he intended to use them.
I see no need to carry them since you have them in git anyway.

> Is there any way to do development on this in the staging version now,
> instead of the wireless-testing tree?

Fine with me, although I do have at least one patch for it that
relates to a pending API change -- I just figured I'd wait for the
round-trip of the patch above and then carry just that patch in my
tree along with the mac80211 API changes that prompted it.  Any other
development makes sense to go to your tree.

Alternatively, I could just drop the patch I have completely or send
it to you to keep until 2.6.29...?

John
-- 
John W. Linville		Linux should be at the core
linville@xxxxxxxxxxxxx			of your literate lifestyle.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/