Re: [PATCH] OMAP: Add keypad driver

[Tony Lindgren <tony@atomide.com>]

Hi,

* Dmitry Torokhov <dmitry.torokhov@gmail.com> [160107 10:54]:
> On Thu, Jan 7, 2016 at 1:37 AM, Dan Carpenter <dan.carpenter@oracle.com> wrote:
> > Hello file (standard input) matches,
> >
> > The patch ad4e09b16ad3: "[PATCH] OMAP: Add keypad driver", leads to the
> > following static checker warning:
> >
> >         drivers/input/keyboard/omap-keypad.c:158 omap_kp_tasklet()
> >         warn: 'keycodes[]' is never negative.
> 
> It looks like you are not resolving commits correctly, the original
> submission did not have this issue.
> 
> >
> > drivers/input/keyboard/omap-keypad.c
...
> >    152  #ifdef NEW_BOARD_LEARNING_MODE
> >    153                          printk(KERN_INFO "omap-keypad: key %d-%d %s\n", col,
> >    154                                 row, (new_state[col] & (1 << row)) ?
> >    155                                 "pressed" : "released");
> >    156  #else
> >    157                          key = keycodes[MATRIX_SCAN_CODE(row, col, row_shift)];
> >    158                          if (key < 0) {
> >                                     ^^^^^^^
> > Never true.  Not sure what was intended.
> 
> It looks like this check was broken by
> da1f026b532ce944d74461497dc6d8c16456466e (Keyboard: omap-keypad: use
> matrix_keypad.h). Previously the driver would expect a list of known
> keys and would scan it and return -1 if key was not found. Now we have
> 2 options:
> 
> 1. Simply remove the check
> 2. Change the condition to "if (key == KEY_RESERVED)"
> 
> I do not really have preference. Tony?

Sounds like the check is not needed if it has not been used for
past five years, so my preference is option #1 then.

Regards,

Tony