Re: [PATCH v3 1/6] hid: new driver for PicoLCD device

From: Bruno PrÃmont
Date: Fri Mar 26 2010 - 05:30:46 EST


On Thu, 25 March 2010 Dmitry Torokhov <dmitry.torokhov@xxxxxxxxx> wrote:
> On Wed, Mar 24, 2010 at 11:40:22PM +0100, Bruno PrÃmont wrote:
> > + for (i = 0; i < PICOLCD_KEYS; i++) {
> > + int key = ((int *)idev->keycode)[i];
>
> Keycodes are now short, not int. Also, just do:
>
> input_set_capability(idev, EV_KEY, data->keycode[i]);
>
> > + if (key < KEY_MAX && key >= 0)
> > + input_set_capability(idev, EV_KEY, key);

Oops, I was not careful enough when switching over...
Thanks for catching,
Bruno
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/