Re: [PATCH 3/3] iio: potentiometer: mcp4531: Add device tree binding

From: kbuild test robot <lkp-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
To: Florian Vaussard
	<florian.vaussard-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Cc: kbuild-all-JC7UmRfGjtg@public.gmane.org,
	devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	Jonathan Cameron <jic23-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
	Hartmut Knaack <knaack.h-Mmb7MZpHnFY@public.gmane.org>,
	Lars-Peter Clausen <lars-Qo5EllUWu/uELgA04lAiVw@public.gmane.org>,
	Peter Meerwald-Stadler
	<pmeerw-jW+XmwGofnusTnJN9+BGXg@public.gmane.org>,
	Rob Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
	Pawel Moll <pawel.moll-5wv7dgnIgG8@public.gmane.org>,
	Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org>,
	Ian Campbell
	<ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org>,
	Kumar Gala <galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>,
	Peter Rosin <peda-koto5C5qi+TLoDKTGw+V6w@public.gmane.org>,
	Slawomir Stepien <sst-IjDXvh/HVVUAvxtiuMwx3w@public.gmane.org>,
	Joachim Eastwood
	<manabian-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	Matt Ranostay <mranostay-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	Cristina Moraru
	<cristina.moraru09-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	linux-iio-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	Florian Vaussard
	<florian.vaussard-EWQkb/GNqlFyDzI6CaY1VQ@public.gmane.org>
Subject: Re: [PATCH 3/3] iio: potentiometer: mcp4531: Add device tree binding
Date: Tue, 21 Jun 2016 15:34:24 +0800	[thread overview]
Message-ID: <201606211524.7QTYmkfT%fengguang.wu@intel.com> (raw)
In-Reply-To: <1466492137-32683-4-git-send-email-florian.vaussard-EWQkb/GNqlFyDzI6CaY1VQ@public.gmane.org>

[-- Attachment #1: Type: text/plain, Size: 1959 bytes --]

Hi,

[auto build test WARNING on iio/togreg]
[also build test WARNING on v4.7-rc4 next-20160620]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Florian-Vaussard/iio-potentiometer-mcp4531-New-parts-and-device-tree/20160621-150032
base:   https://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git togreg
config: x86_64-randconfig-i0-201625 (attached as .config)
compiler: gcc-6 (Debian 6.1.1-1) 6.1.1 20160430
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

All warnings (new ones prefixed by >>):

   drivers/iio/potentiometer/mcp4531.c: In function 'mcp4531_probe':
>> drivers/iio/potentiometer/mcp4531.c:280:11: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]
      devid = (int)of_device_get_match_data(dev);
              ^

vim +280 drivers/iio/potentiometer/mcp4531.c

   264				 const struct i2c_device_id *id)
   265	{
   266		struct device *dev = &client->dev;
   267		unsigned long devid;
   268		struct mcp4531_data *data;
   269		struct iio_dev *indio_dev;
   270		const struct of_device_id *match;
   271	
   272		if (!i2c_check_functionality(client->adapter,
   273					     I2C_FUNC_SMBUS_WORD_DATA)) {
   274			dev_err(dev, "SMBUS Word Data not supported\n");
   275			return -EOPNOTSUPP;
   276		}
   277	
   278		match = of_match_device(of_match_ptr(mcp45xx_of_match), dev);
   279		if (match)
 > 280			devid = (int)of_device_get_match_data(dev);
   281		else
   282			devid = id->driver_data;
   283	
   284		indio_dev = devm_iio_device_alloc(dev, sizeof(*data));
   285		if (!indio_dev)
   286			return -ENOMEM;
   287		data = iio_priv(indio_dev);
   288		i2c_set_clientdata(client, indio_dev);

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[-- Attachment #2: .config.gz --]
[-- Type: application/octet-stream, Size: 28400 bytes --]