goodix_driver_gt9886: UDFPS, double and single tap
squash off the following commits:
goodix_ts: Implement udfps_pressed attribute
referenced both:
Demon000/kernel_motorola_sm7325@8a8a2fa
RealJohnGalt/GaltsGulch-sm8150@bfa8699
It sends a 1 to udfps_pressed when a finger touches fod location only when screen is off
using the gestures thingy in touchscreen driver
Change-Id: Iaf7f4ec214fb601b2534f5ee28fa4c02edcce952
goodix_driver_gt9886: implement double_tap_pressed attribute
Practically the same as udfps pressed: pwnrazr/kernel_raphael_sm8150@dc44a40
https: //github.com/RealJohnGalt/GaltsGulch-sm8150/commit/317330f537f14eedacae71514e4b4bd7b593f289
Change-Id: I1b99ed9da10d02b008244ff26d6a13eda169dd23
goodix_driver_gt9886: remove key_wakeup from dt2w gesture
This is used in previous implementation to wakeup the screen
goodix_driver_gt9886: implement double_tap_enabled
Placeholder for now since I plan on properly disabling the doze modes instead of forcing on all the time
Change-Id: Iccab73664d945872e2ca2a0f16bb1c693814465c
goodix_driver_gt9886: resend gesture command if double tap disabled
since we're forcing double wakeup mode, if dt2w is disabled in android and user accidentally
double tap on screen it will kill screen off fod until screen is turned on and off again.
We need to resend gesture command to prevent this
goodix_driver_gt9886: implement udfps_enabled
Placeholder for now since I plan on properly disabling the doze modes instead of forcing on all the time
Change-Id: I8f789c778e0b085190cff01afe15ae730b077fc0
goodix_driver_gt9886: refactor goodix_sync_ic_stat
Now it can handle dt2w changes too
goodix_driver_gt9886: gesture: also handle both gesture disabled
Now there's a possibility that both gestures are disabled
goodix_driver_gt9886: core: properly use _enabled of new attributes
This should properly set gesture modes based on whether user has it enabled or not in userspace
instead of forcing gestures to be on all the time like before
goodix_driver_gt9886: use switch case for drm notifier
Not sure if it helps much but it looks prettier
Very inspired by: RealJohnGalt/GaltsGulch-sm8150@c1c1557
Change-Id: I5b2009be7d4801deb24c1f646df0354d201422d8
goodix_driver_gt9886: don't run check_gesture_stat for fod status store
This gets called in udfps and double tap enabled store now so quite unnecessary
Change-Id: I8facf6d775ab7fe693e5a686282dad0be284a0e6
input: touchscreen: goodix_driver_gt9886: Remove input event dt2w handling
Change-Id: I8437e5a2b56a57efea9184021ff28639fe526120
goodix_driver_gt9886: use new attributes for gesture handling
remove double wakeup and use double_tap_enabled instead
use udfps_enabled inplace of fod_status for gesture things but keep
for certain touch related things as I'm not sure of it's behaviour there
Change-Id: Ifdf8ed66e7128efb705ef7b53cd8cc7b24eeaf24
goodix_driver_gt9886: refactor set_suspend_func
used chatgpt for this
goodix_driver_gt9886: return early on DRM events if already resumed
IDK why it wasn't like this by default since it's there for suspend
input: goodix: Fix wrongly placed mutex unlock
Xiaomeme added a condition that can keep on iteration over modules
however left mutex unlock before it. This can cause premature mutex
unlock and double-unlock afterwards which leads to undefined behavior.
Mutex unlock should be called only when taking out path.
Change-Id: Ie3595fec0dd46d548e6b14f6b32132e224928021
goodix_driver_gt9886: use PAGE_SIZE instead of 10 for sysfs buffer size
Change-Id: I310883ba0d20d700fa452d47ccb696420d946c6e
goodix_driver_gt9886: use u8 instead of int for udfps* and double_tap*
Save them bytes
Change-Id: I4c63bc9f18ff068d56a039052f128db7f7e0471a
goodix_driver_gt9886: implement single tap sensor
Pretty much the same as udfps/dt2w stuff
Just use one of the FOD gestures as it detects specifically single taps.
Also modify some logic so the appropriate mode gets set in cases like
udfps and dt2w is disabled but st2w is enabled
Change-Id: Idd8de03bf0a633b7553a20f0ea614291e98c3222
Change-Id: I5e9a09848ada86a0ca662c83fe9875f469156778
Signed-off-by:
Pranav Vashi <neobuddy89@gmail.com>
Loading
Please register or sign in to comment