On Tue, Jun 23, 2015 at 11:46:29AM +0100, Tvrtko Ursulin wrote: > > On 06/22/2015 03:17 PM, Daniel Vetter wrote: > >On Mon, Jun 22, 2015 at 12:11:47PM +0100, Damien Lespiau wrote: > >>On Fri, Jun 19, 2015 at 08:27:27PM +0100, Chris Wilson wrote: > >>>Since we only support modesetting by default (disabling modesetting on > >>>the command line prevents i915.ko from loading), having a parameter to > >>>disable modesstting by default is superfluous, i.e. saying > >>>CONFIG_DRM_I915_KMS=n is equivalent to CONFIG_DRM_I915=n. > >>> > >>>Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > >>>Cc: Daniel Veter <daniel.vetter@xxxxxxxx> > >> > >>Reviewed-by: Damien Lespiau <damien.lespiau@xxxxxxxxx> > > > >Slight chance someone with an ums-only setup will get pissed about this. > >But surely interesting to hear about those ;-) > > i915_drv.c: > > #if defined(CONFIG_DRM_I915_KMS) > MODULE_DEVICE_TABLE(pci, pciidlist); > #endif > > Ooops! :) Indeed. Patch submitted and dinq history rectified. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
