<div dir="ltr">merged to dinq, thanks for the patch.</div><br><div class="gmail_quote"><div dir="ltr">On Fri, Dec 9, 2016 at 4:39 PM Rodrigo Vivi <<a href="mailto:rodrigo.vivi@gmail.com">rodrigo.vivi@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br class="gmail_msg">
<br class="gmail_msg">
Reviewed-by: Rodrigo Vivi <<a href="mailto:rodrigo.vivi@intel.com" class="gmail_msg" target="_blank">rodrigo.vivi@intel.com</a>><br class="gmail_msg">
<br class="gmail_msg">
On Fri, Dec 09, 2016 at 11:42:09PM +0530, vathsala nagaraju wrote:<br class="gmail_msg">
> For PSR2 , as per spec, PSR2_CTL bit 31 to be set.<br class="gmail_msg">
> for psr1, bit 31 in SRD_CTL to be set. Reporting<br class="gmail_msg">
> "HW Enabled & Active bit" status for psr2 from SRD_CTL<br class="gmail_msg">
> gives wrong status.<br class="gmail_msg">
><br class="gmail_msg">
> Cc: Rodrigo Vivi <<a href="mailto:rodrigo.vivi@intel.com" class="gmail_msg" target="_blank">rodrigo.vivi@intel.com</a>><br class="gmail_msg">
> Cc: Jim Bride <<a href="mailto:jim.bride@linux.intel.com" class="gmail_msg" target="_blank">jim.bride@linux.intel.com</a>><br class="gmail_msg">
> Signed-off-by: vathsala nagaraju <<a href="mailto:vathsala.nagaraju@intel.com" class="gmail_msg" target="_blank">vathsala.nagaraju@intel.com</a>><br class="gmail_msg">
> ---<br class="gmail_msg">
> drivers/gpu/drm/i915/i915_debugfs.c | 9 ++++++---<br class="gmail_msg">
> 1 file changed, 6 insertions(+), 3 deletions(-)<br class="gmail_msg">
><br class="gmail_msg">
> diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c<br class="gmail_msg">
> index a746130..54e196d 100644<br class="gmail_msg">
> --- a/drivers/gpu/drm/i915/i915_debugfs.c<br class="gmail_msg">
> +++ b/drivers/gpu/drm/i915/i915_debugfs.c<br class="gmail_msg">
> @@ -2567,9 +2567,12 @@ static int i915_edp_psr_status(struct seq_file *m, void *data)<br class="gmail_msg">
> seq_printf(m, "Re-enable work scheduled: %s\n",<br class="gmail_msg">
> yesno(work_busy(&dev_priv->psr.work.work)));<br class="gmail_msg">
><br class="gmail_msg">
> - if (HAS_DDI(dev_priv))<br class="gmail_msg">
> - enabled = I915_READ(EDP_PSR_CTL) & EDP_PSR_ENABLE;<br class="gmail_msg">
> - else {<br class="gmail_msg">
> + if (HAS_DDI(dev_priv)) {<br class="gmail_msg">
> + if (dev_priv->psr.psr2_support)<br class="gmail_msg">
> + enabled = I915_READ(EDP_PSR2_CTL) & EDP_PSR2_ENABLE;<br class="gmail_msg">
> + else<br class="gmail_msg">
> + enabled = I915_READ(EDP_PSR_CTL) & EDP_PSR_ENABLE;<br class="gmail_msg">
> + } else {<br class="gmail_msg">
> for_each_pipe(dev_priv, pipe) {<br class="gmail_msg">
> enum transcoder cpu_transcoder =<br class="gmail_msg">
> intel_pipe_to_cpu_transcoder(dev_priv, pipe);<br class="gmail_msg">
> --<br class="gmail_msg">
> 1.9.1<br class="gmail_msg">
><br class="gmail_msg">
> _______________________________________________<br class="gmail_msg">
> Intel-gfx mailing list<br class="gmail_msg">
> <a href="mailto:Intel-gfx@lists.freedesktop.org" class="gmail_msg" target="_blank">Intel-gfx@lists.freedesktop.org</a><br class="gmail_msg">
> <a href="https://lists.freedesktop.org/mailman/listinfo/intel-gfx" rel="noreferrer" class="gmail_msg" target="_blank">https://lists.freedesktop.org/mailman/listinfo/intel-gfx</a><br class="gmail_msg">
</blockquote></div>