Skip to content

[emontx] Update UART RX/TX pins to GPIO20/GPIO21 for emonWifi#6567

Merged
jesserockz merged 4 commits into
esphome:currentfrom
FredM67:emontx-gpio-emonwifi
Jun 8, 2026
Merged

[emontx] Update UART RX/TX pins to GPIO20/GPIO21 for emonWifi#6567
jesserockz merged 4 commits into
esphome:currentfrom
FredM67:emontx-gpio-emonwifi

Conversation

@FredM67

@FredM67 FredM67 commented May 3, 2026

Copy link
Copy Markdown
Contributor

Description

Update the UART pin assignments in all sample YAML configurations from GPIO16 (RX) / GPIO17 (TX) to GPIO20 (RX) / GPIO21 (TX), which are the pins used on the emonWifi expansion board.

Related issue (if applicable): N/A

Pull request in esphome with YAML changes (if applicable):

  • N/A

Checklist

  • I am merging into next because this is new documentation that has a matching pull-request in esphome as linked above.
    or

  • I am merging into current because this is a fix, change and/or adjustment in the current documentation and is not for a new component or feature.

  • Link added in /src/content/docs/components/index.mdx when creating new documents for new components or cookbook.

@esphome esphome Bot added the current label May 3, 2026
@coderabbitai

coderabbitai Bot commented May 3, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 8de2e520-ead3-4438-9ad7-00742b7cde26

📥 Commits

Reviewing files that changed from the base of the PR and between 554a0d8 and 9d400f5.

📒 Files selected for processing (1)
  • src/content/docs/components/sensor/emontx.mdx

Walkthrough

Documentation updates to the EmonTx component hub guide: two UART configuration examples now specify GPIO20 and GPIO21 instead of GPIO16 and GPIO17 for UART RX and TX pins respectively.

Changes

Documentation: EmonTx UART GPIO Pin Corrections

Layer / File(s) Summary
Documentation Updates
src/content/docs/components/sensor/emontx.mdx
Component/Hub UART example (lines 43–44) and Quick Start example (lines 95–96) updated with corrected GPIO pins: rx_pin: GPIO20 and tx_pin: GPIO21 replace GPIO16 and GPIO17.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately reflects the main change: updating UART pins from GPIO16/GPIO17 to GPIO20/GPIO21 for the emonWifi expansion board in the emontx documentation.
Description check ✅ Passed The description is directly related to the changeset, providing clear context about the UART pin updates and specifying the hardware context (emonWifi expansion board).
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

ESLint skipped: no ESLint configuration detected in root package.json. To enable, add eslint to devDependencies.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@netlify

netlify Bot commented May 3, 2026

Copy link
Copy Markdown

Deploy Preview for esphome ready!

Name Link
🔨 Latest commit 9d400f5
🔍 Latest deploy log https://app.netlify.com/projects/esphome/deploys/6a100a371e04de0009619dc7
😎 Deploy Preview https://deploy-preview-6567--esphome.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

Comment thread src/content/docs/components/sensor/emontx.mdx Outdated
Comment thread src/content/docs/components/sensor/emontx.mdx Outdated
@esphome esphome Bot marked this pull request as draft May 21, 2026 21:08
@esphome

esphome Bot commented May 21, 2026

Copy link
Copy Markdown

Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍

Learn more about our pull request process.

@FredM67 FredM67 marked this pull request as ready for review May 22, 2026 07:51
@esphome esphome Bot requested a review from jesserockz May 22, 2026 07:51
@jesserockz jesserockz merged commit abc8e3d into esphome:current Jun 8, 2026
6 checks passed
bughaver pushed a commit to bughaver/esphome.io that referenced this pull request Jun 8, 2026
…e#6567)

Co-authored-by: Frédéric Metrich <frederic.metrich@siemens.com>
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 10, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants