[PATCH] Fetch clock info before setting up connectors

Paul TBBle Hampson Paul.Hampson at Pobox.com
Sat Jun 2 02:57:40 PDT 2007

RADEONProbePLLParameters sets pll->reference_div, and
RADEONGetPanelInfoFromReg uses it.

This ensures that the former is called before the latter.
 src/radeon_driver.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/radeon_driver.c b/src/radeon_driver.c
index 925b8ee..1c433e2 100644
--- a/src/radeon_driver.c
+++ b/src/radeon_driver.c
@@ -2497,14 +2497,14 @@ static Bool RADEONPreInitControllers(ScrnInfoPtr pScrn, xf86Int10InfoPtr  pInt10
     RADEONGetBIOSInfo(pScrn, pInt10);
+    RADEONGetClockInfo(pScrn);
     if (!RADEONSetupConnectors(pScrn)) {
 	return FALSE;
-    RADEONGetClockInfo(pScrn);
     for (i = 0; i < config->num_output; i++) 
       xf86OutputPtr	      output = config->output[i];

This is for branch randr-1.2, and fixes the problem that
appeared in 117220527de9fd3158f600645bcfcaf46847f45f for
BIOS-less cards.

Basically, the lack of BIOS made RADEONGetPanelInfoFromReg
fall back onto the probed reference_div, which had not yet
been probed.

I hope this formats OK, it's my first use of git-format-patch.

Paul "TBBle" Hampson, B.Sc, LPI, MCSE
On-hiatus Asian Studies student, ANU
The Boss, Bubblesworth Pty Ltd (ABN: 51 095 284 361)
Paul.Hampson at Pobox.com

Of course Pacman didn't influence us as kids. If it did,
we'd be running around in darkened rooms, popping pills and
listening to repetitive music.
 -- Kristian Wilson, Nintendo, Inc, 1989

License: http://creativecommons.org/licenses/by/2.1/au/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
Url : http://lists.x.org/archives/xorg-driver-ati/attachments/20070602/a0d02d5e/attachment.pgp 

More information about the xorg-driver-ati mailing list