* [PATCH] arch/microblaze/kernel/setup.c: Add default 'screen_info" for fixing compiling break
@ 2014-09-14 8:32 Chen Gang
2014-09-14 8:45 ` Geert Uytterhoeven
0 siblings, 1 reply; 10+ messages in thread
From: Chen Gang @ 2014-09-14 8:32 UTC (permalink / raw)
To: Michal Simek, rob.herring, Geert Uytterhoeven, pebolle
Cc: linux-kernel@vger.kernel.org
Add default one just like other architectures have done (e.g. cris,
score, sh ...), the related error with allmodconfig:
LD init/built-in.o
drivers/built-in.o: In function `vgacon_save_screen':
drivers/video/console/.tmp_vgacon.o:(.text+0x834d0): undefined reference to `screen_info'
drivers/video/console/.tmp_vgacon.o:(.text+0x834ec): undefined reference to `screen_info'
drivers/built-in.o: In function `vgacon_resize':
drivers/video/console/.tmp_vgacon.o:(.text+0x83cdc): undefined reference to `screen_info'
drivers/video/console/.tmp_vgacon.o:(.text+0x83d38): undefined reference to `screen_info'
drivers/built-in.o: In function `vgacon_switch':
drivers/video/console/.tmp_vgacon.o:(.text+0x83f50): undefined reference to `screen_info'
drivers/built-in.o:drivers/video/console/.tmp_vgacon.o:(.text+0x8407c): more undefined references to `screen_info' follow
make: *** [vmlinux] Error 1
Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com>
---
arch/microblaze/kernel/setup.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/arch/microblaze/kernel/setup.c b/arch/microblaze/kernel/setup.c
index ab5b488..8756d61 100644
--- a/arch/microblaze/kernel/setup.c
+++ b/arch/microblaze/kernel/setup.c
@@ -18,6 +18,7 @@
#include <linux/console.h>
#include <linux/debugfs.h>
#include <linux/of_fdt.h>
+#include <linux/screen_info.h>
#include <asm/setup.h>
#include <asm/sections.h>
@@ -42,6 +43,8 @@ DEFINE_PER_CPU(unsigned int, ENTRY_SP); /* Saved SP on kernel entry */
DEFINE_PER_CPU(unsigned int, R11_SAVE); /* Temp variable for entry */
DEFINE_PER_CPU(unsigned int, CURRENT_SAVE); /* Saved current pointer */
+struct screen_info screen_info;
+
unsigned int boot_cpuid;
/*
* Placed cmd_line to .data section because can be initialized from
--
1.9.3
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH] arch/microblaze/kernel/setup.c: Add default 'screen_info" for fixing compiling break
2014-09-14 8:32 [PATCH] arch/microblaze/kernel/setup.c: Add default 'screen_info" for fixing compiling break Chen Gang
@ 2014-09-14 8:45 ` Geert Uytterhoeven
2014-09-14 9:02 ` Chen Gang
2014-09-15 8:50 ` Jesper Nilsson
0 siblings, 2 replies; 10+ messages in thread
From: Geert Uytterhoeven @ 2014-09-14 8:45 UTC (permalink / raw)
To: Chen Gang
Cc: Michal Simek, Rob Herring, Paul Bolle,
linux-kernel@vger.kernel.org, Mikael Starvik, Jesper Nilsson
Hi Chen,
On Sun, Sep 14, 2014 at 10:32 AM, Chen Gang <gang.chen.5i5j@gmail.com> wrote:
> Add default one just like other architectures have done (e.g. cris,
> score, sh ...), the related error with allmodconfig:
>
> LD init/built-in.o
> drivers/built-in.o: In function `vgacon_save_screen':
> drivers/video/console/.tmp_vgacon.o:(.text+0x834d0): undefined reference to `screen_info'
> drivers/video/console/.tmp_vgacon.o:(.text+0x834ec): undefined reference to `screen_info'
> drivers/built-in.o: In function `vgacon_resize':
> drivers/video/console/.tmp_vgacon.o:(.text+0x83cdc): undefined reference to `screen_info'
> drivers/video/console/.tmp_vgacon.o:(.text+0x83d38): undefined reference to `screen_info'
> drivers/built-in.o: In function `vgacon_switch':
> drivers/video/console/.tmp_vgacon.o:(.text+0x83f50): undefined reference to `screen_info'
> drivers/built-in.o:drivers/video/console/.tmp_vgacon.o:(.text+0x8407c): more undefined references to `screen_info' follow
> make: *** [vmlinux] Error 1
Can microblaze actually have VGA?
If not, it's better to disable it.
BTW, it's interesting you mention cris, as VGA_CONSOLE depends on !CRIS
in drivers/video/console/Kconfig...
I (still) think it's time to introduce ARCH_MIGHT_HAVE_VGA, and make
VGA_CONSOLE depend on that, cfr.
commit f13399f033ae3d49e1808bacabd83d116844c94e
Merge: 049ffa8ab33a b2c32b32a212
Author: Linus Torvalds <torvalds@linux-foundation.org>
Date: Fri Nov 15 14:05:15 2013 -0800
Merge tag 'for-linus' of
git://linux-c6x.org/git/projects/linux-c6x-upstreaming
Pull Kconfig cleanups from Mark Salter:
"Remove some unused config options from C6X and clean up PC_PARPORT
dependencies. The latter was discussed here:
https://lkml.org/lkml/2013/10/8/12"
* tag 'for-linus' of git://linux-c6x.org/git/projects/linux-c6x-upstreaming:
c6x: remove unused COMMON_CLKDEV Kconfig parameter
Kconfig cleanup (PARPORT_PC dependencies)
x86: select ARCH_MIGHT_HAVE_PC_PARPORT
unicore32: select ARCH_MIGHT_HAVE_PC_PARPORT
sparc: select ARCH_MIGHT_HAVE_PC_PARPORT
sh: select ARCH_MIGHT_HAVE_PC_PARPORT
powerpc: select ARCH_MIGHT_HAVE_PC_PARPORT
parisc: select ARCH_MIGHT_HAVE_PC_PARPORT
mips: select ARCH_MIGHT_HAVE_PC_PARPORT
microblaze: select ARCH_MIGHT_HAVE_PC_PARPORT
m68k: select ARCH_MIGHT_HAVE_PC_PARPORT
ia64: select ARCH_MIGHT_HAVE_PC_PARPORT
arm: select ARCH_MIGHT_HAVE_PC_PARPORT
alpha: select ARCH_MIGHT_HAVE_PC_PARPORT
c6x: remove unused parameter in Kconfig
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] arch/microblaze/kernel/setup.c: Add default 'screen_info" for fixing compiling break
2014-09-14 8:45 ` Geert Uytterhoeven
@ 2014-09-14 9:02 ` Chen Gang
2014-09-16 9:23 ` Michal Simek
2014-09-15 8:50 ` Jesper Nilsson
1 sibling, 1 reply; 10+ messages in thread
From: Chen Gang @ 2014-09-14 9:02 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Michal Simek, Rob Herring, Paul Bolle,
linux-kernel@vger.kernel.org, Mikael Starvik, Jesper Nilsson
On 09/14/2014 04:45 PM, Geert Uytterhoeven wrote:
> Hi Chen,
>
> On Sun, Sep 14, 2014 at 10:32 AM, Chen Gang <gang.chen.5i5j@gmail.com> wrote:
>> Add default one just like other architectures have done (e.g. cris,
>> score, sh ...), the related error with allmodconfig:
>>
>> LD init/built-in.o
>> drivers/built-in.o: In function `vgacon_save_screen':
>> drivers/video/console/.tmp_vgacon.o:(.text+0x834d0): undefined reference to `screen_info'
>> drivers/video/console/.tmp_vgacon.o:(.text+0x834ec): undefined reference to `screen_info'
>> drivers/built-in.o: In function `vgacon_resize':
>> drivers/video/console/.tmp_vgacon.o:(.text+0x83cdc): undefined reference to `screen_info'
>> drivers/video/console/.tmp_vgacon.o:(.text+0x83d38): undefined reference to `screen_info'
>> drivers/built-in.o: In function `vgacon_switch':
>> drivers/video/console/.tmp_vgacon.o:(.text+0x83f50): undefined reference to `screen_info'
>> drivers/built-in.o:drivers/video/console/.tmp_vgacon.o:(.text+0x8407c): more undefined references to `screen_info' follow
>> make: *** [vmlinux] Error 1
>
> Can microblaze actually have VGA?
>
> If not, it's better to disable it.
>
Welcome the related microblaze maintainers ideas for it
> BTW, it's interesting you mention cris, as VGA_CONSOLE depends on !CRIS
> in drivers/video/console/Kconfig...
>
Welcome the related cris maintainers ideas for it.
> I (still) think it's time to introduce ARCH_MIGHT_HAVE_VGA, and make
> VGA_CONSOLE depend on that, cfr.
>
> commit f13399f033ae3d49e1808bacabd83d116844c94e
> Merge: 049ffa8ab33a b2c32b32a212
> Author: Linus Torvalds <torvalds@linux-foundation.org>
> Date: Fri Nov 15 14:05:15 2013 -0800
>
> Merge tag 'for-linus' of
> git://linux-c6x.org/git/projects/linux-c6x-upstreaming
>
> Pull Kconfig cleanups from Mark Salter:
> "Remove some unused config options from C6X and clean up PC_PARPORT
> dependencies. The latter was discussed here:
>
> https://lkml.org/lkml/2013/10/8/12"
>
> * tag 'for-linus' of git://linux-c6x.org/git/projects/linux-c6x-upstreaming:
> c6x: remove unused COMMON_CLKDEV Kconfig parameter
> Kconfig cleanup (PARPORT_PC dependencies)
> x86: select ARCH_MIGHT_HAVE_PC_PARPORT
> unicore32: select ARCH_MIGHT_HAVE_PC_PARPORT
> sparc: select ARCH_MIGHT_HAVE_PC_PARPORT
> sh: select ARCH_MIGHT_HAVE_PC_PARPORT
> powerpc: select ARCH_MIGHT_HAVE_PC_PARPORT
> parisc: select ARCH_MIGHT_HAVE_PC_PARPORT
> mips: select ARCH_MIGHT_HAVE_PC_PARPORT
> microblaze: select ARCH_MIGHT_HAVE_PC_PARPORT
> m68k: select ARCH_MIGHT_HAVE_PC_PARPORT
> ia64: select ARCH_MIGHT_HAVE_PC_PARPORT
> arm: select ARCH_MIGHT_HAVE_PC_PARPORT
> alpha: select ARCH_MIGHT_HAVE_PC_PARPORT
> c6x: remove unused parameter in Kconfig
>
At least for me, the contents above sounds reasonable.
And also thank you very much for always supporting me for sending
'global' patches for Kconfig.
But in honest, what I have done is not quite well, at present. And I
should continue trying...
Thanks.
--
Chen Gang
Open share and attitude like air water and life which God blessed
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] arch/microblaze/kernel/setup.c: Add default 'screen_info" for fixing compiling break
2014-09-14 8:45 ` Geert Uytterhoeven
2014-09-14 9:02 ` Chen Gang
@ 2014-09-15 8:50 ` Jesper Nilsson
2014-09-15 11:10 ` Chen Gang
1 sibling, 1 reply; 10+ messages in thread
From: Jesper Nilsson @ 2014-09-15 8:50 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Chen Gang, Michal Simek, Rob Herring, Paul Bolle,
linux-kernel@vger.kernel.org, Mikael Starvik, Jesper Nilsson
On Sun, Sep 14, 2014 at 10:45:23AM +0200, Geert Uytterhoeven wrote:
> Hi Chen,
>
> On Sun, Sep 14, 2014 at 10:32 AM, Chen Gang <gang.chen.5i5j@gmail.com> wrote:
> > Add default one just like other architectures have done (e.g. cris,
> > score, sh ...), the related error with allmodconfig:
> >
> > LD init/built-in.o
> > drivers/built-in.o: In function `vgacon_save_screen':
> > drivers/video/console/.tmp_vgacon.o:(.text+0x834d0): undefined reference to `screen_info'
> > drivers/video/console/.tmp_vgacon.o:(.text+0x834ec): undefined reference to `screen_info'
> > drivers/built-in.o: In function `vgacon_resize':
> > drivers/video/console/.tmp_vgacon.o:(.text+0x83cdc): undefined reference to `screen_info'
> > drivers/video/console/.tmp_vgacon.o:(.text+0x83d38): undefined reference to `screen_info'
> > drivers/built-in.o: In function `vgacon_switch':
> > drivers/video/console/.tmp_vgacon.o:(.text+0x83f50): undefined reference to `screen_info'
> > drivers/built-in.o:drivers/video/console/.tmp_vgacon.o:(.text+0x8407c): more undefined references to `screen_info' follow
> > make: *** [vmlinux] Error 1
>
> Can microblaze actually have VGA?
>
> If not, it's better to disable it.
>
> BTW, it's interesting you mention cris, as VGA_CONSOLE depends on !CRIS
> in drivers/video/console/Kconfig...
No CRIS platform has ever had a video console.
> I (still) think it's time to introduce ARCH_MIGHT_HAVE_VGA, and make
> VGA_CONSOLE depend on that, cfr.
Sounds reasonable, at least it can avoid the specific depends on !*arch* cruft.
> Gr{oetje,eeting}s,
>
> Geert
/^JN - Jesper Nilsson
--
Jesper Nilsson -- jesper.nilsson@axis.com
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] arch/microblaze/kernel/setup.c: Add default 'screen_info" for fixing compiling break
2014-09-15 8:50 ` Jesper Nilsson
@ 2014-09-15 11:10 ` Chen Gang
2014-09-20 4:08 ` Chen Gang
0 siblings, 1 reply; 10+ messages in thread
From: Chen Gang @ 2014-09-15 11:10 UTC (permalink / raw)
To: Jesper Nilsson, Geert Uytterhoeven
Cc: Michal Simek, Rob Herring, Paul Bolle,
linux-kernel@vger.kernel.org, Mikael Starvik, Jesper Nilsson
On 9/15/14 16:50, Jesper Nilsson wrote:
> On Sun, Sep 14, 2014 at 10:45:23AM +0200, Geert Uytterhoeven wrote:
>> Hi Chen,
>>
>> On Sun, Sep 14, 2014 at 10:32 AM, Chen Gang <gang.chen.5i5j@gmail.com> wrote:
>>> Add default one just like other architectures have done (e.g. cris,
>>> score, sh ...), the related error with allmodconfig:
>>>
>>> LD init/built-in.o
>>> drivers/built-in.o: In function `vgacon_save_screen':
>>> drivers/video/console/.tmp_vgacon.o:(.text+0x834d0): undefined reference to `screen_info'
>>> drivers/video/console/.tmp_vgacon.o:(.text+0x834ec): undefined reference to `screen_info'
>>> drivers/built-in.o: In function `vgacon_resize':
>>> drivers/video/console/.tmp_vgacon.o:(.text+0x83cdc): undefined reference to `screen_info'
>>> drivers/video/console/.tmp_vgacon.o:(.text+0x83d38): undefined reference to `screen_info'
>>> drivers/built-in.o: In function `vgacon_switch':
>>> drivers/video/console/.tmp_vgacon.o:(.text+0x83f50): undefined reference to `screen_info'
>>> drivers/built-in.o:drivers/video/console/.tmp_vgacon.o:(.text+0x8407c): more undefined references to `screen_info' follow
>>> make: *** [vmlinux] Error 1
>>
>> Can microblaze actually have VGA?
>>
>> If not, it's better to disable it.
>>
>> BTW, it's interesting you mention cris, as VGA_CONSOLE depends on !CRIS
>> in drivers/video/console/Kconfig...
>
> No CRIS platform has ever had a video console.
>
>> I (still) think it's time to introduce ARCH_MIGHT_HAVE_VGA, and make
>> VGA_CONSOLE depend on that, cfr.
>
> Sounds reasonable, at least it can avoid the specific depends on !*arch* cruft.
>
Thank you very much, next when I try the ARCH_MIGHT_HAVE_VGA, I shall
process it (remove useless "screen_info").
Thanks.
--
Chen Gang
Open, share, and attitude like air, water, and life which God blessed
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] arch/microblaze/kernel/setup.c: Add default 'screen_info" for fixing compiling break
2014-09-14 9:02 ` Chen Gang
@ 2014-09-16 9:23 ` Michal Simek
2014-09-16 10:04 ` Geert Uytterhoeven
0 siblings, 1 reply; 10+ messages in thread
From: Michal Simek @ 2014-09-16 9:23 UTC (permalink / raw)
To: Chen Gang, Geert Uytterhoeven
Cc: Rob Herring, Paul Bolle, linux-kernel@vger.kernel.org,
Mikael Starvik, Jesper Nilsson
[-- Attachment #1: Type: text/plain, Size: 1780 bytes --]
On 09/14/2014 11:02 AM, Chen Gang wrote:
> On 09/14/2014 04:45 PM, Geert Uytterhoeven wrote:
>> Hi Chen,
>>
>> On Sun, Sep 14, 2014 at 10:32 AM, Chen Gang <gang.chen.5i5j@gmail.com> wrote:
>>> Add default one just like other architectures have done (e.g. cris,
>>> score, sh ...), the related error with allmodconfig:
>>>
>>> LD init/built-in.o
>>> drivers/built-in.o: In function `vgacon_save_screen':
>>> drivers/video/console/.tmp_vgacon.o:(.text+0x834d0): undefined reference to `screen_info'
>>> drivers/video/console/.tmp_vgacon.o:(.text+0x834ec): undefined reference to `screen_info'
>>> drivers/built-in.o: In function `vgacon_resize':
>>> drivers/video/console/.tmp_vgacon.o:(.text+0x83cdc): undefined reference to `screen_info'
>>> drivers/video/console/.tmp_vgacon.o:(.text+0x83d38): undefined reference to `screen_info'
>>> drivers/built-in.o: In function `vgacon_switch':
>>> drivers/video/console/.tmp_vgacon.o:(.text+0x83f50): undefined reference to `screen_info'
>>> drivers/built-in.o:drivers/video/console/.tmp_vgacon.o:(.text+0x8407c): more undefined references to `screen_info' follow
>>> make: *** [vmlinux] Error 1
>>
>> Can microblaze actually have VGA?
>>
>> If not, it's better to disable it.
>>
>
> Welcome the related microblaze maintainers ideas for it
Is this is just about VGA or video console in general?
There is xilinx FB driver and other FB are around too.
Thanks,
Michal
--
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/
Maintainer of Linux kernel - Xilinx Zynq ARM architecture
Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] arch/microblaze/kernel/setup.c: Add default 'screen_info" for fixing compiling break
2014-09-16 9:23 ` Michal Simek
@ 2014-09-16 10:04 ` Geert Uytterhoeven
2014-09-16 10:59 ` Michal Simek
0 siblings, 1 reply; 10+ messages in thread
From: Geert Uytterhoeven @ 2014-09-16 10:04 UTC (permalink / raw)
To: Michal Simek
Cc: Chen Gang, Rob Herring, Paul Bolle, linux-kernel@vger.kernel.org,
Mikael Starvik, Jesper Nilsson
Hi Michal,
On Tue, Sep 16, 2014 at 11:23 AM, Michal Simek <monstr@monstr.eu> wrote:
> On 09/14/2014 11:02 AM, Chen Gang wrote:
>> On 09/14/2014 04:45 PM, Geert Uytterhoeven wrote:
>>> Hi Chen,
>>>
>>> On Sun, Sep 14, 2014 at 10:32 AM, Chen Gang <gang.chen.5i5j@gmail.com> wrote:
>>>> Add default one just like other architectures have done (e.g. cris,
>>>> score, sh ...), the related error with allmodconfig:
>>>>
>>>> LD init/built-in.o
>>>> drivers/built-in.o: In function `vgacon_save_screen':
>>>> drivers/video/console/.tmp_vgacon.o:(.text+0x834d0): undefined reference to `screen_info'
>>>> drivers/video/console/.tmp_vgacon.o:(.text+0x834ec): undefined reference to `screen_info'
>>>> drivers/built-in.o: In function `vgacon_resize':
>>>> drivers/video/console/.tmp_vgacon.o:(.text+0x83cdc): undefined reference to `screen_info'
>>>> drivers/video/console/.tmp_vgacon.o:(.text+0x83d38): undefined reference to `screen_info'
>>>> drivers/built-in.o: In function `vgacon_switch':
>>>> drivers/video/console/.tmp_vgacon.o:(.text+0x83f50): undefined reference to `screen_info'
>>>> drivers/built-in.o:drivers/video/console/.tmp_vgacon.o:(.text+0x8407c): more undefined references to `screen_info' follow
>>>> make: *** [vmlinux] Error 1
>>>
>>> Can microblaze actually have VGA?
>>>
>>> If not, it's better to disable it.
>>>
>>
>> Welcome the related microblaze maintainers ideas for it
>
> Is this is just about VGA or video console in general?
> There is xilinx FB driver and other FB are around too.
These days screen_info is only used by:
- vgacon,
- dummycon on ARM,
- a few fb drivers (efifb, intelfbdrv,sis, vesafb, and vga16fb).
E.g. m68k doesn't have it.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] arch/microblaze/kernel/setup.c: Add default 'screen_info" for fixing compiling break
2014-09-16 10:04 ` Geert Uytterhoeven
@ 2014-09-16 10:59 ` Michal Simek
2014-09-16 14:34 ` Chen Gang
0 siblings, 1 reply; 10+ messages in thread
From: Michal Simek @ 2014-09-16 10:59 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Chen Gang, Rob Herring, Paul Bolle, linux-kernel@vger.kernel.org,
Mikael Starvik, Jesper Nilsson
[-- Attachment #1: Type: text/plain, Size: 2316 bytes --]
On 09/16/2014 12:04 PM, Geert Uytterhoeven wrote:
> Hi Michal,
>
> On Tue, Sep 16, 2014 at 11:23 AM, Michal Simek <monstr@monstr.eu> wrote:
>> On 09/14/2014 11:02 AM, Chen Gang wrote:
>>> On 09/14/2014 04:45 PM, Geert Uytterhoeven wrote:
>>>> Hi Chen,
>>>>
>>>> On Sun, Sep 14, 2014 at 10:32 AM, Chen Gang <gang.chen.5i5j@gmail.com> wrote:
>>>>> Add default one just like other architectures have done (e.g. cris,
>>>>> score, sh ...), the related error with allmodconfig:
>>>>>
>>>>> LD init/built-in.o
>>>>> drivers/built-in.o: In function `vgacon_save_screen':
>>>>> drivers/video/console/.tmp_vgacon.o:(.text+0x834d0): undefined reference to `screen_info'
>>>>> drivers/video/console/.tmp_vgacon.o:(.text+0x834ec): undefined reference to `screen_info'
>>>>> drivers/built-in.o: In function `vgacon_resize':
>>>>> drivers/video/console/.tmp_vgacon.o:(.text+0x83cdc): undefined reference to `screen_info'
>>>>> drivers/video/console/.tmp_vgacon.o:(.text+0x83d38): undefined reference to `screen_info'
>>>>> drivers/built-in.o: In function `vgacon_switch':
>>>>> drivers/video/console/.tmp_vgacon.o:(.text+0x83f50): undefined reference to `screen_info'
>>>>> drivers/built-in.o:drivers/video/console/.tmp_vgacon.o:(.text+0x8407c): more undefined references to `screen_info' follow
>>>>> make: *** [vmlinux] Error 1
>>>>
>>>> Can microblaze actually have VGA?
>>>>
>>>> If not, it's better to disable it.
>>>>
>>>
>>> Welcome the related microblaze maintainers ideas for it
>>
>> Is this is just about VGA or video console in general?
>> There is xilinx FB driver and other FB are around too.
>
> These days screen_info is only used by:
> - vgacon,
> - dummycon on ARM,
> - a few fb drivers (efifb, intelfbdrv,sis, vesafb, and vga16fb).
>
> E.g. m68k doesn't have it.
ok. IRC I think I remember one non mainline FB driver which required it too.
It breaks nothing for me that's why no problem to apply it.
Thanks,
Michal
--
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/
Maintainer of Linux kernel - Xilinx Zynq ARM architecture
Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] arch/microblaze/kernel/setup.c: Add default 'screen_info" for fixing compiling break
2014-09-16 10:59 ` Michal Simek
@ 2014-09-16 14:34 ` Chen Gang
0 siblings, 0 replies; 10+ messages in thread
From: Chen Gang @ 2014-09-16 14:34 UTC (permalink / raw)
To: monstr@monstr.eu
Cc: Geert Uytterhoeven, Rob Herring, Paul Bolle,
linux-kernel@vger.kernel.org, Mikael Starvik, Jesper Nilsson
Thank all of your work. And I also try some patches for microblaze compiler,
hope I can finish within this month (already got much help from related gcc
maintainer). And next month, I shall start Qemu for microblaze.
Thanks again.
发自我的 iPad
在 2014年9月16日，下午6:59，Michal Simek <monstr@monstr.eu> 写道：
>>>> Welcome the related microblaze maintainers ideas for it
>>>
>>> Is this is just about VGA or video console in general?
>>> There is xilinx FB driver and other FB are around too.
>>
>> These days screen_info is only used by:
>> - vgacon,
>> - dummycon on ARM,
>> - a few fb drivers (efifb, intelfbdrv,sis, vesafb, and vga16fb).
>>
>> E.g. m68k doesn't have it.
>
> ok. IRC I think I remember one non mainline FB driver which required it too.
> It breaks nothing for me that's why no problem to apply it.
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] arch/microblaze/kernel/setup.c: Add default 'screen_info" for fixing compiling break
2014-09-15 11:10 ` Chen Gang
@ 2014-09-20 4:08 ` Chen Gang
0 siblings, 0 replies; 10+ messages in thread
From: Chen Gang @ 2014-09-20 4:08 UTC (permalink / raw)
To: Jesper Nilsson, Geert Uytterhoeven
Cc: Michal Simek, Rob Herring, Paul Bolle,
linux-kernel@vger.kernel.org, Mikael Starvik, Jesper Nilsson,
Arnd Bergmann, Andrew Morton
On 09/15/2014 07:10 PM, Chen Gang wrote:
> On 9/15/14 16:50, Jesper Nilsson wrote:
>> On Sun, Sep 14, 2014 at 10:45:23AM +0200, Geert Uytterhoeven wrote:
[...]
>>
>> Sounds reasonable, at least it can avoid the specific depends on !*arch* cruft.
>>
>
> Thank you very much, next when I try the ARCH_MIGHT_HAVE_VGA, I shall
> process it (remove useless "screen_info").
>
Excuse me, after some trying, I guess, at present, I am not suitable to
perform global architecture wide patches (e.g. ARCH_MIGHT_HAVE_VGA). So
welcome any other members to help try.
I guess, in the future, after I finish all architectures allmodconfig,
I shall be suitable for sending global architecture wide patches.
Thanks.
--
Chen Gang
Open, share, and attitude like air, water, and life which God blessed
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2014-09-20 4:08 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-09-14 8:32 [PATCH] arch/microblaze/kernel/setup.c: Add default 'screen_info" for fixing compiling break Chen Gang
2014-09-14 8:45 ` Geert Uytterhoeven
2014-09-14 9:02 ` Chen Gang
2014-09-16 9:23 ` Michal Simek
2014-09-16 10:04 ` Geert Uytterhoeven
2014-09-16 10:59 ` Michal Simek
2014-09-16 14:34 ` Chen Gang
2014-09-15 8:50 ` Jesper Nilsson
2014-09-15 11:10 ` Chen Gang
2014-09-20 4:08 ` Chen Gang
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).