CC: [email protected]
BCC: [email protected]
CC: [email protected]
TO: Matt Johnston <[email protected]>
CC: Jeremy Kerr <[email protected]>
CC: Wolfram Sang <[email protected]>

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 
master
head:   3123109284176b1532874591f7c81f3837bbdc17
commit: f5b8abf9fc3dacd7529d363e26fe8230935d65f8 mctp i2c: MCTP I2C binding 
driver
date:   6 weeks ago
:::::: branch date: 35 hours ago
:::::: commit date: 6 weeks ago
compiler: hppa-linux-gcc (GCC) 11.2.0

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <[email protected]>


cppcheck warnings: (new ones prefixed by >>)
>> drivers/staging/rtl8723bs/core/rtw_mlme_ext.c:5656:16: warning: Local 
>> variable 'null_addr' shadows outer variable [shadowVariable]
    unsigned char null_addr[] = {0x00, 0x00, 0x00, 0x00, 0x00, 0x00};
                  ^
   drivers/staging/rtl8723bs/core/rtw_mlme_ext.c:52:11: note: Shadowed 
declaration
   static u8 null_addr[ETH_ALEN] = {0, 0, 0, 0, 0, 0};
             ^
   drivers/staging/rtl8723bs/core/rtw_mlme_ext.c:5656:16: note: Shadow variable
    unsigned char null_addr[] = {0x00, 0x00, 0x00, 0x00, 0x00, 0x00};
                  ^

cppcheck possible warnings: (new ones prefixed by >>, may not real problems)

>> drivers/staging/rtl8723bs/core/rtw_efuse.c:365:11: warning: Parameter 
>> 'sourdata' can be declared with const [constParameter]
         u8 *sourdata,
             ^
>> drivers/net/mctp/mctp-i2c.c:996:2: warning: There is an unknown macro here 
>> somewhere. Configuration is required. If list_for_each_entry_safe is a macro 
>> then please configure it. [unknownMacro]
    list_for_each_entry_safe(midev, tmp, &mcli->devs, list)
    ^
--
>> drivers/staging/rtl8723bs/core/rtw_ieee80211.c:58:16: warning: Variable 
>> 'dot11_rate_table' can be declared with const [constVariable]
    unsigned char dot11_rate_table[] = {2, 4, 11, 22, 12, 18, 24, 36, 48, 72, 
96, 108, 0}; /*  last element must be zero!! */
                  ^
>> drivers/staging/rtl8723bs/core/rtw_ieee80211.c:278:30: warning: Parameter 
>> 'rateset' can be declared with const [constParameter]
   uint rtw_get_rateset_len(u8 *rateset)
                                ^
>> drivers/staging/rtl8723bs/core/rtw_ieee80211.c:1109:59: warning: Parameter 
>> 'MCS_rate' can be declared with const [constParameter]
   u16 rtw_mcs_rate(u8 bw_40MHz, u8 short_GI, unsigned char *MCS_rate)
                                                             ^
>> drivers/net/phy/micrel.c:1574:9: warning: Uninitialized variable: ret 
>> [uninitvar]
    return ret;
           ^
   drivers/net/phy/micrel.c:1555:19: note: Assuming condition is false
    while (pair_mask && retries--) {
                     ^
   drivers/net/phy/micrel.c:1574:9: note: Uninitialized variable: ret
    return ret;
           ^
--
>> drivers/staging/rtl8723bs/core/rtw_cmd.c:88:2: warning: There is an unknown 
>> macro here somewhere. Configuration is required. If GEN_DRV_CMD_HANDLER is a 
>> macro then please configure it. [unknownMacro]
    GEN_DRV_CMD_HANDLER(0, NULL) /*0*/
    ^
--
>> drivers/staging/rtl8723bs/core/rtw_mlme.c:1998:54: warning: Parameter 
>> 'in_ie' can be declared with const [constParameter]
   int rtw_restruct_wmm_ie(struct adapter *adapter, u8 *in_ie, u8 *out_ie, uint 
in_len, uint initial_out_len)
                                                        ^
--
>> drivers/staging/rtl8723bs/core/rtw_mlme_ext.c:5897:57: warning: Parameter 
>> 'pbuf' can be declared with const [constParameter]
   u8 h2c_msg_hdl(struct adapter *padapter, unsigned char *pbuf)
                                                           ^

vim +996 drivers/net/mctp/mctp-i2c.c

f5b8abf9fc3dac Matt Johnston 2022-02-18   987  
f5b8abf9fc3dac Matt Johnston 2022-02-18   988  static int 
mctp_i2c_remove(struct i2c_client *client)
f5b8abf9fc3dac Matt Johnston 2022-02-18   989  {
f5b8abf9fc3dac Matt Johnston 2022-02-18   990   struct mctp_i2c_client *mcli = 
i2c_get_clientdata(client);
f5b8abf9fc3dac Matt Johnston 2022-02-18   991   struct mctp_i2c_dev *midev = 
NULL, *tmp = NULL;
f5b8abf9fc3dac Matt Johnston 2022-02-18   992  
f5b8abf9fc3dac Matt Johnston 2022-02-18   993   
mutex_lock(&driver_clients_lock);
f5b8abf9fc3dac Matt Johnston 2022-02-18   994   list_del(&mcli->list);
f5b8abf9fc3dac Matt Johnston 2022-02-18   995   /* Remove all child adapter 
netdevs */
f5b8abf9fc3dac Matt Johnston 2022-02-18  @996   list_for_each_entry_safe(midev, 
tmp, &mcli->devs, list)
f5b8abf9fc3dac Matt Johnston 2022-02-18   997           
mctp_i2c_unregister(midev);
f5b8abf9fc3dac Matt Johnston 2022-02-18   998  
f5b8abf9fc3dac Matt Johnston 2022-02-18   999   mctp_i2c_free_client(mcli);
f5b8abf9fc3dac Matt Johnston 2022-02-18  1000   
mutex_unlock(&driver_clients_lock);
f5b8abf9fc3dac Matt Johnston 2022-02-18  1001   /* Callers ignore return code */
f5b8abf9fc3dac Matt Johnston 2022-02-18  1002   return 0;
f5b8abf9fc3dac Matt Johnston 2022-02-18  1003  }
f5b8abf9fc3dac Matt Johnston 2022-02-18  1004  

-- 
0-DAY CI Kernel Test Service
https://01.org/lkp
_______________________________________________
kbuild mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to