Hi, I stumbled on this in k_init.c: nvmode = vcheckmode(cfg.videomode); if ((nvmode & (1 << 4)) && /* VGA_FALCON */ (nvmode & 7) == 4) /* is 16bit */ { nvmode &= ~(1 << 3); /* Set 320 pixels */ } work_out[45] = nvmode; mode = 5; The (limited) documentation I've found for Vcheckmode() says: - "modify video mode to match the possibilities of the attached monitor." (milan development docs) My impression is that the following code is redundant, since this should already be catered for in Vcheckmode(): if ((nvmode & (1 << 4)) && /* VGA_FALCON */ (nvmode & 7) == 4) /* is 16bit */ { nvmode &= ~(1 << 3); /* Set 320 pixels */ }
|