-
Notifications
You must be signed in to change notification settings - Fork 223
feat: support Tmobile TMOHS1 hotspot #393
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
Conversation
8974190
to
412719c
Compare
Oops, I didn't actually intend to mark this ready for review until I included the refactors mentioned above. I could handle those in a separate PR but I'd rather do it here. |
6e5f2cf
to
8f42035
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the ci job for building the daemon is missing.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Just tested on my TMOHS1 and it worked great! One suggestion, having a special error message for a wrong password would be nice. |
Ready to merge this as soon as you rebase @oopsbagel |
Add support for the Tmobile TMOHS1, a Wingtech CT2MHS01-based hotspot with a Qualcomm mdm9607. The TMOHS1 has no screen, only 5 LEDs, two of which are RGB.
The Tmobile TMOHS1 is clearly based on the Wingtech CT2MHS01, and thus the installation process looks very similar between them.
These were originally excluded because they weren't tested, as they are subsets of bands 25, 26, and 66, but are worth enumerating.
7dc9495
to
509023a
Compare
Add support for the Tmobile TMOHS1, a Wingtech CT2MHS01-based hotspot with a Qualcomm mdm9607. The TMOHS1 has no screen, only 5 LEDs, two of which are RGB.
This pull request is a draft until #387 is merged, because it uses common installer code and makes modifications to some of the same files (like
display/src/mod.rs
).I will refactor out some of the other commonalities in the installer between this and the Wingtech before merging.