On Fri, Oct 12, 2012 at 10:11:57AM +0300, Kalle Valo wrote: > Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx> writes: > > > Very nice. > > Indeed. > > > But what is keeping this driver from going to the "proper" place in the > > kernel tree (i.e. drivers/net/...)? I don't have a problem adding it to > > the staging tree, but it would need a TODO file to list what is needed > > to get it out of the staging tree. > > I did a really quick 5 min review of this driver and IMHO this should go > directly to drivers/net/wireless/ (or maybe to drivers/net/wireless/ath, > Luis?), I don't see any reason why this should go to staging. The code > looks clean and is only 2 kLOC. But I'll review this better during the > weekend and also run some build tests. I have no particular objection to that. FWIW, Pontus contacted me and said something like "I've got this driver that sucks and needs a lot of work, should it go to staging?" Perhaps he was being too hard on himself? :-) 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