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

lorawan: Fix issue in unconfirmed_retransmission behavior #15225

Merged

Conversation

zul00
Copy link
Contributor

@zul00 zul00 commented Feb 11, 2022

Summary of changes

Fixes #15224

Fixed issue on LoRaWANStack based on lorawan v1.0.2 specification page 24 line 25-27. The specification mentioned that that end-device shall stop retransmission when receiving ANY downlink on RX1 or RX2 window.

  • Removed conflicting behavior in LoRaWANStack::post_process_tx_with_reception()
  • Updated comment section of nb_trials at lorawan_data_structures.h

Impact of changes

End-device will stop retransmission of unconfirmed uplink on receiving ANY downlink, as specified
by the specificiation.

Migration actions required

Documentation

None


Pull request type

[x] Patch update (Bug fix / Target update / Docs update / Test update / Refactor)
[] Feature update (New feature / Functionality change / New API)
[] Major update (Breaking change E.g. Return code change / API behaviour change)

Test results

[] No Tests required for this change (E.g docs only update)
[] Covered by existing mbed-os tests (Greentea or Unittest)
[x] Tests / results supplied as part of this PR

Reviewers


Fixed issue on LoRaWANStack based on lorawan v1.0.2 specification page
24 line 25-27. The specification mentioned that that end-device shall
stop retransmission when receiving ANY downlink on RX1 or RX2 window.

- Removed conflicting behavior in
  LoRaWANStack::post_process_tx_with_reception()
- Updated comment section of nb_trials at lorawan_data_structures.h
@zul00
Copy link
Contributor Author

zul00 commented Feb 11, 2022

Below is the screenshot from Interop test engine of our DUT using a patched mbed-os. The test is configured as follow:

  • InteropEngine : 1.3.2
  • LRC(Actility) : 1.22.4
  • LoRa region : EU868
  • LoRa class : A

image

@0xc0170
Copy link
Contributor

0xc0170 commented Feb 11, 2022

Thank you the contribution and sharing the tests.

@0xc0170
Copy link
Contributor

0xc0170 commented Feb 11, 2022

CI started

@mergify mergify bot added the needs: CI label Feb 11, 2022
@mbed-ci
Copy link

mbed-ci commented Feb 11, 2022

Jenkins CI Test : ✔️ SUCCESS

Build Number: 1 | 🔒 Jenkins CI Job | 🌐 Logs & Artifacts

CLICK for Detailed Summary

jobs Status
jenkins-ci/mbed-os-ci_build-cloud-example-ARM ✔️
jenkins-ci/mbed-os-ci_build-cloud-example-GCC_ARM ✔️
jenkins-ci/mbed-os-ci_unittests ✔️
jenkins-ci/mbed-os-ci_build-greentea-ARM ✔️
jenkins-ci/mbed-os-ci_build-greentea-GCC_ARM ✔️
jenkins-ci/mbed-os-ci_build-example-ARM ✔️
jenkins-ci/mbed-os-ci_build-example-GCC_ARM ✔️
jenkins-ci/mbed-os-ci_greentea-test ✔️

@0xc0170 0xc0170 added the release-type: patch Indentifies a PR as containing just a patch label Feb 14, 2022
@0xc0170 0xc0170 merged commit c50f490 into ARMmbed:master Feb 14, 2022
@0xc0170 0xc0170 changed the title Fixed issue in unconfirmed_retransmission behavior lorawan: Fix issue in unconfirmed_retransmission behavior Feb 14, 2022
@mergify mergify bot removed the ready for merge label Feb 14, 2022
@mbedmain mbedmain added release-version: 6.16.0 Release-pending and removed release-type: patch Indentifies a PR as containing just a patch Release-pending labels Jun 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Incorrect LoRaWAN behavior on unconfirmed uplink retransmission
4 participants