Re: fb_console_init fix.

[Benjamin Herrenschmidt <benh@kernel.crashing.org>]

On Sat, 2004-02-21 at 08:58, James Simmons wrote:
> Hi!
> 
>   This patch fixes fb_console_init from being called twice. I still need 
> to fix set_con2fb but this helps but this is still important to get in.

I hate those #ifdef MODULE. They are plaguing the drivers too, because
of our crazy initialization scheme...

(Linus: that patch should still go in, doing better is probably a
2.7 target in this case).

If we want the fb stuff to be initialized before other drivers, we
probably want to create an init step before devices and after PCI
probe, but is that really necessary ?

The core fbdev should just need a subsys initcall. fbcon could use
that too and register a notifier. Then we could use the notifier
mecanism to notify fbcon when fbdev's are added (from
register_framebuffer) and get rid of all of the init crap

The only remaining junk is to deal with the setup= stuff, but
then, it could be done by having the drivers provide their
"name" string & setup function in the ops structure passed
on register, and have us call back the driver straight away,
though that's a big change of the way drivers are initialized.

Or we could add an fb_check_setup("name", my_driver_setup) call
to be done by the drivers before register_framebuffer is called


Ben
.




-------------------------------------------------------
SF.Net is sponsored by: Speed Start Your Linux Apps Now.
Build and deploy apps & Web services for Linux with
a free DVD software kit from IBM. Click Now!
http://ads.osdn.com/?ad_id=1356&alloc_id=3438&op=click