Habilitar campos de un form al apretar un radio button

[WorseMzg]

Buneas a todos, primero soy nuevo pero siempre me guio por esta pagina asi que gracias de antemano

Estoy tratando de habilitar y deshabilitar campos al apretar un radio, según lo que entiendo el evento on click me tiene que llamar a una funcion. y es lo que trato de hacer pero no me ejecuta. aqui dejo el codigo. de antemano gracias.

Código:

<form enctype="multipart/form-data" method="post" name="formNoticias" action="">
                <table width="700" border="0" cellpadding="0" cellspacing="0" class="tbNuevaNoticia">
                 
                  <tr>
                    <td class="tituloCampo">¿Desea agregar una galeria de imagenes?</td>
                    <td class="inputCampo"><input type = "radio" name = "galeria" id = "galeria" value = "si" onclick="habilitarGaleria(si)"/>Si -<input type = "radio" name = "galeria" id = "galeria" value = "no" onclick="habilitarGaleria(no)" cheked="cheked" />No</td>
                  </tr>
                  <tr>
                    <td name="galeria" class="tituloCampo">Titulo de la galeria</td>
                    <td name="galeria" class="inputCampo"><input name="NombreGal" type="name" /></td>
                  </tr>
                  <tr>
                    <td name="galeria" class="tituloCampo" style="visibility: hidden;">Seleccione las imagenes:</td>
                    <td name="galeria" class="inputCampo" style="visibility: hidden;"><input name="uploadGaleria[]" type="file" multiple /></td>
                   </table>
            </form>

y aqui el javascript

Código:

function habilitarGaleria(tipo){

  if(tipo == "si")
  {
    document.formNoticias.galeria.style.visibility="visible";
  }
  if (tipo == "no")
  {
    document.formNoticias.galeria.style.visibility="hidden";
  }

}

me imagino que tendre varios errores de sintaxis, pero mas que nada me preocupa la logica de lo que estoy haciendo..


Se agradece