From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753325AbcGTIZq (ORCPT ); Wed, 20 Jul 2016 04:25:46 -0400 Received: from mout.kundenserver.de ([217.72.192.73]:62670 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752730AbcGTIZm (ORCPT ); Wed, 20 Jul 2016 04:25:42 -0400 From: Arnd Bergmann To: Jes Sorensen Cc: linux-wireless@vger.kernel.org, Kalle Valo , Larry Finger , netdev@vger.kernel.org, Greg Kroah-Hartman , Mateusz Kulikowski , devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 2/3] staging/rtl8192e: use s8 instead of char Date: Wed, 20 Jul 2016 10:25:21 +0200 Message-ID: <4806848.QL77nxBrs4@wuerfel> User-Agent: KMail/5.1.3 (Linux/4.4.0-28-generic; KDE/5.18.0; x86_64; ; ) In-Reply-To: References: <20160719153403.2967812-1-arnd@arndb.de> <3858062.NGQDOLnTj0@wuerfel> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" X-Provags-ID: V03:K0:YfRcqXyRg5M1AGBzzr+SiyGsO+pnjsFZL96iP9TjlAoj1QA0/Tp kTssqoYDrBnOBowTrDUxH5SkMzHZKZRnRG2bwXszcV87NV9OVU7aBp2mymbIH/yI3Krxa42 ibXu7mhyCevvriSH8w2O6ETm7kv4W9aeKvFIDiabKQlT+51tWt0akBPFGEdHK7Y0AE8KHOU KzZ0NolO8jNffGS/Q74wg== X-UI-Out-Filterresults: notjunk:1;V01:K0:TkMiUyfZC9k=:mJGANSQx8aCZFVSfx+FQ4n KpqSqprpffpZriCI8KWlL5BrIidUwP3SxBixWfnDwsiHxVlZIry0M0ez27f2ryDMZZSzHoS3w knczCw1W+lrtqH0PNMJOgEUWfr6xjoSLGKYI4oTbeJJ8OaeQxPop5ttMoTZZxaqDDoazNzCtd ojTfllXl5tEWHAiPghxWzvZ+DDQY64Tl7yec8L1/1aKzKY4AMuR8lGJIalqjgoa9HlVd2ee62 H3dtNlrbeajtmk2vucKio7iJVkIH/Is/4xqqoeJM9p0Jnsz5ddwvpeQRVkHwN2gJ74rgxH0AU EypgtNSRwIv/VpG5k3NQv2ti1mNsLoDPec6nUJF1SMLnwmDXumFvZeFGDsPa9NnOLYGtmisav m7335iZe+XPwW8fy8fWJtW1kqyRWVcJsPGZ0RfGmSueTRq0tRR2SU29cLYe5GHkQyibf7aF5Y Ii/TrL88MUjwDVxDzUFqC8oT4u7uzOVBQXMePlJpsoAtu/2/kY6s35r284xRe7DyCXFdFUMtX BBUA/6gQ5nI/GKGr9uu3VWgRjqHeUwZ1FKdYgO11WyVruOMogcTResYQ3yHh8pgOi1odfVK4B FO0Uc20FtyzQWXikvshVZGTv9WQEU1r4gXLNUgZqZZHlfeSYolAdL0fl2ufbVlIN4HEehbqTD tba0mCihndjd8THuu8XrMEedeGezQKouUfBs+V+KVVNM4kUbdPHYI2bSdue7PJhdgBnm14NFU OaBWSkxSCWthAvh0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tuesday, July 19, 2016 12:05:00 PM CEST Jes Sorensen wrote: > Arnd Bergmann writes: > > On Tuesday, July 19, 2016 11:46:04 AM CEST Jes Sorensen wrote: > >> > diff --git a/drivers/staging/rtl8192e/rtl819x_TSProc.c b/drivers/staging/rtl8192e/rtl819x_TSProc.c > >> > index 2c8a526773ed..e0a2fe5e6148 100644 > >> > --- a/drivers/staging/rtl8192e/rtl819x_TSProc.c > >> > +++ b/drivers/staging/rtl8192e/rtl819x_TSProc.c > >> > @@ -323,7 +323,7 @@ bool GetTs(struct rtllib_device *ieee, struct ts_common_info **ppTS, > >> > if (ieee->current_network.qos_data.supported == 0) { > >> > UP = 0; > >> > } else { > >> > - if (!IsACValid(TID)) { > >> > + if (!IsACValid((s8)TID)) { > >> > netdev_warn(ieee->dev, "%s(): TID(%d) is not valid\n", > >> > __func__, TID); > >> > return false; > >> > >> TID is a 4-bit field, it should never go negative. The cast to s8 seems > >> wrong to me, if anything it should be using u8. I do realize the macro > >> IsACValid checks against negative too, but that just looks silly to me. > > > > Ok, I'll remove the extra comparison then to avoid the warning: > > > > staging/rtl8192e/rtl819x_TSProc.c:326:14: error: comparison is always > > true due to limited range of data type [-Werror=type-limits] > > > > I guess it should be a separate patch. I had just stumbled over the > > same thing before resending the patch but decided not to change it > > to keep the patch simple. > > 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. > 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? 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. Arnd