Re: bad panel power sequencing delays, disabling panel

[Daniel Vetter <daniel@ffwll.ch>]

On Wed, Oct 24, 2012 at 01:27:23PM +0800, Daniel J Blueman wrote:
> On 17 October 2012 00:43, Daniel Vetter <daniel@ffwll.ch> wrote:
> > On Tue, Oct 16, 2012 at 6:10 PM, Daniel J Blueman <daniel@quora.org> wrote:
> >> [drm:intel_dp_init], cur t1_t3 0 t8 0 t9 0 t10 0 t11_t12 4000
> >> [drm:intel_dp_init], vbt t1_t3 0 t8 0 t9 0 t10 0 t11_t12 0
> >> [drm:intel_dp_init], panel power up delay 21, power down delay 50,
> >> power cycle delay 400
> >> [drm:intel_dp_init], backlight on delay 5, off delay 5
> >> [drm:intel_dp_init], panel power sequencer register settings: PP_ON
> >> 0x40d20032, PP_OFF 0x1f40032, PP_DIV 0x186904
> >> [drm:intel_dp_i2c_init], i2c_init DPDDC-A
> >> [drm:ironlake_edp_panel_vdd_on], Turn eDP VDD on
> >> [drm:ironlake_edp_panel_vdd_on], eDP VDD already on
> >> [drm:intel_dp_aux_ch], dp_aux_ch timeout status 0x514500c8
> >> [drm:intel_dp_i2c_aux_ch], aux_ch failed -110
> >>
> >> Notable, the nvidia DP init script executed fine; perhaps tracing the
> >> I2C access may be useful?
> >
> > Hm, dp aux transfer don't work (this is the first one in the setup
> > sequence, so it's not an i2c over aux issue with the edid read). I
> > guess the mux is getting in the way. Dave, do you have an idea what's
> > still going wrong? panel power sequence is now operationl. For
> > reference, the wip patch series is at:
> > http://cgit.freedesktop.org/~danvet/drm/log/?h=for-pzanoni
> 
> Booting for-pzanoni as of today, when switching to i915, the backlight
> turns off [1]. Full boot log at
> http://quora.org/2012/mbp-i915-panel-3.txt ; the same situation occurs
> on 3.7-rc2 alas.

Yeah, you have an issue with dp aux not working (since the mux is set
wrongly), not with the panel not being powered on (or at least that's a
secondary issue). So the panel power improvements don't help you.

> Is there a way to trigger re-enumeration of the panels after I
> switcheroo to i915?
> 
> I'm checking with Seth if the mux could be the issue, though the code
> shows there would be an error message if switching failed.

Iirc Dave Airlie has patches to delay the driver init until the mux code
is all set up, so that the driver could request the mux to be switched
while init is ongoing.
-Daniel

> 
> Thanks,
>   Daniel
> 
> --- [1]
> 
> [    3.522057] [drm:intel_dp_i2c_init], i2c_init DPDDC-A
> [    3.522060] [drm:ironlake_edp_panel_vdd_on], Turn eDP VDD on
> [    3.522066] [drm:ironlake_edp_panel_vdd_on], eDP VDD already on
> [    3.524598] [drm:intel_dp_aux_ch], dp_aux_ch timeout status 0x514500c8
> [    3.524601] [drm:intel_dp_i2c_aux_ch], aux_ch failed -110
> [    3.527121] [drm:intel_dp_aux_ch], dp_aux_ch timeout status 0x514500c8
> [    3.527123] [drm:intel_dp_i2c_aux_ch], aux_ch failed -110
> [    3.527144] [drm:ironlake_edp_panel_vdd_off], Turn eDP VDD off 1
> [    3.527147] [drm:ironlake_edp_panel_vdd_on], Turn eDP VDD on
> [    3.527150] [drm:ironlake_edp_panel_vdd_on], eDP VDD already on
> [    3.529677] [drm:intel_dp_aux_ch], dp_aux_ch timeout status 0x514500c8
> [    3.549595] [drm:intel_dp_aux_ch], dp_aux_ch timeout status 0x514500c8
> [    3.564443] input: Apple Inc. Apple Internal Keyboard / Trackpad as
> /devices/pci0000:00/0000:00:1d.0/usb2/2-1/2-1.8/2-1.8.2/2-1.8.2:1.0/input/input4
> [    3.564721] apple 0003:05AC:0262.0001: input,hidraw0: USB HID v1.11
> Keyboard [Apple Inc. Apple Internal Keyboard / Trackpad] on
> usb-0000:00:1d.0-1.8.2/input0
> [    3.569584] [drm:intel_dp_aux_ch], dp_aux_ch timeout status 0x514500c8
> [    3.587024] [drm:ironlake_edp_panel_vdd_off], Turn eDP VDD off 1
> [    3.587035] [drm] failed to retrieve link info, disabling eDP
> -- 
> Daniel J Blueman

-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch