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

Add PT100 on MCU's with 3.3v excitation temptable #16731

Merged
merged 4 commits into from
Jan 31, 2020

Conversation

petrzmax
Copy link
Contributor

@petrzmax petrzmax commented Jan 30, 2020

Description

I was trying to set up PT100 sensor which e3d amp board on my mks robin nano, but I had some problems. Being 100% sure that wiring is ok I was getting about 180 deg at room temperature while on the signal pin a had 1.2V. I was trying different pins and even removed the pull up resistor but still getting same results. I was searching a bit and i have found this comment: #4987 (comment)

@tphan26363 comment sounded logical, so I have recalculated existing temptable. And it worked like a charm! Now I'm getting right temperature.

tl;dr It's probably related to different pullup voltage (3.3 / 5v) so You may want to use different name etc. in the commented sections.

Benefits

Easily use PT100 with amp board on MCU's with 3.3v excitation (STM32 \ LPC176x....)

Related Issues

#4987

@InsanityAutomation
Copy link
Contributor

While in essence I agree, I would consider the naming a bit. The root issue is the excitation, 3.3v or 5v. I would label it as such as some boards may be 5v with a 32 bit mcu.

@petrzmax
Copy link
Contributor Author

petrzmax commented Jan 30, 2020

@InsanityAutomation You are right my friend. I'm a bit lame in giving right names. Do You have some
idea how I should name it?

(A bit new to open source projects question) should I force push ammended commit with fixed names and message or just make another one to my branch?

@InsanityAutomation
Copy link
Contributor

InsanityAutomation commented Jan 30, 2020

just another push to youre branch is fine, doesnt need to be a force push. I would just state excitation voltage on each one

*    20 : Pt100 with circuit in the Ultimainboard V2.x with 5v excitation (AVR)
 *    21 : Pt100 with circuit in the Ultimainboard V2.x with 3.3v excitation (STM32 \ LPC176x....)

As the rest of the circuit with the amp board is still the same. Add in any processors you know for sure are 3.3v.

@petrzmax
Copy link
Contributor Author

@InsanityAutomation Thank You my friend. Pushed it a moment ago ;)

@petrzmax petrzmax changed the title Add PT100 on 32 bit mcu temptable Add PT100 on 32 bit MCU's with 3.3v excitation temptable Jan 30, 2020
@petrzmax petrzmax changed the title Add PT100 on 32 bit MCU's with 3.3v excitation temptable Add PT100 on MCU's with 3.3v excitation temptable Jan 30, 2020
@InsanityAutomation
Copy link
Contributor

Sounds good, just the termistor names file needs a corresponding edit :) After that up to Thinky!

@petrzmax
Copy link
Contributor Author

These should be short I think, so I have added just voltage there ;) I hope it's all good to go now :D

@thinkyhead
Copy link
Member

thinkyhead commented Jan 30, 2020

@InsanityAutomation — Do you anticipate we will want to do this for all the tables? And if so, can we just automate this by adding a factor that gets applied to all tables based on the board's logic level?

@InsanityAutomation
Copy link
Contributor

@thinkyhead no, because most have the thermistor setup through a level adjusted with the sense resistor. This is different because it doesnt go through the normal circuit prepared for it. It goes to a raw analog input.

@thinkyhead
Copy link
Member

thinkyhead commented Jan 30, 2020

And yet the conversion of table 20 using the formula N * 5 / 3.3 lines up exactly with the values in table 21. Seems like we could use this for some cases instead of defining separate tables.

@thinkyhead
Copy link
Member

Just for kicks, I tweaked the thermistor table to expose the relationship between it and table 20, and introducing a method to allow for synthesizing other tables in the future.

@thinkyhead thinkyhead added C: Peripherals T: HAL & APIs Topic related to the HAL and internal APIs. labels Jan 30, 2020
@thinkyhead thinkyhead merged commit e4679c1 into MarlinFirmware:bugfix-2.0.x Jan 31, 2020
@InsanityAutomation
Copy link
Contributor

Perhaps an extension to type 1000 p parameter to apply a multiplier would be a nice wishlist item.

@Bob-the-Kuhn
Copy link
Contributor

I'm confused about what problem you are trying to solve.

In my experience the thermistor table does NOT change if the Vref for the ADC is the same as the excitation to the PT100 circuit. It's the RATIO of Vin to Vmax that is important. That ratio is only dependent on the resistance values of the sensor and the pullup resistor.

NOTE - the PT100 amplifier is more complicated but it is still essentially a pullup resistor tied to the sensor. The amplifier's output (at any given temperature) changes directly proportional to the change in the power supplied to the amplifier. The RATIO of Vin to Vmax remains the same.

For example, a PT100 with a 4.7K pullup will use table 147 if it's on an AVR board or a 32 bit board. A PT100 amplifier system will use table 20 if it's on an AVR board or a 32 bit board.

The same is true if using a PT100 amplifier. On an AVR system the amplifier needs to be given a 5V supply. On a 32 bit board it needs to be given a 3.3V supply. In no case should the PT100 amplifier be powered from 5V on a 32 bit system (most analog inputs are not 5V tolerant). In no case should a pullup resistor be used with a PT100 amplifier. I've had one PT100 amplifier that worked only on a 5V (AVR) system and another that worked on either 5V or 3.3V systems.

In my experience a PT100 system being off by 5-10C at room temperature is usually caused by the ground reference for the PT100 being at a higher voltage (as in 10-20mV) due to current flowing in the thin ground traces on the controller board. It is good practice to run a heavier wire between the analog input port's ground pin and the main ground pin on the controller's power connector. Some improvement is also seen if an additional ground wire can be attached to a ground near the CPU.

If you absolutely must attach the PT100 to an analog input with a pullup resistor then my recommendation is to ditch the PT100 amplifier and wire the PT100 sensor directly to the port.

A custom thermistor table is needed if a PT100 amplifier is connected to a port with a pullup resistor. There's enough variation from amplifier to amplifier that a table customized to that amplifier/controller is a good idea. I have an excel spreadsheet that will generate the table given a couple of voltage measurements.


I suspect you have a wiring error or a bad PT100 amplifier or are connected to a port with a pullup resistor.

Please provide a drawing of how your system is setup. It must include details on what each pin on the PT100 amplifier's output connector is tied to.

@thinkyhead
Copy link
Member

Since the table 21 values are exactly (5.0 / 3.3) times larger than the table 20 values it implies that the signal coming from the device is 5V but the ADC input is expecting current to be relative to 3.3V. If that is the case, I'm not sure how long a 3.3V input will enjoy that.

@petrzmax
Copy link
Contributor Author

@Bob-the-Kuhn I'm working on mks robin nano at the moment so I'm basing on its schematic etc. Here are some informations about this board so we can figure this out:
Endstop has 3 pins: signal, ground and 5V, so all this events should be prepared for 5V.
btw, I have removed R56 resistor.
I'm not sure how it's handled by mcu, but a lot of additional "things" are working on 5V. Also I think that ADC converter working with 3.3v logic should give me max reading at my connected 5V logic amp but it's not doing this. So If this is normal behaviour for this board I'm not worried about breaking something.

Here are schematics:
1
2
3
4
5

As You can see TH1 is connected similiar to X endstop. The difference is that there is HGND and DGND used.
HGND = power ground (High Voltage)
DGNF = digital ground or down (I'm not sure)

Here are all schamatics if You want to check this out.
MKS Robin Nano V1.1_001 SCH.pdf

@petrzmax petrzmax deleted the bugfix-2.0.x-dev branch January 31, 2020 13:44
@tphan26363
Copy link

Just let you know that if you look at the data sheet of some of the chip, The ADC input can withstand +4 above VDD. For example:
https://www.st.com/resource/en/datasheet/stm32f405rg.pdf

GeminiServer pushed a commit to GeminiServer/GeminiServer-Marlin that referenced this pull request Jan 31, 2020
@Bob-the-Kuhn
Copy link
Contributor

The max voltage for an analog input is 4.0V. Above that a parasitic diode to 3.3V gets turned on. It takes some looking through the data sheet but here are the relavent passages:

  • Table 11: VIN 5 V tolerant except when in analog mode ... Input voltage on any other pin 4.0 MAX
  • Table 7 note 4: 5 V tolerant except when in analog mode

It's safest to power the PT100 amplifier from 3.3V. That way you're guaranteed to not cause any problems.

On the other hand the PT100 amplifier has a VERY weak output and you'd need to be over 800C to get to 4V (or have the PT100 amplifier's input open) so I doubt that there's been any damage done.

@InsanityAutomation
Copy link
Contributor

InsanityAutomation commented Feb 1, 2020

This would go back to a thought we discussed in passing a year or so ago of putting a max temp sanity check in each thermistor table. Could protect from damage in this situation.

Just in case anyone planned to fdm mig wire or something :)

@petrzmax
Copy link
Contributor Author

petrzmax commented Feb 1, 2020

@Bob-the-Kuhn Thank You, now everything is clear :)
Have a good day.

tompe-proj added a commit to tompe-proj/Marlin that referenced this pull request Feb 1, 2020
* Set up DIAG endstops for BTT SKR 1.4 / 1.4 Turbo (MarlinFirmware#16727)

* Extend RX/TX pins for up to 8 extruders

* Fix up, improve endstop pin auto-assignment

Followup for MarlinFirmware#16723

* (21) PT100 for MCUs with 3.3v logic (MarlinFirmware#16731)

* [cron] Bump distribution date (2020-01-31)

* Add sanity-check for new Advanced Pause option

Followup to MarlinFirmware#16372

* Include macros for delta ABC

* Update Russian language (MarlinFirmware#16745)

* Fix BTT SKR 1.4 extra endstop pins (MarlinFirmware#16738)

* Option for Trigorilla 1.4 with add-on endstops board (MarlinFirmware#16737)

* Consistent M112 with Emergency Parser (MarlinFirmware#16747)

* Improve mfadd helper script

- Use the original branch name if none is supplied
- Set the remote tracking to the source
- Accept User/Branch or User:Branch syntax

* Clean up i2c encoder, sanitize serial

* Misc cleanup, whitespace

* Encapsulate probe as singleton class (MarlinFirmware#16751)

* G34 automatic point assignment (MarlinFirmware#16473)

* Fix Temperature::over_autostart_threshold (MarlinFirmware#16749)

* Update Russian language (MarlinFirmware#16750)

* Fix CURRENT_STEP_DOWN compile error

* Drop obsolete SD special char handling

See MarlinFirmware#14035

* Probe singleton patch

Followup to MarlinFirmware#16751

* Fix RGB / Neopixel white color bug

See MarlinFirmware#16752

Co-authored-by: Robert Stein <[email protected]>
Co-authored-by: Scott Lahteine <[email protected]>
Co-authored-by: Artur Petrzak <[email protected]>
Co-authored-by: Acenotass <[email protected]>
Co-authored-by: rebel1 <[email protected]>
Co-authored-by: Jason Smith <[email protected]>
Co-authored-by: InsanityAutomation <[email protected]>
mjparme added a commit to mjparme/Marlin that referenced this pull request Feb 1, 2020
* Don't test certain changes

* Reset runout.ran_out on resume (MarlinFirmware#16230)

* Step timing cleanup and rounding fix (MarlinFirmware#16258)

* Add MRR_ESPA/_ESPE (ESP32) boards (MarlinFirmware#16238)

* Add Ender-5 Pro config (MarlinFirmware#16221)

* Add FLYBOARD (STM32F407ZG) (MarlinFirmware#16257)

* Fix STM32 flush of TX (used by UBL) (MarlinFirmware#16197)

* Flash leveling (for some STM32) (MarlinFirmware#16174)

* Some ESP32 patches (MarlinFirmware#16297)

* MKS SGen-L pins EEBF or EFBF scheme (MarlinFirmware#16296)

* Add Rumba32 support for PIO (MarlinFirmware#16202)

* MKS Robin 2 (STM32F407ZE) base support (MarlinFirmware#16270)

* Update Czech language (MarlinFirmware#16305)

* Sync SKR E3 configs (MarlinFirmware#16301)

* Add NOZZLE_AS_PROBE (no probe offsets) (MarlinFirmware#15929)

* Update README with status badge, etc.

* Split up HAL items

* Update mfpub for current MarlinDocumentation

* Fix Max7219 with 256 or more cels

* Fix games menu back item, titles

* Update comments re: NOZZLE_AS_PROBE

* Fix min limit for acc, feed, jerk (MarlinFirmware#16416)

* Improve Renkforce configs (MarlinFirmware#16417)

* Permit ENDSTOP_INTERRUPTS_FEATURE on more STM32 (MarlinFirmware#16412)

* Improve STEVAL_3DP001V1 and future STEVAL_* support (MarlinFirmware#16404)

* MKS Robin ILI9328 TFT support (MarlinFirmware#16401)

* Disable PIDTEMPBED for SKR Mini E3 (MarlinFirmware#16396)

* Return from loop() on non-AVR boards (MarlinFirmware#16390)

* Refactor TMC-related macros and sanity checks (MarlinFirmware#16384)

* Add FILAMENT_UNLOAD_PURGE_FEEDRATE (MarlinFirmware#16372)

* Fix some DOGM warnings (MarlinFirmware#16363)

* Fix warnings in stepper.cpp (MarlinFirmware#16364)

* Fix PrintrBoard build (ignore TMC libraries) (MarlinFirmware#16346)

(In future try to get Teensy processors better supported by `TMCStepper`.)

* Clean up trailing whitespace

* PWM pin not needed for Neopixel brightness / submenu (MarlinFirmware#16345)

* Misc patches preceding DGUS PR

* Fix Visual Micro "Arduino IDE for Visual Studio" support (MarlinFirmware#16418)

* Update Russian language (MarlinFirmware#16426)

* STM32F1: Fix misleading indent / nullptr on FSMC (MarlinFirmware#16431)

* Z-offset edit precision based on value limits (MarlinFirmware#16425)

* Add Ender-5 leadscrew README (MarlinFirmware#16424)

* Add motherboard BIGTREE_SKR_V1.4_TURBO  (MarlinFirmware#16374)

* Enable MULTI_NOZZLE_DUPLICATION for BIBO (MarlinFirmware#16435)

* Touch UI: Fix UBL mesh value editing (MarlinFirmware#16432)

* Sidewinder X1 Config Updates (MarlinFirmware#16315)

* Add Leapfrog Xeed 2015 support (MarlinFirmware#16400)

* Improve JGAurora A1/A5S touch buttons (MarlinFirmware#16394)

* Standardize drivers.h values, add class indirection (MarlinFirmware#16448)

* Update Russian language (MarlinFirmware#16440)

* BTT002: Add runout, PLR, and RGB pins (MarlinFirmware#16442)

* Pins debugging AVR serial pins (MarlinFirmware#16437)

* Fix MIN_PROBE_EDGE bug in default ABL G29 (MarlinFirmware#16367)

* Temporary CI fix for STM32

* Use a default monitor_speed of 250000

* MSG_WATCH => MSG_INFO_SCREEN

* Add EVNOVO (Artillery) Genius config (MarlinFirmware#16320)

* Fixed Creality CR-20 Pro configuration example (MarlinFirmware#16332)

* FYSETC F6 v1.4 board support (MarlinFirmware#16321)

* Add example configs. Expand custom menu. (MarlinFirmware#16286)

- Anet E10
- Geeetech D200
- Geeetech M201
- JGAurora Magic
- MakerFarm Pegasus 12

* Fix Ender-2 display pins, add BTN_ENC (MarlinFirmware#16349)

* Update and fix DGUS (MarlinFirmware#16317)

* Fix serials available on SKR Pro 1.1 (MarlinFirmware#16439)

* Fix daily date bump action

* Action to check PRs

* Update maintainer funding links

* Skip build tests on forks

* Update git helper usage

* Fix EEPROM error with EXTRUDERS == 0 (MarlinFirmware#16464)

* Fix M115 cap with EXTRUDERS == 0 (MarlinFirmware#16459)

* Add option to invert joystick axes (MarlinFirmware#16466)

* Update Russian language (MarlinFirmware#16461)

* Fix permyriad progress bar (MarlinFirmware#16460)

* Improve French language (UBL) (MarlinFirmware#16453)

* Followup to EEPROM patch (MarlinFirmware#16470)

* Remove extra UBL map edit menu item (MarlinFirmware#16451)

* Fix unused var warning (MarlinFirmware#16467)

* Update U20 config to help UBL toggle (MarlinFirmware#16471)

* Move auto_build.py to 'vscode' folder

* Fix MKS_SGEN sanity check (MarlinFirmware#16501)

* Fix EEPROM array size bug (MarlinFirmware#16475)

* Fix SKR Pro BLTouch conflicting timers (MarlinFirmware#16499)

* Provide some missing Arduino macros (MarlinFirmware#16497)

* Fix Change Filament menu item with runout (MarlinFirmware#16485)

* Fix LCD for Tevo Tornado (MarlinFirmware#16474)

* Update workflow skip condition

* Add menu item Tune > Advance K (MarlinFirmware#16488)

* Extend SERIAL_CHAR to take multiple arguments

* Code style and comment tweaks

* Correct ESP32 CPU speed in README (MarlinFirmware#16472)

* Add ARC_SEGMENTS_PER_SEC for finer G2/G3 arcs (MarlinFirmware#16510)

* Fix compile error w/out LCD (MarlinFirmware#16502)

* Add Probe Offsets menu (MarlinFirmware#16444)

* Fix Probe Offset XY edit items (et Français) (MarlinFirmware#16523)

* Update Russian language (MarlinFirmware#16522)

* [cron] Bump distribution date (2020-01-10)

* Prevent Z misaligment on tool change (MarlinFirmware#16518)

* Update Italian language (MarlinFirmware#16527)

* ESP3d integration for ESP32 (MarlinFirmware#16515)

* [cron] Bump distribution date (2020-01-11)

* Migrate actions to default (2.0.x) branch

* Only constrain motion on homed axes (MarlinFirmware#16533)

Co-authored-by: Scott Lahteine <[email protected]>

* Fix probeless delta build (MarlinFirmware#16537)

* Use RECIPROCAL macro (not _RECIP) (MarlinFirmware#16530)

* [cron] Bump distribution date (2020-01-12)

* Restore test_builds action to bugfix

* Update auto_build.py paths for Atom/Sublime

* [cron] Bump distribution date (2020-01-13)

* Add GTM32 (STM32F103VET6) environment (MarlinFirmware#16454)

* Power options formatting

* Geeetech G2Pro configuration (MarlinFirmware#16553)

* Fix Linux upload path detection (MarlinFirmware#16514)

* Add SERVO, TONE timers to variant for better STEVAL_3DP001V1 support (MarlinFirmware#16538)

* Clean up whitespace

* [cron] Bump distribution date (2020-01-14)

* Improved STMicro L64XX stepper driver support (MarlinFirmware#16452)

* Make lcd_power_loss_recovery_cancel exportable

* Fix PLR cancel with ExtUI (MarlinFirmware#16556)

* Bring MP_SCARA config up to date

* Fix Fysetc S6 FLASH_PAGE_SIZE and test build (MarlinFirmware#16560)

* [cron] Bump distribution date (2020-01-15)

* Update more configs

* Move configurations to a separate repo

* Fix build error with unsupported reset flags (MarlinFirmware#16562)

* Steval_3DP001V1 timers, analog inputs, etc. (MarlinFirmware#16565)

* Use Flash EEPROM on BTT002 (MarlinFirmware#16558)

* Get test configs from GitHub

* Fix typo

* Nikon IR support for time lapse photos (MarlinFirmware#16539)

* Adjustable ADC debounce delay (MarlinFirmware#16264)

* [cron] Bump distribution date (2020-01-16)

* STM32 RODATA LENGTH (MarlinFirmware#16580)

* Fix I2C address of MCP4728 on LPC176x (MarlinFirmware#16578)

* Fix GTM32 environment (to prevent a crash)

* Fix some warnings, Melzi pins

* Move macros to stepper/indirection

* Fix TOUCH_UI_FTDI_EVE bugs (MarlinFirmware#16540)

* ESP32 HAL: Fix random pauses during prints (MarlinFirmware#16548)

* [cron] Bump distribution date (2020-01-17)

* STM32 soft SPI. STEVAL_3DP001V1 SD read. M906 tweaks. (MarlinFirmware#16579)

* Clean up HAL ADC, old test scripts

* Adjust GTM32 build flags (MarlinFirmware#16582)

* Shared SPI sanity check (MarlinFirmware#16581)

* More CI test updates

* Update FYSETC S6 pins (MarlinFirmware#16559)

* PINDA v2 temperature sensor / compensation (MarlinFirmware#16293)

* [cron] Bump distribution date (2020-01-18)

* Allow RRW Keypad with any LCD

* [cron] Bump distribution date (2020-01-19)

* [cron] Bump distribution date (2020-01-20)

* Fix M112 with Emergency Parser

* Simplify TMC monitor code

* Updates for L64XX

* Function-like macros

* Use bool in pin compare

* Patch for _STEP_INIT

* Simplify a drivers.h macro (MarlinFirmware#16589)

* SAMD51: ADC for probe temperature compensation (MarlinFirmware#16596)

* Update FYSETC S6 Peripheral Pins (MarlinFirmware#16593)

* Update BTT002 pins (MarlinFirmware#16591)

* Reset shutdown timer on M85 (MarlinFirmware#16587)

* AVR sanity check for MONITOR_DRIVER_STATUS with SW Serial (MarlinFirmware#16421)

* STEVAL_3DP001V1: Easier Serial2 (for WIFI interface) (MarlinFirmware#16599)

* Fix Greek language string (MarlinFirmware#16601)

* Quad Z stepper support (MarlinFirmware#16277)

* LPC1768: Allow I2C master channel override (MarlinFirmware#16584)

* Move steps/mm out of slim menus (MarlinFirmware#16603)

* Fix CHAMBER_MAXTEMP security margin (MarlinFirmware#16600)

* L64xx M906 Fix status variable, formatting (MarlinFirmware#16597)

* Provide methods for M420 + MBL (MarlinFirmware#16602)

* [cron] Bump distribution date (2020-01-21)

* Fix STM401/4xx/STM32F7xx timers, STEVAL_3DP001V1 warning (MarlinFirmware#16621)

* Override for LPC1768 u8g/digipot I2C master ID (MarlinFirmware#16622)

* Release UI on G26 priming timeout (MarlinFirmware#16449)

* Add HMS434 V15 ATSAM board (MarlinFirmware#16620)

* Tweak some config spacing

* G12 defaults per tool. Event G-code for post-toolchange (MarlinFirmware#16554)

* Arc segment radius scaling (MarlinFirmware#16551)

* Update mfdoc, mfpub

* [cron] Bump distribution date (2020-01-22)

* Tweak mfpub stashing

* [cron] Bump distribution date (2020-01-23)

* Enable use of latest TMCStepper on MKS Robin Nano (MarlinFirmware#16652)

* [cron] Bump distribution date (2020-01-24)

* Convert chars only for enabled languages

* Extra debugging for leveling on/off

* Bring configs and drivers.h up to date

* Duet Smart Effector support (MarlinFirmware#16641)

* BigTreeTech GTR V1.0 / Support 8 extruders, heaters, temp sensors, fans (MarlinFirmware#16595)

* [cron] Bump distribution date (2020-01-26)

* Fix DELTA_CALIBRATION_MENU recursive call (MarlinFirmware#16656)

* Fix MKS Base + Digipot compile error (MarlinFirmware#16636)

* Add ESP3DLib idletask entry point (MarlinFirmware#16658)

* Fix Serial defines for M43 on AVR (MarlinFirmware#16649)

* Fix HOMING_BACKOFF_MM for DELTA (MarlinFirmware#16657)

* Clean up PID language defines

* More 8 extruder defines

* Update Slovak language (MarlinFirmware#16646)

* Fix M43 timer report on AVR (MarlinFirmware#16645)

* Fix Spanish (swapped BLTouch strings) (MarlinFirmware#16637)

* SKR 1.3 sensorless endstops hack (MarlinFirmware#16659)

* Fix PID F menu label

* [cron] Bump distribution date (2020-01-27)

* Preserve CWD for write/remove file (MarlinFirmware#16667)

* Rename some temperature members

* Fix Trigorilla 1.4 limit switches

See MarlinFirmware#16612

* Fix serial port redirection (index ≠ port num) (MarlinFirmware#16687)

* G60/G61 Position Save/Restore (MarlinFirmware#16557)

* Remove extraneous G60/G61 lines

* Fix Manual Bed Leveling with multiple extruders (MarlinFirmware#16688)

* [cron] Bump distribution date (2020-01-28)

* Prevent SD access from resetting ESP32 (MarlinFirmware#16690)

* [cron] Bump distribution date (2020-01-29)

* Fix Toolchange (!no_move) return to status (MarlinFirmware#16699)

* [cron] Bump distribution date (2020-01-30)

* Cleanup of old includes, add comments

* Fix HAS_FAN macro (MarlinFirmware#16717)

* Fix probe temp calibration (MarlinFirmware#16718)

* Support for E4d@BOX mainboard (MarlinFirmware#16716)

* Fix missing string for M48 build (MarlinFirmware#16708)

* Fix G60/G61 slots > 8 and compile error (MarlinFirmware#16715)

* Move L64XX index_to_axis to progmem (MarlinFirmware#16697)

* Refactor heater watch, job timer auto-start (MarlinFirmware#16725)

* Auto assign DIAG pins for multi-endstop (MarlinFirmware#16723)

* Tweak G60/G61 slots

* Fix compile error in SdVolume on ESP32 (MarlinFirmware#16728)

* Simultaneous use of rotary encoder and touch buttons (MarlinFirmware#16729)

* Set up DIAG endstops for BTT SKR 1.4 / 1.4 Turbo (MarlinFirmware#16727)

* Extend RX/TX pins for up to 8 extruders

* Fix up, improve endstop pin auto-assignment

Followup for MarlinFirmware#16723

* (21) PT100 for MCUs with 3.3v logic (MarlinFirmware#16731)

* [cron] Bump distribution date (2020-01-31)

* Add sanity-check for new Advanced Pause option

Followup to MarlinFirmware#16372

* Include macros for delta ABC

* Update Russian language (MarlinFirmware#16745)

* Fix BTT SKR 1.4 extra endstop pins (MarlinFirmware#16738)

* Option for Trigorilla 1.4 with add-on endstops board (MarlinFirmware#16737)

* Consistent M112 with Emergency Parser (MarlinFirmware#16747)

* Improve mfadd helper script

- Use the original branch name if none is supplied
- Set the remote tracking to the source
- Accept User/Branch or User:Branch syntax

* Clean up i2c encoder, sanitize serial

* Misc cleanup, whitespace

* Encapsulate probe as singleton class (MarlinFirmware#16751)

* G34 automatic point assignment (MarlinFirmware#16473)

* Fix Temperature::over_autostart_threshold (MarlinFirmware#16749)

* Update Russian language (MarlinFirmware#16750)

* Fix CURRENT_STEP_DOWN compile error

* Drop obsolete SD special char handling

See MarlinFirmware#14035

* Probe singleton patch

Followup to MarlinFirmware#16751

* Fix RGB / Neopixel white color bug

See MarlinFirmware#16752

* Suppress a compile warning

* More 8-extruder fixups

Co-authored-by: Scott Lahteine <[email protected]>
Co-authored-by: Tanguy Pruvot <[email protected]>
Co-authored-by: Jason Smith <[email protected]>
Co-authored-by: Luc <[email protected]>
Co-authored-by: thisiskeithb <[email protected]>
Co-authored-by: FLYmaker <[email protected]>
Co-authored-by: randellhodges <[email protected]>
Co-authored-by: ferengi82 <[email protected]>
Co-authored-by: Luu Lac <[email protected]>
Co-authored-by: petrzjunior <[email protected]>
Co-authored-by: InsanityAutomation <[email protected]>
Co-authored-by: Giuliano Zaro <[email protected]>
Co-authored-by: Dirk O. Kaar <[email protected]>
Co-authored-by: Pascal de Bruijn <[email protected]>
Co-authored-by: Bob Kuhn <[email protected]>
Co-authored-by: Alexander Gavrilenko <[email protected]>
Co-authored-by: swilkens <[email protected]>
Co-authored-by: Lino Barreca <[email protected]>
Co-authored-by: rado79 <[email protected]>
Co-authored-by: Acenotass <[email protected]>
Co-authored-by: ellensp <[email protected]>
Co-authored-by: Walt Sorensen <[email protected]>
Co-authored-by: yedey <[email protected]>
Co-authored-by: George Fu <[email protected]>
Co-authored-by: Vertabreaker <[email protected]>
Co-authored-by: greppp <[email protected]>
Co-authored-by: Jamie <[email protected]>
Co-authored-by: Justin <[email protected]>
Co-authored-by: ManuelMcLure <[email protected]>
Co-authored-by: 0r31 <[email protected]>
Co-authored-by: Alejandro Aguilera <[email protected]>
Co-authored-by: meponderR <[email protected]>
Co-authored-by: Mehdi Beyk Mohamadi <[email protected]>
Co-authored-by: FlyingSamson <[email protected]>
Co-authored-by: Ryan V1 <[email protected]>
Co-authored-by: David Klasinc <[email protected]>
Co-authored-by: gjdodd <[email protected]>
Co-authored-by: Dennis <[email protected]>
Co-authored-by: felixstorm <[email protected]>
Co-authored-by: vivian-ng <[email protected]>
Co-authored-by: tompe-proj <[email protected]>
Co-authored-by: MaukCC <[email protected]>
Co-authored-by: Ryan <[email protected]>
Co-authored-by: Artur Petrzak <[email protected]>
Co-authored-by: yangwenxiong <[email protected]>
Co-authored-by: Roman Moravčík <[email protected]>
Co-authored-by: Robby Candra <[email protected]>
Co-authored-by: Hans007a <[email protected]>
Co-authored-by: Neskik <[email protected]>
Co-authored-by: Mauro <[email protected]>
Co-authored-by: Radek Pietruszewski <[email protected]>
Co-authored-by: chgi <[email protected]>
Co-authored-by: Robert Stein <[email protected]>
Co-authored-by: rebel1 <[email protected]>
@goldjunge91
Copy link

goldjunge91 commented May 9, 2020

Hello, sorry for my bad english but in my case the pt100 with e3d amp board don't work on my skr1.3 i connect it like this picture and make in my config 21 read 1 degree only and when i plug this with same settings on 3.3v i have the same issues

@goldjunge91
Copy link

goldjunge91 commented May 9, 2020

for me on my skr 1.3 with marlin 2.5.3 work this
and change in pins_BTT_SKR_common.h the TEMP_0_PIN to #define TEMP_0_PIN P0_02_A7 for the TX0 pin and for the rx0 pin P0_03_A6

i add this in my config maybe you can make something like this for other user
#if TEMP_SENSOR_0 == 20
#define TEMP_0_PIN P0_03_A6
#endif

bigtreetech/BIGTREETECH-SKR-V1.3#212
@bdelia thanks a lot for that you find the pin numbers

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C: Peripherals T: HAL & APIs Topic related to the HAL and internal APIs.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants