Re: 2.6.13-mm2

[Antonino A. Daplas]

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@xxxxxxx>
>>
>> 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@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/