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.

Sign up for GitHub

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

Jump to bottom

Fixes/timer and sensors api fixes #47

Merged
merged 2 commits into from
Feb 27, 2025
Merged

Fixes/timer and sensors api fixes #47

merged 2 commits into from
Feb 27, 2025

Conversation

srberard
Copy link
Collaborator

Description

Multiple fixes to the timer and sensors API. This addresses issues with threading and multiple exec environments.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Code cleanup/refactoring
  • This change requires a documentation update
  • CI system update
  • Test Coverage update

How Has This Been Tested?

Manually tested on native_sim

Test Configuration (required):

  • Host OS type, version, and arch: Ubuntu 20.04 (arm64)
  • Developer Board make & model: native_sim

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

srberard and others added 2 commits February 27, 2025 13:54
@srberard
Bug fix/timer api (#23)
db8fa4c 
* fix for callback
* get remaining fix and ocre sleep implement
* feature/Make MAX_TIMERS,  MAX_SENSORS,  MAX_CHANNELS_PER_SENSOR configurable values
* sensors_API_refactor
* sensors_update
* sensor_api_refactor
* thread for each container implementation
* fix for container ID in thread create
* fix for container update
* clean up mechanism for timers

---------
Co-authored-by: ArturSir <artur@atym.io>
Signed-off-by: Stephen Berard <stephen.berard@outlook.com>
@srberard
Added devices tree to overlay files (sensors support)
5757cda 
Signed-off-by: Stephen Berard <stephen.berard@outlook.com>
@srberard srberard force-pushed the fixes/timer-and-sensors-API-fixes branch from a84b59c to 5757cda Compare February 27, 2025 12:54
@srberard srberard merged commit 3ddeabd into main Feb 27, 2025
3 checks passed
@srberard srberard deleted the fixes/timer-and-sensors-API-fixes branch February 27, 2025 13:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gonzolively gonzolively Awaiting requested review from gonzolively gonzolively is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@srberard