* [PATCH 14/19] ARM: dts: shmobile/r8a7xxx: replace gpio-key,wakeup with wakeup-source property
[not found] <1445422216-29375-1-git-send-email-sudeep.holla@arm.com>
@ 2015-10-21 10:10 ` Sudeep Holla
2015-10-21 10:18 ` Geert Uytterhoeven
0 siblings, 1 reply; 6+ messages in thread
From: Sudeep Holla @ 2015-10-21 10:10 UTC (permalink / raw)
To: linux-arm-kernel
Though the keyboard driver for GPIO buttons(gpio-keys) will continue to
check for/support the legacy "gpio-key,wakeup" boolean property to
enable gpio buttons as wakeup source, "wakeup-source" is the new
standard binding.
This patch replaces the legacy "gpio-key,wakeup" with the unified
"wakeup-source" property in order to avoid any futher copy-paste
duplication.
Cc: Simon Horman <horms@verge.net.au>
Cc: Magnus Damm <magnus.damm@gmail.com>
Cc: linux-sh@vger.kernel.org
Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>
---
arch/arm/boot/dts/r8a73a4-ape6evm.dts | 2 +-
arch/arm/boot/dts/r8a7740-armadillo800eva.dts | 2 +-
arch/arm/boot/dts/r8a7790-lager.dts | 8 ++++----
arch/arm/boot/dts/r8a7791-koelsch.dts | 22 +++++++++++-----------
arch/arm/boot/dts/sh73a0-kzm9g.dts | 2 +-
5 files changed, 18 insertions(+), 18 deletions(-)
diff --git a/arch/arm/boot/dts/r8a73a4-ape6evm.dts b/arch/arm/boot/dts/r8a73a4-ape6evm.dts
index a4c425923c05..9b7eabd3eba3 100644
--- a/arch/arm/boot/dts/r8a73a4-ape6evm.dts
+++ b/arch/arm/boot/dts/r8a73a4-ape6evm.dts
@@ -110,7 +110,7 @@
gpios = <&pfc 324 GPIO_ACTIVE_LOW>;
linux,code = <KEY_0>;
label = "S16";
- gpio-key,wakeup;
+ wakeup-source;
};
menu-key {
diff --git a/arch/arm/boot/dts/r8a7740-armadillo800eva.dts b/arch/arm/boot/dts/r8a7740-armadillo800eva.dts
index 105d9c95de4a..78a21f2828df 100644
--- a/arch/arm/boot/dts/r8a7740-armadillo800eva.dts
+++ b/arch/arm/boot/dts/r8a7740-armadillo800eva.dts
@@ -85,7 +85,7 @@
gpios = <&pfc 99 GPIO_ACTIVE_LOW>;
linux,code = <KEY_POWER>;
label = "SW3";
- gpio-key,wakeup;
+ wakeup-source;
};
back-key {
diff --git a/arch/arm/boot/dts/r8a7790-lager.dts b/arch/arm/boot/dts/r8a7790-lager.dts
index 37dec5269491..8523c3d03de3 100644
--- a/arch/arm/boot/dts/r8a7790-lager.dts
+++ b/arch/arm/boot/dts/r8a7790-lager.dts
@@ -77,28 +77,28 @@
button@1 {
linux,code = <KEY_1>;
label = "SW2-1";
- gpio-key,wakeup;
+ wakeup-source;
debounce-interval = <20>;
gpios = <&gpio1 14 GPIO_ACTIVE_LOW>;
};
button@2 {
linux,code = <KEY_2>;
label = "SW2-2";
- gpio-key,wakeup;
+ wakeup-source;
debounce-interval = <20>;
gpios = <&gpio1 24 GPIO_ACTIVE_LOW>;
};
button@3 {
linux,code = <KEY_3>;
label = "SW2-3";
- gpio-key,wakeup;
+ wakeup-source;
debounce-interval = <20>;
gpios = <&gpio1 26 GPIO_ACTIVE_LOW>;
};
button@4 {
linux,code = <KEY_4>;
label = "SW2-4";
- gpio-key,wakeup;
+ wakeup-source;
debounce-interval = <20>;
gpios = <&gpio1 28 GPIO_ACTIVE_LOW>;
};
diff --git a/arch/arm/boot/dts/r8a7791-koelsch.dts b/arch/arm/boot/dts/r8a7791-koelsch.dts
index dc158845afdc..48cddfc25f72 100644
--- a/arch/arm/boot/dts/r8a7791-koelsch.dts
+++ b/arch/arm/boot/dts/r8a7791-koelsch.dts
@@ -79,77 +79,77 @@
gpios = <&gpio5 0 GPIO_ACTIVE_LOW>;
linux,code = <KEY_1>;
label = "SW2-1";
- gpio-key,wakeup;
+ wakeup-source;
debounce-interval = <20>;
};
key-2 {
gpios = <&gpio5 1 GPIO_ACTIVE_LOW>;
linux,code = <KEY_2>;
label = "SW2-2";
- gpio-key,wakeup;
+ wakeup-source;
debounce-interval = <20>;
};
key-3 {
gpios = <&gpio5 2 GPIO_ACTIVE_LOW>;
linux,code = <KEY_3>;
label = "SW2-3";
- gpio-key,wakeup;
+ wakeup-source;
debounce-interval = <20>;
};
key-4 {
gpios = <&gpio5 3 GPIO_ACTIVE_LOW>;
linux,code = <KEY_4>;
label = "SW2-4";
- gpio-key,wakeup;
+ wakeup-source;
debounce-interval = <20>;
};
key-a {
gpios = <&gpio7 0 GPIO_ACTIVE_LOW>;
linux,code = <KEY_A>;
label = "SW30";
- gpio-key,wakeup;
+ wakeup-source;
debounce-interval = <20>;
};
key-b {
gpios = <&gpio7 1 GPIO_ACTIVE_LOW>;
linux,code = <KEY_B>;
label = "SW31";
- gpio-key,wakeup;
+ wakeup-source;
debounce-interval = <20>;
};
key-c {
gpios = <&gpio7 2 GPIO_ACTIVE_LOW>;
linux,code = <KEY_C>;
label = "SW32";
- gpio-key,wakeup;
+ wakeup-source;
debounce-interval = <20>;
};
key-d {
gpios = <&gpio7 3 GPIO_ACTIVE_LOW>;
linux,code = <KEY_D>;
label = "SW33";
- gpio-key,wakeup;
+ wakeup-source;
debounce-interval = <20>;
};
key-e {
gpios = <&gpio7 4 GPIO_ACTIVE_LOW>;
linux,code = <KEY_E>;
label = "SW34";
- gpio-key,wakeup;
+ wakeup-source;
debounce-interval = <20>;
};
key-f {
gpios = <&gpio7 5 GPIO_ACTIVE_LOW>;
linux,code = <KEY_F>;
label = "SW35";
- gpio-key,wakeup;
+ wakeup-source;
debounce-interval = <20>;
};
key-g {
gpios = <&gpio7 6 GPIO_ACTIVE_LOW>;
linux,code = <KEY_G>;
label = "SW36";
- gpio-key,wakeup;
+ wakeup-source;
debounce-interval = <20>;
};
};
diff --git a/arch/arm/boot/dts/sh73a0-kzm9g.dts b/arch/arm/boot/dts/sh73a0-kzm9g.dts
index 24b4cd24dceb..710ea035d0f4 100644
--- a/arch/arm/boot/dts/sh73a0-kzm9g.dts
+++ b/arch/arm/boot/dts/sh73a0-kzm9g.dts
@@ -147,7 +147,7 @@
gpios = <&pcf8575 14 GPIO_ACTIVE_LOW>;
linux,code = <KEY_HOME>;
label = "SW1";
- gpio-key,wakeup;
+ wakeup-source;
};
};
--
1.9.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH 14/19] ARM: dts: shmobile/r8a7xxx: replace gpio-key,wakeup with wakeup-source property
2015-10-21 10:10 ` [PATCH 14/19] ARM: dts: shmobile/r8a7xxx: replace gpio-key,wakeup with wakeup-source property Sudeep Holla
@ 2015-10-21 10:18 ` Geert Uytterhoeven
2015-10-21 10:23 ` Sudeep Holla
0 siblings, 1 reply; 6+ messages in thread
From: Geert Uytterhoeven @ 2015-10-21 10:18 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Oct 21, 2015 at 12:10 PM, Sudeep Holla <sudeep.holla@arm.com> wrote:
> Though the keyboard driver for GPIO buttons(gpio-keys) will continue to
> check for/support the legacy "gpio-key,wakeup" boolean property to
> enable gpio buttons as wakeup source, "wakeup-source" is the new
> standard binding.
>
> This patch replaces the legacy "gpio-key,wakeup" with the unified
> "wakeup-source" property in order to avoid any futher copy-paste
> duplication.
Thanks!
I made the same changes to my local tree a few weeks ago, but never found
time to send them out...
> Cc: Simon Horman <horms@verge.net.au>
> Cc: Magnus Damm <magnus.damm@gmail.com>
> Cc: linux-sh@vger.kernel.org
> Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>
Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 14/19] ARM: dts: shmobile/r8a7xxx: replace gpio-key,wakeup with wakeup-source property
2015-10-21 10:18 ` Geert Uytterhoeven
@ 2015-10-21 10:23 ` Sudeep Holla
2015-10-22 0:50 ` Simon Horman
0 siblings, 1 reply; 6+ messages in thread
From: Sudeep Holla @ 2015-10-21 10:23 UTC (permalink / raw)
To: linux-arm-kernel
On 21/10/15 11:18, Geert Uytterhoeven wrote:
> On Wed, Oct 21, 2015 at 12:10 PM, Sudeep Holla <sudeep.holla@arm.com> wrote:
>> Though the keyboard driver for GPIO buttons(gpio-keys) will continue to
>> check for/support the legacy "gpio-key,wakeup" boolean property to
>> enable gpio buttons as wakeup source, "wakeup-source" is the new
>> standard binding.
>>
>> This patch replaces the legacy "gpio-key,wakeup" with the unified
>> "wakeup-source" property in order to avoid any futher copy-paste
>> duplication.
>
> Thanks!
>
> I made the same changes to my local tree a few weeks ago, but never found
> time to send them out...
>
Ah OK, even I had these changes for long. I waited until I fixed all the
binding documents(which was boring ;))
>> Cc: Simon Horman <horms@verge.net.au>
>> Cc: Magnus Damm <magnus.damm@gmail.com>
>> Cc: linux-sh@vger.kernel.org
>> Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>
>
> Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>
>
Thanks
--
Regards,
Sudeep
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 14/19] ARM: dts: shmobile/r8a7xxx: replace gpio-key,wakeup with wakeup-source property
2015-10-21 10:23 ` Sudeep Holla
@ 2015-10-22 0:50 ` Simon Horman
2015-10-22 9:37 ` Sudeep Holla
0 siblings, 1 reply; 6+ messages in thread
From: Simon Horman @ 2015-10-22 0:50 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Oct 21, 2015 at 11:23:03AM +0100, Sudeep Holla wrote:
>
>
> On 21/10/15 11:18, Geert Uytterhoeven wrote:
> >On Wed, Oct 21, 2015 at 12:10 PM, Sudeep Holla <sudeep.holla@arm.com> wrote:
> >>Though the keyboard driver for GPIO buttons(gpio-keys) will continue to
> >>check for/support the legacy "gpio-key,wakeup" boolean property to
> >>enable gpio buttons as wakeup source, "wakeup-source" is the new
> >>standard binding.
> >>
> >>This patch replaces the legacy "gpio-key,wakeup" with the unified
> >>"wakeup-source" property in order to avoid any futher copy-paste
> >>duplication.
> >
> >Thanks!
> >
> >I made the same changes to my local tree a few weeks ago, but never found
> >time to send them out...
> >
>
> Ah OK, even I had these changes for long. I waited until I fixed all the
> binding documents(which was boring ;))
>
> >>Cc: Simon Horman <horms@verge.net.au>
> >>Cc: Magnus Damm <magnus.damm@gmail.com>
> >>Cc: linux-sh@vger.kernel.org
> >>Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>
> >
> >Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>
Can I clarify that you would like me to pick up this patch?
If so I'm happy to do so.
Else:
Acked-by: Simon Horman <horms+renesas@verge.net.au>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 14/19] ARM: dts: shmobile/r8a7xxx: replace gpio-key,wakeup with wakeup-source property
2015-10-22 0:50 ` Simon Horman
@ 2015-10-22 9:37 ` Sudeep Holla
2015-10-23 0:18 ` Simon Horman
0 siblings, 1 reply; 6+ messages in thread
From: Sudeep Holla @ 2015-10-22 9:37 UTC (permalink / raw)
To: linux-arm-kernel
On 22/10/15 01:50, Simon Horman wrote:
> On Wed, Oct 21, 2015 at 11:23:03AM +0100, Sudeep Holla wrote:
>>
>>
>> On 21/10/15 11:18, Geert Uytterhoeven wrote:
>>> On Wed, Oct 21, 2015 at 12:10 PM, Sudeep Holla <sudeep.holla@arm.com> wrote:
>>>> Though the keyboard driver for GPIO buttons(gpio-keys) will continue to
>>>> check for/support the legacy "gpio-key,wakeup" boolean property to
>>>> enable gpio buttons as wakeup source, "wakeup-source" is the new
>>>> standard binding.
>>>>
>>>> This patch replaces the legacy "gpio-key,wakeup" with the unified
>>>> "wakeup-source" property in order to avoid any futher copy-paste
>>>> duplication.
>>>
>>> Thanks!
>>>
>>> I made the same changes to my local tree a few weeks ago, but never found
>>> time to send them out...
>>>
>>
>> Ah OK, even I had these changes for long. I waited until I fixed all the
>> binding documents(which was boring ;))
>>
>>>> Cc: Simon Horman <horms@verge.net.au>
>>>> Cc: Magnus Damm <magnus.damm@gmail.com>
>>>> Cc: linux-sh@vger.kernel.org
>>>> Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>
>>>
>>> Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>
>
> Can I clarify that you would like me to pick up this patch?
> If so I'm happy to do so.
>
Yes you can pick up this patch as it has no dependency on driver changes.
> Else:
>
> Acked-by: Simon Horman <horms+renesas@verge.net.au>
>
Thanks.
--
Regards,
Sudeep
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 14/19] ARM: dts: shmobile/r8a7xxx: replace gpio-key,wakeup with wakeup-source property
2015-10-22 9:37 ` Sudeep Holla
@ 2015-10-23 0:18 ` Simon Horman
0 siblings, 0 replies; 6+ messages in thread
From: Simon Horman @ 2015-10-23 0:18 UTC (permalink / raw)
To: linux-arm-kernel
On Thu, Oct 22, 2015 at 10:37:54AM +0100, Sudeep Holla wrote:
>
>
> On 22/10/15 01:50, Simon Horman wrote:
> >On Wed, Oct 21, 2015 at 11:23:03AM +0100, Sudeep Holla wrote:
> >>
> >>
> >>On 21/10/15 11:18, Geert Uytterhoeven wrote:
> >>>On Wed, Oct 21, 2015 at 12:10 PM, Sudeep Holla <sudeep.holla@arm.com> wrote:
> >>>>Though the keyboard driver for GPIO buttons(gpio-keys) will continue to
> >>>>check for/support the legacy "gpio-key,wakeup" boolean property to
> >>>>enable gpio buttons as wakeup source, "wakeup-source" is the new
> >>>>standard binding.
> >>>>
> >>>>This patch replaces the legacy "gpio-key,wakeup" with the unified
> >>>>"wakeup-source" property in order to avoid any futher copy-paste
> >>>>duplication.
> >>>
> >>>Thanks!
> >>>
> >>>I made the same changes to my local tree a few weeks ago, but never found
> >>>time to send them out...
> >>>
> >>
> >>Ah OK, even I had these changes for long. I waited until I fixed all the
> >>binding documents(which was boring ;))
> >>
> >>>>Cc: Simon Horman <horms@verge.net.au>
> >>>>Cc: Magnus Damm <magnus.damm@gmail.com>
> >>>>Cc: linux-sh@vger.kernel.org
> >>>>Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>
> >>>
> >>>Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>
> >
> >Can I clarify that you would like me to pick up this patch?
> >If so I'm happy to do so.
> >
>
> Yes you can pick up this patch as it has no dependency on driver changes.
Thanks, I have queued this up for v4.5.
It should appear in the devel branch of the renesas tree some time today.
I will include it in the next branch, which is included in the linux-next
tree, once v4.4-rc1 has been released.
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2015-10-23 0:18 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1445422216-29375-1-git-send-email-sudeep.holla@arm.com>
2015-10-21 10:10 ` [PATCH 14/19] ARM: dts: shmobile/r8a7xxx: replace gpio-key,wakeup with wakeup-source property Sudeep Holla
2015-10-21 10:18 ` Geert Uytterhoeven
2015-10-21 10:23 ` Sudeep Holla
2015-10-22 0:50 ` Simon Horman
2015-10-22 9:37 ` Sudeep Holla
2015-10-23 0:18 ` Simon Horman
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).