Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add overlays for Xiaomi Mi 10T Lite/Mi 10i/Redmi Note 9 Pro #643

Open
wants to merge 6 commits into
base: pie
Choose a base branch
from

Conversation

iovxw
Copy link

@iovxw iovxw commented Sep 18, 2022

Tested on vendor V13.0.5.0.SJSCNXM_20220510

iovxw and others added 5 commits September 18, 2022 14:01
Steps to enable:

1. Settings -> Phh Treble Settings -> IMS features
2. Click "Instal IMS APK for Qualcomm vendor" and install it
3. Request IMS network ✅
4. Force the presence of 4G Calling setting ✅
@iovxw
Copy link
Author

iovxw commented Sep 19, 2022

config_is_powerbutton_fps exists
https://cs.android.com/android/platform/superproject/+/android-12.1.0_r8:frameworks/base/core/res/res/values/config.xml;l=4646

Okay, it broke fingerprint settings, need more testing

@iovxw iovxw marked this pull request as draft September 19, 2022 04:12
@iovxw
Copy link
Author

iovxw commented Sep 19, 2022

I can't make config_is_powerbutton_fps work using just overlay, config_sfps_sensor_props didn't help (the provider is Fingerprint21(HIDL) not FingerprintProvider(AIDL))

Without it, the fingerprint set up guide shows wrong position, but at least the unlock animation looks good now

@iovxw iovxw force-pushed the add-gauguin-overlay branch from 0fadcf6 to dafcef3 Compare September 19, 2022 08:41
@iovxw iovxw marked this pull request as ready for review September 19, 2022 08:43
@iovxw
Copy link
Author

iovxw commented Sep 19, 2022

Just for reference if anyone encountered the same issue: LineageOS/android_frameworks_base@21986c0

@iovxw iovxw requested a review from phhusson September 26, 2022 02:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants