Messages in this thread | | | Date | Fri, 26 Mar 2010 10:29:51 +0100 | From | Bruno Prémont <> | Subject | Re: [PATCH v3 1/6] hid: new driver for PicoLCD device |
| |
On Thu, 25 March 2010 Dmitry Torokhov <dmitry.torokhov@gmail.com> 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@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |