Skip to content
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 RF switch for Tracker E #4621

Merged
merged 4 commits into from
Sep 3, 2024
Merged

fix RF switch for Tracker E #4621

merged 4 commits into from
Sep 3, 2024

Conversation

caveman99
Copy link
Member

this creates the same switch matrix we originally had in godmode. Still not working on my version.

@caveman99 caveman99 marked this pull request as ready for review September 3, 2024 13:58
@caveman99
Copy link
Member Author

This should be merged since at the current state SX1262 and SX1280 won't work either.

src/mesh/SX126xInterface.cpp Outdated Show resolved Hide resolved
src/mesh/LR11x0Interface.cpp Outdated Show resolved Hide resolved
@thebentern thebentern requested a review from GUVWAF September 3, 2024 19:05
Copy link
Member

@GUVWAF GUVWAF left a comment

Choose a reason for hiding this comment

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

Looks good. This also includes the fix for the header error in RadioLib, so we should also get less CRC failures now :)

@thebentern thebentern merged commit ff40a3f into master Sep 3, 2024
105 checks passed
@thebentern thebentern deleted the T-1000E branch September 3, 2024 19:21
@caveman99
Copy link
Member Author

caveman99 commented Sep 3, 2024

Looks good. This also includes the fix for the header error in RadioLib, so we should also get less CRC failures now :)

My hope was that this fix (header error) would make my T1000E work again, but alas its still deaf with this
firmware.

@Talie5in
Copy link
Contributor

Talie5in commented Sep 12, 2024

Looks good. This also includes the fix for the header error in RadioLib, so we should also get less CRC failures now :)

My hope was that this fix (header error) would make my T1000E work again, but alas its still deaf with this firmware.

Hey @caveman99 - i was just doing iterative testing through 2.5.0 TP versions, and this is where the t1000e breaks for me after this change.... (dont hear anything other then a really close by Rak4631, basically deaf).

If i go back to 2.5.0.33eb073 i get lots of nodes showing up...
Can see me going through it here - https://discord.com/channels/867578229534359593/1039674919346249728/1283614633311080582

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