no ecco, setInterval devi chiamarlo UNA volta e basta, poi ogni tot la funzione viene rilanciata:

var data, ora, minuti;

function tempo(){
data= new date();
ora= data.getHours();
minuti= data.getMinutes();
//Fai quello che vuoi
}

setInterval(tempo, 1000);

Quello che fai tu chiama OGNI secondo una funzione che incomincia un ciclo infinito che incomincia un ciclo infinito, in modo esponenziale.