Hi,
* Dmitry Torokhov <dmitry.torokhov@xxxxxxxxx> [160107 10:54]:
> On Thu, Jan 7, 2016 at 1:37 AM, Dan Carpenter <dan.carpenter@xxxxxxxxxx> 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
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
