Commit aa947d71 authored by Ping-Ke Shih's avatar Ping-Ke Shih Committed by Greg Kroah-Hartman
Browse files

staging: rtl8723bs: remove unused efuseValue from efuse_OneByteWrite() 



The efuseValue is to store value from register EFUSE_CTRL, and set control
bits including address and write bit. This is no need for RTL8723BS, so
the consumer has been removed. Thus, remove these unused codes are safe.

Otherwiese, clang warns:

rtw_efuse.c:285:6: warning:
	 variable 'efuseValue' set but not used [-Wunused-but-set-variable]
  285 |         u32 efuseValue;
      |             ^

Signed-off-by: default avatarPing-Ke Shih <pkshih@realtek.com>
Tested-by: default avatarPhilipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/20240913002815.5149-3-pkshih@realtek.com


Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent bfa0290f

drivers/staging/rtl8723bs/core/rtw_efuse.c

+0 −9
Original line number Diff line number Diff line
@@ -281,22 +281,13 @@ u8 efuse_OneByteWrite(struct adapter *padapter, u16 addr, u8 data, bool bPseudoT 
{

	u8 tmpidx = 0;

	u8 bResult = false;

	u32 efuseValue;



	if (bPseudoTest)

		return Efuse_Write1ByteToFakeContent(addr, data);





	/*  -----------------e-fuse reg ctrl --------------------------------- */

	/* address */





	efuseValue = rtw_read32(padapter, EFUSE_CTRL);

	efuseValue |= (BIT21 | BIT31);

	efuseValue &= ~(0x3FFFF);

	efuseValue |= ((addr << 8 | data) & 0x3FFFF);





	/*  <20130227, Kordan> 8192E MP chip A-cut had better not set 0x34[11] until B-Cut. */



	/*  <20130121, Kordan> For SMIC EFUSE specificatoin. */