* [PATCH] arm64: compiling issue, need add include/asm/vga.h file @ 2013-04-24 3:44 ` Chen Gang 0 siblings, 0 replies; 34+ messages in thread From: Chen Gang @ 2013-04-24 3:44 UTC (permalink / raw) To: Catalin Marinas, Will Deacon, Arnd Bergmann Cc: linux-arm-kernel, linaro-kernel, linux-kernel For compiling with allmodconfig, need vga.h file, so copy it from arm32. The related information: aarch64-linux-gnu-gcc -Wp,-MD,drivers/gpu/drm/.drm_irq.o.d -nostdinc -isystem /usr/bin/../lib/gcc/aarch64-linux-gnu/4.7.3/include -I/root/linux-next/arch/arm64/include -Iarch/arm64/include/generated -Iinclude -I/root/linux-next/arch/arm64/include/uapi -Iarch/arm64/include/generated/uapi -I/root/linux-next/include/uapi -Iinclude/generated/uapi -include /root/linux-next/include/linux/kconfig.h -D__KERNEL__ -mlittle-endian -Wall -Wundef -Wstrict-prototypes -Wno-trigraphs -fno-strict-aliasing -fno-common -Werror-implicit-function-declaration -Wno-format-security -fno-delete-null-pointer-checks -Os -mgeneral-regs-only -fno-reorder-blocks -fno-ipa-cp-clone -fno-partial-inlining -Wframe-larger-than=2048 -fno-stack-protector -Wno-unused-but-set-variable -fno-omit-frame-pointer -fno-optimize-sibling-calls -g -femit-struct-debug-baseonly -fno-var-tracking -fno-inline-functions-called-once -Wdeclaration-after-statement -Wno-pointer-sign -fno-strict-overflow -fconserve-stack -DCC_HAV E_ASM_GOT O -Iinclude/drm -W -DMODULE -D"KBUILD_STR(s)=#s" -D"KBUILD_BASENAME=KBUILD_STR(drm_irq)" -D"KBUILD_MODNAME=KBUILD_STR(drm)" -c -o drivers/gpu/drm/.tmp_drm_irq.o drivers/gpu/drm/drm_irq.c The related error: include/video/vga.h: At top level: include/video/vga.h:22:21: fatal error: asm/vga.h: No such file or directory compilation terminated. make[3]: *** [drivers/gpu/drm/drm_irq.o] Error 1 make[2]: *** [drivers/gpu/drm] Error 2 make[1]: *** [drivers/gpu] Error 2 make: *** [drivers] Error 2 Signed-off-by: root <root@dhcp122.asianux.net> --- arch/arm64/include/asm/vga.h | 13 +++++++++++++ 1 file changed, 13 insertions(+) create mode 100644 arch/arm64/include/asm/vga.h diff --git a/arch/arm64/include/asm/vga.h b/arch/arm64/include/asm/vga.h new file mode 100644 index 0000000..91f4021 --- /dev/null +++ b/arch/arm64/include/asm/vga.h @@ -0,0 +1,13 @@ +#ifndef ASMARM_VGA_H +#define ASMARM_VGA_H + +#include <linux/io.h> + +extern unsigned long vga_base; + +#define VGA_MAP_MEM(x,s) (vga_base + (x)) + +#define vga_readb(x) (*((volatile unsigned char *)x)) +#define vga_writeb(x,y) (*((volatile unsigned char *)y) = (x)) + +#endif -- 1.7.11.7 ^ permalink raw reply related [flat|nested] 34+ messages in thread
* [PATCH] arm64: compiling issue, need add include/asm/vga.h file @ 2013-04-24 3:44 ` Chen Gang 0 siblings, 0 replies; 34+ messages in thread From: Chen Gang @ 2013-04-24 3:44 UTC (permalink / raw) To: linux-arm-kernel For compiling with allmodconfig, need vga.h file, so copy it from arm32. The related information: aarch64-linux-gnu-gcc -Wp,-MD,drivers/gpu/drm/.drm_irq.o.d -nostdinc -isystem /usr/bin/../lib/gcc/aarch64-linux-gnu/4.7.3/include -I/root/linux-next/arch/arm64/include -Iarch/arm64/include/generated -Iinclude -I/root/linux-next/arch/arm64/include/uapi -Iarch/arm64/include/generated/uapi -I/root/linux-next/include/uapi -Iinclude/generated/uapi -include /root/linux-next/include/linux/kconfig.h -D__KERNEL__ -mlittle-endian -Wall -Wundef -Wstrict-prototypes -Wno-trigraphs -fno-strict-aliasing -fno-common -Werror-implicit-function-declaration -Wno-format-security -fno-delete-null-pointer-checks -Os -mgeneral-regs-only -fno-reorder-blocks -fno-ipa-cp-clone -fno-partial-inlining -Wframe-larger-than=2048 -fno-stack-protector -Wno-unused-but-set-variable -fno-omit-frame-pointer -fno-optimize-sibling-calls -g -femit-struct-debug-baseonly -fno-var-tracking -fno-inline-functions-called-once -Wdeclaration-after-statement -Wno-pointer-sign -fno-strict-overflow -fconserve-stack -DCC_HAV E_ASM_GOT O -Iinclude/drm -W -DMODULE -D"KBUILD_STR(s)=#s" -D"KBUILD_BASENAME=KBUILD_STR(drm_irq)" -D"KBUILD_MODNAME=KBUILD_STR(drm)" -c -o drivers/gpu/drm/.tmp_drm_irq.o drivers/gpu/drm/drm_irq.c The related error: include/video/vga.h: At top level: include/video/vga.h:22:21: fatal error: asm/vga.h: No such file or directory compilation terminated. make[3]: *** [drivers/gpu/drm/drm_irq.o] Error 1 make[2]: *** [drivers/gpu/drm] Error 2 make[1]: *** [drivers/gpu] Error 2 make: *** [drivers] Error 2 Signed-off-by: root <root@dhcp122.asianux.net> --- arch/arm64/include/asm/vga.h | 13 +++++++++++++ 1 file changed, 13 insertions(+) create mode 100644 arch/arm64/include/asm/vga.h diff --git a/arch/arm64/include/asm/vga.h b/arch/arm64/include/asm/vga.h new file mode 100644 index 0000000..91f4021 --- /dev/null +++ b/arch/arm64/include/asm/vga.h @@ -0,0 +1,13 @@ +#ifndef ASMARM_VGA_H +#define ASMARM_VGA_H + +#include <linux/io.h> + +extern unsigned long vga_base; + +#define VGA_MAP_MEM(x,s) (vga_base + (x)) + +#define vga_readb(x) (*((volatile unsigned char *)x)) +#define vga_writeb(x,y) (*((volatile unsigned char *)y) = (x)) + +#endif -- 1.7.11.7 ^ permalink raw reply related [flat|nested] 34+ messages in thread
* Re: [PATCH] arm64: compiling issue, need add include/asm/vga.h file 2013-04-24 3:44 ` Chen Gang @ 2013-04-24 8:22 ` Catalin Marinas -1 siblings, 0 replies; 34+ messages in thread From: Catalin Marinas @ 2013-04-24 8:22 UTC (permalink / raw) To: Chen Gang Cc: Will Deacon, Arnd Bergmann, linux-arm-kernel, linaro-kernel, linux-kernel On Wed, Apr 24, 2013 at 04:44:29AM +0100, Chen Gang wrote: > For compiling with allmodconfig, need vga.h file, so copy it from arm32. I would rather use the generic vga.h. We don't even have a vga_base defined. -- Catalin ^ permalink raw reply [flat|nested] 34+ messages in thread
* [PATCH] arm64: compiling issue, need add include/asm/vga.h file @ 2013-04-24 8:22 ` Catalin Marinas 0 siblings, 0 replies; 34+ messages in thread From: Catalin Marinas @ 2013-04-24 8:22 UTC (permalink / raw) To: linux-arm-kernel On Wed, Apr 24, 2013 at 04:44:29AM +0100, Chen Gang wrote: > For compiling with allmodconfig, need vga.h file, so copy it from arm32. I would rather use the generic vga.h. We don't even have a vga_base defined. -- Catalin ^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: [PATCH] arm64: compiling issue, need add include/asm/vga.h file 2013-04-24 8:22 ` Catalin Marinas @ 2013-04-24 8:44 ` Chen Gang -1 siblings, 0 replies; 34+ messages in thread From: Chen Gang @ 2013-04-24 8:44 UTC (permalink / raw) To: Catalin Marinas Cc: Will Deacon, Arnd Bergmann, linux-arm-kernel, linaro-kernel, linux-kernel On 2013年04月24日 16:22, Catalin Marinas wrote: > On Wed, Apr 24, 2013 at 04:44:29AM +0100, Chen Gang wrote: >> For compiling with allmodconfig, need vga.h file, so copy it from arm32. > > I would rather use the generic vga.h. We don't even have a vga_base > defined. > OK, thanks. I need send patch v2. -- Chen Gang Asianux Corporation ^ permalink raw reply [flat|nested] 34+ messages in thread
* [PATCH] arm64: compiling issue, need add include/asm/vga.h file @ 2013-04-24 8:44 ` Chen Gang 0 siblings, 0 replies; 34+ messages in thread From: Chen Gang @ 2013-04-24 8:44 UTC (permalink / raw) To: linux-arm-kernel On 2013?04?24? 16:22, Catalin Marinas wrote: > On Wed, Apr 24, 2013 at 04:44:29AM +0100, Chen Gang wrote: >> For compiling with allmodconfig, need vga.h file, so copy it from arm32. > > I would rather use the generic vga.h. We don't even have a vga_base > defined. > OK, thanks. I need send patch v2. -- Chen Gang Asianux Corporation ^ permalink raw reply [flat|nested] 34+ messages in thread
* [PATCH v2] arm64: compiling issue, need add include/asm/vga.h file 2013-04-24 8:22 ` Catalin Marinas @ 2013-04-24 10:20 ` Chen Gang -1 siblings, 0 replies; 34+ messages in thread From: Chen Gang @ 2013-04-24 10:20 UTC (permalink / raw) To: Catalin Marinas Cc: Will Deacon, Arnd Bergmann, linux-arm-kernel, linaro-kernel, linux-kernel For compiling with allmodconfig, need vga.h file, so generate it which just only include the asm-generic one. It is firstly used by drivers/gpu/drm/drm_irq.c. The related error: include/video/vga.h:22:21: fatal error: asm/vga.h: No such file or directory Signed-off-by: Chen Gang <gang.chen@asianux.com> --- arch/arm64/include/asm/vga.h | 21 +++++++++++++++++++++ 1 files changed, 21 insertions(+), 0 deletions(-) create mode 100644 arch/arm64/include/asm/vga.h diff --git a/arch/arm64/include/asm/vga.h b/arch/arm64/include/asm/vga.h new file mode 100644 index 0000000..1c49d51 --- /dev/null +++ b/arch/arm64/include/asm/vga.h @@ -0,0 +1,21 @@ +/* + * Copyright (C) 2012 ARM Ltd. + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License version 2 as + * published by the Free Software Foundation. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program. If not, see <http://www.gnu.org/licenses/>. + */ +#ifndef ASMARM_VGA_H +#define ASMARM_VGA_H + +#include <asm-generic/vga.h> + +#endif /* ASMARM_VGA_H */ -- 1.7.7.6 ^ permalink raw reply related [flat|nested] 34+ messages in thread
* [PATCH v2] arm64: compiling issue, need add include/asm/vga.h file @ 2013-04-24 10:20 ` Chen Gang 0 siblings, 0 replies; 34+ messages in thread From: Chen Gang @ 2013-04-24 10:20 UTC (permalink / raw) To: linux-arm-kernel For compiling with allmodconfig, need vga.h file, so generate it which just only include the asm-generic one. It is firstly used by drivers/gpu/drm/drm_irq.c. The related error: include/video/vga.h:22:21: fatal error: asm/vga.h: No such file or directory Signed-off-by: Chen Gang <gang.chen@asianux.com> --- arch/arm64/include/asm/vga.h | 21 +++++++++++++++++++++ 1 files changed, 21 insertions(+), 0 deletions(-) create mode 100644 arch/arm64/include/asm/vga.h diff --git a/arch/arm64/include/asm/vga.h b/arch/arm64/include/asm/vga.h new file mode 100644 index 0000000..1c49d51 --- /dev/null +++ b/arch/arm64/include/asm/vga.h @@ -0,0 +1,21 @@ +/* + * Copyright (C) 2012 ARM Ltd. + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License version 2 as + * published by the Free Software Foundation. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program. If not, see <http://www.gnu.org/licenses/>. + */ +#ifndef ASMARM_VGA_H +#define ASMARM_VGA_H + +#include <asm-generic/vga.h> + +#endif /* ASMARM_VGA_H */ -- 1.7.7.6 ^ permalink raw reply related [flat|nested] 34+ messages in thread
* Re: [PATCH v2] arm64: compiling issue, need add include/asm/vga.h file 2013-04-24 10:20 ` Chen Gang @ 2013-04-24 10:23 ` Will Deacon -1 siblings, 0 replies; 34+ messages in thread From: Will Deacon @ 2013-04-24 10:23 UTC (permalink / raw) To: Chen Gang Cc: Catalin Marinas, Arnd Bergmann, linux-arm-kernel, linaro-kernel, linux-kernel On Wed, Apr 24, 2013 at 11:20:32AM +0100, Chen Gang wrote: > > For compiling with allmodconfig, need vga.h file, so generate it which > just only include the asm-generic one. > > It is firstly used by drivers/gpu/drm/drm_irq.c. > > The related error: > include/video/vga.h:22:21: fatal error: asm/vga.h: No such file or directory > > Signed-off-by: Chen Gang <gang.chen@asianux.com> > --- > arch/arm64/include/asm/vga.h | 21 +++++++++++++++++++++ > 1 files changed, 21 insertions(+), 0 deletions(-) > create mode 100644 arch/arm64/include/asm/vga.h > > diff --git a/arch/arm64/include/asm/vga.h b/arch/arm64/include/asm/vga.h > new file mode 100644 > index 0000000..1c49d51 > --- /dev/null > +++ b/arch/arm64/include/asm/vga.h > @@ -0,0 +1,21 @@ > +/* > + * Copyright (C) 2012 ARM Ltd. > + * > + * This program is free software; you can redistribute it and/or modify > + * it under the terms of the GNU General Public License version 2 as > + * published by the Free Software Foundation. > + * > + * This program is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > + * GNU General Public License for more details. > + * > + * You should have received a copy of the GNU General Public License > + * along with this program. If not, see <http://www.gnu.org/licenses/>. > + */ > +#ifndef ASMARM_VGA_H > +#define ASMARM_VGA_H > + > +#include <asm-generic/vga.h> > + > +#endif /* ASMARM_VGA_H */ Can't you just add this as a generic-y entry in asm/Kbuild? Will ^ permalink raw reply [flat|nested] 34+ messages in thread
* [PATCH v2] arm64: compiling issue, need add include/asm/vga.h file @ 2013-04-24 10:23 ` Will Deacon 0 siblings, 0 replies; 34+ messages in thread From: Will Deacon @ 2013-04-24 10:23 UTC (permalink / raw) To: linux-arm-kernel On Wed, Apr 24, 2013 at 11:20:32AM +0100, Chen Gang wrote: > > For compiling with allmodconfig, need vga.h file, so generate it which > just only include the asm-generic one. > > It is firstly used by drivers/gpu/drm/drm_irq.c. > > The related error: > include/video/vga.h:22:21: fatal error: asm/vga.h: No such file or directory > > Signed-off-by: Chen Gang <gang.chen@asianux.com> > --- > arch/arm64/include/asm/vga.h | 21 +++++++++++++++++++++ > 1 files changed, 21 insertions(+), 0 deletions(-) > create mode 100644 arch/arm64/include/asm/vga.h > > diff --git a/arch/arm64/include/asm/vga.h b/arch/arm64/include/asm/vga.h > new file mode 100644 > index 0000000..1c49d51 > --- /dev/null > +++ b/arch/arm64/include/asm/vga.h > @@ -0,0 +1,21 @@ > +/* > + * Copyright (C) 2012 ARM Ltd. > + * > + * This program is free software; you can redistribute it and/or modify > + * it under the terms of the GNU General Public License version 2 as > + * published by the Free Software Foundation. > + * > + * This program is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > + * GNU General Public License for more details. > + * > + * You should have received a copy of the GNU General Public License > + * along with this program. If not, see <http://www.gnu.org/licenses/>. > + */ > +#ifndef ASMARM_VGA_H > +#define ASMARM_VGA_H > + > +#include <asm-generic/vga.h> > + > +#endif /* ASMARM_VGA_H */ Can't you just add this as a generic-y entry in asm/Kbuild? Will ^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: [PATCH v2] arm64: compiling issue, need add include/asm/vga.h file 2013-04-24 10:23 ` Will Deacon @ 2013-04-24 10:50 ` Chen Gang -1 siblings, 0 replies; 34+ messages in thread From: Chen Gang @ 2013-04-24 10:50 UTC (permalink / raw) To: Will Deacon Cc: Catalin Marinas, Arnd Bergmann, linux-arm-kernel, linaro-kernel, linux-kernel On 2013年04月24日 18:23, Will Deacon wrote: > On Wed, Apr 24, 2013 at 11:20:32AM +0100, Chen Gang wrote: >> > >> > For compiling with allmodconfig, need vga.h file, so generate it which >> > just only include the asm-generic one. >> > >> > It is firstly used by drivers/gpu/drm/drm_irq.c. >> > >> > The related error: >> > include/video/vga.h:22:21: fatal error: asm/vga.h: No such file or directory >> > >> > Signed-off-by: Chen Gang <gang.chen@asianux.com> >> > --- >> > arch/arm64/include/asm/vga.h | 21 +++++++++++++++++++++ >> > 1 files changed, 21 insertions(+), 0 deletions(-) >> > create mode 100644 arch/arm64/include/asm/vga.h >> > >> > diff --git a/arch/arm64/include/asm/vga.h b/arch/arm64/include/asm/vga.h >> > new file mode 100644 >> > index 0000000..1c49d51 >> > --- /dev/null >> > +++ b/arch/arm64/include/asm/vga.h >> > @@ -0,0 +1,21 @@ >> > +/* >> > + * Copyright (C) 2012 ARM Ltd. >> > + * >> > + * This program is free software; you can redistribute it and/or modify >> > + * it under the terms of the GNU General Public License version 2 as >> > + * published by the Free Software Foundation. >> > + * >> > + * This program is distributed in the hope that it will be useful, >> > + * but WITHOUT ANY WARRANTY; without even the implied warranty of >> > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the >> > + * GNU General Public License for more details. >> > + * >> > + * You should have received a copy of the GNU General Public License >> > + * along with this program. If not, see <http://www.gnu.org/licenses/>. >> > + */ >> > +#ifndef ASMARM_VGA_H >> > +#define ASMARM_VGA_H >> > + >> > +#include <asm-generic/vga.h> >> > + >> > +#endif /* ASMARM_VGA_H */ > Can't you just add this as a generic-y entry in asm/Kbuild? OK, thanks, I need send patch v3. -- Chen Gang Asianux Corporation ^ permalink raw reply [flat|nested] 34+ messages in thread
* [PATCH v2] arm64: compiling issue, need add include/asm/vga.h file @ 2013-04-24 10:50 ` Chen Gang 0 siblings, 0 replies; 34+ messages in thread From: Chen Gang @ 2013-04-24 10:50 UTC (permalink / raw) To: linux-arm-kernel On 2013?04?24? 18:23, Will Deacon wrote: > On Wed, Apr 24, 2013 at 11:20:32AM +0100, Chen Gang wrote: >> > >> > For compiling with allmodconfig, need vga.h file, so generate it which >> > just only include the asm-generic one. >> > >> > It is firstly used by drivers/gpu/drm/drm_irq.c. >> > >> > The related error: >> > include/video/vga.h:22:21: fatal error: asm/vga.h: No such file or directory >> > >> > Signed-off-by: Chen Gang <gang.chen@asianux.com> >> > --- >> > arch/arm64/include/asm/vga.h | 21 +++++++++++++++++++++ >> > 1 files changed, 21 insertions(+), 0 deletions(-) >> > create mode 100644 arch/arm64/include/asm/vga.h >> > >> > diff --git a/arch/arm64/include/asm/vga.h b/arch/arm64/include/asm/vga.h >> > new file mode 100644 >> > index 0000000..1c49d51 >> > --- /dev/null >> > +++ b/arch/arm64/include/asm/vga.h >> > @@ -0,0 +1,21 @@ >> > +/* >> > + * Copyright (C) 2012 ARM Ltd. >> > + * >> > + * This program is free software; you can redistribute it and/or modify >> > + * it under the terms of the GNU General Public License version 2 as >> > + * published by the Free Software Foundation. >> > + * >> > + * This program is distributed in the hope that it will be useful, >> > + * but WITHOUT ANY WARRANTY; without even the implied warranty of >> > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the >> > + * GNU General Public License for more details. >> > + * >> > + * You should have received a copy of the GNU General Public License >> > + * along with this program. If not, see <http://www.gnu.org/licenses/>. >> > + */ >> > +#ifndef ASMARM_VGA_H >> > +#define ASMARM_VGA_H >> > + >> > +#include <asm-generic/vga.h> >> > + >> > +#endif /* ASMARM_VGA_H */ > Can't you just add this as a generic-y entry in asm/Kbuild? OK, thanks, I need send patch v3. -- Chen Gang Asianux Corporation ^ permalink raw reply [flat|nested] 34+ messages in thread
* [PATCH v3] arm64: compiling issue, need add include/asm/vga.h file 2013-04-24 10:23 ` Will Deacon @ 2013-04-24 10:58 ` Chen Gang -1 siblings, 0 replies; 34+ messages in thread From: Chen Gang @ 2013-04-24 10:58 UTC (permalink / raw) To: Will Deacon Cc: Catalin Marinas, Arnd Bergmann, linux-arm-kernel, linaro-kernel, linux-kernel For compiling with allmodconfig, need vga.h file, so generate it which just only include the asm-generic one. It is firstly used by drivers/gpu/drm/drm_irq.c. The related error: include/video/vga.h:22:21: fatal error: asm/vga.h: No such file or directory Signed-off-by: Chen Gang <gang.chen@asianux.com> --- arch/arm64/include/asm/Kbuild | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/arch/arm64/include/asm/Kbuild b/arch/arm64/include/asm/Kbuild index a13a4a4..79a642d 100644 --- a/arch/arm64/include/asm/Kbuild +++ b/arch/arm64/include/asm/Kbuild @@ -48,4 +48,5 @@ generic-y += trace_clock.h generic-y += types.h generic-y += unaligned.h generic-y += user.h +generic-y += vga.h generic-y += xor.h -- 1.7.7.6 ^ permalink raw reply related [flat|nested] 34+ messages in thread
* [PATCH v3] arm64: compiling issue, need add include/asm/vga.h file @ 2013-04-24 10:58 ` Chen Gang 0 siblings, 0 replies; 34+ messages in thread From: Chen Gang @ 2013-04-24 10:58 UTC (permalink / raw) To: linux-arm-kernel For compiling with allmodconfig, need vga.h file, so generate it which just only include the asm-generic one. It is firstly used by drivers/gpu/drm/drm_irq.c. The related error: include/video/vga.h:22:21: fatal error: asm/vga.h: No such file or directory Signed-off-by: Chen Gang <gang.chen@asianux.com> --- arch/arm64/include/asm/Kbuild | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/arch/arm64/include/asm/Kbuild b/arch/arm64/include/asm/Kbuild index a13a4a4..79a642d 100644 --- a/arch/arm64/include/asm/Kbuild +++ b/arch/arm64/include/asm/Kbuild @@ -48,4 +48,5 @@ generic-y += trace_clock.h generic-y += types.h generic-y += unaligned.h generic-y += user.h +generic-y += vga.h generic-y += xor.h -- 1.7.7.6 ^ permalink raw reply related [flat|nested] 34+ messages in thread
* Re: [PATCH v3] arm64: compiling issue, need add include/asm/vga.h file 2013-04-24 10:58 ` Chen Gang @ 2013-04-24 11:05 ` Catalin Marinas -1 siblings, 0 replies; 34+ messages in thread From: Catalin Marinas @ 2013-04-24 11:05 UTC (permalink / raw) To: Chen Gang Cc: Will Deacon, Arnd Bergmann, linux-arm-kernel, linaro-kernel, linux-kernel On Wed, Apr 24, 2013 at 11:58:24AM +0100, Chen Gang wrote: > > For compiling with allmodconfig, need vga.h file, so generate it which > just only include the asm-generic one. > > It is firstly used by drivers/gpu/drm/drm_irq.c. > > The related error: > include/video/vga.h:22:21: fatal error: asm/vga.h: No such file or directory > > > Signed-off-by: Chen Gang <gang.chen@asianux.com> Looks fine. Thanks. -- Catalin ^ permalink raw reply [flat|nested] 34+ messages in thread
* [PATCH v3] arm64: compiling issue, need add include/asm/vga.h file @ 2013-04-24 11:05 ` Catalin Marinas 0 siblings, 0 replies; 34+ messages in thread From: Catalin Marinas @ 2013-04-24 11:05 UTC (permalink / raw) To: linux-arm-kernel On Wed, Apr 24, 2013 at 11:58:24AM +0100, Chen Gang wrote: > > For compiling with allmodconfig, need vga.h file, so generate it which > just only include the asm-generic one. > > It is firstly used by drivers/gpu/drm/drm_irq.c. > > The related error: > include/video/vga.h:22:21: fatal error: asm/vga.h: No such file or directory > > > Signed-off-by: Chen Gang <gang.chen@asianux.com> Looks fine. Thanks. -- Catalin ^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: [PATCH v3] arm64: compiling issue, need add include/asm/vga.h file 2013-04-24 11:05 ` Catalin Marinas @ 2013-04-24 11:06 ` Chen Gang -1 siblings, 0 replies; 34+ messages in thread From: Chen Gang @ 2013-04-24 11:06 UTC (permalink / raw) To: Catalin Marinas Cc: Will Deacon, Arnd Bergmann, linux-arm-kernel, linaro-kernel, linux-kernel On 2013年04月24日 19:05, Catalin Marinas wrote: > On Wed, Apr 24, 2013 at 11:58:24AM +0100, Chen Gang wrote: >> > >> > For compiling with allmodconfig, need vga.h file, so generate it which >> > just only include the asm-generic one. >> > >> > It is firstly used by drivers/gpu/drm/drm_irq.c. >> > >> > The related error: >> > include/video/vga.h:22:21: fatal error: asm/vga.h: No such file or directory >> > >> > >> > Signed-off-by: Chen Gang <gang.chen@asianux.com> > Looks fine. Thanks. Thanks, too. -- Chen Gang Asianux Corporation ^ permalink raw reply [flat|nested] 34+ messages in thread
* [PATCH v3] arm64: compiling issue, need add include/asm/vga.h file @ 2013-04-24 11:06 ` Chen Gang 0 siblings, 0 replies; 34+ messages in thread From: Chen Gang @ 2013-04-24 11:06 UTC (permalink / raw) To: linux-arm-kernel On 2013?04?24? 19:05, Catalin Marinas wrote: > On Wed, Apr 24, 2013 at 11:58:24AM +0100, Chen Gang wrote: >> > >> > For compiling with allmodconfig, need vga.h file, so generate it which >> > just only include the asm-generic one. >> > >> > It is firstly used by drivers/gpu/drm/drm_irq.c. >> > >> > The related error: >> > include/video/vga.h:22:21: fatal error: asm/vga.h: No such file or directory >> > >> > >> > Signed-off-by: Chen Gang <gang.chen@asianux.com> > Looks fine. Thanks. Thanks, too. -- Chen Gang Asianux Corporation ^ permalink raw reply [flat|nested] 34+ messages in thread
* [PATCH] arm64: compiling issue for allmodconfig, need "asm/parport.h" by "drivers/parport/parport_pc.c" 2013-04-24 11:05 ` Catalin Marinas @ 2013-05-20 4:19 ` Chen Gang -1 siblings, 0 replies; 34+ messages in thread From: Chen Gang @ 2013-05-20 4:19 UTC (permalink / raw) To: Catalin Marinas Cc: Will Deacon, linux-arm-kernel, linaro-kernel, linux-kernel Need add the default file in 'asm/Kbuild' just like arm has done. The build command is: make EXTRA_CFLAGS=-W ARCH=arm64 allmodconfig make EXTRA_CFLAGS=-W ARCH=arm64 menuconfig set "aarch64-linux-gnu-" as cross compiling prefix. make EXTRA_CFLAGS=-W ARCH=arm64 The related error: drivers/parport/parport_pc.c:67:25: fatal error: asm/parport.h: No such file or directory Signed-off-by: Chen Gang <gang.chen@asianux.com> --- arch/arm64/include/asm/Kbuild | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/arch/arm64/include/asm/Kbuild b/arch/arm64/include/asm/Kbuild index 79a642d..9005e81 100644 --- a/arch/arm64/include/asm/Kbuild +++ b/arch/arm64/include/asm/Kbuild @@ -25,6 +25,7 @@ generic-y += local64.h generic-y += mman.h generic-y += msgbuf.h generic-y += mutex.h +generic-y += parport.h generic-y += pci.h generic-y += percpu.h generic-y += poll.h -- 1.7.7.6 ^ permalink raw reply related [flat|nested] 34+ messages in thread
* [PATCH] arm64: compiling issue for allmodconfig, need "asm/parport.h" by "drivers/parport/parport_pc.c" @ 2013-05-20 4:19 ` Chen Gang 0 siblings, 0 replies; 34+ messages in thread From: Chen Gang @ 2013-05-20 4:19 UTC (permalink / raw) To: linux-arm-kernel Need add the default file in 'asm/Kbuild' just like arm has done. The build command is: make EXTRA_CFLAGS=-W ARCH=arm64 allmodconfig make EXTRA_CFLAGS=-W ARCH=arm64 menuconfig set "aarch64-linux-gnu-" as cross compiling prefix. make EXTRA_CFLAGS=-W ARCH=arm64 The related error: drivers/parport/parport_pc.c:67:25: fatal error: asm/parport.h: No such file or directory Signed-off-by: Chen Gang <gang.chen@asianux.com> --- arch/arm64/include/asm/Kbuild | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/arch/arm64/include/asm/Kbuild b/arch/arm64/include/asm/Kbuild index 79a642d..9005e81 100644 --- a/arch/arm64/include/asm/Kbuild +++ b/arch/arm64/include/asm/Kbuild @@ -25,6 +25,7 @@ generic-y += local64.h generic-y += mman.h generic-y += msgbuf.h generic-y += mutex.h +generic-y += parport.h generic-y += pci.h generic-y += percpu.h generic-y += poll.h -- 1.7.7.6 ^ permalink raw reply related [flat|nested] 34+ messages in thread
* Re: [PATCH] arm64: compiling issue for allmodconfig, need "asm/parport.h" by "drivers/parport/parport_pc.c" 2013-05-20 4:19 ` Chen Gang @ 2013-05-21 13:17 ` Catalin Marinas -1 siblings, 0 replies; 34+ messages in thread From: Catalin Marinas @ 2013-05-21 13:17 UTC (permalink / raw) To: Chen Gang; +Cc: Will Deacon, linux-arm-kernel, linaro-kernel, linux-kernel On Mon, May 20, 2013 at 05:19:31AM +0100, Chen Gang wrote: > > Need add the default file in 'asm/Kbuild' just like arm has done. > > The build command is: > make EXTRA_CFLAGS=-W ARCH=arm64 allmodconfig > make EXTRA_CFLAGS=-W ARCH=arm64 menuconfig > set "aarch64-linux-gnu-" as cross compiling prefix. > make EXTRA_CFLAGS=-W ARCH=arm64 > > The related error: > drivers/parport/parport_pc.c:67:25: fatal error: asm/parport.h: No such file or directory As I don't see when we'd need this driver, you can probably change the Kconfig entry (similar to the VGA_CONSOLE one). Thanks. -- Catalin ^ permalink raw reply [flat|nested] 34+ messages in thread
* [PATCH] arm64: compiling issue for allmodconfig, need "asm/parport.h" by "drivers/parport/parport_pc.c" @ 2013-05-21 13:17 ` Catalin Marinas 0 siblings, 0 replies; 34+ messages in thread From: Catalin Marinas @ 2013-05-21 13:17 UTC (permalink / raw) To: linux-arm-kernel On Mon, May 20, 2013 at 05:19:31AM +0100, Chen Gang wrote: > > Need add the default file in 'asm/Kbuild' just like arm has done. > > The build command is: > make EXTRA_CFLAGS=-W ARCH=arm64 allmodconfig > make EXTRA_CFLAGS=-W ARCH=arm64 menuconfig > set "aarch64-linux-gnu-" as cross compiling prefix. > make EXTRA_CFLAGS=-W ARCH=arm64 > > The related error: > drivers/parport/parport_pc.c:67:25: fatal error: asm/parport.h: No such file or directory As I don't see when we'd need this driver, you can probably change the Kconfig entry (similar to the VGA_CONSOLE one). Thanks. -- Catalin ^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: [PATCH] arm64: compiling issue for allmodconfig, need "asm/parport.h" by "drivers/parport/parport_pc.c" 2013-05-21 13:17 ` Catalin Marinas @ 2013-05-22 0:45 ` Chen Gang -1 siblings, 0 replies; 34+ messages in thread From: Chen Gang @ 2013-05-22 0:45 UTC (permalink / raw) To: Catalin Marinas Cc: Will Deacon, linux-arm-kernel, linaro-kernel, linux-kernel On 05/21/2013 09:17 PM, Catalin Marinas wrote: > On Mon, May 20, 2013 at 05:19:31AM +0100, Chen Gang wrote: >> >> Need add the default file in 'asm/Kbuild' just like arm has done. >> >> The build command is: >> make EXTRA_CFLAGS=-W ARCH=arm64 allmodconfig >> make EXTRA_CFLAGS=-W ARCH=arm64 menuconfig >> set "aarch64-linux-gnu-" as cross compiling prefix. >> make EXTRA_CFLAGS=-W ARCH=arm64 >> >> The related error: >> drivers/parport/parport_pc.c:67:25: fatal error: asm/parport.h: No such file or directory > > As I don't see when we'd need this driver, you can probably change the > Kconfig entry (similar to the VGA_CONSOLE one). > Oh, really it is, it is for PC, it seems except x86, no one else will use it. And for x86, it will be less and less used, in the future. It seems it is a better candidate than VGA_CONSOLE. I will define CONFIG_HAVE_PARPORT_PC for it. :-) Thanks. -- Chen Gang Asianux Corporation ^ permalink raw reply [flat|nested] 34+ messages in thread
* [PATCH] arm64: compiling issue for allmodconfig, need "asm/parport.h" by "drivers/parport/parport_pc.c" @ 2013-05-22 0:45 ` Chen Gang 0 siblings, 0 replies; 34+ messages in thread From: Chen Gang @ 2013-05-22 0:45 UTC (permalink / raw) To: linux-arm-kernel On 05/21/2013 09:17 PM, Catalin Marinas wrote: > On Mon, May 20, 2013 at 05:19:31AM +0100, Chen Gang wrote: >> >> Need add the default file in 'asm/Kbuild' just like arm has done. >> >> The build command is: >> make EXTRA_CFLAGS=-W ARCH=arm64 allmodconfig >> make EXTRA_CFLAGS=-W ARCH=arm64 menuconfig >> set "aarch64-linux-gnu-" as cross compiling prefix. >> make EXTRA_CFLAGS=-W ARCH=arm64 >> >> The related error: >> drivers/parport/parport_pc.c:67:25: fatal error: asm/parport.h: No such file or directory > > As I don't see when we'd need this driver, you can probably change the > Kconfig entry (similar to the VGA_CONSOLE one). > Oh, really it is, it is for PC, it seems except x86, no one else will use it. And for x86, it will be less and less used, in the future. It seems it is a better candidate than VGA_CONSOLE. I will define CONFIG_HAVE_PARPORT_PC for it. :-) Thanks. -- Chen Gang Asianux Corporation ^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: [PATCH] arm64: compiling issue for allmodconfig, need "asm/parport.h" by "drivers/parport/parport_pc.c" 2013-05-22 0:45 ` Chen Gang @ 2013-05-22 7:32 ` Chen Gang -1 siblings, 0 replies; 34+ messages in thread From: Chen Gang @ 2013-05-22 7:32 UTC (permalink / raw) To: Catalin Marinas Cc: Will Deacon, linux-arm-kernel, linaro-kernel, linux-kernel, Geert Uytterhoeven On 05/22/2013 08:45 AM, Chen Gang wrote: > On 05/21/2013 09:17 PM, Catalin Marinas wrote: >> On Mon, May 20, 2013 at 05:19:31AM +0100, Chen Gang wrote: >>> >>> Need add the default file in 'asm/Kbuild' just like arm has done. >>> >>> The build command is: >>> make EXTRA_CFLAGS=-W ARCH=arm64 allmodconfig >>> make EXTRA_CFLAGS=-W ARCH=arm64 menuconfig >>> set "aarch64-linux-gnu-" as cross compiling prefix. >>> make EXTRA_CFLAGS=-W ARCH=arm64 >>> >>> The related error: >>> drivers/parport/parport_pc.c:67:25: fatal error: asm/parport.h: No such file or directory >> >> As I don't see when we'd need this driver, you can probably change the >> Kconfig entry (similar to the VGA_CONSOLE one). >> After check the Kconfig in drivers, we can find quite a few using long list args which related with ARCH. And they are not only depend on ARCHs, but also depend on another things (e.g. VGA_CONSOL also depend on !4xx && !8xx, and another combinations). We also can find another long list for depend on, but not related with ARCH. They seems not quit difficult for reading (in fact, if we separate them into pieces and let them spread into another areas, the reader will not see the whole dependency clearly) For fixing bug, better as local as possible, as independent as possible, it will be OK if only add !ARM64 for PARPORT_PC depend on, and it has no any negative effect. But if we define CONFIG_HAVE_*, we have to let it related with multiple platforms. For design: if can bear current design, better not to touch it, I think current design is still sustainable, and not complex for understanding, so not need redesign. The depends on samples which related with ARCH: ./ide/Kconfig: depends on ALPHA || X86 || IA64 || M32R || MIPS || ARCH_RPC || ARCH_SHARK ./dma/Kconfig: depends on ARCH_IOP32X || ARCH_IOP33X || ARCH_IOP13XX ./misc/Kconfig: depends on (IA64_GENERIC || IA64_SGI_SN2 || IA64_SGI_UV || X86_UV) && SMP ./gpio/Kconfig: depends on PPC_MPC512x || PPC_MPC831x || PPC_MPC834x || PPC_MPC837x || ... ./gpio/Kconfig: depends on GPIOLIB && ARCH_MSM && (ARCH_MSM7X00A || ARCH_MSM7X30 || ARCH_QSD8X50) ./mtd/onenand/Kconfig: depends on ARCH_S3C64XX || ARCH_S5PC100 || ARCH_S5PV210 || ARCH_EXYNOS4 ./net/ethernet/ti/Kconfig: depends on PCI || EISA || AR7 || (ARM && (ARCH_DAVINCI || ARCH_OMAP3 || SOC_AM33XX)) ./net/ethernet/ibm/Kconfig: depends on MCA || PPC_PSERIES || PPC_PSERIES || PPC_DCR || ... ./net/ethernet/smsc/Kconfig: depends on (ARM || M32R || SUPERH || MIPS || BLACKFIN || ... ./net/ethernet/smsc/Kconfig: depends on (ARM || SUPERH || BLACKFIN || MIPS || MN10300) ./rtc/Kconfig: depends on X86 || ALPHA || ARM || M32R || ATARI || PPC || MIPS || SPARC64 ./rtc/Kconfig: depends on ARCH_OMAP15XX || ARCH_OMAP16XX || ARCH_OMAP730 || ARCH_DAVINCI_DA8XX || SOC_AM33XX ./isdn/hisax/Kconfig: depends on PCI && (BROKEN || !(SPARC || PPC || PARISC || M68K || (MIPS && !CPU_LITTLE_ENDIAN) || FRV || (XTENSA && !CPU_LITTLE_ENDIAN))) ./isdn/hisax/Kconfig: depends on PCI && (BROKEN || !(PPC || PARISC || M68K || (MIPS && !CPU_LITTLE_ENDIAN) || FRV || (XTENSA && !CPU_LITTLE_ENDIAN))) ./isdn/hisax/Kconfig: depends on PCI && (BROKEN || !(SPARC || PPC || PARISC || M68K || (MIPS && !CPU_LITTLE_ENDIAN) || FRV || (XTENSA && !CPU_LITTLE_ENDIAN))) ./isdn/hisax/Kconfig: depends on HISAX_NETJET && PCI && (BROKEN || !(SPARC || PPC || PARISC || M68K || (MIPS && !CPU_LITTLE_ENDIAN) || FRV || (XTENSA && !CPU_LITTLE_ENDIAN))) ./input/serio/Kconfig: depends on !PARISC && (!ARM || ARCH_SHARK || FOOTBRIDGE_HOST) && ... ./char/Kconfig: depends on RTC!=y && !IA64 && !ARM && !M32R && !MIPS && !SPARC && !FRV && !S390 && !SUPERH && !AVR32 && !BLACKFIN && !UML ./video/console/Kconfig: depends on !4xx && !8xx && !SPARC && !M68K && !PARISC && !FRV && !SUPERH && !BLACKFIN && !AVR32 && !MN10300 && (!ARM || ARCH_FOOTBRIDGE || ARCH_INTEGRATOR || ARCH_NETWINDER) ./video/Kconfig: depends on FB && (CPU_S3C2416 || ARCH_S3C64XX || ARCH_S5P64X0 || ... ... > > Oh, really it is, it is for PC, it seems except x86, no one else will > use it. And for x86, it will be less and less used, in the future. > > It seems it is a better candidate than VGA_CONSOLE. I will define > CONFIG_HAVE_PARPORT_PC for it. :-) > > Thanks. > -- Chen Gang Asianux Corporation ^ permalink raw reply [flat|nested] 34+ messages in thread
* [PATCH] arm64: compiling issue for allmodconfig, need "asm/parport.h" by "drivers/parport/parport_pc.c" @ 2013-05-22 7:32 ` Chen Gang 0 siblings, 0 replies; 34+ messages in thread From: Chen Gang @ 2013-05-22 7:32 UTC (permalink / raw) To: linux-arm-kernel On 05/22/2013 08:45 AM, Chen Gang wrote: > On 05/21/2013 09:17 PM, Catalin Marinas wrote: >> On Mon, May 20, 2013 at 05:19:31AM +0100, Chen Gang wrote: >>> >>> Need add the default file in 'asm/Kbuild' just like arm has done. >>> >>> The build command is: >>> make EXTRA_CFLAGS=-W ARCH=arm64 allmodconfig >>> make EXTRA_CFLAGS=-W ARCH=arm64 menuconfig >>> set "aarch64-linux-gnu-" as cross compiling prefix. >>> make EXTRA_CFLAGS=-W ARCH=arm64 >>> >>> The related error: >>> drivers/parport/parport_pc.c:67:25: fatal error: asm/parport.h: No such file or directory >> >> As I don't see when we'd need this driver, you can probably change the >> Kconfig entry (similar to the VGA_CONSOLE one). >> After check the Kconfig in drivers, we can find quite a few using long list args which related with ARCH. And they are not only depend on ARCHs, but also depend on another things (e.g. VGA_CONSOL also depend on !4xx && !8xx, and another combinations). We also can find another long list for depend on, but not related with ARCH. They seems not quit difficult for reading (in fact, if we separate them into pieces and let them spread into another areas, the reader will not see the whole dependency clearly) For fixing bug, better as local as possible, as independent as possible, it will be OK if only add !ARM64 for PARPORT_PC depend on, and it has no any negative effect. But if we define CONFIG_HAVE_*, we have to let it related with multiple platforms. For design: if can bear current design, better not to touch it, I think current design is still sustainable, and not complex for understanding, so not need redesign. The depends on samples which related with ARCH: ./ide/Kconfig: depends on ALPHA || X86 || IA64 || M32R || MIPS || ARCH_RPC || ARCH_SHARK ./dma/Kconfig: depends on ARCH_IOP32X || ARCH_IOP33X || ARCH_IOP13XX ./misc/Kconfig: depends on (IA64_GENERIC || IA64_SGI_SN2 || IA64_SGI_UV || X86_UV) && SMP ./gpio/Kconfig: depends on PPC_MPC512x || PPC_MPC831x || PPC_MPC834x || PPC_MPC837x || ... ./gpio/Kconfig: depends on GPIOLIB && ARCH_MSM && (ARCH_MSM7X00A || ARCH_MSM7X30 || ARCH_QSD8X50) ./mtd/onenand/Kconfig: depends on ARCH_S3C64XX || ARCH_S5PC100 || ARCH_S5PV210 || ARCH_EXYNOS4 ./net/ethernet/ti/Kconfig: depends on PCI || EISA || AR7 || (ARM && (ARCH_DAVINCI || ARCH_OMAP3 || SOC_AM33XX)) ./net/ethernet/ibm/Kconfig: depends on MCA || PPC_PSERIES || PPC_PSERIES || PPC_DCR || ... ./net/ethernet/smsc/Kconfig: depends on (ARM || M32R || SUPERH || MIPS || BLACKFIN || ... ./net/ethernet/smsc/Kconfig: depends on (ARM || SUPERH || BLACKFIN || MIPS || MN10300) ./rtc/Kconfig: depends on X86 || ALPHA || ARM || M32R || ATARI || PPC || MIPS || SPARC64 ./rtc/Kconfig: depends on ARCH_OMAP15XX || ARCH_OMAP16XX || ARCH_OMAP730 || ARCH_DAVINCI_DA8XX || SOC_AM33XX ./isdn/hisax/Kconfig: depends on PCI && (BROKEN || !(SPARC || PPC || PARISC || M68K || (MIPS && !CPU_LITTLE_ENDIAN) || FRV || (XTENSA && !CPU_LITTLE_ENDIAN))) ./isdn/hisax/Kconfig: depends on PCI && (BROKEN || !(PPC || PARISC || M68K || (MIPS && !CPU_LITTLE_ENDIAN) || FRV || (XTENSA && !CPU_LITTLE_ENDIAN))) ./isdn/hisax/Kconfig: depends on PCI && (BROKEN || !(SPARC || PPC || PARISC || M68K || (MIPS && !CPU_LITTLE_ENDIAN) || FRV || (XTENSA && !CPU_LITTLE_ENDIAN))) ./isdn/hisax/Kconfig: depends on HISAX_NETJET && PCI && (BROKEN || !(SPARC || PPC || PARISC || M68K || (MIPS && !CPU_LITTLE_ENDIAN) || FRV || (XTENSA && !CPU_LITTLE_ENDIAN))) ./input/serio/Kconfig: depends on !PARISC && (!ARM || ARCH_SHARK || FOOTBRIDGE_HOST) && ... ./char/Kconfig: depends on RTC!=y && !IA64 && !ARM && !M32R && !MIPS && !SPARC && !FRV && !S390 && !SUPERH && !AVR32 && !BLACKFIN && !UML ./video/console/Kconfig: depends on !4xx && !8xx && !SPARC && !M68K && !PARISC && !FRV && !SUPERH && !BLACKFIN && !AVR32 && !MN10300 && (!ARM || ARCH_FOOTBRIDGE || ARCH_INTEGRATOR || ARCH_NETWINDER) ./video/Kconfig: depends on FB && (CPU_S3C2416 || ARCH_S3C64XX || ARCH_S5P64X0 || ... ... > > Oh, really it is, it is for PC, it seems except x86, no one else will > use it. And for x86, it will be less and less used, in the future. > > It seems it is a better candidate than VGA_CONSOLE. I will define > CONFIG_HAVE_PARPORT_PC for it. :-) > > Thanks. > -- Chen Gang Asianux Corporation ^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: [PATCH] arm64: compiling issue for allmodconfig, need "asm/parport.h" by "drivers/parport/parport_pc.c" 2013-05-22 7:32 ` Chen Gang @ 2013-05-28 17:02 ` Catalin Marinas -1 siblings, 0 replies; 34+ messages in thread From: Catalin Marinas @ 2013-05-28 17:02 UTC (permalink / raw) To: Chen Gang Cc: Will Deacon, linux-arm-kernel, linaro-kernel, linux-kernel, Geert Uytterhoeven On Wed, May 22, 2013 at 08:32:20AM +0100, Chen Gang wrote: > On 05/22/2013 08:45 AM, Chen Gang wrote: > > On 05/21/2013 09:17 PM, Catalin Marinas wrote: > >> On Mon, May 20, 2013 at 05:19:31AM +0100, Chen Gang wrote: > >>> > >>> Need add the default file in 'asm/Kbuild' just like arm has done. > >>> > >>> The build command is: > >>> make EXTRA_CFLAGS=-W ARCH=arm64 allmodconfig > >>> make EXTRA_CFLAGS=-W ARCH=arm64 menuconfig > >>> set "aarch64-linux-gnu-" as cross compiling prefix. > >>> make EXTRA_CFLAGS=-W ARCH=arm64 > >>> > >>> The related error: > >>> drivers/parport/parport_pc.c:67:25: fatal error: asm/parport.h: No such file or directory > >> > >> As I don't see when we'd need this driver, you can probably change the > >> Kconfig entry (similar to the VGA_CONSOLE one). > >> > > After check the Kconfig in drivers, we can find quite a few using long > list args which related with ARCH. > > And they are not only depend on ARCHs, but also depend on another > things (e.g. VGA_CONSOL also depend on !4xx && !8xx, and another > combinations). Given this dependency is on the negation of that symbol, you just remove it and have a "depends on HAVE_VGA_CONSOLE". The few architectures that need it will have to define it (X86, ARM with a few configurations). > We also can find another long list for depend on, but not related with > ARCH. > > They seems not quit difficult for reading (in fact, if we separate them > into pieces and let them spread into another areas, the reader will not > see the whole dependency clearly) > > For fixing bug, better as local as possible, as independent as possible, > it will be OK if only add !ARM64 for PARPORT_PC depend on, and it has > no any negative effect. But if we define CONFIG_HAVE_*, we have to let > it related with multiple platforms. > > For design: if can bear current design, better not to touch it, I think > current design is still sustainable, and not complex for understanding, > so not need redesign. We usually clean some of these as we hit them, otherwise people forget about the clean-up later ;). -- Catalin ^ permalink raw reply [flat|nested] 34+ messages in thread
* [PATCH] arm64: compiling issue for allmodconfig, need "asm/parport.h" by "drivers/parport/parport_pc.c" @ 2013-05-28 17:02 ` Catalin Marinas 0 siblings, 0 replies; 34+ messages in thread From: Catalin Marinas @ 2013-05-28 17:02 UTC (permalink / raw) To: linux-arm-kernel On Wed, May 22, 2013 at 08:32:20AM +0100, Chen Gang wrote: > On 05/22/2013 08:45 AM, Chen Gang wrote: > > On 05/21/2013 09:17 PM, Catalin Marinas wrote: > >> On Mon, May 20, 2013 at 05:19:31AM +0100, Chen Gang wrote: > >>> > >>> Need add the default file in 'asm/Kbuild' just like arm has done. > >>> > >>> The build command is: > >>> make EXTRA_CFLAGS=-W ARCH=arm64 allmodconfig > >>> make EXTRA_CFLAGS=-W ARCH=arm64 menuconfig > >>> set "aarch64-linux-gnu-" as cross compiling prefix. > >>> make EXTRA_CFLAGS=-W ARCH=arm64 > >>> > >>> The related error: > >>> drivers/parport/parport_pc.c:67:25: fatal error: asm/parport.h: No such file or directory > >> > >> As I don't see when we'd need this driver, you can probably change the > >> Kconfig entry (similar to the VGA_CONSOLE one). > >> > > After check the Kconfig in drivers, we can find quite a few using long > list args which related with ARCH. > > And they are not only depend on ARCHs, but also depend on another > things (e.g. VGA_CONSOL also depend on !4xx && !8xx, and another > combinations). Given this dependency is on the negation of that symbol, you just remove it and have a "depends on HAVE_VGA_CONSOLE". The few architectures that need it will have to define it (X86, ARM with a few configurations). > We also can find another long list for depend on, but not related with > ARCH. > > They seems not quit difficult for reading (in fact, if we separate them > into pieces and let them spread into another areas, the reader will not > see the whole dependency clearly) > > For fixing bug, better as local as possible, as independent as possible, > it will be OK if only add !ARM64 for PARPORT_PC depend on, and it has > no any negative effect. But if we define CONFIG_HAVE_*, we have to let > it related with multiple platforms. > > For design: if can bear current design, better not to touch it, I think > current design is still sustainable, and not complex for understanding, > so not need redesign. We usually clean some of these as we hit them, otherwise people forget about the clean-up later ;). -- Catalin ^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: [PATCH] arm64: compiling issue for allmodconfig, need "asm/parport.h" by "drivers/parport/parport_pc.c" 2013-05-28 17:02 ` Catalin Marinas @ 2013-05-29 2:06 ` Chen Gang -1 siblings, 0 replies; 34+ messages in thread From: Chen Gang @ 2013-05-29 2:06 UTC (permalink / raw) To: Catalin Marinas Cc: Will Deacon, linux-arm-kernel, linaro-kernel, linux-kernel, Geert Uytterhoeven On 05/29/2013 01:02 AM, Catalin Marinas wrote: >> > >> > After check the Kconfig in drivers, we can find quite a few using long >> > list args which related with ARCH. >> > >> > And they are not only depend on ARCHs, but also depend on another >> > things (e.g. VGA_CONSOL also depend on !4xx && !8xx, and another >> > combinations). > Given this dependency is on the negation of that symbol, you just remove > it and have a "depends on HAVE_VGA_CONSOLE". The few architectures that > need it will have to define it (X86, ARM with a few configurations). > it is "depends on !4xx && !8xx && !SPARC && !M68K && !PARISC && !FRV && !SUPERH && !BLACKFIN && !AVR32 && !MN10300 && (!ARM || ARCH_FOOTBRIDGE || ARCH_INTEGRATOR || ARCH_NETWINDER)" So, not only x86, ARM, but also s390, xtensa, alpha, cris, h8300, ia64, mips... (at least, we do not know, they whether need it or not). >> > We also can find another long list for depend on, but not related with >> > ARCH. >> > >> > They seems not quit difficult for reading (in fact, if we separate them >> > into pieces and let them spread into another areas, the reader will not >> > see the whole dependency clearly) >> > >> > For fixing bug, better as local as possible, as independent as possible, >> > it will be OK if only add !ARM64 for PARPORT_PC depend on, and it has >> > no any negative effect. But if we define CONFIG_HAVE_*, we have to let >> > it related with multiple platforms. >> > >> > For design: if can bear current design, better not to touch it, I think >> > current design is still sustainable, and not complex for understanding, >> > so not need redesign. > We usually clean some of these as we hit them, otherwise people forget > about the clean-up later ;). In fact, I have sent another configuration modification about disabling CONFIG_BUG. it is in discussing... We are sure most of architectures do not need CONFIG_BUG, but for VGA_CONSOLE, we can not be sure most of architectures do not need it. For CONFIG_BUG issue, it will make noisy for most of architectures, but for VGA_CONSOLE issue, at least now, it is only effect with arm64. For CONFIG_BUG issue, it seems we can not provide a satisfied fixing if not get rid of it, but for VGA_CONSOLE issue, at least we have a complete fixing way for it. And now for CONFIG_BUG issue, it is still continuing discussing, and it seems that can not get a result quickly. So for VGA_CONSOLE... ;-) Thanks. -- Chen Gang Asianux Corporation ^ permalink raw reply [flat|nested] 34+ messages in thread
* [PATCH] arm64: compiling issue for allmodconfig, need "asm/parport.h" by "drivers/parport/parport_pc.c" @ 2013-05-29 2:06 ` Chen Gang 0 siblings, 0 replies; 34+ messages in thread From: Chen Gang @ 2013-05-29 2:06 UTC (permalink / raw) To: linux-arm-kernel On 05/29/2013 01:02 AM, Catalin Marinas wrote: >> > >> > After check the Kconfig in drivers, we can find quite a few using long >> > list args which related with ARCH. >> > >> > And they are not only depend on ARCHs, but also depend on another >> > things (e.g. VGA_CONSOL also depend on !4xx && !8xx, and another >> > combinations). > Given this dependency is on the negation of that symbol, you just remove > it and have a "depends on HAVE_VGA_CONSOLE". The few architectures that > need it will have to define it (X86, ARM with a few configurations). > it is "depends on !4xx && !8xx && !SPARC && !M68K && !PARISC && !FRV && !SUPERH && !BLACKFIN && !AVR32 && !MN10300 && (!ARM || ARCH_FOOTBRIDGE || ARCH_INTEGRATOR || ARCH_NETWINDER)" So, not only x86, ARM, but also s390, xtensa, alpha, cris, h8300, ia64, mips... (at least, we do not know, they whether need it or not). >> > We also can find another long list for depend on, but not related with >> > ARCH. >> > >> > They seems not quit difficult for reading (in fact, if we separate them >> > into pieces and let them spread into another areas, the reader will not >> > see the whole dependency clearly) >> > >> > For fixing bug, better as local as possible, as independent as possible, >> > it will be OK if only add !ARM64 for PARPORT_PC depend on, and it has >> > no any negative effect. But if we define CONFIG_HAVE_*, we have to let >> > it related with multiple platforms. >> > >> > For design: if can bear current design, better not to touch it, I think >> > current design is still sustainable, and not complex for understanding, >> > so not need redesign. > We usually clean some of these as we hit them, otherwise people forget > about the clean-up later ;). In fact, I have sent another configuration modification about disabling CONFIG_BUG. it is in discussing... We are sure most of architectures do not need CONFIG_BUG, but for VGA_CONSOLE, we can not be sure most of architectures do not need it. For CONFIG_BUG issue, it will make noisy for most of architectures, but for VGA_CONSOLE issue, at least now, it is only effect with arm64. For CONFIG_BUG issue, it seems we can not provide a satisfied fixing if not get rid of it, but for VGA_CONSOLE issue, at least we have a complete fixing way for it. And now for CONFIG_BUG issue, it is still continuing discussing, and it seems that can not get a result quickly. So for VGA_CONSOLE... ;-) Thanks. -- Chen Gang Asianux Corporation ^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: [PATCH v2] arm64: compiling issue, need add include/asm/vga.h file 2013-04-24 10:20 ` Chen Gang @ 2013-04-24 10:42 ` Catalin Marinas -1 siblings, 0 replies; 34+ messages in thread From: Catalin Marinas @ 2013-04-24 10:42 UTC (permalink / raw) To: Chen Gang Cc: Will Deacon, Arnd Bergmann, linux-arm-kernel, linaro-kernel, linux-kernel On Wed, Apr 24, 2013 at 11:20:32AM +0100, Chen Gang wrote: > For compiling with allmodconfig, need vga.h file, so generate it which > just only include the asm-generic one. > > It is firstly used by drivers/gpu/drm/drm_irq.c. > > The related error: > include/video/vga.h:22:21: fatal error: asm/vga.h: No such file or directory > > Signed-off-by: Chen Gang <gang.chen@asianux.com> > --- > arch/arm64/include/asm/vga.h | 21 +++++++++++++++++++++ > 1 files changed, 21 insertions(+), 0 deletions(-) > create mode 100644 arch/arm64/include/asm/vga.h As Will said, please use asm/Kbuild. > diff --git a/arch/arm64/include/asm/vga.h b/arch/arm64/include/asm/vga.h > new file mode 100644 > index 0000000..1c49d51 > --- /dev/null > +++ b/arch/arm64/include/asm/vga.h > @@ -0,0 +1,21 @@ > +/* > + * Copyright (C) 2012 ARM Ltd. ARM copyright? You either work for ARM or there is a copyright assignment between you and ARM ;) -- Catalin ^ permalink raw reply [flat|nested] 34+ messages in thread
* [PATCH v2] arm64: compiling issue, need add include/asm/vga.h file @ 2013-04-24 10:42 ` Catalin Marinas 0 siblings, 0 replies; 34+ messages in thread From: Catalin Marinas @ 2013-04-24 10:42 UTC (permalink / raw) To: linux-arm-kernel On Wed, Apr 24, 2013 at 11:20:32AM +0100, Chen Gang wrote: > For compiling with allmodconfig, need vga.h file, so generate it which > just only include the asm-generic one. > > It is firstly used by drivers/gpu/drm/drm_irq.c. > > The related error: > include/video/vga.h:22:21: fatal error: asm/vga.h: No such file or directory > > Signed-off-by: Chen Gang <gang.chen@asianux.com> > --- > arch/arm64/include/asm/vga.h | 21 +++++++++++++++++++++ > 1 files changed, 21 insertions(+), 0 deletions(-) > create mode 100644 arch/arm64/include/asm/vga.h As Will said, please use asm/Kbuild. > diff --git a/arch/arm64/include/asm/vga.h b/arch/arm64/include/asm/vga.h > new file mode 100644 > index 0000000..1c49d51 > --- /dev/null > +++ b/arch/arm64/include/asm/vga.h > @@ -0,0 +1,21 @@ > +/* > + * Copyright (C) 2012 ARM Ltd. ARM copyright? You either work for ARM or there is a copyright assignment between you and ARM ;) -- Catalin ^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: [PATCH v2] arm64: compiling issue, need add include/asm/vga.h file 2013-04-24 10:42 ` Catalin Marinas @ 2013-04-24 10:51 ` Chen Gang -1 siblings, 0 replies; 34+ messages in thread From: Chen Gang @ 2013-04-24 10:51 UTC (permalink / raw) To: Catalin Marinas Cc: Will Deacon, Arnd Bergmann, linux-arm-kernel, linaro-kernel, linux-kernel On 2013年04月24日 18:42, Catalin Marinas wrote: > On Wed, Apr 24, 2013 at 11:20:32AM +0100, Chen Gang wrote: >> For compiling with allmodconfig, need vga.h file, so generate it which >> just only include the asm-generic one. >> >> It is firstly used by drivers/gpu/drm/drm_irq.c. >> >> The related error: >> include/video/vga.h:22:21: fatal error: asm/vga.h: No such file or directory >> >> Signed-off-by: Chen Gang <gang.chen@asianux.com> >> --- >> arch/arm64/include/asm/vga.h | 21 +++++++++++++++++++++ >> 1 files changed, 21 insertions(+), 0 deletions(-) >> create mode 100644 arch/arm64/include/asm/vga.h > > As Will said, please use asm/Kbuild. > OK, I will send. >> diff --git a/arch/arm64/include/asm/vga.h b/arch/arm64/include/asm/vga.h >> new file mode 100644 >> index 0000000..1c49d51 >> --- /dev/null >> +++ b/arch/arm64/include/asm/vga.h >> @@ -0,0 +1,21 @@ >> +/* >> + * Copyright (C) 2012 ARM Ltd. > > ARM copyright? You either work for ARM or there is a copyright > assignment between you and ARM ;) > :-) -- Chen Gang Asianux Corporation ^ permalink raw reply [flat|nested] 34+ messages in thread
* [PATCH v2] arm64: compiling issue, need add include/asm/vga.h file @ 2013-04-24 10:51 ` Chen Gang 0 siblings, 0 replies; 34+ messages in thread From: Chen Gang @ 2013-04-24 10:51 UTC (permalink / raw) To: linux-arm-kernel On 2013?04?24? 18:42, Catalin Marinas wrote: > On Wed, Apr 24, 2013 at 11:20:32AM +0100, Chen Gang wrote: >> For compiling with allmodconfig, need vga.h file, so generate it which >> just only include the asm-generic one. >> >> It is firstly used by drivers/gpu/drm/drm_irq.c. >> >> The related error: >> include/video/vga.h:22:21: fatal error: asm/vga.h: No such file or directory >> >> Signed-off-by: Chen Gang <gang.chen@asianux.com> >> --- >> arch/arm64/include/asm/vga.h | 21 +++++++++++++++++++++ >> 1 files changed, 21 insertions(+), 0 deletions(-) >> create mode 100644 arch/arm64/include/asm/vga.h > > As Will said, please use asm/Kbuild. > OK, I will send. >> diff --git a/arch/arm64/include/asm/vga.h b/arch/arm64/include/asm/vga.h >> new file mode 100644 >> index 0000000..1c49d51 >> --- /dev/null >> +++ b/arch/arm64/include/asm/vga.h >> @@ -0,0 +1,21 @@ >> +/* >> + * Copyright (C) 2012 ARM Ltd. > > ARM copyright? You either work for ARM or there is a copyright > assignment between you and ARM ;) > :-) -- Chen Gang Asianux Corporation ^ permalink raw reply [flat|nested] 34+ messages in thread
end of thread, other threads:[~2013-05-29 2:07 UTC | newest] Thread overview: 34+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2013-04-24 3:44 [PATCH] arm64: compiling issue, need add include/asm/vga.h file Chen Gang 2013-04-24 3:44 ` Chen Gang 2013-04-24 8:22 ` Catalin Marinas 2013-04-24 8:22 ` Catalin Marinas 2013-04-24 8:44 ` Chen Gang 2013-04-24 8:44 ` Chen Gang 2013-04-24 10:20 ` [PATCH v2] " Chen Gang 2013-04-24 10:20 ` Chen Gang 2013-04-24 10:23 ` Will Deacon 2013-04-24 10:23 ` Will Deacon 2013-04-24 10:50 ` Chen Gang 2013-04-24 10:50 ` Chen Gang 2013-04-24 10:58 ` [PATCH v3] " Chen Gang 2013-04-24 10:58 ` Chen Gang 2013-04-24 11:05 ` Catalin Marinas 2013-04-24 11:05 ` Catalin Marinas 2013-04-24 11:06 ` Chen Gang 2013-04-24 11:06 ` Chen Gang 2013-05-20 4:19 ` [PATCH] arm64: compiling issue for allmodconfig, need "asm/parport.h" by "drivers/parport/parport_pc.c" Chen Gang 2013-05-20 4:19 ` Chen Gang 2013-05-21 13:17 ` Catalin Marinas 2013-05-21 13:17 ` Catalin Marinas 2013-05-22 0:45 ` Chen Gang 2013-05-22 0:45 ` Chen Gang 2013-05-22 7:32 ` Chen Gang 2013-05-22 7:32 ` Chen Gang 2013-05-28 17:02 ` Catalin Marinas 2013-05-28 17:02 ` Catalin Marinas 2013-05-29 2:06 ` Chen Gang 2013-05-29 2:06 ` Chen Gang 2013-04-24 10:42 ` [PATCH v2] arm64: compiling issue, need add include/asm/vga.h file Catalin Marinas 2013-04-24 10:42 ` Catalin Marinas 2013-04-24 10:51 ` Chen Gang 2013-04-24 10:51 ` Chen Gang
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.