[meta-ti][scarthgap/master][PATCH 1/3] wic: sdimage-2part-efi: Add rootwait kernel param

[meta-ti][scarthgap/master][PATCH 1/3] wic: sdimage-2part-efi: Add rootwait kernel param

[Andrew Davis]

This is important for many boot media such as MMC/SD as there may be a
delay in mounting rootfs which can cause boot failure if kernel does
not wait for the rootfs. Add this as one of the default params given
to the kernel. This is already the case for most U-Boot kernel cmd
lines, this adds the same when doing EFI boot.

Signed-off-by: Andrew Davis <afd@ti.com>
---
 meta-ti-bsp/wic/sdimage-2part-efi.wks.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta-ti-bsp/wic/sdimage-2part-efi.wks.in b/meta-ti-bsp/wic/sdimage-2part-efi.wks.in
index ae38daa7..7252411c 100644
--- a/meta-ti-bsp/wic/sdimage-2part-efi.wks.in
+++ b/meta-ti-bsp/wic/sdimage-2part-efi.wks.in
@@ -3,6 +3,6 @@
 # supports EFI.  Boot files are located in the first vfat partition with extra
 # reserved space.  We cannot use a GPT here.
 
-bootloader --timeout=3 --append="rootfstype=ext4 ${TI_WKS_BOOTLOADER_APPEND}"
+bootloader --timeout=3 --append="rootfstype=ext4 rootwait ${TI_WKS_BOOTLOADER_APPEND}"
 part --source bootimg-efi --sourceparams="loader=${EFI_PROVIDER}" --fstype=vfat --label boot --active --align 1024 --use-uuid --fixed-size 128M
 part / --source rootfs --fstype=ext4 --label root --align 1024 --use-uuid
-- 
2.39.2

[meta-ti][scarthgap/master][PATCH 2/3] wic: sdimage-2part-efi: Add console kernel param

[Andrew Davis]

This is already set for AM64x as that platform requires it, it is valid
for all K3 platforms. Like AM64x other platforms might drop this from
the DTB making it required for them also.

Signed-off-by: Andrew Davis <afd@ti.com>
---
 meta-ti-bsp/conf/machine/include/am64xx.inc | 2 --
 meta-ti-bsp/conf/machine/include/k3.inc     | 2 +-
 2 files changed, 1 insertion(+), 3 deletions(-)

diff --git a/meta-ti-bsp/conf/machine/include/am64xx.inc b/meta-ti-bsp/conf/machine/include/am64xx.inc
index 56b56afe..49e996c8 100644
--- a/meta-ti-bsp/conf/machine/include/am64xx.inc
+++ b/meta-ti-bsp/conf/machine/include/am64xx.inc
@@ -34,5 +34,3 @@ TFA_BOARD = "lite"
 OPTEEMACHINE = "k3-am64x"
 
 MACHINE_ESSENTIAL_EXTRA_RRECOMMENDS += "prueth-fw pruhsr-fw prusw-fw"
-
-TI_WKS_BOOTLOADER_APPEND="console=ttyS2,115200n8"
diff --git a/meta-ti-bsp/conf/machine/include/k3.inc b/meta-ti-bsp/conf/machine/include/k3.inc
index 7ec6b51e..572eb1cc 100644
--- a/meta-ti-bsp/conf/machine/include/k3.inc
+++ b/meta-ti-bsp/conf/machine/include/k3.inc
@@ -51,7 +51,7 @@ MACHINE_FEATURES += "efi"
 
 WKS_FILE ?= "${@bb.utils.contains("MACHINE_FEATURES", "efi", "sdimage-2part-efi.wks.in", "sdimage-2part.wks", d)}"
 
-TI_WKS_BOOTLOADER_APPEND ?= ""
+TI_WKS_BOOTLOADER_APPEND ?= "console=ttyS2,115200n8"
 
 do_image_wic[depends] += "virtual/bootloader:do_deploy"
 
-- 
2.39.2

[meta-ti][scarthgap/master][PATCH 3/3] wic: sdimage-2part-efi: Remove 3 second menu delay

[Andrew Davis]

For all our platforms booting with EFI there is only one option today.
The timeout gives the user 3 seconds to choose a different option, but
as there is only one, this just causes a needless 3 second delay in
boot time. Remove this delay and boot the only option immediately.
If we ever have more than one option we can always add this back.

Signed-off-by: Andrew Davis <afd@ti.com>
---
 meta-ti-bsp/wic/sdimage-2part-efi.wks.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta-ti-bsp/wic/sdimage-2part-efi.wks.in b/meta-ti-bsp/wic/sdimage-2part-efi.wks.in
index 7252411c..c1089762 100644
--- a/meta-ti-bsp/wic/sdimage-2part-efi.wks.in
+++ b/meta-ti-bsp/wic/sdimage-2part-efi.wks.in
@@ -3,6 +3,6 @@
 # supports EFI.  Boot files are located in the first vfat partition with extra
 # reserved space.  We cannot use a GPT here.
 
-bootloader --timeout=3 --append="rootfstype=ext4 rootwait ${TI_WKS_BOOTLOADER_APPEND}"
+bootloader --timeout=0 --append="rootfstype=ext4 rootwait ${TI_WKS_BOOTLOADER_APPEND}"
 part --source bootimg-efi --sourceparams="loader=${EFI_PROVIDER}" --fstype=vfat --label boot --active --align 1024 --use-uuid --fixed-size 128M
 part / --source rootfs --fstype=ext4 --label root --align 1024 --use-uuid
-- 
2.39.2

Re: [meta-ti][scarthgap/master][PATCH 3/3] wic: sdimage-2part-efi: Remove 3 second menu delay

[Denys Dmytriyenko]

On Mon, Jul 28, 2025 at 11:54:37AM -0500, Andrew Davis via lists.yoctoproject.org wrote:
> For all our platforms booting with EFI there is only one option today.
> The timeout gives the user 3 seconds to choose a different option, but
> as there is only one, this just causes a needless 3 second delay in
> boot time. Remove this delay and boot the only option immediately.
> If we ever have more than one option we can always add this back.

Well, menu delay in grub is not only used for selecting among available 
predefined options, but also for interactively editing the command line 
on the fly before booting.


> Signed-off-by: Andrew Davis <afd@ti.com>
> ---
>  meta-ti-bsp/wic/sdimage-2part-efi.wks.in | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/meta-ti-bsp/wic/sdimage-2part-efi.wks.in b/meta-ti-bsp/wic/sdimage-2part-efi.wks.in
> index 7252411c..c1089762 100644
> --- a/meta-ti-bsp/wic/sdimage-2part-efi.wks.in
> +++ b/meta-ti-bsp/wic/sdimage-2part-efi.wks.in
> @@ -3,6 +3,6 @@
>  # supports EFI.  Boot files are located in the first vfat partition with extra
>  # reserved space.  We cannot use a GPT here.
>  
> -bootloader --timeout=3 --append="rootfstype=ext4 rootwait ${TI_WKS_BOOTLOADER_APPEND}"
> +bootloader --timeout=0 --append="rootfstype=ext4 rootwait ${TI_WKS_BOOTLOADER_APPEND}"
>  part --source bootimg-efi --sourceparams="loader=${EFI_PROVIDER}" --fstype=vfat --label boot --active --align 1024 --use-uuid --fixed-size 128M
>  part / --source rootfs --fstype=ext4 --label root --align 1024 --use-uuid
> -- 
> 2.39.2

Re: [meta-ti][scarthgap/master][PATCH 3/3] wic: sdimage-2part-efi: Remove 3 second menu delay

[Khem Raj]

On Mon, Jul 28, 2025 at 11:25 AM Denys Dmytriyenko via
lists.yoctoproject.org <denis=denix.org@lists.yoctoproject.org> wrote:
>
> On Mon, Jul 28, 2025 at 11:54:37AM -0500, Andrew Davis via lists.yoctoproject.org wrote:
> > For all our platforms booting with EFI there is only one option today.
> > The timeout gives the user 3 seconds to choose a different option, but
> > as there is only one, this just causes a needless 3 second delay in
> > boot time. Remove this delay and boot the only option immediately.
> > If we ever have more than one option we can always add this back.
>
> Well, menu delay in grub is not only used for selecting among available
> predefined options, but also for interactively editing the command line
> on the fly before booting.
>

for development, this sounds good, for production not so much.

>
> > Signed-off-by: Andrew Davis <afd@ti.com>
> > ---
> >  meta-ti-bsp/wic/sdimage-2part-efi.wks.in | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/meta-ti-bsp/wic/sdimage-2part-efi.wks.in b/meta-ti-bsp/wic/sdimage-2part-efi.wks.in
> > index 7252411c..c1089762 100644
> > --- a/meta-ti-bsp/wic/sdimage-2part-efi.wks.in
> > +++ b/meta-ti-bsp/wic/sdimage-2part-efi.wks.in
> > @@ -3,6 +3,6 @@
> >  # supports EFI.  Boot files are located in the first vfat partition with extra
> >  # reserved space.  We cannot use a GPT here.
> >
> > -bootloader --timeout=3 --append="rootfstype=ext4 rootwait ${TI_WKS_BOOTLOADER_APPEND}"
> > +bootloader --timeout=0 --append="rootfstype=ext4 rootwait ${TI_WKS_BOOTLOADER_APPEND}"
> >  part --source bootimg-efi --sourceparams="loader=${EFI_PROVIDER}" --fstype=vfat --label boot --active --align 1024 --use-uuid --fixed-size 128M
> >  part / --source rootfs --fstype=ext4 --label root --align 1024 --use-uuid
> > --
> > 2.39.2
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#18840): https://lists.yoctoproject.org/g/meta-ti/message/18840
> Mute This Topic: https://lists.yoctoproject.org/mt/114416911/1997914
> Group Owner: meta-ti+owner@lists.yoctoproject.org
> Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub [raj.khem@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>

Re: [meta-ti][scarthgap/master][PATCH 3/3] wic: sdimage-2part-efi: Remove 3 second menu delay

[Denys Dmytriyenko]

On Mon, Jul 28, 2025 at 11:28:36AM -0700, Khem Raj wrote:
> On Mon, Jul 28, 2025 at 11:25 AM Denys Dmytriyenko via
> lists.yoctoproject.org <denis=denix.org@lists.yoctoproject.org> wrote:
> >
> > On Mon, Jul 28, 2025 at 11:54:37AM -0500, Andrew Davis via lists.yoctoproject.org wrote:
> > > For all our platforms booting with EFI there is only one option today.
> > > The timeout gives the user 3 seconds to choose a different option, but
> > > as there is only one, this just causes a needless 3 second delay in
> > > boot time. Remove this delay and boot the only option immediately.
> > > If we ever have more than one option we can always add this back.
> >
> > Well, menu delay in grub is not only used for selecting among available
> > predefined options, but also for interactively editing the command line
> > on the fly before booting.
> >
> 
> for development, this sounds good, for production not so much.

Sure. And we also disable e.g. U-boot boot prompt delay all the time for our 
customers during production hardening stage, among many other similar items. 
But it is done for the final product - I don't think it should be done in a 
generic BSP layer that many use for development.


> > > Signed-off-by: Andrew Davis <afd@ti.com>
> > > ---
> > >  meta-ti-bsp/wic/sdimage-2part-efi.wks.in | 2 +-
> > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git a/meta-ti-bsp/wic/sdimage-2part-efi.wks.in b/meta-ti-bsp/wic/sdimage-2part-efi.wks.in
> > > index 7252411c..c1089762 100644
> > > --- a/meta-ti-bsp/wic/sdimage-2part-efi.wks.in
> > > +++ b/meta-ti-bsp/wic/sdimage-2part-efi.wks.in
> > > @@ -3,6 +3,6 @@
> > >  # supports EFI.  Boot files are located in the first vfat partition with extra
> > >  # reserved space.  We cannot use a GPT here.
> > >
> > > -bootloader --timeout=3 --append="rootfstype=ext4 rootwait ${TI_WKS_BOOTLOADER_APPEND}"
> > > +bootloader --timeout=0 --append="rootfstype=ext4 rootwait ${TI_WKS_BOOTLOADER_APPEND}"
> > >  part --source bootimg-efi --sourceparams="loader=${EFI_PROVIDER}" --fstype=vfat --label boot --active --align 1024 --use-uuid --fixed-size 128M
> > >  part / --source rootfs --fstype=ext4 --label root --align 1024 --use-uuid
> > > --
> > > 2.39.2

Re: [meta-ti][scarthgap/master][PATCH 3/3] wic: sdimage-2part-efi: Remove 3 second menu delay

[Khem Raj]

On Mon, Jul 28, 2025 at 12:45 PM Denys Dmytriyenko <denis@denix.org> wrote:
>
> On Mon, Jul 28, 2025 at 11:28:36AM -0700, Khem Raj wrote:
> > On Mon, Jul 28, 2025 at 11:25 AM Denys Dmytriyenko via
> > lists.yoctoproject.org <denis=denix.org@lists.yoctoproject.org> wrote:
> > >
> > > On Mon, Jul 28, 2025 at 11:54:37AM -0500, Andrew Davis via lists.yoctoproject.org wrote:
> > > > For all our platforms booting with EFI there is only one option today.
> > > > The timeout gives the user 3 seconds to choose a different option, but
> > > > as there is only one, this just causes a needless 3 second delay in
> > > > boot time. Remove this delay and boot the only option immediately.
> > > > If we ever have more than one option we can always add this back.
> > >
> > > Well, menu delay in grub is not only used for selecting among available
> > > predefined options, but also for interactively editing the command line
> > > on the fly before booting.
> > >
> >
> > for development, this sounds good, for production not so much.
>
> Sure. And we also disable e.g. U-boot boot prompt delay all the time for our
> customers during production hardening stage, among many other similar items.
> But it is done for the final product - I don't think it should be done in a
> generic BSP layer that many use for development.

Right, it could be documented somewhere in README or some such would help
the consumers of the layer.

>
>
> > > > Signed-off-by: Andrew Davis <afd@ti.com>
> > > > ---
> > > >  meta-ti-bsp/wic/sdimage-2part-efi.wks.in | 2 +-
> > > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > > >
> > > > diff --git a/meta-ti-bsp/wic/sdimage-2part-efi.wks.in b/meta-ti-bsp/wic/sdimage-2part-efi.wks.in
> > > > index 7252411c..c1089762 100644
> > > > --- a/meta-ti-bsp/wic/sdimage-2part-efi.wks.in
> > > > +++ b/meta-ti-bsp/wic/sdimage-2part-efi.wks.in
> > > > @@ -3,6 +3,6 @@
> > > >  # supports EFI.  Boot files are located in the first vfat partition with extra
> > > >  # reserved space.  We cannot use a GPT here.
> > > >
> > > > -bootloader --timeout=3 --append="rootfstype=ext4 rootwait ${TI_WKS_BOOTLOADER_APPEND}"
> > > > +bootloader --timeout=0 --append="rootfstype=ext4 rootwait ${TI_WKS_BOOTLOADER_APPEND}"
> > > >  part --source bootimg-efi --sourceparams="loader=${EFI_PROVIDER}" --fstype=vfat --label boot --active --align 1024 --use-uuid --fixed-size 128M
> > > >  part / --source rootfs --fstype=ext4 --label root --align 1024 --use-uuid
> > > > --
> > > > 2.39.2

Re: [meta-ti][scarthgap/master][PATCH 3/3] wic: sdimage-2part-efi: Remove 3 second menu delay

[Ryan Eatmon]

On 7/28/2025 3:20 PM, Khem Raj via lists.yoctoproject.org wrote:
> On Mon, Jul 28, 2025 at 12:45 PM Denys Dmytriyenko <denis@denix.org> wrote:
>>
>> On Mon, Jul 28, 2025 at 11:28:36AM -0700, Khem Raj wrote:
>>> On Mon, Jul 28, 2025 at 11:25 AM Denys Dmytriyenko via
>>> lists.yoctoproject.org <denis=denix.org@lists.yoctoproject.org> wrote:
>>>>
>>>> On Mon, Jul 28, 2025 at 11:54:37AM -0500, Andrew Davis via lists.yoctoproject.org wrote:
>>>>> For all our platforms booting with EFI there is only one option today.
>>>>> The timeout gives the user 3 seconds to choose a different option, but
>>>>> as there is only one, this just causes a needless 3 second delay in
>>>>> boot time. Remove this delay and boot the only option immediately.
>>>>> If we ever have more than one option we can always add this back.
>>>>
>>>> Well, menu delay in grub is not only used for selecting among available
>>>> predefined options, but also for interactively editing the command line
>>>> on the fly before booting.
>>>>
>>>
>>> for development, this sounds good, for production not so much.
>>
>> Sure. And we also disable e.g. U-boot boot prompt delay all the time for our
>> customers during production hardening stage, among many other similar items.
>> But it is done for the final product - I don't think it should be done in a
>> generic BSP layer that many use for development.
> 
> Right, it could be documented somewhere in README or some such would help
> the consumers of the layer.

Is there already something in the system that we could use to drive a 
variable that we can stick into the file?  Like EXTRA_IMAGE_FEATURES?  I 
know there used to be a debug-tweaks that got broken out into multiple 
features.  Maybe create a new one?

TI_WKS_TIMEOUT = '${@bb.utils.contains("IMAGE_FEATURES", 
"wks-debug-timeout", "3", "0 ",d)}'

