Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1361687

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Get rid of intel_dp_dpcd_read_wake()

From Jani Nikula <jani.nikula@linux.intel.com>
Newsgroups linux.kernel
Subject Re: [Intel-gfx] [PATCH 2/2] drm/i915: Get rid of intel_dp_dpcd_read_wake()
Date 2016-03-21 11:40 +0100
Message-ID <rf5pE-4hd-15@gated-at.bofh.it> (permalink)
References <rdIlr-MU-7@gated-at.bofh.it> <rdIlr-MU-5@gated-at.bofh.it> <re3pU-6Yy-11@gated-at.bofh.it> <re5i3-1rG-25@gated-at.bofh.it> <re5L3-23u-1@gated-at.bofh.it>
Organization Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo

Show all headers | View raw


On Fri, 18 Mar 2016, Ville Syrjälä <ville.syrjala@linux.intel.com> wrote:
> On Fri, Mar 18, 2016 at 06:12:35PM +0200, Ville Syrjälä wrote:
>> On Fri, Mar 18, 2016 at 04:13:45PM +0200, Ville Syrjälä wrote:
>> > On Thu, Mar 17, 2016 at 11:40:45AM -0400, Lyude wrote:
>> > > -	drm_dp_dpcd_read(aux, DP_DPCD_REV, buffer, 1);
>> > 
>> > NAK
>> > 
>> > If people keep intentionally breaking my shit I'm going to become
>> > really grumpy soon.
>> 
>> Oh, and just in case someone wants to come up with a better kludge,
>> I just spent a few minutes analyzing the behavior of this crappy
>> monitor a.
>> 
>> What happens is that when the monitor is fully powered up (LED is blue)
>> things are fine. After the monitor goes to sleep (LED turns orange)
>> the first DPCD read will produce garbage. Further DPCD reads are fine,
>> even if I wait a significant amount of time between the reads, as long
>> as the monitor didn't do a power on->off cycle in between. So it looks
>> like it's always just the first read after power down that gets
>> corrupted.
>> 
>> Now I think I'll go and test how writes behave, assuming I can find a
>> decently sized chunk of DPCD address space I can write. And maybe I
>> should also try i2c-over-aux...
>
> The first DPCD write after powerdown also got corrupted. But i2c-over-aux
> seems unaffected for whatever reason.

Did the display go to sleep on its own, or did we do something? In
particular, does DPCD DP_SET_POWER register play a role? What if we skip
writing D3 to it? What if we do that write as the first thing (every
time)?


BR,
Jani.


-- 
Jani Nikula, Intel Open Source Technology Center

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Get rid of  intel_dp_dpcd_read_wake() Ville Syrjälä <ville.syrjala@linux.intel.com> - 2016-03-18 15:20 +0100
  Re: [Intel-gfx] [PATCH 2/2] drm/i915: Get rid of  intel_dp_dpcd_read_wake() Ville Syrjälä <ville.syrjala@linux.intel.com> - 2016-03-18 17:20 +0100
    Re: [Intel-gfx] [PATCH 2/2] drm/i915: Get rid of  intel_dp_dpcd_read_wake() Ville Syrjälä <ville.syrjala@linux.intel.com> - 2016-03-18 17:50 +0100
      Re: [Intel-gfx] [PATCH 2/2] drm/i915: Get rid of  intel_dp_dpcd_read_wake() Daniel Vetter <daniel@ffwll.ch> - 2016-03-18 19:00 +0100
        Re: [Intel-gfx] [PATCH 2/2] drm/i915: Get rid of  intel_dp_dpcd_read_wake() Ville Syrjälä <ville.syrjala@linux.intel.com> - 2016-03-18 19:10 +0100
          Re: [Intel-gfx] [PATCH 2/2] drm/i915: Get rid of  intel_dp_dpcd_read_wake() Lyude Paul <cpaul@redhat.com> - 2016-03-21 17:40 +0100
      Re: [Intel-gfx] [PATCH 2/2] drm/i915: Get rid of intel_dp_dpcd_read_wake() Jani Nikula <jani.nikula@linux.intel.com> - 2016-03-21 11:40 +0100
        Re: [Intel-gfx] [PATCH 2/2] drm/i915: Get rid of  intel_dp_dpcd_read_wake() Ville Syrjälä <ville.syrjala@linux.intel.com> - 2016-03-21 14:40 +0100

csiph-web