Re: [PATCH] drm/kms/radeon: Add kconfig to specify the default modesetting

[Takashi Iwai <tiwai@suse.de>]

At Fri, 21 Aug 2009 10:52:13 +1000,
Dave Airlie wrote:
> 
> On Thu, Aug 20, 2009 at 11:26 PM, Takashi Iwai<tiwai@suse.de> wrote:
> > This patch adds a Kconfig item to specify the default radeon modeset
> > value.  Also fixed the description of CONFIG_DRM_RADEON_KMS.
> >
> > Since KMS gives problems on some Radeon devices, it's safer to disable
> > it as default on distro kernels.  But distros would love to include the
> > feature itself instead of disabling KMS completely.
> >
> > Reference: Novell bnc#527910
> >        https://bugzilla.novell.com/show_bug.cgi?id=527910
> 
> I'm sort of torn on this, when radeon KMS moves out of staging it will
> be disabled by
> default using the current option. It will build everything always and
> disable/enable like this patch
> does. I'm not willing to do that yet esp now in -rcX time as it might
> break builds
> on some other arches.

Yeah, it's a bit late for this kind of change; I share your concern,
too.

> So if a distro isn't shipping a compatible userspace, I'd rather they
> didn't ship
> the kernel support built at all until we move it from staging.
> 
> AFAIK you guys don't use -ati on a lot of cards kms is useful so I'm
> not sure when you'll
> ever have a compatible userspace.

Well, a lack of dynamic switch -- or inconsistent behavior (i915
is built with KMS but not enabled as default) is the problem right
now.  We can live without radeon KMS, but it would be nice if we can
still provide the functionality in any form...


thanks,

Takashi