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

HL7800 driver fixes for TLS and DTLS #83763

Merged
merged 15 commits into from
Jan 20, 2025

Conversation

rerickson1
Copy link
Member

@rerickson1 rerickson1 commented Jan 9, 2025

  • Ensure TCP and UDP socket port numbers are in network order until sending an AT command.
  • Fixes for DNS when using low power modes
  • Add HL7800 support to lwm2m_client, http_get, and aws_iot_mqtt samples. All three samples we tested successfully on a pinnacle_100_dvk.

Fixes
#81672

@rerickson1 rerickson1 self-assigned this Jan 9, 2025
Ensure the proper byte order is used for the network socket in the stack.
When sending the AT commands, the port needs to be converted to
host endianess.
TLS handshakes check that the remote address and port match
what is expected.

Signed-off-by: Ryan Erickson <[email protected]>
Add commented kconfig options for debugging DTLS connections.

Signed-off-by: Ryan Erickson <[email protected]>
@rerickson1 rerickson1 force-pushed the PR/PROD-17883 branch 5 times, most recently from 997f640 to af0f757 Compare January 10, 2025 01:42
@rerickson1 rerickson1 changed the title HL7800 driver fixes HL7800 driver fixes for TLS and DTLS Jan 10, 2025
@rerickson1 rerickson1 marked this pull request as ready for review January 10, 2025 18:24
@zephyrbot zephyrbot added the Release Notes To be mentioned in the release notes label Jan 10, 2025
@rerickson1 rerickson1 requested a review from jukkar January 15, 2025 16:22
@rerickson1 rerickson1 added bug The issue is a bug, or the PR is fixing a bug priority: medium Medium impact/importance bug labels Jan 15, 2025
rerickson1 and others added 10 commits January 16, 2025 10:28
Add an overlay for hl7800 modem configuration.
Adjust the client to wait until the network is ready before
trying to connect to the lwm2m server.

Signed-off-by: Ryan Erickson <[email protected]>
All other APIs that need to reset the modem do so in an async way.
Make the rest API async as well.
This fixes issues where modem events related to the reset
were not generated properly.

Signed-off-by: Ryan Erickson <[email protected]>
When powering off the modem, appropriate state events
should be triggered.

Signed-off-by: Ryan Erickson <[email protected]>
The new state event allows a user to know the overall state of the
HL7800 driver.

Signed-off-by: Ryan Erickson <[email protected]>
Ensure the HL7800 is awake before starting a firmware update.

Signed-off-by: Ryan Erickson <[email protected]>
Ensure the HL7800 stays awake for the entire firmware update.

Signed-off-by: Ryan Erickson <[email protected]>
Update AT command API to return a response.
Add param to set the timeout when waiting for a response.
If the user does not desire a response, the response buffer can
be NULL.

Signed-off-by: Ryan Erickson <[email protected]>
When sleep modes are used, sockets are restored from NV after boot.
If the network changes IP addresses, restore sockets maintain old network
info in the HL7800 firmware.
On boot, restored sockets should be closed to ensure new sockets
are created with updated network info.

Signed-off-by: Ryan Erickson <[email protected]>
On boot cleanup any restored sockets if they exist.

Signed-off-by: Ryan Erickson <[email protected]>
Minor fixes to ensure GET google.com is successful.

Signed-off-by: Ryan Erickson <[email protected]>
@rerickson1 rerickson1 force-pushed the PR/PROD-17883 branch 2 times, most recently from 3a9d599 to 3afc4c2 Compare January 16, 2025 16:36
Add overlay and small adjustments for using the hl7800 modem.
Works out of the box with pinnacle_100_dvk and mg100.

Signed-off-by: Ryan Erickson <[email protected]>
Add support for the pinnacle_100_dvk and mg100.
With these boards, wait to get on the network before trying
to connect to AWS.

Signed-off-by: Ryan Erickson <[email protected]>
Add release notes for HL7800 driver fixes included in the 4.1 release.

Signed-off-by: Ryan Erickson <[email protected]>
@kartben kartben merged commit 05a5688 into zephyrproject-rtos:main Jan 20, 2025
24 checks passed
@kartben
Copy link
Collaborator

kartben commented Jan 20, 2025

@kartben
Copy link
Collaborator

kartben commented Jan 20, 2025

@rerickson1 can you please look into CI failure in main? zephyrproject-rtos/zephyr/actions/runs/12866274954/job/35868523949

sent a hotfix here: #84243

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: HTTP HTTP client/server support area: LWM2M area: Modem Drivers area: Networking area: Samples Samples area: Sockets Networking sockets bug The issue is a bug, or the PR is fixing a bug priority: medium Medium impact/importance bug Release Notes To be mentioned in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants