From mboxrd@z Thu Jan 1 00:00:00 1970 Content-Type: multipart/mixed; boundary="===============3167584164665505788==" MIME-Version: 1.0 From: William Breathitt Gray To: kbuild-all@lists.01.org Subject: Re: [PATCH v6 1/5] counter: Internalize sysfs interface code Date: Wed, 25 Nov 2020 08:02:09 -0500 Message-ID: In-Reply-To: <202011250429.NVAm6jT3-lkp@intel.com> List-Id: --===============3167584164665505788== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable On Wed, Nov 25, 2020 at 04:55:46AM +0800, kernel test robot wrote: > Hi William, > = > I love your patch! Perhaps something to improve: > = > [auto build test WARNING on stm32/stm32-next] > [also build test WARNING on linux/master linus/master v5.10-rc5 next-2020= 1124] > [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/William-Breathitt-Gray/I= ntroduce-the-Counter-character-device-interface/20201123-043634 > base: https://git.kernel.org/pub/scm/linux/kernel/git/atorgue/stm32.git= stm32-next > config: powerpc-randconfig-r016-20201124 (attached as .config) > compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project df9a= e5992889560a8f3c6760b54d5051b47c7bf5) > reproduce (this is a W=3D1 build): > wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbi= n/make.cross -O ~/bin/make.cross > chmod +x ~/bin/make.cross > # install powerpc cross compiling tool for clang build > # apt-get install binutils-powerpc-linux-gnu > # https://github.com/0day-ci/linux/commit/1bc8c3bb021f3736c23d97f= 1a965935821f0ac5a > git remote add linux-review https://github.com/0day-ci/linux > git fetch --no-tags linux-review William-Breathitt-Gray/Introduce= -the-Counter-character-device-interface/20201123-043634 > git checkout 1bc8c3bb021f3736c23d97f1a965935821f0ac5a > # save the attached .config to linux build tree > COMPILER_INSTALL_PATH=3D$HOME/0day COMPILER=3Dclang make.cross AR= CH=3Dpowerpc = > = > If you fix the issue, kindly add following tag as appropriate > Reported-by: kernel test robot > = > All warnings (new ones prefixed by >>): > = > In file included from include/linux/iopoll.h:14: > In file included from include/linux/io.h:13: > In file included from arch/powerpc/include/asm/io.h:604: > arch/powerpc/include/asm/io-defs.h:43:1: warning: performing pointer a= rithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmet= ic] > DEF_PCI_AC_NORET(insb, (unsigned long p, void *b, unsigned long c), > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > arch/powerpc/include/asm/io.h:601:3: note: expanded from macro 'DEF_PC= I_AC_NORET' > __do_##name al; \ > ^~~~~~~~~~~~~~ > :170:1: note: expanded from here > __do_insb > ^ > arch/powerpc/include/asm/io.h:541:56: note: expanded from macro '__do_= insb' > #define __do_insb(p, b, n) readsb((PCI_IO_ADDR)_IO_BASE+(p), (b),= (n)) > ~~~~~~~~~~~~~~~~~~~~~^ > In file included from drivers/counter/ti-eqep.c:15: > In file included from include/linux/regmap.h:20: > In file included from include/linux/iopoll.h:14: > In file included from include/linux/io.h:13: > In file included from arch/powerpc/include/asm/io.h:604: > arch/powerpc/include/asm/io-defs.h:45:1: warning: performing pointer a= rithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmet= ic] > DEF_PCI_AC_NORET(insw, (unsigned long p, void *b, unsigned long c), > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > arch/powerpc/include/asm/io.h:601:3: note: expanded from macro 'DEF_PC= I_AC_NORET' > __do_##name al; \ > ^~~~~~~~~~~~~~ > :172:1: note: expanded from here > __do_insw > ^ > arch/powerpc/include/asm/io.h:542:56: note: expanded from macro '__do_= insw' > #define __do_insw(p, b, n) readsw((PCI_IO_ADDR)_IO_BASE+(p), (b),= (n)) > ~~~~~~~~~~~~~~~~~~~~~^ > In file included from drivers/counter/ti-eqep.c:15: > In file included from include/linux/regmap.h:20: > In file included from include/linux/iopoll.h:14: > In file included from include/linux/io.h:13: > In file included from arch/powerpc/include/asm/io.h:604: > arch/powerpc/include/asm/io-defs.h:47:1: warning: performing pointer a= rithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmet= ic] > DEF_PCI_AC_NORET(insl, (unsigned long p, void *b, unsigned long c), > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > arch/powerpc/include/asm/io.h:601:3: note: expanded from macro 'DEF_PC= I_AC_NORET' > __do_##name al; \ > ^~~~~~~~~~~~~~ > :174:1: note: expanded from here > __do_insl > ^ > arch/powerpc/include/asm/io.h:543:56: note: expanded from macro '__do_= insl' > #define __do_insl(p, b, n) readsl((PCI_IO_ADDR)_IO_BASE+(p), (b),= (n)) > ~~~~~~~~~~~~~~~~~~~~~^ > In file included from drivers/counter/ti-eqep.c:15: > In file included from include/linux/regmap.h:20: > In file included from include/linux/iopoll.h:14: > In file included from include/linux/io.h:13: > In file included from arch/powerpc/include/asm/io.h:604: > arch/powerpc/include/asm/io-defs.h:49:1: warning: performing pointer a= rithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmet= ic] > DEF_PCI_AC_NORET(outsb, (unsigned long p, const void *b, unsigned long= c), > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~= ~~~~ > arch/powerpc/include/asm/io.h:601:3: note: expanded from macro 'DEF_PC= I_AC_NORET' > __do_##name al; \ > ^~~~~~~~~~~~~~ > :176:1: note: expanded from here > __do_outsb > ^ > arch/powerpc/include/asm/io.h:544:58: note: expanded from macro '__do_= outsb' > #define __do_outsb(p, b, n) writesb((PCI_IO_ADDR)_IO_BASE+(p),(b),= (n)) > ~~~~~~~~~~~~~~~~~~~~~^ > In file included from drivers/counter/ti-eqep.c:15: > In file included from include/linux/regmap.h:20: > In file included from include/linux/iopoll.h:14: > In file included from include/linux/io.h:13: > In file included from arch/powerpc/include/asm/io.h:604: > arch/powerpc/include/asm/io-defs.h:51:1: warning: performing pointer a= rithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmet= ic] > DEF_PCI_AC_NORET(outsw, (unsigned long p, const void *b, unsigned long= c), > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~= ~~~~ > arch/powerpc/include/asm/io.h:601:3: note: expanded from macro 'DEF_PC= I_AC_NORET' > __do_##name al; \ > ^~~~~~~~~~~~~~ > :178:1: note: expanded from here > __do_outsw > ^ > arch/powerpc/include/asm/io.h:545:58: note: expanded from macro '__do_= outsw' > #define __do_outsw(p, b, n) writesw((PCI_IO_ADDR)_IO_BASE+(p),(b),= (n)) > ~~~~~~~~~~~~~~~~~~~~~^ > In file included from drivers/counter/ti-eqep.c:15: > In file included from include/linux/regmap.h:20: > In file included from include/linux/iopoll.h:14: > In file included from include/linux/io.h:13: > In file included from arch/powerpc/include/asm/io.h:604: > arch/powerpc/include/asm/io-defs.h:53:1: warning: performing pointer a= rithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmet= ic] > DEF_PCI_AC_NORET(outsl, (unsigned long p, const void *b, unsigned long= c), > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~= ~~~~ > arch/powerpc/include/asm/io.h:601:3: note: expanded from macro 'DEF_PC= I_AC_NORET' > __do_##name al; \ > ^~~~~~~~~~~~~~ > :180:1: note: expanded from here > __do_outsl > ^ > arch/powerpc/include/asm/io.h:546:58: note: expanded from macro '__do_= outsl' > #define __do_outsl(p, b, n) writesl((PCI_IO_ADDR)_IO_BASE+(p),(b),= (n)) > ~~~~~~~~~~~~~~~~~~~~~^ > >> drivers/counter/ti-eqep.c:115:17: warning: unused variable 'ti_qep_t2c= _functions_map' [-Wunused-const-variable] > static const u8 ti_qep_t2c_functions_map[] =3D { > ^ > 13 warnings generated. > = > vim +/ti_qep_t2c_functions_map +115 drivers/counter/ti-eqep.c > = > 114 = > > 115 static const u8 ti_qep_t2c_functions_map[] =3D { > 116 }; > 117 = > = > --- > 0-DAY CI Kernel Test Service, Intel Corporation > https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org This ti_qep_t2c_functions_map array is left over from some code I was testing. It's not used at all -- I simply forgot to remove it -- so I'll make sure to take it out in the next patchset. I'll give this v6 patchset some more time for people to review and comment before I submit the v7 revision. William Breathitt Gray --===============3167584164665505788== Content-Type: application/pgp-signature MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: attachment; filename="signature.asc" LS0tLS1CRUdJTiBQR1AgU0lHTkFUVVJFLS0tLS0KCmlRSXpCQUFCQ2dBZEZpRUVrNUk0UERKMncx Y0RmL2JnaHZwSU5kbTdWSklGQWwrK1ZjUUFDZ2tRaHZwSU5kbTcKVkpJSnVnLzlHaDk0OXpmaW1L VUFmNTZCVW14NSs0aHhnU3BsQUs0MG8zYTNiNFRPTW82UEtFVFJmNlJXUitkegowbjBLWHk3RW9P MVhDb3Y3eVcvcHpqNlZNeXpyVzdRdDVhRWRzUjFmRTFUano2U3Z6czNTenFuK1luS1dBaU9YClZD bHEwcE5SbGZNS3JTZG0rWUk2YTYrb1pZMDRqanFTWit0QThnRXBmbTg3b2F2TkxIdVZZVkd2VnAr VTNrUmkKWG1VbkhsbWR3Vm5rYlIzNUdTR1BTcGlOUVgzK2NQWGpHRmxSN2dieU5teGd3T1U5WmVa OU8rLzVEUXZhQ0NuZwp3MmRQMzN2ZnB4UHlMdERUcVhqTDg4TEJ6WEY0dXRrdkFybVgxUllWWWZu bWNqU2xib2RNSVlOcGJaeFFoNGsvClVXbXZOaDN1eENjREhFSFlURWJjc05VbVQ4VnBteSszY29M V2ppMGJHdWZuclR5OUtyYW12eHhnNWQ1c1ZjU00KLzY0NE1ydW1WUlA0R3Nma25SR04waG12eERk Wis1M1pvOTNmeWhmVDJXL1c5bXJLdmFYY1EwK0s2MlpuT05DeApCWTFzWU5CVS9XOEFxc2dvY3d1 NWNYNmdVWG01VUtVMXdKOSs5OHo3QnlsZTBDUEN6SWIrVTdFTWZmdW1rR0tzCjJNeGVKN1EzcVpE bmJmR3RmUVdlOVI5R0Jjekhlb2NIK3hwT2VjYmVrRXAwUkRDd3dMRHF4Szd5VTIzTURUeTYKYkpj eWl6aXJVdk95aDZ4S25FMjUvbGRWMk5aR3dBMUJpTGc2bTZzV2M5VHZMZEVDempKMnRydVlzbW5q bTdIRAo0UFFvUTBkWHY0MGZaQkJJb2ZkSmdycGJjN2FJejZ2R2piQ0s0bDVmZlUyc3RZcCtmZDA9 Cj1pRHFYCi0tLS0tRU5EIFBHUCBTSUdOQVRVUkUtLS0tLQo= --===============3167584164665505788==--