All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] sysvinit_2.86.bb: add possibility to use USB Serial Console with only rx/tx.
@ 2010-11-26 16:54 matteo.facchinetti
  2010-11-26 17:23 ` Koen Kooi
  0 siblings, 1 reply; 5+ messages in thread
From: matteo.facchinetti @ 2010-11-26 16:54 UTC (permalink / raw)
  To: openembedded-devel; +Cc: Matteo Facchinetti

From: Matteo Facchinetti <engineering@sirius-es.it>

There are many evaluationboard that use mini-usb port like as
a serial port + power supply. Using it as serial console have
to force getty with no need for carrier detect.

Signed-off-by: Matteo Facchinetti <engineering@sirius-es.it>
---
 recipes/sysvinit/sysvinit_2.86.bb |    5 +++++
 1 files changed, 5 insertions(+), 0 deletions(-)

diff --git a/recipes/sysvinit/sysvinit_2.86.bb b/recipes/sysvinit/sysvinit_2.86.bb
index 6ab5290..c34cbc9 100644
--- a/recipes/sysvinit/sysvinit_2.86.bb
+++ b/recipes/sysvinit/sysvinit_2.86.bb
@@ -65,6 +65,11 @@ do_install () {
 	install -m 0644 ${WORKDIR}/inittab ${D}${sysconfdir}/inittab
 	if [ ! -z "${SERIAL_CONSOLE}" ]; then
 		echo "S:2345:respawn:${base_sbindir}/getty ${SERIAL_CONSOLE}" >> ${D}${sysconfdir}/inittab
+	else
+		 
+		if [ ! -z "${USB_SERIAL_CONSOLE}" ]; then
+			echo "S:2345:respawn:${base_sbindir}/getty -L ${USB_SERIAL_CONSOLE}" >> ${D}${sysconfdir}/inittab
+		fi
 	fi
 	if [ "${USE_VT}" = "1" ]; then
 		cat <<EOF >>${D}${sysconfdir}/inittab
-- 
1.7.0.4




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

* Re: [PATCH] sysvinit_2.86.bb: add possibility to use USB Serial Console with only rx/tx.
  2010-11-26 16:54 [PATCH] sysvinit_2.86.bb: add possibility to use USB Serial Console with only rx/tx matteo.facchinetti
@ 2010-11-26 17:23 ` Koen Kooi
  2010-12-02 13:38   ` Matteo Facchinetti
  0 siblings, 1 reply; 5+ messages in thread
From: Koen Kooi @ 2010-11-26 17:23 UTC (permalink / raw)
  To: openembedded-devel

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Where's the patch that adds this new var to the usermanual?

On 26-11-10 17:54, matteo.facchinetti@sirius-es.it wrote:
> From: Matteo Facchinetti <engineering@sirius-es.it>
> 
> There are many evaluationboard that use mini-usb port like as
> a serial port + power supply. Using it as serial console have
> to force getty with no need for carrier detect.
> 
> Signed-off-by: Matteo Facchinetti <engineering@sirius-es.it>
> ---
>  recipes/sysvinit/sysvinit_2.86.bb |    5 +++++
>  1 files changed, 5 insertions(+), 0 deletions(-)
> 
> diff --git a/recipes/sysvinit/sysvinit_2.86.bb b/recipes/sysvinit/sysvinit_2.86.bb
> index 6ab5290..c34cbc9 100644
> --- a/recipes/sysvinit/sysvinit_2.86.bb
> +++ b/recipes/sysvinit/sysvinit_2.86.bb
> @@ -65,6 +65,11 @@ do_install () {
>  	install -m 0644 ${WORKDIR}/inittab ${D}${sysconfdir}/inittab
>  	if [ ! -z "${SERIAL_CONSOLE}" ]; then
>  		echo "S:2345:respawn:${base_sbindir}/getty ${SERIAL_CONSOLE}" >> ${D}${sysconfdir}/inittab
> +	else
> +		 
> +		if [ ! -z "${USB_SERIAL_CONSOLE}" ]; then
> +			echo "S:2345:respawn:${base_sbindir}/getty -L ${USB_SERIAL_CONSOLE}" >> ${D}${sysconfdir}/inittab
> +		fi
>  	fi
>  	if [ "${USE_VT}" = "1" ]; then
>  		cat <<EOF >>${D}${sysconfdir}/inittab

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.5 (Darwin)

iD8DBQFM7+0kMkyGM64RGpERAtPZAKCSHvZA9Eko51AqkoVR476+5tWoIQCgr1/Z
kxtr+B+5lfr+vqIxh6o5qGA=
=JSd7
-----END PGP SIGNATURE-----




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

* Re: [PATCH] sysvinit_2.86.bb: add possibility to use USB Serial Console with only rx/tx.
  2010-11-26 17:23 ` Koen Kooi
@ 2010-12-02 13:38   ` Matteo Facchinetti
  2010-12-05 13:17     ` Scott Anderson
  0 siblings, 1 reply; 5+ messages in thread
From: Matteo Facchinetti @ 2010-12-02 13:38 UTC (permalink / raw)
  To: openembedded-devel

USB_SERIAL_CONSOLE is similar to SERIAL_CONSOLE but SERIAL_CONSOLE 
variabile isn't described too.

IMHO could be need to add a new paragraph describing all kind of CONSOLE 
available for machine conf file.

Where's the right place to add these variabiles in usermanual?



On 11/26/2010 06:23 PM, Koen Kooi wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> Where's the patch that adds this new var to the usermanual?
>
> On 26-11-10 17:54, matteo.facchinetti@sirius-es.it wrote:
>    
>> From: Matteo Facchinetti<engineering@sirius-es.it>
>>
>> There are many evaluationboard that use mini-usb port like as
>> a serial port + power supply. Using it as serial console have
>> to force getty with no need for carrier detect.
>>
>> Signed-off-by: Matteo Facchinetti<engineering@sirius-es.it>
>> ---
>>   recipes/sysvinit/sysvinit_2.86.bb |    5 +++++
>>   1 files changed, 5 insertions(+), 0 deletions(-)
>>
>> diff --git a/recipes/sysvinit/sysvinit_2.86.bb b/recipes/sysvinit/sysvinit_2.86.bb
>> index 6ab5290..c34cbc9 100644
>> --- a/recipes/sysvinit/sysvinit_2.86.bb
>> +++ b/recipes/sysvinit/sysvinit_2.86.bb
>> @@ -65,6 +65,11 @@ do_install () {
>>   	install -m 0644 ${WORKDIR}/inittab ${D}${sysconfdir}/inittab
>>   	if [ ! -z "${SERIAL_CONSOLE}" ]; then
>>   		echo "S:2345:respawn:${base_sbindir}/getty ${SERIAL_CONSOLE}">>  ${D}${sysconfdir}/inittab
>> +	else
>> +		
>> +		if [ ! -z "${USB_SERIAL_CONSOLE}" ]; then
>> +			echo "S:2345:respawn:${base_sbindir}/getty -L ${USB_SERIAL_CONSOLE}">>  ${D}${sysconfdir}/inittab
>> +		fi
>>   	fi
>>   	if [ "${USE_VT}" = "1" ]; then
>>   		cat<<EOF>>${D}${sysconfdir}/inittab
>>      
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.5 (Darwin)
>
> iD8DBQFM7+0kMkyGM64RGpERAtPZAKCSHvZA9Eko51AqkoVR476+5tWoIQCgr1/Z
> kxtr+B+5lfr+vqIxh6o5qGA=
> =JSd7
> -----END PGP SIGNATURE-----
>
>
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
>
>    




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

* Re: [PATCH] sysvinit_2.86.bb: add possibility to use USB Serial Console with only rx/tx.
  2010-12-02 13:38   ` Matteo Facchinetti
@ 2010-12-05 13:17     ` Scott Anderson
  2010-12-06  8:53       ` Matteo Facchinetti
  0 siblings, 1 reply; 5+ messages in thread
From: Scott Anderson @ 2010-12-05 13:17 UTC (permalink / raw)
  To: openembedded-devel

Hi,

I apologize for coming into this late, but could you give a quick explanation of why a
new variable is needed?  In other words, what is wrong with something like:
SERIAL_CONSOLE="-L /dev/ttyUSB0 115200"

  SAA

On Dec 2, 2010, at 5:38 AM, Matteo Facchinetti wrote:

> USB_SERIAL_CONSOLE is similar to SERIAL_CONSOLE but SERIAL_CONSOLE variabile isn't described too.
> 
> IMHO could be need to add a new paragraph describing all kind of CONSOLE available for machine conf file.
> 
> Where's the right place to add these variabiles in usermanual?
> 
> 
> 
> On 11/26/2010 06:23 PM, Koen Kooi wrote:
>> -----BEGIN PGP SIGNED MESSAGE-----
>> Hash: SHA1
>> 
>> Where's the patch that adds this new var to the usermanual?
>> 
>> On 26-11-10 17:54, matteo.facchinetti@sirius-es.it wrote:
>>   
>>> From: Matteo Facchinetti<engineering@sirius-es.it>
>>> 
>>> There are many evaluationboard that use mini-usb port like as
>>> a serial port + power supply. Using it as serial console have
>>> to force getty with no need for carrier detect.
>>> 
>>> Signed-off-by: Matteo Facchinetti<engineering@sirius-es.it>
>>> ---
>>>  recipes/sysvinit/sysvinit_2.86.bb |    5 +++++
>>>  1 files changed, 5 insertions(+), 0 deletions(-)
>>> 
>>> diff --git a/recipes/sysvinit/sysvinit_2.86.bb b/recipes/sysvinit/sysvinit_2.86.bb
>>> index 6ab5290..c34cbc9 100644
>>> --- a/recipes/sysvinit/sysvinit_2.86.bb
>>> +++ b/recipes/sysvinit/sysvinit_2.86.bb
>>> @@ -65,6 +65,11 @@ do_install () {
>>>  	install -m 0644 ${WORKDIR}/inittab ${D}${sysconfdir}/inittab
>>>  	if [ ! -z "${SERIAL_CONSOLE}" ]; then
>>>  		echo "S:2345:respawn:${base_sbindir}/getty ${SERIAL_CONSOLE}">>  ${D}${sysconfdir}/inittab
>>> +	else
>>> +		
>>> +		if [ ! -z "${USB_SERIAL_CONSOLE}" ]; then
>>> +			echo "S:2345:respawn:${base_sbindir}/getty -L ${USB_SERIAL_CONSOLE}">>  ${D}${sysconfdir}/inittab
>>> +		fi
>>>  	fi
>>>  	if [ "${USE_VT}" = "1" ]; then
>>>  		cat<<EOF>>${D}${sysconfdir}/inittab
>>>     
>> -----BEGIN PGP SIGNATURE-----
>> Version: GnuPG v1.4.5 (Darwin)
>> 
>> iD8DBQFM7+0kMkyGM64RGpERAtPZAKCSHvZA9Eko51AqkoVR476+5tWoIQCgr1/Z
>> kxtr+B+5lfr+vqIxh6o5qGA=
>> =JSd7
>> -----END PGP SIGNATURE-----
>> 
>> 
>> _______________________________________________
>> Openembedded-devel mailing list
>> Openembedded-devel@lists.openembedded.org
>> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
>> 
>>   
> 
> 
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
> 




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

* Re: [PATCH] sysvinit_2.86.bb: add possibility to use USB Serial Console with only rx/tx.
  2010-12-05 13:17     ` Scott Anderson
@ 2010-12-06  8:53       ` Matteo Facchinetti
  0 siblings, 0 replies; 5+ messages in thread
From: Matteo Facchinetti @ 2010-12-06  8:53 UTC (permalink / raw)
  To: openembedded-devel

Hi,

Yes, it's true.
In practice there is no difference using SERIAL_CONSOLE with -L 
parameter or USB_SERIAL_CONSOLE.
I have introduced this new variabile only for emphasize that physically 
is different and works in a different way (with -L param).
When make a new conf you could save a lot of time viewing in code that 
is possible to use getty with a suitable configuration for your usb 
console only adding -L.

Otherwise, unless new var adding, I can use SERIAL_CONSOLE with -L  
putting a comment in my machine/config file explaining that. And in a 
second time we can see if is correct to introduce a new var for this scope.

Regards,
Matteo Facchinetti


On 12/05/2010 02:17 PM, Scott Anderson wrote:
> Hi,
>
> I apologize for coming into this late, but could you give a quick explanation of why a
> new variable is needed?  In other words, what is wrong with something like:
> SERIAL_CONSOLE="-L /dev/ttyUSB0 115200"
>
>    SAA
>
> On Dec 2, 2010, at 5:38 AM, Matteo Facchinetti wrote:
>
>    
>> USB_SERIAL_CONSOLE is similar to SERIAL_CONSOLE but SERIAL_CONSOLE variabile isn't described too.
>>
>> IMHO could be need to add a new paragraph describing all kind of CONSOLE available for machine conf file.
>>
>> Where's the right place to add these variabiles in usermanual?
>>
>>
>>
>> On 11/26/2010 06:23 PM, Koen Kooi wrote:
>>      
>>> -----BEGIN PGP SIGNED MESSAGE-----
>>> Hash: SHA1
>>>
>>> Where's the patch that adds this new var to the usermanual?
>>>
>>> On 26-11-10 17:54, matteo.facchinetti@sirius-es.it wrote:
>>>
>>>        
>>>> From: Matteo Facchinetti<engineering@sirius-es.it>
>>>>
>>>> There are many evaluationboard that use mini-usb port like as
>>>> a serial port + power supply. Using it as serial console have
>>>> to force getty with no need for carrier detect.
>>>>
>>>> Signed-off-by: Matteo Facchinetti<engineering@sirius-es.it>
>>>> ---
>>>>   recipes/sysvinit/sysvinit_2.86.bb |    5 +++++
>>>>   1 files changed, 5 insertions(+), 0 deletions(-)
>>>>
>>>> diff --git a/recipes/sysvinit/sysvinit_2.86.bb b/recipes/sysvinit/sysvinit_2.86.bb
>>>> index 6ab5290..c34cbc9 100644
>>>> --- a/recipes/sysvinit/sysvinit_2.86.bb
>>>> +++ b/recipes/sysvinit/sysvinit_2.86.bb
>>>> @@ -65,6 +65,11 @@ do_install () {
>>>>   	install -m 0644 ${WORKDIR}/inittab ${D}${sysconfdir}/inittab
>>>>   	if [ ! -z "${SERIAL_CONSOLE}" ]; then
>>>>   		echo "S:2345:respawn:${base_sbindir}/getty ${SERIAL_CONSOLE}">>   ${D}${sysconfdir}/inittab
>>>> +	else
>>>> +		
>>>> +		if [ ! -z "${USB_SERIAL_CONSOLE}" ]; then
>>>> +			echo "S:2345:respawn:${base_sbindir}/getty -L ${USB_SERIAL_CONSOLE}">>   ${D}${sysconfdir}/inittab
>>>> +		fi
>>>>   	fi
>>>>   	if [ "${USE_VT}" = "1" ]; then
>>>>   		cat<<EOF>>${D}${sysconfdir}/inittab
>>>>
>>>>          
>>> -----BEGIN PGP SIGNATURE-----
>>> Version: GnuPG v1.4.5 (Darwin)
>>>
>>> iD8DBQFM7+0kMkyGM64RGpERAtPZAKCSHvZA9Eko51AqkoVR476+5tWoIQCgr1/Z
>>> kxtr+B+5lfr+vqIxh6o5qGA=
>>> =JSd7
>>> -----END PGP SIGNATURE-----
>>>
>>>
>>> _______________________________________________
>>> Openembedded-devel mailing list
>>> Openembedded-devel@lists.openembedded.org
>>> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
>>>
>>>
>>>        
>>
>> _______________________________________________
>> Openembedded-devel mailing list
>> Openembedded-devel@lists.openembedded.org
>> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
>>
>>      
>
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
>
>    




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

end of thread, other threads:[~2010-12-06  8:54 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-11-26 16:54 [PATCH] sysvinit_2.86.bb: add possibility to use USB Serial Console with only rx/tx matteo.facchinetti
2010-11-26 17:23 ` Koen Kooi
2010-12-02 13:38   ` Matteo Facchinetti
2010-12-05 13:17     ` Scott Anderson
2010-12-06  8:53       ` Matteo Facchinetti

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.