From patchwork Wed Jul 20 15:13:58 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Arnd Bergmann X-Patchwork-Id: 101995 Delivered-To: patch@linaro.org Received: by 10.140.29.52 with SMTP id a49csp38515qga; Wed, 20 Jul 2016 08:18:53 -0700 (PDT) X-Received: by 10.66.76.9 with SMTP id g9mr47821976paw.51.1469027933684; Wed, 20 Jul 2016 08:18:53 -0700 (PDT) Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id s18si3869196pal.122.2016.07.20.08.18.53; Wed, 20 Jul 2016 08:18:53 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-wireless-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-wireless-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-wireless-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754714AbcGTPSv (ORCPT + 1 other); Wed, 20 Jul 2016 11:18:51 -0400 Received: from mout.kundenserver.de ([217.72.192.74]:62231 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754339AbcGTPSs (ORCPT ); Wed, 20 Jul 2016 11:18:48 -0400 Received: from wuerfel.lan. ([78.42.132.4]) by mrelayeu.kundenserver.de (mreue104) with ESMTPA (Nemesis) id 0LrKW4-1bDlHA2WhI-0138cu; Wed, 20 Jul 2016 17:18:24 +0200 From: Arnd Bergmann To: Greg Kroah-Hartman Cc: linux-wireless@vger.kernel.org, Kalle Valo , Larry Finger , netdev@vger.kernel.org, Jes Sorensen , Arnd Bergmann , devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org Subject: [PATCH 1/3] staging/rtl8192u: use s8 instead of char Date: Wed, 20 Jul 2016 17:13:58 +0200 Message-Id: <20160720151818.4142386-1-arnd@arndb.de> X-Mailer: git-send-email 2.9.0 X-Provags-ID: V03:K0:X/ZRMc+IyvuRSXnCnCFAPfmKLM5OHTXEz9a/eNHyvlJUFSZ8KDn 1cvPjoIZ5YXbNZk7J9B/k4woBdWw8coPp7x2N4BDrAUEqReNtm3OULYGAD63+y0Mx1/3wSx RFnst3SdVnsubQ2LmShR5ryTVOVvJuNtyOIazl3WjNF70QnhDJUZt2a/GZlIZgJ15LGBU/c ELFDn4Z0j+zKSGB9SZhdQ== X-UI-Out-Filterresults: notjunk:1; V01:K0:gj2ykz+nlX4=:ZzeCbJmepHsYHXDnsPtrZJ Fn9z3E4gEhloigpVlGwq5Ms1GHn9DsjnMRvrNavvd0OZ8u2t6XJ86rzejcc6Z7tcAz99M41Ec y2FQTEIeTC3mT30HH1KLLC25qBdw1dypUKlevtXXTNQszssJ2u7SlesOkXPIeML+md00qI+Ia Nb7Vv3w4X/LT/mx6zFwcmeJrW1JlcGmdiXIabTiquu0NhnvR0f8UpX3EHX0gvmDrm3ohH8Qye HzVDUC3n1Uxb7NycUl2vAefrx9vJLvgT4cNvfFypuVOQxrTOwbzTjE4Bz+KDcIsddhQJ4iLDP VJ7sIxTy81TN93WXwttMoECofDNnnQmJt4p7YPmHGBLIZKY/IfTvNqcznk7FD71fRF5wlbqgr t/GyZWZsGsvVmBo9nPClHBc7axVExQfjy5ZWvU5ZO85Stk575kaliPRV+ufm4Ypck40y2p7nR fgGeVzuR+yHeuNzRPGAj/Nw1sElbwTGYR5h6FZrB1JVCf40UfkkH8xkvSu22Fc0Wcj7TSnJnS cyEFnwavsk5leJZk5LxKQFnPx5gGDVuMUnD+yh95IE6ege/712zeWwktSiLVdInRVpxCsXnsN qDoNleL9YM1CqhbPTEprxcmPOy28GI8v1IacRrFG9Vik2cfz9h7aCqomKEc/FRyffQMJlc+3w epZNKmxDf+j3lBqmuPpQH8IUMDM8JMiJeNNYPmubtSu+RWQ92TmNvuftmIUZZ5J/BTC0= Sender: linux-wireless-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-wireless@vger.kernel.org Compiling the rtlwifi drivers for ARM with gcc -Wextra warns about lots of incorrect code that results from 'char' being unsigned here, e.g. staging/rtl8192u/r8192U_core.c:4150:16: error: comparison is always false due to limited range of data type [-Werror=type-limits] staging/rtl8192u/r8192U_dm.c:646:50: error: comparison is always false due to limited range of data type [-Werror=type-limits] This patch changes all uses of 'char' in this driver that refer to 8-bit integers to use 's8' instead, which is signed on all architectures. Signed-off-by: Arnd Bergmann Acked-by: Jes Sorensen --- drivers/staging/rtl8192u/ieee80211/ieee80211.h | 4 ++-- drivers/staging/rtl8192u/r8192U.h | 4 ++-- drivers/staging/rtl8192u/r8192U_core.c | 14 +++++++------- 3 files changed, 11 insertions(+), 11 deletions(-) -- 2.9.0 -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211.h b/drivers/staging/rtl8192u/ieee80211/ieee80211.h index 09e9499b7f9d..077ea13eb1e7 100644 --- a/drivers/staging/rtl8192u/ieee80211/ieee80211.h +++ b/drivers/staging/rtl8192u/ieee80211/ieee80211.h @@ -746,7 +746,7 @@ struct ieee80211_rx_stats { bool bisrxaggrsubframe; bool bPacketBeacon; //cosa add for rssi bool bToSelfBA; //cosa add for rssi - char cck_adc_pwdb[4]; //cosa add for rx path selection + s8 cck_adc_pwdb[4]; //cosa add for rx path selection u16 Seq_Num; }; @@ -1814,7 +1814,7 @@ struct ieee80211_device { u32 wpax_type_notify; //{added by David, 2006.9.26} /* QoS related flag */ - char init_wmmparam_flag; + s8 init_wmmparam_flag; /* set on initialization */ u8 qos_support; diff --git a/drivers/staging/rtl8192u/r8192U.h b/drivers/staging/rtl8192u/r8192U.h index 5dba6a28dd9b..b28bc7812caa 100644 --- a/drivers/staging/rtl8192u/r8192U.h +++ b/drivers/staging/rtl8192u/r8192U.h @@ -533,7 +533,7 @@ typedef struct _rt_9x_tx_rate_history { u32 ht_mcs[4][16]; } rt_tx_rahis_t, *prt_tx_rahis_t; typedef struct _RT_SMOOTH_DATA_4RF { - char elements[4][100]; /* array to store values */ + s8 elements[4][100]; /* array to store values */ u32 index; /* index to current array to store */ u32 TotalNum; /* num of valid elements */ u32 TotalVal[4]; /* sum of valid elements */ @@ -1031,7 +1031,7 @@ typedef struct r8192_priv { s8 cck_present_attentuation; u8 cck_present_attentuation_20Mdefault; u8 cck_present_attentuation_40Mdefault; - char cck_present_attentuation_difference; + s8 cck_present_attentuation_difference; bool btxpower_tracking; bool bcck_in_ch14; bool btxpowerdata_readfromEEPORM; diff --git a/drivers/staging/rtl8192u/r8192U_core.c b/drivers/staging/rtl8192u/r8192U_core.c index dd0970facdf5..f36b2d3b1ee9 100644 --- a/drivers/staging/rtl8192u/r8192U_core.c +++ b/drivers/staging/rtl8192u/r8192U_core.c @@ -4209,7 +4209,7 @@ static void rtl8192_process_phyinfo(struct r8192_priv *priv, u8 *buffer, * * Return: 0-100 percentage *---------------------------------------------------------------------------*/ -static u8 rtl819x_query_rxpwrpercentage(char antpower) +static u8 rtl819x_query_rxpwrpercentage(s8 antpower) { if ((antpower <= -100) || (antpower >= 20)) return 0; @@ -4220,9 +4220,9 @@ static u8 rtl819x_query_rxpwrpercentage(char antpower) } /* QueryRxPwrPercentage */ -static u8 rtl819x_evm_dbtopercentage(char value) +static u8 rtl819x_evm_dbtopercentage(s8 value) { - char ret_val; + s8 ret_val; ret_val = value; @@ -4297,8 +4297,8 @@ static void rtl8192_query_rxphystatus(struct r8192_priv *priv, phy_ofdm_rx_status_rxsc_sgien_exintfflag *prxsc; u8 *prxpkt; u8 i, max_spatial_stream, tmp_rxsnr, tmp_rxevm, rxsc_sgien_exflg; - char rx_pwr[4], rx_pwr_all = 0; - char rx_snrX, rx_evmX; + s8 rx_pwr[4], rx_pwr_all = 0; + s8 rx_snrX, rx_evmX; u8 evm, pwdb_all; u32 RSSI, total_rssi = 0; u8 is_cck_rate = 0; @@ -4423,7 +4423,7 @@ static void rtl8192_query_rxphystatus(struct r8192_priv *priv, /* Get Rx snr value in DB */ tmp_rxsnr = pofdm_buf->rxsnr_X[i]; - rx_snrX = (char)(tmp_rxsnr); + rx_snrX = (s8)(tmp_rxsnr); rx_snrX /= 2; priv->stats.rxSNRdB[i] = (long)rx_snrX; @@ -4457,7 +4457,7 @@ static void rtl8192_query_rxphystatus(struct r8192_priv *priv, for (i = 0; i < max_spatial_stream; i++) { tmp_rxevm = pofdm_buf->rxevm_X[i]; - rx_evmX = (char)(tmp_rxevm); + rx_evmX = (s8)(tmp_rxevm); /* Do not use shift operation like "rx_evmX >>= 1" * because the compiler of free build environment will