Hi Grant, kernel test robot noticed the following build warnings: [auto build test WARNING on groeck-staging/hwmon-next] [also build test WARNING on robh/for-next linus/master v6.12-rc4 next-20241025] [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/Grant-Peltier/hwmon-pmbus-isl68137-add-support-for-voltage-divider-on-Vout/20241025-084244 base: https://git.kernel.org/pub/scm/linux/kernel/git/groeck/linux-staging.git hwmon-next patch link: https://lore.kernel.org/r/7138f3c551ce201ddc9b5e3889ce969d1cd0ac57.1729812789.git.grantpeltier93%40gmail.com patch subject: [PATCH v4 1/2] hwmon: (pmbus/isl68137) add support for voltage divider on Vout config: x86_64-randconfig-161-20241025 (https://download.01.org/0day-ci/archive/20241025/202410252204.ut3IIrVm-lkp@xxxxxxxxx/config) compiler: clang version 19.1.2 (https://github.com/llvm/llvm-project 7ba7d8e2f7b6445b60679da826210cdde29eaf8b) reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20241025/202410252204.ut3IIrVm-lkp@xxxxxxxxx/reproduce) If you fix the issue in a separate patch/commit (i.e. not just a new version of the same patch/commit), kindly add following tags | Reported-by: kernel test robot <lkp@xxxxxxxxx> | Closes: https://lore.kernel.org/oe-kbuild-all/202410252204.ut3IIrVm-lkp@xxxxxxxxx/ All warnings (new ones prefixed by >>): In file included from drivers/hwmon/pmbus/isl68137.c:12: In file included from include/linux/i2c.h:19: In file included from include/linux/regulator/consumer.h:35: In file included from include/linux/suspend.h:5: In file included from include/linux/swap.h:9: In file included from include/linux/memcontrol.h:21: In file included from include/linux/mm.h:2213: include/linux/vmstat.h:518:36: warning: arithmetic between different enumeration types ('enum node_stat_item' and 'enum lru_list') [-Wenum-enum-conversion] 518 | return node_stat_name(NR_LRU_BASE + lru) + 3; // skip "nr_" | ~~~~~~~~~~~ ^ ~~~ >> drivers/hwmon/pmbus/isl68137.c:233:3: warning: label followed by a declaration is a C23 extension [-Wc23-extensions] 233 | u64 temp = DIV_U64_ROUND_CLOSEST((u64)word * | ^ 2 warnings generated. vim +233 drivers/hwmon/pmbus/isl68137.c 211 212 static int raa_dmpvr2_write_word_data(struct i2c_client *client, int page, 213 int reg, u16 word) 214 { 215 const struct pmbus_driver_info *info = pmbus_get_driver_info(client); 216 const struct isl68137_data *data = to_isl68137_data(info); 217 int ret; 218 219 switch (reg) { 220 case PMBUS_VOUT_MAX: 221 case PMBUS_VOUT_MARGIN_HIGH: 222 case PMBUS_VOUT_MARGIN_LOW: 223 case PMBUS_VOUT_OV_FAULT_LIMIT: 224 case PMBUS_VOUT_UV_FAULT_LIMIT: 225 case PMBUS_VOUT_COMMAND: 226 /* 227 * In cases where a voltage divider is attached to the target 228 * rail between Vout and the Vsense pin, Vout related PMBus 229 * commands should be scaled based on the expected voltage 230 * at the Vsense pin. 231 * I.e. Vsense = Vout * Rout / Rtotal 232 */ > 233 u64 temp = DIV_U64_ROUND_CLOSEST((u64)word * 234 data->channel[page].vout_voltage_divider[0], 235 data->channel[page].vout_voltage_divider[1]); 236 ret = clamp_val(temp, 0, 0xffff); 237 break; 238 default: 239 ret = -ENODATA; 240 break; 241 } 242 return ret; 243 } 244 -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki
