Re: [PATCH 2/2] Input: charlieplex_keypad: add GPIO charlieplex keypad

[kernel test robot]

Hi Hugo,

kernel test robot noticed the following build warnings:

[auto build test WARNING on ed8a4ef29da3821ee3155d3b1925fa67fc92aae2]

url:    https://github.com/intel-lab-lkp/linux/commits/Hugo-Villeneuve/dt-bindings-input-add-GPIO-charlieplex-keypad/20260204-001338
base:   ed8a4ef29da3821ee3155d3b1925fa67fc92aae2
patch link:    https://lore.kernel.org/r/20260203155023.536103-3-hugo%40hugovil.com
patch subject: [PATCH 2/2] Input: charlieplex_keypad: add GPIO charlieplex keypad
config: parisc-randconfig-r072-20260204 (https://download.01.org/0day-ci/archive/20260204/202602040703.1Bu56Htu-lkp@xxxxxxxxx/config)
compiler: hppa-linux-gcc (GCC) 8.5.0
smatch version: v0.5.0-8994-gd50c5a4c
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260204/202602040703.1Bu56Htu-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/202602040703.1Bu56Htu-lkp@xxxxxxxxx/

All warnings (new ones prefixed by >>):

   drivers/input/keyboard/charlieplex_keypad.c: In function 'charlieplex_keypad_init_gpio':
>> drivers/input/keyboard/charlieplex_keypad.c:120:6: warning: variable 'nkeys' set but not used [-Wunused-but-set-variable]
     int nkeys;
         ^~~~~


vim +/nkeys +120 drivers/input/keyboard/charlieplex_keypad.c

   115	
   116	static int charlieplex_keypad_init_gpio(struct platform_device *pdev,
   117						struct charlieplex_keypad *keypad)
   118	{
   119		bool active_low;
 > 120		int nkeys;
   121		int i;
   122	
   123		keypad->line_gpios = devm_gpiod_get_array(&pdev->dev, "line", GPIOD_IN);
   124		if (IS_ERR(keypad->line_gpios))
   125			return PTR_ERR(keypad->line_gpios);
   126	
   127		keypad->nlines = keypad->line_gpios->ndescs;
   128	
   129		if (keypad->nlines > MATRIX_MAX_ROWS)
   130			return -EINVAL;
   131	
   132		nkeys = (keypad->nlines * keypad->nlines) - keypad->nlines;
   133	
   134		active_low = device_property_read_bool(&pdev->dev, "gpio-activelow");
   135	
   136		for (i = 0; i < keypad->nlines; i++) {
   137			gpiod_set_consumer_name(keypad->line_gpios->desc[i], "charlieplex_kbd_line");
   138	
   139			if (active_low ^ gpiod_is_active_low(keypad->line_gpios->desc[i]))
   140				gpiod_toggle_active_low(keypad->line_gpios->desc[i]);
   141		}
   142	
   143		return 0;
   144	}
   145	

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki