* [PATCH 1/2] input: gpio-keys: add support for disable button from DT
@ 2016-01-12 11:32 ` Laxman Dewangan
0 siblings, 0 replies; 12+ messages in thread
From: Laxman Dewangan @ 2016-01-12 11:32 UTC (permalink / raw)
To: robh+dt, dmitry.torokhov
Cc: pawel.moll, mark.rutland, ijc+devicetree, galak, linus.walleij,
bjorn.andersson, devicetree, linux-kernel, linux-input,
Laxman Dewangan
Add support to disable the button from DT via status property
if given button is not supporting on given platforms.
This will help to re-use the existing dtsi file with status
enable/disable across platforms.
Signed-off-by: Laxman Dewangan <ldewangan@nvidia.com>
---
drivers/input/keyboard/gpio_keys.c | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/drivers/input/keyboard/gpio_keys.c b/drivers/input/keyboard/gpio_keys.c
index bef317f..cf068cb 100644
--- a/drivers/input/keyboard/gpio_keys.c
+++ b/drivers/input/keyboard/gpio_keys.c
@@ -632,6 +632,14 @@ gpio_keys_get_devtree_pdata(struct device *dev)
for_each_child_of_node(node, pp) {
enum of_gpio_flags flags;
+ /* Ignore the button if status of node is disabled */
+ error = of_device_is_available(pp);
+ if (!error) {
+ dev_info(dev, "Button %s is ignored\n", pp->name);
+ pdata->nbuttons--;
+ continue;
+ }
+
button = &pdata->buttons[i++];
button->gpio = of_get_gpio_flags(pp, 0, &flags);
--
2.1.4
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH 1/2] input: gpio-keys: add support for disable button from DT
@ 2016-01-12 11:32 ` Laxman Dewangan
0 siblings, 0 replies; 12+ messages in thread
From: Laxman Dewangan @ 2016-01-12 11:32 UTC (permalink / raw)
To: robh+dt, dmitry.torokhov
Cc: pawel.moll, mark.rutland, ijc+devicetree, galak, linus.walleij,
bjorn.andersson, devicetree, linux-kernel, linux-input,
Laxman Dewangan
Add support to disable the button from DT via status property
if given button is not supporting on given platforms.
This will help to re-use the existing dtsi file with status
enable/disable across platforms.
Signed-off-by: Laxman Dewangan <ldewangan@nvidia.com>
---
drivers/input/keyboard/gpio_keys.c | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/drivers/input/keyboard/gpio_keys.c b/drivers/input/keyboard/gpio_keys.c
index bef317f..cf068cb 100644
--- a/drivers/input/keyboard/gpio_keys.c
+++ b/drivers/input/keyboard/gpio_keys.c
@@ -632,6 +632,14 @@ gpio_keys_get_devtree_pdata(struct device *dev)
for_each_child_of_node(node, pp) {
enum of_gpio_flags flags;
+ /* Ignore the button if status of node is disabled */
+ error = of_device_is_available(pp);
+ if (!error) {
+ dev_info(dev, "Button %s is ignored\n", pp->name);
+ pdata->nbuttons--;
+ continue;
+ }
+
button = &pdata->buttons[i++];
button->gpio = of_get_gpio_flags(pp, 0, &flags);
--
2.1.4
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH 2/2] input: gpio-keys: add support to name the gpio-keys
2016-01-12 11:32 ` Laxman Dewangan
@ 2016-01-12 11:32 ` Laxman Dewangan
-1 siblings, 0 replies; 12+ messages in thread
From: Laxman Dewangan @ 2016-01-12 11:32 UTC (permalink / raw)
To: robh+dt, dmitry.torokhov
Cc: pawel.moll, mark.rutland, ijc+devicetree, galak, linus.walleij,
bjorn.andersson, devicetree, linux-kernel, linux-input,
Laxman Dewangan
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.
Signed-off-by: Laxman Dewangan <ldewangan@nvidia.com>
---
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.
- autorepeat: Boolean, Enable auto repeat feature of Linux input
subsystem.
diff --git a/drivers/input/keyboard/gpio_keys.c b/drivers/input/keyboard/gpio_keys.c
index cf068cb..c95dc3b 100644
--- a/drivers/input/keyboard/gpio_keys.c
+++ b/drivers/input/keyboard/gpio_keys.c
@@ -628,6 +628,8 @@ gpio_keys_get_devtree_pdata(struct device *dev)
pdata->rep = !!of_get_property(node, "autorepeat", NULL);
+ of_property_read_string(node, "gpio-keys,name", &pdata->name);
+
i = 0;
for_each_child_of_node(node, pp) {
enum of_gpio_flags flags;
--
2.1.4
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH 2/2] input: gpio-keys: add support to name the gpio-keys
@ 2016-01-12 11:32 ` Laxman Dewangan
0 siblings, 0 replies; 12+ messages in thread
From: Laxman Dewangan @ 2016-01-12 11:32 UTC (permalink / raw)
To: robh+dt, dmitry.torokhov
Cc: pawel.moll, mark.rutland, ijc+devicetree, galak, linus.walleij,
bjorn.andersson, devicetree, linux-kernel, linux-input,
Laxman Dewangan
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.
Signed-off-by: Laxman Dewangan <ldewangan@nvidia.com>
---
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.
- autorepeat: Boolean, Enable auto repeat feature of Linux input
subsystem.
diff --git a/drivers/input/keyboard/gpio_keys.c b/drivers/input/keyboard/gpio_keys.c
index cf068cb..c95dc3b 100644
--- a/drivers/input/keyboard/gpio_keys.c
+++ b/drivers/input/keyboard/gpio_keys.c
@@ -628,6 +628,8 @@ gpio_keys_get_devtree_pdata(struct device *dev)
pdata->rep = !!of_get_property(node, "autorepeat", NULL);
+ of_property_read_string(node, "gpio-keys,name", &pdata->name);
+
i = 0;
for_each_child_of_node(node, pp) {
enum of_gpio_flags flags;
--
2.1.4
^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH 2/2] input: gpio-keys: add support to name the gpio-keys
2016-01-12 11:32 ` Laxman Dewangan
(?)
@ 2016-01-12 14:43 ` Rob Herring
2016-01-12 14:55 ` Laxman Dewangan
-1 siblings, 1 reply; 12+ messages in thread
From: Rob Herring @ 2016-01-12 14:43 UTC (permalink / raw)
To: Laxman Dewangan
Cc: dmitry.torokhov, pawel.moll, mark.rutland, ijc+devicetree, galak,
linus.walleij, bjorn.andersson, devicetree, linux-kernel,
linux-input
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?
> Signed-off-by: Laxman Dewangan <ldewangan@nvidia.com>
> ---
> 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?
Either way, I'm concerned this doesn't work for other key bindings
which have multiple keys.
> - autorepeat: Boolean, Enable auto repeat feature of Linux input
> subsystem.
>
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 2/2] input: gpio-keys: add support to name the gpio-keys
2016-01-12 14:43 ` Rob Herring
@ 2016-01-12 14:55 ` Laxman Dewangan
0 siblings, 0 replies; 12+ messages in thread
From: Laxman Dewangan @ 2016-01-12 14:55 UTC (permalink / raw)
To: Rob Herring
Cc: dmitry.torokhov, pawel.moll, mark.rutland, ijc+devicetree, galak,
linus.walleij, bjorn.andersson, devicetree, linux-kernel,
linux-input
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@nvidia.com>
>> ---
>> 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.
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 2/2] input: gpio-keys: add support to name the gpio-keys
@ 2016-01-12 14:55 ` Laxman Dewangan
0 siblings, 0 replies; 12+ messages in thread
From: Laxman Dewangan @ 2016-01-12 14:55 UTC (permalink / raw)
To: Rob Herring
Cc: dmitry.torokhov, pawel.moll, mark.rutland, ijc+devicetree, galak,
linus.walleij, bjorn.andersson, devicetree, linux-kernel,
linux-input
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@nvidia.com>
>> ---
>> 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.
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 2/2] input: gpio-keys: add support to name the gpio-keys
@ 2016-01-12 15:23 ` Rob Herring
0 siblings, 0 replies; 12+ messages in thread
From: Rob Herring @ 2016-01-12 15:23 UTC (permalink / raw)
To: Laxman Dewangan
Cc: Dmitry Torokhov, Pawel Moll, Mark Rutland, Ian Campbell,
Kumar Gala, Linus Walleij, Bjorn Andersson, devicetree,
linux-kernel, linux-input
On Tue, Jan 12, 2016 at 8:55 AM, Laxman Dewangan <ldewangan@nvidia.com> wrote:
>
> 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.
Okay, got it.
>>> Signed-off-by: Laxman Dewangan <ldewangan@nvidia.com>
>>> ---
>>> 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?
No, the standard label property. See gpio-leds for an example.
>> 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.
So label is exactly for this purpose of defining user meaningful names.
Rob
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 2/2] input: gpio-keys: add support to name the gpio-keys
@ 2016-01-12 15:23 ` Rob Herring
0 siblings, 0 replies; 12+ messages in thread
From: Rob Herring @ 2016-01-12 15:23 UTC (permalink / raw)
To: Laxman Dewangan
Cc: Dmitry Torokhov, Pawel Moll, Mark Rutland, Ian Campbell,
Kumar Gala, Linus Walleij, Bjorn Andersson,
devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-input-u79uwXL29TY76Z2rM5mHXA
On Tue, Jan 12, 2016 at 8:55 AM, Laxman Dewangan <ldewangan-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> wrote:
>
> 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.
Okay, got it.
>>> Signed-off-by: Laxman Dewangan <ldewangan-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
>>> ---
>>> 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?
No, the standard label property. See gpio-leds for an example.
>> 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.
So label is exactly for this purpose of defining user meaningful names.
Rob
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 2/2] input: gpio-keys: add support to name the gpio-keys
2016-01-12 15:23 ` Rob Herring
(?)
@ 2016-01-12 15:24 ` Laxman Dewangan
2016-01-14 9:16 ` Linus Walleij
-1 siblings, 1 reply; 12+ messages in thread
From: Laxman Dewangan @ 2016-01-12 15:24 UTC (permalink / raw)
To: Rob Herring
Cc: Dmitry Torokhov, Pawel Moll, Mark Rutland, Ian Campbell,
Kumar Gala, Linus Walleij, Bjorn Andersson, devicetree,
linux-kernel, linux-input
On Tuesday 12 January 2016 08:53 PM, Rob Herring wrote:
> On Tue, Jan 12, 2016 at 8:55 AM, Laxman Dewangan <ldewangan@nvidia.com> wrote:
>> On Tuesday 12 January 2016 08:13 PM, Rob Herring wrote:
>>> On Tue, Jan 12, 2016 at 05:02:50PM +0530, Laxman Dewangan wrote:
> So label is exactly for this purpose of defining user meaningful names.
>
Yes, I can use the label for name. I will update the patch.
Just for understanding, is "label" is better property name to pass the
specific name?
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 1/2] input: gpio-keys: add support for disable button from DT
2016-01-12 11:32 ` Laxman Dewangan
(?)
(?)
@ 2016-01-12 21:19 ` Dmitry Torokhov
-1 siblings, 0 replies; 12+ messages in thread
From: Dmitry Torokhov @ 2016-01-12 21:19 UTC (permalink / raw)
To: Laxman Dewangan
Cc: robh+dt, pawel.moll, mark.rutland, ijc+devicetree, galak,
linus.walleij, bjorn.andersson, devicetree, linux-kernel,
linux-input
On Tue, Jan 12, 2016 at 05:02:49PM +0530, Laxman Dewangan wrote:
> Add support to disable the button from DT via status property
> if given button is not supporting on given platforms.
>
> This will help to re-use the existing dtsi file with status
> enable/disable across platforms.
>
> Signed-off-by: Laxman Dewangan <ldewangan@nvidia.com>
> ---
> drivers/input/keyboard/gpio_keys.c | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/drivers/input/keyboard/gpio_keys.c b/drivers/input/keyboard/gpio_keys.c
> index bef317f..cf068cb 100644
> --- a/drivers/input/keyboard/gpio_keys.c
> +++ b/drivers/input/keyboard/gpio_keys.c
> @@ -632,6 +632,14 @@ gpio_keys_get_devtree_pdata(struct device *dev)
> for_each_child_of_node(node, pp) {
> enum of_gpio_flags flags;
>
> + /* Ignore the button if status of node is disabled */
> + error = of_device_is_available(pp);
> + if (!error) {
> + dev_info(dev, "Button %s is ignored\n", pp->name);
I'd rather have it as dev_dbg() please.
> + pdata->nbuttons--;
> + continue;
> + }
> +
> button = &pdata->buttons[i++];
>
> button->gpio = of_get_gpio_flags(pp, 0, &flags);
> --
> 2.1.4
>
--
Dmitry
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 2/2] input: gpio-keys: add support to name the gpio-keys
2016-01-12 15:24 ` Laxman Dewangan
@ 2016-01-14 9:16 ` Linus Walleij
0 siblings, 0 replies; 12+ messages in thread
From: Linus Walleij @ 2016-01-14 9:16 UTC (permalink / raw)
To: Laxman Dewangan
Cc: Rob Herring, Dmitry Torokhov, Pawel Moll, Mark Rutland,
Ian Campbell, Kumar Gala, Bjorn Andersson, devicetree,
linux-kernel, linux-input
On Tue, Jan 12, 2016 at 4:24 PM, Laxman Dewangan <ldewangan@nvidia.com> wrote:
> On Tuesday 12 January 2016 08:53 PM, Rob Herring wrote:
>> On Tue, Jan 12, 2016 at 8:55 AM, Laxman Dewangan <ldewangan@nvidia.com>
>> wrote:
>>>
>>> On Tuesday 12 January 2016 08:13 PM, Rob Herring wrote:
>>>>
>>>> On Tue, Jan 12, 2016 at 05:02:50PM +0530, Laxman Dewangan wrote:
>
>
>> So label is exactly for this purpose of defining user meaningful names.
>>
> Yes, I can use the label for name. I will update the patch.
>
> Just for understanding, is "label" is better property name to pass the
> specific name?
Yes, look at this from
arch/arm64/boot/dts/arm/juno-motherboard.dtsi:
gpio_keys {
compatible = "gpio-keys";
#address-cells = <1>;
#size-cells = <0>;
button@1 {
debounce_interval = <50>;
wakeup-source;
linux,code = <116>;
label = "POWER";
gpios = <&iofpga_gpio0 0 0x4>;
};
button@2 {
debounce_interval = <50>;
wakeup-source;
linux,code = <102>;
label = "HOME";
gpios = <&iofpga_gpio0 1 0x4>;
};
(...)
Currently these names (POWER, HOME etc) appear nicely in
/proc/interrupts, so I guess the same should be used for input?
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2016-01-14 9:18 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-01-12 11:32 [PATCH 1/2] input: gpio-keys: add support for disable button from DT Laxman Dewangan
2016-01-12 11:32 ` Laxman Dewangan
2016-01-12 11:32 ` [PATCH 2/2] input: gpio-keys: add support to name the gpio-keys Laxman Dewangan
2016-01-12 11:32 ` Laxman Dewangan
2016-01-12 14:43 ` Rob Herring
2016-01-12 14:55 ` Laxman Dewangan
2016-01-12 14:55 ` Laxman Dewangan
2016-01-12 15:23 ` Rob Herring
2016-01-12 15:23 ` Rob Herring
2016-01-12 15:24 ` Laxman Dewangan
2016-01-14 9:16 ` Linus Walleij
2016-01-12 21:19 ` [PATCH 1/2] input: gpio-keys: add support for disable button from DT Dmitry Torokhov
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.