On ti, 2016-04-05 at 10:26 +0200, Patrik Jakobsson wrote:
> On Mon, Apr 04, 2016 at 12:34:30PM +0200, Patrik Jakobsson wrote:
> > On Fri, Apr 01, 2016 at 04:02:36PM +0300, Imre Deak wrote:
> > > So far we only power well enabling was synchronous not disabling.
> > > Since
> > > we don't exactly know how the firmware (both DMC and PCU)
> > > synchronizes
> > > against the actual power well state during DC transitions, make
> > > the
> > > disabling also synchronous.
> > >
> > > CC: Mika Kuoppala <mika.kuopp...@linux.intel.com>
> > > CC: Patrik Jakobsson <patrik.jakobs...@linux.intel.com>
> > > Signed-off-by: Imre Deak <imre.d...@intel.com>
> >
> > Reviewed-by: Patrik Jakobsson <patrik.jakobs...@linux.intel.com>
>
> Perhaps I was too quick with the review. I'm getting timeouts when
> trying to
> disable MISC IO and PW1 on SKL. Need to have a closer look at what's
> going on
> here.
The problem is the same that I fixed already on BXT in 04/16. The
firmware doesn't properly save/restore the request bits for these power
wells. It's an existing issue, so good that we found it. I'll follow up
with an updated version of patch 4 that addresses this on SKL as well.
--Imre
>
> -Patrik
>
> >
> > > ---
> > > drivers/gpu/drm/i915/intel_runtime_pm.c | 9 +++++----
> > > 1 file changed, 5 insertions(+), 4 deletions(-)
> > >
> > > diff --git a/drivers/gpu/drm/i915/intel_runtime_pm.c
> > > b/drivers/gpu/drm/i915/intel_runtime_pm.c
> > > index d20fd8f..f5f6e89 100644
> > > --- a/drivers/gpu/drm/i915/intel_runtime_pm.c
> > > +++ b/drivers/gpu/drm/i915/intel_runtime_pm.c
> > > @@ -720,10 +720,6 @@ static void skl_set_power_well(struct
> > > drm_i915_private *dev_priv,
> > >
> > > if (!is_enabled) {
> > > DRM_DEBUG_KMS("Enabling %s\n",
> > > power_well->name);
> > > - if
> > > (wait_for((I915_READ(HSW_PWR_WELL_DRIVER) &
> > > - state_mask), 1))
> > > - DRM_ERROR("%s enable timeout\n",
> > > - power_well->name);
> > > check_fuse_status = true;
> > > }
> > > } else {
> > > @@ -737,6 +733,11 @@ static void skl_set_power_well(struct
> > > drm_i915_private *dev_priv,
> > > bxt_sanitize_power_well_requests(dev_pri
> > > v, power_well);
> > > }
> > >
> > > + if (wait_for(!!(I915_READ(HSW_PWR_WELL_DRIVER) &
> > > state_mask) == enable,
> > > + 1))
> > > + DRM_ERROR("%s %s timeout\n",
> > > + power_well->name, enable ? "enable" :
> > > "disable");
> > > +
> > > if (check_fuse_status) {
> > > if (power_well->data == SKL_DISP_PW_1) {
> > > if (wait_for((I915_READ(SKL_FUSE_STATUS)
> > > &
> > > --
> > > 2.5.0
> > >
> >
> > --
> > ---
> > Intel Sweden AB Registered Office: Knarrarnasgatan 15, 164 40
> > Kista, Stockholm, Sweden Registration Number: 556189-6027
> > _______________________________________________
> > Intel-gfx mailing list
> > Intel-gfx@lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/intel-gfx
>
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
