[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20100204175928.GA20595@srcf.ucam.org>
Date: Thu, 4 Feb 2010 17:59:28 +0000
From: Matthew Garrett <mjg59@...f.ucam.org>
To: Ingo Molnar <mingo@...e.hu>
Cc: Dave Airlie <airlied@...il.com>, Dave Airlie <airlied@...ux.ie>,
torvalds@...ux-foundation.org, linux-kernel@...r.kernel.org,
dri-devel@...ts.sf.net
Subject: Re: hung bootup with "drm/radeon/kms: move radeon KMS on/off
switch out of staging."
On Thu, Feb 04, 2010 at 06:54:45PM +0100, Ingo Molnar wrote:
> But you could claim that it's not a regression because 1) technically the
> code got introduced in drivers/staging/, and staging drivers are not on the
> regression list 2) the Kconfig value is default-off so it can only harm those
> who got lured by a new Kconfig value popping up in -rc7 in a well working
> driver they already have enabled.
>
> So the moving of driver functionality from drivers/staging/ to drivers/ is a
> grey area it appears. Wouldnt it have been better to do this in the next
> merge window, as all other drivers do? It's not new hardware enablement
> either, it's feature enablement for an existing driver.
The reason the option was in staging (as has been mentioned before) was
because the ABI wasn't felt to be stable enough. Upstream is now willing
to commit to that stability, so now seems as good a time to move it as
any. There's no code change and there's no default configuration change,
so I really can't see any way that it can be classed as a regression.
--
Matthew Garrett | mjg59@...f.ucam.org
--
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