From: Lyude <cpaul@redhat.com>
To: "Ville Syrjälä" <ville.syrjala@linux.intel.com>
Cc: David Airlie <airlied@linux.ie>,
Daniel Vetter <daniel.vetter@intel.com>,
intel-gfx@lists.freedesktop.org,
"open list:INTEL DRM DRIVERS (excluding Poulsbo, Moorestow...),
linux-kernel@vger.kernel.org (open list)"
<dri-devel@lists.freedesktop.org>
Subject: Re: [PATCH 0/3] Fixes for HPD
Date: Fri, 17 Jun 2016 18:04:11 -0400 [thread overview]
Message-ID: <1466201051.7789.10.camel@redhat.com> (raw)
In-Reply-To: <1466200705-9405-1-git-send-email-cpaul@redhat.com>
Forgot to mention, Ville: if you could get me an example of how to get
vlv into an infinite loop with these patches I'd appreciate that. I
haven't been able to reproduce this at all with the Valleyview machine
I've got here.
On Fri, 2016-06-17 at 17:58 -0400, Lyude wrote:
> These are a couple of patches intended to fix one of the big problems
> we have
> with a lot of chipsets on i915 right now: in the various forms of
> suspend we
> use in the driver, many of them break HPD while active and can lead
> to some
> seriously confusing situations where they can't get their monitors to
> turn on.
>
> The patches here for fixing Valleyview need to be used with Ville
> Syrjälä's
> patchset for (partially?) preventing valleyview from getting in an
> infinite hpd
> detect loop when doing polling:
>
> https://patchwork.freedesktop.org/series/5884/
>
> It should also be noted some of these are resends, since the original
> patches
> never got picked up by patchwork
>
> Lyude (3):
> drm/i915/vlv: Make intel_crt_reset() per-encoder
> drm/i915/vlv: Reset the ADPA in vlv_display_power_well_init()
> drm/i915: Enable polling when we don't have hpd
>
> Cc: stable@vger.kernel.org
> Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
> Cc: Daniel Vetter <daniel.vetter@intel.com>
>
> drivers/gpu/drm/i915/i915_drv.c | 7 +++-
> drivers/gpu/drm/i915/i915_drv.h | 3 ++
> drivers/gpu/drm/i915/intel_crt.c | 10 ++---
> drivers/gpu/drm/i915/intel_drv.h | 4 +-
> drivers/gpu/drm/i915/intel_hotplug.c | 69
> ++++++++++++++++++++++++++++-----
> drivers/gpu/drm/i915/intel_runtime_pm.c | 10 +++++
> 6 files changed, 86 insertions(+), 17 deletions(-)
>
--
Cheers,
Lyude
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
next prev parent reply other threads:[~2016-06-17 22:04 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-06-17 21:58 [PATCH 0/3] Fixes for HPD Lyude
2016-06-17 21:58 ` Lyude
2016-06-17 21:58 ` [PATCH RESEND 1/3] drm/i915/vlv: Make intel_crt_reset() per-encoder Lyude
2016-06-17 21:58 ` Lyude
2016-06-17 21:58 ` [PATCH RESEND 2/3] drm/i915/vlv: Reset the ADPA in vlv_display_power_well_init() Lyude
2016-06-17 21:58 ` Lyude
2016-06-17 21:58 ` [PATCH 3/3] drm/i915: Enable polling when we don't have hpd Lyude
2016-06-17 21:58 ` Lyude
2016-06-20 13:09 ` [Intel-gfx] " Daniel Vetter
2016-06-20 13:09 ` Daniel Vetter
2016-06-17 22:04 ` Lyude [this message]
2016-06-20 13:22 ` [PATCH 0/3] Fixes for HPD Ville Syrjälä
2016-06-18 6:09 ` ✗ Ro.CI.BAT: failure for " Patchwork
2016-06-20 13:10 ` Daniel Vetter
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1466201051.7789.10.camel@redhat.com \
--to=cpaul@redhat.com \
--cc=airlied@linux.ie \
--cc=daniel.vetter@intel.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=intel-gfx@lists.freedesktop.org \
--cc=ville.syrjala@linux.intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.