Skip to content

LoRaWAN: Fixing transport of fatal TX timeout event #7344

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

Merged
merged 1 commit into from
Jun 28, 2018

Conversation

hasnainvirk
Copy link
Contributor

@hasnainvirk hasnainvirk commented Jun 27, 2018

Description

This commit fixes the issue reported in #7285.
If the radio is unable to transmit, its a fatal error and can happen
both while joining or sending a normal packet. In the case of such
a catastrophy we ought to tell the application that this happened.

A fix for the radio driver will also be patched.

Pull request type

[X] Fix
[ ] Refactor
[ ] New target
[ ] Feature
[ ] Breaking change

Target

Mbed OS 5.9.2

This commit fixes the issue reported in ARMmbed#7285.
If the radio is unable to transmit, its a fatal error and can happen
both while joining or sending a normal packet. In the case of such
a catastrophy we ought to tell the application that this happened.

A fix for the radio driver will also be patched.
@hasnainvirk
Copy link
Contributor Author

@kjbracey-arm @AnttiKauppila @cmonr @0xc0170 Please review.

@hasnainvirk
Copy link
Contributor Author

hasnainvirk commented Jun 27, 2018

@0xc0170
Copy link
Contributor

0xc0170 commented Jun 27, 2018

/morph build

@mbed-ci
Copy link

mbed-ci commented Jun 27, 2018

Build : SUCCESS

Build number : 2462
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/7344/

Triggering tests

/morph test
/morph uvisor-test
/morph export-build
/morph mbed2-build

@mbed-ci
Copy link

mbed-ci commented Jun 27, 2018

@mbed-ci
Copy link

mbed-ci commented Jun 28, 2018

@cmonr cmonr merged commit 34fd640 into ARMmbed:master Jun 28, 2018
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.

5 participants