Re: [PATCH v4 16/18] i2c: allow adapter drivers to override the adapter locking

From: kbuild test robot <lkp@intel.com>
To: Peter Rosin <peda@lysator.liu.se>
Cc: kbuild-all@01.org, linux-kernel@vger.kernel.org,
	Peter Rosin <peda@axentia.se>, Wolfram Sang <wsa@the-dreams.de>,
	Peter Korsgaard <peter.korsgaard@barco.com>,
	Guenter Roeck <linux@roeck-us.net>,
	Jonathan Cameron <jic23@kernel.org>,
	Hartmut Knaack <knaack.h@gmx.de>,
	Lars-Peter Clausen <lars@metafoo.de>,
	Peter Meerwald <pmeerw@pmeerw.net>,
	Antti Palosaari <crope@iki.fi>,
	Mauro Carvalho Chehab <mchehab@osg.samsung.com>,
	Rob Herring <robh+dt@kernel.org>,
	Frank Rowand <frowand.list@gmail.com>,
	Grant Likely <grant.likely@linaro.org>,
	Adriana Reus <adriana.reus@intel.com>,
	Viorel Suman <viorel.suman@intel.com>,
	Krzysztof Kozlowski <k.kozlowski@samsung.com>,
	Terry Heo <terryheo@google.com>,
	Hans Verkuil <hans.verkuil@cisco.com>,
	Arnd Bergmann <arnd@arndb.de>,
	Tommi Rantala <tt.rantala@gmail.com>,
	linux-i2c@vger.kernel.org, linux-iio@vger.kernel.org,
	linux-media@vger.kernel.org, devicetree@vger.kernel.org,
	Peter Rosin <peda@lysator.liu.se>
Subject: Re: [PATCH v4 16/18] i2c: allow adapter drivers to override the adapter locking
Date: Fri, 4 Mar 2016 13:58:50 +0800	[thread overview]
Message-ID: <201603041340.JOBeKIBG%fengguang.wu@intel.com> (raw)
In-Reply-To: <1457044050-15230-17-git-send-email-peda@lysator.liu.se>

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

Hi Peter,

[auto build test WARNING on v4.5-rc6]
[cannot apply to wsa/i2c/for-next sailus-media/master next-20160303]
[if your patch is applied to the wrong git tree, please drop us a note to help improving the system]

url:    https://github.com/0day-ci/linux/commits/Peter-Rosin/i2c-mux-cleanup-and-locking-update/20160304-112229
reproduce: make htmldocs

All warnings (new ones prefixed by >>):

   include/linux/init.h:1: warning: no structured comments found
   kernel/sys.c:1: warning: no structured comments found
   drivers/dma-buf/seqno-fence.c:1: warning: no structured comments found
   drivers/dma-buf/reservation.c:1: warning: no structured comments found
   include/linux/reservation.h:1: warning: no structured comments found
   include/linux/spi/spi.h:540: warning: No description found for parameter 'max_transfer_size'
>> drivers/i2c/i2c-core.c:966: warning: No description found for parameter 'flags'
   drivers/i2c/i2c-core.c:980: warning: No description found for parameter 'flags'
   drivers/i2c/i2c-core.c:994: warning: No description found for parameter 'flags'

vim +/flags +966 drivers/i2c/i2c-core.c

0826374b Michael Lawnick 2010-08-11  950  
97cc4d49 Jean Delvare    2010-10-24  951  	if (parent)
97cc4d49 Jean Delvare    2010-10-24  952  		result = i2c_check_mux_parents(parent, addr);
0826374b Michael Lawnick 2010-08-11  953  
0826374b Michael Lawnick 2010-08-11  954  	if (!result)
0826374b Michael Lawnick 2010-08-11  955  		result = device_for_each_child(&adapter->dev, &addr,
0826374b Michael Lawnick 2010-08-11  956  						i2c_check_mux_children);
0826374b Michael Lawnick 2010-08-11  957  
0826374b Michael Lawnick 2010-08-11  958  	return result;
3b5f794b Jean Delvare    2010-06-03  959  }
3b5f794b Jean Delvare    2010-06-03  960  
9c1600ed David Brownell  2007-05-01  961  /**
7adc3913 Peter Rosin     2016-03-03  962   * i2c_adapter_lock_bus - Get exclusive access to an I2C bus segment
fe61e07e Jean Delvare    2010-08-11  963   * @adapter: Target I2C bus segment
fe61e07e Jean Delvare    2010-08-11  964   */
7adc3913 Peter Rosin     2016-03-03  965  static void i2c_adapter_lock_bus(struct i2c_adapter *adapter, int flags)
fe61e07e Jean Delvare    2010-08-11 @966  {
97cc4d49 Jean Delvare    2010-10-24  967  	struct i2c_adapter *parent = i2c_parent_is_i2c_adapter(adapter);
97cc4d49 Jean Delvare    2010-10-24  968  
97cc4d49 Jean Delvare    2010-10-24  969  	if (parent)
97cc4d49 Jean Delvare    2010-10-24  970  		i2c_lock_adapter(parent);
0826374b Michael Lawnick 2010-08-11  971  	else
fe61e07e Jean Delvare    2010-08-11  972  		rt_mutex_lock(&adapter->bus_lock);
fe61e07e Jean Delvare    2010-08-11  973  }
fe61e07e Jean Delvare    2010-08-11  974  

:::::: The code at line 966 was first introduced by commit
:::::: fe61e07e9ebc890c70d97a1f72ddaad4bee2d848 i2c: Move adapter locking helpers to i2c-core

:::::: TO: Jean Delvare <khali@linux-fr.org>
:::::: CC: Jean Delvare <khali@linux-fr.org>

---
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: 6229 bytes --]