sinara-hw / Thermostat_EEM

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

PoE again #22

Closed jordens closed 3 years ago

jordens commented 3 years ago

A few thoughts:

Looking at the SMSC app note on PoE, they recommend a different circuit for protection: back-to-back 3.3V TVS/Zeners accross RD and TD. Apparently lots of capacitance allowed here (50 pF differential).

Second, I'm a bit worried about the mismatch between the ESD diode datasheet and the schematic symbol:

image image

Unlike the symbol, in the datasheet there is no explicit protection of the data lines to AVDDT_PHY at all. It's only a Zener + forward diode to GND. And that Zener has 8.3 V clamping voltage (1 A) and 6.5 V breakdown (1 mA): higher than the abs max of the PHY (3.3 V + 2 V = 5.3 V), higher than expected from the schematic symbol if it was only a forward diode drop to 3.3V (AVDDT_PHY), and higher than the 3.5 V "punch-through" + forward diode drop or 4.5 V clamping (1A) plus forward diode of the SMSC design.

Third, I think in our design the AVDDT_PHY protection of 8.5 V is still a bit loose compared to the 3.6 V abs max of the PHY. Yes, there is plenty of capacitance to dampen the surge, but still. Needs to be compared to the ~20V*1µs transients that the SMSC App note shows.

TI talks about ESDS312 and LC0-3.3 3.3V clamps/ESD/surge/lightning protection.

Also re https://github.com/sinara-hw/Thermostat/issues/83

This also applies to the other instances where we use that ESD diode pack (USB, 3x I2C, JTAG) and Stabilizer and Thermostat (non-EEM) as well.

gkasprow commented 3 years ago

True, the symbol is wrong! This could explain why PHY died. Good catch! We also need to replace it with the Stabilizer module. Fortunately, I did not update all other designs with this part! What we need is not necessarily Zener diode but fast protection diodes where excessive charge can be dumped to the supply rail. This would work https://datasheet.octopart.com/PRTR5V0U4Y%2C125-Nexperia-datasheet-87582156.pdf

gkasprow commented 3 years ago

I'm not sure if the cathodes can be connected to 3V3 rail. It will limit the max voltage produced by the PHY chip (there is a transformer with center tap). Maybe we should connect it to 5V rail instead...

gkasprow commented 3 years ago

the maximum ratings say: obraz

gkasprow commented 3 years ago

on the other hand, the TX won't exceed 1V. obraz

gkasprow commented 3 years ago

which means that we have AVDDPHY + 0.5V on one output and AVDDPHY -0.5V on complementary output. The TVS diodes should not clip such amplitude

jordens commented 3 years ago

I always thought they were complementary and just switch between 0 and avddphy. I would not expect them to get above avddphy actively. There will always be one inactive half of the winding. Induced voltages in the inactive half should be clamped to avddphy+diode in the phy. No?

gkasprow commented 3 years ago

These are analog signals. 3-state sequence. The transformer central tap is connected to 3V3, so when one output pulls low, the other gets the same value above the bias. The PHY does not have the clamping diode (it accepts up to 5.3V), it's like an open-drain circuit. That's why it can be vulnerable to damage.

jordens commented 3 years ago

Note for the next time we revisit this: We could also try "more robust", "industrial" PHYs like the DP83822.

gkasprow commented 3 years ago

Good ida. But I would do it in case we observe chip failures.