On Mon, 26 Mar 2012 09:45:41 +0200, Daniel Vetter <daniel.vetter at ffwll.ch> wrote: > So don't call it like that. > > Also rip out a confusing comment and instead explain what's really > going on. > > Signed-Off-by: Daniel Vetter <daniel.vetter at ffwll.ch> > --- > +/* > + * This is really ugly: Because old userspace abused the linux agp interface to > + * manage the gtt, we need to claim that all intel devices are agp. For > + * otherwise the drm core refuses to initialize the agp support code. > */ So no new device entries, right? For the patch itself, Reviewed-by: Chris Wilson <chris at chris-wilson.co.uk> -Chris -- Chris Wilson, Intel Open Source Technology Centre