From: kernel test robot <lkp@intel.com> To: Andrea Merello <andrea.merello@gmail.com>, jic23@kernel.org, mchehab+huawei@kernel.org, linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org Cc: kbuild-all@lists.01.org, lars@metafoo.de, robh+dt@kernel.org, andy.shevchenko@gmail.com, matt.ranostay@konsulko.com, ardeleanalex@gmail.com Subject: Re: [v2 09/10] iio: imu: add BNO055 serdev driver Date: Fri, 29 Oct 2021 15:09:37 +0800 [thread overview] Message-ID: <202110291552.iKKrAa3D-lkp@intel.com> (raw) In-Reply-To: <20211028101840.24632-10-andrea.merello@gmail.com> [-- Attachment #1: Type: text/plain, Size: 6416 bytes --] Hi Andrea, Thank you for the patch! Yet something to improve: [auto build test ERROR on linux/master] [also build test ERROR on linus/master v5.15-rc7] [cannot apply to jic23-iio/togreg next-20211028] [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/Andrea-Merello/utils_macro-introduce-find_closest_unsorted/20211028-191851 base: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 2f111a6fd5b5297b4e92f53798ca086f7c7d33a4 config: arc-allyesconfig (attached as .config) compiler: arceb-elf-gcc (GCC) 11.2.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/185166ebe83b933e30af55d4dc7972db6f9a8fb8 git remote add linux-review https://github.com/0day-ci/linux git fetch --no-tags linux-review Andrea-Merello/utils_macro-introduce-find_closest_unsorted/20211028-191851 git checkout 185166ebe83b933e30af55d4dc7972db6f9a8fb8 # save the attached .config to linux build tree COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross ARCH=arc 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/iio/imu/bno055/bno055.c:234:5: error: no previous prototype for 'bno055_calibration_load' [-Werror=missing-prototypes] 234 | int bno055_calibration_load(struct bno055_priv *priv, const struct firmware *fw) | ^~~~~~~~~~~~~~~~~~~~~~~ drivers/iio/imu/bno055/bno055.c: In function 'bno055_fusion_enable_store': >> drivers/iio/imu/bno055/bno055.c:917:13: error: variable 'ret' set but not used [-Werror=unused-but-set-variable] 917 | int ret = 0; | ^~~ drivers/iio/imu/bno055/bno055.c: At top level: >> drivers/iio/imu/bno055/bno055.c:1130:5: error: no previous prototype for 'bno055_debugfs_reg_access' [-Werror=missing-prototypes] 1130 | int bno055_debugfs_reg_access(struct iio_dev *iio_dev, unsigned int reg, | ^~~~~~~~~~~~~~~~~~~~~~~~~ drivers/iio/imu/bno055/bno055.c: In function 'bno055_trigger_handler': >> drivers/iio/imu/bno055/bno055.c:1330:9: error: implicit declaration of function 'for_each_set_bitrange'; did you mean 'for_each_set_bit'? [-Werror=implicit-function-declaration] 1330 | for_each_set_bitrange(start, end, iio_dev->active_scan_mask, | ^~~~~~~~~~~~~~~~~~~~~ | for_each_set_bit >> drivers/iio/imu/bno055/bno055.c:1331:51: error: expected ';' before '{' token 1331 | iio_dev->masklength) { | ^~ | ; >> drivers/iio/imu/bno055/bno055.c:1364:1: error: label 'done' defined but not used [-Werror=unused-label] 1364 | done: | ^~~~ >> drivers/iio/imu/bno055/bno055.c:1327:13: error: unused variable 'ret' [-Werror=unused-variable] 1327 | int ret; | ^~~ >> drivers/iio/imu/bno055/bno055.c:1326:13: error: unused variable 'quat' [-Werror=unused-variable] 1326 | int quat; | ^~~~ >> drivers/iio/imu/bno055/bno055.c:1325:14: error: unused variable 'thr_hit' [-Werror=unused-variable] 1325 | bool thr_hit; | ^~~~~~~ >> drivers/iio/imu/bno055/bno055.c:1324:13: error: unused variable 'buf_idx' [-Werror=unused-variable] 1324 | int buf_idx = 0; | ^~~~~~~ >> drivers/iio/imu/bno055/bno055.c:1323:23: error: unused variable 'mask' [-Werror=unused-variable] 1323 | unsigned long mask; | ^~~~ >> drivers/iio/imu/bno055/bno055.c:1322:14: error: unused variable 'first' [-Werror=unused-variable] 1322 | bool first = true; | ^~~~~ >> drivers/iio/imu/bno055/bno055.c:1321:14: error: unused variable 'xfer_pending' [-Werror=unused-variable] 1321 | bool xfer_pending = false; | ^~~~~~~~~~~~ >> drivers/iio/imu/bno055/bno055.c:1320:37: error: unused variable 'prev_end' [-Werror=unused-variable] 1320 | int xfer_start, start, end, prev_end; | ^~~~~~~~ >> drivers/iio/imu/bno055/bno055.c:1320:13: error: unused variable 'xfer_start' [-Werror=unused-variable] 1320 | int xfer_start, start, end, prev_end; | ^~~~~~~~~~ At top level: >> drivers/iio/imu/bno055/bno055.c:1262:12: error: 'bno055_scan_xfer' defined but not used [-Werror=unused-function] 1262 | static int bno055_scan_xfer(struct bno055_priv *priv, | ^~~~~~~~~~~~~~~~ cc1: all warnings being treated as errors vim +/bno055_calibration_load +234 drivers/iio/imu/bno055/bno055.c 734efd9783b7759 Andrea Merello 2021-10-28 232 734efd9783b7759 Andrea Merello 2021-10-28 233 /* must be called in configuration mode */ 734efd9783b7759 Andrea Merello 2021-10-28 @234 int bno055_calibration_load(struct bno055_priv *priv, const struct firmware *fw) 734efd9783b7759 Andrea Merello 2021-10-28 235 { 734efd9783b7759 Andrea Merello 2021-10-28 236 if (fw->size != BNO055_CALDATA_LEN) { 734efd9783b7759 Andrea Merello 2021-10-28 237 dev_dbg(priv->dev, "Invalid calibration file size %d (expected %d)", 734efd9783b7759 Andrea Merello 2021-10-28 238 fw->size, BNO055_CALDATA_LEN); 734efd9783b7759 Andrea Merello 2021-10-28 239 return -EINVAL; 734efd9783b7759 Andrea Merello 2021-10-28 240 } 734efd9783b7759 Andrea Merello 2021-10-28 241 734efd9783b7759 Andrea Merello 2021-10-28 242 dev_dbg(priv->dev, "loading cal data: %*ph", BNO055_CALDATA_LEN, fw->data); 734efd9783b7759 Andrea Merello 2021-10-28 243 return regmap_bulk_write(priv->regmap, BNO055_CALDATA_START, 734efd9783b7759 Andrea Merello 2021-10-28 244 fw->data, BNO055_CALDATA_LEN); 734efd9783b7759 Andrea Merello 2021-10-28 245 } 734efd9783b7759 Andrea Merello 2021-10-28 246 --- 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: 69504 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com> To: kbuild-all@lists.01.org Subject: Re: [v2 09/10] iio: imu: add BNO055 serdev driver Date: Fri, 29 Oct 2021 15:09:37 +0800 [thread overview] Message-ID: <202110291552.iKKrAa3D-lkp@intel.com> (raw) In-Reply-To: <20211028101840.24632-10-andrea.merello@gmail.com> [-- Attachment #1: Type: text/plain, Size: 6526 bytes --] Hi Andrea, Thank you for the patch! Yet something to improve: [auto build test ERROR on linux/master] [also build test ERROR on linus/master v5.15-rc7] [cannot apply to jic23-iio/togreg next-20211028] [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/Andrea-Merello/utils_macro-introduce-find_closest_unsorted/20211028-191851 base: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 2f111a6fd5b5297b4e92f53798ca086f7c7d33a4 config: arc-allyesconfig (attached as .config) compiler: arceb-elf-gcc (GCC) 11.2.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/185166ebe83b933e30af55d4dc7972db6f9a8fb8 git remote add linux-review https://github.com/0day-ci/linux git fetch --no-tags linux-review Andrea-Merello/utils_macro-introduce-find_closest_unsorted/20211028-191851 git checkout 185166ebe83b933e30af55d4dc7972db6f9a8fb8 # save the attached .config to linux build tree COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross ARCH=arc 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/iio/imu/bno055/bno055.c:234:5: error: no previous prototype for 'bno055_calibration_load' [-Werror=missing-prototypes] 234 | int bno055_calibration_load(struct bno055_priv *priv, const struct firmware *fw) | ^~~~~~~~~~~~~~~~~~~~~~~ drivers/iio/imu/bno055/bno055.c: In function 'bno055_fusion_enable_store': >> drivers/iio/imu/bno055/bno055.c:917:13: error: variable 'ret' set but not used [-Werror=unused-but-set-variable] 917 | int ret = 0; | ^~~ drivers/iio/imu/bno055/bno055.c: At top level: >> drivers/iio/imu/bno055/bno055.c:1130:5: error: no previous prototype for 'bno055_debugfs_reg_access' [-Werror=missing-prototypes] 1130 | int bno055_debugfs_reg_access(struct iio_dev *iio_dev, unsigned int reg, | ^~~~~~~~~~~~~~~~~~~~~~~~~ drivers/iio/imu/bno055/bno055.c: In function 'bno055_trigger_handler': >> drivers/iio/imu/bno055/bno055.c:1330:9: error: implicit declaration of function 'for_each_set_bitrange'; did you mean 'for_each_set_bit'? [-Werror=implicit-function-declaration] 1330 | for_each_set_bitrange(start, end, iio_dev->active_scan_mask, | ^~~~~~~~~~~~~~~~~~~~~ | for_each_set_bit >> drivers/iio/imu/bno055/bno055.c:1331:51: error: expected ';' before '{' token 1331 | iio_dev->masklength) { | ^~ | ; >> drivers/iio/imu/bno055/bno055.c:1364:1: error: label 'done' defined but not used [-Werror=unused-label] 1364 | done: | ^~~~ >> drivers/iio/imu/bno055/bno055.c:1327:13: error: unused variable 'ret' [-Werror=unused-variable] 1327 | int ret; | ^~~ >> drivers/iio/imu/bno055/bno055.c:1326:13: error: unused variable 'quat' [-Werror=unused-variable] 1326 | int quat; | ^~~~ >> drivers/iio/imu/bno055/bno055.c:1325:14: error: unused variable 'thr_hit' [-Werror=unused-variable] 1325 | bool thr_hit; | ^~~~~~~ >> drivers/iio/imu/bno055/bno055.c:1324:13: error: unused variable 'buf_idx' [-Werror=unused-variable] 1324 | int buf_idx = 0; | ^~~~~~~ >> drivers/iio/imu/bno055/bno055.c:1323:23: error: unused variable 'mask' [-Werror=unused-variable] 1323 | unsigned long mask; | ^~~~ >> drivers/iio/imu/bno055/bno055.c:1322:14: error: unused variable 'first' [-Werror=unused-variable] 1322 | bool first = true; | ^~~~~ >> drivers/iio/imu/bno055/bno055.c:1321:14: error: unused variable 'xfer_pending' [-Werror=unused-variable] 1321 | bool xfer_pending = false; | ^~~~~~~~~~~~ >> drivers/iio/imu/bno055/bno055.c:1320:37: error: unused variable 'prev_end' [-Werror=unused-variable] 1320 | int xfer_start, start, end, prev_end; | ^~~~~~~~ >> drivers/iio/imu/bno055/bno055.c:1320:13: error: unused variable 'xfer_start' [-Werror=unused-variable] 1320 | int xfer_start, start, end, prev_end; | ^~~~~~~~~~ At top level: >> drivers/iio/imu/bno055/bno055.c:1262:12: error: 'bno055_scan_xfer' defined but not used [-Werror=unused-function] 1262 | static int bno055_scan_xfer(struct bno055_priv *priv, | ^~~~~~~~~~~~~~~~ cc1: all warnings being treated as errors vim +/bno055_calibration_load +234 drivers/iio/imu/bno055/bno055.c 734efd9783b7759 Andrea Merello 2021-10-28 232 734efd9783b7759 Andrea Merello 2021-10-28 233 /* must be called in configuration mode */ 734efd9783b7759 Andrea Merello 2021-10-28 @234 int bno055_calibration_load(struct bno055_priv *priv, const struct firmware *fw) 734efd9783b7759 Andrea Merello 2021-10-28 235 { 734efd9783b7759 Andrea Merello 2021-10-28 236 if (fw->size != BNO055_CALDATA_LEN) { 734efd9783b7759 Andrea Merello 2021-10-28 237 dev_dbg(priv->dev, "Invalid calibration file size %d (expected %d)", 734efd9783b7759 Andrea Merello 2021-10-28 238 fw->size, BNO055_CALDATA_LEN); 734efd9783b7759 Andrea Merello 2021-10-28 239 return -EINVAL; 734efd9783b7759 Andrea Merello 2021-10-28 240 } 734efd9783b7759 Andrea Merello 2021-10-28 241 734efd9783b7759 Andrea Merello 2021-10-28 242 dev_dbg(priv->dev, "loading cal data: %*ph", BNO055_CALDATA_LEN, fw->data); 734efd9783b7759 Andrea Merello 2021-10-28 243 return regmap_bulk_write(priv->regmap, BNO055_CALDATA_START, 734efd9783b7759 Andrea Merello 2021-10-28 244 fw->data, BNO055_CALDATA_LEN); 734efd9783b7759 Andrea Merello 2021-10-28 245 } 734efd9783b7759 Andrea Merello 2021-10-28 246 --- 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: 69504 bytes --]
next prev parent reply other threads:[~2021-10-29 7:09 UTC|newest] Thread overview: 94+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-07-15 14:17 [PATCH 0/4] Add support for Bosch BNO055 IMU Andrea Merello 2021-07-15 14:17 ` [PATCH 1/4] iio: add modifiers for linear acceleration Andrea Merello 2021-07-17 14:32 ` Jonathan Cameron 2021-07-15 14:17 ` [PATCH 2/4] iio: imu: add Bosch Sensortec BNO055 core driver Andrea Merello 2021-07-15 16:49 ` Andy Shevchenko 2021-07-16 9:19 ` Andrea Merello 2021-07-16 12:39 ` Andy Shevchenko 2021-07-19 7:12 ` Andrea Merello 2021-07-19 10:33 ` Andrea Merello 2021-07-19 9:02 ` Andrea Merello 2021-07-19 11:48 ` Andy Shevchenko 2021-07-19 13:13 ` Andrea Merello 2021-07-16 7:24 ` Alexandru Ardelean 2021-07-16 9:49 ` Andrea Merello 2021-07-17 15:32 ` Jonathan Cameron 2021-07-19 8:30 ` Andrea Merello 2021-07-24 17:08 ` Jonathan Cameron 2021-07-26 14:36 ` Andrea Merello 2021-07-31 18:01 ` Jonathan Cameron 2021-08-04 10:06 ` Andrea Merello 2021-08-04 16:50 ` Jonathan Cameron 2021-08-04 19:27 ` Andy Shevchenko 2021-07-15 14:17 ` [PATCH 3/4] dt-bindings: iio: imu: add bosch BNO055 serdev driver bindings Andrea Merello 2021-07-17 15:39 ` Jonathan Cameron 2021-07-19 8:44 ` Andrea Merello 2021-07-15 14:17 ` [PATCH 4/4] iio: imu: add BNO055 serdev driver Andrea Merello 2021-07-15 17:08 ` kernel test robot 2021-07-15 17:08 ` kernel test robot 2021-07-15 18:14 ` kernel test robot 2021-07-15 18:14 ` kernel test robot 2021-07-17 15:50 ` Jonathan Cameron 2021-07-19 8:49 ` Andrea Merello 2021-07-19 11:55 ` Andy Shevchenko 2021-07-19 12:59 ` Andrea Merello 2021-07-19 14:15 ` Andy Shevchenko 2021-07-19 15:07 ` Andrea Merello 2021-10-28 10:18 ` [v2 00/10] Add support for Bosch BNO055 IMU Andrea Merello 2021-10-28 10:18 ` [v2 01/10] utils_macro: introduce find_closest_unsorted() Andrea Merello 2021-10-28 10:25 ` Andy Shevchenko 2021-11-08 11:05 ` Andrea Merello 2021-10-28 10:18 ` [v2 02/10] iio: document linear acceleration modifiers Andrea Merello 2021-10-28 10:31 ` Andy Shevchenko 2021-11-09 7:48 ` Andrea Merello 2021-10-28 10:40 ` Jonathan Cameron 2021-11-09 8:00 ` Andrea Merello 2021-11-09 17:00 ` Jonathan Cameron 2021-10-28 10:18 ` [v2 03/10] iio: document euler angles modifiers Andrea Merello 2021-10-28 10:33 ` Andy Shevchenko 2021-10-28 10:41 ` Jonathan Cameron 2021-11-09 8:15 ` Andrea Merello 2021-11-09 17:03 ` Jonathan Cameron 2021-10-28 10:18 ` [v2 04/10] iio: add modifiers for linear acceleration Andrea Merello 2021-10-28 10:45 ` Jonathan Cameron 2021-11-09 9:58 ` Andrea Merello 2021-11-09 17:05 ` Jonathan Cameron 2021-10-28 10:18 ` [v2 05/10] iio: add modifers for pitch, yaw, roll Andrea Merello 2021-10-28 10:47 ` Jonathan Cameron 2021-10-28 10:18 ` [v2 06/10] iio: document bno055 private sysfs attributes Andrea Merello 2021-10-28 11:04 ` Jonathan Cameron 2021-11-09 10:22 ` Andrea Merello 2021-11-14 16:20 ` Jonathan Cameron 2022-01-04 11:42 ` Andrea Merello 2022-01-15 15:27 ` Jonathan Cameron 2022-01-17 9:37 ` Andrea Merello 2022-01-22 18:08 ` Jonathan Cameron 2021-10-28 10:18 ` [v2 07/10] iio: imu: add Bosch Sensortec BNO055 core driver Andrea Merello 2021-10-28 13:31 ` Jonathan Cameron 2021-11-09 11:52 ` Andrea Merello 2021-11-14 16:33 ` Jonathan Cameron 2021-10-28 10:18 ` [v2 08/10] dt-bindings: iio: imu: add documentation for Bosch BNO055 bindings Andrea Merello 2021-10-28 12:25 ` Rob Herring 2021-10-28 10:18 ` [v2 09/10] iio: imu: add BNO055 serdev driver Andrea Merello 2021-10-28 12:31 ` Jonathan Cameron 2021-11-09 15:33 ` Andrea Merello 2021-11-14 16:37 ` Jonathan Cameron 2021-10-29 7:09 ` kernel test robot [this message] 2021-10-29 7:09 ` kernel test robot 2021-10-29 12:59 ` kernel test robot 2021-10-29 12:59 ` kernel test robot 2021-10-28 10:18 ` [v2 10/10] iio: imu: add BNO055 I2C driver Andrea Merello 2021-10-28 11:10 ` Jonathan Cameron 2021-11-11 10:12 ` Andrea Merello 2021-11-14 16:38 ` Jonathan Cameron 2021-10-28 22:04 ` Randy Dunlap 2021-11-09 11:56 ` Andrea Merello 2021-11-09 15:47 ` Randy Dunlap 2021-11-09 18:21 ` Joe Perches 2021-11-09 19:11 ` Randy Dunlap 2021-11-09 20:46 ` Joe Perches 2021-11-09 21:20 ` Randy Dunlap 2021-10-29 13:30 ` kernel test robot 2021-10-29 13:30 ` kernel test robot 2021-10-28 10:35 ` [v2 00/10] Add support for Bosch BNO055 IMU Jonathan Cameron 2021-10-28 10:33 ` Andy Shevchenko
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=202110291552.iKKrAa3D-lkp@intel.com \ --to=lkp@intel.com \ --cc=andrea.merello@gmail.com \ --cc=andy.shevchenko@gmail.com \ --cc=ardeleanalex@gmail.com \ --cc=devicetree@vger.kernel.org \ --cc=jic23@kernel.org \ --cc=kbuild-all@lists.01.org \ --cc=lars@metafoo.de \ --cc=linux-iio@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=matt.ranostay@konsulko.com \ --cc=mchehab+huawei@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.