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

[Bug]: "my T-1000-E is deaf"- LR1110 can't communicate with SX1276 devices #4775

Open
1 of 2 tasks
fifieldt opened this issue Sep 19, 2024 · 25 comments
Open
1 of 2 tasks
Labels
bug Something isn't working hardware-support Add hardware support for new devices or modules high-priority Issues that affect core functionality or are "show stoppers" pinned Exclude from stale processing tech debt Code or lib references that are not up to date or propper standards

Comments

@fifieldt
Copy link
Contributor

fifieldt commented Sep 19, 2024

Hardware

Seeed Card Tracker T1000-E

Firmware Version

2.5.0

Description

As discussed in Discord over the past 3 weeks, there are multiple instances of people reporting issues with LR1110 radios. This issue is being created to serve as a central place for recording this information and coordinate testing.

Reported issues include:

  • Not being able to hear nodes more than 50m away
  • Not being able to communicate with SX1276 devices (eg tbeam)
@fifieldt fifieldt added the bug Something isn't working label Sep 19, 2024
@fifieldt
Copy link
Contributor Author

@Talie5in has identified #4621 as the breakpoint.

  • 2.5.0.d6dac17 OK
  • 2.5.0.33eb073 OK
  • 2.5.0.9ac0e26 - Talie5in: doesn't pick up anything other than WisMeshPocket (Rak4631)
  • v2.5.0.9ac0e26 - Fifield: can't talk with tbeam in the same room

@caveman99
Copy link
Member

@Talie5in has identified #4621 as the breakpoint.

Are the other devices on the same firmware level or older/newer? We have radiolib changes scattered in there somewhere.

@fifieldt
Copy link
Contributor Author

Currently trying: 33eb073 with a later version of RadioLib...

@fifieldt
Copy link
Contributor Author

OK, 33eb073 with RadioLib 7.0.0 results in the bad outcome.

By my logic, using the good meshtastic commit with just radiolib changes means the issue might be in RadioLib!

@fifieldt
Copy link
Contributor Author

RadioLib 6.6.0 (working version) is May 29 jgromes/RadioLib@954477b .

Stepping forward through RadioLib with known-good Meshtastic version.

@fifieldt
Copy link
Contributor Author

fifieldt commented Sep 19, 2024

Radiolib w/ Meshtastic 33eb073:
June 30 7a0b5bae9906905f32cd840ec79b2665f509f717 - OK
Jul 30 6d0c226e65ca1835849c9b516255bee642d2ab21 - OK
Aug 29 d3f9eaf30b438faa2406982a75f54be64f681909 - OK
Sep 7 951bfc0625f966b8462be40318b9020558ed0a3e - FAIL

Looks like the breaking change was introduced between Aug 29 and Sep 7.

Aug 29 & prior - T1000E can receive messages from Tbeam.
Sept 7 no messages get through.

@Talie5in
Copy link
Contributor

Talie5in commented Sep 19, 2024

@Talie5in has identified #4621 as the breakpoint.

Are the other devices on the same firmware level or older/newer? We have radiolib changes scattered in there somewhere.

The one example is my WisMeshPockt (Rak4631) on 2.5.0 Alpha (through the entire time I was testing different 2.5.0 TechPreview build - it was out in my car (50m away) and was the only device it would hear past 33eb073

Yes, RadioLib was changed in #4621 as @fifieldt has picked out of my Discord commentary

@fifieldt
Copy link
Contributor Author

OK, it looks like the breaking change is

bc801c70043c2482af8cd81947b855d7325a41ac [PHY] Channel scan configuration

@GUVWAF
Copy link
Member

GUVWAF commented Sep 19, 2024

That RadioLib commit introduces differences in the IRQ flags handling in startReceive(), so if using Meshtastic commit 33eb073 it is expected to break, but PR #4621 also should handle those API changes.

@fifieldt
Copy link
Contributor Author

fifieldt commented Sep 20, 2024

In my testing, I amended the meshtastic code appropriately to deal with the IRQ flag changes.

@fifieldt
Copy link
Contributor Author

fifieldt commented Sep 23, 2024

Potentially related: #4827
False alarm.

@valzzu
Copy link

valzzu commented Sep 25, 2024

works fine on long fast but doesent work on long moderate.
wio tracker is the only node that can communicate with it.

@GUVWAF
Copy link
Member

GUVWAF commented Sep 26, 2024

The incompatibility between SX127x and LR1110 really seems to be because of our non-standard sync word. If I change it on both boards to the default private network (0x12), I can send and receive between them.

@GUVWAF
Copy link
Member

GUVWAF commented Sep 26, 2024

Confirmed LONG_MODERATE doesn't work between LR1110 and other radios, and it's likely due to low data rate optimization (LDRO), which is automatically enabled for SX126x and SX127x when the symbol length is longer than 16ms, but it's not yet there for LR11x0. I'll see if I can add it.

@GUVWAF
Copy link
Member

GUVWAF commented Sep 26, 2024

@valzzu LONG_MODERATE should work after we update RadioLib to include jgromes/RadioLib#1237.

@valzzu
Copy link

valzzu commented Sep 26, 2024

Awesome

@GUVWAF
Copy link
Member

GUVWAF commented Sep 27, 2024

@Talie5in @fifieldt Can you try if #4885 fixes the bad receive performance for you?

In my testing, I amended the meshtastic code appropriately to deal with the IRQ flag changes.

Were you using lora.startReceive(RADIOLIB_LR11X0_RX_TIMEOUT_INF, RADIOLIB_IRQ_RX_DEFAULT_FLAGS, RADIOLIB_IRQ_RX_DEFAULT_MASK, 0)? If so, that only works from RadioLib commit jgromes/RadioLib@3115fc2 (which is the one PR #4621 uses and is three commits after the commit you used).

@Talie5in
Copy link
Contributor

Talie5in commented Sep 28, 2024

Can you try if #4885 fixes the bad receive performance for you?

@GUVWAF Built against 699d81c in PR#4885 and I am getting nodes appearing after a nodeDB reset. 🎉

Some logs attached - of note one interesting bits i've not seen before but are likely normal...

WARN | 04:04:10 490 [Router] Alloc an err=5,to=0x850354e6,idFrom=0x1b7f84e4,id=0x25516740

meshtastic-log-2024-09-28T04-07-35.735Z.log
Additional logging after a reboot

meshtastic-log-2024-09-28T04-15-59.764Z.log

@GUVWAF
Copy link
Member

GUVWAF commented Sep 28, 2024

Great!

WARN | 04:04:10 490 [Router] Alloc an err=5 means "Max. retransmission reached".

@meshtastic-bot
Copy link

This issue has been mentioned on Meshtastic. There might be relevant details there:

https://meshtastic.discourse.group/t/sensecap-t1000-e-not-seeing-t-beams/14913/2

@fifieldt
Copy link
Contributor Author

fifieldt commented Oct 1, 2024

Many thanks @GUVWAF , coming back online a d should be able to test in the next few days.

@fifieldt
Copy link
Contributor Author

Forgot to comment - the LR1110 now works >10km after those changes. The remaining issue is indeed the SX1276 one.

@fifieldt fifieldt changed the title [Bug]: Unclear Issues with LR1110; "my T-1000-E is deaf" [Bug]: "my T-1000-E is deaf"- LR1110 can't communicate with SX1276 devices Oct 22, 2024
@caveman99 caveman99 added tech debt Code or lib references that are not up to date or propper standards high-priority Issues that affect core functionality or are "show stoppers" hardware-support Add hardware support for new devices or modules pinned Exclude from stale processing labels Oct 30, 2024
@egzumer
Copy link

egzumer commented Dec 10, 2024

I found an interesting article about syncword setup between sx127x and sx126x. May help with this issue.
https://blog.classycode.com/lora-sync-word-compatibility-between-sx127x-and-sx126x-460324d1787a

@fifieldt
Copy link
Contributor Author

Word on discord is we got a new contact with Semtech and are discussing this.

@wojo
Copy link

wojo commented Feb 3, 2025

Any update on the new contact from Semtech on a potential field deployable fix?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working hardware-support Add hardware support for new devices or modules high-priority Issues that affect core functionality or are "show stoppers" pinned Exclude from stale processing tech debt Code or lib references that are not up to date or propper standards
Projects
None yet
Development

No branches or pull requests

8 participants