Search Linux Wireless

Re: [PATCH 1/1] New driver: rtl8723au (mac80211)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 03/06/2015 04:15 PM, Jes.Sorensen@xxxxxxxxxx wrote:
From: Jes Sorensen <Jes.Sorensen@xxxxxxxxxx>

This is an alternate driver for the Realtek 8723AU (rtl8723au) written
from scratch utilizing the mac80211 stack.

After spending months cleaning up the vendor provided rtl8723au
driver, which comes with it's own 802.11 stack included, I decided to
rewrite this driver from the bottom up.

Many thanks to Johannes Berg for 802.11 insights and help and Larry
Finger for help with the vendor driver.

The full git log for the development of this driver can be found here:
git git://git.kernel.org/pub/scm/linux/kernel/git/jes/linux.git
     branch rtl8723au-mac80211

This driver is still experimental, but has proven to be rather stable
for me. It lacks some features found in the staging driver, such as
power management, AMPDU, and 40MHz channel support. In addition there
is no AP and monitor support at this point.

Signed-off-by: Jes Sorensen <Jes.Sorensen@xxxxxxxxxx>

Quilt reports the following when this patch is refreshed:

Warning: trailing whitespace in lines 2862,3273,3492,3521 of drivers/net/wireless/rtl8xxxu.c

I have not analyzed all the temporary manipulations of rtl8xxxu_debug to see what you are doing; however, I suggest that you add a module parameter so that debugging can be enabled without rebuilding the module. That way a user who is using a distro binary can enable debugging without the hassle of a full kernel rebuild.

Because this driver is under drivers/net, checkpatch.pl adds additional tests that may not be used in other trees. For instance, it complains when it finds a block comment that starts with a bare "/*". What is usually done is to use "/**" instead.

I think I understand why some of the "#if 0" blocks are present, but others are not clear. For example, I see no value of keeping code that is labelled "only for PCIe". Is it your intention to add the RTL8723AE to this driver?

Running checkpatch.pl on this patch results in total of 24 errors, 99 warnings, and 105 checks. From your mail exchange with Joe Perches, I understand that you will not wish to fix all of these; however, the errors should be handled. The fewer of the warnings or checks that are left, the better, if only to prevent interference from the script kiddies.

I like the clean look of the code. That is particularly impressive given the look of the original. I wish I had the hardware on which to test it. Perhaps I can wedge it into a kernel version that builds and runs on my Radxa Rock,

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




[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Wireless Personal Area Network]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite Hiking]     [MIPS Linux]     [ARM Linux]     [Linux RAID]

  Powered by Linux