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
drivers: sensor: bmm150: Add trigger support for bmm150 magnetometer … #57985
Conversation
2b06420
to
bf3bdb4
Compare
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.
Please add this sensor to tests/drivers/build_all/sensor/sensors_trigger_*.conf
fbe0bc6
to
fa4c3e3
Compare
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.
Please address the failed twister check:
In file included from /__w/zephyr/zephyr/drivers/sensor/bmm150/bmm150_trigger.c:15:
/__w/zephyr/zephyr/drivers/sensor/bmm150/bmm150.h:175:22: error: duplicate member ‘sem’
175 | struct k_sem sem;
| ^~~
/__w/zephyr/zephyr/drivers/sensor/bmm150/bmm150_trigger.c: In function ‘bmm150_handle_interrupts’:
/__w/zephyr/zephyr/drivers/sensor/bmm150/bmm150_trigger.c:24:19: error: ‘struct bmm150_data’ has no member named ‘drdy_handler’; did you mean ‘drdy_handle’?
24 | if (data->drdy_handler) {
| ^~~~~~~~~~~~
| drdy_handle
/__w/zephyr/zephyr/drivers/sensor/bmm150/bmm150_trigger.c:25:23: error: ‘struct bmm150_data’ has no member named ‘drdy_handler’; did you mean ‘drdy_handle’?
25 | data->drdy_handler(dev, data->drdy_trigger);
| ^~~~~~~~~~~~
| drdy_handle
/__w/zephyr/zephyr/drivers/sensor/bmm150/bmm150_trigger.c: In function ‘bmm150_trigger_set’:
/__w/zephyr/zephyr/drivers/sensor/bmm150/bmm150_trigger.c:102:15: error: ‘struct bmm150_data’ has no member named ‘drdy_handler’; did you mean ‘drdy_handle’?
102 | data->drdy_handler = handler;
| ^~~~~~~~~~~~
| drdy_handle
fa4c3e3
to
164bdd3
Compare
77c10b9
to
57d64bd
Compare
@MaureenHelm Do you still have any issues with this patch ? |
57d64bd
to
618c677
Compare
@Cherish-Gww please rebase |
…sensor Add bmm150 magetometer sensor data ready trigger support. Signed-off-by: Weiwei Guo <guoweiwei@syriusrobotics.com>
618c677
to
133b535
Compare
…sensor