sinara-hw / Thermostat_EEM

Thermostat with 4 TEC and 8 sensor channels in EEM form factor
2 stars 1 forks source link

TEC Driver REF pin loading and capacitance #55

Closed nkrackow closed 1 year ago

nkrackow commented 2 years ago

The TEC Driver IC has a Reference pin that outputs the internal reference voltage. This pin should have a 1uF capacitor according to the datasheet, Thermostat curretly has a 10nF. The pin can sink 1mA but only source 10uA. With the current loading the pin is pulled from nominal 1.5V to ~1.05V. Screenshot from 2022-02-21 16-38-08

jordens commented 2 years ago

This was a bug from https://github.com/sinara-hw/Thermostat_EEM/issues/46 I meant adding an 10k R 10n C filter to VREF, after the 1u C and not replacing the 1u with a 10n C 10k R combo. That doesn't work.

jordens commented 2 years ago

Still, there seems to be something loading it. @SingularitySurfer sure you measured the 1.05V on the pin 4 side or R224? In any case the voltage divider at IC14 is broken due to the non-zero impedance of R224.

nkrackow commented 2 years ago

Still, there seems to be something loading it. @SingularitySurfer sure you measured the 1.05V on the pin 4 side or R224? In any case the voltage divider at IC14 is broken due to the non-zero impedance of R224.

Actually I measured on the wrong side of the resistor, sorry! The highlighted net on the pin has 1.5V. Behind R224 there is 1.05V left. Screenshot from 2022-02-22 10-35-43

jordens commented 2 years ago

Ok. Then it's just the divider. And the RC. Should fix both.

gkasprow commented 1 year ago

R224=0R, C168=1uF

jordens commented 1 year ago

@gkasprow I think that's too much load. VREF can do +10µA to -1mA. We are loading it with 25k from 4.3V worst case (see IC14).

gkasprow commented 1 year ago

Right. So let's add another opamp to buffer it. It can be one for all four channels

gkasprow commented 1 year ago

Or, let's replace IC14 with LMV358

gkasprow commented 1 year ago

image

jordens commented 1 year ago

The buffer should also drive the other vref users, e.g. to the CPU.

gkasprow commented 1 year ago

good idea.

gkasprow commented 1 year ago

Fixed in v1.1rc2