Could I apply these patches on top of Cascardo's? Or should I tru individually for better testing ? Rogerio 2010/11/3 Herton Ronaldo Krzesinski <herton@xxxxxxxxxxxxxxx>: > On Tue, 2 Nov 2010 21:48:20 -0200 > Rogerio Luz Coelho <rogluz.news@xxxxxxxxx> wrote: > >> Ok, I see the 8187b is in development. >> >> But PLEASE ask Thadeu Cascardo for his patch as it seems to fix the >> issue with these cards when connected to Laptop (Positivo Laptops in >> Brasil for instance), this would even let the Mandriva folks show >> Positivo that they don't need to ship the Mandriva 2008.0 woith their >> Linux Laptops anymore ... > > He already reposted the patch and this series is based on top of it, so > soon we should see it applied I guess. > >> >> the patch he sent me FIXED the problem where the chip did not connect >> or when it did it would lose the connection right away... >> >> see >> https://bugs.launchpad.net/ubuntu/+source/linux/+bug/215802?comments=all >> >> >> I haven´t aplied any other patches and the one he sent me JUST WORKS, >> tested in a Ubuntu 10.04 64bit (64bit being important since the >> Realtek support told me they did NOT support 64bit with their linux >> drivers) > > Note that this series don't replace patch from Thadeu, it's just more cleanups > and fixes. > >> >> Thanks >> >> Rogerio >> >> 2010/11/2 Herton Ronaldo Krzesinski <herton@xxxxxxxxxxxxxxx>: >> > On 02-11-2010 00:46, Larry Finger wrote: >> >> >> >> On 11/01/2010 09:42 PM, Hin-Tak Leung wrote: >> >>> >> >>> --- On Tue, 2/11/10, Herton Ronaldo Krzesinski<herton@xxxxxxxxxxxxxxx> >> >>> wrote: >> >>> >> >>>> We were using wrong address for BRSR >> >>>> (Basic Rate Set Register) while >> >>>> initializing its value, comparing with Realtek sources, for >> >>>> 8187B case. >> >>>> >> >>>> Also, the same register is initialized in >> >>>> rtl8187b_reg_table, so remove >> >>>> the duplicate initialization from the table. >> >>>> >> >>>> Signed-off-by: Herton Ronaldo Krzesinski<herton@xxxxxxxxxxxxxxx> >> >>>> Acked-by: Larry Finger<Larry.Finger@xxxxxxxxxxxx> >> >>> >> >>> Acked-by: Hin-Tak Leung<htl10@xxxxxxxxxxxxxxxxxxxxx> >> >>> >> >>> ANAPARAM* stands for "anonymous parameters", right? One of these days we >> >>> should give them some meaningful names. >> >> >> >> I have always thought it was for "analog parameter". Am I wrong? >> > >> > Nope, you're right, its listed description is "analog parameter" in 8180 >> > spec. >> > >> >> >> >> Larry >> > >> > -- >> > []'s >> > Herton >> > -- >> > 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 >> > > > > -- > []'s > Herton > -- 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
