On Fri, May 29, 2015 at 05:44:07PM +0100, John.C.Harrison@xxxxxxxxx wrote: > -int intel_ring_begin(struct intel_engine_cs *ring, > +int intel_ring_begin(struct drm_i915_gem_request *req, > int num_dwords) > { > - struct drm_i915_gem_request *req; > - struct drm_i915_private *dev_priv = ring->dev->dev_private; > + struct intel_engine_cs *ring; > + struct drm_i915_private *dev_priv; > int ret; > > + WARN_ON(req == NULL); > + ring = req->ring; What was the point? -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx