CC: [email protected]
In-Reply-To: <[email protected]>
References: <[email protected]>
TO: Ansuel Smith <[email protected]>

Hi Ansuel,

[FYI, it's a private test report for your RFC patch.]
[auto build test WARNING on robh/for-next]
[also build test WARNING on v5.12]
[cannot apply to net-next/master net/master linus/master next-20210505]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]

url:    
https://github.com/0day-ci/linux/commits/Ansuel-Smith/net-mdio-ipq8064-clean-whitespaces-in-define/20210505-063401
base:   https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git for-next
:::::: branch date: 8 hours ago
:::::: commit date: 8 hours ago
config: x86_64-allyesconfig (attached as .config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0

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


cocci warnings: (new ones prefixed by >>)
>> drivers/net/dsa/qca8k.c:496:2-8: preceding lock on line 489
   drivers/net/dsa/qca8k.c:529:2-8: preceding lock on line 522

vim +496 drivers/net/dsa/qca8k.c

69462fe6a39048 Jonathan McDowell 2020-08-01  475  
69462fe6a39048 Jonathan McDowell 2020-08-01  476  static int
69462fe6a39048 Jonathan McDowell 2020-08-01  477  qca8k_vlan_add(struct 
qca8k_priv *priv, u8 port, u16 vid, bool untagged)
69462fe6a39048 Jonathan McDowell 2020-08-01  478  {
69462fe6a39048 Jonathan McDowell 2020-08-01  479        u32 reg;
69462fe6a39048 Jonathan McDowell 2020-08-01  480        int ret;
69462fe6a39048 Jonathan McDowell 2020-08-01  481  
69462fe6a39048 Jonathan McDowell 2020-08-01  482        /*
69462fe6a39048 Jonathan McDowell 2020-08-01  483           We do the right 
thing with VLAN 0 and treat it as untagged while
69462fe6a39048 Jonathan McDowell 2020-08-01  484           preserving the tag 
on egress.
69462fe6a39048 Jonathan McDowell 2020-08-01  485         */
69462fe6a39048 Jonathan McDowell 2020-08-01  486        if (vid == 0)
69462fe6a39048 Jonathan McDowell 2020-08-01  487                return 0;
69462fe6a39048 Jonathan McDowell 2020-08-01  488  
69462fe6a39048 Jonathan McDowell 2020-08-01 @489        
mutex_lock(&priv->reg_mutex);
69462fe6a39048 Jonathan McDowell 2020-08-01  490        ret = 
qca8k_vlan_access(priv, QCA8K_VLAN_READ, vid);
69462fe6a39048 Jonathan McDowell 2020-08-01  491        if (ret < 0)
69462fe6a39048 Jonathan McDowell 2020-08-01  492                goto out;
69462fe6a39048 Jonathan McDowell 2020-08-01  493  
69462fe6a39048 Jonathan McDowell 2020-08-01  494        reg = qca8k_read(priv, 
QCA8K_REG_VTU_FUNC0);
e0bc1bb1e57e47 Ansuel Smith      2021-05-05  495        if (reg < 0)
e0bc1bb1e57e47 Ansuel Smith      2021-05-05 @496                return reg;
69462fe6a39048 Jonathan McDowell 2020-08-01  497        reg |= 
QCA8K_VTU_FUNC0_VALID | QCA8K_VTU_FUNC0_IVL_EN;
69462fe6a39048 Jonathan McDowell 2020-08-01  498        reg &= 
~(QCA8K_VTU_FUNC0_EG_MODE_MASK << QCA8K_VTU_FUNC0_EG_MODE_S(port));
69462fe6a39048 Jonathan McDowell 2020-08-01  499        if (untagged)
69462fe6a39048 Jonathan McDowell 2020-08-01  500                reg |= 
QCA8K_VTU_FUNC0_EG_MODE_UNTAG <<
69462fe6a39048 Jonathan McDowell 2020-08-01  501                                
QCA8K_VTU_FUNC0_EG_MODE_S(port);
69462fe6a39048 Jonathan McDowell 2020-08-01  502        else
69462fe6a39048 Jonathan McDowell 2020-08-01  503                reg |= 
QCA8K_VTU_FUNC0_EG_MODE_TAG <<
69462fe6a39048 Jonathan McDowell 2020-08-01  504                                
QCA8K_VTU_FUNC0_EG_MODE_S(port);
69462fe6a39048 Jonathan McDowell 2020-08-01  505  
69462fe6a39048 Jonathan McDowell 2020-08-01  506        qca8k_write(priv, 
QCA8K_REG_VTU_FUNC0, reg);
69462fe6a39048 Jonathan McDowell 2020-08-01  507        ret = 
qca8k_vlan_access(priv, QCA8K_VLAN_LOAD, vid);
69462fe6a39048 Jonathan McDowell 2020-08-01  508  
69462fe6a39048 Jonathan McDowell 2020-08-01  509  out:
69462fe6a39048 Jonathan McDowell 2020-08-01  510        
mutex_unlock(&priv->reg_mutex);
69462fe6a39048 Jonathan McDowell 2020-08-01  511  
69462fe6a39048 Jonathan McDowell 2020-08-01  512        return ret;
69462fe6a39048 Jonathan McDowell 2020-08-01  513  }
69462fe6a39048 Jonathan McDowell 2020-08-01  514  

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/[email protected]

Attachment: .config.gz
Description: application/gzip

_______________________________________________
kbuild mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to