* [PATCH v2] platform/x86/pcengines-apuv2: use KEY_RESTART for front button
@ 2019-07-25 19:06 Enrico Weigelt, metux IT consult
2019-08-01 15:00 ` Andy Shevchenko
0 siblings, 1 reply; 4+ messages in thread
From: Enrico Weigelt, metux IT consult @ 2019-07-25 19:06 UTC (permalink / raw)
To: linux-kernel; +Cc: info, dvhart, andy, platform-driver-x86
From: Enrico Weigelt <info@metux.net>
The keycode KEY_RESTART is more appropriate for the front button,
as most people use it for things like restart or factory reset.
Signed-off-by: Enrico Weigelt <info@metux.net>
Fixes: f8eb0235f65989fc5521c40c78d1261e7f25cdbe ("x86: pcengines apuv2 gpio/leds/keys platform driver")
---
drivers/platform/x86/pcengines-apuv2.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/platform/x86/pcengines-apuv2.c b/drivers/platform/x86/pcengines-apuv2.c
index b0d3110..4138007c 100644
--- a/drivers/platform/x86/pcengines-apuv2.c
+++ b/drivers/platform/x86/pcengines-apuv2.c
@@ -93,7 +93,7 @@
static struct gpio_keys_button apu2_keys_buttons[] = {
{
- .code = KEY_SETUP,
+ .code = KEY_RESTART,
.active_low = 1,
.desc = "front button",
.type = EV_KEY,
--
1.9.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v2] platform/x86/pcengines-apuv2: use KEY_RESTART for front button
2019-07-25 19:06 [PATCH v2] platform/x86/pcengines-apuv2: use KEY_RESTART for front button Enrico Weigelt, metux IT consult
@ 2019-08-01 15:00 ` Andy Shevchenko
2019-08-05 12:06 ` Enrico Weigelt, metux IT consult
0 siblings, 1 reply; 4+ messages in thread
From: Andy Shevchenko @ 2019-08-01 15:00 UTC (permalink / raw)
To: Enrico Weigelt, metux IT consult
Cc: Linux Kernel Mailing List, Darren Hart, Andy Shevchenko, Platform Driver
On Thu, Jul 25, 2019 at 10:06 PM Enrico Weigelt, metux IT consult
<info@metux.net> wrote:
>
> From: Enrico Weigelt <info@metux.net>
>
> The keycode KEY_RESTART is more appropriate for the front button,
> as most people use it for things like restart or factory reset.
>
Applied, thanks!
> Signed-off-by: Enrico Weigelt <info@metux.net>
> Fixes: f8eb0235f65989fc5521c40c78d1261e7f25cdbe ("x86: pcengines apuv2 gpio/leds/keys platform driver")
Please, use allowed minimum (12 nowadays) characters of hash.
> ---
> drivers/platform/x86/pcengines-apuv2.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/platform/x86/pcengines-apuv2.c b/drivers/platform/x86/pcengines-apuv2.c
> index b0d3110..4138007c 100644
> --- a/drivers/platform/x86/pcengines-apuv2.c
> +++ b/drivers/platform/x86/pcengines-apuv2.c
> @@ -93,7 +93,7 @@
>
> static struct gpio_keys_button apu2_keys_buttons[] = {
> {
> - .code = KEY_SETUP,
> + .code = KEY_RESTART,
> .active_low = 1,
> .desc = "front button",
> .type = EV_KEY,
> --
> 1.9.1
>
--
With Best Regards,
Andy Shevchenko
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2] platform/x86/pcengines-apuv2: use KEY_RESTART for front button
2019-08-01 15:00 ` Andy Shevchenko
@ 2019-08-05 12:06 ` Enrico Weigelt, metux IT consult
2019-08-16 10:07 ` Andy Shevchenko
0 siblings, 1 reply; 4+ messages in thread
From: Enrico Weigelt, metux IT consult @ 2019-08-05 12:06 UTC (permalink / raw)
To: Andy Shevchenko, Enrico Weigelt, metux IT consult
Cc: Linux Kernel Mailing List, Darren Hart, Andy Shevchenko, Platform Driver
On 01.08.19 17:00, Andy Shevchenko wrote:
Hi,
> On Thu, Jul 25, 2019 at 10:06 PM Enrico Weigelt, metux IT consult
> <info@metux.net> wrote:
>>
>> From: Enrico Weigelt <info@metux.net>
>>
>> The keycode KEY_RESTART is more appropriate for the front button,
>> as most people use it for things like restart or factory reset.
>>
>
> Applied, thanks!
thx.
>
>> Signed-off-by: Enrico Weigelt <info@metux.net>
>> Fixes: f8eb0235f65989fc5521c40c78d1261e7f25cdbe ("x86: pcengines apuv2 gpio/leds/keys platform driver")
>
> Please, use allowed minimum (12 nowadays) characters of hash.
hmm, do you happen to know how to configure tig to show the short hash ?
--mtx
--
Enrico Weigelt, metux IT consult
Free software and Linux embedded engineering
info@metux.net -- +49-151-27565287
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2] platform/x86/pcengines-apuv2: use KEY_RESTART for front button
2019-08-05 12:06 ` Enrico Weigelt, metux IT consult
@ 2019-08-16 10:07 ` Andy Shevchenko
0 siblings, 0 replies; 4+ messages in thread
From: Andy Shevchenko @ 2019-08-16 10:07 UTC (permalink / raw)
To: Enrico Weigelt, metux IT consult
Cc: Enrico Weigelt, metux IT consult, Linux Kernel Mailing List,
Darren Hart, Andy Shevchenko, Platform Driver
On Mon, Aug 5, 2019 at 3:06 PM Enrico Weigelt, metux IT consult
<lkml@metux.net> wrote:
> On 01.08.19 17:00, Andy Shevchenko wrote:
> > On Thu, Jul 25, 2019 at 10:06 PM Enrico Weigelt, metux IT consult
> > <info@metux.net> wrote:
> >> From: Enrico Weigelt <info@metux.net>
> >> Fixes: f8eb0235f65989fc5521c40c78d1261e7f25cdbe ("x86: pcengines apuv2 gpio/leds/keys platform driver")
> >
> > Please, use allowed minimum (12 nowadays) characters of hash.
>
> hmm, do you happen to know how to configure tig to show the short hash ?
I have configured '--oneline' to show 12.
--
With Best Regards,
Andy Shevchenko
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-08-16 10:07 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-07-25 19:06 [PATCH v2] platform/x86/pcengines-apuv2: use KEY_RESTART for front button Enrico Weigelt, metux IT consult
2019-08-01 15:00 ` Andy Shevchenko
2019-08-05 12:06 ` Enrico Weigelt, metux IT consult
2019-08-16 10:07 ` Andy Shevchenko
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).