Re: [PATCH 2/2] Input: gpio_keys - Set ABS params when using axes
On Tue, Nov 01, 2016 at 11:25:04AM +0100, Paul Cercueil wrote: > The gpio-keys supports the EV_ABS event type, but does not actually > configure the input device to work with that mode. > > This patch configures the axis corresponding to button->code as being > in the range [-1,+1]. This makes it possible to use gpio-keys to > implement a hat (using the ABS_HAT0X /ABS_HAT0Y axes, with two GPIOs > each with values -1/+1). What if I am emitting values of 10? Also, reporting limits on axis is not a requirement, it is simply a hint. If your userspace refuses axis with unrefined (aka 0) min and max you need to fix it. > > Signed-off-by: Paul Cercueil> --- > drivers/input/keyboard/gpio_keys.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/drivers/input/keyboard/gpio_keys.c > b/drivers/input/keyboard/gpio_keys.c > index 7018c49..1f2850a 100644 > --- a/drivers/input/keyboard/gpio_keys.c > +++ b/drivers/input/keyboard/gpio_keys.c > @@ -540,6 +540,9 @@ static int gpio_keys_setup_key(struct platform_device > *pdev, > > input_set_capability(input, button->type ?: EV_KEY, button->code); > > + if (button->type == EV_ABS) > + input_set_abs_params(input, button->code, -1, 1, 0, 0); > + > /* >* Install custom action to cancel release timer and >* workqueue item. > -- > 2.9.3 > Thanks. -- Dmitry
Re: [PATCH 2/2] Input: gpio_keys - Set ABS params when using axes
On Tue, Nov 01, 2016 at 11:25:04AM +0100, Paul Cercueil wrote: > The gpio-keys supports the EV_ABS event type, but does not actually > configure the input device to work with that mode. > > This patch configures the axis corresponding to button->code as being > in the range [-1,+1]. This makes it possible to use gpio-keys to > implement a hat (using the ABS_HAT0X /ABS_HAT0Y axes, with two GPIOs > each with values -1/+1). What if I am emitting values of 10? Also, reporting limits on axis is not a requirement, it is simply a hint. If your userspace refuses axis with unrefined (aka 0) min and max you need to fix it. > > Signed-off-by: Paul Cercueil > --- > drivers/input/keyboard/gpio_keys.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/drivers/input/keyboard/gpio_keys.c > b/drivers/input/keyboard/gpio_keys.c > index 7018c49..1f2850a 100644 > --- a/drivers/input/keyboard/gpio_keys.c > +++ b/drivers/input/keyboard/gpio_keys.c > @@ -540,6 +540,9 @@ static int gpio_keys_setup_key(struct platform_device > *pdev, > > input_set_capability(input, button->type ?: EV_KEY, button->code); > > + if (button->type == EV_ABS) > + input_set_abs_params(input, button->code, -1, 1, 0, 0); > + > /* >* Install custom action to cancel release timer and >* workqueue item. > -- > 2.9.3 > Thanks. -- Dmitry
[PATCH 2/2] Input: gpio_keys - Set ABS params when using axes
The gpio-keys supports the EV_ABS event type, but does not actually configure the input device to work with that mode. This patch configures the axis corresponding to button->code as being in the range [-1,+1]. This makes it possible to use gpio-keys to implement a hat (using the ABS_HAT0X /ABS_HAT0Y axes, with two GPIOs each with values -1/+1). Signed-off-by: Paul Cercueil--- drivers/input/keyboard/gpio_keys.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/input/keyboard/gpio_keys.c b/drivers/input/keyboard/gpio_keys.c index 7018c49..1f2850a 100644 --- a/drivers/input/keyboard/gpio_keys.c +++ b/drivers/input/keyboard/gpio_keys.c @@ -540,6 +540,9 @@ static int gpio_keys_setup_key(struct platform_device *pdev, input_set_capability(input, button->type ?: EV_KEY, button->code); + if (button->type == EV_ABS) + input_set_abs_params(input, button->code, -1, 1, 0, 0); + /* * Install custom action to cancel release timer and * workqueue item. -- 2.9.3
[PATCH 2/2] Input: gpio_keys - Set ABS params when using axes
The gpio-keys supports the EV_ABS event type, but does not actually configure the input device to work with that mode. This patch configures the axis corresponding to button->code as being in the range [-1,+1]. This makes it possible to use gpio-keys to implement a hat (using the ABS_HAT0X /ABS_HAT0Y axes, with two GPIOs each with values -1/+1). Signed-off-by: Paul Cercueil --- drivers/input/keyboard/gpio_keys.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/input/keyboard/gpio_keys.c b/drivers/input/keyboard/gpio_keys.c index 7018c49..1f2850a 100644 --- a/drivers/input/keyboard/gpio_keys.c +++ b/drivers/input/keyboard/gpio_keys.c @@ -540,6 +540,9 @@ static int gpio_keys_setup_key(struct platform_device *pdev, input_set_capability(input, button->type ?: EV_KEY, button->code); + if (button->type == EV_ABS) + input_set_abs_params(input, button->code, -1, 1, 0, 0); + /* * Install custom action to cancel release timer and * workqueue item. -- 2.9.3