Re: [PATCH] sysvinit_2.86.bb: add possibility to use USB Serial Console with only rx/tx.

[Matteo Facchinetti <matteo.facchinetti@sirius-es.it>]

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
>
>