Just something to consider. The Millis() function on the arduino platform resets after around 50 days or so. Make sure you have a case for that so the clock doesn't randomly barf
I'm now using one of the hardware timers for the clock, so no worries there