public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Bill Davidsen <davidsen@tmr.com>
To: Jon Smirl <jonsmirl@gmail.com>
Cc: Mike Mestnik <cheako911@yahoo.com>,
	Dave Airlie <airlied@linux.ie>,
	dri-devel@lists.sourceforge.net,
	lkml <linux-kernel@vger.kernel.org>
Subject: Re: Merging DRM and fbdev
Date: Mon, 04 Oct 2004 15:48:00 -0400	[thread overview]
Message-ID: <4161A8F0.4040402@tmr.com> (raw)
In-Reply-To: <9e47339104100311566f66eb43@mail.gmail.com>

Jon Smirl wrote:
> On Sun, 3 Oct 2004 11:38:39 -0700 (PDT), Mike Mestnik
> <cheako911@yahoo.com> wrote:
> 
>>What about moving the DRM and FB specific code into there own per card
>>libs?
>>
>>radeon - attached to hardware
>>   radeon-drm
>>      drm - library
>>   radeon-fb
>>      fb - library
>>         fbcon - library
> 
> 
> Fell free to convert the merged radeon driver in to a driver plus two
> libs if you want. I'll accept the patch back. You'll need to wait
> until I get the merged driver working.
> 
> What I don't want is two independent implementations of the hardware
> initialization code like we currently have. The point of merging is to
> make sure that a single logical driver programs the hardware is a
> consistent way.
> 
> We spend so much time talking about splitting the radeon driver into
> pieces. But I don't hear anyone saying I can't ship my product because
> the radeon driver is 120K and all I can handle is 60K. I'm not going
> to spend a week's work breaking things up and testing them just
> because of some theoretical need for a non-existant embedded system.
> When this hypothetical embedded system shows up the people making the
> money off from the system can do the work.

Perhaps there might be some feedback from the embedded folks and/or 
those who decide if these changes are what they want to go in the 
kernel. If you're going to do something like this, one of the embedded 
vendors might want to contribute to development. Clearly smaller 
software parts have advantages, if resources were available to do it 
split as part of the modification.

That would probably reduce the maintenence effort in the future as well.

-- 
    -bill davidsen (davidsen@tmr.com)
"The secret to procrastination is to put things off until the
  last possible moment - but no longer"  -me

  reply	other threads:[~2004-10-04 19:49 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-10-03  3:55 Merging DRM and fbdev Jon Smirl
2004-10-03  7:26 ` Dave Airlie
2004-10-03 15:33   ` Jon Smirl
2004-10-03 15:50     ` Vladimir Dergachev
2004-10-03 16:24       ` Jon Smirl
2004-10-03 16:59         ` Vladimir Dergachev
2004-10-03 17:55           ` Jon Smirl
2004-10-03 20:37             ` Vladimir Dergachev
2004-10-03 22:42               ` Jon Smirl
2004-10-03 22:16                 ` Alan Cox
2004-10-03 22:19       ` Alan Cox
2004-10-04  1:56         ` Vladimir Dergachev
2004-10-03 18:38     ` Mike Mestnik
2004-10-03 18:56       ` Jon Smirl
2004-10-04 19:48         ` Bill Davidsen [this message]
2004-10-03 16:46   ` Jon Smirl
2004-10-04 17:47     ` Tonnerre
2004-10-04 18:42       ` Jon Smirl

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=4161A8F0.4040402@tmr.com \
    --to=davidsen@tmr.com \
    --cc=airlied@linux.ie \
    --cc=cheako911@yahoo.com \
    --cc=dri-devel@lists.sourceforge.net \
    --cc=jonsmirl@gmail.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