Re: asm-ppc header issues when building ARCH=powerpc

On Wed, Aug 22, 2007 at 11:16:03PM -0500, Kumar Gala wrote:

On Aug 22, 2007, at 10:33 PM, David Gibson wrote:

quoted

quoted

My analysis of <asm/bootinfo.h> usage:

./drivers/macintosh/adb-iop.c:#include <asm/bootinfo.h>	 remove
./drivers/char/vme_scc.c:#include <asm/bootinfo.h>		68k only
./drivers/char/serial167.c:#include <asm/bootinfo.h>		68k only
./drivers/serial/dz.c:#include <asm/bootinfo.h>		 decstation
./drivers/mtd/devices/ms02-nv.c:#include <asm/bootinfo.h>	decstation
./drivers/net/macsonic.c:#include <asm/bootinfo.h>		68k
./drivers/net/jazzsonic.c:#include <asm/bootinfo.h>		mips
./drivers/video/pmag-aa-fb.c:#include <asm/bootinfo.h>		mips
./drivers/video/maxinefb.c:#include <asm/bootinfo.h>		mips
./drivers/video/logo/logo.c:#include <asm/bootinfo.h>		mips
./drivers/video/macfb.c:#include <asm/bootinfo.h>		68k
./drivers/video/valkyriefb.c:#include <asm/bootinfo.h>		68k

Uh.. I'm pretty sure valkyriefb.c is for (old) PowerMacs, not 68k.

It appears to be both.  If you look at the include its protected by a  
#ifdef CONFIG_MAC which we is only defined on m68k.

According to drivers/video/macfb.c, both the Quadra 630 and Performa 588
have the Valkyrie chip for their video. I don't remember which ppc based
models have it, but it's definitely on both architectures.

Just as an extra note, the file drivers/macintosh/adb-iop.c is m68k only,
so you should probably leave that alone as well. It probably doesn't need
that header, but the change should really come from the 68k side of things.

	Brad Boyer
	flar@allandria.com