Hi Nishanth,
I love your patch! Perhaps something to improve:
[auto build test WARNING on jic23-iio/togreg] [also build test WARNING on linus/master v5.19-rc4 next-20220630] [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/Nishanth-Menon/iio-adc-ti-adc... base: https://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git togreg config: microblaze-randconfig-s032-20220629 compiler: microblaze-linux-gcc (GCC) 11.3.0 reproduce: wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # apt-get install sparse # sparse version: v0.6.4-39-gce1a6720-dirty # https://github.com/intel-lab-lkp/linux/commit/d5184722ec9ae186da9bed1497e480... git remote add linux-review https://github.com/intel-lab-lkp/linux git fetch --no-tags linux-review Nishanth-Menon/iio-adc-ti-adc128s052-Fix-number-of-channels-when-device-tree-is-used/20220701-070342 git checkout d5184722ec9ae186da9bed1497e4804297f2040b # save the config file mkdir build_dir && cp config build_dir/.config COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.3.0 make.cross C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' O=build_dir ARCH=microblaze SHELL=/bin/bash drivers/iio/adc/
If you fix the issue, kindly add following tag where applicable Reported-by: kernel test robot lkp@intel.com
sparse warnings: (new ones prefixed by >>)
drivers/iio/adc/ti-adc128s052.c:184:50: sparse: sparse: Using plain integer as NULL pointer drivers/iio/adc/ti-adc128s052.c:185:50: sparse: sparse: incorrect type in initializer (different base types) @@ expected void const *data @@ got int @@
drivers/iio/adc/ti-adc128s052.c:185:50: sparse: expected void const *data drivers/iio/adc/ti-adc128s052.c:185:50: sparse: got int drivers/iio/adc/ti-adc128s052.c:186:50: sparse: sparse: incorrect type in initializer (different base types) @@ expected void const *data @@ got int @@ drivers/iio/adc/ti-adc128s052.c:186:50: sparse: expected void const *data drivers/iio/adc/ti-adc128s052.c:186:50: sparse: got int drivers/iio/adc/ti-adc128s052.c:187:50: sparse: sparse: incorrect type in initializer (different base types) @@ expected void const *data @@ got int @@ drivers/iio/adc/ti-adc128s052.c:187:50: sparse: expected void const *data drivers/iio/adc/ti-adc128s052.c:187:50: sparse: got int drivers/iio/adc/ti-adc128s052.c:188:50: sparse: sparse: incorrect type in initializer (different base types) @@ expected void const *data @@ got int @@ drivers/iio/adc/ti-adc128s052.c:188:50: sparse: expected void const *data drivers/iio/adc/ti-adc128s052.c:188:50: sparse: got int drivers/iio/adc/ti-adc128s052.c:189:50: sparse: sparse: incorrect type in initializer (different base types) @@ expected void const *data @@ got int @@ drivers/iio/adc/ti-adc128s052.c:189:50: sparse: expected void const *data drivers/iio/adc/ti-adc128s052.c:189:50: sparse: got int drivers/iio/adc/ti-adc128s052.c:190:50: sparse: sparse: incorrect type in initializer (different base types) @@ expected void const *data @@ got int @@ drivers/iio/adc/ti-adc128s052.c:190:50: sparse: expected void const *data drivers/iio/adc/ti-adc128s052.c:190:50: sparse: got int
vim +184 drivers/iio/adc/ti-adc128s052.c
182 183 static const struct of_device_id adc128_of_match[] = {
184 { .compatible = "ti,adc128s052", .data = 0}, 185 { .compatible = "ti,adc122s021", .data = 1},
186 { .compatible = "ti,adc122s051", .data = 1}, 187 { .compatible = "ti,adc122s101", .data = 1}, 188 { .compatible = "ti,adc124s021", .data = 2}, 189 { .compatible = "ti,adc124s051", .data = 2}, 190 { .compatible = "ti,adc124s101", .data = 2}, 191 { /* sentinel */ }, 192 }; 193 MODULE_DEVICE_TABLE(of, adc128_of_match); 194