From: Jonathan Cameron <jic23@kernel.org> To: kernel test robot <lkp@intel.com> Cc: Jagath Jog J <jagathjog1996@gmail.com>, dan@dlrobertson.com, andy.shevchenko@gmail.com, llvm@lists.linux.dev, kbuild-all@lists.01.org, linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v4 9/9] iio: accel: bma400: Add support for activity and inactivity events Date: Sun, 24 Apr 2022 17:20:02 +0100 [thread overview] Message-ID: <20220424172002.7dd48dfa@jic23-huawei> (raw) In-Reply-To: <202204211211.febbJ6fy-lkp@intel.com> On Thu, 21 Apr 2022 14:45:05 +0800 kernel test robot <lkp@intel.com> wrote: > Hi Jagath, > > Thank you for the patch! Perhaps something to improve: > > [auto build test WARNING on jic23-iio/togreg] > [also build test WARNING on linus/master v5.18-rc3 next-20220420] > [cannot apply to linux/master] > [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/intel-lab-lkp/linux/commits/Jagath-Jog-J/iio-accel-bma400-Add-buffer-step-and-activity-inactivity/20220421-051244 > base: https://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git togreg > config: mips-randconfig-r031-20220420 (https://download.01.org/0day-ci/archive/20220421/202204211211.febbJ6fy-lkp@intel.com/config) > compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project bac6cd5bf85669e3376610cfc4c4f9ca015e7b9b) > 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 mips cross compiling tool for clang build > # apt-get install binutils-mips-linux-gnu > # https://github.com/intel-lab-lkp/linux/commit/b33d9910aa7588ec8db7c1694dbc03c3ed200ebb > git remote add linux-review https://github.com/intel-lab-lkp/linux > git fetch --no-tags linux-review Jagath-Jog-J/iio-accel-bma400-Add-buffer-step-and-activity-inactivity/20220421-051244 > git checkout b33d9910aa7588ec8db7c1694dbc03c3ed200ebb > # save the config file > mkdir build_dir && cp config build_dir/.config > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=mips SHELL=/bin/bash drivers/iio/accel/ > > 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/iio/accel/bma400_core.c:1047:17: warning: result of comparison of constant 18446744073709551615 with expression of type 'typeof (_Generic((msk), char: (unsigned char)0, unsigned char: (unsigned char)0, signed char: (unsigned char)0, unsigned short: (unsigned short)0, short: (unsigned short)0, unsigned int: (unsigned int)0, int: (unsigned int)0, unsigned long: (unsigned long)0, long: (unsigned long)0, unsigned long long: (unsigned long long)0, long long: (unsigned long long)0, default: (msk)))' (aka 'unsigned int') is always false [-Wtautological-constant-out-of-range-compare] > field_value = FIELD_PREP(msk, state); > ^~~~~~~~~~~~~~~~~~~~~~ > include/linux/bitfield.h:114:3: note: expanded from macro 'FIELD_PREP' > __BF_FIELD_CHECK(_mask, 0ULL, _val, "FIELD_PREP: "); \ > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > include/linux/bitfield.h:71:53: note: expanded from macro '__BF_FIELD_CHECK' > BUILD_BUG_ON_MSG(__bf_cast_unsigned(_mask, _mask) > \ > ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~ > include/linux/build_bug.h:39:58: note: expanded from macro 'BUILD_BUG_ON_MSG' > #define BUILD_BUG_ON_MSG(cond, msg) compiletime_assert(!(cond), msg) > ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~ > include/linux/compiler_types.h:352:22: note: expanded from macro 'compiletime_assert' > _compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__) > ~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > include/linux/compiler_types.h:340:23: note: expanded from macro '_compiletime_assert' > __compiletime_assert(condition, msg, prefix, suffix) > ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > include/linux/compiler_types.h:332:9: note: expanded from macro '__compiletime_assert' > if (!(condition)) \ > ^~~~~~~~~ > drivers/iio/accel/bma400_core.c:1053:17: warning: result of comparison of constant 18446744073709551615 with expression of type 'typeof (_Generic((msk), char: (unsigned char)0, unsigned char: (unsigned char)0, signed char: (unsigned char)0, unsigned short: (unsigned short)0, short: (unsigned short)0, unsigned int: (unsigned int)0, int: (unsigned int)0, unsigned long: (unsigned long)0, long: (unsigned long)0, unsigned long long: (unsigned long long)0, long long: (unsigned long long)0, default: (msk)))' (aka 'unsigned int') is always false [-Wtautological-constant-out-of-range-compare] > field_value = FIELD_PREP(msk, state); > ^~~~~~~~~~~~~~~~~~~~~~ > include/linux/bitfield.h:114:3: note: expanded from macro 'FIELD_PREP' > __BF_FIELD_CHECK(_mask, 0ULL, _val, "FIELD_PREP: "); \ > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > include/linux/bitfield.h:71:53: note: expanded from macro '__BF_FIELD_CHECK' > BUILD_BUG_ON_MSG(__bf_cast_unsigned(_mask, _mask) > \ > ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~ > include/linux/build_bug.h:39:58: note: expanded from macro 'BUILD_BUG_ON_MSG' > #define BUILD_BUG_ON_MSG(cond, msg) compiletime_assert(!(cond), msg) > ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~ > include/linux/compiler_types.h:352:22: note: expanded from macro 'compiletime_assert' > _compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__) > ~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > include/linux/compiler_types.h:340:23: note: expanded from macro '_compiletime_assert' > __compiletime_assert(condition, msg, prefix, suffix) > ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > include/linux/compiler_types.h:332:9: note: expanded from macro '__compiletime_assert' > if (!(condition)) \ > ^~~~~~~~~ > 2 warnings generated. > > > vim +1047 drivers/iio/accel/bma400_core.c > > 1035 > 1036 static int bma400_activity_event_en(struct bma400_data *data, > 1037 enum iio_event_direction dir, > 1038 int state) > 1039 { > 1040 int ret, reg, msk, value, field_value; > 1041 > 1042 switch (dir) { > 1043 case IIO_EV_DIR_RISING: > 1044 reg = BMA400_GEN1INT_CONFIG0; > 1045 msk = BMA400_INT_GEN1_MSK; > 1046 value = 2; > > 1047 field_value = FIELD_PREP(msk, state); Ah. Will need to clamp state to 0/1 field_value = FIELD_PREP(msk, state ? 1 : 0); perhaps? > 1048 break; > 1049 case IIO_EV_DIR_FALLING: > 1050 reg = BMA400_GEN2INT_CONFIG0; > 1051 msk = BMA400_INT_GEN2_MSK; > 1052 value = 0; > 1053 field_value = FIELD_PREP(msk, state); > 1054 break; > 1055 default: > 1056 return -EINVAL; > 1057 } > 1058 > 1059 /* Enabling all axis for interrupt evaluation */ > 1060 ret = regmap_write(data->regmap, reg, 0xF8); > 1061 if (ret) > 1062 return ret; > 1063 > 1064 /* OR combination of all axis for interrupt evaluation */ > 1065 ret = regmap_write(data->regmap, reg + BMA400_GEN_CONFIG1_OFF, value); > 1066 if (ret) > 1067 return ret; > 1068 > 1069 /* Initial value to avoid interrupts while enabling*/ > 1070 ret = regmap_write(data->regmap, reg + BMA400_GEN_CONFIG2_OFF, 0x0A); > 1071 if (ret) > 1072 return ret; > 1073 > 1074 /* Initial duration value to avoid interrupts while enabling*/ > 1075 ret = regmap_write(data->regmap, reg + BMA400_GEN_CONFIG31_OFF, 0x0F); > 1076 if (ret) > 1077 return ret; > 1078 > 1079 ret = regmap_update_bits(data->regmap, BMA400_INT1_MAP_REG, msk, > 1080 field_value); > 1081 if (ret) > 1082 return ret; > 1083 > 1084 ret = regmap_update_bits(data->regmap, BMA400_INT_CONFIG0_REG, msk, > 1085 field_value); > 1086 if (ret) > 1087 return ret; > 1088 > 1089 set_mask_bits(&data->generic_event_en, msk, field_value); > 1090 return 0; > 1091 } > 1092 >
WARNING: multiple messages have this Message-ID (diff)
From: Jonathan Cameron <jic23@kernel.org> To: kbuild-all@lists.01.org Subject: Re: [PATCH v4 9/9] iio: accel: bma400: Add support for activity and inactivity events Date: Sun, 24 Apr 2022 17:20:02 +0100 [thread overview] Message-ID: <20220424172002.7dd48dfa@jic23-huawei> (raw) In-Reply-To: <202204211211.febbJ6fy-lkp@intel.com> [-- Attachment #1: Type: text/plain, Size: 8473 bytes --] On Thu, 21 Apr 2022 14:45:05 +0800 kernel test robot <lkp@intel.com> wrote: > Hi Jagath, > > Thank you for the patch! Perhaps something to improve: > > [auto build test WARNING on jic23-iio/togreg] > [also build test WARNING on linus/master v5.18-rc3 next-20220420] > [cannot apply to linux/master] > [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/intel-lab-lkp/linux/commits/Jagath-Jog-J/iio-accel-bma400-Add-buffer-step-and-activity-inactivity/20220421-051244 > base: https://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git togreg > config: mips-randconfig-r031-20220420 (https://download.01.org/0day-ci/archive/20220421/202204211211.febbJ6fy-lkp(a)intel.com/config) > compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project bac6cd5bf85669e3376610cfc4c4f9ca015e7b9b) > 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 mips cross compiling tool for clang build > # apt-get install binutils-mips-linux-gnu > # https://github.com/intel-lab-lkp/linux/commit/b33d9910aa7588ec8db7c1694dbc03c3ed200ebb > git remote add linux-review https://github.com/intel-lab-lkp/linux > git fetch --no-tags linux-review Jagath-Jog-J/iio-accel-bma400-Add-buffer-step-and-activity-inactivity/20220421-051244 > git checkout b33d9910aa7588ec8db7c1694dbc03c3ed200ebb > # save the config file > mkdir build_dir && cp config build_dir/.config > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=mips SHELL=/bin/bash drivers/iio/accel/ > > 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/iio/accel/bma400_core.c:1047:17: warning: result of comparison of constant 18446744073709551615 with expression of type 'typeof (_Generic((msk), char: (unsigned char)0, unsigned char: (unsigned char)0, signed char: (unsigned char)0, unsigned short: (unsigned short)0, short: (unsigned short)0, unsigned int: (unsigned int)0, int: (unsigned int)0, unsigned long: (unsigned long)0, long: (unsigned long)0, unsigned long long: (unsigned long long)0, long long: (unsigned long long)0, default: (msk)))' (aka 'unsigned int') is always false [-Wtautological-constant-out-of-range-compare] > field_value = FIELD_PREP(msk, state); > ^~~~~~~~~~~~~~~~~~~~~~ > include/linux/bitfield.h:114:3: note: expanded from macro 'FIELD_PREP' > __BF_FIELD_CHECK(_mask, 0ULL, _val, "FIELD_PREP: "); \ > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > include/linux/bitfield.h:71:53: note: expanded from macro '__BF_FIELD_CHECK' > BUILD_BUG_ON_MSG(__bf_cast_unsigned(_mask, _mask) > \ > ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~ > include/linux/build_bug.h:39:58: note: expanded from macro 'BUILD_BUG_ON_MSG' > #define BUILD_BUG_ON_MSG(cond, msg) compiletime_assert(!(cond), msg) > ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~ > include/linux/compiler_types.h:352:22: note: expanded from macro 'compiletime_assert' > _compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__) > ~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > include/linux/compiler_types.h:340:23: note: expanded from macro '_compiletime_assert' > __compiletime_assert(condition, msg, prefix, suffix) > ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > include/linux/compiler_types.h:332:9: note: expanded from macro '__compiletime_assert' > if (!(condition)) \ > ^~~~~~~~~ > drivers/iio/accel/bma400_core.c:1053:17: warning: result of comparison of constant 18446744073709551615 with expression of type 'typeof (_Generic((msk), char: (unsigned char)0, unsigned char: (unsigned char)0, signed char: (unsigned char)0, unsigned short: (unsigned short)0, short: (unsigned short)0, unsigned int: (unsigned int)0, int: (unsigned int)0, unsigned long: (unsigned long)0, long: (unsigned long)0, unsigned long long: (unsigned long long)0, long long: (unsigned long long)0, default: (msk)))' (aka 'unsigned int') is always false [-Wtautological-constant-out-of-range-compare] > field_value = FIELD_PREP(msk, state); > ^~~~~~~~~~~~~~~~~~~~~~ > include/linux/bitfield.h:114:3: note: expanded from macro 'FIELD_PREP' > __BF_FIELD_CHECK(_mask, 0ULL, _val, "FIELD_PREP: "); \ > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > include/linux/bitfield.h:71:53: note: expanded from macro '__BF_FIELD_CHECK' > BUILD_BUG_ON_MSG(__bf_cast_unsigned(_mask, _mask) > \ > ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~ > include/linux/build_bug.h:39:58: note: expanded from macro 'BUILD_BUG_ON_MSG' > #define BUILD_BUG_ON_MSG(cond, msg) compiletime_assert(!(cond), msg) > ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~ > include/linux/compiler_types.h:352:22: note: expanded from macro 'compiletime_assert' > _compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__) > ~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > include/linux/compiler_types.h:340:23: note: expanded from macro '_compiletime_assert' > __compiletime_assert(condition, msg, prefix, suffix) > ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > include/linux/compiler_types.h:332:9: note: expanded from macro '__compiletime_assert' > if (!(condition)) \ > ^~~~~~~~~ > 2 warnings generated. > > > vim +1047 drivers/iio/accel/bma400_core.c > > 1035 > 1036 static int bma400_activity_event_en(struct bma400_data *data, > 1037 enum iio_event_direction dir, > 1038 int state) > 1039 { > 1040 int ret, reg, msk, value, field_value; > 1041 > 1042 switch (dir) { > 1043 case IIO_EV_DIR_RISING: > 1044 reg = BMA400_GEN1INT_CONFIG0; > 1045 msk = BMA400_INT_GEN1_MSK; > 1046 value = 2; > > 1047 field_value = FIELD_PREP(msk, state); Ah. Will need to clamp state to 0/1 field_value = FIELD_PREP(msk, state ? 1 : 0); perhaps? > 1048 break; > 1049 case IIO_EV_DIR_FALLING: > 1050 reg = BMA400_GEN2INT_CONFIG0; > 1051 msk = BMA400_INT_GEN2_MSK; > 1052 value = 0; > 1053 field_value = FIELD_PREP(msk, state); > 1054 break; > 1055 default: > 1056 return -EINVAL; > 1057 } > 1058 > 1059 /* Enabling all axis for interrupt evaluation */ > 1060 ret = regmap_write(data->regmap, reg, 0xF8); > 1061 if (ret) > 1062 return ret; > 1063 > 1064 /* OR combination of all axis for interrupt evaluation */ > 1065 ret = regmap_write(data->regmap, reg + BMA400_GEN_CONFIG1_OFF, value); > 1066 if (ret) > 1067 return ret; > 1068 > 1069 /* Initial value to avoid interrupts while enabling*/ > 1070 ret = regmap_write(data->regmap, reg + BMA400_GEN_CONFIG2_OFF, 0x0A); > 1071 if (ret) > 1072 return ret; > 1073 > 1074 /* Initial duration value to avoid interrupts while enabling*/ > 1075 ret = regmap_write(data->regmap, reg + BMA400_GEN_CONFIG31_OFF, 0x0F); > 1076 if (ret) > 1077 return ret; > 1078 > 1079 ret = regmap_update_bits(data->regmap, BMA400_INT1_MAP_REG, msk, > 1080 field_value); > 1081 if (ret) > 1082 return ret; > 1083 > 1084 ret = regmap_update_bits(data->regmap, BMA400_INT_CONFIG0_REG, msk, > 1085 field_value); > 1086 if (ret) > 1087 return ret; > 1088 > 1089 set_mask_bits(&data->generic_event_en, msk, field_value); > 1090 return 0; > 1091 } > 1092 >
next prev parent reply other threads:[~2022-04-24 16:11 UTC|newest] Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-04-20 21:10 [PATCH v4 0/9] iio: accel: bma400: Add buffer, step and activity/inactivity Jagath Jog J 2022-04-20 21:10 ` [PATCH v4 1/9] iio: accel: bma400: Fix the scale min and max macro values Jagath Jog J 2022-04-27 12:28 ` Andy Shevchenko 2022-04-20 21:10 ` [PATCH v4 2/9] iio: accel: bma400: Reordering of header files Jagath Jog J 2022-04-20 21:10 ` [PATCH v4 3/9] iio: accel: bma400: conversion to device-managed function Jagath Jog J 2022-04-20 21:11 ` [PATCH v4 4/9] iio: accel: bma400: Add triggered buffer support Jagath Jog J 2022-04-27 12:34 ` Andy Shevchenko 2022-05-01 16:20 ` Jonathan Cameron 2022-05-01 20:25 ` Jagath Jog J 2022-05-07 16:00 ` Jonathan Cameron 2022-04-20 21:11 ` [PATCH v4 5/9] iio: accel: bma400: Add separate channel for step counter Jagath Jog J 2022-04-27 12:34 ` Andy Shevchenko 2022-04-27 21:01 ` Jagath Jog J 2022-04-20 21:11 ` [PATCH v4 6/9] iio: accel: bma400: Add step change event Jagath Jog J 2022-05-01 16:31 ` Jonathan Cameron 2022-05-01 20:27 ` Jagath Jog J 2022-04-20 21:11 ` [PATCH v4 7/9] iio: accel: bma400: Add activity recognition support Jagath Jog J 2022-04-20 21:11 ` [PATCH v4 8/9] iio: accel: bma400: Add debugfs register access support Jagath Jog J 2022-05-01 16:34 ` Jonathan Cameron 2022-04-20 21:11 ` [PATCH v4 9/9] iio: accel: bma400: Add support for activity and inactivity events Jagath Jog J 2022-04-21 6:45 ` kernel test robot 2022-04-24 16:20 ` Jonathan Cameron [this message] 2022-04-24 16:20 ` Jonathan Cameron 2022-04-27 3:01 ` Jagath Jog J 2022-04-27 3:01 ` Jagath Jog J 2022-04-27 13:45 ` Andy Shevchenko 2022-04-27 13:45 ` Andy Shevchenko 2022-04-26 11:04 ` [PATCH v4 0/9] iio: accel: bma400: Add buffer, step and activity/inactivity Andy Shevchenko 2022-04-26 11:05 ` Andy Shevchenko 2022-04-26 12:28 ` Jagath Jog J 2022-05-01 16:36 ` Jonathan Cameron
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=20220424172002.7dd48dfa@jic23-huawei \ --to=jic23@kernel.org \ --cc=andy.shevchenko@gmail.com \ --cc=dan@dlrobertson.com \ --cc=jagathjog1996@gmail.com \ --cc=kbuild-all@lists.01.org \ --cc=linux-iio@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=lkp@intel.com \ --cc=llvm@lists.linux.dev \ /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.