Re: [PATCH v2] iio: Add SEMTECH SX9310/9311 sensor driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Daniel,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on iio/togreg]
[also build test WARNING on linux/master linus/master v5.5 next-20200207]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system. BTW, we also suggest to use '--base' option to specify the
base tree in git format-patch, please see https://stackoverflow.com/a/37406982]

url:    https://github.com/0day-ci/linux/commits/Daniel-Campello/iio-Add-SEMTECH-SX9310-9311-sensor-driver/20200209-180706
base:   https://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git togreg
reproduce:
        # apt-get install sparse
        # sparse version: v0.6.1-162-g98276e61-dirty
        make ARCH=x86_64 allmodconfig
        make C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__'

If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@xxxxxxxxx>


sparse warnings: (new ones prefixed by >>)

>> drivers/iio/proximity/sx9310.c:184:9: sparse: sparse: Using plain integer as NULL pointer
   drivers/iio/proximity/sx9310.c:185:9: sparse: sparse: Using plain integer as NULL pointer
   drivers/iio/proximity/sx9310.c:186:9: sparse: sparse: Using plain integer as NULL pointer

vim +184 drivers/iio/proximity/sx9310.c

   162	
   163	#define SX9310_CHANNEL(idx, name)					 \
   164		{								 \
   165			.type = IIO_PROXIMITY,					 \
   166			.info_mask_separate = BIT(IIO_CHAN_INFO_RAW),		 \
   167			.info_mask_shared_by_all = BIT(IIO_CHAN_INFO_SAMP_FREQ), \
   168			.indexed = 1,						 \
   169			.channel = idx,						 \
   170			.extend_name = name,					 \
   171			.address = SX9310_REG_DIFF_MSB,				 \
   172			.event_spec = sx9310_events,				 \
   173			.num_event_specs = ARRAY_SIZE(sx9310_events),		 \
   174			.scan_index = idx,					 \
   175			.scan_type = {						 \
   176				.sign = 's',					 \
   177				.realbits = 12,					 \
   178				.storagebits = 16,				 \
   179				.endianness = IIO_BE,				 \
   180			},							 \
   181		}
   182	
   183	static const struct iio_chan_spec sx9310_channels[] = {
 > 184		SX9310_CHANNEL(0, 0),		/* CS0 */
   185		SX9310_CHANNEL(1, 0),		/* CS1 */
   186		SX9310_CHANNEL(2, 0),		/* CS2 */
   187		SX9310_CHANNEL(3, "COMB"),	/* COMB */
   188	
   189		IIO_CHAN_SOFT_TIMESTAMP(4),
   190	};
   191	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@xxxxxxxxxxxx



[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux