On Tue, Jan 29, 2008 at 11:22:41AM +0100, Heiko Schocher wrote: > + model = "MGCOGE"; > + compatible = "fsl,mgcoge";
keymile,mgcoge > + PowerPC,[EMAIL PROTECTED] { [snip] > + compatible = "fsl,mpc8248-localbus", All of these 8248s should be 8247. > + // Temporary -- will go away once kernel uses ranges for > get_immrbase(). > + reg = <0xf0000000 0x00053000>; The patch to use ranges for get_immrbase() just went in, so we can drop this now. > + [EMAIL PROTECTED] { > + compatible = "fsl,cpm-muram-data"; > + reg = <0 0x1100 0x1140 > + 0xec0 0x9800 0x800>; This doesn't look right. You're excluding 0x40 bytes at 0x1100, which is where planetcore puts SMC1. However, you're using SMC2 -- and I'm guessing aren't using planetcore, since this isn't an embedded planet board. If you're using u-boot, this should be: reg = <0x80 0x1f80 0x9800 0x800>; > + /* Monitor port/SMC2 */ > + smc2: [EMAIL PROTECTED] { > + device_type = "serial"; > + compatible = "fsl,mpc8248-smc-uart", > + "fsl,cpm2-smc-uart"; > + reg = <0x11a90 0x20 0x1100 0x40 0x88fc 4>; If you're using u-boot, this should be <0x11a90 0x20 0x40 0x40>. > + current-speed = <0x1c200>; This should be decimal. -Scott _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev