Re: [Intel-gfx] [PATCH 05/22] drm/i915: enable force wake, disable LLC on VLV
This has been discussed elsewhere, but I summarize here for transparency: This makes some of the display registers take the NEEDS_FORCE_WAKE path in reg reads and writes, breaking things. Rebased on top of recent drm-intel-next-queued with Ville's IS_DISPLAYREG nuking commits this works all right, though. BR, Jani. On Sat, 02 Feb 2013, Jesse Barnes wrote: > Signed-off-by: Jesse Barnes > --- > drivers/gpu/drm/i915/i915_drv.c |4 > 1 file changed, 4 insertions(+) > > diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c > index 13b9b4f..b35b479 100644 > --- a/drivers/gpu/drm/i915/i915_drv.c > +++ b/drivers/gpu/drm/i915/i915_drv.c > @@ -276,6 +276,8 @@ static const struct intel_device_info > intel_valleyview_m_info = { > .has_bsd_ring = 1, > .has_blt_ring = 1, > .is_valleyview = 1, > + .has_force_wake = 1, > + .has_llc = 0, > }; > > static const struct intel_device_info intel_valleyview_d_info = { > @@ -285,6 +287,8 @@ static const struct intel_device_info > intel_valleyview_d_info = { > .has_bsd_ring = 1, > .has_blt_ring = 1, > .is_valleyview = 1, > + .has_force_wake = 1, > + .has_llc = 0, > }; > > static const struct intel_device_info intel_haswell_d_info = { > -- > 1.7.9.5 > > ___ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/intel-gfx ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx
Re: [Intel-gfx] [PATCH 05/22] drm/i915: enable force wake, disable LLC on VLV
On Sat, Feb 02, 2013 at 01:56:09PM +0100, Jesse Barnes wrote: > Signed-off-by: Jesse Barnes > --- > drivers/gpu/drm/i915/i915_drv.c |4 > 1 file changed, 4 insertions(+) > > diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c > index 13b9b4f..b35b479 100644 > --- a/drivers/gpu/drm/i915/i915_drv.c > +++ b/drivers/gpu/drm/i915/i915_drv.c > @@ -276,6 +276,8 @@ static const struct intel_device_info > intel_valleyview_m_info = { > .has_bsd_ring = 1, > .has_blt_ring = 1, > .is_valleyview = 1, > + .has_force_wake = 1, > + .has_llc = 0, > }; > > static const struct intel_device_info intel_valleyview_d_info = { > @@ -285,6 +287,8 @@ static const struct intel_device_info > intel_valleyview_d_info = { > .has_bsd_ring = 1, > .has_blt_ring = 1, > .is_valleyview = 1, > + .has_force_wake = 1, > + .has_llc = 0, > }; It looks like we never set these flags to 0 explicitly. It's using named initializers so the rest gets initialized to 0 anyway. > > static const struct intel_device_info intel_haswell_d_info = { > -- > 1.7.9.5 > > ___ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Ville Syrjälä Intel OTC ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx
[Intel-gfx] [PATCH 05/22] drm/i915: enable force wake, disable LLC on VLV
Signed-off-by: Jesse Barnes --- drivers/gpu/drm/i915/i915_drv.c |4 1 file changed, 4 insertions(+) diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c index 13b9b4f..b35b479 100644 --- a/drivers/gpu/drm/i915/i915_drv.c +++ b/drivers/gpu/drm/i915/i915_drv.c @@ -276,6 +276,8 @@ static const struct intel_device_info intel_valleyview_m_info = { .has_bsd_ring = 1, .has_blt_ring = 1, .is_valleyview = 1, + .has_force_wake = 1, + .has_llc = 0, }; static const struct intel_device_info intel_valleyview_d_info = { @@ -285,6 +287,8 @@ static const struct intel_device_info intel_valleyview_d_info = { .has_bsd_ring = 1, .has_blt_ring = 1, .is_valleyview = 1, + .has_force_wake = 1, + .has_llc = 0, }; static const struct intel_device_info intel_haswell_d_info = { -- 1.7.9.5 ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx