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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <202002100001.BwAzqv0v%lkp@intel.com>
Date:   Mon, 10 Feb 2020 00:47:28 +0800
From:   kbuild test robot <lkp@...el.com>
To:     Daniel Campello <campello@...omium.org>
Cc:     kbuild-all@...ts.01.org, LKML <linux-kernel@...r.kernel.org>,
        Daniel Campello <campello@...omium.org>,
        Gwendal Grignou <gwendal@...omium.org>,
        Enrico Granata <egranata@...omium.org>,
        Christophe JAILLET <christophe.jaillet@...adoo.fr>,
        Andreas Klinger <ak@...klinger.de>, linux-iio@...r.kernel.org,
        Hartmut Knaack <knaack.h@....de>,
        Thomas Gleixner <tglx@...utronix.de>,
        Lars-Peter Clausen <lars@...afoo.de>,
        Jonathan Cameron <jic23@...nel.org>,
        Peter Meerwald-Stadler <pmeerw@...erw.net>
Subject: Re: [PATCH v2] iio: Add SEMTECH SX9310/9311 sensor driver

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@...el.com>


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@lists.01.org