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

update Golioth Zephyr SDK to v0.16.0 #111

Merged
merged 2 commits into from
Nov 26, 2024
Merged

Conversation

szczys
Copy link
Collaborator

@szczys szczys commented Nov 25, 2024

This is based on NCS v2.8.0

Update async callbacks:

  • Remove golioth_response param
  • Add golioth_status and golioth_coap_rsp_code params

@szczys szczys force-pushed the szczys/bump-golioth-0.16.0 branch from 78c7272 to 63af32b Compare November 25, 2024 22:57
@szczys szczys marked this pull request as draft November 25, 2024 23:00
This is based on NCS v2.8.0

- Update async callbacks:
  - Remove golioth_response param
  - Add golioth_status and golioth_coap_rsp_code params
- Replace deprecated CONFIG_EVENTFD_MAX with CONFIG_ZVFS_OPEN_MAX
- Replace deprecated CONFIG_POSIX_MAX_FDS with CONFIG_ZVFS_OPEN_MAX
- Use CONFIG_MBEDTLS_LEGACY_CRYPTO_C to enable support for p-384 curves

Signed-off-by: Mike Szczys <[email protected]>
@szczys szczys force-pushed the szczys/bump-golioth-0.16.0 branch from 63af32b to 950ff93 Compare November 25, 2024 23:22
Prevents a situation where the device blocks all other processes while
trying to constantly read from the sensors.

Signed-off-by: Mike Szczys <[email protected]>
@szczys szczys marked this pull request as ready for review November 26, 2024 16:01
Copy link
Contributor

@MarkoPura MarkoPura left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@szczys szczys merged commit 5ef19e9 into main Nov 26, 2024
3 checks passed
@szczys szczys deleted the szczys/bump-golioth-0.16.0 branch November 26, 2024 19:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants