Re: [STATUS] fbdev api.

From: Antonino Daplas (adaplas@pol.net)
Date: Tue Dec 03 2002 - 22:32:07 EST


On Wed, 2002-12-04 at 03:18, James Simmons wrote:
>
> > Attached is a patch against linux-2.5.50 + your fbdev.diff.
>
> Applied :-)
>
> > b. Another rewrite of fbcon_show_logo() so it's more understandable
> > (hopefully). I also added support for the rest of the visuals, but
> > untested yet.
> > Not tested:
> > static psuedocolor, mono01, and mono10.
>
> I have a mono hga card.
>
Can you apply the following patch to include logo drawing support for all formats :-)?

diff -Naur linux-2.5.50-js/drivers/video/cfbimgblt.c linux/drivers/video/cfbimgblt.c
--- linux-2.5.50-js/drivers/video/cfbimgblt.c 2002-12-04 03:14:19.000000000 +0000
+++ linux/drivers/video/cfbimgblt.c 2002-12-04 03:13:57.000000000 +0000
@@ -123,11 +123,11 @@
                         shift = start_index;
                 }
                 while (n--) {
- if (p->fix.visual == FB_VISUAL_PSEUDOCOLOR)
- color = *src & bitmask;
                         if (p->fix.visual == FB_VISUAL_TRUECOLOR ||
                             p->fix.visual == FB_VISUAL_DIRECTCOLOR )
                                 color = palette[*src] & bitmask;
+ else
+ color = *src & bitmask;
                         val |= SHIFT_HIGH(color, shift);
                         if (shift >= null_bits) {
                                 FB_WRITEL(val, dst++);

> > c. prevent fbcon module from loading if no fbdev is registered. Also
> > made fbcon module unsafe to unload (for now). This is optional, of course.
>
> It is a good idea until we have the ability to switch back to text mode.
>
It's not the switch back to text mode, that's very doable (see my other
reply). It's during give_up_console() at fbcon module exit. At this
point, the console suddenly disappears and freezes the system. Maybe we
can save the global "conswitchp" during fbcon module init, then
something like this at fbcon module exit:

void __exit fb_console_exit(void)
{
        give_up_console(&fbcon);
        take_over_console(saved_conswitchp, ...);
}

Is this feasible?

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/



This archive was generated by hypermail 2b29 : Sat Dec 07 2002 - 22:00:17 EST