-
Notifications
You must be signed in to change notification settings - Fork 235
#451 Implement UPSEQ #452
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
base: main
Are you sure you want to change the base?
#451 Implement UPSEQ #452
Conversation
|
Tested with the 10.3" Carta1300 display ED103MC1. Worked wonderful! Nice investigation! ❤️ Needs to fixed before merging:
|
|
@jonasschnelli EpdDisplayType need to be turned into bitmap flags to allow combination. that one is fixed! the other one I still need to research why it triggers. @vroland please take a look at this too will add you as a reviewer when done |
|
@jonasschnelli if you can please do a fullclean, pull this branch again, and build. |
|
Hey, nice work on finding this out! I'm wondering though, why implement it as a new board type? Does it actually require different hardware? Since it may also be useful to enable with normal v7. |
|
Did you check if that power-on sequence impairs any of the other displays? Since it's mentioned in the docs, maybe it should just be the default? |
|
Actually it does work with all the others I tested and it can of course work with v7. I’m working in a v7_103 PCB that is still not ready to become open source but soon I will send you a new board so you can try it out. In this version the 40 pin FPC comes already mirrored and supporting the modern 16 bit displays from Eink.Thanks for your review! |
|
Ok, then that sounds like it makes sense to have that routine for all the boards with a TPS65185 and factor it out into a common function that checks fo rthe DISPLAY_UPSEQ_MC2 display flag and then sets up the power up sequence if set. |
|
Great then I can refactor this and do it for v6, v7. Although v6 didn’t had 16 bit display support maybe there will be some 8 bit panels requiring this up sequence. |
|
Updated. Thanks for your review @vroland |

Implements a solution for #451