Return-path: Received: from mail-it0-f50.google.com ([209.85.214.50]:37753 "EHLO mail-it0-f50.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754188AbcGTO0C (ORCPT ); Wed, 20 Jul 2016 10:26:02 -0400 Subject: Re: [PATCH 2/3] staging/rtl8192e: use s8 instead of char To: Jes Sorensen , Arnd Bergmann References: <20160719153403.2967812-1-arnd@arndb.de> <3858062.NGQDOLnTj0@wuerfel> <4806848.QL77nxBrs4@wuerfel> Cc: linux-wireless@vger.kernel.org, Kalle Valo , netdev@vger.kernel.org, Greg Kroah-Hartman , Mateusz Kulikowski , devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org, Andrea Merello From: Larry Finger Message-ID: <29c970ef-0bd5-38f2-48fd-91adb01fc8a9@lwfinger.net> (sfid-20160720_162610_680098_A58BD139) Date: Wed, 20 Jul 2016 09:25:59 -0500 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=windows-1252; format=flowed Sender: linux-wireless-owner@vger.kernel.org List-ID: On 07/20/2016 06:25 AM, Jes Sorensen wrote: > Arnd Bergmann writes: >> On Tuesday, July 19, 2016 12:05:00 PM CEST Jes Sorensen wrote: >>> Arnd Bergmann writes: >>> I think that would be better, albeit not a big issue. >> >> Ok, and since Kalle applied the first patch to his tree, I'm now sending >> a series of three patches that are all for Greg, which also avoids some >> possible confusion. > > Awesome! > >>> I'd like to get rid of all the drivers/staging/rtl* drivers eventually >> >> That would be great, yes. >> >> Can you clarify what the long-term plan is? I see that >> drivers/net/wireless/realtek/rtlwifi/ has most of the PCIe parts >> and one USB device (rtl8192cu/rtl8188cus) while >> drivers/net/wireless/rtl8xxx has all the USB parts including >> that one. >> >> Does that mean we want the staging drivers for PCIe devices >> to get merged into rtlwifi, and the remaining USB drivers to get >> replaced by r8xxxu? > > Well it really all depends on how much time I have and how much others > step up and help contribute to the code. For rtl8xxxu my plans are as > follows: > > 1) rtl8188eu support, since this is the most widely distributed USB > dongle which isn't currently supported by a non staging driver. I am > currently working on this together with Andrea Merello. > > 2) Beacon support for IBSS and AP mode - hopefully this should make it > possible to default rtl8xxxu for rtl8192cu/rtl8188cu devices and disable > them in rtlwifi. > > 3) SDIO device support > > 4) PCI device support > > 5) 802.11ac device support > > 3/4/5 not necessarily in that order. There really is no reason why > rtl8xxxu shouldn't have SDIO and PCI device support added so the core > code can be shared. > >> As one data point that I can provide (but you are probably >> aware of), I could never get my rtl8188cus stick to work with >> rtlwifi, but I found the older r8712u device to work fine with >> the staging/rtl8712 driver. > > I'd love to hear if the rtl8188cus works better with rtl8xxxu. For the > rtl8712 device, rtl8192su?, then potentially that could be added to > rtl8xxxu as well, but it's not a top priority on my list right now. I would be very pleased to eliminate all the rtl* drivers from staging, particularly the USB varieties. I have been very disappointed with the USB group at Realtek. They dropped rtl8192cu on me, and then have provided no support. As I know nothing of the chip internals, there has been little I could do to improve the driver. In addition, they have never implemented any of the code improvements in any new USB drivers. Jes has done a wonderful job of making sense of the crap drivers from Realtek, and rtl8xxxu is a very good start. Adding SDIO support for the RTL8723BS would be a plus. At the moment, there is only an out-of-tree driver at GitHub, which I refuse to submit to staging. At the moment, it appears that every tablet with this chip needs special treatment. In addition, the SDIO support seems to be buggy. The drivers in question and my comments are as follows: rtl8712u and rtl8188eu - replaced by new code in rtl8xxxu. rtl8723au - already in rtl8xxxu rtl8192e - This chip is so rarely used that it does not make much difference what we do. If someone wants to integrate it into rtlwifi, I would have no objection, but I do not expect to contribute to the effort. rtl8192u - This one is even rarer. I have no sample, thus I would be unable to test. rtl8192cu - Replace with rtl8xxxu as soon as the new driver supports beacons. Good riddance. Larry