linux-fbdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Ville Syrjälä" <syrjala@sci.fi>
To: linux-fbdev-devel@lists.sourceforge.net
Subject: Re: [PATCH 06/20] vt8623fb: New framebuffer	driver for VIA VT8623
Date: Sun, 6 May 2007 04:36:17 +0300	[thread overview]
Message-ID: <20070506013617.GA29760@sci.fi> (raw)
In-Reply-To: <20070505233834.GA19120@skynet.be>

On Sun, May 06, 2007 at 01:38:34AM +0200, Luc Verhaegen wrote:
> On Sun, May 06, 2007 at 01:31:47AM +0200, Luc Verhaegen wrote:
> > On Sun, May 06, 2007 at 07:07:37AM +0800, Antonino A. Daplas wrote:
> > > 
> > > Yep, just looking at that patch, it will be a difficult one to clean up.
> > > 
> > > Tony
> > 
> > It takes a special kind of idiot to take something like that on and 
> > keep it up too.
> 
> So you're better off with the rudimentary driver, at least that can be 
> fixed on reasonable terms.

It would be nice to have a clean driver with enough features so
people don't have to hunt around in a maze of different driver versions
to get the features they need. It's a shame people have just forked the
driver several times without really trying to work on a common codebase
and get it included in the kernel.

I think a big portion of the blame falls on VIA since they are the
original authors and the they have decided not to work with the community.
Maybe that's changing now that they posted the patches (even though they
aren't clean). I didn't really read the viafb patch thread so I missed the
details. Are they are prepared to clean it up or do they just expect to
dump the code here and forget about it?

-- 
Ville Syrjälä
syrjala@sci.fi
http://www.sci.fi/~syrjala/

-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/

  reply	other threads:[~2007-05-06  1:36 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-05-05 14:49 [PATCH 06/20] vt8623fb: New framebuffer driver for VIA VT8623 Antonino A. Daplas
2007-05-05 18:15 ` Ville Syrjälä
2007-05-05 20:08   ` Ondrej Zajicek
2007-05-05 20:34     ` Ville Syrjälä
2007-05-05 22:48       ` Luc Verhaegen
2007-05-05 23:07         ` Antonino A. Daplas
2007-05-05 23:31           ` Luc Verhaegen
2007-05-05 23:38             ` Luc Verhaegen
2007-05-06  1:36               ` Ville Syrjälä [this message]
2007-05-06  2:07                 ` Luc Verhaegen
2007-05-05 23:03       ` Antonino A. Daplas
2007-05-08  6:28 ` Andrew Morton
2007-05-08 12:28   ` [PATCH] vt8623fb: Fix compile error if CONFIG_MTRR=n Antonino A. Daplas

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=20070506013617.GA29760@sci.fi \
    --to=syrjala@sci.fi \
    --cc=linux-fbdev-devel@lists.sourceforge.net \
    /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).