Re: [PATCH 2/2] power: supply: Add bd718(15/28/78) charger driver
From: Dan Carpenter
Date: Tue Aug 19 2025 - 02:15:10 EST
Hi Andreas,
kernel test robot noticed the following build warnings:
url: https://github.com/intel-lab-lkp/linux/commits/Andreas-Kemnade/mfd-bd71828-bd71815-prepare-for-power-supply-support/20250817-032146
base: 8f5ae30d69d7543eee0d70083daf4de8fe15d585
patch link: https://lore.kernel.org/r/20250816-bd71828-charger-v1-2-71b11bde5c73%40kemnade.info
patch subject: [PATCH 2/2] power: supply: Add bd718(15/28/78) charger driver
config: loongarch-randconfig-r073-20250818 (https://download.01.org/0day-ci/archive/20250819/202508191303.UzPStkjj-lkp@xxxxxxxxx/config)
compiler: loongarch64-linux-gcc (GCC) 15.1.0
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>
| Reported-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
| Closes: https://lore.kernel.org/r/202508191303.UzPStkjj-lkp@xxxxxxxxx/
smatch warnings:
drivers/power/supply/bd71828-power.c:298 bd71828_get_temp() error: uninitialized symbol 't'.
drivers/power/supply/bd71828-power.c:559 bd71828_battery_get_property() error: uninitialized symbol 'tmp'.
vim +/t +298 drivers/power/supply/bd71828-power.c
c57d31029550a0 Andreas Kemnade 2025-08-16 286 static int bd71828_get_temp(struct bd71828_power *pwr, int *temp)
c57d31029550a0 Andreas Kemnade 2025-08-16 287 {
c57d31029550a0 Andreas Kemnade 2025-08-16 288 uint16_t t;
c57d31029550a0 Andreas Kemnade 2025-08-16 289 int ret;
c57d31029550a0 Andreas Kemnade 2025-08-16 290 int tmp = 200 * 10000;
c57d31029550a0 Andreas Kemnade 2025-08-16 291
c57d31029550a0 Andreas Kemnade 2025-08-16 292 ret = bd7182x_read16_himask(pwr, pwr->regs->btemp_vth,
c57d31029550a0 Andreas Kemnade 2025-08-16 293 BD71828_MASK_VM_BTMP_U, &t);
c57d31029550a0 Andreas Kemnade 2025-08-16 294 if (ret || t > 3200)
c57d31029550a0 Andreas Kemnade 2025-08-16 295 dev_err(pwr->dev,
c57d31029550a0 Andreas Kemnade 2025-08-16 296 "Failed to read system min average voltage\n");
We should return the error here.
c57d31029550a0 Andreas Kemnade 2025-08-16 297
c57d31029550a0 Andreas Kemnade 2025-08-16 @298 tmp -= 625ULL * (unsigned int)t;
If bd7182x_read16_himask() fails then t is uninitialized or invalid.
c57d31029550a0 Andreas Kemnade 2025-08-16 299 *temp = tmp / 1000;
c57d31029550a0 Andreas Kemnade 2025-08-16 300
c57d31029550a0 Andreas Kemnade 2025-08-16 301 return ret;
c57d31029550a0 Andreas Kemnade 2025-08-16 302 }
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki