Skip to content

Enable PDP type IPV6 and IPV4V6 to support more cellular service prov… #13432

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
Sep 17, 2020

Conversation

felser
Copy link
Contributor

@felser felser commented Aug 14, 2020

Enable PDP type IPV6 and IPV4V6 to support more cellular service providers

Summary of changes

Adds support for more service providers on Telit radios.
This fixes connectivity failures for MTS_DRAGONFLY_F411RE/F413RH builds that are not using AT&T.

Impact of changes

This affects any cellular platform that uses the Telit radio HE910 code.

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

Tested the mbed-os-example cellular on LE910 Telit radios with AT&T, Sprint and Verizon carriers. Context ID contained only IPV4, only IPV4V6 or a combination of both. All connections were successful.

[] 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


@felser
Copy link
Contributor Author

felser commented Aug 14, 2020

I'm not sure if this is the best solution or if instead there should be support added separately for LE910 radios. So far this is the only change I have needed to support a range of Telit radios using the mbed included HE910 code. I figured I'd submit the PR and discussions can occur. FYI, I am out of the office the next couple days.

@ciarmcom ciarmcom added the release-type: patch Indentifies a PR as containing just a patch label Aug 14, 2020
@ciarmcom ciarmcom requested a review from a team August 14, 2020 14:00
@ciarmcom
Copy link
Member

@felser, thank you for your changes.
@ARMmbed/mbed-os-maintainers please review.

@felser
Copy link
Contributor Author

felser commented Aug 24, 2020

Is there more you need from me to move this along?

@maclobdell
Copy link
Contributor

@ARMmbed/mbed-os-maintainers can you comment on this?

@mbed-ci
Copy link

mbed-ci commented Sep 10, 2020

Jenkins CI Test : ✔️ SUCCESS

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

CLICK for Detailed Summary

jobs Status
jenkins-ci/mbed-os-ci_unittests ✔️
jenkins-ci/mbed-os-ci_build-GCC_ARM ✔️
jenkins-ci/mbed-os-ci_build-ARM ✔️
jenkins-ci/mbed-os-ci_greentea-test ✔️
jenkins-ci/mbed-os-ci_dynamic-memory-usage ✔️
jenkins-ci/mbed-os-ci_cloud-client-pytest ✔️

@0xc0170 0xc0170 merged commit 4809f51 into ARMmbed:master Sep 17, 2020
@mergify mergify bot removed the ready for merge label Sep 17, 2020
@mbedmain mbedmain added release-version: 6.4.0 Release-pending and removed release-type: patch Indentifies a PR as containing just a patch Release-pending labels Oct 20, 2020
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.

6 participants