arch/sh/boards/mach-hp6xx/hp6xx_apm.c:32:22: warning: variable 'backup' set but not used
From: kernel test robot
Date: Fri Jul 07 2023 - 15:43:06 EST
Hi Masahiro,
First bad commit (maybe != root cause):
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 4f6b6c2b2f86b7878a770736bf478d8a263ff0bc
commit: 706afcea16cd83fecb7c2229ccc31bb237ffdbef sh: Fix -Wmissing-include-dirs warnings for various platforms
date: 2 days ago
config: sh-hp6xx_defconfig (https://download.01.org/0day-ci/archive/20230708/202307080311.ZgsVj2PR-lkp@xxxxxxxxx/config)
compiler: sh4-linux-gcc (GCC) 12.3.0
reproduce: (https://download.01.org/0day-ci/archive/20230708/202307080311.ZgsVj2PR-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/202307080311.ZgsVj2PR-lkp@xxxxxxxxx/
All warnings (new ones prefixed by >>):
arch/sh/boards/mach-hp6xx/hp6xx_apm.c: In function 'hp6x0_apm_get_power_status':
>> arch/sh/boards/mach-hp6xx/hp6xx_apm.c:32:22: warning: variable 'backup' set but not used [-Wunused-but-set-variable]
32 | int battery, backup, charging, percentage;
| ^~~~~~
vim +/backup +32 arch/sh/boards/mach-hp6xx/hp6xx_apm.c
dd4f99b42dcce8 arch/sh/boards/hp6xx/hp6xx_apm.c Paul Mundt 2008-03-06 29
0a9b0db19262db arch/sh/boards/hp6xx/hp6xx_apm.c Paul Mundt 2007-01-24 30 static void hp6x0_apm_get_power_status(struct apm_power_info *info)
3aa770e7972723 arch/sh/boards/hp6xx/hp6xx_apm.c Andriy Skulysh 2006-09-27 31 {
0a9b0db19262db arch/sh/boards/hp6xx/hp6xx_apm.c Paul Mundt 2007-01-24 @32 int battery, backup, charging, percentage;
3aa770e7972723 arch/sh/boards/hp6xx/hp6xx_apm.c Andriy Skulysh 2006-09-27 33 u8 pgdr;
3aa770e7972723 arch/sh/boards/hp6xx/hp6xx_apm.c Andriy Skulysh 2006-09-27 34
0a9b0db19262db arch/sh/boards/hp6xx/hp6xx_apm.c Paul Mundt 2007-01-24 35 battery = adc_single(ADC_CHANNEL_BATTERY);
0a9b0db19262db arch/sh/boards/hp6xx/hp6xx_apm.c Paul Mundt 2007-01-24 36 backup = adc_single(ADC_CHANNEL_BACKUP);
0a9b0db19262db arch/sh/boards/hp6xx/hp6xx_apm.c Paul Mundt 2007-01-24 37 charging = adc_single(ADC_CHANNEL_CHARGE);
3aa770e7972723 arch/sh/boards/hp6xx/hp6xx_apm.c Andriy Skulysh 2006-09-27 38
3aa770e7972723 arch/sh/boards/hp6xx/hp6xx_apm.c Andriy Skulysh 2006-09-27 39 percentage = 100 * (battery - HP680_BATTERY_MIN) /
3aa770e7972723 arch/sh/boards/hp6xx/hp6xx_apm.c Andriy Skulysh 2006-09-27 40 (HP680_BATTERY_MAX - HP680_BATTERY_MIN);
3aa770e7972723 arch/sh/boards/hp6xx/hp6xx_apm.c Andriy Skulysh 2006-09-27 41
8b03c040e4efaa arch/sh/boards/hp6xx/hp6xx_apm.c Kristoffer Ericson 2008-03-04 42 /* % of full battery */
8b03c040e4efaa arch/sh/boards/hp6xx/hp6xx_apm.c Kristoffer Ericson 2008-03-04 43 info->battery_life = percentage;
8b03c040e4efaa arch/sh/boards/hp6xx/hp6xx_apm.c Kristoffer Ericson 2008-03-04 44
8b03c040e4efaa arch/sh/boards/hp6xx/hp6xx_apm.c Kristoffer Ericson 2008-03-04 45 /* We want our estimates in minutes */
8b03c040e4efaa arch/sh/boards/hp6xx/hp6xx_apm.c Kristoffer Ericson 2008-03-04 46 info->units = 0;
8b03c040e4efaa arch/sh/boards/hp6xx/hp6xx_apm.c Kristoffer Ericson 2008-03-04 47
8b03c040e4efaa arch/sh/boards/hp6xx/hp6xx_apm.c Kristoffer Ericson 2008-03-04 48 /* Extremely(!!) rough estimate, we will replace this with a datalist later on */
8b03c040e4efaa arch/sh/boards/hp6xx/hp6xx_apm.c Kristoffer Ericson 2008-03-04 49 info->time = (2 * battery);
8b03c040e4efaa arch/sh/boards/hp6xx/hp6xx_apm.c Kristoffer Ericson 2008-03-04 50
0a9b0db19262db arch/sh/boards/hp6xx/hp6xx_apm.c Paul Mundt 2007-01-24 51 info->ac_line_status = (battery > HP680_BATTERY_AC_ON) ?
3aa770e7972723 arch/sh/boards/hp6xx/hp6xx_apm.c Andriy Skulysh 2006-09-27 52 APM_AC_ONLINE : APM_AC_OFFLINE;
3aa770e7972723 arch/sh/boards/hp6xx/hp6xx_apm.c Andriy Skulysh 2006-09-27 53
9d56dd3b083a3b arch/sh/boards/mach-hp6xx/hp6xx_apm.c Paul Mundt 2010-01-26 54 pgdr = __raw_readb(PGDR);
3aa770e7972723 arch/sh/boards/hp6xx/hp6xx_apm.c Andriy Skulysh 2006-09-27 55 if (pgdr & PGDR_MAIN_BATTERY_OUT) {
0a9b0db19262db arch/sh/boards/hp6xx/hp6xx_apm.c Paul Mundt 2007-01-24 56 info->battery_status = APM_BATTERY_STATUS_NOT_PRESENT;
0a9b0db19262db arch/sh/boards/hp6xx/hp6xx_apm.c Paul Mundt 2007-01-24 57 info->battery_flag = 0x80;
3aa770e7972723 arch/sh/boards/hp6xx/hp6xx_apm.c Andriy Skulysh 2006-09-27 58 } else if (charging < 8) {
0a9b0db19262db arch/sh/boards/hp6xx/hp6xx_apm.c Paul Mundt 2007-01-24 59 info->battery_status = APM_BATTERY_STATUS_CHARGING;
0a9b0db19262db arch/sh/boards/hp6xx/hp6xx_apm.c Paul Mundt 2007-01-24 60 info->battery_flag = 0x08;
8b03c040e4efaa arch/sh/boards/hp6xx/hp6xx_apm.c Kristoffer Ericson 2008-03-04 61 info->ac_line_status = 0x01;
3aa770e7972723 arch/sh/boards/hp6xx/hp6xx_apm.c Andriy Skulysh 2006-09-27 62 } else if (percentage <= APM_CRITICAL) {
0a9b0db19262db arch/sh/boards/hp6xx/hp6xx_apm.c Paul Mundt 2007-01-24 63 info->battery_status = APM_BATTERY_STATUS_CRITICAL;
0a9b0db19262db arch/sh/boards/hp6xx/hp6xx_apm.c Paul Mundt 2007-01-24 64 info->battery_flag = 0x04;
3aa770e7972723 arch/sh/boards/hp6xx/hp6xx_apm.c Andriy Skulysh 2006-09-27 65 } else if (percentage <= APM_LOW) {
0a9b0db19262db arch/sh/boards/hp6xx/hp6xx_apm.c Paul Mundt 2007-01-24 66 info->battery_status = APM_BATTERY_STATUS_LOW;
0a9b0db19262db arch/sh/boards/hp6xx/hp6xx_apm.c Paul Mundt 2007-01-24 67 info->battery_flag = 0x02;
3aa770e7972723 arch/sh/boards/hp6xx/hp6xx_apm.c Andriy Skulysh 2006-09-27 68 } else {
0a9b0db19262db arch/sh/boards/hp6xx/hp6xx_apm.c Paul Mundt 2007-01-24 69 info->battery_status = APM_BATTERY_STATUS_HIGH;
0a9b0db19262db arch/sh/boards/hp6xx/hp6xx_apm.c Paul Mundt 2007-01-24 70 info->battery_flag = 0x01;
3aa770e7972723 arch/sh/boards/hp6xx/hp6xx_apm.c Andriy Skulysh 2006-09-27 71 }
3aa770e7972723 arch/sh/boards/hp6xx/hp6xx_apm.c Andriy Skulysh 2006-09-27 72 }
3aa770e7972723 arch/sh/boards/hp6xx/hp6xx_apm.c Andriy Skulysh 2006-09-27 73
:::::: The code at line 32 was first introduced by commit
:::::: 0a9b0db19262dbb09f3a34195e68cafd5dc3fa10 [APM] SH: Convert to use shared APM emulation.
:::::: TO: Paul Mundt <lethal@xxxxxxxxxxxx>
:::::: CC: Ralf Baechle <ralf@xxxxxxxxxxxxxx>
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki