diff options
author | Furquan Shaikh <furquan@google.com> | 2018-10-19 15:34:33 -0700 |
---|---|---|
committer | chrome-bot <chrome-bot@chromium.org> | 2018-10-30 01:04:57 -0700 |
commit | 09a5e0a9398a1ca9e953969d5c10d3b60cda8eac (patch) | |
tree | 609fd9dfbc271cf71513e52df5d4473749e87e70 /board/poppy/board.h | |
parent | 23a82485cfd2e893fa147dd0fd877ff783316c8f (diff) | |
download | chrome-ec-09a5e0a9398a1ca9e953969d5c10d3b60cda8eac.tar.gz |
dptf: Get rid of CONFIG_DPTF_DEVICE_ORIENTATION
CONFIG_DPTF_DEVICE_ORIENTATION was added to indicate mode change to
the host to allow it to read the tablet mode flag from shared EC
memory and select the right DPTF table to load (if supported).
However, this config seems unnecessary because of the following
reasons:
1. Host sets SCI mask to indicate to the EC which events it wants to
process. Thus, even if the EC sets mode change flag, it will not be
notified to the host unless it supports mode change event.
2. Additionally, if host supports mode change event, but does not
support multiple DPTF tables, then EC ACPI code takes care of ensuring
that there is a thermal event handler present to reload tables.
3. CONFIG_DPTF_DEVICE_ORIENTATION was defined for almost all new x86
boards.
BUG=b:117844490
BRANCH=None
TEST=make -j buildall
Change-Id: Ic4097ae047e2d559673a321da4df86514f902993
Signed-off-by: Furquan Shaikh <furquan@google.com>
Reviewed-on: https://chromium-review.googlesource.com/1292359
Commit-Ready: Furquan Shaikh <furquan@chromium.org>
Tested-by: Furquan Shaikh <furquan@chromium.org>
Reviewed-by: Gwendal Grignou <gwendal@chromium.org>
Diffstat (limited to 'board/poppy/board.h')
-rw-r--r-- | board/poppy/board.h | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/board/poppy/board.h b/board/poppy/board.h index 358b83aaa1..8b0a0f5944 100644 --- a/board/poppy/board.h +++ b/board/poppy/board.h @@ -22,7 +22,6 @@ #define CONFIG_BUTTON_TRIGGERED_RECOVERY #define CONFIG_DETACHABLE_BASE #define CONFIG_DPTF -#define CONFIG_DPTF_DEVICE_ORIENTATION #define CONFIG_EMULATED_SYSRQ #define CONFIG_FLASH_SIZE 0x80000 #define CONFIG_FPU |