Hi,
vergiss das hoffnungslos veraltete all-Objekt und sprich das p-Tag über die ID an:
document.getElementById("counter").innerHTML = ...
Wobei Dir bei der innerHTML-Methode klar sein muss, dass dieses input-Feld dann futsch ist. Dieser Mischmasch sieht irgendwie so aus, als wüsstest Du selbst nicht genau, wie das aussehen soll...
LG