Re: [PATCH] i2c: use void pointers for supplying data for reads and writes

From: kbuild test robot
Date: Sun Apr 02 2017 - 21:09:54 EST


Hi Dmitry,

[auto build test ERROR on wsa/i2c/for-next]
[also build test ERROR on v4.11-rc4 next-20170331]
[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/Dmitry-Torokhov/i2c-use-void-pointers-for-supplying-data-for-reads-and-writes/20170403-074306
base: https://git.kernel.org/pub/scm/linux/kernel/git/wsa/linux.git i2c/for-next
config: i386-randconfig-a0-201714 (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
# save the attached .config to linux build tree
make ARCH=i386

All errors (new ones prefixed by >>):

drivers/iio/adc/max1363.c: In function 'max1363_probe':
>> drivers/iio/adc/max1363.c:1633:12: error: assignment from incompatible pointer type [-Werror=incompatible-pointer-types]
st->send = i2c_master_send;
^
drivers/iio/adc/max1363.c:1634:12: error: assignment from incompatible pointer type [-Werror=incompatible-pointer-types]
st->recv = i2c_master_recv;
^
Cyclomatic Complexity 1 arch/x86/include/asm/bitops.h:constant_test_bit
Cyclomatic Complexity 1 arch/x86/include/asm/bitops.h:variable_test_bit
Cyclomatic Complexity 1 arch/x86/include/asm/bitops.h:fls
Cyclomatic Complexity 1 arch/x86/include/asm/arch_hweight.h:__arch_hweight32
Cyclomatic Complexity 2 include/linux/bitops.h:hweight_long
Cyclomatic Complexity 1 include/linux/log2.h:__ilog2_u32
Cyclomatic Complexity 1 include/linux/kernel.h:kstrtoul
Cyclomatic Complexity 1 include/asm-generic/getorder.h:__get_order
Cyclomatic Complexity 3 include/linux/bitmap.h:bitmap_copy
Cyclomatic Complexity 3 include/linux/bitmap.h:bitmap_subset
Cyclomatic Complexity 3 include/linux/bitmap.h:bitmap_weight
Cyclomatic Complexity 1 include/linux/err.h:PTR_ERR
Cyclomatic Complexity 1 include/linux/err.h:IS_ERR
Cyclomatic Complexity 1 include/linux/device.h:devm_kzalloc
Cyclomatic Complexity 1 include/linux/device.h:dev_get_drvdata
Cyclomatic Complexity 1 include/linux/device.h:dev_set_drvdata
Cyclomatic Complexity 1 include/linux/i2c.h:i2c_get_clientdata
Cyclomatic Complexity 1 include/linux/i2c.h:i2c_set_clientdata
Cyclomatic Complexity 1 include/linux/i2c.h:i2c_get_functionality
Cyclomatic Complexity 1 include/linux/i2c.h:i2c_check_functionality
Cyclomatic Complexity 28 include/linux/slab.h:kmalloc_index
Cyclomatic Complexity 67 include/linux/slab.h:kmalloc_large
Cyclomatic Complexity 5 include/linux/slab.h:kmalloc
Cyclomatic Complexity 1 include/linux/iio/iio.h:dev_to_iio_dev
Cyclomatic Complexity 1 include/linux/iio/iio.h:iio_priv
Cyclomatic Complexity 1 include/linux/iio/iio.h:iio_buffer_enabled
Cyclomatic Complexity 2 include/linux/iio/buffer.h:iio_push_to_buffers_with_timestamp
Cyclomatic Complexity 4 drivers/iio/adc/max1363.c:max1363_match_mode
Cyclomatic Complexity 1 drivers/iio/adc/max1363.c:max1363_write_basic_config
Cyclomatic Complexity 1 drivers/iio/adc/max1363.c:max1363_set_scan_mode
Cyclomatic Complexity 2 drivers/iio/adc/max1363.c:max1363_read_thresh
Cyclomatic Complexity 7 drivers/iio/adc/max1363.c:max1363_write_thresh
Cyclomatic Complexity 6 drivers/iio/adc/max1363.c:__max1363_check_event_mask
Cyclomatic Complexity 2 drivers/iio/adc/max1363.c:max1363_update_scan_mode
Cyclomatic Complexity 2 drivers/iio/adc/max1363.c:max1363_initial_setup
Cyclomatic Complexity 3 drivers/iio/adc/max1363.c:max1363_alloc_scan_masks
Cyclomatic Complexity 8 drivers/iio/adc/max1363.c:max1363_read_single_chan
Cyclomatic Complexity 4 drivers/iio/adc/max1363.c:max1363_read_raw
Cyclomatic Complexity 2 drivers/iio/adc/max1363.c:max1363_read_event_config
Cyclomatic Complexity 5 drivers/iio/adc/max1363.c:max1363_monitor_store_freq
Cyclomatic Complexity 16 drivers/iio/adc/max1363.c:max1363_monitor_mode_update
Cyclomatic Complexity 6 drivers/iio/adc/max1363.c:max1363_write_event_config
Cyclomatic Complexity 1 drivers/iio/adc/max1363.c:max1363_monitor_show_freq
Cyclomatic Complexity 2 drivers/iio/adc/max1363.c:max1363_remove
Cyclomatic Complexity 19 drivers/iio/adc/max1363.c:max1363_probe
Cyclomatic Complexity 2 drivers/iio/adc/max1363.c:max1363_event_handler
Cyclomatic Complexity 8 drivers/iio/adc/max1363.c:max1363_trigger_handler
Cyclomatic Complexity 3 drivers/iio/adc/max1363.c:max1363_smbus_recv
Cyclomatic Complexity 3 drivers/iio/adc/max1363.c:max1363_smbus_send
Cyclomatic Complexity 1 drivers/iio/adc/max1363.c:max1363_driver_init
Cyclomatic Complexity 1 drivers/iio/adc/max1363.c:max1363_driver_exit
cc1: some warnings being treated as errors

vim +1633 drivers/iio/adc/max1363.c

a405b00e Guenter Roeck 2013-02-03 1617 if (!IS_ERR(vref)) {
a405b00e Guenter Roeck 2013-02-03 1618 int vref_uv;
a405b00e Guenter Roeck 2013-02-03 1619
a405b00e Guenter Roeck 2013-02-03 1620 ret = regulator_enable(vref);
a405b00e Guenter Roeck 2013-02-03 1621 if (ret)
a405b00e Guenter Roeck 2013-02-03 1622 goto error_disable_reg;
a405b00e Guenter Roeck 2013-02-03 1623 st->vref = vref;
a405b00e Guenter Roeck 2013-02-03 1624 vref_uv = regulator_get_voltage(vref);
a405b00e Guenter Roeck 2013-02-03 1625 if (vref_uv <= 0) {
a405b00e Guenter Roeck 2013-02-03 1626 ret = -EINVAL;
a405b00e Guenter Roeck 2013-02-03 1627 goto error_disable_reg;
a405b00e Guenter Roeck 2013-02-03 1628 }
a405b00e Guenter Roeck 2013-02-03 1629 st->vref_uv = vref_uv;
a405b00e Guenter Roeck 2013-02-03 1630 }
a405b00e Guenter Roeck 2013-02-03 1631
61bdda69 Vivien Didelot 2013-10-20 1632 if (i2c_check_functionality(client->adapter, I2C_FUNC_I2C)) {
61bdda69 Vivien Didelot 2013-10-20 @1633 st->send = i2c_master_send;
61bdda69 Vivien Didelot 2013-10-20 1634 st->recv = i2c_master_recv;
61bdda69 Vivien Didelot 2013-10-20 1635 } else if (i2c_check_functionality(client->adapter, I2C_FUNC_SMBUS_BYTE)
61bdda69 Vivien Didelot 2013-10-20 1636 && st->chip_info->bits == 8) {
61bdda69 Vivien Didelot 2013-10-20 1637 st->send = max1363_smbus_send;
61bdda69 Vivien Didelot 2013-10-20 1638 st->recv = max1363_smbus_recv;
61bdda69 Vivien Didelot 2013-10-20 1639 } else {
61bdda69 Vivien Didelot 2013-10-20 1640 ret = -EOPNOTSUPP;
61bdda69 Vivien Didelot 2013-10-20 1641 goto error_disable_reg;

:::::: The code at line 1633 was first introduced by commit
:::::: 61bdda69222c09efd8943d240cd2a8e0bb659d82 iio:adc:max1363 support SMBus for 8-bit devices

:::::: TO: Vivien Didelot <vivien.didelot@xxxxxxxxxxxxxxxxxxxx>
:::::: CC: Jonathan Cameron <jic23@xxxxxxxxxx>

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

Attachment: .config.gz
Description: application/gzip