Re: [PATCH] arc: kernel: add default extern variable 'screen_info' in "setup.c"

2013-12-23 Thread Chen Gang
Hello James: For metag, also meet the same issue, could you help to fix it in metag tree? Thanks. On 12/06/2013 08:58 AM, rkuo wrote: > On Thu, Nov 28, 2013 at 05:07:05PM +0800, Chen Gang wrote: >> Hello Vineet: >> >> hexagon meets the same issue, could you help send related patch for it? >> >>

Re: [PATCH] arc: kernel: add default extern variable 'screen_info' in "setup.c"

2013-12-05 Thread Chen Gang
On 12/06/2013 08:58 AM, rkuo wrote: > On Thu, Nov 28, 2013 at 05:07:05PM +0800, Chen Gang wrote: >> Hello Vineet: >> >> hexagon meets the same issue, could you help send related patch for it? >> >> If you have no time, please let me know, I should send it within next >> month. >> >> BTW: I assume h

Re: [PATCH] arc: kernel: add default extern variable 'screen_info' in "setup.c"

2013-12-05 Thread rkuo
On Thu, Nov 28, 2013 at 05:07:05PM +0800, Chen Gang wrote: > Hello Vineet: > > hexagon meets the same issue, could you help send related patch for it? > > If you have no time, please let me know, I should send it within next > month. > > BTW: I assume hexagon need not support VGA_CONSOLE, either

Re: [PATCH] arc: kernel: add default extern variable 'screen_info' in "setup.c"

2013-11-28 Thread Chen Gang
Hello Vineet: hexagon meets the same issue, could you help send related patch for it? If you have no time, please let me know, I should send it within next month. BTW: I assume hexagon need not support VGA_CONSOLE, either, if it is incorrect, please let me know thanks. > On 10/24/2013 07:50 PM

Re: [PATCH] arc: kernel: add default extern variable 'screen_info' in "setup.c"

2013-10-25 Thread Chen Gang
On 10/24/2013 07:51 PM, Geert Uytterhoeven wrote: > Hi Chen, > > On Thu, Oct 24, 2013 at 1:23 PM, Chen Gang wrote: >>> Lot's of negative dependencies are missing for VGA_CONSOLE. >>> I think only alpha, some arm, ia64, some mips, some powerpc, and x86 >>> may use it. >>> >> >> in "arch" sub-direc

Re: [PATCH] arc: kernel: add default extern variable 'screen_info' in "setup.c"

2013-10-25 Thread Chen Gang
On 10/24/2013 07:50 PM, Vineet Gupta wrote: > On 10/24/2013 04:56 PM, Chen Gang wrote: >> If what you said is correct, could you help to send related patch for >> it? if I send this kind of patches, it will be easily redirected to >> "/dev/null". > > Geert has explained the state of things quite n

Re: [PATCH] arc: kernel: add default extern variable 'screen_info' in "setup.c"

2013-10-24 Thread Geert Uytterhoeven
Hi Chen, On Thu, Oct 24, 2013 at 1:23 PM, Chen Gang wrote: >> Lot's of negative dependencies are missing for VGA_CONSOLE. >> I think only alpha, some arm, ia64, some mips, some powerpc, and x86 >> may use it. >> > > in "arch" sub-directory, contents 13/30 (43.3%) architectures contents > VGA_CONS

Re: [PATCH] arc: kernel: add default extern variable 'screen_info' in "setup.c"

2013-10-24 Thread Vineet Gupta
On 10/24/2013 04:56 PM, Chen Gang wrote: > If what you said is correct, could you help to send related patch for > it? if I send this kind of patches, it will be easily redirected to > "/dev/null". Geert has explained the state of things quite nicely. Please send your patch to introduce HAVE_xxx.

Re: [PATCH] arc: kernel: add default extern variable 'screen_info' in "setup.c"

2013-10-24 Thread Chen Gang
On 10/24/2013 04:30 PM, Geert Uytterhoeven wrote: > On Thu, Oct 24, 2013 at 4:56 AM, Chen Gang wrote: >>> NAK. >> >> OK, thanks. At least this patch is incorrect. >> >> >>> ARC will never have VGA console. You need to add !ARC to relevant Kconfig. >>> However that approach is frowned upon in gene

Re: [PATCH] arc: kernel: add default extern variable 'screen_info' in "setup.c"

2013-10-24 Thread Geert Uytterhoeven
On Thu, Oct 24, 2013 at 4:56 AM, Chen Gang wrote: >> NAK. > > OK, thanks. At least this patch is incorrect. > > >> ARC will never have VGA console. You need to add !ARC to relevant Kconfig. >> However that approach is frowned upon in general. The current way to doing >> such things is to define

Re: [PATCH] arc: kernel: add default extern variable 'screen_info' in "setup.c"

2013-10-23 Thread Chen Gang
sianux.com] > Sent: Wednesday, October 23, 2013 4:39 PM > To: vgu...@synopsys.com; Arnd Bergmann; sachin.ka...@linaro.org; Paul > Gortmaker; James Hogan > Cc: linux-kernel@vger.kernel.org > Subject: [PATCH] arc: kernel: add default extern variable 'screen_info' in >

RE: [PATCH] arc: kernel: add default extern variable 'screen_info' in "setup.c"

2013-10-23 Thread Vineet Gupta
From: Chen Gang [gang.c...@asianux.com] Sent: Wednesday, October 23, 2013 4:39 PM To: vgu...@synopsys.com; Arnd Bergmann; sachin.ka...@linaro.org; Paul Gortmaker; James Hogan Cc: linux-kernel@vger.kernel.org Subject: [PATCH] arc: kernel: add default extern variable 'screen_inf

[PATCH] arc: kernel: add default extern variable 'screen_info' in "setup.c"

2013-10-23 Thread Chen Gang
Add default 'screen_info' just like some of other architectures (e.g. cris, score, sh, tile), or can not pass compiling. The related error (with allmodconfig): drivers/built-in.o: In function `vgacon_save_screen': drivers/video/console/vgacon.c:1347: undefined reference to `screen_info' dri