Re: mach64 breakage in 2.6.22

[Ville Syrjälä <syrjala@sci.fi>]

On Thu, Jun 14, 2007 at 10:25:28AM +0200, Olaf Hering wrote:
> On Sat, May 05, Andrew Morton wrote:
> 
> > ftp://ftp.kernel.org/pub/linux/kernel/people/akpm/patches/2.6/2.6.21/2.6.21-mm1/
> 
> > +atyfb-reorganize-clock-init.patch
> 
> This change breaks the display on an ibook1 with 800x600 lcd.

Are you sure? I would expect
atyfb-halve-xclk-with-mobility-and-32bit-memory.patch to cause more
problems.

> It is commit b4e124c138558a0cff51398ddff9a8e44ed0b529 in 2.6.22-rc4.
> The used config is arch/powerpc/configs/pmac32_defconfig
> 
> It seems that the recognized display width is only 640, the remain space
> up to 800 is repeated with the content from colum 0. The chars around
> colum 640 are mostly garbage.
> 
> .....
> atyfb: using auxiliary register aperture
> atyfb: 3D RAGE Mobility L (Mach64 LN, AGP 2x) [0x4c4e rev 0x64]
> atyfb: 4M SDRAM (2:1) (32-bit), 14.31818 MHz XTAL, 230 MHz PLL, 70 Mhz MCLK, 27 MHz XCLK

This seems to confirm my suspicion.

Please try this patch:

---
 drivers/video/aty/atyfb_base.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Index: linux-2.6.22-rc2/drivers/video/aty/atyfb_base.c
===================================================================
--- linux-2.6.22-rc2.orig/drivers/video/aty/atyfb_base.c
+++ linux-2.6.22-rc2/drivers/video/aty/atyfb_base.c
@@ -2295,7 +2295,7 @@ static int __devinit aty_init(struct fb_
 	 * and set the frequency manually. */
 	if (machine_is_compatible("PowerBook2,1")) {
 		par->pll_limits.mclk = 70;
-		par->pll_limits.xclk = 53;
+		par->pll_limits.xclk = 106;
 	}
 #endif
 
-- 
Ville Syrjälä
syrjala@sci.fi
http://www.sci.fi/~syrjala/
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/