bootloader --timeout=${TI_WKS_TIMEOUT} --append="rootfstype=ext4 
rootwait ${TI_WKS_BOOTLOADER_APPEND}"


Would that work?


>>
>>
>>>>> Signed-off-by: Andrew Davis <afd@ti.com>
>>>>> ---
>>>>>   meta-ti-bsp/wic/sdimage-2part-efi.wks.in | 2 +-
>>>>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>>>>
>>>>> diff --git a/meta-ti-bsp/wic/sdimage-2part-efi.wks.in b/meta-ti-bsp/wic/sdimage-2part-efi.wks.in
>>>>> index 7252411c..c1089762 100644
>>>>> --- a/meta-ti-bsp/wic/sdimage-2part-efi.wks.in
>>>>> +++ b/meta-ti-bsp/wic/sdimage-2part-efi.wks.in
>>>>> @@ -3,6 +3,6 @@
>>>>>   # supports EFI.  Boot files are located in the first vfat partition with extra
>>>>>   # reserved space.  We cannot use a GPT here.
>>>>>
>>>>> -bootloader --timeout=3 --append="rootfstype=ext4 rootwait ${TI_WKS_BOOTLOADER_APPEND}"
>>>>> +bootloader --timeout=0 --append="rootfstype=ext4 rootwait ${TI_WKS_BOOTLOADER_APPEND}"
>>>>>   part --source bootimg-efi --sourceparams="loader=${EFI_PROVIDER}" --fstype=vfat --label boot --active --align 1024 --use-uuid --fixed-size 128M
>>>>>   part / --source rootfs --fstype=ext4 --label root --align 1024 --use-uuid
>>>>> --
>>>>> 2.39.2
>>>>>
>>>>>
>>>>> -=-=-=-=-=-=-=-=-=-=-=-
>>>>> Links: You receive all messages sent to this group.
>>>>> View/Reply Online (#18848): https://lists.yoctoproject.org/g/meta-ti/message/18848
>>>>> Mute This Topic: https://lists.yoctoproject.org/mt/114416911/6551054
>>>>> Group Owner: meta-ti+owner@lists.yoctoproject.org
>>>>> Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub [reatmon@ti.com]
>>>>> -=-=-=-=-=-=-=-=-=-=-=-
>>>>>

-- 
Ryan Eatmon                reatmon@ti.com
-----------------------------------------
Texas Instruments, Inc.  -  LCPD  -  MGTS

Re: [meta-ti][scarthgap/master][PATCH 3/3] wic: sdimage-2part-efi: Remove 3 second menu delay

[Khem Raj]

On Mon, Jul 28, 2025 at 1:33 PM Ryan Eatmon <reatmon@ti.com> wrote:
>
>
>
> On 7/28/2025 3:20 PM, Khem Raj via lists.yoctoproject.org wrote:
> > On Mon, Jul 28, 2025 at 12:45 PM Denys Dmytriyenko <denis@denix.org> wrote:
> >>
> >> On Mon, Jul 28, 2025 at 11:28:36AM -0700, Khem Raj wrote:
> >>> On Mon, Jul 28, 2025 at 11:25 AM Denys Dmytriyenko via
> >>> lists.yoctoproject.org <denis=denix.org@lists.yoctoproject.org> wrote:
> >>>>
> >>>> On Mon, Jul 28, 2025 at 11:54:37AM -0500, Andrew Davis via lists.yoctoproject.org wrote:
> >>>>> For all our platforms booting with EFI there is only one option today.
> >>>>> The timeout gives the user 3 seconds to choose a different option, but
> >>>>> as there is only one, this just causes a needless 3 second delay in
> >>>>> boot time. Remove this delay and boot the only option immediately.
> >>>>> If we ever have more than one option we can always add this back.
> >>>>
> >>>> Well, menu delay in grub is not only used for selecting among available
> >>>> predefined options, but also for interactively editing the command line
> >>>> on the fly before booting.
> >>>>
> >>>
> >>> for development, this sounds good, for production not so much.
> >>
> >> Sure. And we also disable e.g. U-boot boot prompt delay all the time for our
> >> customers during production hardening stage, among many other similar items.
> >> But it is done for the final product - I don't think it should be done in a
> >> generic BSP layer that many use for development.
> >
> > Right, it could be documented somewhere in README or some such would help
> > the consumers of the layer.
>
> Is there already something in the system that we could use to drive a
> variable that we can stick into the file?  Like EXTRA_IMAGE_FEATURES?  I
> know there used to be a debug-tweaks that got broken out into multiple
> features.  Maybe create a new one?
>
> TI_WKS_TIMEOUT = '${@bb.utils.contains("IMAGE_FEATURES",
> "wks-debug-timeout", "3", "0 ",d)}'
>
> bootloader --timeout=${TI_WKS_TIMEOUT} --append="rootfstype=ext4
> rootwait ${TI_WKS_BOOTLOADER_APPEND}"
>
>
> Would that work?

probably, it can make it easier for sure.

>
>
> >>
> >>
> >>>>> Signed-off-by: Andrew Davis <afd@ti.com>
> >>>>> ---
> >>>>>   meta-ti-bsp/wic/sdimage-2part-efi.wks.in | 2 +-
> >>>>>   1 file changed, 1 insertion(+), 1 deletion(-)
> >>>>>
> >>>>> diff --git a/meta-ti-bsp/wic/sdimage-2part-efi.wks.in b/meta-ti-bsp/wic/sdimage-2part-efi.wks.in
> >>>>> index 7252411c..c1089762 100644
> >>>>> --- a/meta-ti-bsp/wic/sdimage-2part-efi.wks.in
> >>>>> +++ b/meta-ti-bsp/wic/sdimage-2part-efi.wks.in
> >>>>> @@ -3,6 +3,6 @@
> >>>>>   # supports EFI.  Boot files are located in the first vfat partition with extra
> >>>>>   # reserved space.  We cannot use a GPT here.
> >>>>>
> >>>>> -bootloader --timeout=3 --append="rootfstype=ext4 rootwait ${TI_WKS_BOOTLOADER_APPEND}"
> >>>>> +bootloader --timeout=0 --append="rootfstype=ext4 rootwait ${TI_WKS_BOOTLOADER_APPEND}"
> >>>>>   part --source bootimg-efi --sourceparams="loader=${EFI_PROVIDER}" --fstype=vfat --label boot --active --align 1024 --use-uuid --fixed-size 128M
> >>>>>   part / --source rootfs --fstype=ext4 --label root --align 1024 --use-uuid
> >>>>> --
> >>>>> 2.39.2
> >>>>>
> >>>>>
> >>>>> -=-=-=-=-=-=-=-=-=-=-=-
> >>>>> Links: You receive all messages sent to this group.
> >>>>> View/Reply Online (#18848): https://lists.yoctoproject.org/g/meta-ti/message/18848
> >>>>> Mute This Topic: https://lists.yoctoproject.org/mt/114416911/6551054
> >>>>> Group Owner: meta-ti+owner@lists.yoctoproject.org
> >>>>> Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub [reatmon@ti.com]
> >>>>> -=-=-=-=-=-=-=-=-=-=-=-
> >>>>>
>
> --
> Ryan Eatmon                reatmon@ti.com
> -----------------------------------------
> Texas Instruments, Inc.  -  LCPD  -  MGTS

Re: [meta-ti][scarthgap/master][PATCH 3/3] wic: sdimage-2part-efi: Remove 3 second menu delay

[Denys Dmytriyenko]

On Mon, Jul 28, 2025 at 03:33:23PM -0500, Ryan Eatmon wrote:
> 
> 
> On 7/28/2025 3:20 PM, Khem Raj via lists.yoctoproject.org wrote:
> >On Mon, Jul 28, 2025 at 12:45 PM Denys Dmytriyenko <denis@denix.org> wrote:
> >>
> >>On Mon, Jul 28, 2025 at 11:28:36AM -0700, Khem Raj wrote:
> >>>On Mon, Jul 28, 2025 at 11:25 AM Denys Dmytriyenko via
> >>>lists.yoctoproject.org <denis=denix.org@lists.yoctoproject.org> wrote:
> >>>>
> >>>>On Mon, Jul 28, 2025 at 11:54:37AM -0500, Andrew Davis via lists.yoctoproject.org wrote:
> >>>>>For all our platforms booting with EFI there is only one option today.
> >>>>>The timeout gives the user 3 seconds to choose a different option, but
> >>>>>as there is only one, this just causes a needless 3 second delay in
> >>>>>boot time. Remove this delay and boot the only option immediately.
> >>>>>If we ever have more than one option we can always add this back.
> >>>>
> >>>>Well, menu delay in grub is not only used for selecting among available
> >>>>predefined options, but also for interactively editing the command line
> >>>>on the fly before booting.
> >>>>
> >>>
> >>>for development, this sounds good, for production not so much.
> >>
> >>Sure. And we also disable e.g. U-boot boot prompt delay all the time for our
> >>customers during production hardening stage, among many other similar items.
> >>But it is done for the final product - I don't think it should be done in a
> >>generic BSP layer that many use for development.
> >
> >Right, it could be documented somewhere in README or some such would help
> >the consumers of the layer.
> 
> Is there already something in the system that we could use to drive
> a variable that we can stick into the file?  Like
> EXTRA_IMAGE_FEATURES?  I know there used to be a debug-tweaks that
> got broken out into multiple features.  Maybe create a new one?
> 
> TI_WKS_TIMEOUT = '${@bb.utils.contains("IMAGE_FEATURES",
> "wks-debug-timeout", "3", "0 ",d)}'

