[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20090616184544.GC28853@kroah.com>
Date: Tue, 16 Jun 2009 11:45:44 -0700
From: Greg KH <greg@...ah.com>
To: Dave Airlie <airlied@...il.com>
Cc: Dave Airlie <airlied@...ux.ie>, torvalds@...ux-foundation.org,
dri-devel@...ts.sf.net, linux-kernel@...r.kernel.org
Subject: Re: [git pull] drm - fixes + radeon KMS (part 2)
On Mon, Jun 15, 2009 at 12:43:38PM +1000, Dave Airlie wrote:
> On Mon, Jun 15, 2009 at 12:22 PM, Greg KH<greg@...ah.com> wrote:
> > On Mon, Jun 15, 2009 at 03:08:56AM +0100, Dave Airlie wrote:
> >>
> >> Hi Linus,
> >>
> >> Please pull the 'drm-linus' branch from
> >> ssh://master.kernel.org/pub/scm/linux/kernel/git/airlied/drm-2.6.git drm-linus
> >>
> >> This is big. It contains the initial TTM memory manager + ATI radeon KMS
> >> support. Currently the KMS code is part of the DRM radeon driver however
> >> it is very clearly separated internally from the old codepaths. We've
> >> elected to keep the radeon KMS Kconfig build/enable under staging for now
> >> since we may have some ABI tweaks to sort out in this release cycle,
> >> however the code is all in the drm. I don't think this enables crap
> >> tainting, but at least no-one will find kms by accident.
> >
> > No, the module loader looks for stuff in drivers/staging/ to cause a
> > "taint".
> >
> > But why not just keep the Kconfig stuff in your own directory, and
> > depend on CONFIG_STAGING if you want to not have it show up for "normal"
> > users? It seems odd to put anything in drivers/staging/Kconfig for
> > something that is not in drivers/staging.
> >
> > I'm guessing this Kconfig change was not in linux-next? Or had it been
> > there and I just missed it somehow?
> >
>
> No since I was on holidays until the just before the merge window
> opened, the patches
> did get posted to lkml but missed your cc.
>
> Well we'd like to make sure people go via the staging menus to get at
> the kconfig option
> for now, granted it probably doesn't matter whether it goes in staging
> menus or in drm depends
> on CONFIG_STAGING. Up to you I can post a patch after this merge to
> move it to drm.
Ok, I'll defer to your judgement here.
thanks,
greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists