Hi,
It looks okay to me,
Some people add diodes at the motor connections (I usually dont myself but if youre building for a long time project or a commercial unit then they are worth considering). This is to protect the driver chip because when motors change directions or stop etc, the magnetic field breaks down and can back up into the chip again (which might get damaged), the diodes direct the power away from the chip and into the battery or power bus allowing it to be re used instead of being wasted and possibly smoking youre driver ic.
See this website for a better explanation:
http://www.robotroom.com/HBridge.html He's using a different motor driver but the principles are the same. Obviously you need to select your diodes to be able to handle your own motor requirements.