On ti, 2016-08-09 at 11:46 +0100, Chris Wilson wrote: > On Tue, Aug 09, 2016 at 01:39:02PM +0300, Joonas Lahtinen wrote: > > > > On su, 2016-08-07 at 15:45 +0100, Chris Wilson wrote: > > > @@ -5327,7 +5320,7 @@ static const struct drm_info_list i915_debugfs_list[] = { > > > {"i915_capabilities", i915_capabilities, 0}, > > > {"i915_gem_objects", i915_gem_object_info, 0}, > > > {"i915_gem_gtt", i915_gem_gtt_info, 0}, > > > - {"i915_gem_pinned", i915_gem_gtt_info, 0, (void *) PINNED_LIST}, > > > + {"i915_gem_pinned", i915_gem_gtt_info, 0, 0}, > > "i915_gem_pin_display" then? Otherwise it's a fragile change. > Sure. Sold with that change? Also remove the "i915_gem_gtt", alias and rename the function appropriately, with those; Reviewed-by: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx> > -Chris > -- Joonas Lahtinen Open Source Technology Center Intel Corporation _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx