sinara-hw / Thermostat_EEM

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

CPU signal assignment #3

Closed gkasprow closed 3 years ago

gkasprow commented 3 years ago

@jordens Please have a look at the CPU signal assignments. It should be coherent with the STM32MX project Thermostat_8CH.zip

jordens commented 3 years ago

Looks good!

jordens commented 3 years ago

@gkasprow Could you elaborate a bit on the ideal diode design? All I know is this. Doesn't the body diode always power downstream EEMs ~and also lead to POE_PWR_SRC being at best one body diode drop below 12V~?

gkasprow commented 3 years ago

the MOSFET is off by default. When forward currents starts passing via FET diode, the from EEM to 12V rail, the T10A gets disable and MOSFET is on. Even cunducting FET has enough voltage drop to keep the T10 in off state.

jordens commented 3 years ago

Ah. I just didn't look close enough. The body diode is reverse biased in the situation I was thinking of.

jordens commented 3 years ago

@gkasprow I added a couple items to my list above. Really good.

jordens commented 3 years ago

And finally one idea: can we add headers to support bridge and kelvin measurements (for load cells etc) on the ADC channels? Just a 16 way DNP IDC header (vexc+, vexc- per channel, the sense side of the 5k resistors) and two 0R jumpers per channel added between vexc+ vsense+ and vexc-/vsense- to optionally break those up.

jordens commented 3 years ago

@gkasprow I added and clarified a couple of the points above.

gkasprow commented 3 years ago

Let's add 8pin IDC connectors, it's easier to fanout them to DSUB using IDC connectors

gkasprow commented 3 years ago

I moved the PWM_maxINeg1 to PB5 to free the PC7 which is now TIM8_OC2 and steers the fan PWM

gkasprow commented 3 years ago

I don't want to make the excitation switchable using MOS switches. What I can accept is a miniature relay. Shall I go for it or 0R resistors are fine? You can always place jumpers on neighboring pins of the IDC8 connectors to enable 2-wire mode without a need for soldering 0R resistors.

gkasprow commented 3 years ago

I rearranged the placement:

gkasprow commented 3 years ago

@jordens do you want to run all channels with full power? 30W PoE would not be sufficient. We would also need a stronger DC/DC converter.

jordens commented 3 years ago

ACK the timer move, the IDC connectors, and the arrangements

We'll do active power limiting for both temperature and total supply load. It would be nice to get a current shunt and CPU ADC channel on the 5V or 12V rail in addition to the per-channel power measurements that we already have. If you already have nice 5V DC-DC blocks that can do a bit more current in mind, that would be interesting when using the Molex/barrel. But more PoE power is a bit awkward infrastructure-wise.

If you don't like to support alternating excitation with switches then let's leave it. Testpoints for GPIO01 instead please.

For the measurement configurations I'd like to ensure the following (I hope the notation is ok):

Forget about the I2C on the D-44 connector. I don't think that's worth it. Let's do identification and management on the current controler.

gkasprow commented 3 years ago

@jordens The scenario could be simplified using one DPDT relay. Reed relays are recommended because they don't require minimum operating current as regular relays do. Can you make a paper drawing of the idea? I got lost with these Rr+- What do you mean by the topology on the board? PLacement of test points? We don't have any free CPU pins, but I can add I2C current shunt. stronger DC/DC is not a problem, I can also use two existing ones in parallel.

jordens commented 3 years ago

Alternatively if you think it may be a good idea to I2C on the header board, then we could reduce the number of wires per TEC to 2. People seem to be talking about 5 A per contact and some 20 mOhm or thereabouts.

gkasprow commented 3 years ago

What we can do is to reduce the number of TEC wires to two. We will spare 8 pins. Maybe, with SPDT relays we could scrap the internal IDC connectors ?

jordens commented 3 years ago

Poor man's sketch of the four different use cases I have in mind and the IDC/D-44 and ADC connectivity.

image

Hope that's clearer.

I'd prefer to have at least the four Vref- on the D-44 connector. That would support 16 NTCs with still really good measurement quality.

jordens commented 3 years ago

To support that kind of chopper mode, we'd need a DPDT relay per ADC to commutate Vexc+- between Vss/Vdd. The relay would be driven from GPIO0 by the ADC sequencer automatically (MUX_IO). It would work with all four use cases and can be disabled/held fixed. The configurable ADC delay for that looks perfect.

jordens commented 3 years ago

If we can put all four Vref+ and the four Vref- onto the D-44 connector (by scrapping the third TEC wire) then we can scrap the IDC connectors. Vexc+- and 6 wire mode isn't that interesting to me. On the D-44 we'd have then 4x2 TEC+, 4x2 TEC-, 8 Vsense+, 8 Vsense-, 4 Vref+, 4 Vref-, SCL, SDA, 3V3_MP, GND. Exactly 44 contacts.

jordens commented 3 years ago

A 3 m 22 AWG wire has 150 mOhm resistance. That with forward and return and two contacts, that drops 0.45 V at 3 A. Which is a bit annoying. 3 contacts would drop 0.3 V. Then again, if drop becomes critical, we can just pigtail fatter cables from the D-44. The D-44 still looks like the best connector for the panel. The rest is a question of cabling.

gkasprow commented 3 years ago

One can also use P3V3 and GND as vexc+-, probably using 3V LDO.

gkasprow commented 3 years ago

I will use MIC45212 which can give off 14A of current.

gkasprow commented 3 years ago

@jordens If we route also the GND and VDD, which is the case with I2C, then all possible variants could be possible. Since we don't need live reconfiguration, I would simply add DIP switches that configure the resistor network. 8 position DIP switch would do the job. Alternate actuation is also ok, we need only DPDT reed relay (20$). Reed relays also introduce very low thermoelectric potential.

gkasprow commented 3 years ago

Such schematic should do the job. obraz

gkasprow commented 3 years ago

I'm not sure if we can use alternate excitation due to limitation of REF inputs obraz REF inputs can be buffered so we can use identical protection/filtering as in the case of analog inputs.

gkasprow commented 3 years ago

This is the proposed protection of VREF inputs obraz

jordens commented 3 years ago

The REF+- wiring, MIC45212 look good. I don't like the DIP switches for the resistor network configurations. I'd rather just have this done in rework. With the Vexc commutation voltage offsets (due to thermoelectrics or MOSFET resistances) would not be a problem. They are cancelled by the measurement methods. But if -- as you say -- REF+ >= 1 V + REF- (I'm somewhat unsure whether this is required) then we can't do it this way. Let's leave it without Vexc commutation and just put test points onto GPIO0/1. I don't like the idea of placing the commutation between Vref and Vsense instead. Also what's the lifetime cycles of those reed relays? We'd be switching them at 20-50 Hz. That's 1e6/day.

gkasprow commented 3 years ago

1e6 is too much. They have 1e7..1e8 without load Do you afraid of the DIP switch's performance? They are not hermetic and after some time the contacts will degrade. We can of course substitute them with reed relays but that would be costly. Another option is OPTO-MOS relays that have a few tens of Ohm ON resistance, 1e-12 off current, and no leakage to GND at all.

jordens commented 3 years ago

My thought was to commutate Vexc between successive samples to zero out thermoelectric voltages etc. That can be implemented with the ADC sequencer at maybe 25 Hz. Vexc+ does not need to be very stable, not even between the two polarities, it already has 66 Ohm impedance in Thermostat-v2.0. Only REF+- need to track Vref+- accurately. But if we believe that REF+ > REF- then we can forget about it. Note that the manual also says "The device functions with a reference magnitude from 1 V to AVDD1" suggestiing that the magnitude of the differential counts. But then it says "REF− can span from AVSS to AVDD1 − 1 V" and "REF+ can span from AVSS + 1 V to AVDD1" which mostly exclude it. Let's consider actively commuting Vexc dead. I don't like the DIP switches to implement the different resistor options in the signal path (or for commutating) because they are physically large, thus take up space and can have significant temperature gradients with thermoelectric potentials. And most importantly I'd expect that they won't remove the need to change resistors for some applications and the sensor end of the cable will need reconfiguration and soldering anyway. If the single-ended ratiometric option and the bridge measurements prove useful and work well, then I'd be happy to reconsider the DIP switches. What do you think?

Also, if you go for a separate LDO for Vexc+, can we up the voltage (AVDD1 = Vexc+, and maybe also AVDD2) a bit towards the 5 V the ADC can handle? I'd expect dynamic range to increase a bit. I think we can share one LDO among all ADCs.

gkasprow commented 3 years ago

that's not that bad with DIP switches area. obraz The idea was to just label them 1,2,3,4 where every number means one of the sensor scenarios. So the user simply toggles certain combinations of switches and gets desired input configuration. I already implemented dedicated LDOs for analog and digital ADC rails. 5V AVDD is OK, the same for Vexc. The ADC can work with neraly any combination of IOVDD and AVDD obraz obraz What if we toggle excitation of the input signals instead of REF?

jordens commented 3 years ago

I still don't see us making much use of the DIP switches. If I needed DIP-switchable configurations and could live with them in the signal paths, I'd put them on the header board on other side of the cable.

Commutating Vsense+- doesn't help with thermoelectric voltages (you need to commutate the drive as well) and the ADC can already mux the inputs in any configuration. We'd need to commutate Vref+- while leaving the REF+- polarity fixed. That's tricky since that commutation would sit in the signal path where it easily introduces non-symmetric offsets.

jordens commented 3 years ago

Do you think it makes sense to use two AD7172-4 instead of four AD7172-2? Price per chip is the same, programming and software is compatible. The only downside I see is the lack of the temperature sensor and internal reference (which we don't use).

jordens commented 3 years ago

Could you explain the reset circuit with the diode between the SWD/JTAG NRST and the CPUT NRST/IC13 reset generator? The reset generator is push-pull but this way the JTAG/SWD can't reset the CPU (only monitor the reset?) and the connection seems useless then.

gkasprow commented 3 years ago

True, I was already writing that since we have 3V3 and GND, we can realize all scenarios on the header board. So no resistors are needed. I would simply put only default configuration, rest of them can be build using dedicated header boards. In case of ADC the price is not a big contribution, it's better to use components already used in other designs.

gkasprow commented 3 years ago

the reset chip is push-pull, it keeps the RSTn pin low for a certain time after power on. Then its output is high and the debugger can pull RST low. The net name should be called RSTn, not RST The JTAG JRST is not connected now.

jordens commented 3 years ago

Ah. I mixed up where µC NRST is connected (RST and not CPU_RESETn).

You are right about the resistors not being needed at all here. They could and maybe should all be on the header board! But then we definitely need Vexc- (i.e. GND) and Vexc+ (P5V0A from the LDO, not 3V3) on the D44 connector.

I'm still unsure whether it's better to have I2C on the header board or instead use those four contacts to beef up all TEC connections to three contacts.

jordens commented 3 years ago

And it's not the best idea to connect AIN4 to REF- since REF- is available as a dedicated mux input already. If you don't have another idea, I think it's best to connect AIN4 to REFOUT (2.5 V reference that we don't use, which is not available as a mux input which gives us a nice (indirect) handle on measuring all relevant frontend voltages single-ended and absolute (measure REFOUT/REF- against REF+/REF-, then measure AIN0-3/REF- and AVDD/AVSS against REF+/REF-).

gkasprow commented 3 years ago

why not use 3V3 LDO on the header board and supply I2C from Vexc+? I2C won't be used often. In this way we would keep all 4 vref pairs

jordens commented 3 years ago

Yes. Sounds good.

gkasprow commented 3 years ago

So, to conclude, I will keep only the default symmetrical ratiometric connection. obraz

jordens commented 3 years ago

Yes. Exactly like that. Connect Vref- to REF- and Vref+ to REF+. Bring VREF_5V0 and GND to the connector. No need for the net-ties NT6 and NT2 anymore.

gkasprow commented 3 years ago

Right, the net ties were needed to avoid the wrong GND routing

jordens commented 3 years ago

I'd like to change the TEC_U_Meas circuit to be referenced to the MAX Vref 1.5V (not (CPU ADC_REF + REF_GND)/2). That makes these MAX voltages all consistently referenced. Could you also remove the DAC_FB circuitry? I don't want to use it and instead find the overall offset with ITEC or TEC_U_Meas vs the MAX Vref 1.5V. At least DNP R69 by default and feel free to use those CPU ADC inputs for something else (sensing the 12V current shunt for example).

jordens commented 3 years ago

I'd also like to be able to measure the P3V3 voltage because the PWMs are proportional to it. The best way I see is to connect VBAT to a scaled P3V3 and then measure VBAT relative to VREFINT (~1.2V) or P5V0A/CPU_ADC_REF.

gkasprow commented 3 years ago

Right, we have ITEC and TEC_U so these are suffienct means to verify that DAC works. We have free ADC inputs, so we can measure 3V3 directly.

gkasprow commented 3 years ago

obraz

gkasprow commented 3 years ago

the 3V CPU reference generation uses 1% resistors and opamp buffer with offset of 7mV max . Do we need better precision?

gkasprow commented 3 years ago

I mean this circuit obraz

jordens commented 3 years ago

No. 1% and 7mV offset is fine for me.

gkasprow commented 3 years ago

I added a new schematic release. Please check it.