GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

drivers/generic/gpio_keys: Add trigger for wakeup event action 

Currently without any trigger the wakeup event is generated on both the
rising and falling edges of the GPIO input. Add support to specify the
trigger explicitly so that the configuration can be passed to the
kernel.

BRANCH=octopus
BUG=b:117953118
TEST=Ensure that the system boots to ChromeOS. Ensure that the stylus tools
open on pen eject. Ensure that the system wakes on Pen Eject. Ensure that
the system enters S0ix and S3 states after the pen is ejected. Ensure that
the system enters S0ix and S3 states when the pen remains inserted in its
holder. Ensured that the system does not wake when the pen is inserted.
Ensure that the suspend_stress_test runs successfully for 25 iterations
with the pen placed in its holder and ejected from its holder.

Change-Id: Ifb08ba01106031aa2655c1ae2faab284926f1ceb
Signed-off-by: Karthikeyan Ramasubramanian <kramasub@chromium.org>
Reviewed-on: https://review.coreboot.org/c/30451
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Furquan Shaikh <furquan@google.com>
This commit is contained in:
Karthikeyan Ramasubramanian 2018-12-26 21:37:24 -07:00 committed by Patrick Georgi
parent cff4507289
commit c4427393c5
2 changed files with 14 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/drivers/generic/gpio_keys/chip.h
View File

@ -30,6 +30,13 @@ enum {
SW_PEN_INSERTED = 0xf, SW_PEN_INSERTED = 0xf,
}; };
/* Trigger for wakeup event action */
enum {
EV_ACT_ANY,
EV_ACT_ASSERTED,
EV_ACT_DEASSERTED,
};
/* Details of the child node defining key */ /* Details of the child node defining key */
struct key_info { struct key_info {
/* Device name of the child node - Mandatory */ /* Device name of the child node - Mandatory */
@ -47,6 +54,8 @@ struct key_info {
bool is_wakeup_source; bool is_wakeup_source;
/* Wake GPE */ /* Wake GPE */
unsigned int wake; unsigned int wake;
/* Trigger for Wakeup Event Action as defined in EV_ACT_* enum */
unsigned int wakeup_event_action;
/* Can this key be disabled? */ /* Can this key be disabled? */
bool can_be_disabled; bool can_be_disabled;
/* Debounce interval time in milliseconds */ /* Debounce interval time in milliseconds */

6
src/drivers/generic/gpio_keys/gpio_keys.c
View File

@ -43,8 +43,12 @@ static struct acpi_dp *gpio_keys_add_child_node(
if (key->is_wakeup_source) if (key->is_wakeup_source)
acpi_dp_add_integer(dsd, "wakeup-source", acpi_dp_add_integer(dsd, "wakeup-source",
key->is_wakeup_source); key->is_wakeup_source);
if (key->wake) if (key->wake) {
acpigen_write_PRW(key->wake, 3); acpigen_write_PRW(key->wake, 3);
acpi_dp_add_integer(dsd, "wakeup-event-action",
key->wakeup_event_action);
}
if (key->can_be_disabled) if (key->can_be_disabled)
acpi_dp_add_integer(dsd, "linux,can-disable", acpi_dp_add_integer(dsd, "linux,can-disable",
key->can_be_disabled); key->can_be_disabled);