New Dual 2ghz Powermac G5

[Benjamin Herrenschmidt]

On Tue, 2005-08-09 at 16:40 -0400, Keith Mitchell wrote:
> I just got in a bunch of new Dual 2.0ghz Powermacs and one out of the 
> bunch so far has a noisy fan problem.  I have been loading YDL 4.0.91 
> which has kernel 2.6.12 plus some patches and I did go back and verify 
> that it had the last patch I saw related to this 
> (http://patchwork.ozlabs.org/linuxppc64/patch?id=846).  All of the 
> machines all identify themselves as running OpenFirmware 5.2.4f1 and all 
> came in at once with the same config.
> 
> The machine with the problem so far issues the following messages:
> 
> kernel: /u3 at 0,f8000000/i2c at f8001000: Missing interrupt or address !
> kernel: /u3 at 0,f8000000/i2c at f8001000: Missing interrupt or address !
> kernel: /u3 at 0,f8000000/i2c at f8001000: Missing interrupt or address !
> kernel: /u3 at 0,f8000000/i2c at f8001000: Missing interrupt or address !
> 
> 
> Is this a known issue?

Yes, here is the fix:

The workaround for broken device-tree that prevents fan control from
working on recent G5 models need to be "enabled" for machines with
revision 0x37 of the bridge in addition to machines with revision 0x35.

Signed-off-by: Geoff Levand <geoffrey.levand at am.sony.com>
Signed-off-by: Benjamin Herrenschmidt <benh at kernel.crashing.org>

--- a/arch/ppc64/kernel/prom_init.c     2005-07-27 18:34:40.000000000 -0700
+++ b/arch/ppc64/kernel/prom_init.c     2005-08-09 16:18:04.000000000 -0700
@@ -1803,7 +1803,7 @@
        if (prom_getprop(u3, "device-rev", &u3_rev, sizeof(u3_rev))
            == PROM_ERROR)
                return;
-       if (u3_rev != 0x35)
+       if (u3_rev != 0x35 && u3_rev != 0x37)
                return;
        /* does it need fixup ? */
        if (prom_getproplen(i2c, "interrupts") > 0)