linux-fbdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Antonino A. Daplas" <adaplas@hotpop.com>
To: Petr Stehlik <pstehlik@sophics.cz>,
	linux-fbdev-devel@lists.sourceforge.net
Cc: Geert Uytterhoeven <geert@linux-m68k.org>,
	"Christian T. Steigies" <cts@debian.org>
Subject: Re: patch fixing background next to penguin logo in monochrome
Date: Wed, 11 Aug 2004 22:23:11 +0000 (UTC)
Date: Sun, 12 Sep 2004 06:16:41 +0800	[thread overview]
Message-ID: <200409120616.43518.adaplas@hotpop.com> (raw)
In-Reply-To: <1092254183.4209.16.camel@joy.home>

On Thursday 12 August 2004 03:56, Petr Stehlik wrote:
> Hi all,
>
> I have developed a simple workaround for a problem existing in
> framebuffer when it interprets VGA attribute bits in monochrome mode.
> Instead of empty chars we get underlines - you can see what I mean at
> http://joy.sophics.cz/horizlines.jpg - the image shows the background
> where the penguin logo will appear during linux kernel boot up. The logo
> then looks like incorrectly drawn (due to the horizontal lines) and
> people are complaining or reporting this as a sign of a kernel problems
> with their hardware setup or something.

During the take_over_console() part in drivers/char/console.c, the
the framebuffer is initialized first before the character attributes are
updated. So, during the framebuffer initialization, when the space for
the logo is created, the space is erased using vc->vc_video_erase_char.  And
because the attributes are not updated yet, the vc_video_erase_char has still
the attributes of the initial console. If the initial console happens to be
color capable (vc->vc_can_do_color != 0), then the attributes will be 
misinterpreted when the monochrome console redraws the space for the logo.
You get the underline.

Anyway, the simplest solution is to use a vc->vc_video_erase_char with the
underline attribute removed.  Check drivers/video/fbcon.c and look at the
function fbcon_setup. Note, the logo space is erased using the scr_memsetw
function with conp->vc_video_erase_char. Just replace them with
(conp->vc_video_erase_char & ~0x400) if conp->can_do_color is false. (0x400
is the underline attribute). 

I haven't tried this in 2.4, but a similar fix is already present in the latest 2.6 rc
and mm tree. (I still need to add the bold attribute support though).

Tony




-------------------------------------------------------
SF.Net email is sponsored by Shop4tech.com-Lowest price on Blank Media
100pk Sonic DVD-R 4x for only $29 -100pk Sonic DVD+R for only $33
Save 50% off Retail on Ink & Toner - Free Shipping and Free Gift.
http://www.shop4tech.com/z/Inkjet_Cartridges/9_108_r285

  reply	other threads:[~2004-08-11 22:18 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-08-11 19:56 patch fixing background next to penguin logo in monochrome Petr Stehlik
2004-08-11 22:23 ` Antonino A. Daplas [this message]
2004-08-12 12:02   ` Petr Stehlik
2004-08-12 20:01     ` Geert Uytterhoeven
2004-08-12 22:30       ` Antonino A. Daplas
2004-08-12 23:53         ` Antonino A. Daplas
2004-08-13  8:05           ` Geert Uytterhoeven
2004-08-15  0:43             ` Antonino A. Daplas
2004-08-15 12:25         ` Geert Uytterhoeven
2004-08-16  3:11           ` Antonino A. Daplas
2004-08-17  7:45             ` Geert Uytterhoeven

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=200409120616.43518.adaplas@hotpop.com \
    --to=adaplas@hotpop.com \
    --cc=adaplas@pol.net \
    --cc=cts@debian.org \
    --cc=geert@linux-m68k.org \
    --cc=linux-fbdev-devel@lists.sourceforge.net \
    --cc=pstehlik@sophics.cz \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).