fbdev/atyfb: Fix DSP config on some PowerMacs & PowerBooks
commit94eebffcf94729a92344dcfb1bc58ec1d830c5d2
authorRisto Suominen <Risto.Suominen@gmail.com>
Tue, 13 Jan 2009 20:09:30 +0000 (13 20:09 +0000)
committerGreg Kroah-Hartman <gregkh@suse.de>
Fri, 6 Feb 2009 22:00:36 +0000 (6 14:00 -0800)
tree11624fba9ee80ad1aec8bc40a19b5811cadfe4c0
parent8e9c63009d8bef93ec4c1d7ca8c97839423114c7
fbdev/atyfb: Fix DSP config on some PowerMacs & PowerBooks

commit 7fbb7cadd062baf299fd8b26a80ea99da0c3fe01 upstream.

Since the complete re-write in 2.6.10, some PowerMacs (At least PowerMac 5500
and PowerMac G3 Beige rev A) with ATI Mach64 chip have suffered from unstable
columns in their framebuffer image. This seems to depend on a value (4) read
from PLL_EXT_CNTL register, which leads to incorrect DSP config parameters to
be written to the chip. This patch uses a value calculated by aty_init_pll_ct
instead, as a starting point.

There are questions as to whether this should be extended to other platforms
or maybe made dependent on specific chip types, but in the meantime, this has
been tested on various powermacs and works for them so let's commit it.

Signed-off-by: Risto Suominen <Risto.Suominen@gmail.com>
Tested-by: Michael Pettersson <mike@it.uu.se>
Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/video/aty/mach64_ct.c