Re: [PATCH 2/2] input: gpio-keys: add support to name the gpio-keys

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On Tuesday 12 January 2016 08:13 PM, Rob Herring wrote:
On Tue, Jan 12, 2016 at 05:02:50PM +0530, Laxman Dewangan wrote:
Add support from DT to name the gpio-keys driver to have
desired key name for this input system.

This helps userspace code to get name and perform proper
event to key mapping in some of cases. In android, the
user space map the key code to application code using
key character layout and for this it uses the name of
input system.
Why is specifying the key code not enough?

This is the name for input system instead of individual keys.
In android system, the key mapping is done in user space. By having name offers to specifying the customized key character layout instead of generic one form android which is fall-back.



Signed-off-by: Laxman Dewangan <ldewangan@xxxxxxxxxx>
---
  Documentation/devicetree/bindings/input/gpio-keys.txt | 1 +
  drivers/input/keyboard/gpio_keys.c                    | 2 ++
  2 files changed, 3 insertions(+)

diff --git a/Documentation/devicetree/bindings/input/gpio-keys.txt b/Documentation/devicetree/bindings/input/gpio-keys.txt
index cf1333d..2e6bcd2 100644
--- a/Documentation/devicetree/bindings/input/gpio-keys.txt
+++ b/Documentation/devicetree/bindings/input/gpio-keys.txt
@@ -4,6 +4,7 @@ Required properties:
  	- compatible = "gpio-keys";
Optional properties:
+	- gpio-keys,name: String, name of the gpio-keys input system.
gpio-keys is not a vendor.

Would "label" work for you instead?

"label" means node name?


Either way, I'm concerned this doesn't work for other key bindings
which have multiple keys.



This is not for the individual keys, its for input system like /sys/class/input/input0/name.


--
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



[Index of Archives]     [Linux Media Devel]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Linux Wireless Networking]     [Linux Omap]

  Powered by Linux