Hi Egil,

[auto build test WARNING on net-next/master]

url:    
https://github.com/0day-ci/linux/commits/Egil-Hjelmeland/net-dsa-lan9303-unicast-offload-fdb-mdb-STP/20170727-074246
config: x86_64-randconfig-x000-201730 (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

Note: it may well be a FALSE warning. FWIW you are at least aware of it now.
http://gcc.gnu.org/wiki/Better_Uninitialized_Warnings

All warnings (new ones prefixed by >>):

   drivers/net//dsa/lan9303-core.c: In function 'lan9303_port_stp_state_set':
>> drivers/net//dsa/lan9303-core.c:945:16: warning: 'portstate' may be used 
>> uninitialized in this function [-Wmaybe-uninitialized]
     int portmask, portstate;
                   ^~~~~~~~~

vim +/portstate +945 drivers/net//dsa/lan9303-core.c

   941  
   942  static void lan9303_port_stp_state_set(struct dsa_switch *ds, int port,
   943                                         u8 state)
   944  {
 > 945          int portmask, portstate;
   946          struct lan9303 *chip = ds->priv;
   947  
   948          dev_dbg(chip->dev, "%s(port %d, state %d)\n",
   949                  __func__, port, state);
   950          if (!chip->is_bridged)
   951                  return;
   952  
   953          switch (state) {
   954          case BR_STATE_DISABLED:
   955                  portstate = LAN9303_SWE_PORT_STATE_DISABLED_PORT0;
   956                  break;
   957          case BR_STATE_BLOCKING:
   958          case BR_STATE_LISTENING:
   959                  portstate = LAN9303_SWE_PORT_STATE_BLOCKING_PORT0;
   960                  break;
   961          case BR_STATE_LEARNING:
   962                  portstate = LAN9303_SWE_PORT_STATE_LEARNING_PORT0;
   963                  break;
   964          case BR_STATE_FORWARDING:
   965                  portstate = LAN9303_SWE_PORT_STATE_FORWARDING_PORT0;
   966                  break;
   967          default:
   968                  dev_err(chip->dev, "%s(port %d, state %d)\n",
   969                          __func__, port, state);
   970          }
   971          portmask = 0x3 << (port * 2);
   972          portstate     <<= (port * 2);
   973          lan9303_write_switch_reg_mask(chip, LAN9303_SWE_PORT_STATE,
   974                                        portstate, portmask);
   975  }
   976  

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Attachment: .config.gz
Description: application/gzip

Reply via email to