All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Souza, Jose" <jose.souza@intel.com>
To: "tjaalton@ubuntu.com" <tjaalton@ubuntu.com>,
	"intel-gfx@lists.freedesktop.org"
	<intel-gfx@lists.freedesktop.org>
Cc: "Nikula, Jani" <jani.nikula@intel.com>
Subject: Re: [PATCH v3 2/2] drm/i915: Enable hotplug retry
Date: Tue, 9 Jul 2019 17:40:08 +0000	[thread overview]
Message-ID: <73c7ac9c606f05508e48a09973dc64fb88d77a95.camel@intel.com> (raw)
In-Reply-To: <7a097f16-b4c9-a907-8d25-0502a4d06ef6@ubuntu.com>

On Tue, 2019-07-09 at 12:26 +0300, Timo Aaltonen wrote:
> On 2.7.2019 23.41, Souza, Jose wrote:
> > On Tue, 2019-07-02 at 23:29 +0300, Timo Aaltonen wrote:
> > > On 2.7.2019 22.54, Souza, Jose wrote:
> > > > Here a dmesg output of this patch working in a unpowered type-c
> > > > dongle:
> > > > https://gist.github.com/zehortigoza/93c54b03fb65237cc1a2e193acef61a8
> > > > 
> > > > With the latest type-c patches from Imre it is becoming really
> > > > hard
> > > > to
> > > > reproduce this but is still possible, also looks like due some
> > > > internal
> > > > error on the dongle it being re-discovered by USB sub-system.
> > > > 
> > > > I added this to the patches bellow have more log information:
> > > > https://gist.github.com/zehortigoza/baecabeb7097b9322723b6caf5a9ced5
> > > > Let me know if you think this or something similar should be
> > > > squashed
> > > > to this patch, I think it is not necessary.
> > > 
> > > FWIW, we've tested these on a WHL which is suffering from HDMI
> > > unplug
> > > still showing the display connected, and it's working fine now.
> > > 
> > > 
> > 
> > Thanks, I will add your:
> > 
> > Tested-by: Timo Aaltonen <tjaalton@ubuntu.com>
> 
> That's fine. When can we expect these to land on dinq?
> 
> 
> 

As soon as someone reviews it.

Imre can you review the the second patch? I will find someone to review
the first one.
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

  reply	other threads:[~2019-07-09 17:40 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-06-28 21:39 [PATCH v3 1/2] drm/i915: Add support for retrying hotplug José Roberto de Souza
2019-06-28 21:39 ` [PATCH v3 2/2] drm/i915: Enable hotplug retry José Roberto de Souza
2019-07-02 19:54   ` Souza, Jose
2019-07-02 20:29     ` Timo Aaltonen
2019-07-02 20:41       ` Souza, Jose
2019-07-09  9:26         ` Timo Aaltonen
2019-07-09 17:40           ` Souza, Jose [this message]
2019-07-10 14:12   ` Imre Deak
2019-06-28 23:34 ` ✓ Fi.CI.BAT: success for series starting with [v3,1/2] drm/i915: Add support for retrying hotplug Patchwork
2019-06-29 12:20 ` ✓ Fi.CI.IGT: " Patchwork
2019-07-10 11:20 ` [PATCH v3 1/2] " Rodrigo Vivi
2019-07-10 22:17   ` Souza, Jose

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=73c7ac9c606f05508e48a09973dc64fb88d77a95.camel@intel.com \
    --to=jose.souza@intel.com \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=jani.nikula@intel.com \
    --cc=tjaalton@ubuntu.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.