[yocto] thud, beaglebone-yocto.conf: SERIAL_CONSOLES setting

[Heiko Schocher]

Hello Kevin, Robert,

Am 06.11.2018 um 09:10 schrieb Heiko Schocher:
> Hello Kevin, Robert,
> 
> Am 05.11.2018 um 06:26 schrieb Kevin Hao:
>> On Sun, Nov 04, 2018 at 12:10:00PM +0200, Robert Berger wrote:
>>> Hi,
>>>
>>> On 02.11.18 16:27, Khem Raj wrote:
>>>>
>>>> omap serial is obsolete why does linux-yocto keeps using it.
>>>> seondly, machine config should enable both consoles ttyO0 and ttyS0 if
>>>> you know that at least one kernel is using ttyO0
>>>>
>>> How about picking whatever works for you in the kernel conf and in machine
>>> conf?
>>>
>>> SERIAL_CONSOLES = "115200;ttyS0 115200;ttyO0"
>>> SERIAL_CONSOLES_CHECK = "${SERIAL_CONSOLES}"
>>>
>>> Like this on the first boot either ttyO0 or ttyS0 should be picked
>>> automatically.
>>
>> Yes, this is doable. Would you mind send a patch?
> 
> Sorry for answering so late... good hint, I missed SERIAL_CONSOLES_CHECK
> 
> I try this change and report, give me some time...


diff --git a/meta-yocto-bsp/conf/machine/beaglebone-yocto.conf 
b/meta-yocto-bsp/conf/machine/beaglebone-yocto.conf
index e911e75004..def3a2ae06 100644
--- a/meta-yocto-bsp/conf/machine/beaglebone-yocto.conf
+++ b/meta-yocto-bsp/conf/machine/beaglebone-yocto.conf
@@ -20,7 +20,8 @@ WKS_FILE ?= "beaglebone-yocto.wks"
  IMAGE_INSTALL_append = " kernel-devicetree kernel-image-zimage"
  do_image_wic[depends] += "mtools-native:do_populate_sysroot dosfstools-native:do_populate_sysroot"

-SERIAL_CONSOLES = "115200;ttyO0"
+SERIAL_CONSOLES = "115200;ttyS0 115200;ttyO0"
+SERIAL_CONSOLES_CHECK = "${SERIAL_CONSOLES}"

  PREFERRED_PROVIDER_virtual/kernel ?= "linux-yocto"
  PREFERRED_VERSION_linux-yocto ?= "4.18%"

and on my beagleboneblack linux kernels with 8250 serial driver enabled
(console ttyS0) and kernel omap_serial driver enabled (console ttyO0)
are booting.

Unfortunately it took me some time, until I realized that my settings
in auto.conf do not work, because in beaglebone-yocto.conf

SERIAL_CONSOLES = "115200;ttyO0"

is set ... May a

SERIAL_CONSOLES ?= "115200;ttyS0 115200;ttyO0"

is friendlier ?

Should I send a formal patch?

bye,
Heiko
-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-52   Fax: +49-8142-66989-80   Email: hs at denx.de