lkml.org 
[lkml]   [2010]   [Apr]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 10/11] viafb: rework the I2C support in the VIA framebuffer driver
Jonathan Corbet schrieb:
> On Fri, 23 Apr 2010 23:12:54 +0200
> Florian Tobias Schandinat <FlorianSchandinat@gmx.de> wrote:
>
>> 1. while the module loads the screen shows some strange colours. The
>> colours itself do not worry me but it probably at least means that the
>> module load time was increased significantly (2-3 seconds, I guess)
>>
>> 2. most annoyingly after this patch I can no longer get a working VGA
>> console after unbinding the module. Normally I just do:
>
> Hmm...compiling the driver as a module is not really something that
> makes sense on the OLPC platform, so I've never done it.
>
> #1 is weird, there should be nothing there that slows initialization
> down. Putting in some prints and looking at the timing would be most
> helpful here.

Actually that is probably a mistake on my side. I had the impression
that it was much longer but didn't take into account that the old
behaviour allowed the VGA console to work until viafb was completly
loaded and fbcon took over while the new one immediately destroys the
screen and shows random things until it is completely loaded. However
the time between hitting enter for the module load command and getting a
working fbcon really is at least nearly the same.

> As for #2, I can certainly say that I've never unloaded this code, so
> that's an untested path. I'll have a look and see if I can see
> anything obvious.

Well as for the behaviour change described above I think the problem
might be in the load path. At least when I faked an exit as when memory
size detection or ioremapping fails (which is a very common issue that
really needs a workaround) I get the same unusable VGA console. This
needs to be fixed.
This whole I2C stuff seems incredibly unstable I even have indicators
that the current code might be to blame for freezing the machine on some
configurations with P4M900 IGP. I just try to prevent it to get even
worse...


Thanks,

Florian Tobias Schandinat


\
 
 \ /
  Last update: 2010-04-24 00:43    [W:0.077 / U:0.464 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site