From 0e4b40db0a361edca1cd9d4070bc2345648b76b6 Mon Sep 17 00:00:00 2001 From: Bruno Martins Date: Thu, 16 Jul 2020 23:04:49 +0100 Subject: [PATCH] sm6250-common: power: Scan input devices for eligible DT2W node It is no longer a surprise for anyone to see Xiaomi shipping the same device with different displays and different displays usually mean different touchscreen drivers. The current double tap handling in the power HAL only takes into account one specific node, meant to be set in the device tree. However, if the touchscreen was registered as a different input device it would cause the feature to be broken. Address the issue by iterating through the input devices in order to find one supported touchscreen input to be used. Co-authored-by: LuK1337 Change-Id: I2f7fb4a8b0c9cd1a16d6c2b93602d285a191f170 --- BoardConfigCommon.mk | 3 ++ power/power-mode.cpp | 105 +++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 108 insertions(+) create mode 100644 power/power-mode.cpp diff --git a/BoardConfigCommon.mk b/BoardConfigCommon.mk index b1492d8..182a2db 100644 --- a/BoardConfigCommon.mk +++ b/BoardConfigCommon.mk @@ -147,6 +147,9 @@ BOARD_USES_QCOM_HARDWARE := true BOARD_VENDOR := xiaomi TARGET_BOARD_PLATFORM := atoll +# Power +TARGET_POWERHAL_MODE_EXT := $(COMMON_PATH)/power/power-mode.cpp + # Properties TARGET_ODM_PROP += $(COMMON_PATH)/odm.prop TARGET_PRODUCT_PROP += $(COMMON_PATH)/product.prop diff --git a/power/power-mode.cpp b/power/power-mode.cpp new file mode 100644 index 0000000..d9972c9 --- /dev/null +++ b/power/power-mode.cpp @@ -0,0 +1,105 @@ +/* + * Copyright (C) 2021 The LineageOS Project + * + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ + +#include +#include +#include +#include + +namespace { +int open_ts_input() { + int fd = -1; + DIR* dir = opendir("/dev/input"); + + if (dir != NULL) { + struct dirent* ent; + + while ((ent = readdir(dir)) != NULL) { + if (ent->d_type == DT_CHR) { + char absolute_path[PATH_MAX] = {0}; + char name[80] = {0}; + + strcpy(absolute_path, "/dev/input/"); + strcat(absolute_path, ent->d_name); + + fd = open(absolute_path, O_RDWR); + if (ioctl(fd, EVIOCGNAME(sizeof(name) - 1), &name) > 0) { + if (strcmp(name, "fts_ts") == 0 || strcmp(name, "NVTCapacitiveTouchScreen") == 0) + break; + } + + close(fd); + fd = -1; + } + } + + closedir(dir); + } + + return fd; +} +} // anonymous namespace + +namespace aidl { +namespace google { +namespace hardware { +namespace power { +namespace impl { +namespace pixel { + +static constexpr int kInputEventWakeupModeOff = 4; +static constexpr int kInputEventWakeupModeOn = 5; + +using ::aidl::android::hardware::power::Mode; + +bool isDeviceSpecificModeSupported(Mode type, bool* _aidl_return) { + switch (type) { + case Mode::DOUBLE_TAP_TO_WAKE: + *_aidl_return = true; + return true; + default: + return false; + } +} + +bool setDeviceSpecificMode(Mode type, bool enabled) { + switch (type) { + case Mode::DOUBLE_TAP_TO_WAKE: { + int fd = open_ts_input(); + if (fd == -1) { + LOG(WARNING) + << "DT2W won't work because no supported touchscreen input devices were found"; + return false; + } + struct input_event ev; + ev.type = EV_SYN; + ev.code = SYN_CONFIG; + ev.value = enabled ? kInputEventWakeupModeOn : kInputEventWakeupModeOff; + write(fd, &ev, sizeof(ev)); + close(fd); + } + return true; + default: + return false; + } +} + +} // namespace pixel +} // namespace impl +} // namespace power +} // namespace hardware +} // namespace google +} // namespace aidl