From d5371c2669f0fc7a660b5f28ab61e19859912da5 Mon Sep 17 00:00:00 2001 From: John Galt Date: Fri, 10 Mar 2023 10:16:27 -0500 Subject: [PATCH] [REAPPLY/FIXUP] goodix: always return 1 for screen_state On some devices this worsens goodix performance due to userspace driver decision. Signed-off-by: Harish --- drivers/input/fingerprint/goodix/gf_spi.c | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/drivers/input/fingerprint/goodix/gf_spi.c b/drivers/input/fingerprint/goodix/gf_spi.c index 2d86746278d8..483d6677a727 100644 --- a/drivers/input/fingerprint/goodix/gf_spi.c +++ b/drivers/input/fingerprint/goodix/gf_spi.c @@ -646,9 +646,9 @@ static ssize_t screen_state_get(struct device *device, struct device_attribute *attribute, char *buffer) { - struct gf_dev *gfDev = dev_get_drvdata(device); +// struct gf_dev *gfDev = dev_get_drvdata(device); - return scnprintf(buffer, PAGE_SIZE, "%i\n", gfDev->screen_state); + return scnprintf(buffer, PAGE_SIZE, "%i\n", 1); } static inline ssize_t udfps_pressed_get(struct device *device, @@ -839,9 +839,6 @@ static int __always_inline goodix_fb_state_chg_callback( } #endif } - gf_dev->screen_state = 0; - sysfs_notify(&gf_dev->spi->dev.kobj, - NULL, dev_attr_screen_state.attr.name); break; case MSM_DRM_BLANK_UNBLANK: if (gf_dev->device_available == 1) { @@ -855,9 +852,6 @@ static int __always_inline goodix_fb_state_chg_callback( SIGIO, POLL_IN); #endif } - gf_dev->screen_state = 1; - sysfs_notify(&gf_dev->spi->dev.kobj, - NULL, dev_attr_screen_state.attr.name); break; default: pr_info("%s defalut\n", __func__);