drivers: Lsm9ds1 mag driver #73530
drivers: Lsm9ds1 mag driver #73530
Conversation
zephyrbot
added
area: Devicetree Binding
zephyrbot
requested review from
MaureenHelm,
decsny,
galak,
kartben,
nashif,
teburd,
tristan-google,
ubieda and
yperess
|
@miggazElquez just curious, why filing two separate PRs for the same driver? |
miggazElquez
force-pushed
the
lsm9ds1_mag_driver
branch
from
04de46f to
1e7ed4d
Compare
It was asked by @jeppenodgaard, so I did as asked. And the two driver are independent, even if it's the same sensor, from a logical point of view they are totally different (different adress on the bus, different registers...) so it kind of make sense. |
miggazElquez
force-pushed
the
lsm9ds1_mag_driver
branch
3 times, most recently
from
b024f13 to
0a939d7
Compare
miggazElquez
force-pushed
the
lsm9ds1_mag_driver
branch
3 times, most recently
from
6ba3f3d to
9378ce9
Compare
miggazElquez
force-pushed
the
lsm9ds1_mag_driver
branch
from
9378ce9 to
cee9ae6
Compare
MaureenHelm
left a comment
MaureenHelm
left a comment
There was a problem hiding this comment.
LGTM, but needs to be rebased.
@avisconti please take a look
miggazElquez
force-pushed
the
lsm9ds1_mag_driver
branch
from
cee9ae6 to
d3f310d
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. |
miggazElquez
force-pushed
the
lsm9ds1_mag_driver
branch
from
d3f310d to
ffd3355
Compare
|
I had forgotten this PR, I rebased on main and modified the code to use |
miggazElquez
force-pushed
the
lsm9ds1_mag_driver
branch
from
ffd3355 to
21e4c39
Compare
miggazElquez
force-pushed
the
lsm9ds1_mag_driver
branch
from
21e4c39 to
0e6f589
Compare
|
Rebased on main to fix merge conflict |
This commit adds a description for the magnetometer part of the lsm9ds1 sensor. Signed-off-by: Miguel Gazquez <[email protected]>
This driver supports the magnetometer in the ST lsm9ds1 sensor, on the I2C bus. This driver does not support the triggers. This driver use the stmemsc HAL. link: https://www.st.com/resource/en/datasheet/lsm9ds1.pdf Signed-off-by: Miguel Gazquez <[email protected]>
…e tree Describe the magnetometer part of the lsm9ds1 sensor, available in the Arduino Nano 33 BLE through I2C. Signed-off-by: Miguel Gazquez <[email protected]>
miggazElquez
force-pushed
the
lsm9ds1_mag_driver
branch
from
0e6f589 to
11af0e4
Compare
|
|
@MaureenHelm @avisconti Could you please take a look ? |
8 participants
This PR add a driver for the magnetometer of the lsm9ds1 sensor (https://www.st.com/en/mems-and-sensors/lsm9ds1.html).
The four first commits (3a3b731f503d719a63df5cb28ce2c655be853d5f..e64fe2cbea0d985e8064de17b67bcc23301b6a2b) are part of #73141.