From: kernel test robot <lkp@intel.com> To: Florinel Iordache <florinel.iordache@nxp.com>, davem@davemloft.net, netdev@vger.kernel.org, andrew@lunn.ch, f.fainelli@gmail.com, hkallweit1@gmail.com, linux@armlinux.org.uk Cc: kbuild-all@lists.01.org, clang-built-linux@googlegroups.com, devicetree@vger.kernel.org, linux-doc@vger.kernel.org, robh+dt@kernel.org, mark.rutland@arm.com Subject: Re: [PATCH net-next v3 5/7] net: phy: enable qoriq backplane support Date: Thu, 25 Jun 2020 08:35:25 +0800 [thread overview] Message-ID: <202006250846.o4cQISE9%lkp@intel.com> (raw) In-Reply-To: <1592832924-31733-6-git-send-email-florinel.iordache@nxp.com> [-- Attachment #1: Type: text/plain, Size: 2876 bytes --] Hi Florinel, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on net-next/master] url: https://github.com/0day-ci/linux/commits/Florinel-Iordache/doc-net-add-backplane-documentation/20200622-223623 base: https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git 29a720c1042f469c8fea317cb5e7f496b116e07d config: x86_64-allyesconfig (attached as .config) compiler: clang version 11.0.0 (https://github.com/llvm/llvm-project 1d4c87335d5236ea1f35937e1014980ba961ae34) 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 # install x86_64 cross compiling tool for clang build # apt-get install binutils-x86-64-linux-gnu # save the attached .config to linux build tree COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64 If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <lkp@intel.com> All warnings (new ones prefixed by >>): >> drivers/net/phy/backplane/qoriq_backplane.c:150:18: warning: no previous prototype for function 'get_serdes_type' [-Wmissing-prototypes] enum serdes_type get_serdes_type(struct device_node *serdes_node) ^ drivers/net/phy/backplane/qoriq_backplane.c:150:1: note: declare 'static' if the function is not intended to be used outside of this translation unit enum serdes_type get_serdes_type(struct device_node *serdes_node) ^ static 1 warning generated. -- >> drivers/net/phy/backplane/qoriq_serdes_10g.c:120:13: warning: bitwise or with non-zero value always evaluates to true [-Wtautological-bitwise-compare] if (ln_req | LANE_RX) { ~~~~~~~^~~~~~~~~ drivers/net/phy/backplane/qoriq_serdes_10g.c:115:13: warning: bitwise or with non-zero value always evaluates to true [-Wtautological-bitwise-compare] if (ln_req | LANE_TX) { ~~~~~~~^~~~~~~~~ 2 warnings generated. vim +/get_serdes_type +150 drivers/net/phy/backplane/qoriq_backplane.c 149 > 150 enum serdes_type get_serdes_type(struct device_node *serdes_node) 151 { 152 enum serdes_type serdes = SERDES_INVAL; 153 const char *serdes_comp; 154 int comp_no, i, ret; 155 156 comp_no = of_property_count_strings(serdes_node, "compatible"); 157 for (i = 0; i < comp_no; i++) { 158 ret = of_property_read_string_index(serdes_node, "compatible", 159 i, &serdes_comp); 160 if (ret == 0) { 161 if (!strcasecmp(serdes_comp, "serdes-10g")) { 162 serdes = SERDES_10G; 163 break; 164 } 165 } 166 } 167 168 return serdes; 169 } 170 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org [-- Attachment #2: .config.gz --] [-- Type: application/gzip, Size: 75316 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com> To: kbuild-all@lists.01.org Subject: Re: [PATCH net-next v3 5/7] net: phy: enable qoriq backplane support Date: Thu, 25 Jun 2020 08:35:25 +0800 [thread overview] Message-ID: <202006250846.o4cQISE9%lkp@intel.com> (raw) In-Reply-To: <1592832924-31733-6-git-send-email-florinel.iordache@nxp.com> [-- Attachment #1: Type: text/plain, Size: 2944 bytes --] Hi Florinel, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on net-next/master] url: https://github.com/0day-ci/linux/commits/Florinel-Iordache/doc-net-add-backplane-documentation/20200622-223623 base: https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git 29a720c1042f469c8fea317cb5e7f496b116e07d config: x86_64-allyesconfig (attached as .config) compiler: clang version 11.0.0 (https://github.com/llvm/llvm-project 1d4c87335d5236ea1f35937e1014980ba961ae34) 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 # install x86_64 cross compiling tool for clang build # apt-get install binutils-x86-64-linux-gnu # save the attached .config to linux build tree COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64 If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <lkp@intel.com> All warnings (new ones prefixed by >>): >> drivers/net/phy/backplane/qoriq_backplane.c:150:18: warning: no previous prototype for function 'get_serdes_type' [-Wmissing-prototypes] enum serdes_type get_serdes_type(struct device_node *serdes_node) ^ drivers/net/phy/backplane/qoriq_backplane.c:150:1: note: declare 'static' if the function is not intended to be used outside of this translation unit enum serdes_type get_serdes_type(struct device_node *serdes_node) ^ static 1 warning generated. -- >> drivers/net/phy/backplane/qoriq_serdes_10g.c:120:13: warning: bitwise or with non-zero value always evaluates to true [-Wtautological-bitwise-compare] if (ln_req | LANE_RX) { ~~~~~~~^~~~~~~~~ drivers/net/phy/backplane/qoriq_serdes_10g.c:115:13: warning: bitwise or with non-zero value always evaluates to true [-Wtautological-bitwise-compare] if (ln_req | LANE_TX) { ~~~~~~~^~~~~~~~~ 2 warnings generated. vim +/get_serdes_type +150 drivers/net/phy/backplane/qoriq_backplane.c 149 > 150 enum serdes_type get_serdes_type(struct device_node *serdes_node) 151 { 152 enum serdes_type serdes = SERDES_INVAL; 153 const char *serdes_comp; 154 int comp_no, i, ret; 155 156 comp_no = of_property_count_strings(serdes_node, "compatible"); 157 for (i = 0; i < comp_no; i++) { 158 ret = of_property_read_string_index(serdes_node, "compatible", 159 i, &serdes_comp); 160 if (ret == 0) { 161 if (!strcasecmp(serdes_comp, "serdes-10g")) { 162 serdes = SERDES_10G; 163 break; 164 } 165 } 166 } 167 168 return serdes; 169 } 170 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org [-- Attachment #2: config.gz --] [-- Type: application/gzip, Size: 75316 bytes --]
next prev parent reply other threads:[~2020-06-25 0:38 UTC|newest] Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-06-22 13:35 [PATCH net-next v2 0/9] net: ethernet backplane support on DPAA1 Florinel Iordache 2020-06-22 13:35 ` [PATCH net-next v3 1/7] doc: net: add backplane documentation Florinel Iordache 2020-06-22 13:35 ` [PATCH net-next v3 2/7] dt-bindings: net: add backplane dt bindings Florinel Iordache 2020-06-22 22:20 ` Florian Fainelli 2020-06-24 12:55 ` [EXT] " Florinel Iordache 2020-06-26 18:55 ` Florian Fainelli 2020-06-29 21:58 ` Rob Herring 2020-06-22 13:35 ` [PATCH net-next v3 3/7] net: fman: add kr support for dpaa1 mac Florinel Iordache 2020-06-22 13:35 ` [PATCH net-next v3 4/7] net: phy: add backplane kr driver support Florinel Iordache 2020-06-22 14:24 ` Andrew Lunn 2020-06-22 14:39 ` [EXT] " Florinel Iordache 2020-06-26 19:05 ` Florian Fainelli 2020-06-29 13:23 ` Florinel Iordache 2020-06-22 15:08 ` Madalin Bucur (OSS) 2020-06-26 19:02 ` Florian Fainelli 2020-06-29 13:58 ` Russell King - ARM Linux admin 2020-06-30 0:51 ` Andrew Lunn 2020-06-22 16:29 ` kernel test robot 2020-06-22 16:29 ` kernel test robot 2020-06-22 21:46 ` Jakub Kicinski 2020-06-22 13:35 ` [PATCH net-next v3 5/7] net: phy: enable qoriq backplane support Florinel Iordache 2020-06-22 17:47 ` kernel test robot 2020-06-22 17:47 ` kernel test robot 2020-06-25 0:35 ` kernel test robot [this message] 2020-06-25 0:35 ` kernel test robot 2020-06-22 13:35 ` [PATCH net-next v3 6/7] net: phy: add bee algorithm for kr training Florinel Iordache 2020-06-22 13:35 ` [PATCH net-next v3 7/7] arm64: dts: add serdes and mdio description Florinel Iordache 2020-06-26 19:08 ` Florian Fainelli 2020-06-29 12:32 ` [EXT] " Florinel Iordache
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=202006250846.o4cQISE9%lkp@intel.com \ --to=lkp@intel.com \ --cc=andrew@lunn.ch \ --cc=clang-built-linux@googlegroups.com \ --cc=davem@davemloft.net \ --cc=devicetree@vger.kernel.org \ --cc=f.fainelli@gmail.com \ --cc=florinel.iordache@nxp.com \ --cc=hkallweit1@gmail.com \ --cc=kbuild-all@lists.01.org \ --cc=linux-doc@vger.kernel.org \ --cc=linux@armlinux.org.uk \ --cc=mark.rutland@arm.com \ --cc=netdev@vger.kernel.org \ --cc=robh+dt@kernel.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: linkBe 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.