Should probably make it more generic, not wks. I'd imagine it can be shared 
for U-boot and EFI/GRUB, so maybe bootloader-timeout or something?


> bootloader --timeout=${TI_WKS_TIMEOUT} --append="rootfstype=ext4
> rootwait ${TI_WKS_BOOTLOADER_APPEND}"
> 
> 
> Would that work?
> 
> 
> >>
> >>
> >>>>>Signed-off-by: Andrew Davis <afd@ti.com>
> >>>>>---
> >>>>>  meta-ti-bsp/wic/sdimage-2part-efi.wks.in | 2 +-
> >>>>>  1 file changed, 1 insertion(+), 1 deletion(-)
> >>>>>
> >>>>>diff --git a/meta-ti-bsp/wic/sdimage-2part-efi.wks.in b/meta-ti-bsp/wic/sdimage-2part-efi.wks.in
> >>>>>index 7252411c..c1089762 100644
> >>>>>--- a/meta-ti-bsp/wic/sdimage-2part-efi.wks.in
> >>>>>+++ b/meta-ti-bsp/wic/sdimage-2part-efi.wks.in
> >>>>>@@ -3,6 +3,6 @@
> >>>>>  # supports EFI.  Boot files are located in the first vfat partition with extra
> >>>>>  # reserved space.  We cannot use a GPT here.
> >>>>>
> >>>>>-bootloader --timeout=3 --append="rootfstype=ext4 rootwait ${TI_WKS_BOOTLOADER_APPEND}"
> >>>>>+bootloader --timeout=0 --append="rootfstype=ext4 rootwait ${TI_WKS_BOOTLOADER_APPEND}"
> >>>>>  part --source bootimg-efi --sourceparams="loader=${EFI_PROVIDER}" --fstype=vfat --label boot --active --align 1024 --use-uuid --fixed-size 128M
> >>>>>  part / --source rootfs --fstype=ext4 --label root --align 1024 --use-uuid
> >>>>>--
> >>>>>2.39.2
> >>>>>
> >>>>>
> >>>>>-=-=-=-=-=-=-=-=-=-=-=-
> >>>>>Links: You receive all messages sent to this group.
> >>>>>View/Reply Online (#18848): https://lists.yoctoproject.org/g/meta-ti/message/18848
> >>>>>Mute This Topic: https://lists.yoctoproject.org/mt/114416911/6551054
> >>>>>Group Owner: meta-ti+owner@lists.yoctoproject.org
> >>>>>Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub [reatmon@ti.com]
> >>>>>-=-=-=-=-=-=-=-=-=-=-=-
> >>>>>

Re: [meta-ti][scarthgap/master][PATCH 3/3] wic: sdimage-2part-efi: Remove 3 second menu delay

[Andrew Davis]

On 7/28/25 5:59 PM, Denys Dmytriyenko wrote:
> On Mon, Jul 28, 2025 at 03:33:23PM -0500, Ryan Eatmon wrote:
>>
>>
>> On 7/28/2025 3:20 PM, Khem Raj via lists.yoctoproject.org wrote:
>>> On Mon, Jul 28, 2025 at 12:45 PM Denys Dmytriyenko <denis@denix.org> wrote:
>>>>
>>>> On Mon, Jul 28, 2025 at 11:28:36AM -0700, Khem Raj wrote:
>>>>> On Mon, Jul 28, 2025 at 11:25 AM Denys Dmytriyenko via
>>>>> lists.yoctoproject.org <denis=denix.org@lists.yoctoproject.org> wrote:
>>>>>>
>>>>>> On Mon, Jul 28, 2025 at 11:54:37AM -0500, Andrew Davis via lists.yoctoproject.org wrote:
>>>>>>> For all our platforms booting with EFI there is only one option today.
>>>>>>> The timeout gives the user 3 seconds to choose a different option, but
>>>>>>> as there is only one, this just causes a needless 3 second delay in
>>>>>>> boot time. Remove this delay and boot the only option immediately.
>>>>>>> If we ever have more than one option we can always add this back.
>>>>>>
>>>>>> Well, menu delay in grub is not only used for selecting among available
>>>>>> predefined options, but also for interactively editing the command line
>>>>>> on the fly before booting.
>>>>>>
>>>>>
>>>>> for development, this sounds good, for production not so much.
>>>>
>>>> Sure. And we also disable e.g. U-boot boot prompt delay all the time for our
>>>> customers during production hardening stage, among many other similar items.
>>>> But it is done for the final product - I don't think it should be done in a
>>>> generic BSP layer that many use for development.
>>>
>>> Right, it could be documented somewhere in README or some such would help
>>> the consumers of the layer.
>>
>> Is there already something in the system that we could use to drive
>> a variable that we can stick into the file?  Like
>> EXTRA_IMAGE_FEATURES?  I know there used to be a debug-tweaks that
>> got broken out into multiple features.  Maybe create a new one?
>>
>> TI_WKS_TIMEOUT = '${@bb.utils.contains("IMAGE_FEATURES",
>> "wks-debug-timeout", "3", "0 ",d)}'
> 
> Should probably make it more generic, not wks. I'd imagine it can be shared
> for U-boot and EFI/GRUB, so maybe bootloader-timeout or something?
> 

Right now we have both a 2 second delay in U-Boot, and then a 3 second delay
in the loaded EFI chainloader (GRUB). Which means after all the work we do to
push down our boot times, we still just sit for 5 seconds waiting in case some
developer has a terminal attached and wants to modify some option.. If you
are a developer then you should know how to add back the boot delay, for
everyone else it should be default disabled.

Having a "bootloader-timeout" var we can set would be nice, but it should
set the U-Boot timeout and the EFI timeout should still be default 0 IMHO.
No reason for two different delays in boot to wait for extra dev uart input.

I've got a patchset in the works to switch us away from GRUB to uefi-kernel
anyway, which means direct kernel boot through EFI. No middle-man EFI
chainloader like GRUB or systemd-boot needed. Just U-Boot -> Kernel like
it should be on an embedded system.

Andrew

> 
>> bootloader --timeout=${TI_WKS_TIMEOUT} --append="rootfstype=ext4
>> rootwait ${TI_WKS_BOOTLOADER_APPEND}"
>>
>>
>> Would that work?
>>
>>
>>>>
>>>>
>>>>>>> Signed-off-by: Andrew Davis <afd@ti.com>
>>>>>>> ---
>>>>>>>   meta-ti-bsp/wic/sdimage-2part-efi.wks.in | 2 +-
>>>>>>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>>>>>>
>>>>>>> diff --git a/meta-ti-bsp/wic/sdimage-2part-efi.wks.in b/meta-ti-bsp/wic/sdimage-2part-efi.wks.in
>>>>>>> index 7252411c..c1089762 100644
>>>>>>> --- a/meta-ti-bsp/wic/sdimage-2part-efi.wks.in
>>>>>>> +++ b/meta-ti-bsp/wic/sdimage-2part-efi.wks.in
>>>>>>> @@ -3,6 +3,6 @@
>>>>>>>   # supports EFI.  Boot files are located in the first vfat partition with extra
>>>>>>>   # reserved space.  We cannot use a GPT here.
>>>>>>>
>>>>>>> -bootloader --timeout=3 --append="rootfstype=ext4 rootwait ${TI_WKS_BOOTLOADER_APPEND}"
>>>>>>> +bootloader --timeout=0 --append="rootfstype=ext4 rootwait ${TI_WKS_BOOTLOADER_APPEND}"
>>>>>>>   part --source bootimg-efi --sourceparams="loader=${EFI_PROVIDER}" --fstype=vfat --label boot --active --align 1024 --use-uuid --fixed-size 128M
>>>>>>>   part / --source rootfs --fstype=ext4 --label root --align 1024 --use-uuid
>>>>>>> --
>>>>>>> 2.39.2
>>>>>>>
>>>>>>>
>>>>>>> -=-=-=-=-=-=-=-=-=-=-=-
>>>>>>> Links: You receive all messages sent to this group.
>>>>>>> View/Reply Online (#18848): https://lists.yoctoproject.org/g/meta-ti/message/18848
>>>>>>> Mute This Topic: https://lists.yoctoproject.org/mt/114416911/6551054
>>>>>>> Group Owner: meta-ti+owner@lists.yoctoproject.org
>>>>>>> Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub [reatmon@ti.com]
>>>>>>> -=-=-=-=-=-=-=-=-=-=-=-
>>>>>>>

Re: [meta-ti][scarthgap/master][PATCH 3/3] wic: sdimage-2part-efi: Remove 3 second menu delay

[Ryan Eatmon]

On 7/29/2025 1:03 PM, Andrew Davis wrote:
> On 7/28/25 5:59 PM, Denys Dmytriyenko wrote:
>> On Mon, Jul 28, 2025 at 03:33:23PM -0500, Ryan Eatmon wrote:
>>>
>>>
>>> On 7/28/2025 3:20 PM, Khem Raj via lists.yoctoproject.org wrote:
>>>> On Mon, Jul 28, 2025 at 12:45 PM Denys Dmytriyenko <denis@denix.org> 
>>>> wrote:
>>>>>
>>>>> On Mon, Jul 28, 2025 at 11:28:36AM -0700, Khem Raj wrote:
>>>>>> On Mon, Jul 28, 2025 at 11:25 AM Denys Dmytriyenko via
>>>>>> lists.yoctoproject.org <denis=denix.org@lists.yoctoproject.org> 
>>>>>> wrote:
>>>>>>>
>>>>>>> On Mon, Jul 28, 2025 at 11:54:37AM -0500, Andrew Davis via 
>>>>>>> lists.yoctoproject.org wrote:
>>>>>>>> For all our platforms booting with EFI there is only one option 
>>>>>>>> today.
>>>>>>>> The timeout gives the user 3 seconds to choose a different 
>>>>>>>> option, but
>>>>>>>> as there is only one, this just causes a needless 3 second delay in
>>>>>>>> boot time. Remove this delay and boot the only option immediately.
>>>>>>>> If we ever have more than one option we can always add this back.
>>>>>>>
>>>>>>> Well, menu delay in grub is not only used for selecting among 
>>>>>>> available
>>>>>>> predefined options, but also for interactively editing the 
>>>>>>> command line
>>>>>>> on the fly before booting.
>>>>>>>
>>>>>>
>>>>>> for development, this sounds good, for production not so much.
>>>>>
>>>>> Sure. And we also disable e.g. U-boot boot prompt delay all the 
>>>>> time for our
>>>>> customers during production hardening stage, among many other 
>>>>> similar items.
>>>>> But it is done for the final product - I don't think it should be 
>>>>> done in a
>>>>> generic BSP layer that many use for development.
>>>>
>>>> Right, it could be documented somewhere in README or some such would 
>>>> help
>>>> the consumers of the layer.
>>>
>>> Is there already something in the system that we could use to drive
>>> a variable that we can stick into the file?  Like
>>> EXTRA_IMAGE_FEATURES?  I know there used to be a debug-tweaks that
>>> got broken out into multiple features.  Maybe create a new one?
>>>
>>> TI_WKS_TIMEOUT = '${@bb.utils.contains("IMAGE_FEATURES",
>>> "wks-debug-timeout", "3", "0 ",d)}'
>>
>> Should probably make it more generic, not wks. I'd imagine it can be 
>> shared
>> for U-boot and EFI/GRUB, so maybe bootloader-timeout or something?
>>
> 
> Right now we have both a 2 second delay in U-Boot, and then a 3 second 
> delay
> in the loaded EFI chainloader (GRUB). Which means after all the work we 
> do to
> push down our boot times, we still just sit for 5 seconds waiting in 
> case some
> developer has a terminal attached and wants to modify some option.. If you
> are a developer then you should know how to add back the boot delay, for
> everyone else it should be default disabled.

