From: kernel test robot <lkp@intel.com>
To: Moksh Panicker <mokshpanicker.7@gmail.com>, gregkh@linuxfoundation.org
Cc: llvm@lists.linux.dev, oe-kbuild-all@lists.linux.dev,
linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org,
Moksh Panicker <mokshpanicker.7@gmail.com>
Subject: Re: [PATCH] staging: rtl8723bs: fix coding style in rtw_efuse.c
Date: Fri, 15 May 2026 05:48:12 +0800 [thread overview]
Message-ID: <202605150500.7TJ6yTim-lkp@intel.com> (raw)
In-Reply-To: <20260510063314.34066-1-mokshpanicker.7@gmail.com>
Hi Moksh,
kernel test robot noticed the following build errors:
[auto build test ERROR on staging/staging-testing]
url: https://github.com/intel-lab-lkp/linux/commits/Moksh-Panicker/staging-rtl8723bs-fix-coding-style-in-rtw_efuse-c/20260514-222556
base: staging/staging-testing
patch link: https://lore.kernel.org/r/20260510063314.34066-1-mokshpanicker.7%40gmail.com
patch subject: [PATCH] staging: rtl8723bs: fix coding style in rtw_efuse.c
config: sparc64-allmodconfig (https://download.01.org/0day-ci/archive/20260515/202605150500.7TJ6yTim-lkp@intel.com/config)
compiler: clang version 23.0.0git (https://github.com/llvm/llvm-project 5bac06718f502014fade905512f1d26d578a18f3)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260515/202605150500.7TJ6yTim-lkp@intel.com/reproduce)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202605150500.7TJ6yTim-lkp@intel.com/
All errors (new ones prefixed by >>):
>> drivers/staging/rtl8723bs/core/rtw_efuse.c:83:1: error: function definition is not allowed here
83 | {
| ^
drivers/staging/rtl8723bs/core/rtw_efuse.c:136:1: error: function definition is not allowed here
136 | {
| ^
drivers/staging/rtl8723bs/core/rtw_efuse.c:167:1: error: function definition is not allowed here
167 | {
| ^
drivers/staging/rtl8723bs/core/rtw_efuse.c:176:1: error: function definition is not allowed here
176 | {
| ^
drivers/staging/rtl8723bs/core/rtw_efuse.c:186:1: error: function definition is not allowed here
186 | {
| ^
drivers/staging/rtl8723bs/core/rtw_efuse.c:213:1: error: function definition is not allowed here
213 | {
| ^
drivers/staging/rtl8723bs/core/rtw_efuse.c:245:1: error: function definition is not allowed here
245 | {
| ^
>> drivers/staging/rtl8723bs/core/rtw_efuse.c:253:29: error: expected '}'
253 | } /* rtw_efuse_shadow_read*/
| ^
drivers/staging/rtl8723bs/core/rtw_efuse.c:45:1: note: to match this '{'
45 | {
| ^
8 errors generated.
vim +83 drivers/staging/rtl8723bs/core/rtw_efuse.c
554c0a3abf216c Hans de Goede 2017-03-29 80
554c0a3abf216c Hans de Goede 2017-03-29 81 /* 11/16/2008 MH Read one byte from real Efuse. */
ceeea324fb96ba Moksh Panicker 2026-05-10 82 u8 rtw_efuse_one_byte_read(struct adapter *padapter, u16 addr, u8 *data)
554c0a3abf216c Hans de Goede 2017-03-29 @83 {
554c0a3abf216c Hans de Goede 2017-03-29 84 u32 tmpidx = 0;
554c0a3abf216c Hans de Goede 2017-03-29 85 u8 bResult;
554c0a3abf216c Hans de Goede 2017-03-29 86 u8 readbyte;
554c0a3abf216c Hans de Goede 2017-03-29 87
554c0a3abf216c Hans de Goede 2017-03-29 88 /* <20130121, Kordan> For SMIC EFUSE specificatoin. */
554c0a3abf216c Hans de Goede 2017-03-29 89 /* 0x34[11]: SW force PGMEN input of efuse to high. (for the bank selected by 0x34[9:8]) */
554c0a3abf216c Hans de Goede 2017-03-29 90 /* PHY_SetMacReg(padapter, 0x34, BIT11, 0); */
6a620bb2f4c1b4 Michael Steinmötzger 2026-05-09 91 rtw_write16(padapter, 0x34, rtw_read16(padapter, 0x34) & (~BIT(11)));
554c0a3abf216c Hans de Goede 2017-03-29 92
554c0a3abf216c Hans de Goede 2017-03-29 93 /* -----------------e-fuse reg ctrl --------------------------------- */
554c0a3abf216c Hans de Goede 2017-03-29 94 /* address */
554c0a3abf216c Hans de Goede 2017-03-29 95 rtw_write8(padapter, EFUSE_CTRL + 1, (u8)(addr & 0xff));
554c0a3abf216c Hans de Goede 2017-03-29 96 rtw_write8(padapter, EFUSE_CTRL + 2, ((u8)((addr >> 8) & 0x03)) |
554c0a3abf216c Hans de Goede 2017-03-29 97 (rtw_read8(padapter, EFUSE_CTRL + 2) & 0xFC));
554c0a3abf216c Hans de Goede 2017-03-29 98
554c0a3abf216c Hans de Goede 2017-03-29 99 /* rtw_write8(padapter, EFUSE_CTRL+3, 0x72); read cmd */
554c0a3abf216c Hans de Goede 2017-03-29 100 /* Write bit 32 0 */
554c0a3abf216c Hans de Goede 2017-03-29 101 readbyte = rtw_read8(padapter, EFUSE_CTRL + 3);
554c0a3abf216c Hans de Goede 2017-03-29 102 rtw_write8(padapter, EFUSE_CTRL + 3, (readbyte & 0x7f));
554c0a3abf216c Hans de Goede 2017-03-29 103
554c0a3abf216c Hans de Goede 2017-03-29 104 while (!(0x80 & rtw_read8(padapter, EFUSE_CTRL + 3)) && (tmpidx < 1000)) {
554c0a3abf216c Hans de Goede 2017-03-29 105 mdelay(1);
554c0a3abf216c Hans de Goede 2017-03-29 106 tmpidx++;
554c0a3abf216c Hans de Goede 2017-03-29 107 }
554c0a3abf216c Hans de Goede 2017-03-29 108 if (tmpidx < 100) {
554c0a3abf216c Hans de Goede 2017-03-29 109 *data = rtw_read8(padapter, EFUSE_CTRL);
554c0a3abf216c Hans de Goede 2017-03-29 110 bResult = true;
554c0a3abf216c Hans de Goede 2017-03-29 111 } else {
554c0a3abf216c Hans de Goede 2017-03-29 112 *data = 0xff;
554c0a3abf216c Hans de Goede 2017-03-29 113 bResult = false;
554c0a3abf216c Hans de Goede 2017-03-29 114 }
554c0a3abf216c Hans de Goede 2017-03-29 115
554c0a3abf216c Hans de Goede 2017-03-29 116 return bResult;
554c0a3abf216c Hans de Goede 2017-03-29 117 }
554c0a3abf216c Hans de Goede 2017-03-29 118
554c0a3abf216c Hans de Goede 2017-03-29 119 /*-----------------------------------------------------------------------------
554c0a3abf216c Hans de Goede 2017-03-29 120 * Function: Efuse_ReadAllMap
554c0a3abf216c Hans de Goede 2017-03-29 121 *
554c0a3abf216c Hans de Goede 2017-03-29 122 * Overview: Read All Efuse content
554c0a3abf216c Hans de Goede 2017-03-29 123 *
554c0a3abf216c Hans de Goede 2017-03-29 124 * Input: NONE
554c0a3abf216c Hans de Goede 2017-03-29 125 *
554c0a3abf216c Hans de Goede 2017-03-29 126 * Output: NONE
554c0a3abf216c Hans de Goede 2017-03-29 127 *
554c0a3abf216c Hans de Goede 2017-03-29 128 * Return: NONE
554c0a3abf216c Hans de Goede 2017-03-29 129 *
554c0a3abf216c Hans de Goede 2017-03-29 130 * Revised History:
554c0a3abf216c Hans de Goede 2017-03-29 131 * When Who Remark
554c0a3abf216c Hans de Goede 2017-03-29 132 * 11/11/2008 MHC Create Version 0.
554c0a3abf216c Hans de Goede 2017-03-29 133 *
35b79fd9fbdce9 Ignacio Pena 2025-07-16 134 */
e8605159aec9bc Michael Straube 2025-08-23 135 static void Efuse_ReadAllMap(struct adapter *padapter, u8 efuseType, u8 *Efuse)
554c0a3abf216c Hans de Goede 2017-03-29 136 {
554c0a3abf216c Hans de Goede 2017-03-29 137 u16 mapLen = 0;
554c0a3abf216c Hans de Goede 2017-03-29 138
236faa3b92d336 Michael Straube 2025-08-24 139 Hal_EfusePowerSwitch(padapter, true);
554c0a3abf216c Hans de Goede 2017-03-29 140
fd865573df743e Siwanan Bungtong 2026-02-10 141 Hal_GetEfuseDefinition(padapter, efuseType, TYPE_EFUSE_MAP_LEN, &mapLen);
554c0a3abf216c Hans de Goede 2017-03-29 142
0124378e9ed4fd Michael Straube 2025-08-23 143 Hal_ReadEFuse(padapter, efuseType, 0, mapLen, Efuse);
554c0a3abf216c Hans de Goede 2017-03-29 144
236faa3b92d336 Michael Straube 2025-08-24 145 Hal_EfusePowerSwitch(padapter, false);
554c0a3abf216c Hans de Goede 2017-03-29 146 }
554c0a3abf216c Hans de Goede 2017-03-29 147
554c0a3abf216c Hans de Goede 2017-03-29 148 /*-----------------------------------------------------------------------------
554c0a3abf216c Hans de Goede 2017-03-29 149 * Function: efuse_ShadowRead1Byte
554c0a3abf216c Hans de Goede 2017-03-29 150 * efuse_ShadowRead2Byte
554c0a3abf216c Hans de Goede 2017-03-29 151 * efuse_ShadowRead4Byte
554c0a3abf216c Hans de Goede 2017-03-29 152 *
554c0a3abf216c Hans de Goede 2017-03-29 153 * Overview: Read from efuse init map by one/two/four bytes !!!!!
554c0a3abf216c Hans de Goede 2017-03-29 154 *
554c0a3abf216c Hans de Goede 2017-03-29 155 * Input: NONE
554c0a3abf216c Hans de Goede 2017-03-29 156 *
554c0a3abf216c Hans de Goede 2017-03-29 157 * Output: NONE
554c0a3abf216c Hans de Goede 2017-03-29 158 *
554c0a3abf216c Hans de Goede 2017-03-29 159 * Return: NONE
554c0a3abf216c Hans de Goede 2017-03-29 160 *
554c0a3abf216c Hans de Goede 2017-03-29 161 * Revised History:
554c0a3abf216c Hans de Goede 2017-03-29 162 * When Who Remark
554c0a3abf216c Hans de Goede 2017-03-29 163 * 11/12/2008 MHC Create Version 0.
554c0a3abf216c Hans de Goede 2017-03-29 164 *
35b79fd9fbdce9 Ignacio Pena 2025-07-16 165 */
5641eeec0f5835 Ross Schmidt 2020-11-09 166 static void efuse_ShadowRead1Byte(struct adapter *padapter, u16 Offset, u8 *Value)
554c0a3abf216c Hans de Goede 2017-03-29 167 {
554c0a3abf216c Hans de Goede 2017-03-29 168 struct eeprom_priv *pEEPROM = GET_EEPROM_EFUSE_PRIV(padapter);
554c0a3abf216c Hans de Goede 2017-03-29 169
554c0a3abf216c Hans de Goede 2017-03-29 170 *Value = pEEPROM->efuse_eeprom_data[Offset];
554c0a3abf216c Hans de Goede 2017-03-29 171
554c0a3abf216c Hans de Goede 2017-03-29 172 } /* EFUSE_ShadowRead1Byte */
554c0a3abf216c Hans de Goede 2017-03-29 173
554c0a3abf216c Hans de Goede 2017-03-29 174 /* Read Two Bytes */
5641eeec0f5835 Ross Schmidt 2020-11-09 175 static void efuse_ShadowRead2Byte(struct adapter *padapter, u16 Offset, u16 *Value)
554c0a3abf216c Hans de Goede 2017-03-29 176 {
554c0a3abf216c Hans de Goede 2017-03-29 177 struct eeprom_priv *pEEPROM = GET_EEPROM_EFUSE_PRIV(padapter);
554c0a3abf216c Hans de Goede 2017-03-29 178
554c0a3abf216c Hans de Goede 2017-03-29 179 *Value = pEEPROM->efuse_eeprom_data[Offset];
554c0a3abf216c Hans de Goede 2017-03-29 180 *Value |= pEEPROM->efuse_eeprom_data[Offset + 1] << 8;
554c0a3abf216c Hans de Goede 2017-03-29 181
554c0a3abf216c Hans de Goede 2017-03-29 182 } /* EFUSE_ShadowRead2Byte */
554c0a3abf216c Hans de Goede 2017-03-29 183
554c0a3abf216c Hans de Goede 2017-03-29 184 /* Read Four Bytes */
5641eeec0f5835 Ross Schmidt 2020-11-09 185 static void efuse_ShadowRead4Byte(struct adapter *padapter, u16 Offset, u32 *Value)
554c0a3abf216c Hans de Goede 2017-03-29 186 {
554c0a3abf216c Hans de Goede 2017-03-29 187 struct eeprom_priv *pEEPROM = GET_EEPROM_EFUSE_PRIV(padapter);
554c0a3abf216c Hans de Goede 2017-03-29 188
554c0a3abf216c Hans de Goede 2017-03-29 189 *Value = pEEPROM->efuse_eeprom_data[Offset];
554c0a3abf216c Hans de Goede 2017-03-29 190 *Value |= pEEPROM->efuse_eeprom_data[Offset + 1] << 8;
554c0a3abf216c Hans de Goede 2017-03-29 191 *Value |= pEEPROM->efuse_eeprom_data[Offset + 2] << 16;
554c0a3abf216c Hans de Goede 2017-03-29 192 *Value |= pEEPROM->efuse_eeprom_data[Offset + 3] << 24;
554c0a3abf216c Hans de Goede 2017-03-29 193
554c0a3abf216c Hans de Goede 2017-03-29 194 } /* efuse_ShadowRead4Byte */
554c0a3abf216c Hans de Goede 2017-03-29 195
554c0a3abf216c Hans de Goede 2017-03-29 196 /*-----------------------------------------------------------------------------
3f188a336f9582 Linus Probert 2026-04-04 197 * Function: rtw_efuse_shadow_map_update
554c0a3abf216c Hans de Goede 2017-03-29 198 *
554c0a3abf216c Hans de Goede 2017-03-29 199 * Overview: Transfer current EFUSE content to shadow init and modify map.
554c0a3abf216c Hans de Goede 2017-03-29 200 *
554c0a3abf216c Hans de Goede 2017-03-29 201 * Input: NONE
554c0a3abf216c Hans de Goede 2017-03-29 202 *
554c0a3abf216c Hans de Goede 2017-03-29 203 * Output: NONE
554c0a3abf216c Hans de Goede 2017-03-29 204 *
554c0a3abf216c Hans de Goede 2017-03-29 205 * Return: NONE
554c0a3abf216c Hans de Goede 2017-03-29 206 *
554c0a3abf216c Hans de Goede 2017-03-29 207 * Revised History:
554c0a3abf216c Hans de Goede 2017-03-29 208 * When Who Remark
554c0a3abf216c Hans de Goede 2017-03-29 209 * 11/13/2008 MHC Create Version 0.
554c0a3abf216c Hans de Goede 2017-03-29 210 *
35b79fd9fbdce9 Ignacio Pena 2025-07-16 211 */
3f188a336f9582 Linus Probert 2026-04-04 212 void rtw_efuse_shadow_map_update(struct adapter *padapter, u8 efuseType)
554c0a3abf216c Hans de Goede 2017-03-29 213 {
554c0a3abf216c Hans de Goede 2017-03-29 214 struct eeprom_priv *pEEPROM = GET_EEPROM_EFUSE_PRIV(padapter);
554c0a3abf216c Hans de Goede 2017-03-29 215 u16 mapLen = 0;
554c0a3abf216c Hans de Goede 2017-03-29 216
fd865573df743e Siwanan Bungtong 2026-02-10 217 Hal_GetEfuseDefinition(padapter, efuseType, TYPE_EFUSE_MAP_LEN, &mapLen);
554c0a3abf216c Hans de Goede 2017-03-29 218
d2ab9916f0122c Michael Straube 2018-06-30 219 if (pEEPROM->bautoload_fail_flag)
554c0a3abf216c Hans de Goede 2017-03-29 220 memset(pEEPROM->efuse_eeprom_data, 0xFF, mapLen);
d2ab9916f0122c Michael Straube 2018-06-30 221 else
e8605159aec9bc Michael Straube 2025-08-23 222 Efuse_ReadAllMap(padapter, efuseType, pEEPROM->efuse_eeprom_data);
554c0a3abf216c Hans de Goede 2017-03-29 223
554c0a3abf216c Hans de Goede 2017-03-29 224 /* PlatformMoveMemory((void *)&pHalData->EfuseMap[EFUSE_MODIFY_MAP][0], */
554c0a3abf216c Hans de Goede 2017-03-29 225 /* void *)&pHalData->EfuseMap[EFUSE_INIT_MAP][0], mapLen); */
3f188a336f9582 Linus Probert 2026-04-04 226 } /* rtw_efuse_shadow_map_update */
554c0a3abf216c Hans de Goede 2017-03-29 227
554c0a3abf216c Hans de Goede 2017-03-29 228 /*-----------------------------------------------------------------------------
7a590f2660887a Linus Probert 2026-04-04 229 * Function: rtw_efuse_shadow_read
554c0a3abf216c Hans de Goede 2017-03-29 230 *
554c0a3abf216c Hans de Goede 2017-03-29 231 * Overview: Read from efuse init map !!!!!
554c0a3abf216c Hans de Goede 2017-03-29 232 *
554c0a3abf216c Hans de Goede 2017-03-29 233 * Input: NONE
554c0a3abf216c Hans de Goede 2017-03-29 234 *
554c0a3abf216c Hans de Goede 2017-03-29 235 * Output: NONE
554c0a3abf216c Hans de Goede 2017-03-29 236 *
554c0a3abf216c Hans de Goede 2017-03-29 237 * Return: NONE
554c0a3abf216c Hans de Goede 2017-03-29 238 *
554c0a3abf216c Hans de Goede 2017-03-29 239 * Revised History:
554c0a3abf216c Hans de Goede 2017-03-29 240 * When Who Remark
554c0a3abf216c Hans de Goede 2017-03-29 241 * 11/12/2008 MHC Create Version 0.
554c0a3abf216c Hans de Goede 2017-03-29 242 *
35b79fd9fbdce9 Ignacio Pena 2025-07-16 243 */
7a590f2660887a Linus Probert 2026-04-04 244 void rtw_efuse_shadow_read(struct adapter *padapter, u8 Type, u16 Offset, u32 *Value)
554c0a3abf216c Hans de Goede 2017-03-29 245 {
554c0a3abf216c Hans de Goede 2017-03-29 246 if (Type == 1)
554c0a3abf216c Hans de Goede 2017-03-29 247 efuse_ShadowRead1Byte(padapter, Offset, (u8 *)Value);
554c0a3abf216c Hans de Goede 2017-03-29 248 else if (Type == 2)
554c0a3abf216c Hans de Goede 2017-03-29 249 efuse_ShadowRead2Byte(padapter, Offset, (u16 *)Value);
554c0a3abf216c Hans de Goede 2017-03-29 250 else if (Type == 4)
554c0a3abf216c Hans de Goede 2017-03-29 251 efuse_ShadowRead4Byte(padapter, Offset, (u32 *)Value);
554c0a3abf216c Hans de Goede 2017-03-29 252
7a590f2660887a Linus Probert 2026-04-04 @253 } /* rtw_efuse_shadow_read*/
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
prev parent reply other threads:[~2026-05-14 21:52 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-05-10 6:33 [PATCH] staging: rtl8723bs: fix coding style in rtw_efuse.c Moksh Panicker
2026-05-10 16:08 ` Greg KH
2026-05-14 19:50 ` kernel test robot
2026-05-14 21:48 ` kernel test robot [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=202605150500.7TJ6yTim-lkp@intel.com \
--to=lkp@intel.com \
--cc=gregkh@linuxfoundation.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-staging@lists.linux.dev \
--cc=llvm@lists.linux.dev \
--cc=mokshpanicker.7@gmail.com \
--cc=oe-kbuild-all@lists.linux.dev \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.