Re: [linuxsh-dev] sh-sci.c patch

[M. R. B. <ma...@uw...>]

On Tue, 13 Feb 2001, Bryan Rittmeyer wrote:

> Hello,
> 
> Today I tried building the latest CVS kernel with a full GNU CVS
> toolchain, unpatched (yes, I _am_ completely crazy). Everything built OK
> except for sh-sci.c,
> which expects __sh3__ to be defined only for SH3 CPUs. However, the new
> toolchain seems to define __sh3__ even for the SH4 (maybe the old gcc in
> LinuxSH CVS does also? I don't remember). Anyway to get vmlinux to
> build, I modified the sh-sci.c pre-processor logic to use CONFIG_CPU
> instead... here's the patch if anyone else is having this problem:
>

I posted a question regarding this a couple days ago.  From my
understanding, the -m4-nofpu switch in the arch/sh/Makefile is the
culprit, in the gcc specs it defines __sh3__ with no mention of __SH4__. 
I suspect that switch could break other things semi-randomly in the
future, I also remember problems with the include/asm-sh/irq.h header.  My
interim solution is to just change the flag in the kernel Makefile to -m4. 
Thanks for the patch.

M. R.