It looks OK to me. Just make sure the ground of your 12V supply is tied to the ground for the Arduino.