Re: CVS commit: syssrc/sys/dev

[Bang Jun-Young <junyoung%mogua.com@localhost>]

On Wed, Jun 26, 2002 at 12:07:44AM +0300, Matthias Drochner wrote:
> 
> Module Name:  syssrc
> Committed By: drochner
> Date:         Tue Jun 25 21:07:44 UTC 2002
> 
> Modified Files:
>       syssrc/sys/dev/ic: vga.c vgavar.h
>       syssrc/sys/dev/isa: vga_isa.c
>       syssrc/sys/dev/pci: vga_pci.c
> 
> Log Message:
> allow to overwrite the builtin VGA font if necessary,
> make the number of available font slots variable,
> set up a "quirk" mechanism to tell the generic vga code about crippled
> VGA adapters which ignore the "fontsel" TS register,
> initiate the quirk table with an ATI chip which happened to be on a board
> I tested with.
> Afaik quite a number of ATI chips suffers from the "loaded fonts don't
> work" problem - these should be added.
> Bad side effect of my change: The builtin font will be kicked out
> always if a VGA_CONSOLE_SCREENTYPE is specified which needs a loaded
> font. In early console initialization, we don't know much about the
> graphics card, so we have to assume the worst (ie ATI:-).

Hmm...this doesn't look good to me. IMHO, my old patch works better.

What I was considering is to add `options ATI_BROKEN_FONTSEL' (or
better name) to config files and make the patch compilable optionally
for ATI users. The patch doesn't do any harm to non-ATI graphics cards,
just making screen switching a bit slower. A number of people 
reported me that it worked fine.

If you're okay, I'll commit my patch.

Jun-Young

-- 
Bang Jun-Young <junyoung%mogua.com@localhost>