Date: Sun, 14 Jun 2009 19:22:35 -0700
From: Greg KH <greg@kroah.com>
To: Dave Airlie <airlied@linux.ie>
Cc: torvalds@linux-foundation.org, dri-devel@lists.sf.net,
       linux-kernel@vger.kernel.org
Subject: Re: [git pull] drm - fixes + radeon KMS (part 2)
Message-ID: <20090615022235.GA12905@kroah.com>
References: <alpine.DEB.2.00.0906150302250.20307@skynet.skynet.ie>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <alpine.DEB.2.00.0906150302250.20307@skynet.skynet.ie>
User-Agent: Mutt/1.5.19 (2009-01-05)
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1412
Lines: 34

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?

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/