On 01/14/2014 11:40 AM, Andrea Merello wrote:
Hello Bernhard,
Are you still curious to test something ? :)
I attach a patch that should apply to wireless-testing..
I still have several items pending on my TODO list, including things
to tidy/fix, and things that are currently missing, however I'm
getting encouraging results on my setup..
If this works for you, and if other people in CC here agree, I may
starting split this mega-patch in something more suitable for kernel
maintainers to merge (currently it even changes few misleading names
in the common register struct, affecting also rtl8187 driver)..
Let me know if it works :)
Andrea,
Building your version gets the following warning:
CC [M] drivers/net/wireless/rtl818x/rtl8187/rfkill.o
drivers/net/wireless/rtl818x/rtl8187/dev.c: In function ‘rtl8187_set_anaparam’:
drivers/net/wireless/rtl818x/rtl8187/dev.c:596:3: warning: passing argument 2 of
‘rtl818x_iowrite8’ from incompatible pointer type [enabled by default]
In file included from drivers/net/wireless/rtl818x/rtl8187/dev.c:32:0:
drivers/net/wireless/rtl818x/rtl8187/rtl8187.h:237:20: note: expected ‘u8 *’ but
argument is of type ‘__le16 *’
The easy fix is to change that to an ...iowrite16() call. So far, my testing to
see if writing that extra byte messes up the RTL8187B has not shown any
problems; however, I have not yet tested on a big-endian machine.
Have you given any thought to renaming the driver? It seems to me that rtl8180
was probably not right when it drove both RTL8180 and RTL8185, and it is even
less appropriate today.
The driver connects to the following APs:
802.11n running WPA2
802.11g running WPA1
802.11g running WEP
Looks good.
Larry
--
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
