On Wed, Jul 27, 2016 at 01:20:42PM +0300, Joonas Lahtinen wrote:
> On ti, 2016-07-26 at 08:42 +0100, Chris Wilson wrote:
> > On Tue, Jul 26, 2016 at 10:08:32AM +0300, Joonas Lahtinen wrote:
> > > 
> > > On ma, 2016-07-25 at 18:32 +0100, Chris Wilson wrote:
> > > > 
> > > > -/**
> > > > - * i915_gem_init_ggtt - Initialize GEM for Global GTT
> > > > - * @dev: DRM device
> > > > - */
> > > > -void i915_gem_init_ggtt(struct drm_device *dev)
> > > > 
> > > > +static void init_global_gtt(struct drm_i915_private *dev_priv)
> > > Why not s/global_gtt/ggtt/ for easier grepping.
> > init_global_vm() so that you don't think it is just about the ggtt.
> 
> Agreed.

I ended up inlining it :|
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to