On ke, 2016-07-27 at 11:34 +0100, Chris Wilson wrote: > 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 :| Well, we'll live with that, too. Regards, Joonas > -Chris > -- Joonas Lahtinen Open Source Technology Center Intel Corporation _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
