-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
[mcp3204] diff mode update #5623
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
Conversation
Added documentation for diff mode analog readings.
✅ Deploy Preview for esphome ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
swoboda1337
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This needs to use the next branch
Developer docs are here: https://developers.esphome.io/contributing/docs/
|
Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍 |
|
I fixed the branch but none of the files are changed |
|
Thanks. My first PR here and its quite confusing. Do I need to do anything
else for this diff mode?
…On Fri, Nov 14, 2025, 16:35 Jonathan Swoboda ***@***.***> wrote:
*swoboda1337* left a comment (esphome/esphome-docs#5623)
<#5623 (comment)>
I fixed the branch
—
Reply to this email directly, view it on GitHub
<#5623 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAJE6M225DU4DH77FFGB6T34XZCJAVCNFSM6AAAAACMCZ6N66VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTKMZTGMZDKNJUG4>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
|
I also need to figure out, when in diff mode, how to report an error if pin
= 2. Well, I can promise this in another two to three years, you can count
on it :). Thanks again.
…On Fri, Nov 14, 2025, 16:59 Jonathan Swoboda ***@***.***> wrote:
*swoboda1337* left a comment (esphome/esphome-docs#5623)
<#5623 (comment)>
Thanks. My first PR here and its quite confusing. Do I need to do anything
else for this diff mode?
Well if you look at the changed files there are none? So yeah you have to
add the docs still :)
image.png (view on web)
<https://github.com/user-attachments/assets/4cbde77e-8c66-4433-af55-2a5122d133cf>
—
Reply to this email directly, view it on GitHub
<#5623 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAJE6O4RVYEPJESEIFCC3T34X36DAVCNFSM6AAAAACMCZ6N66VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTKMZTGQZTIOJUHE>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
swoboda1337
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are no changes to review
|
Moved to PR #5634 |

Added documentation for diff mode analog readings on a MCP3204/MCP3208.
Description:
This PR adds documentation for the new differential mode feature for the MCP3204/MCP3208 ADC components. Differential mode allows the chip to measure the voltage difference between two analog input pins, rather than measuring a single pin's voltage relative to ground. This is useful for applications requiring precise measurements of small voltage differences, noise rejection in sensor readings.
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
/components/index.rstwhen creating new documents for new components or cookbook.