From: kernel test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: Re: [RFC net-next 2/2] net: dsa: add Realtek RTL8366S switch driver
Date: Wed, 17 Feb 2021 15:37:33 +0800 [thread overview]
Message-ID: <202102171519.C68a7MwF-lkp@intel.com> (raw)
In-Reply-To: <20210217062139.7893-3-dqfext@gmail.com>
[-- Attachment #1: Type: text/plain, Size: 10322 bytes --]
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-Realtek-RTL8366S-SR/20210217-143046
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/3cd6e7f27c49d9e06810a3ee0b03fbb6f20a40b9
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@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(a)lists.01.org
[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 30527 bytes --]
next prev parent reply other threads:[~2021-02-17 7:37 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-02-17 6:21 [RFC net-next 0/2] DSA driver for Realtek RTL8366S/SR DENG Qingfang
2021-02-17 6:21 ` [RFC net-next 1/2] net: dsa: add Realtek RTL8366S switch tag DENG Qingfang
2021-02-17 7:07 ` Heiner Kallweit
2021-02-17 11:01 ` Linus Walleij
2021-02-17 11:28 ` Heiner Kallweit
2021-02-17 10:55 ` Linus Walleij
2021-02-17 12:28 ` DENG Qingfang
2021-02-27 23:47 ` Linus Walleij
2021-02-28 11:32 ` DENG Qingfang
2021-03-01 13:06 ` Linus Walleij
2021-02-17 6:21 ` [RFC net-next 2/2] net: dsa: add Realtek RTL8366S switch driver DENG Qingfang
2021-02-17 7:37 ` kernel test robot [this message]
2021-02-17 9:57 ` kernel test robot
2021-02-17 11:12 ` Linus Walleij
2021-02-17 12:45 ` DENG Qingfang
2021-02-17 12:25 ` Vladimir Oltean
2021-02-17 15:44 ` Linus Walleij
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=202102171519.C68a7MwF-lkp@intel.com \
--to=lkp@intel.com \
--cc=kbuild-all@lists.01.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.