From nobody Wed Apr 1 09:44:32 2026 Received: from mail-lf1-f49.google.com (mail-lf1-f49.google.com [209.85.167.49]) (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 EF9732EACF2 for ; Mon, 30 Mar 2026 21:24:25 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.167.49 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1774905867; cv=none; b=FfHevj/NravNCLVLtdmjOf9veWYopL4PS6jB0mkcelo+iDb1RKhmbjxl2n7Tm/s8TH+dCiUfxYnT7bkfqL6sfHFK7Y2vaN7f473WWtwX3pCn3gJm13WfTYyyN2wJT5aRv0bEPZv8TjcFGGR7tVHmyvKJbbZ7z/xqxw01qNM2WxI= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1774905867; c=relaxed/simple; bh=NC1dDUDHwEvDYMc464MBEttGhB+u9I2Wuw0gcp9Luls=; h=Date:From:To:Subject:Message-ID:MIME-Version:Content-Type: Content-Disposition; b=BQ/0czvmIwaWuHkjCDWexDU3p3n2TYIki5vnj5Igb0oziXka1PocyJ+EJxTafX4gJgWaEJhJB+sh6fJ8vawP0+nyBYRzdaXy0t1aDUwZ5ubk0rmYqnGzuvXPGmPoSSYmcMPDom7TXvzs0xg2qZq/W+/vhOuT+di9s6cxwNKJN50= 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=l6/BljCE; arc=none smtp.client-ip=209.85.167.49 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="l6/BljCE" Received: by mail-lf1-f49.google.com with SMTP id 2adb3069b0e04-5a2853b1f8cso4905746e87.3 for ; Mon, 30 Mar 2026 14:24:25 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1774905864; x=1775510664; darn=vger.kernel.org; h=content-disposition:mime-version:message-id:subject:to:from:date :from:to:cc:subject:date:message-id:reply-to; bh=ygFLLyOp6SIhFrThJg/jYbhK7mG8/wkyRqBEku4dv60=; b=l6/BljCEjLC8y/4AaMOyWLCN1aTfIlTECcHOi/ysTtTkFS3VtsmaNtxw2kOSidp9lO F/TB/IlJuDQ+iCcRKAYEUxLRFeayA5pvSIvlrAWZWgCuQWPQRuPgInD5tESEOLRyiHfr 6HcGDTwAwI43sAODvjmIOZ6UGIZnDegj/tw0jQgRtpttzeuK5fBzwAvtfESO25Q2fkh4 R23pmsYQWCdf5ok1+KcQ6ErdZsWTNf3CU8uWBgqToMXMqToDJ/0XerVoYRf7WJXMw3KT l2teE6sLD/zgn+zk7PV5i/YdpOpYV7bQKmD6Q12r1lWzd/VtLrEW7XrB58QBp8jQgS3q dpiw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1774905864; x=1775510664; h=content-disposition:mime-version:message-id:subject:to:from:date :x-gm-gg:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=ygFLLyOp6SIhFrThJg/jYbhK7mG8/wkyRqBEku4dv60=; b=S6YC90Y6JtkKk0ShnwbkCRrkF2xc/G7NzmsVe7el+UoGMcz7Mby+l6Tr49YoawDWNy Xbb8PswbxM76dZNEyN/gN+QKSUSxdd+BwZahQbsveyI2JqRGTfC3evw3MfMl0XQGyVpi hQs9JHLzXNtJeXcF5Uyu8t2F6HspxlySYLo9mevGlXVwQKf++h4qVqAxJBPi3VKOUbPn 9+tczpjAMPmI55wh+p5CYcf5ub+M27zsKX7Rc+BHXflUhArPYifm5+yphh3Dy3F5SGSe EoEBkPb22ZGyKBK7HINN1lAEIWvWnsC/XejWkJwDVvXZ03WBEPYEWME659t5z/u6I5HL Zfcg== X-Forwarded-Encrypted: i=1; AJvYcCVOwJS1juIQef//ayRPkE2jFO4mfZMS52S+YkZbygfEKa18XfIU1LELl3fUaDU/LK966q5x3FtClzsR26M=@vger.kernel.org X-Gm-Message-State: AOJu0YwM8cqd4N5XEUHObylMyNuviznr84wl3tswH0pJybCG1v526umY kJQNy0Wa/q3A1YC4YGb2au9us8KaePiO2DRkN7xIP3QCa9Ro/TLxeIcwYihrHg== X-Gm-Gg: ATEYQzxJpYZUFWlj/Aeq01oigVl9LBhIHM2ACHxd5Kld0fkP0r6BIteY73CESJCC+Pc m45KVcDkEOXPqayiAbuNcCs1IHCnRjINeMn5wMMNNVM78uNc+F8uMfVLB81WTmbD3wpyQMCNq1A gn+cKyQRTn+G3ktZC629JPZXmZ0+vAEawXOok5miUilphpl/wcnQSOO9t9RprZyhqfOqJriCD6P 6VQwNY9Oy1ABWtf/7fLul6lc1sYFNtUu0igoJcN9yzw/q+s1Tc7m6iRGcaxMl7VkeuuYkMfw2UT FMEE29ocGKCrPD68sQ+21BB7E1GMfH2NI8rkaecznoMR13hxSeQgkpR9/2toiLyoiqf1LlZyidr x6DymQwgDOaUS+1oli5/Ph1zgwaVnEwHfZexzxfoVTI3d8UrhaJ4nVx7R+baEVtFS2HlsQ0tJqg 0ixYa4cZnyRQcF9ALj9XGHJMiSe6GqIxKqoMtCC9sfuq7vfm/EWs9Q9seIV66lfg== X-Received: by 2002:a05:6512:b99:b0:5a2:7a31:9199 with SMTP id 2adb3069b0e04-5a2ab7fa6e1mr6019850e87.14.1774905863751; Mon, 30 Mar 2026 14:24:23 -0700 (PDT) Received: from localhost (c188-150-77-196.bredband.tele2.se. [188.150.77.196]) by smtp.gmail.com with ESMTPSA id 2adb3069b0e04-5a2bce53835sm164e87.63.2026.03.30.14.24.23 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 30 Mar 2026 14:24:23 -0700 (PDT) Date: Mon, 30 Mar 2026 23:24:22 +0200 From: Linus Probert To: Greg Kroah-Hartman , linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org Subject: [PATCH] staging: rtl8723bs: clean up function names Message-ID: Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Renames camel case functions in rtw_efuse.h and also removes two function declarations that were never implemented nor called in the source. Naming problems found using checkpatch.pl tool. The unused functions were discovered while applying the changes. Signed-off-by: Linus Probert --- drivers/staging/rtl8723bs/core/rtw_efuse.c | 34 +++++++++---------- .../staging/rtl8723bs/hal/rtl8723b_hal_init.c | 33 ++++++++++-------- drivers/staging/rtl8723bs/hal/sdio_halinit.c | 2 +- drivers/staging/rtl8723bs/include/rtw_efuse.h | 14 ++++---- 4 files changed, 43 insertions(+), 40 deletions(-) diff --git a/drivers/staging/rtl8723bs/core/rtw_efuse.c b/drivers/staging/r= tl8723bs/core/rtw_efuse.c index 099320e4eb50..7b72b05da489 100644 --- a/drivers/staging/rtl8723bs/core/rtw_efuse.c +++ b/drivers/staging/rtl8723bs/core/rtw_efuse.c @@ -10,7 +10,7 @@ =20 /* 11/16/2008 MH Add description. Get current efuse area enabled word!!. = */ u8 -Efuse_CalculateWordCnts(u8 word_en) +rtw_efuse_calculate_word_counts(u8 word_en) { u8 word_cnts =3D 0; =20 @@ -26,7 +26,7 @@ Efuse_CalculateWordCnts(u8 word_en) } =20 /*------------------------------------------------------------------------= ----- - * Function: EFUSE_Read1Byte + * Function: efuse_read_1_byte * * Overview: Copy from WMAC fot EFUSE read 1 byte. * @@ -42,7 +42,7 @@ Efuse_CalculateWordCnts(u8 word_en) * */ u8 -EFUSE_Read1Byte( +rtw_efuse_read_1_byte( struct adapter *Adapter, u16 Address) { @@ -79,11 +79,11 @@ u16 Address) } else return 0xFF; =20 -} /* EFUSE_Read1Byte */ +} /* efuse_read_1_byte */ =20 /* 11/16/2008 MH Read one byte from real Efuse. */ u8 -efuse_OneByteRead( +rtw_efuse_one_byte_read( struct adapter *padapter, u16 addr, u8 *data) @@ -170,7 +170,7 @@ static void Efuse_ReadAllMap(struct adapter *padapter, = u8 efuseType, u8 *Efuse) * 11/12/2008 MHC Create Version 0. * */ -static void efuse_ShadowRead1Byte(struct adapter *padapter, u16 Offset, u8= *Value) +static void efuse_shadow_read_1_byte(struct adapter *padapter, u16 Offset,= u8 *Value) { struct eeprom_priv *pEEPROM =3D GET_EEPROM_EFUSE_PRIV(padapter); =20 @@ -179,7 +179,7 @@ static void efuse_ShadowRead1Byte(struct adapter *padap= ter, u16 Offset, u8 *Valu } /* EFUSE_ShadowRead1Byte */ =20 /* Read Two Bytes */ -static void efuse_ShadowRead2Byte(struct adapter *padapter, u16 Offset, u1= 6 *Value) +static void efuse_shadow_read_2_byte(struct adapter *padapter, u16 Offset,= u16 *Value) { struct eeprom_priv *pEEPROM =3D GET_EEPROM_EFUSE_PRIV(padapter); =20 @@ -189,7 +189,7 @@ static void efuse_ShadowRead2Byte(struct adapter *padap= ter, u16 Offset, u16 *Val } /* EFUSE_ShadowRead2Byte */ =20 /* Read Four Bytes */ -static void efuse_ShadowRead4Byte(struct adapter *padapter, u16 Offset, u3= 2 *Value) +static void efuse_shadow_read_4_byte(struct adapter *padapter, u16 Offset,= u32 *Value) { struct eeprom_priv *pEEPROM =3D GET_EEPROM_EFUSE_PRIV(padapter); =20 @@ -201,7 +201,7 @@ static void efuse_ShadowRead4Byte(struct adapter *padap= ter, u16 Offset, u32 *Val } /* efuse_ShadowRead4Byte */ =20 /*------------------------------------------------------------------------= ----- - * Function: EFUSE_ShadowMapUpdate + * Function: efuse_shadow_map_update * * Overview: Transfer current EFUSE content to shadow init and modify map. * @@ -216,7 +216,7 @@ static void efuse_ShadowRead4Byte(struct adapter *padap= ter, u16 Offset, u32 *Val * 11/13/2008 MHC Create Version 0. * */ -void EFUSE_ShadowMapUpdate(struct adapter *padapter, u8 efuseType) +void rtw_efuse_shadow_map_update(struct adapter *padapter, u8 efuseType) { struct eeprom_priv *pEEPROM =3D GET_EEPROM_EFUSE_PRIV(padapter); u16 mapLen =3D 0; @@ -230,11 +230,11 @@ void EFUSE_ShadowMapUpdate(struct adapter *padapter, = u8 efuseType) =20 /* PlatformMoveMemory((void *)&pHalData->EfuseMap[EFUSE_MODIFY_MAP][0], */ /* void *)&pHalData->EfuseMap[EFUSE_INIT_MAP][0], mapLen); */ -} /* EFUSE_ShadowMapUpdate */ +} /* efuse_shadow_map_update */ =20 =20 /*------------------------------------------------------------------------= ----- - * Function: EFUSE_ShadowRead + * Function: efuse_shadow_read * * Overview: Read from efuse init map !!!!! * @@ -249,13 +249,13 @@ void EFUSE_ShadowMapUpdate(struct adapter *padapter, = u8 efuseType) * 11/12/2008 MHC Create Version 0. * */ -void EFUSE_ShadowRead(struct adapter *padapter, u8 Type, u16 Offset, u32 *= Value) +void rtw_efuse_shadow_read(struct adapter *padapter, u8 Type, u16 Offset, = u32 *Value) { if (Type =3D=3D 1) - efuse_ShadowRead1Byte(padapter, Offset, (u8 *)Value); + efuse_shadow_read_1_byte(padapter, Offset, (u8 *)Value); else if (Type =3D=3D 2) - efuse_ShadowRead2Byte(padapter, Offset, (u16 *)Value); + efuse_shadow_read_2_byte(padapter, Offset, (u16 *)Value); else if (Type =3D=3D 4) - efuse_ShadowRead4Byte(padapter, Offset, (u32 *)Value); + efuse_shadow_read_4_byte(padapter, Offset, (u32 *)Value); =20 -} /* EFUSE_ShadowRead*/ +} /* efuse_shadow_read*/ diff --git a/drivers/staging/rtl8723bs/hal/rtl8723b_hal_init.c b/drivers/st= aging/rtl8723bs/hal/rtl8723b_hal_init.c index e794fe3caf9d..3dff8aa2badb 100644 --- a/drivers/staging/rtl8723bs/hal/rtl8723b_hal_init.c +++ b/drivers/staging/rtl8723bs/hal/rtl8723b_hal_init.c @@ -651,7 +651,7 @@ static void hal_ReadEFuse_WiFi( hal_EfuseSwitchToBank(padapter, 0); =20 while (AVAILABLE_EFUSE_ADDR(eFuse_Addr)) { - efuse_OneByteRead(padapter, eFuse_Addr++, &efuseHeader); + rtw_efuse_one_byte_read(padapter, eFuse_Addr++, &efuseHeader); if (efuseHeader =3D=3D 0xFF) break; =20 @@ -659,7 +659,7 @@ static void hal_ReadEFuse_WiFi( if (EXT_HEADER(efuseHeader)) { /* extended header */ offset =3D GET_HDR_OFFSET_2_0(efuseHeader); =20 - efuse_OneByteRead(padapter, eFuse_Addr++, &efuseExtHdr); + rtw_efuse_one_byte_read(padapter, eFuse_Addr++, &efuseExtHdr); if (ALL_WORDS_DISABLED(efuseExtHdr)) continue; =20 @@ -678,16 +678,18 @@ static void hal_ReadEFuse_WiFi( for (i =3D 0; i < EFUSE_MAX_WORD_UNIT; i++) { /* Check word enable condition in the section */ if (!(wden & (0x01<bautoload_fail_flag) { /* autoload OK. */ if (!pEEPROM->EepromOrEfuse) { /* Read EFUSE real map to shadow. */ - EFUSE_ShadowMapUpdate(padapter, EFUSE_WIFI); + rtw_efuse_shadow_map_update(padapter, EFUSE_WIFI); memcpy(PROMContent, pEEPROM->efuse_eeprom_data, HWSET_MAX_SIZE_8723B); } } else {/* autoload fail */ if (!pEEPROM->EepromOrEfuse) - EFUSE_ShadowMapUpdate(padapter, EFUSE_WIFI); + rtw_efuse_shadow_map_update(padapter, EFUSE_WIFI); memcpy(PROMContent, pEEPROM->efuse_eeprom_data, HWSET_MAX_SIZE_8723B); } } @@ -1447,7 +1451,7 @@ void Hal_EfuseParsePackageType_8723B( u8 efuseContent; =20 Hal_EfusePowerSwitch(padapter, true); - efuse_OneByteRead(padapter, 0x1FB, &efuseContent); + rtw_efuse_one_byte_read(padapter, 0x1FB, &efuseContent); Hal_EfusePowerSwitch(padapter, false); =20 package =3D efuseContent & 0x7; @@ -1555,7 +1559,8 @@ void Hal_ReadRFGainOffset( =20 if (!AutoloadFail) { Adapter->eeprompriv.EEPROMRFGainOffset =3D PROMContent[EEPROM_RF_GAIN_OF= FSET]; - Adapter->eeprompriv.EEPROMRFGainVal =3D EFUSE_Read1Byte(Adapter, EEPROM_= RF_GAIN_VAL); + Adapter->eeprompriv.EEPROMRFGainVal =3D rtw_efuse_read_1_byte(Adapter, + EEPROM_RF_GAIN_VAL); } else { Adapter->eeprompriv.EEPROMRFGainOffset =3D 0; Adapter->eeprompriv.EEPROMRFGainVal =3D 0xFF; diff --git a/drivers/staging/rtl8723bs/hal/sdio_halinit.c b/drivers/staging= /rtl8723bs/hal/sdio_halinit.c index f2f73c65a636..3b3fac981a27 100644 --- a/drivers/staging/rtl8723bs/hal/sdio_halinit.c +++ b/drivers/staging/rtl8723bs/hal/sdio_halinit.c @@ -574,7 +574,7 @@ static bool HalDetectPwrDownMode(struct adapter *Adapte= r) struct pwrctrl_priv *pwrctrlpriv =3D adapter_to_pwrctl(Adapter); =20 =20 - EFUSE_ShadowRead(Adapter, 1, 0x7B/*EEPROM_RF_OPT3_92C*/, (u32 *)&tmpvalue= ); + rtw_efuse_shadow_read(Adapter, 1, 0x7B/*EEPROM_RF_OPT3_92C*/, (u32 *)&tmp= value); =20 /* 2010/08/25 MH INF priority > PDN Efuse value. */ if (tmpvalue & BIT4 && pwrctrlpriv->reg_pdnmode) diff --git a/drivers/staging/rtl8723bs/include/rtw_efuse.h b/drivers/stagin= g/rtl8723bs/include/rtw_efuse.h index 191ffdf593d4..808ba94a5998 100644 --- a/drivers/staging/rtl8723bs/include/rtw_efuse.h +++ b/drivers/staging/rtl8723bs/include/rtw_efuse.h @@ -31,7 +31,7 @@ enum { #define EFUSE_REPEAT_THRESHOLD_ 3 =20 /* */ -/* The following is for BT Efuse definition */ +/* The following is for BT Efuse definition */ /* */ #define EFUSE_BT_MAX_MAP_LEN 1024 #define EFUSE_MAX_BANK 4 @@ -68,13 +68,11 @@ struct efuse_hal { u8 fakeBTEfuseModifiedMap[EFUSE_BT_MAX_MAP_LEN]; }; =20 -u8 Efuse_CalculateWordCnts(u8 word_en); -u8 efuse_OneByteRead(struct adapter *padapter, u16 addr, u8 *data); +u8 rtw_efuse_calculate_word_counts(u8 word_en); +u8 rtw_efuse_one_byte_read(struct adapter *padapter, u16 addr, u8 *data); =20 -u8 EFUSE_Read1Byte(struct adapter *padapter, u16 Address); -void EFUSE_ShadowMapUpdate(struct adapter *padapter, u8 efuseType); -void EFUSE_ShadowRead(struct adapter *padapter, u8 Type, u16 Offset, u32 *= Value); -void Rtw_Hal_ReadMACAddrFromFile(struct adapter *padapter); -u32 Rtw_Hal_readPGDataFromConfigFile(struct adapter *padapter); +u8 rtw_efuse_read_1_byte(struct adapter *padapter, u16 Address); +void rtw_efuse_shadow_map_update(struct adapter *padapter, u8 efuseType); +void rtw_efuse_shadow_read(struct adapter *padapter, u8 Type, u16 Offset, = u32 *Value); =20 #endif --=20 2.53.0