Sólo hay que cambiar `.val` por `.value`.

<!-- begin snippet: js hide: false console: true babel: false -->

<!-- language: lang-js -->

    function valor() {
        var n1 = document.getElementById("n1").value;
        var n2 = document.getElementById("n2").value;
        var resultado = parseInt(n1, 10) + parseInt(n2, 10);
        document.getElementById("resultado").value = resultado;
    }

<!-- language: lang-html -->

    <input name="n1" type="text" id="n1" size="4" maxlength="5" />
    <input name="n1" type="text" id="n2" size="4" maxlength="5" />
    <input name="suma" type="button" onClick="valor()" value="suma" />
    <input type="text" id="resultado" />

<!-- end snippet -->

**Nota:** Además, en `parseInt()` se debería poner siempre la base (10).  
Por ejemplo, `parseInt(n1, 10)`.