Re: [PATCH 04/24] C6X: build infrastructure

Mark Salter
2011-08-24T14:10:47Z

On Wed, 2011-08-24 at 14:47 +0100, David Woodhouse wrote:
> On Tue, 2011-08-23 at 09:23 -0400, Mark Salter wrote:
> >
> > > > +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