From nobody Sun Feb 8 13:09:04 2026 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 2759AEB64D9 for ; Mon, 19 Jun 2023 15:10:23 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230345AbjFSPKV (ORCPT ); Mon, 19 Jun 2023 11:10:21 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:59276 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229657AbjFSPKH (ORCPT ); Mon, 19 Jun 2023 11:10:07 -0400 Received: from mail-ed1-x52c.google.com (mail-ed1-x52c.google.com [IPv6:2a00:1450:4864:20::52c]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 31EDAC2 for ; Mon, 19 Jun 2023 08:10:06 -0700 (PDT) Received: by mail-ed1-x52c.google.com with SMTP id 4fb4d7f45d1cf-519274f7b05so4863439a12.3 for ; Mon, 19 Jun 2023 08:10:06 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20221208; t=1687187404; x=1689779404; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=lQqWKTsadGM4IA7NNaSI9I45VeGk8Yh+tmcH6FyzFNE=; b=JHlNw2RyBWoUL/sErR78SZSPa75mtmwBlcmoNP2bL0W7lnYa0ONfKZp6x/l1BbuEer 9EcXj7B6c/m6LmZ8NwsnvAdPC2hhon2df7Jksh6mpQie0htEKef32nOpk19NN/5GeN7j hMp2pqCPGiYrKkYKcKSiiFUhh9z5XR4N9y/zIFmN/y3c3ua4dx4lVPF0f6+42B5yiIrv aqS2Gtadueh68UeeKmRcemzyE7fxSMx3ngQkeRl20kUmDq5JhIwwbnG7YxmI0hSbjY4l 7umb0YWJwMuNeC2CkJS8mofy6ScwCtVN1TIydbXeYuVyZhWiKulydwufiUF9fYFMPwkr IRzQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1687187404; x=1689779404; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=lQqWKTsadGM4IA7NNaSI9I45VeGk8Yh+tmcH6FyzFNE=; b=MWf2RVIW8SKTj/giqd2G7lO1lPZdopqSbyWuhk8exbwLy4tv9GpGPn7NhsC6FxF2kL PU6ixNUZHLb5aPlrPFJ8k94MXNpJX88Dyf17rsG4bT/ApDHpUVvxFZgULPkFHVh7Q88T Zvv9HGqxcW6tUleo/viG7h6NK6n7ujeiGj4g/gLG2u0MILcvQneGk/Iy9oDJeO+BNv4r 6PQMACwYyg+PzPWeGxRByvl2nQFcVPZ7knCw0Y4OJl8ia2u1WDkajLvZIqzg7cJMUGWf Ub4/r+XtTN3pZvfzMURm9/8qE7DhOCh7GFglMYzNCk2EnWvw/EK1MIc3r31zAuU9GzFj P5jA== X-Gm-Message-State: AC+VfDzBRAdVwsM6DCy3V30aOvh+sjIpIqXv4OgZ5P5A5BPcj/JmSxCf WrBjCmetmd1rMguDfdMAZFConX6Bpj4= X-Google-Smtp-Source: ACHHUZ5TQwmANF6neAknP5H+AWyJqbwoNg/jyFn+m7PYNFR6ihuE1bqV3XdLZciPiRJQBaskEwaQzw== X-Received: by 2002:a05:6402:2cd:b0:518:72b8:1fe9 with SMTP id b13-20020a05640202cd00b0051872b81fe9mr6804650edx.25.1687187404670; Mon, 19 Jun 2023 08:10:04 -0700 (PDT) Received: from localhost.localdomain (ip5f5abbd5.dynamic.kabel-deutschland.de. [95.90.187.213]) by smtp.gmail.com with ESMTPSA id p6-20020a056402044600b0050488d1d376sm13297186edw.0.2023.06.19.08.10.03 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 19 Jun 2023 08:10:04 -0700 (PDT) From: Michael Straube To: gregkh@linuxfoundation.org Cc: philipp.g.hortmann@gmail.com, linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org, Michael Straube Subject: [PATCH 1/4] staging: rtl8192e: clean up brace coding style issues Date: Mon, 19 Jun 2023 17:09:50 +0200 Message-ID: <20230619150953.22484-2-straube.linux@gmail.com> X-Mailer: git-send-email 2.41.0 In-Reply-To: <20230619150953.22484-1-straube.linux@gmail.com> References: <20230619150953.22484-1-straube.linux@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" Clean up brace coding style in if/else statements to improve readability and clear checkpatch issues. CHECK: braces {} should be used on all arms of this statement CHECK: Unbalanced braces around else statement Signed-off-by: Michael Straube Tested-by: Philipp Hortmann --- .../staging/rtl8192e/rtl8192e/r8192E_dev.c | 3 ++- .../staging/rtl8192e/rtl8192e/r8192E_phy.c | 10 +++++--- drivers/staging/rtl8192e/rtl8192e/rtl_dm.c | 3 ++- drivers/staging/rtl8192e/rtl8192e/rtl_ps.c | 3 ++- drivers/staging/rtl8192e/rtl8192e/rtl_wx.c | 4 +-- drivers/staging/rtl8192e/rtllib_softmac.c | 25 +++++++++++-------- drivers/staging/rtl8192e/rtllib_softmac_wx.c | 4 +-- 7 files changed, 30 insertions(+), 22 deletions(-) diff --git a/drivers/staging/rtl8192e/rtl8192e/r8192E_dev.c b/drivers/stagi= ng/rtl8192e/rtl8192e/r8192E_dev.c index 7683853e7c46..e5925899402c 100644 --- a/drivers/staging/rtl8192e/rtl8192e/r8192E_dev.c +++ b/drivers/staging/rtl8192e/rtl8192e/r8192E_dev.c @@ -765,8 +765,9 @@ void rtl92e_link_change(struct net_device *dev) ; else priv->receive_config =3D reg |=3D RCR_CBSSID; - } else + } else { priv->receive_config =3D reg &=3D ~RCR_CBSSID; + } =20 rtl92e_writel(dev, RCR, reg); } diff --git a/drivers/staging/rtl8192e/rtl8192e/r8192E_phy.c b/drivers/stagi= ng/rtl8192e/rtl8192e/r8192E_phy.c index 7b0da55fa7aa..24180670ea0b 100644 --- a/drivers/staging/rtl8192e/rtl8192e/r8192E_phy.c +++ b/drivers/staging/rtl8192e/rtl8192e/r8192E_phy.c @@ -39,8 +39,9 @@ void rtl92e_set_bb_reg(struct net_device *dev, u32 dwRegA= ddr, u32 dwBitMask, BitShift =3D _rtl92e_calculate_bit_shift(dwBitMask); NewValue =3D (OriginalValue & ~dwBitMask) | (dwData << BitShift); rtl92e_writel(dev, dwRegAddr, NewValue); - } else + } else { rtl92e_writel(dev, dwRegAddr, dwData); + } } =20 u32 rtl92e_get_bb_reg(struct net_device *dev, u32 dwRegAddr, u32 dwBitMask) @@ -163,10 +164,10 @@ void rtl92e_set_rf_reg(struct net_device *dev, enum r= f90_radio_path eRFPath, =20 _rtl92e_phy_rf_fw_write(dev, eRFPath, RegAddr, New_Value); - } else + } else { _rtl92e_phy_rf_fw_write(dev, eRFPath, RegAddr, Data); + } udelay(200); - } else { if (BitMask !=3D bMask12Bits) { Original_Value =3D _rtl92e_phy_rf_read(dev, eRFPath, @@ -175,8 +176,9 @@ void rtl92e_set_rf_reg(struct net_device *dev, enum rf9= 0_radio_path eRFPath, New_Value =3D (Original_Value & ~BitMask) | (Data << BitShift); =20 _rtl92e_phy_rf_write(dev, eRFPath, RegAddr, New_Value); - } else + } else { _rtl92e_phy_rf_write(dev, eRFPath, RegAddr, Data); + } } } =20 diff --git a/drivers/staging/rtl8192e/rtl8192e/rtl_dm.c b/drivers/staging/r= tl8192e/rtl8192e/rtl_dm.c index adb6f12c89d8..37c275cac40b 100644 --- a/drivers/staging/rtl8192e/rtl8192e/rtl_dm.c +++ b/drivers/staging/rtl8192e/rtl8192e/rtl_dm.c @@ -622,8 +622,9 @@ static void _rtl92e_dm_tx_power_tracking_callback_tssi(= struct net_device *dev) } else if (priv->rtllib->current_network.channel !=3D 14 && priv->bcck= _in_ch14) { priv->bcck_in_ch14 =3D false; rtl92e_dm_cck_txpower_adjust(dev, priv->bcck_in_ch14); - } else + } else { rtl92e_dm_cck_txpower_adjust(dev, priv->bcck_in_ch14); + } } =20 if (priv->cck_present_attn_diff <=3D -12 || diff --git a/drivers/staging/rtl8192e/rtl8192e/rtl_ps.c b/drivers/staging/r= tl8192e/rtl8192e/rtl_ps.c index 5ecade840d75..1c49d5da68eb 100644 --- a/drivers/staging/rtl8192e/rtl8192e/rtl_ps.c +++ b/drivers/staging/rtl8192e/rtl8192e/rtl_ps.c @@ -219,8 +219,9 @@ void rtl92e_leisure_ps_enter(struct net_device *dev) =20 if (priv->rtllib->ps =3D=3D RTLLIB_PS_DISABLED) _rtl92e_ps_set_mode(dev, RTLLIB_PS_MBCAST | RTLLIB_PS_UNICAST); - } else + } else { psc->LpsIdleCount++; + } } } =20 diff --git a/drivers/staging/rtl8192e/rtl8192e/rtl_wx.c b/drivers/staging/r= tl8192e/rtl8192e/rtl_wx.c index 4436442b56dd..b680757cfcf6 100644 --- a/drivers/staging/rtl8192e/rtl8192e/rtl_wx.c +++ b/drivers/staging/rtl8192e/rtl8192e/rtl_wx.c @@ -568,9 +568,9 @@ static int _rtl92e_wx_set_frag(struct net_device *dev, if (priv->hw_radio_off) return 0; =20 - if (wrqu->frag.disabled) + if (wrqu->frag.disabled) { priv->rtllib->fts =3D DEFAULT_FRAG_THRESHOLD; - else { + } else { if (wrqu->frag.value < MIN_FRAG_THRESHOLD || wrqu->frag.value > MAX_FRAG_THRESHOLD) return -EINVAL; diff --git a/drivers/staging/rtl8192e/rtllib_softmac.c b/drivers/staging/rt= l8192e/rtllib_softmac.c index 7494c7e6fff6..a47614c837ce 100644 --- a/drivers/staging/rtl8192e/rtllib_softmac.c +++ b/drivers/staging/rtl8192e/rtllib_softmac.c @@ -809,8 +809,9 @@ static struct sk_buff *rtllib_probe_resp(struct rtllib_= device *ieee, erpinfo_content =3D 0; if (ieee->current_network.buseprotection) erpinfo_content |=3D ERP_UseProtection; - } else + } else { erp_len =3D 0; + } =20 crypt =3D ieee->crypt_info.crypt[ieee->crypt_info.tx_keyidx]; encrypt =3D ieee->host_encrypt && crypt && crypt->ops && @@ -1383,9 +1384,9 @@ static void rtllib_associate_step1(struct rtllib_devi= ce *ieee, u8 *daddr) =20 skb =3D rtllib_authentication_req(beacon, ieee, 0, daddr); =20 - if (!skb) + if (!skb) { rtllib_associate_abort(ieee); - else { + } else { ieee->link_state =3D RTLLIB_ASSOCIATING_AUTHENTICATING; netdev_dbg(ieee->dev, "Sending authentication request\n"); softmac_mgmt_xmit(skb, ieee); @@ -1408,9 +1409,9 @@ static void rtllib_auth_challenge(struct rtllib_devic= e *ieee, u8 *challenge, =20 skb =3D rtllib_authentication_req(beacon, ieee, chlen + 2, beacon->bssid); =20 - if (!skb) + if (!skb) { rtllib_associate_abort(ieee); - else { + } else { c =3D skb_put(skb, chlen + 2); *(c++) =3D MFIE_TYPE_CHALLENGE; *(c++) =3D chlen; @@ -1439,9 +1440,9 @@ static void rtllib_associate_step2(struct rtllib_devi= ce *ieee) =20 ieee->softmac_stats.tx_ass_rq++; skb =3D rtllib_association_req(beacon, ieee); - if (!skb) + if (!skb) { rtllib_associate_abort(ieee); - else { + } else { softmac_mgmt_xmit(skb, ieee); mod_timer(&ieee->associate_timer, jiffies + (HZ / 2)); } @@ -1583,11 +1584,12 @@ inline void rtllib_softmac_new_net(struct rtllib_de= vice *ieee, net->ssid_len =3D net->hidden_ssid_len; ssidbroad =3D 1; } - } else + } else { ssidmatch =3D (ieee->current_network.ssid_len =3D=3D net->ssid_len) && (!strncmp(ieee->current_network.ssid, net->ssid, net->ssid_len)); + } =20 /* if the user set the AP check if match. * if the network does not broadcast essid we check the @@ -2536,8 +2538,9 @@ static void rtllib_start_ibss_wq(void *data) ieee->current_network.rates[3] =3D RTLLIB_BASIC_RATE_MASK | RTLLIB_CCK_RATE_11MB; =20 - } else + } else { ieee->current_network.rates_len =3D 0; + } =20 if (ieee->modulation & RTLLIB_OFDM_MODULATION) { ieee->current_network.rates_ex_len =3D 8; @@ -3119,10 +3122,10 @@ void notify_wx_assoc_event(struct rtllib_device *ie= ee) return; =20 wrqu.ap_addr.sa_family =3D ARPHRD_ETHER; - if (ieee->link_state =3D=3D MAC80211_LINKED) + if (ieee->link_state =3D=3D MAC80211_LINKED) { memcpy(wrqu.ap_addr.sa_data, ieee->current_network.bssid, ETH_ALEN); - else { + } else { netdev_info(ieee->dev, "%s(): Tell user space disconnected\n", __func__); eth_zero_addr(wrqu.ap_addr.sa_data); diff --git a/drivers/staging/rtl8192e/rtllib_softmac_wx.c b/drivers/staging= /rtl8192e/rtllib_softmac_wx.c index be82f0a655e8..783948fd67ca 100644 --- a/drivers/staging/rtl8192e/rtllib_softmac_wx.c +++ b/drivers/staging/rtl8192e/rtllib_softmac_wx.c @@ -231,9 +231,9 @@ int rtllib_wx_set_rts(struct rtllib_device *ieee, struct iw_request_info *info, union iwreq_data *wrqu, char *extra) { - if (wrqu->rts.disabled || !wrqu->rts.fixed) + if (wrqu->rts.disabled || !wrqu->rts.fixed) { ieee->rts =3D DEFAULT_RTS_THRESHOLD; - else { + } else { if (wrqu->rts.value < MIN_RTS_THRESHOLD || wrqu->rts.value > MAX_RTS_THRESHOLD) return -EINVAL; --=20 2.41.0 From nobody Sun Feb 8 13:09:04 2026 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id DAB3FEB64DB for ; Mon, 19 Jun 2023 15:10:25 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230496AbjFSPKY (ORCPT ); Mon, 19 Jun 2023 11:10:24 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:57576 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229806AbjFSPKI (ORCPT ); Mon, 19 Jun 2023 11:10:08 -0400 Received: from mail-lf1-x132.google.com (mail-lf1-x132.google.com [IPv6:2a00:1450:4864:20::132]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 6F6D59B for ; Mon, 19 Jun 2023 08:10:07 -0700 (PDT) Received: by mail-lf1-x132.google.com with SMTP id 2adb3069b0e04-4f866a3d8e4so2630480e87.0 for ; Mon, 19 Jun 2023 08:10:07 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20221208; t=1687187406; x=1689779406; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=KEaQN5x4Wxywu5DRGerlvVJNwGjReSSnSGgqsLpGfq8=; b=RPt8uIPW6Z1I8YsH+/4gGjf0coJJvCyrShdrTfreq02XGa44yMc4IRgwBtAIVfG7Sl qwMmN7FMMFw9wCZGht7sIWTqD9T+cFXqNqLVfiGkZcbsmFAC4eNxzTfJYrJoTADVa+cQ B9RdCqbWEDDEBrolg3fidkt+ooGT5anjwastHqnb6Eubr8Ff51O6/uuQ7FfSiiynmVHf +Azn0Bm82yG/+vPWJUtget0XUxX22Yt7lkfCUn5rCtd8CwBxfH4G4bEJZupfcXZZTFYM EUXWFnME/0fehi4NmXwkML8S63PbG3kKJxxvJ7hYnoSYHT8vCLzamjfXTfiuCXNm/opX JcwQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1687187406; x=1689779406; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=KEaQN5x4Wxywu5DRGerlvVJNwGjReSSnSGgqsLpGfq8=; b=iHQiC/jv2S2O4lIL8WQobi9DTNRie7UBPwXpKxM4d5GCT15K1CWXAJbBGaCGSxm7ru MtyNmKhIp0wbEe985LkkcX8dV1s2b4xBETymzLYCKShsF9h4r0DPbsn07UOaUjaRuuWu gf+/KpS5XuBg2xyG797J5voHE7WrIf+P9VKafd9dgzm88e6t0cyUx4KYk7x+BJBzM0B8 T6hY99bmmdIObcXX8r5daajN9MEsSuIg2Mh5tpL5REd3AougqaojmafWPRWRauCrW1n5 cLfa3BWuQFYP5z6AHe/HR7dj8Qj0g/1f3G4mYmrJyW73rREMqZAaq04nD8OnkvC+LAxj MstQ== X-Gm-Message-State: AC+VfDxecQCPo/vZuXTUWeMLXl4li/WXBMQaOtAM440Bb3mhPiH31E2Y O1co+LtxB+XkKyaV7h4m6J5p9T6hPGg= X-Google-Smtp-Source: ACHHUZ4aQi79GTD852ryOqtRfHZUKyif4wukn1JIVkmcg7rnOJKoGWfxHwWWuUqgnv4t3SLFyR6k5A== X-Received: by 2002:a05:6512:47a:b0:4f8:7494:17d7 with SMTP id x26-20020a056512047a00b004f8749417d7mr1260966lfd.59.1687187405320; Mon, 19 Jun 2023 08:10:05 -0700 (PDT) Received: from localhost.localdomain (ip5f5abbd5.dynamic.kabel-deutschland.de. [95.90.187.213]) by smtp.gmail.com with ESMTPSA id p6-20020a056402044600b0050488d1d376sm13297186edw.0.2023.06.19.08.10.04 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 19 Jun 2023 08:10:05 -0700 (PDT) From: Michael Straube To: gregkh@linuxfoundation.org Cc: philipp.g.hortmann@gmail.com, linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org, Michael Straube Subject: [PATCH 2/4] staging: rtl8192e: convert else if sequence to switch Date: Mon, 19 Jun 2023 17:09:51 +0200 Message-ID: <20230619150953.22484-3-straube.linux@gmail.com> X-Mailer: git-send-email 2.41.0 In-Reply-To: <20230619150953.22484-1-straube.linux@gmail.com> References: <20230619150953.22484-1-straube.linux@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" Convert a sequence of else if statements that all check the same variable to a single switch statement. With a switch statement it is easier to see what is going on. Additionally this clears a checkpatch warning. WARNING: braces {} are not necessary for any arm of this statement Signed-off-by: Michael Straube Tested-by: Philipp Hortmann --- drivers/staging/rtl8192e/rtllib_softmac.c | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/drivers/staging/rtl8192e/rtllib_softmac.c b/drivers/staging/rt= l8192e/rtllib_softmac.c index a47614c837ce..7b3cbe28e562 100644 --- a/drivers/staging/rtl8192e/rtllib_softmac.c +++ b/drivers/staging/rtl8192e/rtllib_softmac.c @@ -2830,15 +2830,19 @@ void rtllib_start_protocol(struct rtllib_device *ie= ee) * attempts does not fail just because the user provide the essid * and the nic is still checking for the AP MAC ?? */ - if (ieee->iw_mode =3D=3D IW_MODE_INFRA) { + switch (ieee->iw_mode) { + case IW_MODE_INFRA: rtllib_start_bss(ieee); - } else if (ieee->iw_mode =3D=3D IW_MODE_ADHOC) { + break; + case IW_MODE_ADHOC: rtllib_start_ibss(ieee); - - } else if (ieee->iw_mode =3D=3D IW_MODE_MASTER) { + break; + case IW_MODE_MASTER: rtllib_start_master_bss(ieee); - } else if (ieee->iw_mode =3D=3D IW_MODE_MONITOR) { + break; + case IW_MODE_MONITOR: rtllib_start_monitor_mode(ieee); + break; } } =20 --=20 2.41.0 From nobody Sun Feb 8 13:09:04 2026 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 6F97FEB64DA for ; Mon, 19 Jun 2023 15:10:27 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230511AbjFSPK0 (ORCPT ); Mon, 19 Jun 2023 11:10:26 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:57610 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229808AbjFSPKI (ORCPT ); Mon, 19 Jun 2023 11:10:08 -0400 Received: from mail-ed1-x531.google.com (mail-ed1-x531.google.com [IPv6:2a00:1450:4864:20::531]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 9E5C3AD for ; Mon, 19 Jun 2023 08:10:07 -0700 (PDT) Received: by mail-ed1-x531.google.com with SMTP id 4fb4d7f45d1cf-5187aa18410so3682543a12.0 for ; Mon, 19 Jun 2023 08:10:07 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20221208; t=1687187406; x=1689779406; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=snu+S5y4+SEbvZxuNqaJKxNoj771q1fsuo3zQR1oJlc=; b=XW/t5IoD5oqI+Vp+Gl3zbLsWvZdsIRxACNRjO6CSI8qrSw6uMY8OMl2ObU5ogfm8Jd kFPxcrhVvDpKqg8Oc4lHvP+UnbhL7iMvu7VNoj2QJcOF1J8OqILf24OKTsaKbt6HdcUA gBqWMDIusLDXq+qIhvHVgJPXrnxYmPHnZ4+p8JXaYIbZyxk4x9SgYUHiRU1RmujvhLRA Rcz9+E6LWceMrPFUru4iqnVJaokucWJzy6FgsiA0EdCcvklgV8J0k1BN2NTFYN+F6ydu zVLdWd4VL7wpKuSAuf7T3/xv+SXkLVSNEt2DFMtVIowRlb/JkhWhKGbG7+hTqX60zLOq QtAQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1687187406; x=1689779406; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=snu+S5y4+SEbvZxuNqaJKxNoj771q1fsuo3zQR1oJlc=; b=NVdavCN84Fd3p28SQvtX+MsTH6p7Ujbc0MgoCic/tQBHsGo8Yv5JSIGJH3M8svybxn 3A/B/lakrIuN5/BdF6r3y/sRc7mGRsMpyC84Elx55Ach2SXk10CHv9xHnuvPijjXEKoi xx7b9G7xHlFRlwWonmZUuK5xLD9RRjPYrMpYU5XfqPyD+0BIYSqwVb4L4yd0K297+Icl 2hgpiKitEpbz3k+heB6fgj9qkbHWCtxZMgnf/ZorubYGjgczgeUpUk+2afo3srYt4iV9 K2/qHFwwPTKuly1eQE5MOajh/52ENgkxChTKdwIqqpwLvT9u3wkbOlXGka9CLFAfK3kD zCLw== X-Gm-Message-State: AC+VfDzUHS6JuzGNlV0b4IuxYz/A5Z/1bttc26mXI0eCRzr0j7paljzQ 67cpOI4VM8uHL3xMis0Fji9ewe19IKo= X-Google-Smtp-Source: ACHHUZ6p1rzi/cv+fMtaGRWBi/CZoN77pk2/R0btD0J8YIKBkR21JIJ9whPigKMIpHXJff9kZtRl2A== X-Received: by 2002:aa7:db89:0:b0:51a:265a:8fca with SMTP id u9-20020aa7db89000000b0051a265a8fcamr5143511edt.27.1687187405909; Mon, 19 Jun 2023 08:10:05 -0700 (PDT) Received: from localhost.localdomain (ip5f5abbd5.dynamic.kabel-deutschland.de. [95.90.187.213]) by smtp.gmail.com with ESMTPSA id p6-20020a056402044600b0050488d1d376sm13297186edw.0.2023.06.19.08.10.05 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 19 Jun 2023 08:10:05 -0700 (PDT) From: Michael Straube To: gregkh@linuxfoundation.org Cc: philipp.g.hortmann@gmail.com, linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org, Michael Straube Subject: [PATCH 3/4] staging: rtl8192e: remove return statement from void function Date: Mon, 19 Jun 2023 17:09:52 +0200 Message-ID: <20230619150953.22484-4-straube.linux@gmail.com> X-Mailer: git-send-email 2.41.0 In-Reply-To: <20230619150953.22484-1-straube.linux@gmail.com> References: <20230619150953.22484-1-straube.linux@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" Remove unnecessary return statement from the void function rtl92e_config_mac(). Issue found by checkpatch. WARNING: void function return statements are not generally useful Signed-off-by: Michael Straube Tested-by: Philipp Hortmann --- drivers/staging/rtl8192e/rtl8192e/r8192E_phy.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/rtl8192e/rtl8192e/r8192E_phy.c b/drivers/stagi= ng/rtl8192e/rtl8192e/r8192E_phy.c index 24180670ea0b..b0857c5709dd 100644 --- a/drivers/staging/rtl8192e/rtl8192e/r8192E_phy.c +++ b/drivers/staging/rtl8192e/rtl8192e/r8192E_phy.c @@ -268,7 +268,6 @@ void rtl92e_config_mac(struct net_device *dev) rtl92e_set_bb_reg(dev, pdwArray[i], pdwArray[i + 1], pdwArray[i + 2]); } - return; } =20 static void _rtl92e_phy_config_bb(struct net_device *dev, u8 ConfigType) --=20 2.41.0 From nobody Sun Feb 8 13:09:04 2026 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 8480DEB64D9 for ; Mon, 19 Jun 2023 15:10:30 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231215AbjFSPK3 (ORCPT ); Mon, 19 Jun 2023 11:10:29 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:56858 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229824AbjFSPKJ (ORCPT ); Mon, 19 Jun 2023 11:10:09 -0400 Received: from mail-ed1-x532.google.com (mail-ed1-x532.google.com [IPv6:2a00:1450:4864:20::532]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 1FA6FCF for ; Mon, 19 Jun 2023 08:10:08 -0700 (PDT) Received: by mail-ed1-x532.google.com with SMTP id 4fb4d7f45d1cf-519608ddbf7so3694011a12.2 for ; Mon, 19 Jun 2023 08:10:08 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20221208; t=1687187406; x=1689779406; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=09Nls6tXLgOvm0ng8/7m8dRNZ/h0ThOPEUKr3YHTZzs=; b=pme+MsdXK1u77Qcjve1hUQzq86C1M6dYAPy1m3hfdiAtb72eyv2TOoLP6TWC9DPPSg me1SSIX9+gs9DZl/C0tp0aap+CN0oV07PPnIZNPnqcbjhzEkZqekyXN/me6HPs70P0z8 hwDeeCACZLSSq0s1dErFwVZo6S5ESwUrlFxobMlVtoX8wPqdB4pQVHPCBHhWJ9KaPDSU fN1MxkXkZY6gMQvzxetHaBMvQY3SjmkQ0uulHc2Ht1J3f8sN67MuTaweNkXBM7GfB2AY pRFJKzNVAEzdL5akaO0o8yL49L/CV/+D6XJxH7+BFAy4Y4dYhXL+52aNZacdCfn9vR2o 2opA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1687187406; x=1689779406; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=09Nls6tXLgOvm0ng8/7m8dRNZ/h0ThOPEUKr3YHTZzs=; b=ZMlODmeC7C8zbVdoMTK9zb1NIJ2tjW4za0pyYHnZJnHqjladFl5L2OJEVAPtqKHXMS GT6RbKgN8kh9Ra8wDHJldx74DrMvC9sifM4vZG76KIrr60MB1DgSYCUYQRZAkR1qhzyT Z+zXPhL6eL4IjX084PkVr3d1HgnJVLNhAKAtS+VFMrbRD8fWuOOeZh4eZ8DOuUUnLczl lKipi75Nwq4kaHlLkk8EKpiy/gFBb+AGY2qeSail6eu3bNRdiyCzCgIFRFC/zxe8UlA8 vq4v6ooJ8ZRLjeWfLtk+1JMi/DNQ/xougyiNNjjTL0v1I3YGItzfV6uX+P3c7u7N6GRg Dkpg== X-Gm-Message-State: AC+VfDz7Q4qysn7790DLdakc/FZqXZbveEllUR/h0B47LgK40bY6g94W XT3ql36Sn4UxX0LPTFNKUhM= X-Google-Smtp-Source: ACHHUZ69hXtQAYUmB8YoITV8a1MdHg2JXDtk41CV4kQ7vR5XjOQLePMVMM/AeZQdilYcUSqBH2RLVw== X-Received: by 2002:aa7:d0c8:0:b0:51a:4a25:bfbd with SMTP id u8-20020aa7d0c8000000b0051a4a25bfbdmr3392543edo.36.1687187406486; Mon, 19 Jun 2023 08:10:06 -0700 (PDT) Received: from localhost.localdomain (ip5f5abbd5.dynamic.kabel-deutschland.de. [95.90.187.213]) by smtp.gmail.com with ESMTPSA id p6-20020a056402044600b0050488d1d376sm13297186edw.0.2023.06.19.08.10.06 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 19 Jun 2023 08:10:06 -0700 (PDT) From: Michael Straube To: gregkh@linuxfoundation.org Cc: philipp.g.hortmann@gmail.com, linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org, Michael Straube Subject: [PATCH 4/4] staging: rtl8192e: remove comparison to true Date: Mon, 19 Jun 2023 17:09:53 +0200 Message-ID: <20230619150953.22484-5-straube.linux@gmail.com> X-Mailer: git-send-email 2.41.0 In-Reply-To: <20230619150953.22484-1-straube.linux@gmail.com> References: <20230619150953.22484-1-straube.linux@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" Remove a comparison to true reported by checkpatch. CHECK: Using comparison to true is error prone Signed-off-by: Michael Straube Tested-by: Philipp Hortmann --- drivers/staging/rtl8192e/rtl8192e/rtl_wx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8192e/rtl8192e/rtl_wx.c b/drivers/staging/r= tl8192e/rtl8192e/rtl_wx.c index b680757cfcf6..88975dc804c6 100644 --- a/drivers/staging/rtl8192e/rtl8192e/rtl_wx.c +++ b/drivers/staging/rtl8192e/rtl8192e/rtl_wx.c @@ -391,7 +391,7 @@ static int _rtl92e_wx_set_scan(struct net_device *dev, rt_state =3D priv->rtllib->rf_power_state; if (!priv->up) return -ENETDOWN; - if (priv->rtllib->link_detect_info.bBusyTraffic =3D=3D true) + if (priv->rtllib->link_detect_info.bBusyTraffic) return -EAGAIN; =20 if (wrqu->data.flags & IW_SCAN_THIS_ESSID) { --=20 2.41.0