From: "Ville Syrjälä" <syrjala@sci.fi>
To: Risto Suominen <risto.suominen@gmail.com>,
Alex Kern <alex.kern@gmx.de>,
linux-kernel@vger.kernel.org, benh@kernel.crashing.org,
linux-fbdev-devel@lists.sourceforge.net
Subject: Re: [Linux-fbdev-devel] [PATCH] atyfb: fix distorted image on PowerMacs
Date: Tue, 3 Feb 2009 20:56:25 +0200 [thread overview]
Message-ID: <20090203185624.GC22401@sci.fi> (raw)
In-Reply-To: <20090122201204.GD4996@sci.fi>
On Thu, Jan 22, 2009 at 10:12:04PM +0200, Ville Syrjälä wrote:
> On Thu, Jan 22, 2009 at 08:22:48PM +0200, Risto Suominen wrote:
> > 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.
>
> AFAICS it should be the right thing to do on other systems too. I have
> a good collection of mach64 pci cards and laptops so I'll try to find
> some time next week to test that theory on x86.
Well, the experiment failed on the first Mobility I tried. The FIFO
calculations aren't obvious from the code and there's no mention how
to actually calculate them in the programming guide. So I guess special
casing powermacs is the best way forward.
--
Ville Syrjälä
syrjala@sci.fi
http://www.sci.fi/~syrjala/
next prev parent reply other threads:[~2009-02-03 18:56 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-01-22 18:22 [PATCH] atyfb: fix distorted image on PowerMacs Risto Suominen
2009-01-22 20:12 ` Ville Syrjälä
2009-02-03 18:56 ` Ville Syrjälä [this message]
2009-02-03 21:06 ` [Linux-fbdev-devel] " Benjamin Herrenschmidt
2009-02-03 21:06 ` Benjamin Herrenschmidt
2009-01-23 14:21 ` Mikael Pettersson
2009-01-30 3:42 ` Benjamin Herrenschmidt
2009-01-30 4:37 ` Andrew Morton
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20090203185624.GC22401@sci.fi \
--to=syrjala@sci.fi \
--cc=alex.kern@gmx.de \
--cc=benh@kernel.crashing.org \
--cc=linux-fbdev-devel@lists.sourceforge.net \
--cc=linux-kernel@vger.kernel.org \
--cc=risto.suominen@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.