Skip to content

Conversation

@pabigot
Copy link
Contributor

@pabigot pabigot commented Dec 16, 2019

Define a binding for the Aosong DHT family of temperature/humidity sensors. Remove the Kconfig settings, and update the driver to use devicetree information.

Add a sample.

@pabigot pabigot added the area: Sensors Sensors label Dec 16, 2019
@pabigot pabigot requested a review from galak December 16, 2019 16:24
@pabigot pabigot requested a review from nashif as a code owner December 16, 2019 17:15
@zephyrbot zephyrbot added the area: Tests Issues related to a particular existing or missing test label Dec 16, 2019
@pabigot pabigot force-pushed the pabigot/20191216b branch 2 times, most recently from 0e5d5e7 to 78a847e Compare December 16, 2019 18:58
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This would read better:

After providing a devicetree overlay that specifies the sensor location, build this sample app using:

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated.

Copy link
Contributor

@dbkinder dbkinder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

docs LGTM, thanks!

Copy link
Member

@MaureenHelm MaureenHelm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

one nit, but otherwise looks great. thanks @pabigot

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this should be const

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done; thanks.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you amended the wrong commit. the change should be in sensor: dht: convert from Kconfig to devicetree

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Moved to the right commit.

Define a binding for the Aosong DHT family of temperature/humidity
sensors.  Remove the Kconfig settings, and update the driver to use
devicetree information.

Signed-off-by: Peter Bigot <[email protected]>
Add a sample to test the Aosong temperature/humidity sensor.

Signed-off-by: Peter Bigot <[email protected]>
@MaureenHelm MaureenHelm merged commit 816d2c4 into zephyrproject-rtos:master Dec 19, 2019
@pabigot pabigot deleted the pabigot/20191216b branch December 19, 2019 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: Devicetree area: Samples Samples area: Sensors Sensors area: Tests Issues related to a particular existing or missing test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants