feat: initial Miyoo Mini Flip compatibility; hardware detection, lid close events, lid close settings options#1860
Conversation
|
@xXJSONDeruloXx Thank you very much for this PR. I will test these additions and get back to you as soon as possible. |
|
@xXJSONDeruloXx I’m curious how you created MainUI 285 clean and MainUI 285 expert ? |
|
@xXJSONDeruloXx Ok, |
|
@Sichroteph thanks for the review, updated this to point to same bin |
|
I tested this and the hinge/lid isn'tbeing detected still, i have however started my own Onion Fork for the Mini Flip & got most stuff functional if you'd like me to make a PR with my changes instead?
|
|
Yes please, that would be great. We’ll make sure to credit both of you. |
|
I'm happy to test today when I get back from the office if anyone would like me to 👍 |
|
@Dark98 I can not find your fork, but if you like I would be happy to implement your working version of lid detection and co author the commits |
|
@xXJSONDeruloXx @Sichroteph @ajb3932 I'll get them PR'ed later today :)
|
|
well, now I wanna race ya to find a working method lol |
|
ok have my flip mini now, was able to test this and with recent commit I am having lid close and open sleep wake, and tweaks options to change from sleep to shutdown does appear to be working too, so lid shut shuts down |
|
I've had it working for a while so I technically won the race :P (tenlevels beat everyone 😂) No point me making a PR if you've now got it working :) Aside from a few visual glitches & chargingState, I think I have most stuff working on my fork so if you want help with anything like USB-C audio for example
|
|
Demo with the 3 lid close settings in tweaks: |
Is chargingState working properly for you?(i didnt test when i tested your build at first) I cant get it working on my fork 😅 |
|
@Dark98 can you link me to your fork? |
It's currently private :P sorry😅 |
|
@ajb3932 @Sichroteph ready to test |
|
pushed a few housekeeping commits, plus I realized when lid close is set to shutdown and game is running, we were doing hard shutdown, now its same as half sec power button press, where it saves state and shuts down gracefully with also did some A/B testing with the screen blanking concept vs the fallback for sleep, negligible heat and battery life, so instead of the logic overhead I reverted that to mirror behavior of brief power btn press |
|
per discussions in the discord, here is a release off this branch, will encourage testing and report back or point users to this PR to gather feedback https://github.com/xXJSONDeruloXx/Onion/releases/tag/v4.5-dev-pr-1860 |
|
285headphone.c |
|
@fengfrw i prefer to keep the headphone change a separate PR |
|
I just tested your build on my mini flip. It works great. Thx for all your effort. Two observations: |
|
Addressed all comments/change requests and tested on my device. also went ahead and added the tenlevels headphone detection slightly modified for deviceid rather than hal sensor detect. Also on sleep to shutdown if set I am having that be respected on lid close shutdown, I think that grants the most consistent behavior across devices. if sleep to shutdown is not set, then lid close sleeps indefinitely as before. LMK if any other changes needed |
|
Can this merge? |
Long time Onion Enjoyer, first time PR-er.
I do not have a MM flip yet to test, but these additions are all in reference to the implementation work done in Allium to accommodate the flip's hardware and implement features around the lid events.
Please let me know if this is at all helpful, and what needs to be changed to get this in a merge ready state. Anyone with MM Flip hardware any testing would be greatly appreciated.