Skip to content

Merge/template v2.7.1 #17

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

Open
wants to merge 32 commits into
base: main
Choose a base branch
from
Open

Merge/template v2.7.1 #17

wants to merge 32 commits into from

Conversation

szczys
Copy link
Contributor

@szczys szczys commented May 14, 2025

No description provided.

hasheddan and others added 30 commits September 16, 2024 11:22
Fixes formatting to adhere to .clang-format.

Signed-off-by: Daniel Mangum <[email protected]>
Fixes formatting to adhere to .clang-format.

Signed-off-by: Daniel Mangum <[email protected]>
When using LTE, we may start fetching sensor readings before a connection
is established with Golioth. Readings can be shown on a local display
during this period, but attempting to send to Golioth will fail. This
updates the sensor reading streaming, which is just a counter in this case,
to only attempt to stream readings to Golioth when connected. This mirrors
the behavior of the battery monitor.

Signed-off-by: Daniel Mangum <[email protected]>
Adds a log message indicating the streaming data to Golioth was skipped due
to no connection established.

Signed-off-by: Daniel Mangum <[email protected]>
The new version of upload artifact disallows repeat upload to the same
directory. Instead, artifacts should be uploaded using unique names, then
gathered into a single archive using the merge and pattern parameters.

- Split board name into its own step
- Use board name for build, artifact names, and upload archive name

Signed-off-by: Mike Szczys <[email protected]>
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]>
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]>
see CHANGELOG for details

Signed-off-by: Mike Szczys <[email protected]>
Add ecdsa to the pip install command. This package is needed since Golioth
Firmware SDK v0.16.0 (probably part of NCS v2.8.0 but unconfirmed).
Unfortunately the Zephyr requirements.txt does not include this package.

Signed-off-by: Mike Szczys <[email protected]>
see CHANGELOG for details

Signed-off-by: Mike Szczys <[email protected]>
This commit converts README.rst to README.md. No substantive changes have
been made.

Signed-off-by: Mike Szczys <[email protected]>
- add overview of functionality
- add a supported hardware overview
- move Golioth Features above Local set up
- Separate Stream from State
- Add Stream example JSON
- Add LightDB State Example JSON
- Clarify LightDB State functionality
- update OTA instructions for Packages/Cohorts/Deployments
- Move pipelines higher in the document
- Reverence the pipelines section from the Stream section
- Move hardware variants lower in the document
- Simplify the hardware variants section
- rename opening section
- make all headings after the first subheadings
- fix mention of LightDB Stream
- properly style admonitions for GitHub
- add section for additional sensors/components
- remove hardware variations sections
- add mention of use cases and link to projects site
- add section for asking questions

Signed-off-by: Mike Szczys <[email protected]>
Signed-off-by: Mike Szczys <[email protected]>
Although all boards currently supported are using nRF9160, this repository
is fully compatible with the nRF9151 when config/overlays are copied over
to reflect that board name. Support this and other variants by changing the
IFDEF statements to CONFIG_SOC_SERIES_NRF91X as suggested by user issue:
khttps://github.com/golioth/reference-design-template/issues/120

Signed-off-by: Mike Szczys <[email protected]>
This prepares the battery monitor library to support other sources of
voltage measurement.

Signed-off-by: Mike Szczys <[email protected]>
Add support for reading the Elixir battery voltage and percentage using the
MAX17262 fuel gauge.

By default the library will now use the fuel gauge if a devicetree node for
the MAX17262 is enabled. If not, the library will use a voltage divider
that has the devicetree path `vbatt`.

The library should only be used when one of the two above cases is
present for a board.

Signed-off-by: Mike Szczys <[email protected]>
Remove the battery monitor code from this repository into its own
repository. Add battery monitor using west.yml.

Signed-off-by: Mike Szczys <[email protected]>
- Itemize supported service with links back to relevant docs
- Settings: Add link directly to Device Settings menu
- RPC: explicitly reference the device view tab for RPC

Signed-off-by: Mike Szczys <[email protected]>
The Golioth Aludel-Mini board has been replaced by the Golioth
Aludel-Elixir and is no longer available. This commit removes support and
CI testing for the Mini.

Signed-off-by: Mike Szczys <[email protected]>
- silence warning when neither LED nor Ostentus is present
- Use IF_ENABLED instead of #ifdef to wrap LET set/unset call

Signed-off-by: Mike Szczys <[email protected]>
- Move MBEDTLS_SSL_IN_CONTENT_LEN and MBEDTLS_SSL_OUT_CONTENT_LEN to
  nrf9160_ns.conf as it is only used by NRF_SECURITY
- The network info library only supports Nordic devices. Enable it in
  nrf9160_ns.conf and use Kconfig to conditionally build the function calls

Signed-off-by: Mike Szczys <[email protected]>
see CHANGELOG for details

Signed-off-by: Mike Szczys <[email protected]>
Whitespace in project name is now forbidden by west, leading to the
following error:

> UnboundLocalError: cannot access local variable 'context' where it is not associated with a value

Signed-off-by: Mike Szczys <[email protected]>
see CHANGELOG for details

Signed-off-by: Mike Szczys <[email protected]>
@szczys szczys requested review from ChrisGammell and MarkoPura May 14, 2025 15:35
release template_v2.7.1
@szczys szczys force-pushed the merge/template_v2.7.1 branch from 15c8501 to d701a2f Compare May 14, 2025 16:58
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

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.

3 participants