Bug 27083 - Commit "use lcd pll limits when available" in drm-radeon-testing breaks my laptop lvds
Summary: Commit "use lcd pll limits when available" in drm-radeon-testing breaks my l...
Status: RESOLVED FIXED
Alias: None
Product: DRI
Classification: Unclassified
Component: DRM/Radeon (show other bugs)
Version: DRI git
Hardware: x86-64 (AMD64) Linux (All)
: medium normal
Assignee: Default DRI bug account
QA Contact:
URL:
Whiteboard:
Keywords:
: 27106 (view as bug list)
Depends on:
Blocks:
 
Reported: 2010-03-15 05:15 UTC by Stefano Carignano
Modified: 2010-03-17 20:30 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:

Attachments
remove lvds quirks (1.46 KB, patch)
2010-03-15 10:08 UTC, Alex Deucher 		no flags Details | Splinter Review
disable lcd pll code (605 bytes, patch)
2010-03-16 07:01 UTC, Alex Deucher 		no flags Details | Splinter Review
View All

Description Stefano Carignano 2010-03-15 05:15:51 UTC 
As the summary says, commit 86cb2bbfda2cf402aee46779ee90bbb7d915482b (drm/radeon/kms: use lcd pll limits when available) in drm-radeon-testing produces unwatchable fuzziness on my laptop screen (toshiba a300, hd3470 mobility). Reverting the commit brings the laptop back to an usable state.
Some system info was already attached at the (old, similar, fixed) bug 
http://bugs.freedesktop.org/show_bug.cgi?id=26358
Comment 1 Alex Deucher 2010-03-15 10:08:40 UTC 
Created attachment 34076 [details] [review]
remove lvds quirks

Perhaps with the new code the quirk from the other bug isn't necessary.  Does this patch help?  It removes the LVDS quirks.
Comment 2 Alexandre Derumier 2010-03-16 02:44:41 UTC 
i confirm, with the pll limits patch, i doesn't need the lvds quirk for my dell studio laptop 15.
Comment 3 Stefano Carignano 2010-03-16 02:58:03 UTC 
seems to be working here as well, thanks!
Comment 4 Alex Deucher 2010-03-16 06:56:08 UTC 
*** Bug 27106 has been marked as a duplicate of this bug. ***
Comment 5 Alex Deucher 2010-03-16 07:01:21 UTC 
Created attachment 34106 [details] [review]
disable lcd pll code

I suspect the pll algo fixes are actually what fixed things.  Can you try this patch as well (in addition to the previous one)?
Comment 6 Christian Schmidt 2010-03-16 09:46:47 UTC 
Both patches together work for my duplicate T500 bug.
Comment 7 Stefano Carignano 2010-03-17 01:43:39 UTC 
the second patch didn't break anything here as well :)
Comment 8 Alex Deucher 2010-03-17 20:30:41 UTC 
I've sent the combined patch to Dave.  thanks!

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.