* [PATCH] input: cros_ec_keyb: Add an EC event for sysrq
@ 2017-03-30 22:18 Rajat Jain
2017-04-01 17:16 ` Dmitry Torokhov
0 siblings, 1 reply; 4+ messages in thread
From: Rajat Jain @ 2017-03-30 22:18 UTC (permalink / raw)
To: Dmitry Torokhov, linux-input, linux-kernel
Cc: Lee Jones, rajatxjain, dtor, Rajat Jain
Some form factors (detachables / tablets) may not have a keyboard and
thus user may have to resort to using a defined EC UI to send sysrq(s)
to the kernel in order to collect crash info etc. This UI typically
is in the form of user pressing volume / power buttons in some specific
sequence and for some specific time. Add a new EC event that allows EC
to communicate the sysrq to the AP.
(We're skipping event number 5 because it has been reserved for
something else)
Signed-off-by: Rajat Jain <rajatja@google.com>
---
drivers/input/keyboard/cros_ec_keyb.c | 7 +++++++
include/linux/mfd/cros_ec_commands.h | 4 ++++
2 files changed, 11 insertions(+)
diff --git a/drivers/input/keyboard/cros_ec_keyb.c b/drivers/input/keyboard/cros_ec_keyb.c
index 6a250d65f8fe..00a3a830cd3a 100644
--- a/drivers/input/keyboard/cros_ec_keyb.c
+++ b/drivers/input/keyboard/cros_ec_keyb.c
@@ -30,6 +30,7 @@
#include <linux/notifier.h>
#include <linux/platform_device.h>
#include <linux/slab.h>
+#include <linux/sysrq.h>
#include <linux/input/matrix_keypad.h>
#include <linux/mfd/cros_ec.h>
#include <linux/mfd/cros_ec_commands.h>
@@ -260,6 +261,12 @@ static int cros_ec_keyb_work(struct notifier_block *nb,
ckdev->ec->event_size);
break;
+ case EC_MKBP_EVENT_SYSRQ:
+ val = get_unaligned_le32(&ckdev->ec->event_data.data.sysrq);
+ dev_dbg(ckdev->dev, "sysrq code from EC : %#x\n", val);
+ handle_sysrq(val);
+ break;
+
case EC_MKBP_EVENT_BUTTON:
case EC_MKBP_EVENT_SWITCH:
/*
diff --git a/include/linux/mfd/cros_ec_commands.h b/include/linux/mfd/cros_ec_commands.h
index f1ef6388c233..c93e7e0300ef 100644
--- a/include/linux/mfd/cros_ec_commands.h
+++ b/include/linux/mfd/cros_ec_commands.h
@@ -2041,6 +2041,9 @@ enum ec_mkbp_event {
/* The state of the switches have changed. */
EC_MKBP_EVENT_SWITCH = 4,
+ /* EC sent a sysrq command */
+ EC_MKBP_EVENT_SYSRQ = 6,
+
/* Number of MKBP events */
EC_MKBP_EVENT_COUNT,
};
@@ -2053,6 +2056,7 @@ union ec_response_get_next_data {
uint32_t buttons;
uint32_t switches;
+ uint32_t sysrq;
} __packed;
struct ec_response_get_next_event {
--
2.12.2.564.g063fe858b8-goog
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] input: cros_ec_keyb: Add an EC event for sysrq
2017-03-30 22:18 [PATCH] input: cros_ec_keyb: Add an EC event for sysrq Rajat Jain
@ 2017-04-01 17:16 ` Dmitry Torokhov
2017-04-03 9:56 ` Lee Jones
0 siblings, 1 reply; 4+ messages in thread
From: Dmitry Torokhov @ 2017-04-01 17:16 UTC (permalink / raw)
To: Lee Jones; +Cc: linux-input, linux-kernel, Rajat Jain, rajatxjain, dtor
On Thu, Mar 30, 2017 at 03:18:07PM -0700, Rajat Jain wrote:
> Some form factors (detachables / tablets) may not have a keyboard and
> thus user may have to resort to using a defined EC UI to send sysrq(s)
> to the kernel in order to collect crash info etc. This UI typically
> is in the form of user pressing volume / power buttons in some specific
> sequence and for some specific time. Add a new EC event that allows EC
> to communicate the sysrq to the AP.
>
> (We're skipping event number 5 because it has been reserved for
> something else)
>
> Signed-off-by: Rajat Jain <rajatja@google.com>
Lee, is it OK to route through my tree or you want to take it?
> ---
> drivers/input/keyboard/cros_ec_keyb.c | 7 +++++++
> include/linux/mfd/cros_ec_commands.h | 4 ++++
> 2 files changed, 11 insertions(+)
>
> diff --git a/drivers/input/keyboard/cros_ec_keyb.c b/drivers/input/keyboard/cros_ec_keyb.c
> index 6a250d65f8fe..00a3a830cd3a 100644
> --- a/drivers/input/keyboard/cros_ec_keyb.c
> +++ b/drivers/input/keyboard/cros_ec_keyb.c
> @@ -30,6 +30,7 @@
> #include <linux/notifier.h>
> #include <linux/platform_device.h>
> #include <linux/slab.h>
> +#include <linux/sysrq.h>
> #include <linux/input/matrix_keypad.h>
> #include <linux/mfd/cros_ec.h>
> #include <linux/mfd/cros_ec_commands.h>
> @@ -260,6 +261,12 @@ static int cros_ec_keyb_work(struct notifier_block *nb,
> ckdev->ec->event_size);
> break;
>
> + case EC_MKBP_EVENT_SYSRQ:
> + val = get_unaligned_le32(&ckdev->ec->event_data.data.sysrq);
> + dev_dbg(ckdev->dev, "sysrq code from EC : %#x\n", val);
> + handle_sysrq(val);
> + break;
> +
> case EC_MKBP_EVENT_BUTTON:
> case EC_MKBP_EVENT_SWITCH:
> /*
> diff --git a/include/linux/mfd/cros_ec_commands.h b/include/linux/mfd/cros_ec_commands.h
> index f1ef6388c233..c93e7e0300ef 100644
> --- a/include/linux/mfd/cros_ec_commands.h
> +++ b/include/linux/mfd/cros_ec_commands.h
> @@ -2041,6 +2041,9 @@ enum ec_mkbp_event {
> /* The state of the switches have changed. */
> EC_MKBP_EVENT_SWITCH = 4,
>
> + /* EC sent a sysrq command */
> + EC_MKBP_EVENT_SYSRQ = 6,
> +
> /* Number of MKBP events */
> EC_MKBP_EVENT_COUNT,
> };
> @@ -2053,6 +2056,7 @@ union ec_response_get_next_data {
>
> uint32_t buttons;
> uint32_t switches;
> + uint32_t sysrq;
> } __packed;
>
> struct ec_response_get_next_event {
> --
> 2.12.2.564.g063fe858b8-goog
>
--
Dmitry
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] input: cros_ec_keyb: Add an EC event for sysrq
2017-04-01 17:16 ` Dmitry Torokhov
@ 2017-04-03 9:56 ` Lee Jones
2017-04-03 18:54 ` Dmitry Torokhov
0 siblings, 1 reply; 4+ messages in thread
From: Lee Jones @ 2017-04-03 9:56 UTC (permalink / raw)
To: Dmitry Torokhov; +Cc: linux-input, linux-kernel, Rajat Jain, rajatxjain, dtor
On Sat, 01 Apr 2017, Dmitry Torokhov wrote:
> On Thu, Mar 30, 2017 at 03:18:07PM -0700, Rajat Jain wrote:
> > Some form factors (detachables / tablets) may not have a keyboard and
> > thus user may have to resort to using a defined EC UI to send sysrq(s)
> > to the kernel in order to collect crash info etc. This UI typically
> > is in the form of user pressing volume / power buttons in some specific
> > sequence and for some specific time. Add a new EC event that allows EC
> > to communicate the sysrq to the AP.
> >
> > (We're skipping event number 5 because it has been reserved for
> > something else)
> >
> > Signed-off-by: Rajat Jain <rajatja@google.com>
>
> Lee, is it OK to route through my tree or you want to take it?
Looks trivial enough:
Acked-by: Lee Jones <lee.jones@linaro.org>
> > ---
> > drivers/input/keyboard/cros_ec_keyb.c | 7 +++++++
> > include/linux/mfd/cros_ec_commands.h | 4 ++++
> > 2 files changed, 11 insertions(+)
> >
> > diff --git a/drivers/input/keyboard/cros_ec_keyb.c b/drivers/input/keyboard/cros_ec_keyb.c
> > index 6a250d65f8fe..00a3a830cd3a 100644
> > --- a/drivers/input/keyboard/cros_ec_keyb.c
> > +++ b/drivers/input/keyboard/cros_ec_keyb.c
> > @@ -30,6 +30,7 @@
> > #include <linux/notifier.h>
> > #include <linux/platform_device.h>
> > #include <linux/slab.h>
> > +#include <linux/sysrq.h>
> > #include <linux/input/matrix_keypad.h>
> > #include <linux/mfd/cros_ec.h>
> > #include <linux/mfd/cros_ec_commands.h>
> > @@ -260,6 +261,12 @@ static int cros_ec_keyb_work(struct notifier_block *nb,
> > ckdev->ec->event_size);
> > break;
> >
> > + case EC_MKBP_EVENT_SYSRQ:
> > + val = get_unaligned_le32(&ckdev->ec->event_data.data.sysrq);
> > + dev_dbg(ckdev->dev, "sysrq code from EC : %#x\n", val);
> > + handle_sysrq(val);
> > + break;
> > +
> > case EC_MKBP_EVENT_BUTTON:
> > case EC_MKBP_EVENT_SWITCH:
> > /*
> > diff --git a/include/linux/mfd/cros_ec_commands.h b/include/linux/mfd/cros_ec_commands.h
> > index f1ef6388c233..c93e7e0300ef 100644
> > --- a/include/linux/mfd/cros_ec_commands.h
> > +++ b/include/linux/mfd/cros_ec_commands.h
> > @@ -2041,6 +2041,9 @@ enum ec_mkbp_event {
> > /* The state of the switches have changed. */
> > EC_MKBP_EVENT_SWITCH = 4,
> >
> > + /* EC sent a sysrq command */
> > + EC_MKBP_EVENT_SYSRQ = 6,
> > +
> > /* Number of MKBP events */
> > EC_MKBP_EVENT_COUNT,
> > };
> > @@ -2053,6 +2056,7 @@ union ec_response_get_next_data {
> >
> > uint32_t buttons;
> > uint32_t switches;
> > + uint32_t sysrq;
> > } __packed;
> >
> > struct ec_response_get_next_event {
>
--
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] input: cros_ec_keyb: Add an EC event for sysrq
2017-04-03 9:56 ` Lee Jones
@ 2017-04-03 18:54 ` Dmitry Torokhov
0 siblings, 0 replies; 4+ messages in thread
From: Dmitry Torokhov @ 2017-04-03 18:54 UTC (permalink / raw)
To: Lee Jones; +Cc: linux-input, linux-kernel, Rajat Jain, rajatxjain, dtor
On Mon, Apr 03, 2017 at 10:56:59AM +0100, Lee Jones wrote:
> On Sat, 01 Apr 2017, Dmitry Torokhov wrote:
>
> > On Thu, Mar 30, 2017 at 03:18:07PM -0700, Rajat Jain wrote:
> > > Some form factors (detachables / tablets) may not have a keyboard and
> > > thus user may have to resort to using a defined EC UI to send sysrq(s)
> > > to the kernel in order to collect crash info etc. This UI typically
> > > is in the form of user pressing volume / power buttons in some specific
> > > sequence and for some specific time. Add a new EC event that allows EC
> > > to communicate the sysrq to the AP.
> > >
> > > (We're skipping event number 5 because it has been reserved for
> > > something else)
> > >
> > > Signed-off-by: Rajat Jain <rajatja@google.com>
> >
> > Lee, is it OK to route through my tree or you want to take it?
>
> Looks trivial enough:
>
> Acked-by: Lee Jones <lee.jones@linaro.org>
Applied, thank you.
>
> > > ---
> > > drivers/input/keyboard/cros_ec_keyb.c | 7 +++++++
> > > include/linux/mfd/cros_ec_commands.h | 4 ++++
> > > 2 files changed, 11 insertions(+)
> > >
> > > diff --git a/drivers/input/keyboard/cros_ec_keyb.c b/drivers/input/keyboard/cros_ec_keyb.c
> > > index 6a250d65f8fe..00a3a830cd3a 100644
> > > --- a/drivers/input/keyboard/cros_ec_keyb.c
> > > +++ b/drivers/input/keyboard/cros_ec_keyb.c
> > > @@ -30,6 +30,7 @@
> > > #include <linux/notifier.h>
> > > #include <linux/platform_device.h>
> > > #include <linux/slab.h>
> > > +#include <linux/sysrq.h>
> > > #include <linux/input/matrix_keypad.h>
> > > #include <linux/mfd/cros_ec.h>
> > > #include <linux/mfd/cros_ec_commands.h>
> > > @@ -260,6 +261,12 @@ static int cros_ec_keyb_work(struct notifier_block *nb,
> > > ckdev->ec->event_size);
> > > break;
> > >
> > > + case EC_MKBP_EVENT_SYSRQ:
> > > + val = get_unaligned_le32(&ckdev->ec->event_data.data.sysrq);
> > > + dev_dbg(ckdev->dev, "sysrq code from EC : %#x\n", val);
> > > + handle_sysrq(val);
> > > + break;
> > > +
> > > case EC_MKBP_EVENT_BUTTON:
> > > case EC_MKBP_EVENT_SWITCH:
> > > /*
> > > diff --git a/include/linux/mfd/cros_ec_commands.h b/include/linux/mfd/cros_ec_commands.h
> > > index f1ef6388c233..c93e7e0300ef 100644
> > > --- a/include/linux/mfd/cros_ec_commands.h
> > > +++ b/include/linux/mfd/cros_ec_commands.h
> > > @@ -2041,6 +2041,9 @@ enum ec_mkbp_event {
> > > /* The state of the switches have changed. */
> > > EC_MKBP_EVENT_SWITCH = 4,
> > >
> > > + /* EC sent a sysrq command */
> > > + EC_MKBP_EVENT_SYSRQ = 6,
> > > +
> > > /* Number of MKBP events */
> > > EC_MKBP_EVENT_COUNT,
> > > };
> > > @@ -2053,6 +2056,7 @@ union ec_response_get_next_data {
> > >
> > > uint32_t buttons;
> > > uint32_t switches;
> > > + uint32_t sysrq;
> > > } __packed;
> > >
> > > struct ec_response_get_next_event {
> >
>
> --
> Lee Jones
> Linaro STMicroelectronics Landing Team Lead
> Linaro.org │ Open source software for ARM SoCs
> Follow Linaro: Facebook | Twitter | Blog
--
Dmitry
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2017-04-03 18:54 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-03-30 22:18 [PATCH] input: cros_ec_keyb: Add an EC event for sysrq Rajat Jain
2017-04-01 17:16 ` Dmitry Torokhov
2017-04-03 9:56 ` Lee Jones
2017-04-03 18:54 ` 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.