Re: [PATCH kernel v3] powerpc/makefile: Do not redefine $(CPP) for preprocessor

2021-05-17 Thread Segher Boessenkool
Hi! On Mon, May 17, 2021 at 01:23:11PM +1000, Alexey Kardashevskiy wrote: > On 5/14/21 18:46, Segher Boessenkool wrote: > >On Fri, May 14, 2021 at 11:42:32AM +0900, Masahiro Yamada wrote: > >>In my best guess, the reason why powerpc adding the endian flag to CPP > >>is this line in

Re: [PATCH kernel v3] powerpc/makefile: Do not redefine $(CPP) for preprocessor

2021-05-16 Thread Alexey Kardashevskiy
On 5/14/21 18:46, Segher Boessenkool wrote: Hi! On Fri, May 14, 2021 at 11:42:32AM +0900, Masahiro Yamada wrote: In my best guess, the reason why powerpc adding the endian flag to CPP is this line in arch/powerpc/kernel/vdso64/vdso64.lds.S #ifdef __LITTLE_ENDIAN__

Re: [PATCH kernel v3] powerpc/makefile: Do not redefine $(CPP) for preprocessor

2021-05-14 Thread Segher Boessenkool
Hi! On Fri, May 14, 2021 at 11:42:32AM +0900, Masahiro Yamada wrote: > In my best guess, the reason why powerpc adding the endian flag to CPP > is this line in arch/powerpc/kernel/vdso64/vdso64.lds.S > > #ifdef __LITTLE_ENDIAN__ > OUTPUT_FORMAT("elf64-powerpcle", "elf64-powerpcle",

Re: [PATCH kernel v3] powerpc/makefile: Do not redefine $(CPP) for preprocessor

2021-05-13 Thread Alexey Kardashevskiy
On 14/05/2021 12:42, Masahiro Yamada wrote: On Fri, May 14, 2021 at 3:59 AM Nathan Chancellor wrote: On 5/13/2021 4:59 AM, Alexey Kardashevskiy wrote: The $(CPP) (do only preprocessing) macro is already defined in Makefile. However POWERPC redefines it and adds $(KBUILD_CFLAGS) which

Re: [PATCH kernel v3] powerpc/makefile: Do not redefine $(CPP) for preprocessor

2021-05-13 Thread Masahiro Yamada
On Fri, May 14, 2021 at 3:59 AM Nathan Chancellor wrote: > > On 5/13/2021 4:59 AM, Alexey Kardashevskiy wrote: > > The $(CPP) (do only preprocessing) macro is already defined in Makefile. > > However POWERPC redefines it and adds $(KBUILD_CFLAGS) which results > > in flags duplication. Which is

Re: [PATCH kernel v3] powerpc/makefile: Do not redefine $(CPP) for preprocessor

2021-05-13 Thread Alexey Kardashevskiy
On 14/05/2021 04:59, Nathan Chancellor wrote: On 5/13/2021 4:59 AM, Alexey Kardashevskiy wrote: The $(CPP) (do only preprocessing) macro is already defined in Makefile. However POWERPC redefines it and adds $(KBUILD_CFLAGS) which results in flags duplication. Which is not a big deal by

Re: [PATCH kernel v3] powerpc/makefile: Do not redefine $(CPP) for preprocessor

2021-05-13 Thread Nathan Chancellor
On 5/13/2021 4:59 AM, Alexey Kardashevskiy wrote: The $(CPP) (do only preprocessing) macro is already defined in Makefile. However POWERPC redefines it and adds $(KBUILD_CFLAGS) which results in flags duplication. Which is not a big deal by itself except for the flags which depend on other flags

[PATCH kernel v3] powerpc/makefile: Do not redefine $(CPP) for preprocessor

2021-05-13 Thread Alexey Kardashevskiy
The $(CPP) (do only preprocessing) macro is already defined in Makefile. However POWERPC redefines it and adds $(KBUILD_CFLAGS) which results in flags duplication. Which is not a big deal by itself except for the flags which depend on other flags and the compiler checks them as it parses the