Summary: | [gm45] Spurious detection of VGA | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | DRI | Reporter: | Chris Wilson <chris> | ||||||
Component: | DRM/Intel | Assignee: | Daniel Vetter <daniel> | ||||||
Status: | CLOSED FIXED | QA Contact: | Intel GFX Bugs mailing list <intel-gfx-bugs> | ||||||
Severity: | normal | ||||||||
Priority: | high | CC: | ben, chris, daniel, jbarnes | ||||||
Version: | XOrg git | ||||||||
Hardware: | Other | ||||||||
OS: | All | ||||||||
Whiteboard: | |||||||||
i915 platform: | i915 features: | ||||||||
Attachments: |
|
Description
Chris Wilson
2012-09-14 20:18:36 UTC
Created attachment 67186 [details]
Xorg.log
Created attachment 67187 [details]
dmesg
[ 1.251393] [drm:drm_helper_probe_single_connector_modes], [CONNECTOR:9:VGA-1] [ 1.264048] [drm:intel_crt_detect], CRT detected via hotplug [ 1.268022] [drm:gmbus_xfer], GMBUS [i915 gmbus vga] NAK for addr: 0050 r(1) [ 1.268027] [drm:drm_do_probe_ddc_edid], drm: skipping non-existent adapter i915 gmbus vga [ 1.268029] [drm:intel_crt_get_edid], CRT GMBUS EDID read failed, retry using GPIO bit-banging [ 1.268285] [drm:drm_do_probe_ddc_edid], drm: skipping non-existent adapter i915 gmbus vga [ 1.305977] [drm:drm_do_probe_ddc_edid], drm: skipping non-existent adapter i915 gmbus dpb [ 1.305994] [drm:drm_helper_probe_single_connector_modes], [CONNECTOR:9:VGA-1] probed modes : [ 1.305997] [drm:drm_mode_debug_printmodeline], Modeline 28:"1024x768" 60 65000 1024 1048 1184 1344 768 771 777 806 0x40 0xa [ 1.306001] [drm:drm_mode_debug_printmodeline], Modeline 26:"800x600" 60 40000 800 840 968 1056 600 601 605 628 0x40 0x5 [ 1.306006] [drm:drm_mode_debug_printmodeline], Modeline 25:"800x600" 56 36000 800 824 896 1024 600 601 603 625 0x40 0x5 [ 1.306010] [drm:drm_mode_debug_printmodeline], Modeline 27:"848x480" 60 33750 848 864 976 1088 480 486 494 517 0x40 0x5 [ 1.306013] [drm:drm_mode_debug_printmodeline], Modeline 24:"640x480" 60 25175 640 656 752 800 480 489 492 525 0x40 0xa Last known good: 8c3f929b So this looks to be a private commit of mine, so consider this a heads up: commit 27dacc5fca64c535bffb384400539f381db7ca2f Author: Daniel Vetter <daniel.vetter@ffwll.ch> Date: Thu Sep 6 14:22:22 2012 +0200 drm/i915: reorder setup sequence to have irqs for output setup Otherwise the new&shiny irq-driven gmbus and dp aux code won't work that well. Noticed since the dp aux code doesn't have an automatic fallback with a timeout (since the hw provides for that already). Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch> I've tried to reproduce on my gm45, but couldn't. But there's definitely something broken with hpd irq setup after this patch - we set up dev_priv->hotplug_enabled_mask in the output setup functions, but now already set up the hotplug irq _before_ calling those ... Works with updated gmbus-irq branch. The ghost is back. Patch, this time hopefully for real: https://patchwork.kernel.org/patch/1854161/ commit 20afbda209d708be66944907966486d0c1331cb8 Author: Daniel Vetter <daniel.vetter@ffwll.ch> Date: Tue Dec 11 14:05:07 2012 +0100 drm/i915: Fixup hpd irq register setup ordering For GMCH platforms we set up the hpd irq registers in the irq postinstall hook. But since we only enable the irq sources we actually need in PORT_HOTPLUG_EN/STATUS, taking dev_priv->hotplug_supported_mask into account, no hpd interrupt sources is enabled since |
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.