All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ander Conselvan De Oliveira <conselvan2@gmail.com>
To: Shubhangi Shrivastava <shubhangi.shrivastava@intel.com>,
	Daniel Vetter <daniel@ffwll.ch>
Cc: intel-gfx@lists.freedesktop.org
Subject: Re: [PATCH 0/6] Fixing sink count related detection over
Date: Wed, 18 Nov 2015 11:23:45 +0200	[thread overview]
Message-ID: <1447838625.2608.3.camel@gmail.com> (raw)
In-Reply-To: <564C0FAB.1010106@intel.com>

On Wed, 2015-11-18 at 11:12 +0530, Shubhangi Shrivastava wrote:
> 
> On Wednesday 18 November 2015 01:23 AM, Daniel Vetter wrote:
> > On Mon, Nov 02, 2015 at 06:25:10PM +0530, Shubhangi Shrivastava wrote:
> > > This patch set cleans up DP detection logic to bring all DPCD
> > > operations at one place and to create a clear demarcation
> > > between handling of long and short pulses. This simplifies
> > > fixing of sink count related detection for DP panels.
> > > 
> > > Patches:
> > > 1. First two patches clean up intel_dp_detect and form a new
> > > function which will include all DPCD related operations.
> > > 2. Second patch splits up intel_dp_check_link_status to form
> > > a new function which will handle short pulse requests.
> > > 3. Last three patches fixes the detection logic related to
> > > sink count i.e detect changes in sink count and handle them
> > > appropriately.
> > > 
> > > Note: this is tested on BXT with non-mst panels,
> > > will get back ASAP with results for MST panels too.
> > Sivakumar and Ander are working on reorganizing all the DP hpd handling,
> > at least they have been before I went on vacation. I think it would make
> > sense to land that work first, before we start to apply functional fixes.
> > 
> > Please coordinate with them.
> > -Daniel
> Yes.. The reorganization of DP HPD handling is in progress.
> I am working with Siva on the same. :)

I thought this was the series that reorganizes HPD. Should I be looking at some
other patches too?

Ander

> We have found certain bugs in MST related code, pointed out
> by Ander. Once it is fixed, we will re-upload the patches.
> > 
> > > Shubhangi Shrivastava (6):
> > >    drm/i915: Splitting intel_dp_detect
> > >    drm/i915: Cleaning up intel_dp_hpd_pulse
> > >    drm/i915: Splitting intel_dp_check_link_status
> > >    drm/i915: Save sink_count for tracking changes to it
> > >    drm/i915: read sink_count dpcd always
> > >    drm/i915: force full detect on sink count change
> > > 
> > >   drivers/gpu/drm/i915/intel_dp.c  | 170 +++++++++++++++++++++++++--------
> > > ------
> > >   drivers/gpu/drm/i915/intel_drv.h |   1 +
> > >   2 files changed, 110 insertions(+), 61 deletions(-)
> > > 
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

  reply	other threads:[~2015-11-18  9:23 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-11-02 12:55 [PATCH 0/6] Fixing sink count related detection over Shubhangi Shrivastava
2015-11-02 12:55 ` [PATCH 1/6] drm/i915: Splitting intel_dp_detect Shubhangi Shrivastava
2015-11-02 13:05   ` kbuild test robot
2015-11-02 12:55 ` [PATCH 2/6] drm/i915: Cleaning up intel_dp_hpd_pulse Shubhangi Shrivastava
2015-11-02 12:55 ` [PATCH 3/6] drm/i915: Splitting intel_dp_check_link_status Shubhangi Shrivastava
2015-11-02 13:11   ` kbuild test robot
2015-11-02 12:55 ` [PATCH 4/6] drm/i915: Save sink_count for tracking changes to it Shubhangi Shrivastava
2015-11-02 12:55 ` [PATCH 5/6] drm/i915: read sink_count dpcd always Shubhangi Shrivastava
2015-11-02 12:55 ` [PATCH 6/6] drm/i915: force full detect on sink count change Shubhangi Shrivastava
2015-11-17 19:53 ` [PATCH 0/6] Fixing sink count related detection over Daniel Vetter
2015-11-18  5:42   ` Shubhangi Shrivastava
2015-11-18  9:23     ` Ander Conselvan De Oliveira [this message]
2015-11-18  9:36       ` Shubhangi Shrivastava
  -- strict thread matches above, loose matches on Subject: below --
2015-11-06 12:28 Shubhangi Shrivastava
2016-01-05 12:50 Shubhangi Shrivastava

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=1447838625.2608.3.camel@gmail.com \
    --to=conselvan2@gmail.com \
    --cc=daniel@ffwll.ch \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=shubhangi.shrivastava@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.