I have to drive a 3.3v laser diode with a 5v logic out. A logic LO will turn the laser ON, where a logic HI will turn the laser OFF. Below is the schematic. Did I design this circuit correctly, or is there something wrong? If so, is the correction simple, or is it totally messed up and there's a better way to do this? Thanks.