Re: [GIT repo] loongson: Merge and Clean up fuloong(2e), fuloong(2f) and yeeloong(2f) support

[Wu Zhangjin <wuzhangjin@gmail.com>]

On Sat, 2009-05-16 at 01:09 +0800, Zhang Le wrote:
> On 18:38 Fri 15 May     , Wu Zhangjin wrote:
> > On Fri, 2009-05-15 at 10:25 +0200, Arnaud Patard wrote:
> > > - even if it should not affect the kernel, compiling with
> > >   -march=loongson2f even for 2e (you're matching on loongson2 so 2e and
> > >   2f) looks weird.
> > 
> > sorry, this is really a very obvious error, in 2e, -march=loongson2e
> > should be used.
> > 
> > to fix this problem, perhaps we can add two new kernel options:
> > 
> > config CPU_LOONGSON2E
> > 	bool
> > 
> > config CPU_LOONGSON2F
> > 	bool
> > 
> > and then use this solution:
> > 
> > config FULOONG2E
> > 	...
> > 	select CPU_LOONGSON2E
> > 	...
> > 
> > config YEELOONG2F
> > 	...
> > 	select CPU_LOONGSON2F
> > 	...
> > 
> > cflags-$(CONFIG_CPU_LOONGSON2E)  += -march=loongson2e -Wa,--trap
> > cflags-$(CONFIG_CPU_LOONGSON2F)  += -march=loongson2f -Wa,--trap
> > 
> > is this solution okay?
> 
> I have tried to solve this problem once. At that time, there is no loongson-dev
> list. So I send my patch to yanhua and zhang fuxin directly. It seems that they
> didn't forward that patch to you.
> 
> For the whole patch, please check attachment. It does not applicable on top of
> your work any more.
> 
> But here is how I deal with the CPU types:
> 
> @@ -951,14 +929,20 @@ choice
>        prompt "CPU type"
>        default CPU_R4X00
> 
> -config CPU_LOONGSON2
> -       bool "Loongson 2"
> -       depends on SYS_HAS_CPU_LOONGSON2
> -       select CPU_SUPPORTS_32BIT_KERNEL
> -       select CPU_SUPPORTS_64BIT_KERNEL
> -       select CPU_SUPPORTS_HIGHMEM
> +config CPU_LOONGSON2E
> +       bool "Loongson 2E"
> +       depends on SYS_HAS_CPU_LOONGSON2E
> +       select CPU_LOONGSON2
> +       help
> +         The Loongson 2E processor implements the MIPS III instruction set
> +         with many extensions.
> +
> +config CPU_LOONGSON2F
> +       bool "Loongson 2F"
> +       depends on SYS_HAS_CPU_LOONGSON2F
> +       select CPU_LOONGSON2
>         help
> -         The Loongson 2E/2F processor implements the MIPS III instruction set
> +         The Loongson 2F processor implements the MIPS III instruction set
>          with many extensions.
> 
>  config CPU_MIPS32_R1
> @@ -1171,7 +1155,16 @@ config CPU_SB1
> 
>  endchoice
> 
> -config SYS_HAS_CPU_LOONGSON2
> +config CPU_LOONGSON2
> +       bool
> +       select CPU_SUPPORTS_32BIT_KERNEL
> +       select CPU_SUPPORTS_64BIT_KERNEL
> +       select CPU_SUPPORTS_HIGHMEM
> +
> +config SYS_HAS_CPU_LOONGSON2E
> +       bool
> +
> +config SYS_HAS_CPU_LOONGSON2F
>         bool
> 
>  config SYS_HAS_CPU_MIPS32_R1
> 

applied it, thanks!