Skip to content

Conversation

oopsbagel
Copy link
Collaborator

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.

@oopsbagel oopsbagel force-pushed the tmobile-tmohs1 branch 3 times, most recently from 8974190 to 412719c Compare June 14, 2025 05:01
@oopsbagel oopsbagel marked this pull request as ready for review June 20, 2025 08:05
@oopsbagel
Copy link
Collaborator Author

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.

@oopsbagel oopsbagel force-pushed the tmobile-tmohs1 branch 4 times, most recently from 6e5f2cf to 8f42035 Compare June 25, 2025 07:18
@oopsbagel oopsbagel requested review from untitaker, wgreenberg, gaykitty and cooperq and removed request for untitaker and wgreenberg June 25, 2025 07:26
untitaker
untitaker previously approved these changes Jun 25, 2025
Copy link
Collaborator

@untitaker untitaker left a 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.

untitaker
untitaker previously approved these changes Jun 25, 2025
gaykitty
gaykitty previously approved these changes Jun 26, 2025
Copy link
Collaborator

@gaykitty gaykitty left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@gaykitty
Copy link
Collaborator

Just tested on my TMOHS1 and it worked great! One suggestion, having a special error message for a wrong password would be nice.

@cooperq
Copy link
Collaborator

cooperq commented Jul 16, 2025

Ready to merge this as soon as you rebase @oopsbagel

cooperq
cooperq previously approved these changes Jul 16, 2025
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.
@oopsbagel oopsbagel dismissed stale reviews from cooperq, gaykitty, and untitaker via 509023a July 16, 2025 23:14
@oopsbagel oopsbagel requested review from gaykitty, untitaker and cooperq and removed request for untitaker July 16, 2025 23:25
@cooperq cooperq merged commit ca0151f into EFForg:main Jul 16, 2025
32 checks passed
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.

4 participants