From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from smtp.codeaurora.org ([198.145.29.96]:34702 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751090AbbJWRHr (ORCPT ); Fri, 23 Oct 2015 13:07:47 -0400 From: Kalle Valo To: Jes Sorensen Cc: Xose Vazquez Perez , linux-wireless , Larry Finger Subject: Re: overlap of devices in rtl8xxxu and rtl8192cu References: <562A2711.9040700@gmail.com> <87ziz91xag.fsf@kamboji.qca.qualcomm.com> Date: Fri, 23 Oct 2015 20:07:37 +0300 In-Reply-To: (Jes Sorensen's message of "Fri, 23 Oct 2015 10:26:06 -0400") Message-ID: <87oafp1pk6.fsf@kamboji.qca.qualcomm.com> (sfid-20151023_190750_573921_BFACE5AE) MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Sender: linux-wireless-owner@vger.kernel.org List-ID: Jes Sorensen writes: > Kalle Valo writes: >> Jes Sorensen writes: >> >>> Xose Vazquez Perez writes: >>>> >>>> There is overlap of devices between rtl8xxxu and rtl8192cu: >>>> USB_VENDER_ID_REALTEK, 0x8176 >>>> USB_VENDER_ID_REALTEK, 0x8178 >>>> USB_VENDER_ID_REALTEK, 0x817f >>>> 0x7392, 0x7811 >>> >>> Yes >> >> What, are you saying that you didn't fix that? I did say in the last >> review round that we should not have duplicated device entries within >> drivers/net/wireless and I assumed you took care of that. >> >>> and there is also a clash with USB IDs between >>> drivers/staging/rtl8192u and drivers/net/wireless/realtek/rtlwifi - >>> this is nothing new. >> >> We should do things properly in under drivers/, I don't care what >> happens in staging and that's no excuse to do things badly. > > As I explained earlier, there is no way I can do that unless we remove > rtlwifi's rtl8192cu driver and that cannot happen due to the feature > differences. There are plenty of other conflicts and this is not a bug. Like I said earlier in rtl8xxxu review, I don't want to have two drivers supporting the same device. So either rtl8xxxu or rtlwifi supports the devices above, not both. And to my knowledge rtl8xxxu is still lacking in features, and it's a new driver, so it's best that rtlwifi continues to support these for the time being. Once rtl8xxxu catches up and we have a clear consensus we can make the switch. I'll send a patch in a jiffy, merge window is getting closer so we don't have any extra time. -- Kalle Valo