Re: [Intel-gfx] [PATCH v2] drm/i915/gvt: clean up intel_gvt.h as interface for i915 core

2016-10-20 Thread Zhenyu Wang
On 2016.10.20 09:12:02 +0100, Chris Wilson wrote: > On Thu, Oct 20, 2016 at 04:02:39PM +0800, Zhenyu Wang wrote: > > void intel_gvt_clean_device(struct drm_i915_private *dev_priv) > > { > > - struct intel_gvt *gvt = _priv->gvt; > > + struct intel_gvt *gvt = to_gvt(dev_priv); > > > > if

Re: [Intel-gfx] [PATCH v2] drm/i915/gvt: clean up intel_gvt.h as interface for i915 core

2016-10-20 Thread Chris Wilson
On Thu, Oct 20, 2016 at 04:02:39PM +0800, Zhenyu Wang wrote: > void intel_gvt_clean_device(struct drm_i915_private *dev_priv) > { > - struct intel_gvt *gvt = _priv->gvt; > + struct intel_gvt *gvt = to_gvt(dev_priv); > > if (WARN_ON(!gvt->initialized)) > return; > @@

[Intel-gfx] [PATCH v2] drm/i915/gvt: clean up intel_gvt.h as interface for i915 core

2016-10-20 Thread Zhenyu Wang
i915 core should only call functions and structures exposed through intel_gvt.h. Remove internal gvt.h and i915_pvinfo.h. Change for internal intel_gvt structure as private handler which not requires to expose gvt internal structure for i915 core. v2: Fix per Chris's comment - carefully handle