From: Dan Carpenter <dan.carpenter@oracle.com> To: Pavel Skripkin <paskripkin@gmail.com> Cc: kernel test robot <lkp@intel.com>, Larry.Finger@lwfinger.net, phil@philpotter.co.uk, gregkh@linuxfoundation.org, straube.linux@gmail.com, fmdefrancesco@gmail.com, clang-built-linux@googlegroups.com, kbuild-all@lists.01.org, linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org Subject: Re: [PATCH v3 3/6] staging: r8188eu: add error handling of rtw_read8 Date: Wed, 25 Aug 2021 15:51:59 +0300 [thread overview] Message-ID: <20210825125159.GU1931@kadam> (raw) In-Reply-To: <9743e1ee-75f4-6231-427b-8c7f659fb252@gmail.com> On Wed, Aug 25, 2021 at 03:17:06PM +0300, Pavel Skripkin wrote: > On 8/25/21 3:05 PM, kernel test robot wrote: > > Hi Pavel, > > > > Thank you for the patch! Yet something to improve: > > > > [auto build test ERROR on staging/staging-testing] > > > > url: https://github.com/0day-ci/linux/commits/Pavel-Skripkin/staging-r8188eu-remove-read-write-_macreg/20210824-162756 > > base: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git 093991aaadf0fbb34184fa37a46e7a157da3f386 > > config: arm-buildonly-randconfig-r001-20210825 (attached as .config) > > compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project ea08c4cd1c0869ec5024a8bb3f5cdf06ab03ae83) > > 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 arm cross compiling tool for clang build > > # apt-get install 11.1.1 > > # https://github.com/0day-ci/linux/commit/d4e4bbed4e59df37967086f60fe92cb1b4504d37 > > git remote add linux-review https://github.com/0day-ci/linux > > git fetch --no-tags linux-review Pavel-Skripkin/staging-r8188eu-remove-read-write-_macreg/20210824-162756 > > git checkout d4e4bbed4e59df37967086f60fe92cb1b4504d37 > > # save the attached .config to linux build tree > > mkdir build_dir > > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross O=build_dir ARCH=arm SHELL=/bin/bash drivers/staging/r8188eu/ > > > > 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/staging/r8188eu/hal/usb_halinit.c:2022:3: error: expected expression > > u8 tmp; > > ^ > > 1 error generated. > > > > Kconfig warnings: (for reference only) > > WARNING: unmet direct dependencies detected for QCOM_SCM > > Depends on (ARM || ARM64) && HAVE_ARM_SMCCC > > Selected by > > - ARM_QCOM_SPM_CPUIDLE && CPU_IDLE && (ARM || ARM64) && (ARCH_QCOM || COMPILE_TEST && !ARM64 && MMU > > > > > > We need to fix Kconfig, ok, I will take a look at it later > This is not related to your patch. Ignore it. > > vim +2022 drivers/staging/r8188eu/hal/usb_halinit.c > > > > > 2020 case HW_VAR_BCN_VALID: > > 2021 /* BCN_VALID, BIT(16) of REG_TDECTRL = BIT(0) of REG_TDECTRL+2, write 1 to clear, Clear by sw */ > > > 2022 u8 tmp; > > Hm, I don't know anything about ARM compilers, so should I wrap this code > block with {}? Yep. > > My local gcc 11.1.1 (x86_64) does not produce any warnings/errors > You should figure out whats up with that because it shouldn't compile with the gcc options that the kernel uses. regards, dan carpenter
WARNING: multiple messages have this Message-ID (diff)
From: Dan Carpenter <dan.carpenter@oracle.com> To: kbuild-all@lists.01.org Subject: Re: [PATCH v3 3/6] staging: r8188eu: add error handling of rtw_read8 Date: Wed, 25 Aug 2021 15:51:59 +0300 [thread overview] Message-ID: <20210825125159.GU1931@kadam> (raw) In-Reply-To: <9743e1ee-75f4-6231-427b-8c7f659fb252@gmail.com> [-- Attachment #1: Type: text/plain, Size: 2935 bytes --] On Wed, Aug 25, 2021 at 03:17:06PM +0300, Pavel Skripkin wrote: > On 8/25/21 3:05 PM, kernel test robot wrote: > > Hi Pavel, > > > > Thank you for the patch! Yet something to improve: > > > > [auto build test ERROR on staging/staging-testing] > > > > url: https://github.com/0day-ci/linux/commits/Pavel-Skripkin/staging-r8188eu-remove-read-write-_macreg/20210824-162756 > > base: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git 093991aaadf0fbb34184fa37a46e7a157da3f386 > > config: arm-buildonly-randconfig-r001-20210825 (attached as .config) > > compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project ea08c4cd1c0869ec5024a8bb3f5cdf06ab03ae83) > > 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 arm cross compiling tool for clang build > > # apt-get install 11.1.1 > > # https://github.com/0day-ci/linux/commit/d4e4bbed4e59df37967086f60fe92cb1b4504d37 > > git remote add linux-review https://github.com/0day-ci/linux > > git fetch --no-tags linux-review Pavel-Skripkin/staging-r8188eu-remove-read-write-_macreg/20210824-162756 > > git checkout d4e4bbed4e59df37967086f60fe92cb1b4504d37 > > # save the attached .config to linux build tree > > mkdir build_dir > > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross O=build_dir ARCH=arm SHELL=/bin/bash drivers/staging/r8188eu/ > > > > 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/staging/r8188eu/hal/usb_halinit.c:2022:3: error: expected expression > > u8 tmp; > > ^ > > 1 error generated. > > > > Kconfig warnings: (for reference only) > > WARNING: unmet direct dependencies detected for QCOM_SCM > > Depends on (ARM || ARM64) && HAVE_ARM_SMCCC > > Selected by > > - ARM_QCOM_SPM_CPUIDLE && CPU_IDLE && (ARM || ARM64) && (ARCH_QCOM || COMPILE_TEST && !ARM64 && MMU > > > > > > We need to fix Kconfig, ok, I will take a look at it later > This is not related to your patch. Ignore it. > > vim +2022 drivers/staging/r8188eu/hal/usb_halinit.c > > > > > 2020 case HW_VAR_BCN_VALID: > > 2021 /* BCN_VALID, BIT(16) of REG_TDECTRL = BIT(0) of REG_TDECTRL+2, write 1 to clear, Clear by sw */ > > > 2022 u8 tmp; > > Hm, I don't know anything about ARM compilers, so should I wrap this code > block with {}? Yep. > > My local gcc 11.1.1 (x86_64) does not produce any warnings/errors > You should figure out whats up with that because it shouldn't compile with the gcc options that the kernel uses. regards, dan carpenter
next prev parent reply other threads:[~2021-08-25 12:52 UTC|newest] Thread overview: 140+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-08-20 17:07 [PATCH RFC 0/3] staging: r8188eu: avoid uninit value bugs Pavel Skripkin 2021-08-20 17:07 ` [PATCH RFC 1/3] staging: r8188eu: add proper rtw_read* error handling Pavel Skripkin 2021-08-20 20:54 ` kernel test robot 2021-08-20 21:50 ` Pavel Skripkin 2021-08-20 23:41 ` Phillip Potter 2021-08-20 23:41 ` Phillip Potter 2021-08-21 5:55 ` Fabio M. De Francesco 2021-08-21 10:35 ` Pavel Skripkin 2021-08-21 12:11 ` Fabio M. De Francesco 2021-08-20 17:07 ` [PATCH RFC 2/3] staging: r8188eu: add error handling to ReadFuse Pavel Skripkin 2021-08-20 23:51 ` Phillip Potter 2021-08-20 23:51 ` Phillip Potter 2021-08-21 3:59 ` Fabio M. De Francesco 2021-08-20 17:07 ` [PATCH RFC 3/3] staging: r8188eu: add error argument to read_macreg Pavel Skripkin 2021-08-20 23:18 ` Phillip Potter 2021-08-20 23:18 ` Phillip Potter 2021-08-21 10:38 ` Pavel Skripkin 2021-08-20 23:12 ` [PATCH RFC 0/3] staging: r8188eu: avoid uninit value bugs Phillip Potter 2021-08-20 23:12 ` Phillip Potter 2021-08-21 10:42 ` Pavel Skripkin 2021-08-22 9:53 ` Fabio M. De Francesco 2021-08-22 10:09 ` Pavel Skripkin 2021-08-22 10:59 ` Fabio M. De Francesco 2021-08-22 11:34 ` Fabio M. De Francesco 2021-08-22 12:10 ` Pavel Skripkin 2021-08-22 12:39 ` Greg KH 2021-08-22 12:50 ` Pavel Skripkin 2021-08-22 13:06 ` Greg KH 2021-08-22 13:21 ` Fabio M. De Francesco 2021-08-22 13:30 ` Greg KH 2021-08-22 13:31 ` Pavel Skripkin 2021-08-22 14:35 ` [PATCH RFC v2 0/6] " Pavel Skripkin 2021-08-22 14:35 ` [PATCH RFC v2 1/6] staging: r8188eu: remove {read,write}_macreg Pavel Skripkin 2021-08-22 14:35 ` [PATCH RFC v2 2/6] staging: r8188eu: add helper macro for printing registers Pavel Skripkin 2021-08-22 14:35 ` [PATCH RFC v2 3/6] staging: r8188eu: add error handling of rtw_read8 Pavel Skripkin 2021-08-22 16:49 ` kernel test robot 2021-08-22 14:35 ` [PATCH RFC v2 4/6] staging: r8188eu: add error handling of rtw_read16 Pavel Skripkin 2021-08-22 14:36 ` [PATCH RFC v2 5/6] staging: r8188eu: add error handling of rtw_read32 Pavel Skripkin 2021-08-23 23:33 ` Phillip Potter 2021-08-23 23:33 ` Phillip Potter 2021-08-24 0:10 ` Fabio M. De Francesco 2021-08-24 6:40 ` Pavel Skripkin 2021-08-24 8:38 ` Fabio M. De Francesco 2021-08-24 8:47 ` Pavel Skripkin 2021-08-24 8:53 ` Pavel Skripkin 2021-08-24 9:46 ` Fabio M. De Francesco 2021-08-24 22:10 ` Phillip Potter 2021-08-24 22:10 ` Phillip Potter 2021-08-24 22:07 ` Phillip Potter 2021-08-24 22:07 ` Phillip Potter 2021-08-24 6:53 ` Pavel Skripkin 2021-08-24 7:25 ` [PATCH v3 0/6] staging: r8188eu: avoid uninit value bugs Pavel Skripkin 2021-08-24 7:27 ` [PATCH v3 1/6] staging: r8188eu: remove {read,write}_macreg Pavel Skripkin 2021-08-26 10:39 ` Greg KH 2021-08-26 10:40 ` Greg KH 2021-08-24 7:27 ` [PATCH v3 2/6] staging: r8188eu: add helper macro for printing registers Pavel Skripkin 2021-08-26 10:37 ` Greg KH 2021-08-24 7:27 ` [PATCH v3 3/6] staging: r8188eu: add error handling of rtw_read8 Pavel Skripkin 2021-08-25 12:05 ` kernel test robot 2021-08-25 12:05 ` kernel test robot 2021-08-25 12:17 ` Pavel Skripkin 2021-08-25 12:17 ` Pavel Skripkin 2021-08-25 12:51 ` Dan Carpenter [this message] 2021-08-25 12:51 ` Dan Carpenter 2021-08-25 13:02 ` Pavel Skripkin 2021-08-25 13:02 ` Pavel Skripkin 2021-08-25 13:34 ` Dan Carpenter 2021-08-25 13:34 ` Dan Carpenter 2021-08-25 13:44 ` Pavel Skripkin 2021-08-25 13:44 ` Pavel Skripkin 2021-08-25 17:11 ` Nick Desaulniers 2021-08-25 17:11 ` Nick Desaulniers 2021-08-25 17:11 ` Nick Desaulniers 2021-08-26 11:08 ` Dan Carpenter 2021-08-26 11:08 ` Dan Carpenter 2021-08-25 23:45 ` Fabio M. De Francesco 2021-08-26 5:13 ` Pavel Skripkin 2021-08-26 8:21 ` David Laight 2021-08-26 8:27 ` Pavel Skripkin 2021-08-26 10:19 ` David Laight 2021-08-26 11:21 ` Dan Carpenter 2021-08-27 8:14 ` David Laight 2021-08-27 8:22 ` Pavel Skripkin 2021-08-27 9:07 ` Dan Carpenter 2021-08-27 9:16 ` Pavel Skripkin 2021-08-27 9:23 ` Dan Carpenter 2021-08-30 11:21 ` kernel test robot 2021-08-30 11:21 ` kernel test robot 2021-08-24 7:27 ` [PATCH v3 4/6] staging: r8188eu: add error handling of rtw_read16 Pavel Skripkin 2021-08-25 4:35 ` Fabio M. De Francesco 2021-08-25 8:22 ` Pavel Skripkin 2021-08-25 9:48 ` Fabio M. De Francesco 2021-08-25 9:55 ` Pavel Skripkin 2021-08-25 10:06 ` Dan Carpenter 2021-08-25 10:13 ` Pavel Skripkin 2021-08-25 10:38 ` Dan Carpenter 2021-08-25 10:41 ` Pavel Skripkin 2021-08-25 11:06 ` Fabio M. De Francesco 2021-08-25 11:11 ` Fabio M. De Francesco 2021-08-25 11:31 ` Dan Carpenter 2021-08-25 12:11 ` Fabio M. De Francesco 2021-08-25 10:51 ` Fabio M. De Francesco 2021-08-26 10:50 ` Greg KH 2021-08-26 10:58 ` Pavel Skripkin 2021-08-24 7:27 ` [PATCH v3 5/6] staging: r8188eu: add error handling of rtw_read32 Pavel Skripkin 2021-08-25 4:40 ` Fabio M. De Francesco 2021-08-26 8:51 ` David Laight 2021-08-26 9:22 ` Pavel Skripkin 2021-08-26 9:27 ` Pavel Skripkin 2021-08-26 10:22 ` David Laight 2021-08-26 10:55 ` Pavel Skripkin 2021-08-26 10:59 ` David Laight 2021-08-26 20:03 ` Pavel Skripkin 2021-08-27 7:12 ` gregkh 2021-08-27 7:16 ` Pavel Skripkin 2021-08-24 7:27 ` [PATCH v3 6/6] staging: r8188eu: make ReadEFuse return an int Pavel Skripkin 2021-08-25 10:13 ` [PATCH v3 0/6] staging: r8188eu: avoid uninit value bugs Fabio M. De Francesco 2021-08-27 7:49 ` Kari Argillander 2021-08-27 7:52 ` Pavel Skripkin 2021-08-24 6:58 ` [PATCH RFC v2 5/6] staging: r8188eu: add error handling of rtw_read32 Dan Carpenter 2021-08-24 7:01 ` Pavel Skripkin 2021-08-24 15:07 ` Fabio M. De Francesco 2021-08-22 14:36 ` [PATCH RFC v2 6/6] staging: r8188eu: make ReadEFuse return an int Pavel Skripkin 2021-08-22 15:30 ` [PATCH RFC v2 0/6] staging: r8188eu: avoid uninit value bugs Pavel Skripkin 2021-08-22 16:05 ` Michael Straube 2021-08-22 16:26 ` Pavel Skripkin 2021-08-22 23:52 ` Phillip Potter 2021-08-22 23:52 ` Phillip Potter 2021-08-22 17:36 ` Fabio M. De Francesco 2021-08-22 17:38 ` Pavel Skripkin 2021-08-22 20:06 ` Fabio M. De Francesco 2021-08-22 20:19 ` Pavel Skripkin 2021-08-23 0:12 ` Phillip Potter 2021-08-23 0:12 ` Phillip Potter 2021-08-23 6:38 ` Pavel Skripkin 2021-08-23 6:44 ` Pavel Skripkin 2021-08-22 16:03 ` [PATCH RFC 0/3] " Fabio M. De Francesco 2021-08-22 16:15 ` Pavel Skripkin 2021-08-22 15:04 ` Phillip Potter 2021-08-22 15:04 ` Phillip Potter
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=20210825125159.GU1931@kadam \ --to=dan.carpenter@oracle.com \ --cc=Larry.Finger@lwfinger.net \ --cc=clang-built-linux@googlegroups.com \ --cc=fmdefrancesco@gmail.com \ --cc=gregkh@linuxfoundation.org \ --cc=kbuild-all@lists.01.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-staging@lists.linux.dev \ --cc=lkp@intel.com \ --cc=paskripkin@gmail.com \ --cc=phil@philpotter.co.uk \ --cc=straube.linux@gmail.com \ /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.