-
Notifications
You must be signed in to change notification settings - Fork 86
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
fix Tolino Epos 2 to the correct light controller #531
Conversation
app/src/main/java/org/koreader/launcher/device/LightsFactory.kt
Outdated
Show resolved
Hide resolved
Co-authored-by: Benoit Pierre <benoit.pierre@gmail.com>
Makes no sense to revert. It must be a FW thingy. @rola25: are you on the lastest firmware for your device? Which one? Maybe I'm missing something, |
I mean, |
We did, it was my fault here: https://github.com/koreader/android-luajit-launcher/pull/513/files#diff-7d3fed5a40ea52bd9274153219933a8cac5bdacd5622db82e3d45c8ed5fcbcb4L13. |
Thanks for the info, @hugleo Now makes sense. Then please also change |
For TOLINO_EPOS1 TolinoNtx seems to be good. koreader/koreader#8482 (comment) |
Please no. Both on the same place. (Root!). But the FW version thingy might apply. Newer firmwares might be compatible with the non-root implementation. The old root method might be invalid... |
OK, done. AFAIK, the root driver was developed and tested for the TOLINO EPOS 2. Good to know that they might still release new firmware for old TOLINOs even today. Next time, I'll buy a TOLINO! :D |
Thanks!
Both were "Root" before the regression.
:))) I mean in general, like Onyxes or new tolino "vision" devices. |
But they really do: https://mytolino.com/software-updates-for-tolino-ereaders/ EDIT: 24 is the day not the year. The year is 2014. Is really too old :) Seems the OP can be the only one with the device alive. ls -l |
Hi, no I'm not on the latest firmware, mine is 16.1.0. I also have a Tolino Epos 1 - on weekend I can test koreader there. |
Can you test the Tolino Epos 1 on the eInk test and tell if will work TolinoRoot or TolinoNtx lights? This way we will have at least two reports on the device and can commit to the best option. |
I tested it on the Tolino Epos 1 -> warmth is working there with both drivers, ntx and tolinoroot ... very strange ;-) OK i debugged further: Epos1: Epos2: |
@pazos I think that's all, go for the root here as work for both devices. |
Isn't non-root preferable? |
Reading this comment: koreader/koreader#12843 is implying root I guess. |
All non recent Nooks and Tolinos come with a closed version of android where their stock reading app is a big monolithic launcher. Usually the way to "gain access" involves a way to get adb working and using adb to install an alternate launcher that lets you install and launch other apks. New Tolino (vision?) comes with an Onyx/Mebook like launcher and with recent-ish android versions. Those devices are already "opened" but obtaining root in there might be difficult. No idea. tl;dr: if v15 and v16 of the tolino epos1 firmware work with the non-root controller we can upgrade. Otherwise stay as is, root is easily achievable in "closed" tolinos. |
What do you prefer? merge this as is or revert to ntx? |
@hugleo koreader/koreader#12843 (comment) Now I think the non-root is better. Hopefully works on v16 too. Who knows? |
hmm. let be ntx for now then. |
fix koreader/koreader#8482 (comment)
This change is