Messages in this thread | | | Date | Sun, 11 Sep 2005 04:21:41 +0800 | From | "Antonino A. Daplas" <> | Subject | Re: 2.6.13-mm2 |
| |
Manuel Lauss wrote: > >> Ah, yes, sorry about that. Can you try this patch? >> >> Fix kernel oops when CONFIG_FB_I810_I2C is set to 'n'. >> >> Signed-off-by: Antonino Daplas <adaplas@pol.net> >> >> diff --git a/drivers/video/i810/i810_main.c >> b/drivers/video/i810/i810_main.c >> --- a/drivers/video/i810/i810_main.c >> +++ b/drivers/video/i810/i810_main.c >> @@ -1830,7 +1830,7 @@ static void __devinit i810fb_find_init_m >> { >> struct fb_videomode mode; >> struct fb_var_screeninfo var; >> - struct fb_monspecs *specs = NULL; >> + struct fb_monspecs *specs = &info->monspecs; >> int found = 0; >> #ifdef CONFIG_FB_I810_I2C >> int i; >> @@ -1853,12 +1853,11 @@ static void __devinit i810fb_find_init_m >> if (!err) >> printk("i810fb_init_pci: DDC probe successful\n"); >> >> - fb_edid_to_monspecs(par->edid, &info->monspecs); >> + fb_edid_to_monspecs(par->edid, specs); >> >> - if (info->monspecs.modedb == NULL) >> + if (specs->modedb == NULL) >> printk("i810fb_init_pci: Unable to get Mode Database\n"); >> >> - specs = &info->monspecs; >> fb_videomode_to_modelist(specs->modedb, specs->modedb_len, >> &info->modelist); >> if (specs->modedb != NULL) { > > Thanks, it boots now, but doesnt set video mode. Spews > a bunch of "i810fb: invalid video mode" lines and defaults > to 640x480.
Set CONFIG_FB_I810_I2C to n. Your display does not have an EDID block. Even getting the EDID from the BIOS failed.
Tony - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |