[Intel-gfx] [PATCH] drm/i915/chv: WaDisablePwrmtrEvent:chv on CHV only
Ville Syrjälä
ville.syrjala at linux.intel.com
Tue Jun 10 19:16:28 CEST 2014
On Tue, Jun 10, 2014 at 06:34:18PM +0300, Jani Nikula wrote:
> On Tue, 10 Jun 2014, Jani Nikula <jani.nikula at linux.intel.com> wrote:
> > On Mon, 09 Jun 2014, Damien Lespiau <damien.lespiau at intel.com> wrote:
> >> On Mon, Jun 09, 2014 at 10:06:49AM -0700, Tom.O'Rourke at intel.com wrote:
> >>> From: Tom O'Rourke <Tom.O'Rourke at intel.com>
> >>>
> >>> In gen8_enable_rps, don't write CHV registers unless IS_CHERRYVIEW.
> >>>
> >>> Signed-off-by: Tom O'Rourke <Tom.O'Rourke at intel.com>
> >>
> >> A lovely catch.
> >
> > Sadly gen8_enable_rps does not get called on chv, so the fix is wrong.
>
> To elaborate, I think we need a patch dropping the wa altogether (which
> we can queue for 3.15 through stable because the change affects
> broadwell) and another patch, if needed, adding the wa in the chv
> specific function.
This is just a merge mishap in one the chv patches. Someone just
needs to send a patch that moves the misapplied stuff to the
appropriate chv function.
--
Ville Syrjälä
Intel OTC
More information about the Intel-gfx
mailing list