diff options
| author | Jett Rink <jettrink@chromium.org> | 2018-01-17 16:53:59 -0800 |
|---|---|---|
| committer | chrome-bot <chrome-bot@chromium.org> | 2018-01-18 05:09:37 -0800 |
| commit | d52ba1a4d4801fdba129be64fa4c1c53a7ced817 (patch) | |
| tree | 65cfc3e4ddbc51ff66abfee280783c83ae09a3d6 /common/backlight_lid.c | |
| parent | 289347ad6aa4aae56cb90b42b6655ae49ecbb337 (diff) | |
| download | chrome-ec-d52ba1a4d4801fdba129be64fa4c1c53a7ced817.tar.gz | |
backlight: Adding support for active low GPIO signal
BUG=b:72007261
BRANCH=none
TEST=Verified with grunt board (with is active low)
Change-Id: I9a58148b8d92065bec982071ed1d97a466197e9a
Signed-off-by: Jett Rink <jettrink@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/872233
Reviewed-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Randall Spangler <rspangler@chromium.org>
Reviewed-by: Edward Hill <ecgh@chromium.org>
Diffstat (limited to 'common/backlight_lid.c')
| -rw-r--r-- | common/backlight_lid.c | 21 |
1 files changed, 17 insertions, 4 deletions
diff --git a/common/backlight_lid.c b/common/backlight_lid.c index 30b8b75eb7..9f2c984ec0 100644 --- a/common/backlight_lid.c +++ b/common/backlight_lid.c @@ -11,6 +11,19 @@ #include "host_command.h" #include "lid_switch.h" + +/** + * Activate/Deactivate the backlight GPIO pin considering active high or low. + */ +void enable_backlight(int enabled) +{ +#ifdef CONFIG_BACKLIGHT_LID_ACTIVE_LOW + gpio_set_level(GPIO_ENABLE_BACKLIGHT_L, !enabled); +#else + gpio_set_level(GPIO_ENABLE_BACKLIGHT, enabled); +#endif +} + /** * Update backlight state. */ @@ -18,14 +31,14 @@ static void update_backlight(void) { #ifdef CONFIG_BACKLIGHT_REQ_GPIO /* Enable the backlight if lid is open AND requested by AP */ - gpio_set_level(GPIO_ENABLE_BACKLIGHT, lid_is_open() && - gpio_get_level(CONFIG_BACKLIGHT_REQ_GPIO)); + enable_backlight(lid_is_open() && + gpio_get_level(CONFIG_BACKLIGHT_REQ_GPIO)); #else /* * Enable backlight if lid is open; this is AND'd with the request from * the AP in hardware. */ - gpio_set_level(GPIO_ENABLE_BACKLIGHT, lid_is_open()); + enable_backlight(lid_is_open()); #endif } DECLARE_HOOK(HOOK_LID_CHANGE, update_backlight, HOOK_PRIO_DEFAULT); @@ -60,7 +73,7 @@ static int switch_command_enable_backlight(struct host_cmd_handler_args *args) { const struct ec_params_switch_enable_backlight *p = args->params; - gpio_set_level(GPIO_ENABLE_BACKLIGHT, p->enabled); + enable_backlight(p->enabled); return EC_RES_SUCCESS; } |