-
Notifications
You must be signed in to change notification settings - Fork 7.7k
tests: drivers: build_all: sensor: add more sensors #91928
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
Conversation
ba1d7fa
to
f943b11
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.
Looks good. WRT adding the AFBR-S50 to the tests, we'd have to add test doubles to the HAL dependencies, as they're implemented by the binary blob.
If that sounds good. Let me know and I can take a stab at it.
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!
Update the dt snippet so that it includes all the required properties Signed-off-by: Benjamin Cabé <[email protected]>
This was missed when the driver was contributed in 8f789f8 Signed-off-by: Benjamin Cabé <[email protected]>
This was missed when the driver was contributed in 2759ada Signed-off-by: Benjamin Cabé <[email protected]>
This was missed when the driver was contributed in 756e699 Signed-off-by: Benjamin Cabé <[email protected]>
c89a4c8
Rebased to fix merge conflicts in the i2c.dtsi file |
|
@MaureenHelm please refresh so that it doesn't go to rebase jail again :) Thanks! |
add previously untested sensors
adi,adxl366
everlight,als-pt19
maxim,ds3231-sensor
I was going to also add
brcm,afbr-s50
but it requires an Arm target ; it might still make sense to add a config for testing such kind of sensors, but this is for another PR I guess. However, I kept the commit that is fixing the yaml file for this sensor's binding so that the snippet in the documentation is complete (cc @ubieda @bperseghetti )