From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-lf1-f51.google.com (mail-lf1-f51.google.com [209.85.167.51]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 3E5DC1F30A9 for ; Sun, 12 Jul 2026 15:34:34 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.167.51 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1783870475; cv=none; b=L11IX37lreFKjxSfUXcWL7dN+Yy8vgaoXWHvHa67gRT7roE2UtXFm9yH0m7uejPRrSoUpkiXy6JuCb+BVRxMzqwksfEJC1CKUFebWSPbr5ZlC/raDOAMUU/PvdCvDD4MG1WPxzHECDFPs4UwHCzYdtef1gyFRVrRSTT24TxqdbM= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1783870475; c=relaxed/simple; bh=NrkEMKv/OjVK8Al5t2JOrX+5lNBTCMuIpTfnJq0FZKg=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=eNupX+51Aao17nGwY0FBy1umWK3g0vAbxk4vI4+86D8K+NrwqaffTaHExYbxIaoSJ+UzSDh6ZTzutMXoQusX1Ss+/AaNos1dj/O0pQbLJr5bcPTMC4jKn/M15FDvoy1pfR9BgtrhCl7FtBSxdhMeT1zhr4+0RdScX60hTMdh/+M= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=Rt8w0kxh; arc=none smtp.client-ip=209.85.167.51 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="Rt8w0kxh" Received: by mail-lf1-f51.google.com with SMTP id 2adb3069b0e04-5b015b2d792so2609893e87.3 for ; Sun, 12 Jul 2026 08:34:34 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1783870472; x=1784475272; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to:content-type; bh=OQ1k+7+OKgDOPXNXeWWLucG/wIn4+UMOi+5rR6YEEIk=; b=Rt8w0kxh11P6yvBSzStsiXNEjLaPgYIG3qMxQQRG/BkD/M5pJW8UmRddl2SNvQQAts zxtsXZX9xsolwI2BKQ9CHiAV09sxM0J79nGf4Of7tHXA5tStDrsmQ0fgQen4d0/7BbRR kL+ZorPxyasfuqZJlsZPaBOFkNVwjI0Lv1hrE0uKKWJTH16u9X5UfdWTK/mrHkxxTGEY Qk6rjnDUE78ZNAl2vbUKXCWTzntDIcn1yKAxQkeSbfIA5CHERwLZVf5YGINoOsqNymMK 6belur7n5/2860/jYgnFRezyLxVN9JvnOVPYGZewbnMamdwN4O9f6xXNg354bv6xjVy9 ngEg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1783870472; x=1784475272; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to:content-type; bh=OQ1k+7+OKgDOPXNXeWWLucG/wIn4+UMOi+5rR6YEEIk=; b=Z5bxzne0k0WDIt8HUOGhrqC9UIk1VzqPDhSm7/NCTlNi3ukZDI0zZkuxsGkYzinAwL czyGakL2U2gfaD0E4rxz4qvelZBYhOCC1J4w6a6hkdsyy32mb3g27eW0f/ptGh8cNF92 3hxWg5URUqEq4rD/VfGR5mLI7GV5L2yx3nTk3R7Hdgk5QjQ/13f7LxPauOTblWT95RyL g0/nQ/xw0yrmKLpcFSvqB4WwC8zqXesNT/5rvfd+5jbzIkZKyGwbbC/aUnt7G/eVISZL 8gREgreZofgb/CMNwQKyjpHS+XXwVs97DdQSBUQeB/E3DSPTm09EOgqbTdENQXXwnn5H Fssg== X-Forwarded-Encrypted: i=1; AHgh+Rrj61uKNgVCKbCp3wPEllSQIFuBckt56eumSLn6X+fUCroKajQe6rJxOmCgCeBHJhG9pqPCc+Ftk3MO27c=@vger.kernel.org X-Gm-Message-State: AOJu0YwQqwDzifq2Q6V8NVLiuREmvm1+O9YanZ4t+X4pcQYh3vuWY12K 6gVffnVKwx0H7c5tb2UUV8we2Mq8SEGfqwLuCGmLJ2GpY7+S66UONmCDrAaK3TayVFA= X-Gm-Gg: AfdE7clCn7gFVVtNB7riSAmxyejWJ95WL6vaueUC5I6gGleeXkgz7YI5CeV6nCKS4A1 jMJl0gb0UNpzjEA3tgXAgi8rFarzz/IJY8o4QsUNlnMvPUiJ2TCtfkoYEL+uqhYxw0HcAXGhf3i H6siVhlCIyd0MmuXmXP6LhrvmPwVfmjaKGJIOLhjpTvzENnAZ6W8uqrSgtTDaLhzTjDjwz62LVR T1hrlDUCJaj2mHkBMVOacJ6K0NceqsAFFkcV0mJgd5ErIe+HPEvy9SSyRbeCgr3HILn559niwsM OThonhQBz5vAok21uCR8mvGVWvBDlYhdlqJAbwFLWneIGl29Sq+d/Bj5TRJlJvBBawnbQroJip0 2K/XeRR7MkAyqryHcKPYAh9GjsTSS8R8miNdqeAqY76mTWBAI5CTFoE9P9EnkQ65gcJnnXyOXvF 1AM+mhk3XOjiHRvj2MJJrPHI9xuJa3Racw95kToIhCNhb0ighefu6dOqa0Gnh3BmXDrL+SIXKAp s6qI+SgdyiP6RI= X-Received: by 2002:ac2:48ac:0:b0:5b0:bc3:dc11 with SMTP id 2adb3069b0e04-5b023669296mr1082053e87.22.1783870472267; Sun, 12 Jul 2026 08:34:32 -0700 (PDT) Received: from archlinux ([94.180.100.43]) by smtp.gmail.com with ESMTPSA id 2adb3069b0e04-5b01caa66c0sm2191773e87.63.2026.07.12.08.34.30 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 12 Jul 2026 08:34:31 -0700 (PDT) From: Arsenii Pashchenko To: gregkh@linuxfoundation.org Cc: linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org, ulijg308@gmail.com Subject: [PATCH] staging: rtl8723bs: remove unnecessary braces Date: Sun, 12 Jul 2026 22:34:23 +0700 Message-ID: <20260712153423.130472-1-ulijg308@gmail.com> X-Mailer: git-send-email 2.55.0 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Remove redundant curly braces from single-statement blocks in rtl8723b_hal_init.c to adhere to the Linux kernel coding style. This resolves warnings reported by checkpatch.pl. Signed-off-by: Arsenii Pashchenko --- .../staging/rtl8723bs/hal/rtl8723b_hal_init.c | 51 +++++++++---------- 1 file changed, 25 insertions(+), 26 deletions(-) diff --git a/drivers/staging/rtl8723bs/hal/rtl8723b_hal_init.c b/drivers/staging/rtl8723bs/hal/rtl8723b_hal_init.c index a364f4507..7862cea83 100644 --- a/drivers/staging/rtl8723bs/hal/rtl8723b_hal_init.c +++ b/drivers/staging/rtl8723bs/hal/rtl8723b_hal_init.c @@ -194,9 +194,9 @@ static s32 polling_fwdl_chksum( yield(); } while (jiffies_to_msecs(jiffies-start) < timeout_ms || cnt < min_cnt); - if (!(value32 & FWDL_ChkSum_rpt)) { + if (!(value32 & FWDL_ChkSum_rpt)) goto exit; - } + if (g_fwdl_chksum_fail) { g_fwdl_chksum_fail--; @@ -235,9 +235,9 @@ static s32 _FWFreeToGo(struct adapter *adapter, u32 min_cnt, u32 timeout_ms) yield(); } while (jiffies_to_msecs(jiffies - start) < timeout_ms || cnt < min_cnt); - if (!(value32 & WINTINI_RDY)) { + if (!(value32 & WINTINI_RDY)) goto exit; - } + if (g_fwdl_wintint_rdy_fail) { g_fwdl_wintint_rdy_fail--; @@ -1031,9 +1031,9 @@ void UpdateHalRAMask8723B(struct adapter *padapter, u32 mac_id, u8 rssi_level) rate_bitmap = hal_btcoex_GetRaMask(padapter); mask &= ~rate_bitmap; - if (pHalData->fw_ractrl) { + if (pHalData->fw_ractrl) rtl8723b_set_FwMacIdConfig_cmd(padapter, mac_id, psta->raid, psta->bw_mode, short_gi_rate, mask); - } + /* set correct initial date rate for each mac_id */ pdmpriv->INIDATA_RATE[mac_id] = psta->init_rate; @@ -1184,9 +1184,9 @@ void Hal_EfuseParseIDCode(struct adapter *padapter, u8 *hwinfo) /* Check 0x8129 again for making sure autoload status!! */ EEPROMId = le16_to_cpu(*((__le16 *)hwinfo)); - if (EEPROMId != RTL_EEPROM_ID) { + if (EEPROMId != RTL_EEPROM_ID) pEEPROM->bautoload_fail_flag = true; - } else + else pEEPROM->bautoload_fail_flag = false; } @@ -1566,13 +1566,13 @@ u8 SCMapping_8723B(struct adapter *Adapter, struct pkt_attrib *pattrib) if (pattrib->bwmode == CHANNEL_WIDTH_40) { SCSettingOfDesc = HT_DATA_SC_DONOT_CARE; } else if (pattrib->bwmode == CHANNEL_WIDTH_20) { - if (pHalData->nCur40MhzPrimeSC == HAL_PRIME_CHNL_OFFSET_UPPER) { + if (pHalData->nCur40MhzPrimeSC == HAL_PRIME_CHNL_OFFSET_UPPER) SCSettingOfDesc = HT_DATA_SC_20_UPPER_OF_40MHZ; - } else if (pHalData->nCur40MhzPrimeSC == HAL_PRIME_CHNL_OFFSET_LOWER) { + else if (pHalData->nCur40MhzPrimeSC == HAL_PRIME_CHNL_OFFSET_LOWER) SCSettingOfDesc = HT_DATA_SC_20_LOWER_OF_40MHZ; - } else { + else SCSettingOfDesc = HT_DATA_SC_DONOT_CARE; - } + } } else { SCSettingOfDesc = HT_DATA_SC_DONOT_CARE; @@ -1596,9 +1596,9 @@ static void rtl8723b_cal_txdesc_chksum(struct tx_desc *ptxdesc) /* Thomas, Lucas@SD4, 20130515 */ count = 16; - for (index = 0; index < count; index++) { + for (index = 0; index < count; index++) checksum |= le16_to_cpu(*(__le16 *)(usPtr + index)); - } + ptxdesc->txdw7 |= cpu_to_le32(checksum & 0x0000ffff); } @@ -1772,11 +1772,10 @@ static void rtl8723b_fill_default_txdesc( ptxdesc->mbssid = pattrib->mbssid & 0xF; ptxdesc->rty_lmt_en = 1; /* retry limit enable */ - if (pattrib->retry_ctrl) { + if (pattrib->retry_ctrl) ptxdesc->data_rt_lmt = 6; - } else { + else ptxdesc->data_rt_lmt = 12; - } ptxdesc->datarate = MRateToHwRate(pmlmeext->tx_rate); @@ -1867,9 +1866,9 @@ void rtl8723b_fill_fake_txdesc( SET_TX_DESC_HWSEQ_SEL_8723B(pDesc, 0); } - if (IsBTQosNull) { + if (IsBTQosNull) SET_TX_DESC_BT_INT_8723B(pDesc, 1); - } + SET_TX_DESC_USE_RATE_8723B(pDesc, 1); /* use data rate which is set by Sw */ SET_TX_DESC_OWN_8723B((u8 *)pDesc, 1); @@ -2184,9 +2183,9 @@ void CCX_FwC2HTxRpt_8723b(struct adapter *padapter, u8 *pdata, u8 len) #define GET_8723B_C2H_TX_RPT_LIFE_TIME_OVER(_Header) LE_BITS_TO_1BYTE((_Header + 0), 6, 1) #define GET_8723B_C2H_TX_RPT_RETRY_OVER(_Header) LE_BITS_TO_1BYTE((_Header + 0), 7, 1) - if (GET_8723B_C2H_TX_RPT_RETRY_OVER(pdata) | GET_8723B_C2H_TX_RPT_LIFE_TIME_OVER(pdata)) { + if (GET_8723B_C2H_TX_RPT_RETRY_OVER(pdata) | GET_8723B_C2H_TX_RPT_LIFE_TIME_OVER(pdata)) rtw_ack_tx_done(&padapter->xmitpriv, RTW_SCTX_DONE_CCX_PKT_FAIL); - } + /* else if (seq_no != padapter->xmitpriv.seq_no) { rtw_ack_tx_done(&padapter->xmitpriv, RTW_SCTX_DONE_CCX_PKT_FAIL); @@ -2555,9 +2554,9 @@ void SetHwReg8723B(struct adapter *padapter, u8 variable, u8 *val) /* Forece leave RF low power mode for 1T1R to prevent conficting setting in Fw power */ /* saving sequence. 2010.06.07. Added by tynli. Suggested by SD3 yschang. */ - if (psmode != PS_MODE_ACTIVE) { + if (psmode != PS_MODE_ACTIVE) ODM_RF_Saving(&pHalData->odmpriv, true); - } + /* if (psmode != PS_MODE_ACTIVE) { */ /* rtl8723b_set_lowpwr_lps_cmd(padapter, true); */ @@ -2871,11 +2870,11 @@ u8 GetHalDefVar8723B(struct adapter *padapter, enum hal_def_variable variable, v break; case HAL_DEF_TX_PAGE_BOUNDARY: - if (!padapter->registrypriv.wifi_spec) { + if (!padapter->registrypriv.wifi_spec) *(u8 *)pval = TX_PAGE_BOUNDARY_8723B; - } else { + else *(u8 *)pval = WMM_NORMAL_TX_PAGE_BOUNDARY_8723B; - } + break; case HAL_DEF_MACID_SLEEP: -- 2.55.0