Re: [PATCH 04/24] C6X: build infrastructure
From: Mark Salter <hidden>
Date: 2011-08-24 14:10:47
On Wed, 2011-08-24 at 14:47 +0100, David Woodhouse wrote:
On Tue, 2011-08-23 at 09:23 -0400, Mark Salter wrote:quoted
quoted
quoted
+cflags-y := -D__linux__ -D__TMS320C6X__Why is __linux__ and __TMS3206C6X__ neeeded? and why are they not needed for CHECKFLAGS ?I think these two defines were left over from trying to compensate for differences between TI compiler and GCC. They aren't needed.Last time I looked, the Coda file system was broken and would not compile if __linux__ wasn't defined. This is the *only* thing that prevents you from building a kernel with a generic $ARCH-elf toolchain, IIRC.
The reason we defined it in the C6X Makefile was that the TI compiler did not define it for us. GCC does define it, so we're good without it the makefile now. It looks like coda still needs it as well as drm, scsi/aic7xxx driver, and a couple other places. --Mark