Hooking up TLC5940 to 12V

. Is this the correct way to hook up the LEDs to a 12 V rail?

Looks like it. The +12 is connected to the anodes of the LEDs and the -ve from the 12V is connected to the arduino ground.
The TLC5940 chip can stand about 16V so the 12V is fine.