Bug 23752 - drm/radeon/kms -- hang during boot unless using "nomodeset"
Summary: drm/radeon/kms -- hang during boot unless using "nomodeset"
Alias: None
Product: Drivers
Classification: Unclassified
Component: Video(DRI - non Intel) (show other bugs)
Hardware: All Linux
: P1 normal
Assignee: drivers_video-dri
URL: https://bugs.freedesktop.org/show_bug...
Depends on:
Blocks: 21782
  Show dependency tree
Reported: 2010-11-25 15:37 UTC by John W. Linville
Modified: 2010-12-02 21:40 UTC (History)
4 users (show)

See Also:
Kernel Version: 2.6.37-rc3
Regression: Yes
Bisected commit-id:

lspci-info.txt (19.66 KB, text/plain)
2010-11-25 15:37 UTC, John W. Linville
vbios.rom (64.00 KB, application/octet-stream)
2010-11-26 00:01 UTC, John W. Linville
fix ddc line vbios bug (1.78 KB, patch)
2010-11-30 00:48 UTC, Alex Deucher
Details | Diff

Description John W. Linville 2010-11-25 15:37:22 UTC
Created attachment 38212 [details]

Boot hangs very early unless nomodeset is used.  No disk access is observed and box never becomes available on the network.

The box is an HP 8530p laptop.  Normal functionality is restored by reverting this patch:

commit bdd91b2b571c80dacfca88667d935f9907e62931
Author: Tyson Whitehead <twhitehead@gmail.com>
Date:   Mon Nov 8 16:08:30 2010 +0000

    drm/radeon/kms: fix bugs in ddc and cd path router code
    This is a follow on to:
    (drm/radeon/kms: add support for clock/data path routers)
    That patch completed mux support for ddc and cd line routing
    between connectors.  This patch fixes an indexing typo that was
    resulting in the atom bios router objects not always being walked,
    ensures the validity entries for the reused router structure are
    reset for every connector object walked, and corrects the masking
    operations used to update the mux control bits.
    Signed-off-by: Tyson Whitehead <twhitehead@gmail.com>
    Signed-off-by: Alex Deucher <alexdeucher@gmail.com>
    Signed-off-by: Dave Airlie <airlied@redhat.com>
Comment 1 Alex Deucher 2010-11-25 16:16:36 UTC
Can you attach a copy of the video bios from that laptop?  See this link for how to dump the vbios:
Comment 2 John W. Linville 2010-11-26 00:01:24 UTC
Created attachment 38242 [details]
Comment 3 Alex Deucher 2010-11-30 00:48:31 UTC
Created attachment 38592 [details]
fix ddc line vbios bug

This patch should fix the issue and get the LCD EDID working as well.
Comment 4 John W. Linville 2010-11-30 02:44:36 UTC
Seems good to me (i.e. boot is successful w/o using nomodeset)!  Not sure how to check the EDID bit...?
Comment 5 Alex Deucher 2010-11-30 05:14:14 UTC
(In reply to comment #4)
> Seems good to me (i.e. boot is successful w/o using nomodeset)!  Not sure how
> to check the EDID bit...?

Great I'll send it to Dave.  xrandr --verbose will show you whether or not there is an EDID for a particular output.  The panel timing is also available via a table in the vbios so the EDID only supplies supplemental info (like physical display size).
Comment 6 Florian Mickler 2010-11-30 10:44:18 UTC
Patch: https://bugzilla.kernel.org/attachment.cgi?id=38592
Comment 7 Rafael J. Wysocki 2010-11-30 21:32:51 UTC
Handled-By : Alex Deucher <alexdeucher@gmail.com>
Comment 8 Rafael J. Wysocki 2010-12-02 21:40:05 UTC
Fixed by commit 3074adc drm/radeon/kms: add workaround for dce3 ddc line vbios bug .

Note You need to log in before you can comment on or make changes to this bug.