>
> Acked-by: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx>
> Reviewed-by: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx>
>
>
>
> On Tue, 2016-12-13 at 18:57 -0200, Paulo Zanoni wrote:
> > We've been ignoring the poor bugzilla reporters that say PSR causes
> > system lockups and all other sorts of problems. The earliest bug
> > report is from April, so I think we can use the "revert the offending
> > commit if no fixes are presented within 8 months" rule here.
> >
Would it means that, since we cannot fix and disable it by default then PSR is going to be deprecated ? I recall that on reported bugs we pay attention whether or not default parameter values is used or not to proceed on fix.
-Yann
> > Fixes: 9b58e352b463 ("drm/i915: Enable PSR by default on Haswell and
> > Broadwell.")
> > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97602
> > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97515
> > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=96736
> > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=96704
> > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=96569
> > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=95176
> > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=94985
> > Cc: <stable@xxxxxxxxxxxxxxx> # v4.6+
> > Cc: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx>
> > Cc: Jim Bride <jim.bride@xxxxxxxxxxxxxxx>
> > Signed-off-by: Paulo Zanoni <paulo.r.zanoni@xxxxxxxxx>
> > ---
> > drivers/gpu/drm/i915/intel_psr.c | 10 +++-------
> > 1 file changed, 3 insertions(+), 7 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/i915/intel_psr.c
> > b/drivers/gpu/drm/i915/intel_psr.c
> > index d5f8d03..6aca8ff 100644
> > --- a/drivers/gpu/drm/i915/intel_psr.c
> > +++ b/drivers/gpu/drm/i915/intel_psr.c
> > @@ -823,13 +823,9 @@ void intel_psr_init(struct drm_i915_private
> *dev_priv)
> > dev_priv->psr_mmio_base = IS_HASWELL(dev_priv) ?
> > HSW_EDP_PSR_BASE : BDW_EDP_PSR_BASE;
> >
> > - /* Per platform default */
> > - if (i915.enable_psr == -1) {
> > - if (IS_HASWELL(dev_priv) || IS_BROADWELL(dev_priv))
> > - i915.enable_psr = 1;
> > - else
> > - i915.enable_psr = 0;
> > - }
> > + /* Per platform default: all disabled. */
> > + if (i915.enable_psr == -1)
> > + i915.enable_psr = 0;
> >
> > /* Set link_standby x link_off defaults */
> > if (IS_HASWELL(dev_priv) || IS_BROADWELL(dev_priv))
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
