- 5V logic cannot be directly connected to the base. Use a current limiting resistor (and usually a B-E resistor as well, to keep it fast).
- The output won't pull up to full +V, due to the LED. This might not be a problem (is there another pull-up on the connection?).
- The output is not short, overvoltage or reverse protected, so I don't think I would call it "safety critical".
- What kind of "safety critical" application do you have in mind? I sincerely hope you aren't doing something like medical life support or mission-critical, and asking questions to the internet about it. I would seriously suggest spending real money talking to consultants about "safety critical" systems.
Tim
Thank you T3sl4co1l for your reply!
Sorry! I should have included more of the circuit...
-There is of course a base-resistor (but it is outside the snapshot)
-There is also a separate pull-up on the 24V side, so no worries.
-There is also a current-limit resistor on the output, which actually goes to the base of an PNP.
Safety-critical was wrongly put, or wrongly understood. What I mean is robust. For example, is it more robust than using a dedicated chip?
Don't worry, you wont see this in a pacemaker or any medical...