From: Jani Nikula <jani.nikula@linux.intel.com>
To: intel-gfx@lists.freedesktop.org
Cc: Egbert Eich <eich@suse.de>
Subject: Re: [PATCH 0/5] drm/i915 Avoid long delays when reading EDID on eDP
Date: Tue, 25 Nov 2014 11:08:42 +0200 [thread overview]
Message-ID: <87sih7r6jp.fsf@intel.com> (raw)
In-Reply-To: <1416849387-20984-1-git-send-email-eich@suse.de>
On Mon, 24 Nov 2014, Egbert Eich <eich@suse.de> wrote:
> For eDP in the Intel driver pps_lock()/unlock() need to be called before
> initiating an I2C/AUX channel transfer. These operations can be quite
> expensive - especially on values for HZ lower than 1000.
> It is therefore better to perfrom this locking/unlocking only once,
> ie at the beginning and at the end of the entire I2C transfer.
> The current design of drm_dp_helper.c doesn't allow this.
> This patchset modifies drm_dp_helper.c and moves the locking/unlocking
> operation to the top.
> This fixes the long delay observed in
> https://bugs.freedesktop.org/show_bug.cgi?id=86201
>
> Egbert Eich (4):
> drm/DP: Create pointer to generic DPCD access function
> drm/DP: Export drm_dp_i2c_xfer() DP helper function
> drm/DP: Export drm_dp_dpcd_access() DP helper function
These three need to be sent to dri-devel, with cc: Thierry Reding
<thierry.reding@gmail.com>.
BR,
Jani.
> drm/i915/eDP: Move pps_lock() and edp_panel_vdd_on() to top
>
> Ville Syrjälä (1):
> drm/i915: Try to avoid pps_{lock,unlock}() on DP ports
>
> drivers/gpu/drm/drm_dp_helper.c | 11 ++--
> drivers/gpu/drm/i915/intel_dp.c | 132 +++++++++++++++++++++++++++++++--------
> drivers/gpu/drm/i915/intel_drv.h | 5 ++
> include/drm/drm_dp_helper.h | 14 +++++
> 4 files changed, 133 insertions(+), 29 deletions(-)
>
> --
> 1.8.4.5
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx
--
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
next prev parent reply other threads:[~2014-11-25 9:08 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-11-24 17:16 [PATCH 0/5] drm/i915 Avoid long delays when reading EDID on eDP Egbert Eich
2014-11-24 17:16 ` [PATCH 1/5] drm/i915: Try to avoid pps_{lock, unlock}() on DP ports Egbert Eich
2014-11-24 18:28 ` Jani Nikula
2014-11-24 17:16 ` [PATCH 2/5] drm/DP: Create pointer to generic DPCD access function Egbert Eich
2014-11-24 17:16 ` [PATCH 3/5] drm/DP: Export drm_dp_i2c_xfer() DP helper function Egbert Eich
2014-11-24 17:16 ` [PATCH 4/5] drm/DP: Export drm_dp_dpcd_access() " Egbert Eich
2014-11-24 17:16 ` [PATCH 5/5] drm/i915/eDP: Move pps_lock() and edp_panel_vdd_on() to top Egbert Eich
2014-11-25 9:08 ` Jani Nikula [this message]
2014-11-25 13:13 ` [PATCH 0/5] drm/i915 Avoid long delays when reading EDID on eDP Daniel Vetter
2014-11-25 17:20 ` Egbert Eich
2014-11-26 8:57 ` 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=87sih7r6jp.fsf@intel.com \
--to=jani.nikula@linux.intel.com \
--cc=eich@suse.de \
--cc=intel-gfx@lists.freedesktop.org \
/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.