Continue with the tasks still unchecked above plus:
The things you checked look good 🥇
Sheet 1:
[x] Rev 1.1.4 now (this should be a global attribute that updates all the sheets, you messed up somehow)
Sheet 3:
[x] Can you move the thermistor closer to the microcontroller base, like A5 :)
[x] Do you want test points on the I2C buses?
[x] Do a library update all
Sheet 4:
[x] Addr = 0x29 | R/!W (don't forget the bitwise OR pipe symbol)
Sheet 5:
[x] Bulk cap is for VBATT-5. It will see the di/dt and want the extra stored energy
[x] Do you want the current shunt amplifier that is 50x or 100x gain? 2.5V (ADC range) / 2 (positive and negative) / 50 (gain) / 0.05 ohm (shunt) = +/- 0.5A = 0.12mA/LSB theoretically.
[x] No decoupling for the current shunt amplifiers, sad for days
Sheet 6:
[x] ADC's VDD (for analog circuitry) should be AVDD sourced. VDRIVE (for driving digital logic) should and is 3.3V sourced
[x] Decoupling caps are weird, each power pin should have one and make sure to have the correct voltage and ground
Sheet 9
I unchecked a thing you didn't do
Sheet 10:
[x] Why two decoupling caps of the same size (capacitance and package). If you want the decoupling talk I can which would give you a reason to have more sizes for a single pin
[x] Unused ADC inputs AHHH, tie em low
[x] Is a temperature monitoring ADC a primary or secondary bus kind of component? (hint: it isn't the one that is currently connected :) )
[x] I see your note, slack would have been faster 👍
Continue with the tasks still unchecked above plus: The things you checked look good 🥇
Sheet 1:
Sheet 3:
Sheet 4:
pipe
symbol)Sheet 5:
Sheet 6:
Sheet 9 I unchecked a thing you didn't do
Sheet 10: