linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] arch: avr32: Makefile: add '-D__linux__' flag for gcc-4.4.7 using.
@ 2014-02-01 12:35 Chen Gang
  2014-02-01 12:56 ` Richard Weinberger
  2014-02-17  9:50 ` Hans-Christian Egtvedt
  0 siblings, 2 replies; 5+ messages in thread
From: Chen Gang @ 2014-02-01 12:35 UTC (permalink / raw)
  To: hskinnemoen, egtvedt; +Cc: linux-kernel

For avr32 cross compiler, do not define '__linux__' internally, so it
will cause issue with allmodconfig.

The related error:

    CC [M]  fs/coda/psdev.o
  In file included from include/linux/coda.h:64,
                   from fs/coda/psdev.c:45:
  include/uapi/linux/coda.h:221: error: expected specifier-qualifier-list before 'u_quad_t'

The related toolchain version (which only download, not re-compile):

  [root@gchen linux-next]# /upstream/toolchain/download/avr32-gnu-toolchain-linux_x86/bin/avr32-gcc -v
  Using built-in specs.
  Target: avr32
  Configured with: /data2/home/toolsbuild/jenkins-knuth/workspace/avr32-gnu-toolchain/src/gcc/configure --target=avr32 --host=i686-pc-linux-gnu --build=x86_64-pc-linux-gnu --prefix=/home/toolsbuild/jenkins-knuth/workspace/avr32-gnu-toolchain/avr32-gnu-toolchain-linux_x86 --enable-languages=c,c++ --disable-nls --disable-libssp --disable-libstdcxx-pch --with-dwarf2 --enable-version-specific-runtime-libs --disable-shared --enable-doc --with-mpfr-lib=/home/toolsbuild/jenkins-knuth/workspace/avr32-gnu-toolchain/avr32-gnu-toolchain-linux_x86/lib --with-mpfr-include=/home/toolsbuild/jenkins-knuth/workspace/avr32-gnu-toolchain/avr32-gnu-toolchain-linux_x86/include --with-gmp=/home/toolsbuild/jenkins-knuth/workspace/avr32-gnu-toolchain/avr32-gnu-toolchain-linux_x86 --with-mpc=/home/toolsbuild/jenkins-knuth/workspace/avr32-gnu-toolchain/avr32-gnu-toolchain-linux_x86 --enable-__cxa_atexit --disable-shared --with-newlib --with-pkgversion=AVR_32_bit_GNU_Toolchain_3.4.2_435 --with-bugurl=http://www
.atmel.com/avr
  Thread model: single
  gcc version 4.4.7 (AVR_32_bit_GNU_Toolchain_3.4.2_435) 


Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com>
---
 arch/avr32/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/avr32/Makefile b/arch/avr32/Makefile
index 22fb665..dba48a5 100644
--- a/arch/avr32/Makefile
+++ b/arch/avr32/Makefile
@@ -11,7 +11,7 @@ all: uImage vmlinux.elf
 
 KBUILD_DEFCONFIG	:= atstk1002_defconfig
 
-KBUILD_CFLAGS	+= -pipe -fno-builtin -mno-pic
+KBUILD_CFLAGS	+= -pipe -fno-builtin -mno-pic -D__linux__
 KBUILD_AFLAGS	+= -mrelax -mno-pic
 KBUILD_CFLAGS_MODULE += -mno-relax
 LDFLAGS_vmlinux	+= --relax
-- 
1.7.11.7

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH] arch: avr32: Makefile: add '-D__linux__' flag for gcc-4.4.7 using.
  2014-02-01 12:35 [PATCH] arch: avr32: Makefile: add '-D__linux__' flag for gcc-4.4.7 using Chen Gang
@ 2014-02-01 12:56 ` Richard Weinberger
  2014-02-01 13:46   ` Chen Gang
  2014-02-17  9:50 ` Hans-Christian Egtvedt
  1 sibling, 1 reply; 5+ messages in thread
From: Richard Weinberger @ 2014-02-01 12:56 UTC (permalink / raw)
  To: Chen Gang; +Cc: hskinnemoen, egtvedt, linux-kernel

On Sat, Feb 1, 2014 at 1:35 PM, Chen Gang <gang.chen.5i5j@gmail.com> wrote:
> For avr32 cross compiler, do not define '__linux__' internally, so it
> will cause issue with allmodconfig.

Isn't this a toolchain bug?

> The related error:
>
>     CC [M]  fs/coda/psdev.o
>   In file included from include/linux/coda.h:64,
>                    from fs/coda/psdev.c:45:
>   include/uapi/linux/coda.h:221: error: expected specifier-qualifier-list before 'u_quad_t'
>
> The related toolchain version (which only download, not re-compile):
>
>   [root@gchen linux-next]# /upstream/toolchain/download/avr32-gnu-toolchain-linux_x86/bin/avr32-gcc -v
>   Using built-in specs.
>   Target: avr32
>   Configured with: /data2/home/toolsbuild/jenkins-knuth/workspace/avr32-gnu-toolchain/src/gcc/configure --target=avr32 --host=i686-pc-linux-gnu --build=x86_64-pc-linux-gnu --prefix=/home/toolsbuild/jenkins-knuth/workspace/avr32-gnu-toolchain/avr32-gnu-toolchain-linux_x86 --enable-languages=c,c++ --disable-nls --disable-libssp --disable-libstdcxx-pch --with-dwarf2 --enable-version-specific-runtime-libs --disable-shared --enable-doc --with-mpfr-lib=/home/toolsbuild/jenkins-knuth/workspace/avr32-gnu-toolchain/avr32-gnu-toolchain-linux_x86/lib --with-mpfr-include=/home/toolsbuild/jenkins-knuth/workspace/avr32-gnu-toolchain/avr32-gnu-toolchain-linux_x86/include --with-gmp=/home/toolsbuild/jenkins-knuth/workspace/avr32-gnu-toolchain/avr32-gnu-toolchain-linux_x86 --with-mpc=/home/toolsbuild/jenkins-knuth/workspace/avr32-gnu-toolchain/avr32-gnu-toolchain-linux_x86 --enable-__cxa_atexit --disable-shared --with-newlib --with-pkgversion=AVR_32_bit_GNU_Toolchain_3.4.2_435 --with-bugurl=http://www
> .atmel.com/avr
>   Thread model: single
>   gcc version 4.4.7 (AVR_32_bit_GNU_Toolchain_3.4.2_435)
>
>
> Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com>
> ---
>  arch/avr32/Makefile | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/avr32/Makefile b/arch/avr32/Makefile
> index 22fb665..dba48a5 100644
> --- a/arch/avr32/Makefile
> +++ b/arch/avr32/Makefile
> @@ -11,7 +11,7 @@ all: uImage vmlinux.elf
>
>  KBUILD_DEFCONFIG       := atstk1002_defconfig
>
> -KBUILD_CFLAGS  += -pipe -fno-builtin -mno-pic
> +KBUILD_CFLAGS  += -pipe -fno-builtin -mno-pic -D__linux__
>  KBUILD_AFLAGS  += -mrelax -mno-pic
>  KBUILD_CFLAGS_MODULE += -mno-relax
>  LDFLAGS_vmlinux        += --relax
> --
> 1.7.11.7
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/



-- 
Thanks,
//richard

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] arch: avr32: Makefile: add '-D__linux__' flag for gcc-4.4.7 using.
  2014-02-01 12:56 ` Richard Weinberger
@ 2014-02-01 13:46   ` Chen Gang
  2014-02-16 11:40     ` Chen Gang
  0 siblings, 1 reply; 5+ messages in thread
From: Chen Gang @ 2014-02-01 13:46 UTC (permalink / raw)
  To: Richard Weinberger; +Cc: hskinnemoen, egtvedt, linux-kernel

On 02/01/2014 08:56 PM, Richard Weinberger wrote:
> On Sat, Feb 1, 2014 at 1:35 PM, Chen Gang <gang.chen.5i5j@gmail.com> wrote:
>> For avr32 cross compiler, do not define '__linux__' internally, so it
>> will cause issue with allmodconfig.
> 
> Isn't this a toolchain bug?

Hmm... for me, it is not toolchain's bug, it is a feature issue in gcc
old version, and in newer version it is OK (in my memory, after gcc-4.7,
it is OK).

In our "arch/*", really contents some another architectures which
already use "-D__linux__" in their "Makefile". :-)

Thanks.

> 
>> The related error:
>>
>>     CC [M]  fs/coda/psdev.o
>>   In file included from include/linux/coda.h:64,
>>                    from fs/coda/psdev.c:45:
>>   include/uapi/linux/coda.h:221: error: expected specifier-qualifier-list before 'u_quad_t'
>>
>> The related toolchain version (which only download, not re-compile):
>>
>>   [root@gchen linux-next]# /upstream/toolchain/download/avr32-gnu-toolchain-linux_x86/bin/avr32-gcc -v
>>   Using built-in specs.
>>   Target: avr32
>>   Configured with: /data2/home/toolsbuild/jenkins-knuth/workspace/avr32-gnu-toolchain/src/gcc/configure --target=avr32 --host=i686-pc-linux-gnu --build=x86_64-pc-linux-gnu --prefix=/home/toolsbuild/jenkins-knuth/workspace/avr32-gnu-toolchain/avr32-gnu-toolchain-linux_x86 --enable-languages=c,c++ --disable-nls --disable-libssp --disable-libstdcxx-pch --with-dwarf2 --enable-version-specific-runtime-libs --disable-shared --enable-doc --with-mpfr-lib=/home/toolsbuild/jenkins-knuth/workspace/avr32-gnu-toolchain/avr32-gnu-toolchain-linux_x86/lib --with-mpfr-include=/home/toolsbuild/jenkins-knuth/workspace/avr32-gnu-toolchain/avr32-gnu-toolchain-linux_x86/include --with-gmp=/home/toolsbuild/jenkins-knuth/workspace/avr32-gnu-toolchain/avr32-gnu-toolchain-linux_x86 --with-mpc=/home/toolsbuild/jenkins-knuth/workspace/avr32-gnu-toolchain/avr32-gnu-toolchain-linux_x86 --enable-__cxa_atexit --disable-shared --with-newlib --with-pkgversion=AVR_32_bit_GNU_Toolchain_3.4.2_435 --with-bugurl=http://
www
>> .atmel.com/avr
>>   Thread model: single
>>   gcc version 4.4.7 (AVR_32_bit_GNU_Toolchain_3.4.2_435)
>>
>>
>> Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com>
>> ---
>>  arch/avr32/Makefile | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/arch/avr32/Makefile b/arch/avr32/Makefile
>> index 22fb665..dba48a5 100644
>> --- a/arch/avr32/Makefile
>> +++ b/arch/avr32/Makefile
>> @@ -11,7 +11,7 @@ all: uImage vmlinux.elf
>>
>>  KBUILD_DEFCONFIG       := atstk1002_defconfig
>>
>> -KBUILD_CFLAGS  += -pipe -fno-builtin -mno-pic
>> +KBUILD_CFLAGS  += -pipe -fno-builtin -mno-pic -D__linux__
>>  KBUILD_AFLAGS  += -mrelax -mno-pic
>>  KBUILD_CFLAGS_MODULE += -mno-relax
>>  LDFLAGS_vmlinux        += --relax
>> --
>> 1.7.11.7
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
>> the body of a message to majordomo@vger.kernel.org
>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>> Please read the FAQ at  http://www.tux.org/lkml/
> 
> 
> 


-- 
Chen Gang

Open, share and attitude like air, water and life which God blessed

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] arch: avr32: Makefile: add '-D__linux__' flag for gcc-4.4.7 using.
  2014-02-01 13:46   ` Chen Gang
@ 2014-02-16 11:40     ` Chen Gang
  0 siblings, 0 replies; 5+ messages in thread
From: Chen Gang @ 2014-02-16 11:40 UTC (permalink / raw)
  To: Richard Weinberger; +Cc: hskinnemoen, egtvedt, linux-kernel

Hello Maintainers:

Please help check this patch, when you have free time.

Thanks.


On 02/01/2014 09:46 PM, Chen Gang wrote:
> On 02/01/2014 08:56 PM, Richard Weinberger wrote:
>> On Sat, Feb 1, 2014 at 1:35 PM, Chen Gang <gang.chen.5i5j@gmail.com> wrote:
>>> For avr32 cross compiler, do not define '__linux__' internally, so it
>>> will cause issue with allmodconfig.
>>
>> Isn't this a toolchain bug?
> 
> Hmm... for me, it is not toolchain's bug, it is a feature issue in gcc
> old version, and in newer version it is OK (in my memory, after gcc-4.7,
> it is OK).
> 
> In our "arch/*", really contents some another architectures which
> already use "-D__linux__" in their "Makefile". :-)
> 
> Thanks.
> 
>>
>>> The related error:
>>>
>>>     CC [M]  fs/coda/psdev.o
>>>   In file included from include/linux/coda.h:64,
>>>                    from fs/coda/psdev.c:45:
>>>   include/uapi/linux/coda.h:221: error: expected specifier-qualifier-list before 'u_quad_t'
>>>
>>> The related toolchain version (which only download, not re-compile):
>>>
>>>   [root@gchen linux-next]# /upstream/toolchain/download/avr32-gnu-toolchain-linux_x86/bin/avr32-gcc -v
>>>   Using built-in specs.
>>>   Target: avr32
>>>   Configured with: /data2/home/toolsbuild/jenkins-knuth/workspace/avr32-gnu-toolchain/src/gcc/configure --target=avr32 --host=i686-pc-linux-gnu --build=x86_64-pc-linux-gnu --prefix=/home/toolsbuild/jenkins-knuth/workspace/avr32-gnu-toolchain/avr32-gnu-toolchain-linux_x86 --enable-languages=c,c++ --disable-nls --disable-libssp --disable-libstdcxx-pch --with-dwarf2 --enable-version-specific-runtime-libs --disable-shared --enable-doc --with-mpfr-lib=/home/toolsbuild/jenkins-knuth/workspace/avr32-gnu-toolchain/avr32-gnu-toolchain-linux_x86/lib --with-mpfr-include=/home/toolsbuild/jenkins-knuth/workspace/avr32-gnu-toolchain/avr32-gnu-toolchain-linux_x86/include --with-gmp=/home/toolsbuild/jenkins-knuth/workspace/avr32-gnu-toolchain/avr32-gnu-toolchain-linux_x86 --with-mpc=/home/toolsbuild/jenkins-knuth/workspace/avr32-gnu-toolchain/avr32-gnu-toolchain-linux_x86 --enable-__cxa_atexit --disable-shared --with-newlib --with-pkgversion=AVR_32_bit_GNU_Toolchain_3.4.2_435 --with-bugurl=http:/
/
> www
>>> .atmel.com/avr
>>>   Thread model: single
>>>   gcc version 4.4.7 (AVR_32_bit_GNU_Toolchain_3.4.2_435)
>>>
>>>
>>> Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com>
>>> ---
>>>  arch/avr32/Makefile | 2 +-
>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/arch/avr32/Makefile b/arch/avr32/Makefile
>>> index 22fb665..dba48a5 100644
>>> --- a/arch/avr32/Makefile
>>> +++ b/arch/avr32/Makefile
>>> @@ -11,7 +11,7 @@ all: uImage vmlinux.elf
>>>
>>>  KBUILD_DEFCONFIG       := atstk1002_defconfig
>>>
>>> -KBUILD_CFLAGS  += -pipe -fno-builtin -mno-pic
>>> +KBUILD_CFLAGS  += -pipe -fno-builtin -mno-pic -D__linux__
>>>  KBUILD_AFLAGS  += -mrelax -mno-pic
>>>  KBUILD_CFLAGS_MODULE += -mno-relax
>>>  LDFLAGS_vmlinux        += --relax
>>> --
>>> 1.7.11.7
>>> --
>>> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
>>> the body of a message to majordomo@vger.kernel.org
>>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>>> Please read the FAQ at  http://www.tux.org/lkml/
>>
>>
>>
> 
> 


-- 
Chen Gang

Open, share and attitude like air, water and life which God blessed

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] arch: avr32: Makefile: add '-D__linux__' flag for gcc-4.4.7 using.
  2014-02-01 12:35 [PATCH] arch: avr32: Makefile: add '-D__linux__' flag for gcc-4.4.7 using Chen Gang
  2014-02-01 12:56 ` Richard Weinberger
@ 2014-02-17  9:50 ` Hans-Christian Egtvedt
  1 sibling, 0 replies; 5+ messages in thread
From: Hans-Christian Egtvedt @ 2014-02-17  9:50 UTC (permalink / raw)
  To: Chen Gang; +Cc: hskinnemoen, linux-kernel

Around Sat 01 Feb 2014 20:35:54 +0800 or thereabout, Chen Gang wrote:
> For avr32 cross compiler, do not define '__linux__' internally, so it
> will cause issue with allmodconfig.
> 
> The related error:
> 
>     CC [M]  fs/coda/psdev.o
>   In file included from include/linux/coda.h:64,
>                    from fs/coda/psdev.c:45:
>   include/uapi/linux/coda.h:221: error: expected specifier-qualifier-list before 'u_quad_t'
> 
> The related toolchain version (which only download, not re-compile):
> 
>   [root@gchen linux-next]# /upstream/toolchain/download/avr32-gnu-toolchain-linux_x86/bin/avr32-gcc -v
>   Using built-in specs.
>   Target: avr32
>   Configured with: /data2/home/toolsbuild/jenkins-knuth/workspace/avr32-gnu-toolchain/src/gcc/configure --target=avr32 --host=i686-pc-linux-gnu --build=x86_64-pc-linux-gnu --prefix=/home/toolsbuild/jenkins-knuth/workspace/avr32-gnu-toolchain/avr32-gnu-toolchain-linux_x86 --enable-languages=c,c++ --disable-nls --disable-libssp --disable-libstdcxx-pch --with-dwarf2 --enable-version-specific-runtime-libs --disable-shared --enable-doc --with-mpfr-lib=/home/toolsbuild/jenkins-knuth/workspace/avr32-gnu-toolchain/avr32-gnu-toolchain-linux_x86/lib --with-mpfr-include=/home/toolsbuild/jenkins-knuth/workspace/avr32-gnu-toolchain/avr32-gnu-toolchain-linux_x86/include --with-gmp=/home/toolsbuild/jenkins-knuth/workspace/avr32-gnu-toolchain/avr32-gnu-toolchain-linux_x86 --with-mpc=/home/toolsbuild/jenkins-knuth/workspace/avr32-gnu-toolchain/avr32-gnu-toolchain-linux_x86 --enable-__cxa_atexit --disable-shared --with-newlib --with-pkgversion=AVR_32_bit_GNU_Toolchain_3.4.2_435 --with-bugurl=http://www
> .atmel.com/avr
>   Thread model: single
>   gcc version 4.4.7 (AVR_32_bit_GNU_Toolchain_3.4.2_435) 
> 
> Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com>

Acked-by: Hans-Christian Egtvedt <egtvedt@samfundet.no>

I will merge this into the avr32 tree in the for-linus branch. AFAICT this is
also suitable for stable, so I'll add the appropriate CC.

> ---
>  arch/avr32/Makefile | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/avr32/Makefile b/arch/avr32/Makefile
> index 22fb665..dba48a5 100644
> --- a/arch/avr32/Makefile
> +++ b/arch/avr32/Makefile
> @@ -11,7 +11,7 @@ all: uImage vmlinux.elf
>  
>  KBUILD_DEFCONFIG	:= atstk1002_defconfig
>  
> -KBUILD_CFLAGS	+= -pipe -fno-builtin -mno-pic
> +KBUILD_CFLAGS	+= -pipe -fno-builtin -mno-pic -D__linux__
>  KBUILD_AFLAGS	+= -mrelax -mno-pic
>  KBUILD_CFLAGS_MODULE += -mno-relax
>  LDFLAGS_vmlinux	+= --relax
-- 
mvh
Hans-Christian Egtvedt

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2014-02-17  9:50 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-02-01 12:35 [PATCH] arch: avr32: Makefile: add '-D__linux__' flag for gcc-4.4.7 using Chen Gang
2014-02-01 12:56 ` Richard Weinberger
2014-02-01 13:46   ` Chen Gang
2014-02-16 11:40     ` Chen Gang
2014-02-17  9:50 ` Hans-Christian Egtvedt

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).