Dee Sharpe
Sent from my iPhone
Begin forwarded message:
From: Dee Sharpe <[EMAIL PROTECTED]>
Date: June 2, 2008 8:05:41 PM CDT
To: Eric Anholt <[EMAIL PROTECTED]>
Subject: Re: in-kernel DRM tree move around....
Please don't get rid of the shared-core directory. Having that
visual separation makes it easier to understand the codebase for
developers who want to port DRM to other (non-*nix) platforms. I am
in the process of porting it over to Syllable & I find the 'core'
directory structure very helpful. I've even created a syllable-core
directory in my dev tree. Just my $0.02.
Dee Sharpe
Sent from my iPhone
On May 29, 2008, at 2:41 PM, Eric Anholt <[EMAIL PROTECTED]> wrote:
On Thu, 2008-05-29 at 10:46 +1000, Dave Airlie wrote:
Hi,
So I've been growing more annoyed with the current layout of the drm
tree in the kernel,
a) it lives under char.
b) everything in one directory.
c) header files in one directory.
d) no header files exposed to userspace.
http://git.kernel.org/?p=linux/kernel/git/airlied/drm-2.6.git;a=commitdiff;h=7df9a948d0f849466e3de259cccb49bc54cbad68
is a proposal to create drivers/gpu/drm, (I may move AGP in there as
well later). It also creates per-driver subdirs.
This to me solves the a-d problems.
There is also an out-of-tree kernel drm which I'll probably give
some
more thought later.
I like it.
For upstream DRM, my first thought was something like:
drm/src/i915/i915_*.[ch]
drm/src/r128/r128_*.[ch]
drm/src/drm.h
drm/src/bsd/drm_*.[ch]
drm/src/bsd/Makefile
drm/src/bsd/i915/Makefile
drm/src/bsd/r128/Makefile
drm/src/linux/drm_*.[ch]
drm/src/linux/Makefile
Get rid of the -core absurdity, and move the shared code to top-
level of
the kernel instead of alongside other kernel code.
But maybe nicer for merging on both of our parts would be:
drm/linux/Makefile
drm/linux/drm*.[ch]
drm/linux/i915/Makefile
drm/linux/i915/i915_*.[ch]
drm/linux/i915/Makefile
drm/linux/r128/r128_*.[ch]
drm/bsd/drm.h -> ../linux/drm.h
drm/bsd/drm*.[ch]
drm/bsd/Makefile
drm/bsd/i915/Makefile
drm/bsd/i915/i915_drv.c
drm/bsd/i915/i915_*.[ch] -> ../../linux/i915/i915_*.[ch]
drm/bsd/r128/Makefile
drm/bsd/r128/r128_drv.c
drm/bsd/r128/r128_*.[ch] -> ../../linux/r128/r128_*.[ch]
--
Eric Anholt [EMAIL PROTECTED]
[EMAIL PROTECTED] [EMAIL PROTECTED]
---
---
-------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
--
_______________________________________________
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel
-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
--
_______________________________________________
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel
