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

[FR] Onboard/Motherboard-based Thermistor Support #20175

Closed
thisiskeithb opened this issue Nov 17, 2020 · 9 comments
Closed

[FR] Onboard/Motherboard-based Thermistor Support #20175

thisiskeithb opened this issue Nov 17, 2020 · 9 comments
Labels

Comments

@thisiskeithb
Copy link
Member

thisiskeithb commented Nov 17, 2020

Description

Boards like the Einsy Rambo, BTT002, SKR E3 Turbo, and motherboard used in the Eryone ER-20 have an onboard 100K thermistor, but motherboard/onboard thermistors are not natively supported in Marlin.

There's a "hacky" way to get readings where you define the onboard thermistor as a TEMP_CHAMBER_PIN which gives you feedback on the LCD & associated thermal protections, but that's not ideal.

Feature Workflow

  1. Support onboard thermistor with something like TEMP_MOTHERBOARD_PIN
  2. Show temperature over serial and optionally on the LCD
  3. Enjoy thermal protections of motherboard-based thermistors!
@thisiskeithb thisiskeithb added the T: Feature Request Features requested by users. label Nov 17, 2020
@3DFans
Copy link

3DFans commented Nov 17, 2020

thx @thisiskeithb

@ETE-Design
Copy link
Contributor

Would also be great if a 2'nd bed could be added as an option... I have 2 Zone's on my bed and right now I use Chamber to show 2'nd Zone Temp... But then it also need a 2'nd Heater Pin defined to make it work...

@slowbro
Copy link
Member

slowbro commented Jun 28, 2021

Would also be great if a 2'nd bed could be added as an option... I have 2 Zone's on my bed and right now I use Chamber to show 2'nd Zone Temp... But then it also need a 2'nd Heater Pin defined to make it work...

The "Redundant Temp Sensor" option I made would probably do this for you.

@ETE-Design
Copy link
Contributor

@slowbro In this PR? Would also be great with alternative bitmap for it ;-)

@thisiskeithb
Copy link
Member Author

Would also be great if a 2'nd bed could be added as an option... I have 2 Zone's on my bed and right now I use Chamber to show 2'nd Zone Temp... But then it also need a 2'nd Heater Pin defined to make it work...

This should be a different Feature Request.

@slowbro
Copy link
Member

slowbro commented Jun 28, 2021

@slowbro In this PR? Would also be great with alternative bitmap for it ;-)

Ah, I misinterpreted your first message, I thought you were just trying to ensure the zones were the same temp. Agreed with @thisiskeithb, you should open a separate feature request for that :)

@slowbro
Copy link
Member

slowbro commented Jun 28, 2021

Since I've been recently spending a lot of time in the temperature module, I'll take this and add a new TEMP_SENSOR_BOARD option.

@thisiskeithb
Copy link
Member Author

Closing since the feature request was complete and I'll open a separate ticket if we can't figure out why it's causing lockups.

@github-actions
Copy link

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked and limited conversation to collaborators Sep 10, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

4 participants