mov a, p1 ; mov the value of p1 to acc
push acc ; value of acc is preserved

That is obvious. Do not state the obvious. The Why is what is important

mov a, p1 ; Read the keyboard
push acc ; save the Keyboard value for later.

Is better (assuming it is correct)

A block comment stating what a section is supposed to do is desirable.
A comment on every line is not always needed.

Note that for all newbies reading this in the future:
mov a, p1 ; mov the value of p1 to acc

This is the clasic bad comment written by people force to comment their code. Do it right, You will be glad next year when you need to make changes.