Hola:
Supongo que ya funciona (acabo de probarlo...)
Un par de apuntes para que simplifiques el código:
Si pones como parámetro el elemento (o el formulario) puedes simplificar la sintaxis y sería fácil re-utilizarlo:
onchange="opcion(this)" -> el parámetro es el select...
onchange="opcion(this.form)" -> el parámetro es el formulario...
Luego en la definición para referenciar al formulario sería
function opcion(f) {
// si f es el select, el formulario sería f.form... y el elemento cm se obtendría con f.form.cm
// si f es el form el elemento cm sería f.cm
Y la función se simplificaría pasándole el formulario como parámetro (onchange="opcion.this.form"):
function opcion(f) {
f.cm.readOnly = f.cp.readOnly = (f.camion.value != "0")
}
Saludos