public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Thomas Winischhofer <tw@webit.com>
To: linux-kernel@vger.kernel.org
Subject: @Linus, Marcello, (Alan?) (regards sisfb)
Date: Fri, 21 Dec 2001 15:40:08 +0100	[thread overview]
Message-ID: <3C2349C8.1DF70E5F@falke.mail> (raw)


Hi, as you might know, the sisfb driver did not work correctly on
systems with SiS630 and LCD panels, i.e laptops. The problem was the
type of video bridge used, which is LVDS in most cases.

The current driver contained in the kernel can't even work in theory, 
because 

1) the tables that should have contained necessary data for refresh and
timing were empty, and

2) this data depends on the type of LCD panels attached and is not the
same on all machines.

So, the whole theory with this driver is a failure.

I therefore re-wrote (and commented) the sisfb driver to read out the
BIOS data, instead of its very own tables. On non-SiS630 or non-LVDS
systems, the old code is used.

>From the feedback I got, I know that the driver is working in about 50%
of the SiS630-LVDS-machines out there, depending on what revision of the
SiS630 and LCD panel being used. This is 50% more than before!

The driver has been tested on systems without a video bridge as well
(ie. desktop machines) and it works there as well. This means I didn't
break it for other configurations.

The dark side: The driver is - when used on SiS630 and LVDS - not
LinuxBIOS-capable (because it depends on data stored in the SiS-BIOS)

Are you willing to include the new driver in the kernel?

It's available here: http://members.aon.at/~twinisch/sisfb.tar.gz

I have not made a patch, because this patch would be about as twice as
huge than the whole code. But if you insist, I could make a patch as
well. The code is based on 2.4.16. AFAIK there were no changes to sisfb
up to current 17-rc2. The archive is to be extracted over the existing
code while in /usr/src/linux/drivers/video/sis/

Thomas

-- 
Thomas Winischhofer
Vienna/Austria                  Check it out:
mailto:tw@webit.com              http://www.webit.com/tw


             reply	other threads:[~2001-12-21 14:53 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-12-21 14:40 Thomas Winischhofer [this message]
2001-12-21 15:50 ` @Linus, Marcello, (Alan?) (regards sisfb) Alan Cox
2001-12-21 16:31   ` Thomas Winischhofer
2001-12-21 16:56     ` Alan Cox
2001-12-25 17:02 ` Thomas Winischhofer
2001-12-26 11:49   ` bert hubert

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=3C2349C8.1DF70E5F@falke.mail \
    --to=tw@webit.com \
    --cc=linux-kernel@vger.kernel.org \
    /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