From 71990e0beece6e60ffbffccfdf56e5a71641a120 Mon Sep 17 00:00:00 2001 From: Paulo Cesar Pereira de Andrade Date: Mon, 20 Oct 2008 15:09:09 -0200 Subject: Don't change M1XCLK unless option specified in xorg.conf. Also add more complete description of programmable clocks. --- src/smi_501.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) (limited to 'src/smi_501.c') diff --git a/src/smi_501.c b/src/smi_501.c index 1816423..26669cf 100644 --- a/src/smi_501.c +++ b/src/smi_501.c @@ -178,10 +178,6 @@ SMI501_HWInit(ScrnInfoPtr pScrn) mode->clock.f.m_divider = 1; mode->clock.f.m_shift = 0; - /* FIXME probably should not "touch" m1clk. A value other then 112Mhz - * will instant lock on my test prototype, "or" maybe it just means - * that m1clk value must be equal to mclk value? (and mclk must be - * set first!?) */ switch (pSmi->MCLK) { case 168000: /* 336/1/1 */ mode->clock.f.m1_select = 1; @@ -199,11 +195,15 @@ SMI501_HWInit(ScrnInfoPtr pScrn) mode->clock.f.m1_shift = 1; break; case 112000: /* 336/3/0 */ - default: mode->clock.f.m1_select = 1; mode->clock.f.m1_divider = 1; mode->clock.f.m1_shift = 0; break; + default: + /* Do nothing. Use what was configured by the kernel. + * Accordingly to SMI, this should be 144Mhz for 6ns sdram, + * or 112Mhz for other types of sdram. */ + break; } -- cgit v1.2.3