Re: mach64 breakage in 2.6.22

Ville Syrjälä
2007-06-14T18:03:27Z

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