function readonly(chk) {
// Du lässt dir chk übergeben
if(chk){
// fragest es noch ab
document.getElementById(felder[i]).style.backgroundColor = "#ffffff";
document.getElementById(felder[i]).disabled = !(document.getElementById('klick').checked);
// Und holst es dir dann nocheinmal? True/False oder !chk würden es auch machen.
Ansonsten solltest du dich um die Dinge kümmern, die Martin angemerkt hat.