Skip to contentSkip to conversation

staging: rtl8723bs: fix CamelCase warnings in rtw_efuse.c

PATCH In review Bruce Ou 2 participants 4 messages Jun 23 Staging

lore
Share & tools
lore

Bruce Ou

author
#0 lore raw

PATCH 1/1 staging: rtl8723bs: fix CamelCase warnings in rtw_efuse.c

---
 drivers/staging/rtl8723bs/core/rtw_efuse.c | 42 +++++++++++-----------
 1 file changed, 21 insertions(+), 21 deletions(-)
drivers/staging/rtl8723bs/core/rtw_efuse.c +21 -21
index 099320e4e..b5dc5892f 100644
@@ -46,31 +46,31 @@ EFUSE_Read1Byte(
 struct adapter *Adapter,
 u16		Address)
 {
-	u8 Bytetemp = {0x00};
+	u8 byte_temp = {0x00};
 	u8 temp = {0x00};
 	u32 k = 0;
-	u16 contentLen = 0;
+	u16 content_len = 0;
 
-	Hal_GetEfuseDefinition(Adapter, EFUSE_WIFI, TYPE_EFUSE_REAL_CONTENT_LEN, &contentLen);
+	Hal_GetEfuseDefinition(Adapter, EFUSE_WIFI, TYPE_EFUSE_REAL_CONTENT_LEN, &content_len);
 
-	if (Address < contentLen) {/* E-fuse 512Byte */
+	if (Address < content_len) {/* E-fuse 512Byte */
 		/* Write E-fuse Register address bit0~7 */
 		temp = Address & 0xFF;
 		rtw_write8(Adapter, EFUSE_CTRL + 1, temp);
-		Bytetemp = rtw_read8(Adapter, EFUSE_CTRL + 2);
+		byte_temp = rtw_read8(Adapter, EFUSE_CTRL + 2);
 		/* Write E-fuse Register address bit8~9 */
-		temp = ((Address >> 8) & 0x03) | (Bytetemp & 0xFC);
+		temp = ((Address >> 8) & 0x03) | (byte_temp & 0xFC);
 		rtw_write8(Adapter, EFUSE_CTRL + 2, temp);
 
 		/* Write 0x30[31]= 0 */
-		Bytetemp = rtw_read8(Adapter, EFUSE_CTRL + 3);
-		temp = Bytetemp & 0x7F;
+		byte_temp = rtw_read8(Adapter, EFUSE_CTRL + 3);
+		temp = byte_temp & 0x7F;
 		rtw_write8(Adapter, EFUSE_CTRL + 3, temp);
 
 		/* Wait Write-ready (0x30[31]= 1) */
-		Bytetemp = rtw_read8(Adapter, EFUSE_CTRL + 3);
-		while (!(Bytetemp & 0x80)) {
-			Bytetemp = rtw_read8(Adapter, EFUSE_CTRL + 3);
+		byte_temp = rtw_read8(Adapter, EFUSE_CTRL + 3);
+		while (!(byte_temp & 0x80)) {
+			byte_temp = rtw_read8(Adapter, EFUSE_CTRL + 3);
 			k++;
 			if (k == 1000)
 				break;
@@ -139,15 +139,15 @@ u8	*data)
  * 11/11/2008	MHC		Create Version 0.
  *
  */
-static void Efuse_ReadAllMap(struct adapter *padapter, u8 efuseType, u8 *Efuse)
+static void Efuse_ReadAllMap(struct adapter *padapter, u8 efuse_type, u8 *Efuse)
 {
-	u16 mapLen = 0;
+	u16 map_len = 0;
 
 	Hal_EfusePowerSwitch(padapter, true);
 
-	Hal_GetEfuseDefinition(padapter, efuseType, TYPE_EFUSE_MAP_LEN, &mapLen);
+	Hal_GetEfuseDefinition(padapter, efuse_type, TYPE_EFUSE_MAP_LEN, &map_len);
 
-	Hal_ReadEFuse(padapter, efuseType, 0, mapLen, Efuse);
+	Hal_ReadEFuse(padapter, efuse_type, 0, map_len, Efuse);
 
 	Hal_EfusePowerSwitch(padapter, false);
 }
@@ -216,20 +216,20 @@ static void efuse_ShadowRead4Byte(struct adapter *padapter, u16 Offset, u32 *Val
  * 11/13/2008	MHC		Create Version 0.
  *
  */
-void EFUSE_ShadowMapUpdate(struct adapter *padapter, u8 efuseType)
+void EFUSE_ShadowMapUpdate(struct adapter *padapter, u8 efuse_type)
 {
 	struct eeprom_priv *pEEPROM = GET_EEPROM_EFUSE_PRIV(padapter);
-	u16 mapLen = 0;
+	u16 map_len = 0;
 
-	Hal_GetEfuseDefinition(padapter, efuseType, TYPE_EFUSE_MAP_LEN, &mapLen);
+	Hal_GetEfuseDefinition(padapter, efuse_type, TYPE_EFUSE_MAP_LEN, &map_len);
 
 	if (pEEPROM->bautoload_fail_flag)
-		memset(pEEPROM->efuse_eeprom_data, 0xFF, mapLen);
+		memset(pEEPROM->efuse_eeprom_data, 0xFF, map_len);
 	else
-		Efuse_ReadAllMap(padapter, efuseType, pEEPROM->efuse_eeprom_data);
+		Efuse_ReadAllMap(padapter, efuse_type, pEEPROM->efuse_eeprom_data);
 
 	/* PlatformMoveMemory((void *)&pHalData->EfuseMap[EFUSE_MODIFY_MAP][0], */
-	/* void *)&pHalData->EfuseMap[EFUSE_INIT_MAP][0], mapLen); */
+	/* void *)&pHalData->EfuseMap[EFUSE_INIT_MAP][0], map_len); */
 } /*  EFUSE_ShadowMapUpdate */
 
 
Signature · 1 line
2.43.0
#1 lore raw
Quoted 2 lines · Bruce Ou

---

No commit message. You're working against an old tree as well. You need to work against linux-next or devel-next.

Please, don't send a patch at a time, one by one throughout the day. Send them as a patchset.

regards,
dan carpenter

author
#2 lore raw

Hi Dan,

Thank you for your review and guidance!

I have switched my local tree to the latest staging/staging-next branch and squashed all the CamelCase local variable cleanups in rtw_efuse.c into a single, comprehensive patch with a proper commit message.

I will send the new patch shortly.

Regards,
Bruce Ou

Quoted 14 lines · Dan Carpenter <[email protected]> 于2026年6月23日周二 17:21写道:
Quoted 2 lines · Bruce Ou

---

No commit message. You're working against an old tree as well. You need to work against linux-next or devel-next.

Please, don't send a patch at a time, one by one throughout the day. Send them as a patchset.

regards,
dan carpenter

#3 lore raw
Quoted 9 lines · bruce ou

Hi Dan,

Thank you for your review and guidance!

I have switched my local tree to the latest staging/staging-next branch and squashed all the CamelCase local variable cleanups in rtw_efuse.c into a single, comprehensive patch with a proper commit message.

I will send the new patch shortly.

Not as one patch, but as a collection of patches in a patchset.

regards,
dan carpenter