tree:
https://git.kernel.org/pub/scm/linux/kernel/git/groeck/linux-staging.git
hwmon-playground
head: 4e5fc259a7d202d7e13818052a6cdaff0f602b35
commit: 0bad0ac867f208392aaef0d39889357256e55706 [2/12] hwmon: (pmbus) Add 'phase'
parameter where needed for multi-phase support
config: i386-allyesconfig (attached as .config)
compiler: gcc-7 (Debian 7.5.0-3) 7.5.0
reproduce:
git checkout 0bad0ac867f208392aaef0d39889357256e55706
# save the attached .config to linux build tree
make ARCH=i386
If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>):
drivers/hwmon/pmbus/ibm-cffps.c: In function 'ibm_cffps_read_word_data':
> drivers/hwmon/pmbus/ibm-cffps.c:351:8: error: too few arguments
to function 'pmbus_read_word_data'
rc = pmbus_read_word_data(client,
page, CFFPS_12VCS_VOUT_CMD);
^~~~~~~~~~~~~~~~~~~~
In file included from drivers/hwmon/pmbus/ibm-cffps.c:19:0:
drivers/hwmon/pmbus/pmbus.h:462:5: note: declared here
int pmbus_read_word_data(struct i2c_client *client, int page, int phase,
^~~~~~~~~~~~~~~~~~~~
vim +/pmbus_read_word_data +351 drivers/hwmon/pmbus/ibm-cffps.c
f69316d62c7066 Edward A. James 2017-08-21 326
f69316d62c7066 Edward A. James 2017-08-21 327 static int ibm_cffps_read_word_data(struct
i2c_client *client, int page,
0bad0ac867f208 Guenter Roeck 2020-01-14 328 int phase, int reg)
f69316d62c7066 Edward A. James 2017-08-21 329 {
f69316d62c7066 Edward A. James 2017-08-21 330 int rc, mfr;
f69316d62c7066 Edward A. James 2017-08-21 331
f69316d62c7066 Edward A. James 2017-08-21 332 switch (reg) {
f69316d62c7066 Edward A. James 2017-08-21 333 case PMBUS_STATUS_WORD:
0bad0ac867f208 Guenter Roeck 2020-01-14 334 rc = pmbus_read_word_data(client, page,
phase, reg);
f69316d62c7066 Edward A. James 2017-08-21 335 if (rc < 0)
f69316d62c7066 Edward A. James 2017-08-21 336 return rc;
f69316d62c7066 Edward A. James 2017-08-21 337
f69316d62c7066 Edward A. James 2017-08-21 338 mfr = pmbus_read_byte_data(client,
page,
f69316d62c7066 Edward A. James 2017-08-21 339 PMBUS_STATUS_MFR_SPECIFIC);
f69316d62c7066 Edward A. James 2017-08-21 340 if (mfr < 0)
f69316d62c7066 Edward A. James 2017-08-21 341 /*
f69316d62c7066 Edward A. James 2017-08-21 342 * Return the status register instead
of an error,
f69316d62c7066 Edward A. James 2017-08-21 343 * since we successfully read status.
f69316d62c7066 Edward A. James 2017-08-21 344 */
f69316d62c7066 Edward A. James 2017-08-21 345 return rc;
f69316d62c7066 Edward A. James 2017-08-21 346
f69316d62c7066 Edward A. James 2017-08-21 347 if (mfr & CFFPS_MFR_PS_KILL)
f69316d62c7066 Edward A. James 2017-08-21 348 rc |= PB_STATUS_OFF;
f69316d62c7066 Edward A. James 2017-08-21 349 break;
1952d79a0d2635 Eddie James 2019-12-19 350 case PMBUS_VIRT_READ_VMON:
1952d79a0d2635 Eddie James 2019-12-19 @351 rc = pmbus_read_word_data(client, page,
CFFPS_12VCS_VOUT_CMD);
1952d79a0d2635 Eddie James 2019-12-19 352 break;
f69316d62c7066 Edward A. James 2017-08-21 353 default:
f69316d62c7066 Edward A. James 2017-08-21 354 rc = -ENODATA;
f69316d62c7066 Edward A. James 2017-08-21 355 break;
f69316d62c7066 Edward A. James 2017-08-21 356 }
f69316d62c7066 Edward A. James 2017-08-21 357
f69316d62c7066 Edward A. James 2017-08-21 358 return rc;
f69316d62c7066 Edward A. James 2017-08-21 359 }
f69316d62c7066 Edward A. James 2017-08-21 360
:::::: The code at line 351 was first introduced by commit
:::::: 1952d79a0d2635c9130a48a598dd2f1eea70c45b hwmon: (pmbus/ibm-cffps) Add the VMON
property for version 2
:::::: TO: Eddie James <eajames(a)linux.ibm.com>
:::::: CC: Guenter Roeck <linux(a)roeck-us.net>
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org Intel Corporation