Skip to content
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 support for ds18s20 #71286

Merged
merged 1 commit into from
May 13, 2024

Conversation

fgihl
Copy link
Contributor

@fgihl fgihl commented Apr 9, 2024

Added support for the older ds18s20 inside the (newer) ds18b20.

dts/bindings/sensor/maxim,ds18b20.yaml Outdated Show resolved Hide resolved
dts/bindings/sensor/maxim,ds18b20.yaml Outdated Show resolved Hide resolved
@str4t0m str4t0m self-requested a review April 10, 2024 08:47
@fgihl fgihl force-pushed the fgihl/ds18s20 branch 2 times, most recently from 5fdd5ab to 7857345 Compare April 10, 2024 16:12
@zephyrbot zephyrbot added the area: W1 1-Wire label Apr 21, 2024
@fgihl fgihl force-pushed the fgihl/ds18s20 branch 3 times, most recently from 7753494 to cca18c7 Compare April 22, 2024 19:52
@fgihl fgihl requested a review from decsny April 25, 2024 20:22
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.

Please add a node with the new compatible to tests/drivers/build_all/sensor/w1.dtsi

drivers/sensor/maxim/ds18b20/ds18b20.c Outdated Show resolved Hide resolved
drivers/sensor/maxim/ds18b20/ds18b20.c Show resolved Hide resolved
@zephyrbot zephyrbot added the Release Notes To be mentioned in the release notes label Apr 27, 2024
dts/bindings/sensor/maxim,ds18s20.yaml Outdated Show resolved Hide resolved
drivers/sensor/maxim/ds18b20/ds18b20.c Outdated Show resolved Hide resolved
drivers/sensor/maxim/ds18b20/ds18b20.c Outdated Show resolved Hide resolved
drivers/sensor/maxim/ds18b20/ds18b20.c Outdated Show resolved Hide resolved
tests/drivers/build_all/sensor/w1.dtsi Outdated Show resolved Hide resolved
@fgihl fgihl force-pushed the fgihl/ds18s20 branch 2 times, most recently from 17ae304 to 927c526 Compare May 3, 2024 20:03
@fgihl
Copy link
Contributor Author

fgihl commented May 3, 2024

rebased to solve conflict in release note file

@fgihl fgihl force-pushed the fgihl/ds18s20 branch 2 times, most recently from 81e8cb2 to 8733004 Compare May 3, 2024 20:31
Added support for the older ds18s20 inside the (newer) ds18b20.

Signed-off-by: Fredrik Gihl <fgihl@hotmail.com>
@nashif nashif merged commit 2d31d45 into zephyrproject-rtos:main May 13, 2024
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Devicetree Binding PR modifies or adds a Device Tree binding area: Sensors Sensors area: W1 1-Wire platform: ADI Analog Devices, Inc. Release Notes To be mentioned in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants