Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] add support for virtual terminal
@ 2011-11-07 10:21 Yegor Yefremov
  2011-11-07 10:32 ` Baruch Siach
  0 siblings, 1 reply; 3+ messages in thread
From: Yegor Yefremov @ 2011-11-07 10:21 UTC (permalink / raw)
  To: buildroot

introduces new option "Enable virtual console on tty1"
to enable virtual terminal on tty1

Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com>
---
 fs/skeleton/etc/inittab    |    3 +++
 target/generic/Config.in   |    3 +++
 target/generic/Makefile.in |   14 ++++++++++++++
 3 files changed, 20 insertions(+)

Index: b/fs/skeleton/etc/inittab
===================================================================
--- a/fs/skeleton/etc/inittab
+++ b/fs/skeleton/etc/inittab
@@ -23,6 +23,9 @@
 # now run any rc scripts
 ::sysinit:/etc/init.d/rcS
 
+# Put a getty on a virtual terminal
+#tty1::respawn:/sbin/getty tty1 38400
+
 # Put a getty on the serial port
 #ttyS0::respawn:/sbin/getty -L ttyS0 115200 vt100 # GENERIC_SERIAL
 
Index: b/target/generic/Config.in
===================================================================
--- a/target/generic/Config.in
+++ b/target/generic/Config.in
@@ -116,6 +116,9 @@
 	default "57600"		if BR2_TARGET_GENERIC_GETTY_BAUDRATE_57600
 	default "115200"	if BR2_TARGET_GENERIC_GETTY_BAUDRATE_115200
 
+config BR2_TARGET_GENERIC_TTY_PORT
+	bool "Enable virtual console on tty1"
+
 config BR2_TARGET_GENERIC_REMOUNT_ROOTFS_RW
 	bool "remount root filesystem read-write during boot"
 	default y
Index: b/target/generic/Makefile.in
===================================================================
--- a/target/generic/Makefile.in
+++ b/target/generic/Makefile.in
@@ -23,6 +23,14 @@
 	$(SED) '/# GENERIC_SERIAL$$/s~^.*#~$(shell echo $(TARGET_GENERIC_GETTY) | tail -c+4)::respawn:/sbin/getty -L $(TARGET_GENERIC_GETTY) $(TARGET_GENERIC_GETTY_BAUDRATE) vt100 #~' \
 		$(TARGET_DIR)/etc/inittab
 
+# Put a getty on virtual terminal
+target-generic-do-getty-tty1:
+	$(SED) 's/#tty1/tty1/' $(TARGET_DIR)/etc/inittab
+
+# Don't put a getty on virtual terminal
+target-generic-dont-getty-tty1:
+	$(SED) 's/tty1/#tty1/' $(TARGET_DIR)/etc/inittab
+
 # Find commented line, if any, and remove leading '#'s
 target-generic-do-remount-rw:
 	$(SED) '/^#.*# REMOUNT_ROOTFS_RW$$/s~^#\+~~' $(TARGET_DIR)/etc/inittab
@@ -39,6 +47,12 @@
 TARGETS += target-generic-issue
 endif
 
+ifeq ($(BR2_TARGET_GENERIC_TTY_PORT),y)
+TARGETS += target-generic-do-getty-tty1
+else
+TARGETS += target-generic-dont-getty-tty1
+endif
+
 ifeq ($(BR2_ROOTFS_SKELETON_DEFAULT),y)
 ifeq ($(BR2_PACKAGE_SYSVINIT),y)
 TARGETS += target-generic-getty-sysvinit

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

* [Buildroot] [PATCH] add support for virtual terminal
  2011-11-07 10:21 [Buildroot] [PATCH] add support for virtual terminal Yegor Yefremov
@ 2011-11-07 10:32 ` Baruch Siach
  2011-11-07 11:48   ` Yegor Yefremov
  0 siblings, 1 reply; 3+ messages in thread
From: Baruch Siach @ 2011-11-07 10:32 UTC (permalink / raw)
  To: buildroot

Hi Yegor,

On Mon, Nov 07, 2011 at 11:21:29AM +0100, Yegor Yefremov wrote:
> introduces new option "Enable virtual console on tty1"
> to enable virtual terminal on tty1
> 
> Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com>
> ---

[snip]

> +# Put a getty on a virtual terminal
> +#tty1::respawn:/sbin/getty tty1 38400

You should probably add '# GENERIC_VT' in this line. See below.

> +
>  # Put a getty on the serial port
>  #ttyS0::respawn:/sbin/getty -L ttyS0 115200 vt100 # GENERIC_SERIAL

[snip]

> Index: b/target/generic/Makefile.in
> ===================================================================
> --- a/target/generic/Makefile.in
> +++ b/target/generic/Makefile.in
> @@ -23,6 +23,14 @@
>  	$(SED) '/# GENERIC_SERIAL$$/s~^.*#~$(shell echo $(TARGET_GENERIC_GETTY) | tail -c+4)::respawn:/sbin/getty -L $(TARGET_GENERIC_GETTY) $(TARGET_GENERIC_GETTY_BAUDRATE) vt100 #~' \
>  		$(TARGET_DIR)/etc/inittab
>  
> +# Put a getty on virtual terminal
> +target-generic-do-getty-tty1:
> +	$(SED) 's/#tty1/tty1/' $(TARGET_DIR)/etc/inittab
> +
> +# Don't put a getty on virtual terminal
> +target-generic-dont-getty-tty1:
> +	$(SED) 's/tty1/#tty1/' $(TARGET_DIR)/etc/inittab

This might cause unintended changes in /etc/inittab, since the 'tty1' string 
appears in more places. Add '# GENERIC_VT' as above, and only change matching 
lines. See how GENERIC_SERIAL is being used

baruch

-- 
                                                     ~. .~   Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
   - baruch at tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il -

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

* [Buildroot] [PATCH] add support for virtual terminal
  2011-11-07 10:32 ` Baruch Siach
@ 2011-11-07 11:48   ` Yegor Yefremov
  0 siblings, 0 replies; 3+ messages in thread
From: Yegor Yefremov @ 2011-11-07 11:48 UTC (permalink / raw)
  To: buildroot

Hi Baruch,

> On Mon, Nov 07, 2011 at 11:21:29AM +0100, Yegor Yefremov wrote:
>> introduces new option "Enable virtual console on tty1"
>> to enable virtual terminal on tty1
>>
>> Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com>
>> ---
> [snip]
>
>> +# Put a getty on a virtual terminal
>> +#tty1::respawn:/sbin/getty tty1 38400
> You should probably add '# GENERIC_VT' in this line. See below.
>
>> +
>>  # Put a getty on the serial port
>>  #ttyS0::respawn:/sbin/getty -L ttyS0 115200 vt100 # GENERIC_SERIAL
> [snip]
>
>> Index: b/target/generic/Makefile.in
>> ===================================================================
>> --- a/target/generic/Makefile.in
>> +++ b/target/generic/Makefile.in
>> @@ -23,6 +23,14 @@
>>  	$(SED) '/# GENERIC_SERIAL$$/s~^.*#~$(shell echo $(TARGET_GENERIC_GETTY) | tail -c+4)::respawn:/sbin/getty -L $(TARGET_GENERIC_GETTY) $(TARGET_GENERIC_GETTY_BAUDRATE) vt100 #~' \
>>  		$(TARGET_DIR)/etc/inittab
>>  
>> +# Put a getty on virtual terminal
>> +target-generic-do-getty-tty1:
>> +	$(SED) 's/#tty1/tty1/' $(TARGET_DIR)/etc/inittab
>> +
>> +# Don't put a getty on virtual terminal
>> +target-generic-dont-getty-tty1:
>> +	$(SED) 's/tty1/#tty1/' $(TARGET_DIR)/etc/inittab
> This might cause unintended changes in /etc/inittab, since the 'tty1' string 
> appears in more places. Add '# GENERIC_VT' as above, and only change matching 
> lines. See how GENERIC_SERIAL is being used

Thanks. Changes implemented.

Yegor

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

end of thread, other threads:[~2011-11-07 11:48 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-11-07 10:21 [Buildroot] [PATCH] add support for virtual terminal Yegor Yefremov
2011-11-07 10:32 ` Baruch Siach
2011-11-07 11:48   ` Yegor Yefremov

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox