Skip to content

DS18B20 Add onewire_bus UART Backend#756

Merged
suda-morris merged 2 commits intoespressif:masterfrom
Narukara:feat/ds18b20_uart
Apr 8, 2026
Merged

DS18B20 Add onewire_bus UART Backend#756
suda-morris merged 2 commits intoespressif:masterfrom
Narukara:feat/ds18b20_uart

Conversation

@Narukara
Copy link
Copy Markdown
Contributor

@Narukara Narukara commented Apr 7, 2026

ESP-BSP Pull Request checklist

Note: For new BSPs create a PR with this link.

  • Version of modified component bumped
  • CI passing

Change description

This PR updates the ds18b20_read example to support the new UART backend introduced in onewire_bus 1.1.0, while keeping RMT support.

It adds menuconfig options to select the 1-Wire backend (RMT/UART) and updates the example/CI build variants accordingly so backend-specific builds are selected based on SoC capability.

No functional changes were made to the ds18b20 driver itself.


Note

Low Risk
Low risk: changes are limited to the ds18b20_read example, its build variants, and dependency metadata; no driver logic is modified. Main risk is CI/build matrix behavior changing due to new backend- and SoC-capability gating.

Overview
Adds selectable 1-Wire backend support to the ds18b20_read example, allowing either RMT or UART to be chosen via new menuconfig options (main/Kconfig.projbuild) and wiring this into ds18b20_example_main.c.

Updates CI/build-test rules to build backend-specific variants and skip them when the target SoC lacks the required peripheral support, and adds example sdkconfig.bsp.rmt / sdkconfig.bsp.uart presets.

Bumps the ds18b20 component version to 0.3.1 and updates its onewire_bus dependency to ^1.1.0; documentation/changelog are updated accordingly.

Reviewed by Cursor Bugbot for commit 71695c5. Bugbot is set up for automated code reviews on this repo. Configure here.

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Apr 7, 2026

CLA assistant check
All committers have signed the CLA.

Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 92bffef. Configure here.

Comment thread components/.build-test-rules.yml
Comment thread components/ds18b20/examples/ds18b20_read/sdkconfig.bsp.uart
Comment thread components/ds18b20/examples/ds18b20_read/sdkconfig.bsp.rmt
Copy link
Copy Markdown
Collaborator

@suda-morris suda-morris left a comment

Choose a reason for hiding this comment

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

LGTM

@Narukara Narukara force-pushed the feat/ds18b20_uart branch from 92bffef to 71695c5 Compare April 8, 2026 02:32
@cursor
Copy link
Copy Markdown

cursor bot commented Apr 8, 2026

You have used all of your free Bugbot PR reviews.

To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

@suda-morris suda-morris merged commit 0b722d3 into espressif:master Apr 8, 2026
33 of 90 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants