Re: [zd1211-devs] [PATCH] UW2453 for TL-WN322G support + 2.6.22.x
Status: Beta
Brought to you by:
mayne
From: Hin-Tak L. <hin...@ya...> - 2009-08-08 14:25:36
|
--- On Fri, 7/8/09, "Sören Wellhöfer" <Soe...@gm...> wrote: > While adding support for the > TL-WN322G wireless usb adapter on an embedded system I > devised a patch to be applied against the 2.6.22.x kernel to > make it work. > > It basically integrates support for the uw2453 chip into > the zd1211rw driver framework of mentioned kernel and fixes > a few common issues associated with the device. > > Among the problems taken care of are those which are > described in this mailing list under the following > subjects: > > * [zd1211-devs] tp-link WN322G > * [zd1211-devs] zd1211 on asus A9RP > > Cheers, > Sören > > Signed-off-by: Sören Wellhöfer <soe...@gm...> I appreciate the effort of sharing such changes, personally - don't know if Linus is taking patches for such old kernels, probably not, so the the signed-off is probably unnecessary :-). I have one comment though - the patch is neither minimal (you added a couple of strictly-speakingnot-necessary debug print statements, and there is a whitespace change noted below), nor maximal - maximal would be back-porting the current code, i.e. a subset of the compat-wireless objective. Have you looked at compat-wireless? I don't know if it runs as earlier as 2.6.22, but I think it does. > diff -uprN > 2.6.22.6-vanilla/drivers/net/wireless/zd1211rw/zd_rf.c > 2.6.22.6/drivers/net/wireless/zd1211rw/zd_rf.c > --- > 2.6.22.6-vanilla/drivers/net/wireless/zd1211rw/zd_rf.c > 2009-04-23 11:36:28.000000000 +0200 > +++ > 2.6.22.6/drivers/net/wireless/zd1211rw/zd_rf.c > 2009-08-04 15:55:05.000000000 +0200 > @@ -25,7 +25,7 @@ > > static const char * const rfs[] = { > [0] > = "unknown RF0", > - [1] > = "unknown RF1", > + [1] > = "unknown RF1", > [UW2451_RF] = > "UW2451_RF", > [UCHIP_RF] = > "UCHIP_RF", > [AL2230_RF] = > "AL2230_RF", White space change, probably unintentional or indicative of a bug? |