Re: [PATCH 02/10] i2c-mux: move select and deselect ops to i2c_mux_core
Hi Peter, [auto build test ERROR on wsa/i2c/for-next] [also build test ERROR on v4.4-rc8 next-20160104] [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/20160104-231355 base: https://git.kernel.org/pub/scm/linux/kernel/git/wsa/linux i2c/for-next config: x86_64-randconfig-s4-01042328 (attached as .config) reproduce: # save the attached .config to linux build tree make ARCH=x86_64 All error/warnings (new ones prefixed by >>): drivers/media/dvb-frontends/si2168.c: In function 'si2168_probe': drivers/media/dvb-frontends/si2168.c:712:37: warning: passing argument 1 of 'i2c_add_mux_adapter' from incompatible pointer type [-Wincompatible-pointer-types] dev->adapter = i2c_add_mux_adapter(client->adapter, &client->dev, ^ In file included from drivers/media/dvb-frontends/si2168_priv.h:23:0, from drivers/media/dvb-frontends/si2168.c:17: include/linux/i2c-mux.h:53:21: note: expected 'struct i2c_mux_core *' but argument is of type 'struct i2c_adapter *' struct i2c_adapter *i2c_add_mux_adapter(struct i2c_mux_core *muxc, ^ >> drivers/media/dvb-frontends/si2168.c:713:4: warning: passing argument 3 of >> 'i2c_add_mux_adapter' makes integer from pointer without a cast >> [-Wint-conversion] client, 0, 0, 0, si2168_select, si2168_deselect); ^ In file included from drivers/media/dvb-frontends/si2168_priv.h:23:0, from drivers/media/dvb-frontends/si2168.c:17: include/linux/i2c-mux.h:53:21: note: expected 'u32 {aka unsigned int}' but argument is of type 'struct i2c_client *' struct i2c_adapter *i2c_add_mux_adapter(struct i2c_mux_core *muxc, ^ >> drivers/media/dvb-frontends/si2168.c:712:17: error: too many arguments to >> function 'i2c_add_mux_adapter' dev->adapter = i2c_add_mux_adapter(client->adapter, &client->dev, ^ In file included from drivers/media/dvb-frontends/si2168_priv.h:23:0, from drivers/media/dvb-frontends/si2168.c:17: include/linux/i2c-mux.h:53:21: note: declared here struct i2c_adapter *i2c_add_mux_adapter(struct i2c_mux_core *muxc, ^ vim +/i2c_add_mux_adapter +712 drivers/media/dvb-frontends/si2168.c 845f3505 Antti Palosaari 2014-04-10 706ret = -ENOMEM; 37b4e43d Olli Salonen2014-08-05 707dev_err(&client->dev, "kzalloc() failed\n"); 1ee5e7dd Antti Palosaari 2014-12-05 708goto err; 845f3505 Antti Palosaari 2014-04-10 709} 845f3505 Antti Palosaari 2014-04-10 710 845f3505 Antti Palosaari 2014-04-10 711/* create mux i2c adapter for tuner */ 6307b560 Antti Palosaari 2014-11-25 @712dev->adapter = i2c_add_mux_adapter(client->adapter, &client->dev, 6307b560 Antti Palosaari 2014-11-25 @713client, 0, 0, 0, si2168_select, si2168_deselect); bd01c766 Antti Palosaari 2014-11-25 714if (dev->adapter == NULL) { 4d6efc7a Luis Alves 2014-07-17 715ret = -ENODEV; 346d4900 Antti Palosaari 2014-12-05 716goto err_kfree; :: The code at line 712 was first introduced by commit :: 6307b560800fd92b915a851186653317460039ad [media] si2168: carry pointer to client instead of state :: TO: Antti Palosaari :: CC: Mauro Carvalho Chehab --- 0-DAY kernel test infrastructureOpen Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation .config.gz Description: Binary data
Re: [PATCH 02/10] i2c-mux: move select and deselect ops to i2c_mux_core
Hi Peter, [auto build test WARNING on wsa/i2c/for-next] [also build test WARNING on v4.4-rc8 next-20160104] [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/20160104-231355 base: https://git.kernel.org/pub/scm/linux/kernel/git/wsa/linux i2c/for-next reproduce: # apt-get install sparse make ARCH=x86_64 allmodconfig make C=1 CF=-D__CHECK_ENDIAN__ sparse warnings: (new ones prefixed by >>) include/linux/compiler.h:228:8: sparse: attribute 'no_sanitize_address': unknown attribute >> drivers/iio/imu/inv_mpu6050/inv_mpu_core.c:845:46: sparse: too many >> arguments for function i2c_add_mux_adapter drivers/iio/imu/inv_mpu6050/inv_mpu_core.c: In function 'inv_mpu_probe': drivers/iio/imu/inv_mpu6050/inv_mpu_core.c:845:40: warning: passing argument 1 of 'i2c_add_mux_adapter' from incompatible pointer type [-Wincompatible-pointer-types] st->mux_adapter = i2c_add_mux_adapter(client->adapter, ^ In file included from drivers/iio/imu/inv_mpu6050/inv_mpu_core.c:26:0: include/linux/i2c-mux.h:53:21: note: expected 'struct i2c_mux_core *' but argument is of type 'struct i2c_adapter *' struct i2c_adapter *i2c_add_mux_adapter(struct i2c_mux_core *muxc, ^ drivers/iio/imu/inv_mpu6050/inv_mpu_core.c:847:12: warning: passing argument 3 of 'i2c_add_mux_adapter' makes integer from pointer without a cast [-Wint-conversion] indio_dev, ^ In file included from drivers/iio/imu/inv_mpu6050/inv_mpu_core.c:26:0: include/linux/i2c-mux.h:53:21: note: expected 'u32 {aka unsigned int}' but argument is of type 'struct iio_dev *' struct i2c_adapter *i2c_add_mux_adapter(struct i2c_mux_core *muxc, ^ drivers/iio/imu/inv_mpu6050/inv_mpu_core.c:845:20: error: too many arguments to function 'i2c_add_mux_adapter' st->mux_adapter = i2c_add_mux_adapter(client->adapter, ^ In file included from drivers/iio/imu/inv_mpu6050/inv_mpu_core.c:26:0: include/linux/i2c-mux.h:53:21: note: declared here struct i2c_adapter *i2c_add_mux_adapter(struct i2c_mux_core *muxc, ^ -- include/linux/compiler.h:228:8: sparse: attribute 'no_sanitize_address': unknown attribute >> drivers/media/dvb-frontends/si2168.c:712:43: sparse: too many arguments for >> function i2c_add_mux_adapter drivers/media/dvb-frontends/si2168.c: In function 'si2168_probe': drivers/media/dvb-frontends/si2168.c:712:37: warning: passing argument 1 of 'i2c_add_mux_adapter' from incompatible pointer type [-Wincompatible-pointer-types] dev->adapter = i2c_add_mux_adapter(client->adapter, &client->dev, ^ In file included from drivers/media/dvb-frontends/si2168_priv.h:23:0, from drivers/media/dvb-frontends/si2168.c:17: include/linux/i2c-mux.h:53:21: note: expected 'struct i2c_mux_core *' but argument is of type 'struct i2c_adapter *' struct i2c_adapter *i2c_add_mux_adapter(struct i2c_mux_core *muxc, ^ drivers/media/dvb-frontends/si2168.c:713:4: warning: passing argument 3 of 'i2c_add_mux_adapter' makes integer from pointer without a cast [-Wint-conversion] client, 0, 0, 0, si2168_select, si2168_deselect); ^ In file included from drivers/media/dvb-frontends/si2168_priv.h:23:0, from drivers/media/dvb-frontends/si2168.c:17: include/linux/i2c-mux.h:53:21: note: expected 'u32 {aka unsigned int}' but argument is of type 'struct i2c_client *' struct i2c_adapter *i2c_add_mux_adapter(struct i2c_mux_core *muxc, ^ drivers/media/dvb-frontends/si2168.c:712:17: error: too many arguments to function 'i2c_add_mux_adapter' dev->adapter = i2c_add_mux_adapter(client->adapter, &client->dev, ^ In file included from drivers/media/dvb-frontends/si2168_priv.h:23:0, from drivers/media/dvb-frontends/si2168.c:17: include/linux/i2c-mux.h:53:21: note: declared here struct i2c_adapter *i2c_add_mux_adapter(struct i2c_mux_core *muxc, ^ -- include/linux/compiler.h:228:8: sparse: attribute 'no_sanitize_address': unknown attribute >> drivers/media/dvb-frontends/m88ds3103.c:1470:47: sparse: too many arguments >> for function i2c_add_mux_adapter drivers/media/dvb-frontends/m88ds3103.c: In function 'm88ds3103_probe': drivers/media/dvb-frontends/m88ds3103.c:1470:41: warning: passing argument 1 of 'i2c_add_mux_adapter' from incompatible pointer type [-Wincompatible-pointer-types] dev->i2c_adapter = i2c_add_mux_adapter(client->adapter, &client->dev, ^ In file included from drivers/media/dvb-frontends/m88ds3103_priv.h:24:
Re: [PATCH 02/10] i2c-mux: move select and deselect ops to i2c_mux_core
Hi Peter, [auto build test ERROR on wsa/i2c/for-next] [also build test ERROR on v4.4-rc8 next-20160104] [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/20160104-231355 base: https://git.kernel.org/pub/scm/linux/kernel/git/wsa/linux i2c/for-next config: i386-randconfig-i0-01042049 (attached as .config) reproduce: # save the attached .config to linux build tree make ARCH=i386 All error/warnings (new ones prefixed by >>): drivers/iio/imu/inv_mpu6050/inv_mpu_core.c: In function 'inv_mpu_probe': drivers/iio/imu/inv_mpu6050/inv_mpu_core.c:845:40: warning: passing argument 1 of 'i2c_add_mux_adapter' from incompatible pointer type [-Wincompatible-pointer-types] st->mux_adapter = i2c_add_mux_adapter(client->adapter, ^ In file included from drivers/iio/imu/inv_mpu6050/inv_mpu_core.c:26:0: include/linux/i2c-mux.h:53:21: note: expected 'struct i2c_mux_core *' but argument is of type 'struct i2c_adapter *' struct i2c_adapter *i2c_add_mux_adapter(struct i2c_mux_core *muxc, ^ >> drivers/iio/imu/inv_mpu6050/inv_mpu_core.c:847:12: warning: passing argument >> 3 of 'i2c_add_mux_adapter' makes integer from pointer without a cast >> [-Wint-conversion] indio_dev, ^ In file included from drivers/iio/imu/inv_mpu6050/inv_mpu_core.c:26:0: include/linux/i2c-mux.h:53:21: note: expected 'u32 {aka unsigned int}' but argument is of type 'struct iio_dev *' struct i2c_adapter *i2c_add_mux_adapter(struct i2c_mux_core *muxc, ^ >> drivers/iio/imu/inv_mpu6050/inv_mpu_core.c:845:20: error: too many arguments >> to function 'i2c_add_mux_adapter' st->mux_adapter = i2c_add_mux_adapter(client->adapter, ^ In file included from drivers/iio/imu/inv_mpu6050/inv_mpu_core.c:26:0: include/linux/i2c-mux.h:53:21: note: declared here struct i2c_adapter *i2c_add_mux_adapter(struct i2c_mux_core *muxc, ^ -- drivers/of/unittest.c: In function 'unittest_i2c_mux_probe': drivers/of/unittest.c:1730:38: warning: passing argument 1 of 'i2c_add_mux_adapter' from incompatible pointer type [-Wincompatible-pointer-types] stm->adap[i] = i2c_add_mux_adapter(adap, dev, client, ^ In file included from drivers/of/unittest.c:24:0: include/linux/i2c-mux.h:53:21: note: expected 'struct i2c_mux_core *' but argument is of type 'struct i2c_adapter *' struct i2c_adapter *i2c_add_mux_adapter(struct i2c_mux_core *muxc, ^ >> drivers/of/unittest.c:1730:49: warning: passing argument 3 of >> 'i2c_add_mux_adapter' makes integer from pointer without a cast >> [-Wint-conversion] stm->adap[i] = i2c_add_mux_adapter(adap, dev, client, ^ In file included from drivers/of/unittest.c:24:0: include/linux/i2c-mux.h:53:21: note: expected 'u32 {aka unsigned int}' but argument is of type 'struct i2c_client *' struct i2c_adapter *i2c_add_mux_adapter(struct i2c_mux_core *muxc, ^ >> drivers/of/unittest.c:1730:18: error: too many arguments to function >> 'i2c_add_mux_adapter' stm->adap[i] = i2c_add_mux_adapter(adap, dev, client, ^ In file included from drivers/of/unittest.c:24:0: include/linux/i2c-mux.h:53:21: note: declared here struct i2c_adapter *i2c_add_mux_adapter(struct i2c_mux_core *muxc, ^ vim +/i2c_add_mux_adapter +845 drivers/iio/imu/inv_mpu6050/inv_mpu_core.c 09a642b7 Ge Gao 2013-02-02 839result = iio_device_register(indio_dev); 09a642b7 Ge Gao 2013-02-02 840if (result) { 09a642b7 Ge Gao 2013-02-02 841 dev_err(&st->client->dev, "IIO register fail %d\n", result); 09a642b7 Ge Gao 2013-02-02 842goto out_remove_trigger; 09a642b7 Ge Gao 2013-02-02 843} 09a642b7 Ge Gao 2013-02-02 844 3a2ecc3d Srinivas Pandruvada 2014-12-05 @845st->mux_adapter = i2c_add_mux_adapter(client->adapter, 3a2ecc3d Srinivas Pandruvada 2014-12-05 846 &client->dev, 3a2ecc3d Srinivas Pandruvada 2014-12-05 @847 indio_dev, 3a2ecc3d Srinivas Pandruvada 2014-12-05 848 0, 0, 0, 3a2ecc3d Srinivas Pandruvada 2014-12-05 849 inv_mpu6050_select_bypass, 3a2ecc3d Srinivas Pandruvada 2014-12-05 850 inv_mpu6050_deselect_bypass); :: The code at line 845 was first introduced by commit :: 3a2ecc3d2dce6e051b6afc319bb380c829e4e4fd iio: imu: inv_mpu6050: Add i2c mux for by
Re: [PATCH 02/10] i2c-mux: move select and deselect ops to i2c_mux_core
Hi Peter, [auto build test ERROR on wsa/i2c/for-next] [also build test ERROR on v4.4-rc8 next-20160104] [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/20160104-231355 base: https://git.kernel.org/pub/scm/linux/kernel/git/wsa/linux i2c/for-next config: x86_64-randconfig-i0-201601 (attached as .config) reproduce: # save the attached .config to linux build tree make ARCH=x86_64 All error/warnings (new ones prefixed by >>): drivers/media/dvb-frontends/m88ds3103.c: In function 'm88ds3103_probe': drivers/media/dvb-frontends/m88ds3103.c:1470:41: warning: passing argument 1 of 'i2c_add_mux_adapter' from incompatible pointer type [-Wincompatible-pointer-types] dev->i2c_adapter = i2c_add_mux_adapter(client->adapter, &client->dev, ^ In file included from drivers/media/dvb-frontends/m88ds3103_priv.h:24:0, from drivers/media/dvb-frontends/m88ds3103.c:17: include/linux/i2c-mux.h:53:21: note: expected 'struct i2c_mux_core *' but argument is of type 'struct i2c_adapter *' struct i2c_adapter *i2c_add_mux_adapter(struct i2c_mux_core *muxc, ^ >> drivers/media/dvb-frontends/m88ds3103.c:1471:13: warning: passing argument 3 >> of 'i2c_add_mux_adapter' makes integer from pointer without a cast >> [-Wint-conversion] dev, 0, 0, 0, m88ds3103_select, ^ In file included from drivers/media/dvb-frontends/m88ds3103_priv.h:24:0, from drivers/media/dvb-frontends/m88ds3103.c:17: include/linux/i2c-mux.h:53:21: note: expected 'u32 {aka unsigned int}' but argument is of type 'struct m88ds3103_dev *' struct i2c_adapter *i2c_add_mux_adapter(struct i2c_mux_core *muxc, ^ >> drivers/media/dvb-frontends/m88ds3103.c:1470:21: error: too many arguments >> to function 'i2c_add_mux_adapter' dev->i2c_adapter = i2c_add_mux_adapter(client->adapter, &client->dev, ^ In file included from drivers/media/dvb-frontends/m88ds3103_priv.h:24:0, from drivers/media/dvb-frontends/m88ds3103.c:17: include/linux/i2c-mux.h:53:21: note: declared here struct i2c_adapter *i2c_add_mux_adapter(struct i2c_mux_core *muxc, ^ -- drivers/media/dvb-frontends/rtl2830.c: In function 'rtl2830_probe': drivers/media/dvb-frontends/rtl2830.c:868:37: warning: passing argument 1 of 'i2c_add_mux_adapter' from incompatible pointer type [-Wincompatible-pointer-types] dev->adapter = i2c_add_mux_adapter(client->adapter, &client->dev, ^ In file included from drivers/media/dvb-frontends/rtl2830_priv.h:24:0, from drivers/media/dvb-frontends/rtl2830.c:18: include/linux/i2c-mux.h:53:21: note: expected 'struct i2c_mux_core *' but argument is of type 'struct i2c_adapter *' struct i2c_adapter *i2c_add_mux_adapter(struct i2c_mux_core *muxc, ^ >> drivers/media/dvb-frontends/rtl2830.c:869:4: warning: passing argument 3 of >> 'i2c_add_mux_adapter' makes integer from pointer without a cast >> [-Wint-conversion] client, 0, 0, 0, rtl2830_select, NULL); ^ In file included from drivers/media/dvb-frontends/rtl2830_priv.h:24:0, from drivers/media/dvb-frontends/rtl2830.c:18: include/linux/i2c-mux.h:53:21: note: expected 'u32 {aka unsigned int}' but argument is of type 'struct i2c_client *' struct i2c_adapter *i2c_add_mux_adapter(struct i2c_mux_core *muxc, ^ >> drivers/media/dvb-frontends/rtl2830.c:868:17: error: too many arguments to >> function 'i2c_add_mux_adapter' dev->adapter = i2c_add_mux_adapter(client->adapter, &client->dev, ^ In file included from drivers/media/dvb-frontends/rtl2830_priv.h:24:0, from drivers/media/dvb-frontends/rtl2830.c:18: include/linux/i2c-mux.h:53:21: note: declared here struct i2c_adapter *i2c_add_mux_adapter(struct i2c_mux_core *muxc, ^ vim +/i2c_add_mux_adapter +1470 drivers/media/dvb-frontends/m88ds3103.c f01919e8f Antti Palosaari 2015-04-16 1464 goto err_kfree; 56ea37da3 Antti Palosaari 2015-10-03 1465 ret = m88ds3103_update_bits(dev, 0x23, 0x10, 0x10); 395d00d1c Antti Palosaari 2013-02-25 1466 if (ret) f01919e8f Antti Palosaari 2015-04-16 1467 goto err_kfree; 395d00d1c Antti Palosaari 2013-02-25 1468 44b9055b4 Antti Palosaari 2013-11-19 1469 /* create mux i2c adapter for tuner */ f01919e8f Antti Palosaari 2015-04-16 @1470 dev->i2c_adapter = i2c_add_mux_adapter(client->adapter, &client->dev, f01919e8f Antti Palosaari 2015-04-16 @1471 dev, 0, 0, 0, m88ds3103_select, 478932b16