drivers: sensor: adiltc2990: add adi ltc2990 support #60842
Merged
Conversation
This file contains 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
zephyrbot
added
area: Devicetree Binding
jilaypandya
force-pushed
the
feature/add-analogdevices-ltc2990-support
branch
2 times, most recently
from
68c4887
to
0c8db7b
Compare
jilaypandya
changed the title
jilaypandya
force-pushed
the
feature/add-analogdevices-ltc2990-support
branch
5 times, most recently
from
9292ccc
to
203f730
Compare
MaureenHelm
requested changes
Jul 31, 2023
There was a problem hiding this comment.
Thank you for your contribution!
Please combine commits drivers: sensor: adds adltc2990 to CMakeLists.txt and Kconfig and driver: sensor: adds basic support for analog devices ltc2990. You also need to add this sensor to tests/drivers/build_all/sensor/i2c.dtsi to build the driver in CI.
jilaypandya
force-pushed
the
feature/add-analogdevices-ltc2990-support
branch
3 times, most recently
from
42b82bd
to
461e5be
Compare
jilaypandya
force-pushed
the
feature/add-analogdevices-ltc2990-support
branch
from
461e5be
to
67442c2
Compare
jilaypandya
force-pushed
the
feature/add-analogdevices-ltc2990-support
branch
from
67442c2
to
bb1991e
Compare
MaureenHelm
requested changes
Aug 18, 2023
jilaypandya
force-pushed
the
feature/add-analogdevices-ltc2990-support
branch
from
bb1991e
to
0310557
Compare
jilaypandya
force-pushed
the
feature/add-analogdevices-ltc2990-support
branch
from
0310557
to
e65ebda
Compare
jilaypandya
force-pushed
the
feature/add-analogdevices-ltc2990-support
branch
3 times, most recently
from
2dc66e9
to
c27b8f0
Compare
jilaypandya
force-pushed
the
feature/add-analogdevices-ltc2990-support
branch
from
c27b8f0
to
28c92e1
Compare
yperess
requested changes
Sep 8, 2023
jilaypandya
force-pushed
the
feature/add-analogdevices-ltc2990-support
branch
3 times, most recently
from
3d7f290
to
89bb77e
Compare
jilaypandya
force-pushed
the
feature/add-analogdevices-ltc2990-support
branch
from
89bb77e
to
11def69
Compare
|
@MaureenHelm @yperess, I have incorporated the requested changes, hope the PR is fine now. Would be great if could get a further review or approval:-) |
jilaypandya
force-pushed
the
feature/add-analogdevices-ltc2990-support
branch
2 times, most recently
from
ed9a3e6
to
8702626
Compare
MaureenHelm
requested changes
Sep 12, 2023
jilaypandya
force-pushed
the
feature/add-analogdevices-ltc2990-support
branch
from
8702626
to
9874e83
Compare
This commit adds adi,adltc2990.yaml to dts/bindings/sensor. Signed-off-by: Jilay Pandya <jilay.pandya@zeiss.com>
jilaypandya
force-pushed
the
feature/add-analogdevices-ltc2990-support
branch
from
9874e83
to
669c253
Compare
This commit adds Kconfig variables to configure ADLTC2990. This commit adds basic driver code for analog devices ltc2990 sensor. Signed-off-by: Jilay Pandya <jilay.pandya@zeiss.com>
jilaypandya
force-pushed
the
feature/add-analogdevices-ltc2990-support
branch
from
669c253
to
c788457
Compare
jilaypandya
commented
Sep 14, 2023
MaureenHelm
approved these changes
Sep 14, 2023
yperess
approved these changes
Sep 18, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This PR adds basic support of analog devices LTC2990 Quad I2C Voltage, Current and Temperature Monitor.
This sensor was tested with MIMXRT1062xxx6A for monitoring 4 single ended voltages with following output.