[pcf8563] Fix default I2C address from 0xA3 to 0x51#6407
[pcf8563] Fix default I2C address from 0xA3 to 0x51#6407swoboda1337 wants to merge 1 commit intoesphome:currentfrom
Conversation
There was a problem hiding this comment.
Pull request overview
Fixes the documented default I²C address for the PCF8563 RTC in the ESPHome docs, aligning it with the 7-bit address format used by ESPHome’s I2C layer.
Changes:
- Update the
pcf8563time component docs to use0x51as the defaultaddressinstead of0xA3.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
WalkthroughUpdated the PCF8563 real-time clock component documentation to correct the default I²C address from Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Suggested labels
Suggested reviewers
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
✅ Deploy Preview for esphome ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Description
Fix the documented default I2C address for the PCF8563 RTC from
0xA3(8-bit read address from datasheet) to0x51(7-bit address used by ESPHome's I2C layer).Pull request in esphome with YAML changes (if applicable):
Checklist
I am merging into
nextbecause this is new documentation that has a matching pull-request in esphome as linked above.or
I am merging into
currentbecause 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.mdxwhen creating new documents for new components or cookbook.