drivers/lis331dlh: add accelerometer driver#22399
Open
MastenaM wants to merge 6 commits into
Open
Conversation
MastenaM
requested review from
MichelRottleuthner,
aabadie and
leandrolanzieri
as code owners
|
Hey @MastenaM, thank you for your first contribution! We really appreciate it! If you haven't already, please take a look at our contributing guidelines before the review process starts. Also, due to how the GitHub review system works, please avoid force-pushing or squashing your commits unless asked to by a maintainer (or unless your commit is still in "draft commit" stage). Lastly, make sure to comply with our AI Policy when using AI. Your pull request will be reviewed as soon as possible. |
github-actions
Bot
added
Area: tests
maribu
changed the title
maribu
reviewed
Jun 22, 2026
maribu
left a comment
maribu
left a comment
Member
There was a problem hiding this comment.
I'm very impressed with the quality of the PR :) Thanks and welcome to the team of RIOT contributors!
Co-authored-by: Marian Buschsieweke <maribu@users.noreply.github.com>
Contributor
AnnsAnns
added
AI: Co-Authored
crasbe
added
the
State: waiting for author
Contributor
|
Any updates on this matter? |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 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.
Contribution description
This PR adds support for the ST LIS331DLH 3-axis accelerometer.
The contribution includes:
The driver provides initialization and raw XYZ acceleration data reading over I2C.
Testing procedure
The driver was tested by building the dedicated RIOT driver test application:
shell make -C tests/drivers/lis331dlh clean make -C tests/drivers/lis331dlh
Tested board:
text nucleo-f401re
The application builds successfully without warnings or errors and produces:
text driver_lis331dlh_test.elf
Additionally:
shell git diff --check
was used to verify that no whitespace issues are present.
Issues/PRs references
None.
Declaration of AI-Tools / LLMs usage:
AI-Tools / LLMs that were used are: