samples: sensor: Add adxl345 streaming and trigger configurations #80568
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
MaureenHelm
force-pushed
the
adxl345-rtio-profiling
branch
2 times, most recently
from
4c5681a to
ff47bee
Compare
|
This pull request has been marked as stale because it has been open (more than) 60 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this pull request will automatically be closed in 14 days. Note, that you can always re-open a closed pull request at any time. |
MaureenHelm
force-pushed
the
adxl345-rtio-profiling
branch
from
ff47bee to
585384c
Compare
Member
Author
ubieda
previously approved these changes
May 14, 2025
MaureenHelm
force-pushed
the
adxl345-rtio-profiling
branch
from
e27ba37 to
5f29959
Compare
In order to prevent not serving all events that would clear the interrupt line. This patch also removes FIFO servicing through fetch/get APIs, as this is only exposed through streaming mode. Signed-off-by: Luis Ubieda <[email protected]>
Sets the adxl345 driver thread name for easier debugging. Signed-off-by: Maureen Helm <[email protected]>
MaureenHelm
force-pushed
the
adxl345-rtio-profiling
branch
from
5f29959 to
f22933c
Compare
github-actions
bot
added
area: Samples
github-actions
bot
requested review from
asemjonovs,
avisconti,
galak,
jeppenodgaard,
jfischer-no,
kartben,
microbuilder and
nashif
github-actions
bot
requested review from
ozersa,
teburd,
tristan-google,
ttmut,
yasinustunerg and
yperess
kartben
reviewed
May 16, 2025
Comment on lines
98
to
99
| <6 0 &gpio1 4 0>, /* IO5/CS2 */ | ||
| <7 0 &gpio1 5 0>; /* IO6/CS3 */ |
Contributor
There was a problem hiding this comment.
copy paste error? (IO5 / IO6 appear twice)
Specifies which gpio pins are connected to the pmod spi header on the apard32690 board. This will enable using the adxl345 accelerometer interrupt when attaching the pmod_acl shield. Signed-off-by: Maureen Helm <[email protected]>
Fixes the pmod_acl shield board to specify the default accelerometer output data rate with a preprocessor macro that corresponds to an encoded register field value, rather than an integer value in Hertz. Signed-off-by: Maureen Helm <[email protected]>
Specifies which gpio pin on the pmod header is attached to the adxl345 accelerometer interrupt. Signed-off-by: Maureen Helm <[email protected]>
Changes the accelerometer polling sample to include data with the FIFO watermark trigger instead of dropping it. Signed-off-by: Maureen Helm <[email protected]>
Adds adxl345 streaming and trigger configurations to accelerometer samples for profiling the performance impact of adding rtio support to max32 spi and adxl345 accelerometer drivers. Signed-off-by: Maureen Helm <[email protected]>
Configures the accelerometer polling and trigger samples to disable the uart console when the rtt console is enabled. Having both consoles enabled simultaneously doesn't currently work on the apard32690 platform and it's redundant anyway. Signed-off-by: Maureen Helm <[email protected]>
MaureenHelm
force-pushed
the
adxl345-rtio-profiling
branch
from
f22933c to
d83bf9a
Compare
|
kartben
approved these changes
May 16, 2025
ubieda
approved these changes
May 19, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Follow-up to #80115 using adxl345 instead of adxl362
Adds adxl345 streaming and trigger configurations to accelerometer
samples for profiling the performance impact of adding rtio support to
max32 spi and adxl345 accelerometer drivers.
Signed-off-by: Maureen Helm [email protected]
c: @vladislav-pejic @dimitrije-lilic