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

Merge template v2.4.1 #10

Merged
merged 198 commits into from
Nov 4, 2024
Merged

Merge template v2.4.1 #10

merged 198 commits into from
Nov 4, 2024

Conversation

szczys
Copy link
Collaborator

@szczys szczys commented Oct 31, 2024

Merge Reference Design Template at template_v2.4.1

resolves https://github.com/golioth/devrel-issue-tracker/issues/430

szczys and others added 30 commits November 14, 2022 10:33
Run the hello app by after setting credentials in the shell.

Signed-off-by: Mike Szczys <[email protected]>
Signed-off-by: Mike Szczys <[email protected]>
Defining `buiid/setings/board_root` makes Zephyr build system aware of
downstream boards defined in `boards/` directory.
Those were copy-pasted from another board, so fix it now.

Signed-off-by: Marcin Niestroj <[email protected]>
Just make sure that this board definition follows other nRF9160 boards
default settings.

Signed-off-by: Marcin Niestroj <[email protected]>
Credentials are set via the serial terminal so there is no need for a
separate credentials file.

Signed-off-by: Mike Szczys <[email protected]>
This commit updates the manifest file to use a separate repository for
Zephyr board definitions of custom Golioth boards. The repo will clone
into app/golioth-boards and the project is now configured to use that
directory by default for the BOARD_ROOT.

Signed-off-by: Mike Szczys <[email protected]>
* Pull Golioth up to tip of main to incorporate latest OTA fixes
* Add DFU functionality

Signed-off-by: Mike Szczys <[email protected]>
Signed-off-by: Mike Szczys <[email protected]>
Move all DFU-related code to app_dfu.c

Signed-off-by: Mike Szczys <[email protected]>
* Fix headings
* Add instrcutions about LOOP_DELAY_S setting

Signed-off-by: Mike Szczys <[email protected]>
Signed-off-by: Mike Szczys <[email protected]>
Settings service is used to adjust the LOOP_DELAY_S which is used for
the main loop sleep. When a new setting is received, the main loop
wakes, immediately taking a reading, then uses new value for sleep. The
user button will now also wake the main loop.

Signed-off-by: Mike Szczys <[email protected]>
Use the app_work.c file to run all of your specialized functions and
keep main and other template files unchanged.

Signed-off-by: Mike Szczys <[email protected]>
* Only perform update if received value is different from local value
* Increase LOOP_DELAY_S to 12 hours (was 100 seconds)

Signed-off-by: Mike Szczys <[email protected]>
DFU uses multiple c/h files and will be easier to understand if they are
in their own folder. Rename app_dfu to golioth_dfu for clarity.

Signed-off-by: Mike Szczys <[email protected]>
* Move settings to its own file
* Update settings changed-value check to be more readable

Signed-off-by: Mike Szczys <[email protected]>
Registering a settings callback is now handles in app_settings.c by
calling app_register_settings(client); from on_connect in main.

Signed-off-by: Mike Szczys <[email protected]>
Use `template` or `rd_template` as a generic name for the template
project.

Signed-off-by: Mike Szczys <[email protected]>
Signed-off-by: Mike Szczys <[email protected]>
The duplicate observations workaround was previously committed without
an incrementing iteration count which caused obersavtions to never be
registered.

Signed-off-by: Mike Szczys <[email protected]>
This commit adds device state to the template. It will observe a
`desired` endpoint and report to a `status` endpoint. (Both endpoints
are configurable in app_state.h).

The approach uses the JSON library for parsing, and tests two values
`example_int0` and `example_int1` for bounding [0..9999].

Signed-off-by: Mike Szczys <[email protected]>
szczys and others added 22 commits May 31, 2024 12:02
See CHANGELOG for details

Signed-off-by: Mike Szczys <[email protected]>
Signed-off-by: Chris Wilson <[email protected]>
The default Golioth Pipeline for a new project applies to CBOR data and not
JSON data. This updates the data type to use CBOR.

NOTE: By default the pipeline will not include the path with the data. The
RD Template sends to the "sensor" path, but LightDB data will not be shown
under that path unless an inject transform is added.

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

Signed-off-by: Mike Szczys <[email protected]>
Specify application version using VERSION file in application root
directory. This is a standard mechanism provided by Zephyr.

CONFIG_MCUBOOT_IMGTOOL_SIGN_VERSION is automatically set to proper version
according to VERSION file contents.

Signed-off-by: Marcin Niestroj <[email protected]>
Add the default CBOR to LightDB Stream pipeline and directions on how to
use it.

Signed-off-by: Mike Szczys <[email protected]>
libostentus is now a Zephyr driver. Update the module version and change
all API calls to match the driver syntax. Add reset and firmware version
read when app first runs.

Signed-off-by: Mike Szczys <[email protected]>
Signed-off-by: Mike Szczys <[email protected]>
Update to a set of board definitions that use Hardware Model v2 for Zephyr
v3.7.0. Update board overlay files accordingly.

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

Signed-off-by: Mike Szczys <[email protected]>
We need to substitute forward-slash for underscore when naming artifacts
for the new Zephyr board names. The regex approach taken fails when using
sh but will pass when the workflow is instructed to use bash.

Signed-off-by: Mike Szczys <[email protected]>
We use the `template_v#.#.#` to differentiate template tags from tags on
reference designs that fork this repository.

Add the `template_` prefix to the default release number presented when
running the GitHub action to generate a new draft release.

Signed-off-by: Mike Szczys <[email protected]>
Use --sysbuild for all board as recommended. This solves the deprecation
warnings when building for the nRF91 without sysbuild.

Bump the golioth-zephyr-boards module version to incorporate changes
necessary to use sysbuild with aludel_elixir.

Signed-off-by: Mike Szczys <[email protected]>
Nordic recommends that multi-artifact builds (like mcuboot + app in this
repository) use a static partition table. The resolves the build warning:

---------------------------------------------------------------------
--- WARNING: Using a bootloader without pm_static.yml.            ---
--- There are cases where a deployed product can consist of       ---
--- multiple images, and only a subset of these images can be     ---
--- upgraded through a firmware update mechanism. In such cases,  ---
--- the upgradable images must have partitions that are static    ---
--- and are matching the partition map used by the bootloader     ---
--- programmed onto the device.                                   ---
---------------------------------------------------------------------

Signed-off-by: Mike Szczys <[email protected]>
We were previously testing for only the aludel_mini but building for all
boards on every PR will help catch digressions sooner.

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

Signed-off-by: Mike Szczys <[email protected]>
@szczys szczys force-pushed the merge/template_v2.4.1 branch from a11be82 to ac075e4 Compare November 1, 2024 20:48
@szczys szczys marked this pull request as ready for review November 1, 2024 20:59
During merge commit these sensors were accidentally placed on the i2c bus.

Signed-off-by: Mike Szczys <[email protected]>
Copy link
Collaborator

@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.

Awesome work! LGTM!

@szczys szczys merged commit ad68f13 into main Nov 4, 2024
3 checks passed
@szczys szczys deleted the merge/template_v2.4.1 branch November 4, 2024 19:48
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.

6 participants