From: kernel test robot <rong.a.chen@intel.com>
To: Jason Gerecke <killertofu@gmail.com>,
linux-i2c@vger.kernel.org,
Wolfram Sang <wsa-dev@sang-engineering.com>
Cc: Ping Cheng <pinglinux@gmail.com>,
"Tobita, Tatsunosuke" <tatsunosuke.tobita@wacom.com>,
Jason Gerecke <jason.gerecke@wacom.com>,
llvm@lists.linux.dev, kbuild-all@lists.01.org
Subject: Re: [PATCH] i2c: Use u8 type in i2c transfer calls
Date: Wed, 20 Jul 2022 08:20:53 +0800 [thread overview]
Message-ID: <0551a3ad-8c42-78fe-5b50-ebbc003e55e6@intel.com> (raw)
In-Reply-To: <202207190634.ToyhlXSz-lkp@intel.com>
Hi Jason,
I love your patch! Yet something to improve:
[auto build test ERROR on wsa/i2c/for-next]
[also build test ERROR on linus/master v5.19-rc7 next-20220718]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]
url:
https://github.com/intel-lab-lkp/linux/commits/Jason-Gerecke/i2c-Use-u8-type-in-i2c-transfer-calls/20220718-233658
base: https://git.kernel.org/pub/scm/linux/kernel/git/wsa/linux.git
i2c/for-next
config: hexagon-randconfig-r026-20220718
(https://download.01.org/0day-ci/archive/20220719/202207190634.ToyhlXSz-lkp@intel.com/config)
compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project
d74b88c69dc2644bd0dc5d64e2d7413a0d4040e5)
reproduce (this is a W=1 build):
wget
https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross
-O ~/bin/make.cross
chmod +x ~/bin/make.cross
#
https://github.com/intel-lab-lkp/linux/commit/9732240c23a365c0590f05ce83196869235a2ea7
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review
Jason-Gerecke/i2c-Use-u8-type-in-i2c-transfer-calls/20220718-233658
git checkout 9732240c23a365c0590f05ce83196869235a2ea7
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1
O=build_dir ARCH=hexagon SHELL=/bin/bash drivers/iio/adc/
If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <lkp@intel.com>
All errors (new ones prefixed by >>):
>> drivers/iio/adc/max1363.c:1645:12: error: incompatible function pointer types assigning to 'int (*)(const struct i2c_client *, const char *, int)' from 'int (const struct i2c_client *, const u8 *, int)' (aka 'int (const struct i2c_client *, const unsigned char *, int)') [-Werror,-Wincompatible-function-pointer-types]
st->send = i2c_master_send;
^ ~~~~~~~~~~~~~~~
>> drivers/iio/adc/max1363.c:1646:12: error: incompatible function pointer types assigning to 'int (*)(const struct i2c_client *, char *, int)' from 'int (const struct i2c_client *, u8 *, int)' (aka 'int (const struct i2c_client *, unsigned char *, int)') [-Werror,-Wincompatible-function-pointer-types]
st->recv = i2c_master_recv;
^ ~~~~~~~~~~~~~~~
2 errors generated.
vim +1645 drivers/iio/adc/max1363.c
a1ff6d25261315 drivers/iio/adc/max1363.c Alexandru Ardelean
2021-09-26 1584 fc52692c49969e drivers/iio/adc/max1363.c
Greg Kroah-Hartman 2012-12-21 1585 static int max1363_probe(struct
i2c_client *client,
d1325cf45077bd drivers/staging/iio/adc/max1363_core.c Jonathan Cameron
2009-08-18 1586 const struct i2c_device_id *id)
d1325cf45077bd drivers/staging/iio/adc/max1363_core.c Jonathan Cameron
2009-08-18 1587 {
cd4361c7e2e077 drivers/staging/iio/adc/max1363_core.c Michael Hennerich
2012-02-22 1588 int ret;
3dba81ba803ca0 drivers/staging/iio/adc/max1363_core.c Jonathan Cameron
2011-04-15 1589 struct max1363_state *st;
3dba81ba803ca0 drivers/staging/iio/adc/max1363_core.c Jonathan Cameron
2011-04-15 1590 struct iio_dev *indio_dev;
a405b00e4470b1 drivers/iio/adc/max1363.c Guenter Roeck
2013-02-03 1591 struct regulator *vref;
d1325cf45077bd drivers/staging/iio/adc/max1363_core.c Jonathan Cameron
2009-08-18 1592 6917e1d9af2c50 drivers/iio/adc/max1363.c
Sachin Kamat 2013-07-22 1593 indio_dev =
devm_iio_device_alloc(&client->dev,
6917e1d9af2c50 drivers/iio/adc/max1363.c Sachin Kamat
2013-07-22 1594 sizeof(struct max1363_state));
6917e1d9af2c50 drivers/iio/adc/max1363.c Sachin Kamat
2013-07-22 1595 if (!indio_dev)
6917e1d9af2c50 drivers/iio/adc/max1363.c Sachin Kamat
2013-07-22 1596 return -ENOMEM;
b77b8f8c20c83c drivers/staging/iio/adc/max1363.c Jonathan Cameron
2012-10-06 1597 a1ff6d25261315 drivers/iio/adc/max1363.c
Alexandru Ardelean 2021-09-26 1598 ret =
devm_iio_map_array_register(&client->dev, indio_dev,
a1ff6d25261315 drivers/iio/adc/max1363.c Alexandru Ardelean
2021-09-26 1599 client->dev.platform_data);
0b27d678c7fbeb drivers/staging/iio/adc/max1363_core.c Jonathan Cameron
2012-04-10 1600 if (ret < 0)
6917e1d9af2c50 drivers/iio/adc/max1363.c Sachin Kamat
2013-07-22 1601 return ret;
b77b8f8c20c83c drivers/staging/iio/adc/max1363.c Jonathan Cameron
2012-10-06 1602 3dba81ba803ca0
drivers/staging/iio/adc/max1363_core.c Jonathan Cameron 2011-04-15
1603 st = iio_priv(indio_dev);
b77b8f8c20c83c drivers/staging/iio/adc/max1363.c Jonathan Cameron
2012-10-06 1604 bf09cddb2cd402 drivers/iio/adc/max1363.c
Rohit Sarkar 2020-03-15 1605 mutex_init(&st->lock);
7c3e8675f37503 drivers/iio/adc/max1363.c Guenter Roeck
2013-02-03 1606 st->reg = devm_regulator_get(&client->dev, "vcc");
a1ff6d25261315 drivers/iio/adc/max1363.c Alexandru Ardelean
2021-09-26 1607 if (IS_ERR(st->reg))
a1ff6d25261315 drivers/iio/adc/max1363.c Alexandru Ardelean
2021-09-26 1608 return PTR_ERR(st->reg);
b77b8f8c20c83c drivers/staging/iio/adc/max1363.c Jonathan Cameron
2012-10-06 1609 b77b8f8c20c83c drivers/staging/iio/adc/max1363.c
Jonathan Cameron 2012-10-06 1610 ret = regulator_enable(st->reg);
b77b8f8c20c83c drivers/staging/iio/adc/max1363.c Jonathan Cameron
2012-10-06 1611 if (ret)
a1ff6d25261315 drivers/iio/adc/max1363.c Alexandru Ardelean
2021-09-26 1612 return ret;
b77b8f8c20c83c drivers/staging/iio/adc/max1363.c Jonathan Cameron
2012-10-06 1613 a1ff6d25261315 drivers/iio/adc/max1363.c
Alexandru Ardelean 2021-09-26 1614 ret =
devm_add_action_or_reset(&client->dev, max1363_reg_disable, st->reg);
a1ff6d25261315 drivers/iio/adc/max1363.c Alexandru Ardelean
2021-09-26 1615 if (ret)
a1ff6d25261315 drivers/iio/adc/max1363.c Alexandru Ardelean
2021-09-26 1616 return ret;
3dba81ba803ca0 drivers/staging/iio/adc/max1363_core.c Jonathan Cameron
2011-04-15 1617 f84ff467c0ce5b drivers/iio/adc/max1363.c
Jonathan Cameron 2020-06-28 1618 st->chip_info =
device_get_match_data(&client->dev);
794ac821cc44e4 drivers/iio/adc/max1363.c Julia Lawall
2018-05-21 1619 if (!st->chip_info)
3dba81ba803ca0 drivers/staging/iio/adc/max1363_core.c Jonathan Cameron
2011-04-15 1620 st->chip_info =
&max1363_chip_info_tbl[id->driver_data];
3dba81ba803ca0 drivers/staging/iio/adc/max1363_core.c Jonathan Cameron
2011-04-15 1621 st->client = client;
3dba81ba803ca0 drivers/staging/iio/adc/max1363_core.c Jonathan Cameron
2011-04-15 1622 a405b00e4470b1 drivers/iio/adc/max1363.c
Guenter Roeck 2013-02-03 1623 st->vref_uv =
st->chip_info->int_vref_mv * 1000;
55b40d37311807 drivers/iio/adc/max1363.c Guenter Roeck
2014-01-27 1624 vref = devm_regulator_get_optional(&client->dev,
"vref");
a405b00e4470b1 drivers/iio/adc/max1363.c Guenter Roeck
2013-02-03 1625 if (!IS_ERR(vref)) {
a405b00e4470b1 drivers/iio/adc/max1363.c Guenter Roeck
2013-02-03 1626 int vref_uv;
a405b00e4470b1 drivers/iio/adc/max1363.c Guenter Roeck
2013-02-03 1627 a405b00e4470b1 drivers/iio/adc/max1363.c
Guenter Roeck 2013-02-03 1628 ret = regulator_enable(vref);
a405b00e4470b1 drivers/iio/adc/max1363.c Guenter Roeck
2013-02-03 1629 if (ret)
a1ff6d25261315 drivers/iio/adc/max1363.c Alexandru Ardelean
2021-09-26 1630 return ret;
a1ff6d25261315 drivers/iio/adc/max1363.c Alexandru Ardelean
2021-09-26 1631 a1ff6d25261315 drivers/iio/adc/max1363.c
Alexandru Ardelean 2021-09-26 1632 ret =
devm_add_action_or_reset(&client->dev, max1363_reg_disable, vref);
a1ff6d25261315 drivers/iio/adc/max1363.c Alexandru Ardelean
2021-09-26 1633 if (ret)
a1ff6d25261315 drivers/iio/adc/max1363.c Alexandru Ardelean
2021-09-26 1634 return ret;
a1ff6d25261315 drivers/iio/adc/max1363.c Alexandru Ardelean
2021-09-26 1635 a405b00e4470b1 drivers/iio/adc/max1363.c
Guenter Roeck 2013-02-03 1636 st->vref = vref;
a405b00e4470b1 drivers/iio/adc/max1363.c Guenter Roeck
2013-02-03 1637 vref_uv = regulator_get_voltage(vref);
a1ff6d25261315 drivers/iio/adc/max1363.c Alexandru Ardelean
2021-09-26 1638 if (vref_uv <= 0)
a1ff6d25261315 drivers/iio/adc/max1363.c Alexandru Ardelean
2021-09-26 1639 return -EINVAL;
a1ff6d25261315 drivers/iio/adc/max1363.c Alexandru Ardelean
2021-09-26 1640 a405b00e4470b1 drivers/iio/adc/max1363.c
Guenter Roeck 2013-02-03 1641 st->vref_uv = vref_uv;
a405b00e4470b1 drivers/iio/adc/max1363.c Guenter Roeck
2013-02-03 1642 }
a405b00e4470b1 drivers/iio/adc/max1363.c Guenter Roeck
2013-02-03 1643 61bdda69222c09 drivers/iio/adc/max1363.c
Vivien Didelot 2013-10-20 1644 if
(i2c_check_functionality(client->adapter, I2C_FUNC_I2C)) {
61bdda69222c09 drivers/iio/adc/max1363.c Vivien Didelot
2013-10-20 @1645 st->send = i2c_master_send;
61bdda69222c09 drivers/iio/adc/max1363.c Vivien Didelot
2013-10-20 @1646 st->recv = i2c_master_recv;
61bdda69222c09 drivers/iio/adc/max1363.c Vivien Didelot
2013-10-20 1647 } else if (i2c_check_functionality(client->adapter,
I2C_FUNC_SMBUS_BYTE)
61bdda69222c09 drivers/iio/adc/max1363.c Vivien Didelot
2013-10-20 1648 && st->chip_info->bits == 8) {
61bdda69222c09 drivers/iio/adc/max1363.c Vivien Didelot
2013-10-20 1649 st->send = max1363_smbus_send;
61bdda69222c09 drivers/iio/adc/max1363.c Vivien Didelot
2013-10-20 1650 st->recv = max1363_smbus_recv;
61bdda69222c09 drivers/iio/adc/max1363.c Vivien Didelot
2013-10-20 1651 } else {
a1ff6d25261315 drivers/iio/adc/max1363.c Alexandru Ardelean
2021-09-26 1652 return -EOPNOTSUPP;
61bdda69222c09 drivers/iio/adc/max1363.c Vivien Didelot
2013-10-20 1653 }
61bdda69222c09 drivers/iio/adc/max1363.c Vivien Didelot
2013-10-20 1654 cd4361c7e2e077
drivers/staging/iio/adc/max1363_core.c Michael Hennerich 2012-02-22
1655 ret = max1363_alloc_scan_masks(indio_dev);
cd4361c7e2e077 drivers/staging/iio/adc/max1363_core.c Michael Hennerich
2012-02-22 1656 if (ret)
a1ff6d25261315 drivers/iio/adc/max1363.c Alexandru Ardelean
2021-09-26 1657 return ret;
82020b0ef15de0 drivers/staging/iio/adc/max1363_core.c Jonathan Cameron
2010-05-04 1658 845bd12ac78f92
drivers/staging/iio/adc/max1363_core.c Jonathan Cameron 2011-05-18
1659 indio_dev->name = id->name;
8e7d967244a8ee drivers/staging/iio/adc/max1363_core.c Jonathan Cameron
2011-08-30 1660 indio_dev->channels = st->chip_info->channels;
8e7d967244a8ee drivers/staging/iio/adc/max1363_core.c Jonathan Cameron
2011-08-30 1661 indio_dev->num_channels = st->chip_info->num_channels;
6fe8135fccd66a drivers/staging/iio/adc/max1363_core.c Jonathan Cameron
2011-05-18 1662 indio_dev->info = st->chip_info->info;
6fe8135fccd66a drivers/staging/iio/adc/max1363_core.c Jonathan Cameron
2011-05-18 1663 indio_dev->modes = INDIO_DIRECT_MODE;
d1325cf45077bd drivers/staging/iio/adc/max1363_core.c Jonathan Cameron
2009-08-18 1664 ret = max1363_initial_setup(st);
58f0a255454dc2 drivers/staging/iio/adc/max1363_core.c Jonathan Cameron
2011-05-18 1665 if (ret < 0)
a1ff6d25261315 drivers/iio/adc/max1363.c Alexandru Ardelean
2021-09-26 1666 return ret;
d1325cf45077bd drivers/staging/iio/adc/max1363_core.c Jonathan Cameron
2009-08-18 1667 a1ff6d25261315 drivers/iio/adc/max1363.c
Alexandru Ardelean 2021-09-26 1668 ret =
devm_iio_triggered_buffer_setup(&client->dev, indio_dev, NULL,
7a1aeba7ed0d5a drivers/iio/adc/max1363.c Lars-Peter Clausen
2013-09-16 1669 &max1363_trigger_handler, NULL);
d1325cf45077bd drivers/staging/iio/adc/max1363_core.c Jonathan Cameron
2009-08-18 1670 if (ret)
a1ff6d25261315 drivers/iio/adc/max1363.c Alexandru Ardelean
2021-09-26 1671 return ret;
d1325cf45077bd drivers/staging/iio/adc/max1363_core.c Jonathan Cameron
2009-08-18 1672 6fe8135fccd66a
drivers/staging/iio/adc/max1363_core.c Jonathan Cameron 2011-05-18
1673 if (client->irq) {
7c3e8675f37503 drivers/iio/adc/max1363.c Guenter Roeck
2013-02-03 1674 ret = devm_request_threaded_irq(&client->dev,
st->client->irq,
aaf370db7dad57 drivers/staging/iio/adc/max1363_core.c Jonathan Cameron
2011-05-18 1675 NULL,
aaf370db7dad57 drivers/staging/iio/adc/max1363_core.c Jonathan Cameron
2011-05-18 1676 &max1363_event_handler,
aaf370db7dad57 drivers/staging/iio/adc/max1363_core.c Jonathan Cameron
2011-05-18 1677 IRQF_TRIGGER_RISING | IRQF_ONESHOT,
aaf370db7dad57 drivers/staging/iio/adc/max1363_core.c Jonathan Cameron
2011-05-18 1678 "max1363_event",
aaf370db7dad57 drivers/staging/iio/adc/max1363_core.c Jonathan Cameron
2011-05-18 1679 indio_dev);
298cd976e07f1a drivers/staging/iio/adc/max1363_core.c Jonathan Cameron
2011-05-18 1680 440a5200347734
drivers/staging/iio/adc/max1363_core.c Jonathan Cameron 2010-06-26
1681 if (ret)
d1325cf45077bd drivers/staging/iio/adc/max1363_core.c Jonathan Cameron
2009-08-18 1682 return ret;
d1325cf45077bd drivers/staging/iio/adc/max1363_core.c Jonathan Cameron
2009-08-18 1683 }
d1325cf45077bd drivers/staging/iio/adc/max1363_core.c Jonathan Cameron
2009-08-18 1684 a1ff6d25261315 drivers/iio/adc/max1363.c
Alexandru Ardelean 2021-09-26 1685 return
devm_iio_device_register(&client->dev, indio_dev);
d1325cf45077bd drivers/staging/iio/adc/max1363_core.c Jonathan Cameron
2009-08-18 1686 }
d1325cf45077bd drivers/staging/iio/adc/max1363_core.c Jonathan Cameron
2009-08-18 1687
--
0-DAY CI Kernel Test Service
https://01.org/lkp
next parent reply other threads:[~2022-07-20 0:21 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <202207190634.ToyhlXSz-lkp@intel.com>
2022-07-20 0:20 ` kernel test robot [this message]
2022-07-20 19:01 ` [PATCH] i2c: Use u8 type in i2c transfer calls Jason Gerecke
2022-07-28 14:26 ` Jason Gerecke
2022-07-28 20:47 ` Andy Shevchenko
2022-07-28 22:48 ` Jason Gerecke
2022-07-31 12:43 ` Jonathan Cameron
2022-07-18 15:34 Jason Gerecke
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=0551a3ad-8c42-78fe-5b50-ebbc003e55e6@intel.com \
--to=rong.a.chen@intel.com \
--cc=jason.gerecke@wacom.com \
--cc=kbuild-all@lists.01.org \
--cc=killertofu@gmail.com \
--cc=linux-i2c@vger.kernel.org \
--cc=llvm@lists.linux.dev \
--cc=pinglinux@gmail.com \
--cc=tatsunosuke.tobita@wacom.com \
--cc=wsa-dev@sang-engineering.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).