Nitpick: AD8541 used as overtemperature comparators

sinara-hw / Stabilizer_Piezo_Driver

SImple piezo driver mezzanine for Stabilizer

7 stars 3 forks source link

Nitpick: AD8541 used as overtemperature comparators #6

Closed dnadlinger closed 3 years ago

dnadlinger commented 3 years ago

Just scrolled through the release PDF out of curiosity, and noticed that 2xAD8541 are used as comparators/LED drivers for the overtemperature circuit. Since those aren't appear anywhere else on the BOM, isn't that a bit of overkill, i.e. wouldn't pretty much any cheap dual comparator for 1/10th the price also do the trick?

gkasprow commented 3 years ago

I wrote the same in of of my reviews

JakubKajzer commented 3 years ago

AD8541 changed to LM358

dnadlinger commented 3 years ago

(Haven't reviewed the changes myself, but then again, it's not my role to do so. :) )