Re: 2.6.13-mm2

Antonino A. Daplas
2005-09-10T20:22:48Z

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