-
-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
lis2dw12: Add support for lis2dw12 accelerometer #6312
Conversation
f9dd113
to
eb53991
Compare
This |
Thanks. In general it seems fine to me. I have a few comments:
Also, @dmbutyugin do you have any comments on this PR? -Kevin |
I think |
fbe1665
to
7a3579b
Compare
|
Thanks. It looks fine to me. I'll give a couple of days to see if there are additional comments - otherwise I'll look to commit. Cheers, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks. I had just a few comments. I didn't test the accelerometer though, so I cannot comment whether and how it works either in the current state or with the suggested mode change. Besides that, I don't have any major feedback for the support of this accelerometer.
6d7e210
to
52ff364
Compare
lis2dw12 is an accelerometer from STMicroelectronics(https://www.st.com/resource/en/datasheet/lis2dw12.pdf) With better performance than the ADXL345 according to the datasheet. Signed-off-by: XM.Zhou from BigTreeTech [email protected] Signed-off-by: Alan.Ma from BigTreeTech [email protected]
Signed-off-by: Alan.Ma from BigTreeTech [email protected]
Signed-off-by: Alan.Ma from BigTreeTech [email protected]
Thanks. -Kevin |
lis2dw12 is an accelerometer from STMicroelectronics
With better performance than the ADXL345 according to the datasheet.