On Mon, Sep 13, 2010 at 09:28:07AM -0700, Dmitry Torokhov wrote:
> On Thu, Sep 09, 2010 at 03:40:04PM +0300, Ville Syrjälä wrote:
> > On Wed, Sep 08, 2010 at 12:42:05AM -0700, Dmitry Torokhov wrote:
> > > Switch the code to use new style of getkeycode and setkeycode
> > > methods to allow retrieving and setting keycodes not only by
> > > their scancodes but also by index.
> > >
> > > Signed-off-by: Dmitry Torokhov <d...@mail.ru>
> > > ---
> > >
> > > drivers/input/misc/ati_remote2.c | 93
> > > +++++++++++++++++++++++++++-----------
> > > 1 files changed, 65 insertions(+), 28 deletions(-)
> > >
> > > diff --git a/drivers/input/misc/ati_remote2.c
> > > b/drivers/input/misc/ati_remote2.c
> > > index 2325765..b2e0d82 100644
> > > --- a/drivers/input/misc/ati_remote2.c
> > > +++ b/drivers/input/misc/ati_remote2.c
> > > @@ -483,51 +483,88 @@ static void ati_remote2_complete_key(struct urb
> > > *urb)
> > > }
> > >
> > > static int ati_remote2_getkeycode(struct input_dev *idev,
> > > - unsigned int scancode, unsigned int *keycode)
> > > + struct input_keymap_entry *ke)
> > > {
> > > struct ati_remote2 *ar2 = input_get_drvdata(idev);
> > > unsigned int mode;
> > > - int index;
> > > + int offset;
> > > + unsigned int index;
> > > + unsigned int scancode;
> > > +
> > > + if (ke->flags & INPUT_KEYMAP_BY_INDEX) {
> > > + index = ke->index;
> > > + if (index >= (ATI_REMOTE2_MODES - 1) *
> > ^^^^
> > That -1 looks wrong. Same in setkeycode().
> >
>
> Yes, indeed. Thanks for noticing.
I fixed this bug locally and gave this a short whirl with my RWII.
I tried both the old and new style keycode ioctls. Everything
worked as expected.
So if you want more tags feel free to add my Acked-by and Tested-by
for this (assuming the off-by-one fix is included) and you can add my
Tested-by for patch 1/6 as well.
--
Ville Syrjälä
syrj...@sci.fi
http://www.sci.fi/~syrjala/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