Agreed.  It would just be nice to have some nice easy way in the 
local.conf to add the delay back in if you want.


> Having a "bootloader-timeout" var we can set would be nice, but it should
> set the U-Boot timeout and the EFI timeout should still be default 0 IMHO.
> No reason for two different delays in boot to wait for extra dev uart 
> input.
> 
> I've got a patchset in the works to switch us away from GRUB to uefi-kernel
> anyway, which means direct kernel boot through EFI. No middle-man EFI
> chainloader like GRUB or systemd-boot needed. Just U-Boot -> Kernel like
> it should be on an embedded system.
> 
> Andrew
> 
>>
>>> bootloader --timeout=${TI_WKS_TIMEOUT} --append="rootfstype=ext4
>>> rootwait ${TI_WKS_BOOTLOADER_APPEND}"
>>>
>>>
>>> Would that work?
>>>
>>>
>>>>>
>>>>>
>>>>>>>> Signed-off-by: Andrew Davis <afd@ti.com>
>>>>>>>> ---
>>>>>>>>   meta-ti-bsp/wic/sdimage-2part-efi.wks.in | 2 +-
>>>>>>>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>>>>>>>
>>>>>>>> diff --git a/meta-ti-bsp/wic/sdimage-2part-efi.wks.in 
>>>>>>>> b/meta-ti-bsp/wic/sdimage-2part-efi.wks.in
>>>>>>>> index 7252411c..c1089762 100644
>>>>>>>> --- a/meta-ti-bsp/wic/sdimage-2part-efi.wks.in
>>>>>>>> +++ b/meta-ti-bsp/wic/sdimage-2part-efi.wks.in
>>>>>>>> @@ -3,6 +3,6 @@
>>>>>>>>   # supports EFI.  Boot files are located in the first vfat 
>>>>>>>> partition with extra
>>>>>>>>   # reserved space.  We cannot use a GPT here.
>>>>>>>>
>>>>>>>> -bootloader --timeout=3 --append="rootfstype=ext4 rootwait 
>>>>>>>> ${TI_WKS_BOOTLOADER_APPEND}"
>>>>>>>> +bootloader --timeout=0 --append="rootfstype=ext4 rootwait 
>>>>>>>> ${TI_WKS_BOOTLOADER_APPEND}"
>>>>>>>>   part --source bootimg-efi 
>>>>>>>> --sourceparams="loader=${EFI_PROVIDER}" --fstype=vfat --label 
>>>>>>>> boot --active --align 1024 --use-uuid --fixed-size 128M
>>>>>>>>   part / --source rootfs --fstype=ext4 --label root --align 1024 
>>>>>>>> --use-uuid
>>>>>>>> -- 
>>>>>>>> 2.39.2
>>>>>>>>
>>>>>>>>
>>>>>>>> -=-=-=-=-=-=-=-=-=-=-=-
>>>>>>>> Links: You receive all messages sent to this group.
>>>>>>>> View/Reply Online (#18848): 
>>>>>>>> https://lists.yoctoproject.org/g/meta-ti/message/18848
>>>>>>>> Mute This Topic: 
>>>>>>>> https://lists.yoctoproject.org/mt/114416911/6551054
>>>>>>>> Group Owner: meta-ti+owner@lists.yoctoproject.org
>>>>>>>> Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub 
>>>>>>>> [reatmon@ti.com]
>>>>>>>> -=-=-=-=-=-=-=-=-=-=-=-
>>>>>>>>
> 

-- 
Ryan Eatmon                reatmon@ti.com
-----------------------------------------
Texas Instruments, Inc.  -  LCPD  -  MGTS