... en sí la llamada a tu función esta bien... habrá que verla para ver qué hace... además de que supongo no estás pasando los parametros de forma correcta ¿qué son p1 y p2?... quiero adivinar a que son campos de exto... así que quizá puedas agregarles una id y ocupar getElementById()...
Código:
<input type=" text" name="p1" id= "p1">
<input type="button" onClick="alert(getElementById('p1').value)">
... espero te sirva. Suerte!
