Subject: Re: [PATCH v7 net-next 5/9] drivers: net: dsa: add fdb entry flags to drivers

From: kernel test robot <[email-masked]>
Date: Tue, 11 Oct 2022 12:51:30 +0800

Hi Hans,

Thank you for the patch! Yet something to improve:

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

url:    https://github.com/intel-lab-lkp/linux/commits/Hans-J-Schultz/Extend-locked-port-feature-with-FDB-locked-flag-MAC-Auth-MAB/20221010-125833
base:   https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git 0326074ff4652329f2a1a9c8685104576bd8d131
config: arm-allyesconfig
compiler: arm-linux-gnueabi-gcc (GCC) 12.1.0
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # https://github.com/intel-lab-lkp/linux/commit/c43f9a7ac8a269ec0741676ba077322536bd3b23
        git remote add linux-review https://github.com/intel-lab-lkp/linux
        git fetch --no-tags linux-review Hans-J-Schultz/Extend-locked-port-feature-with-FDB-locked-flag-MAC-Auth-MAB/20221010-125833
        git checkout c43f9a7ac8a269ec0741676ba077322536bd3b23
        # save the config file
        mkdir build_dir && cp config build_dir/.config
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=arm SHELL=/bin/bash

If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot <[email-masked]>

All errors (new ones prefixed by >>):

>> drivers/net/dsa/rzn1_a5psw.c:759:25: error: initialization of 'int (*)(struct dsa_switch *, int,  const unsigned char *, u16,  u16,  struct dsa_db)' {aka 'int (*)(struct dsa_switch *, int,  const unsigned char *, short unsigned int,  short unsigned int,  struct dsa_db)'} from incompatible pointer type 'int (*)(struct dsa_switch *, int,  const unsigned char *, u16,  struct dsa_db)' {aka 'int (*)(struct dsa_switch *, int,  const unsigned char *, short unsigned int,  struct dsa_db)'} [-Werror=incompatible-pointer-types]
     759 |         .port_fdb_add = a5psw_port_fdb_add,
         |                         ^~~~~~~~~~~~~~~~~~
   drivers/net/dsa/rzn1_a5psw.c:759:25: note: (near initialization for 'a5psw_switch_ops.port_fdb_add')
   drivers/net/dsa/rzn1_a5psw.c:760:25: error: initialization of 'int (*)(struct dsa_switch *, int,  const unsigned char *, u16,  u16,  struct dsa_db)' {aka 'int (*)(struct dsa_switch *, int,  const unsigned char *, short unsigned int,  short unsigned int,  struct dsa_db)'} from incompatible pointer type 'int (*)(struct dsa_switch *, int,  const unsigned char *, u16,  struct dsa_db)' {aka 'int (*)(struct dsa_switch *, int,  const unsigned char *, short unsigned int,  struct dsa_db)'} [-Werror=incompatible-pointer-types]
     760 |         .port_fdb_del = a5psw_port_fdb_del,
         |                         ^~~~~~~~~~~~~~~~~~
   drivers/net/dsa/rzn1_a5psw.c:760:25: note: (near initialization for 'a5psw_switch_ops.port_fdb_del')
   cc1: some warnings being treated as errors

vim +759 drivers/net/dsa/rzn1_a5psw.c

888cdb892b61d7 Clément Léger 2022-06-24  736  
888cdb892b61d7 Clément Léger 2022-06-24  737  static const struct dsa_switch_ops a5psw_switch_ops = {
888cdb892b61d7 Clément Léger 2022-06-24  738  	.get_tag_protocol = a5psw_get_tag_protocol,
888cdb892b61d7 Clément Léger 2022-06-24  739  	.setup = a5psw_setup,
888cdb892b61d7 Clément Léger 2022-06-24  740  	.port_disable = a5psw_port_disable,
888cdb892b61d7 Clément Léger 2022-06-24  741  	.port_enable = a5psw_port_enable,
888cdb892b61d7 Clément Léger 2022-06-24  742  	.phylink_get_caps = a5psw_phylink_get_caps,
888cdb892b61d7 Clément Léger 2022-06-24  743  	.phylink_mac_select_pcs = a5psw_phylink_mac_select_pcs,
888cdb892b61d7 Clément Léger 2022-06-24  744  	.phylink_mac_link_down = a5psw_phylink_mac_link_down,
888cdb892b61d7 Clément Léger 2022-06-24  745  	.phylink_mac_link_up = a5psw_phylink_mac_link_up,
888cdb892b61d7 Clément Léger 2022-06-24  746  	.port_change_mtu = a5psw_port_change_mtu,
888cdb892b61d7 Clément Léger 2022-06-24  747  	.port_max_mtu = a5psw_port_max_mtu,
c7243fd4a62f2f Clément Léger 2022-06-24  748  	.get_sset_count = a5psw_get_sset_count,
c7243fd4a62f2f Clément Léger 2022-06-24  749  	.get_strings = a5psw_get_strings,
c7243fd4a62f2f Clément Léger 2022-06-24  750  	.get_ethtool_stats = a5psw_get_ethtool_stats,
c7243fd4a62f2f Clément Léger 2022-06-24  751  	.get_eth_mac_stats = a5psw_get_eth_mac_stats,
c7243fd4a62f2f Clément Léger 2022-06-24  752  	.get_eth_ctrl_stats = a5psw_get_eth_ctrl_stats,
c7243fd4a62f2f Clément Léger 2022-06-24  753  	.get_rmon_stats = a5psw_get_rmon_stats,
888cdb892b61d7 Clément Léger 2022-06-24  754  	.set_ageing_time = a5psw_set_ageing_time,
888cdb892b61d7 Clément Léger 2022-06-24  755  	.port_bridge_join = a5psw_port_bridge_join,
888cdb892b61d7 Clément Léger 2022-06-24  756  	.port_bridge_leave = a5psw_port_bridge_leave,
888cdb892b61d7 Clément Léger 2022-06-24  757  	.port_stp_state_set = a5psw_port_stp_state_set,
888cdb892b61d7 Clément Léger 2022-06-24  758  	.port_fast_age = a5psw_port_fast_age,
5edf246c686986 Clément Léger 2022-06-24 @759  	.port_fdb_add = a5psw_port_fdb_add,
5edf246c686986 Clément Léger 2022-06-24  760  	.port_fdb_del = a5psw_port_fdb_del,
5edf246c686986 Clément Léger 2022-06-24  761  	.port_fdb_dump = a5psw_port_fdb_dump,
888cdb892b61d7 Clément Léger 2022-06-24  762  };
888cdb892b61d7 Clément Léger 2022-06-24  763  

-- 
0-DAY CI Kernel Test Service
https://01.org/lkp