-
Notifications
You must be signed in to change notification settings - Fork 7k
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: sensors: Add driver for LM95234 temperature sensor #71169
drivers: sensors: Add driver for LM95234 temperature sensor #71169
Conversation
d0276fc
to
f2bd91b
Compare
f2bd91b
to
1d01ebc
Compare
d4c2196
to
63ccabd
Compare
Rebased to main to fix a conflict. @jeppenodgaard I think your comments are all addressed? |
Yes. LGTM. There is no resolve buttons for some reason. |
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.
A couple nits, but otherwise LGTM
63ccabd
to
b0e3dd6
Compare
Updated as suggested. |
Ping - any additional comments? |
Add a driver for the National/TI LM95234 Quad Remote Diode and Local Temperature Sensor with SMBus Interface and TruTherm Technology. Signed-off-by: Robert Hancock <robert.hancock@calian.com>
b0e3dd6
to
c8e46cc
Compare
Fixed conflicts in the tests/drivers/build_all/sensor/i2c.dtsi file, no other changes from last rev. |
Add a driver for the National/TI LM95234 Quad Remote Diode and Local Temperature Sensor with SMBus Interface and TruTherm Technology.