Hi DENG,
[FYI, it's a private test report for your RFC patch.]
[auto build test ERROR on net-next/master]
url:
https://github.com/0day-ci/linux/commits/DENG-Qingfang/DSA-driver-for-Rea...
base:
https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git
06b334f08b4f0e53be64160392be4c37db28a413
config: parisc-randconfig-m031-20210216 (attached as .config)
compiler: hppa-linux-gcc (GCC) 9.3.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/0day-ci/linux/commit/3cd6e7f27c49d9e06810a3ee0b03fbb6f...
git remote add linux-review
https://github.com/0day-ci/linux
git fetch --no-tags linux-review
DENG-Qingfang/DSA-driver-for-Realtek-RTL8366S-SR/20210217-143046
git checkout 3cd6e7f27c49d9e06810a3ee0b03fbb6f20a40b9
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=parisc
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>):
drivers/net/dsa/rtl8366s.c: In function 'rtl8366s_port_vlan_filtering':
> drivers/net/dsa/rtl8366s.c:928:9: error: too few arguments to
function 'rtl8366_vlan_filtering'
928 | return
rtl8366_vlan_filtering(ds, port, vlan_filtering);
| ^~~~~~~~~~~~~~~~~~~~~~
In file included from drivers/net/dsa/rtl8366s.c:14:
drivers/net/dsa/realtek-smi-core.h:133:5: note: declared here
133 | int rtl8366_vlan_filtering(struct dsa_switch *ds, int port, bool
vlan_filtering,
| ^~~~~~~~~~~~~~~~~~~~~~
In file included from include/linux/skbuff.h:13,
from include/linux/if_ether.h:19,
from include/linux/etherdevice.h:20,
from drivers/net/dsa/rtl8366s.c:10:
drivers/net/dsa/rtl8366s.c: In function 'rtl8366s_port_lag_join':
> include/linux/list.h:522:18: error: invalid type argument of
'->' (have 'struct list_head')
522 |
list_entry((ptr)->next, type, member)
| ^~
include/linux/kernel.h:693:26: note: in definition of macro 'container_of'
693 | void *__mptr = (void *)(ptr); \
| ^~~
include/linux/list.h:522:2: note: in expansion of macro 'list_entry'
522 | list_entry((ptr)->next, type, member)
| ^~~~~~~~~~
include/linux/list.h:628:13: note: in expansion of macro 'list_first_entry'
628 | for (pos = list_first_entry(head, typeof(*pos), member); \
| ^~~~~~~~~~~~~~~~
drivers/net/dsa/rtl8366s.c:1049:2: note: in expansion of macro
'list_for_each_entry'
1049 | list_for_each_entry(dp, dp->dst->ports, list) {
| ^~~~~~~~~~~~~~~~~~~
In file included from <command-line>:
> include/linux/list.h:522:18: error: invalid type argument of
'->' (have 'struct list_head')
522 |
list_entry((ptr)->next, type, member)
| ^~
include/linux/compiler_types.h:300:9: note: in definition of macro
'__compiletime_assert'
300 | if (!(condition)) \
| ^~~~~~~~~
include/linux/compiler_types.h:320:2: note: in expansion of macro
'_compiletime_assert'
320 | _compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__)
| ^~~~~~~~~~~~~~~~~~~
include/linux/build_bug.h:39:37: note: in expansion of macro
'compiletime_assert'
39 | #define BUILD_BUG_ON_MSG(cond, msg) compiletime_assert(!(cond), msg)
| ^~~~~~~~~~~~~~~~~~
include/linux/kernel.h:694:2: note: in expansion of macro 'BUILD_BUG_ON_MSG'
694 | BUILD_BUG_ON_MSG(!__same_type(*(ptr), ((type *)0)->member) && \
| ^~~~~~~~~~~~~~~~
include/linux/kernel.h:694:20: note: in expansion of macro '__same_type'
694 | BUILD_BUG_ON_MSG(!__same_type(*(ptr), ((type *)0)->member) && \
| ^~~~~~~~~~~
include/linux/list.h:511:2: note: in expansion of macro 'container_of'
511 | container_of(ptr, type, member)
| ^~~~~~~~~~~~
include/linux/list.h:522:2: note: in expansion of macro 'list_entry'
522 | list_entry((ptr)->next, type, member)
| ^~~~~~~~~~
include/linux/list.h:628:13: note: in expansion of macro 'list_first_entry'
628 | for (pos = list_first_entry(head, typeof(*pos), member); \
| ^~~~~~~~~~~~~~~~
drivers/net/dsa/rtl8366s.c:1049:2: note: in expansion of macro
'list_for_each_entry'
1049 | list_for_each_entry(dp, dp->dst->ports, list) {
| ^~~~~~~~~~~~~~~~~~~
> include/linux/list.h:522:18: error: invalid type argument of
'->' (have 'struct list_head')
522 |
list_entry((ptr)->next, type, member)
| ^~
include/linux/compiler_types.h:300:9: note: in definition of macro
'__compiletime_assert'
300 | if (!(condition)) \
| ^~~~~~~~~
include/linux/compiler_types.h:320:2: note: in expansion of macro
'_compiletime_assert'
320 | _compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__)
| ^~~~~~~~~~~~~~~~~~~
include/linux/build_bug.h:39:37: note: in expansion of macro
'compiletime_assert'
39 | #define BUILD_BUG_ON_MSG(cond, msg) compiletime_assert(!(cond), msg)
| ^~~~~~~~~~~~~~~~~~
include/linux/kernel.h:694:2: note: in expansion of macro 'BUILD_BUG_ON_MSG'
694 | BUILD_BUG_ON_MSG(!__same_type(*(ptr), ((type *)0)->member) && \
| ^~~~~~~~~~~~~~~~
include/linux/kernel.h:695:6: note: in expansion of macro '__same_type'
695 | !__same_type(*(ptr), void), \
| ^~~~~~~~~~~
include/linux/list.h:511:2: note: in expansion of macro 'container_of'
511 | container_of(ptr, type, member)
| ^~~~~~~~~~~~
include/linux/list.h:522:2: note: in expansion of macro 'list_entry'
522 | list_entry((ptr)->next, type, member)
| ^~~~~~~~~~
include/linux/list.h:628:13: note: in expansion of macro 'list_first_entry'
628 | for (pos = list_first_entry(head, typeof(*pos), member); \
| ^~~~~~~~~~~~~~~~
drivers/net/dsa/rtl8366s.c:1049:2: note: in expansion of macro
'list_for_each_entry'
1049 | list_for_each_entry(dp, dp->dst->ports, list) {
| ^~~~~~~~~~~~~~~~~~~
In file included from include/linux/preempt.h:11,
from include/linux/spinlock.h:51,
from include/linux/mmzone.h:8,
from include/linux/gfp.h:6,
from include/linux/mm.h:10,
from include/linux/bvec.h:14,
from include/linux/skbuff.h:17,
from include/linux/if_ether.h:19,
from include/linux/etherdevice.h:20,
from drivers/net/dsa/rtl8366s.c:10:
> include/linux/list.h:619:16: error: invalid operands to binary ==
(have 'const struct list_head *' and 'struct list_head')
619 |
(&pos->member == (head))
| ^~ ~~~~~~
include/linux/list.h:629:8: note: in expansion of macro 'list_entry_is_head'
629 | !list_entry_is_head(pos, head, member); \
| ^~~~~~~~~~~~~~~~~~
drivers/net/dsa/rtl8366s.c:1049:2: note: in expansion of macro
'list_for_each_entry'
1049 | list_for_each_entry(dp, dp->dst->ports, list) {
| ^~~~~~~~~~~~~~~~~~~
drivers/net/dsa/rtl8366s.c: At top level:
> drivers/net/dsa/rtl8366s.c:1130:25: error: initialization of
'int (*)(struct dsa_switch *, int, bool, struct netlink_ext_ack *)' {aka
'int (*)(struct dsa_switch *, int, _Bool, struct netlink_ext_ack *)'} from
incompatible pointer type 'int (*)(struct dsa_switch *, int, bool)' {aka 'int
(*)(struct dsa_switch *, int, _Bool)'} [-Werror=incompatible-pointer-types]
1130 | .port_vlan_filtering = rtl8366s_port_vlan_filtering,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/net/dsa/rtl8366s.c:1130:25: note: (near initialization for
'rtl8366s_switch_ops.port_vlan_filtering')
> drivers/net/dsa/rtl8366s.c:1158:41: error: static declaration of
'rtl8366s_variant' follows non-static declaration
1158 | static const
struct realtek_smi_variant rtl8366s_variant = {
| ^~~~~~~~~~~~~~~~
In file included from drivers/net/dsa/rtl8366s.c:14:
drivers/net/dsa/realtek-smi-core.h:146:41: note: previous declaration of
'rtl8366s_variant' was here
146 | extern const struct realtek_smi_variant rtl8366s_variant;
| ^~~~~~~~~~~~~~~~
drivers/net/dsa/rtl8366s.c:1158:41: warning: 'rtl8366s_variant' defined but not
used [-Wunused-const-variable=]
1158 | static const struct realtek_smi_variant rtl8366s_variant = {
| ^~~~~~~~~~~~~~~~
cc1: some warnings being treated as errors
vim +/rtl8366_vlan_filtering +928 drivers/net/dsa/rtl8366s.c
906
907 static int rtl8366s_port_vlan_filtering(struct dsa_switch *ds, int port,
908 bool vlan_filtering)
909 {
910 struct realtek_smi *smi = ds->priv;
911 unsigned int val;
912 int ret;
913
914 /* Enable/Disable VLAN ingress filtering */
915 val = BIT(port);
916 ret = regmap_update_bits(smi->map, RTL8366S_VLAN_MEMBERINGRESS_REG,
917 val, vlan_filtering ? val : 0);
918 if (ret)
919 return ret;
920
921 /* Disable/Enable keep original tagged/untagged */
922 val = FIELD_PREP(RTL8366S_EGRESS_KEEP_FORMAT_MASK, val);
923 ret = regmap_update_bits(smi->map, RTL8366S_EGRESS_KEEP_FORMAT_REG,
924 val, vlan_filtering ? 0 : val);
925 if (ret)
926 return ret;
927
928 return rtl8366_vlan_filtering(ds, port, vlan_filtering);
929 }
930
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org