Re: [PATCH v2 2/3] iio: light: ROHM BH1745 colour sensor

kernel test robot
2024-06-05T07:52:52Z

Hi Mudit,

kernel test robot noticed the following build warnings:

[auto build test WARNING on jic23-iio/togreg]
[also build test WARNING on linus/master v6.10-rc2 next-20240605]
[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#_base_tree_information]

url: https://github.com/intel-lab-lkp/linux/commits/Mudit-Sharma/iio-light-ROHM-BH1745-colour-sensor/20240604-002405
base: https://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git togreg
patch link: https://lore.kernel.org/r/20240603162122.165943-2-muditsharma.info%40gmail.com
patch subject: [PATCH v2 2/3] iio: light: ROHM BH1745 colour sensor
config: csky-randconfig-r112-20240605 (https://download.01.org/0day-ci/archive/20240605/202406051506.orRmnC3s-lkp@intel.com/config)
compiler: csky-linux-gcc (GCC) 13.2.0
reproduce: (https://download.01.org/0day-ci/archive/20240605/202406051506.orRmnC3s-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202406051506.orRmnC3s-lkp@intel.com/

sparse warnings: (new ones prefixed by >>)
>> drivers/iio/light/bh1745.c:99:3: sparse: sparse: symbol 'bh1745_int_source' was not declared. Should it be static?
>> drivers/iio/light/bh1745.c:105:3: sparse: sparse: symbol 'bh1745_gain' was not declared. Should it be static?
>> drivers/iio/light/bh1745.c:114:3: sparse: sparse: symbol 'bh1745_measurement_time' was not declared. Should it be static?
>> drivers/iio/light/bh1745.c:121:3: sparse: sparse: symbol 'bh1745_presistence_value' was not declared. Should it be static?

vim +/bh1745_int_source +99 drivers/iio/light/bh1745.c

93
94 enum {
95 BH1745_INT_SOURCE_RED,
96 BH1745_INT_SOURCE_GREEN,
97 BH1745_INT_SOURCE_BLUE,
98 BH1745_INT_SOURCE_CLEAR,
> 99 } bh1745_int_source;
100
101 enum {
102 BH1745_ADC_GAIN_1X,
103 BH1745_ADC_GAIN_2X,
104 BH1745_ADC_GAIN_16X,
> 105 } bh1745_gain;
106
107 enum {
108 BH1745_MEASUREMENT_TIME_160MS,
109 BH1745_MEASUREMENT_TIME_320MS,
110 BH1745_MEASUREMENT_TIME_640MS,
111 BH1745_MEASUREMENT_TIME_1280MS,
112 BH1745_MEASUREMENT_TIME_2560MS,
113 BH1745_MEASUREMENT_TIME_5120MS,
> 114 } bh1745_measurement_time;
115
116 enum {
117 BH1745_PRESISTENCE_UPDATE_TOGGLE,
118 BH1745_PRESISTENCE_UPDATE_EACH_MEASUREMENT,
119 BH1745_PRESISTENCE_UPDATE_FOUR_MEASUREMENT,
120 BH1745_PRESISTENCE_UPDATE_EIGHT_MEASUREMENT,
> 121 } bh1745_presistence_value;
122

--
0-DAY CI Kernel Test Service