diff mbox series

[wrynose] am62pxx-evm: Add tisdk-uenv configuration for display support

Message ID 20261111101358.3203148-1-r-sharma3@ti.com
State Superseded
Delegated to: Ryan Eatmon
Headers show
Series [wrynose] am62pxx-evm: Add tisdk-uenv configuration for display support | expand

Commit Message

Rahul Sharma Nov. 11, 2026, 10:13 a.m. UTC 
Add U-Boot environment configuration to enable OLDI display output
on the AM62Pxx EVM. This includes:

- Create tisdk-uenv bbappend for am62pxx machine
- Add uEnv.txt with OLDI simplefb device tree overlay

The configuration loads k3-am62p5-sk-oldi-simplefb.dtbo overlay
to enable display output through the OLDI interface.

Signed-off-by: Rahul Sharma <r-sharma3@ti.com>
---
 .../recipes-tisdk/tisdk-uenv/tisdk-uenv.bbappend     |  5 +++++
 .../tisdk-uenv/tisdk-uenv/am62pxx-evm/uEnv.txt       | 12 ++++++++++++
 2 files changed, 17 insertions(+)
 create mode 100644 meta-ti-bsp/recipes-tisdk/tisdk-uenv/tisdk-uenv.bbappend
 create mode 100644 meta-ti-bsp/recipes-tisdk/tisdk-uenv/tisdk-uenv/am62pxx-evm/uEnv.txt

Comments

PRC Automation Nov. 11, 2026, 10:16 a.m. UTC | #1 
meta-ti / na / 20261111101358.3203148-1-r-sharma3

PRC Results: FAIL

=========================================================
  check-yocto-patches: PASS
=========================================================
Patches
----------------------------------------
WARN - [wrynose][PATCH] am62pxx-evm: Add tisdk-uenv configuration for display support
    WARN: Commit message does not include file/recipe name: am62pxx-evm: Add tisdk-uenv configuration for display support. (COMMIT-MESSAGE-2)
        patch
    
    WARN: Commit message should not include directory path to recipe: am62pxx-evm: Add tisdk-uenv configuration for display support.
              am62pxx-evm (COMMIT-MESSAGE-3)
        patch
    
    For details on the above errors/warnings visit: https://lists.yoctoproject.org/g/meta-ti/wiki/40887



=========================================================
  apply-yocto-patch: PASS
=========================================================
wrynose
=====================
Summary:
- Patch Series: [wrynose][PATCH] am62pxx-evm: Add tisdk-uenv configuration for display support
- Submitter: From: Rahul Sharma <r-sharma3@ti.com>
- Date: Date: Wed, 11 Nov 2026 15:43:58 +0530
- Num Patches: 1
- Mailing List (public inbox) Commit SHA: 56275f2b2ea875e77d1bf0be8d391eec93a5af6f

Applied to:
- Repository: lcpd-prc-meta-ti
- Base Branch: wrynose-wip
- Commit Author: Rahul Sharma <r-sharma3@ti.com>
- Commit Subject: am62lxx-evm: Add DSI config fragments for display support
- Commit SHA: ce7e8077fa980d8e634db3340f6f9e8d81a12123

Patches
----------------------------------------
All patches applied



=========================================================
  check-yocto-repo: FAIL
=========================================================
wrynose
=====================
FAIL
    WARN: .bbappend files might need a guard to make them check-layer compliant. (GUARD-1)
        meta-ti-bsp/recipes-tisdk/tisdk-uenv/tisdk-uenv.bbappend
    
    For details on the above errors/warnings visit: https://lists.yoctoproject.org/g/meta-ti/wiki/40887
Thakkar, Devarsh Nov. 11, 2026, 11:37 a.m. UTC | #2 
On 11/11/2026 3:43 PM, Rahul Sharma via meta-ti wrote:
> Add U-Boot environment configuration to enable OLDI display output
> on the AM62Pxx EVM. This includes:
> 
> - Create tisdk-uenv bbappend for am62pxx machine
> - Add uEnv.txt with OLDI simplefb device tree overlay
> 
> The configuration loads k3-am62p5-sk-oldi-simplefb.dtbo overlay
> to enable display output through the OLDI interface.
> 
> Signed-off-by: Rahul Sharma <r-sharma3@ti.com>
> ---
>  .../recipes-tisdk/tisdk-uenv/tisdk-uenv.bbappend     |  5 +++++
>  .../tisdk-uenv/tisdk-uenv/am62pxx-evm/uEnv.txt       | 12 ++++++++++++
>  2 files changed, 17 insertions(+)
>  create mode 100644 meta-ti-bsp/recipes-tisdk/tisdk-uenv/tisdk-uenv.bbappend
>  create mode 100644 meta-ti-bsp/recipes-tisdk/tisdk-uenv/tisdk-uenv/am62pxx-evm/uEnv.txt
> 
> diff --git a/meta-ti-bsp/recipes-tisdk/tisdk-uenv/tisdk-uenv.bbappend b/meta-ti-bsp/recipes-tisdk/tisdk-uenv/tisdk-uenv.bbappend
> new file mode 100644
> index 00000000..bc061b1c
> --- /dev/null
> +++ b/meta-ti-bsp/recipes-tisdk/tisdk-uenv/tisdk-uenv.bbappend
> @@ -0,0 +1,5 @@
> +FILESEXTRAPATHS:prepend := "${THISDIR}/${PN}/${MACHINE}:"
> +
> +SRC_URI:append:am62pxx = "\
> +    file://uEnv.txt \
> +"
> diff --git a/meta-ti-bsp/recipes-tisdk/tisdk-uenv/tisdk-uenv/am62pxx-evm/uEnv.txt b/meta-ti-bsp/recipes-tisdk/tisdk-uenv/tisdk-uenv/am62pxx-evm/uEnv.txt
> new file mode 100644
> index 00000000..0ffda50d
> --- /dev/null
> +++ b/meta-ti-bsp/recipes-tisdk/tisdk-uenv/tisdk-uenv/am62pxx-evm/uEnv.txt
> @@ -0,0 +1,12 @@
> +# This uEnv.txt file can contain additional environment settings that you
> +# want to set in U-Boot at boot time.  This can be simple variables such
> +# as the serverip or custom variables.  The format of this file is:
> +#    variable=value
> +# NOTE: This file will be evaluated after the bootcmd is run and the
> +#       bootcmd must be set to load this file if it exists (this is the
> +#       default on all newer U-Boot images.  This also means that some
> +#       variables such as bootdelay cannot be changed by this file since
> +#       it is not evaluated until the bootcmd is run.
> +
> +optargs=vt.global_cursor_default=0

I don't think above is related to this patch. Let's remove it for now
and keep the cursor enabled.

With above change,
Reviewed-by: Devarsh Thakkar <devarsht@ti.com>

Regards
Devarsh

> +name_overlays=ti/k3-am62p5-sk-oldi-simplefb.dtbo
> 
> 
> 
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#20518): https://lists.yoctoproject.org/g/meta-ti/message/20518
> Mute This Topic: https://lists.yoctoproject.org/mt/122209407/6740599
> Group Owner: meta-ti+owner@lists.yoctoproject.org
> Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub [devarsht@ti.com]
> -=-=-=-=-=-=-=-=-=-=-=-
Ryan Eatmon Nov. 11, 2026, 4:05 p.m. UTC | #3 
On 11/11/2026 4:13 AM, Rahul Sharma wrote:
> Add U-Boot environment configuration to enable OLDI display output
> on the AM62Pxx EVM. This includes:
> 
> - Create tisdk-uenv bbappend for am62pxx machine
> - Add uEnv.txt with OLDI simplefb device tree overlay
> 
> The configuration loads k3-am62p5-sk-oldi-simplefb.dtbo overlay
> to enable display output through the OLDI interface.
> 
> Signed-off-by: Rahul Sharma <r-sharma3@ti.com>
> ---
>   .../recipes-tisdk/tisdk-uenv/tisdk-uenv.bbappend     |  5 +++++
>   .../tisdk-uenv/tisdk-uenv/am62pxx-evm/uEnv.txt       | 12 ++++++++++++
>   2 files changed, 17 insertions(+)
>   create mode 100644 meta-ti-bsp/recipes-tisdk/tisdk-uenv/tisdk-uenv.bbappend
>   create mode 100644 meta-ti-bsp/recipes-tisdk/tisdk-uenv/tisdk-uenv/am62pxx-evm/uEnv.txt
> 
> diff --git a/meta-ti-bsp/recipes-tisdk/tisdk-uenv/tisdk-uenv.bbappend b/meta-ti-bsp/recipes-tisdk/tisdk-uenv/tisdk-uenv.bbappend
> new file mode 100644
> index 00000000..bc061b1c
> --- /dev/null
> +++ b/meta-ti-bsp/recipes-tisdk/tisdk-uenv/tisdk-uenv.bbappend
> @@ -0,0 +1,5 @@
> +FILESEXTRAPATHS:prepend := "${THISDIR}/${PN}/${MACHINE}:"
> +
> +SRC_URI:append:am62pxx = "\
> +    file://uEnv.txt \
> +"
> diff --git a/meta-ti-bsp/recipes-tisdk/tisdk-uenv/tisdk-uenv/am62pxx-evm/uEnv.txt b/meta-ti-bsp/recipes-tisdk/tisdk-uenv/tisdk-uenv/am62pxx-evm/uEnv.txt
> new file mode 100644
> index 00000000..0ffda50d
> --- /dev/null
> +++ b/meta-ti-bsp/recipes-tisdk/tisdk-uenv/tisdk-uenv/am62pxx-evm/uEnv.txt
> @@ -0,0 +1,12 @@
> +# This uEnv.txt file can contain additional environment settings that you
> +# want to set in U-Boot at boot time.  This can be simple variables such
> +# as the serverip or custom variables.  The format of this file is:
> +#    variable=value
> +# NOTE: This file will be evaluated after the bootcmd is run and the
> +#       bootcmd must be set to load this file if it exists (this is the
> +#       default on all newer U-Boot images.  This also means that some
> +#       variables such as bootdelay cannot be changed by this file since
> +#       it is not evaluated until the bootcmd is run.
> +
> +optargs=vt.global_cursor_default=0
> +name_overlays=ti/k3-am62p5-sk-oldi-simplefb.dtbo

NAK.  You are modifying a file in meta-ti-bsp that is owned by 
meta-arago.  The BSP layer should not be changing something from a 
distro layer.  You should just send a patch to the meta-arago list to 
change this.

But I need to ask, why are we trying to change things like this in the 
uenv file?  Is no better way of doing this in the defconfig for building 
uboot on this platform?
Andrew Davis Nov. 11, 2026, 4:20 p.m. UTC | #4 
On 11/11/26 10:05 AM, Ryan Eatmon via meta-ti wrote:
> 
> 
> On 11/11/2026 4:13 AM, Rahul Sharma wrote:
>> Add U-Boot environment configuration to enable OLDI display output
>> on the AM62Pxx EVM. This includes:
>>
>> - Create tisdk-uenv bbappend for am62pxx machine
>> - Add uEnv.txt with OLDI simplefb device tree overlay
>>
>> The configuration loads k3-am62p5-sk-oldi-simplefb.dtbo overlay
>> to enable display output through the OLDI interface.
>>
>> Signed-off-by: Rahul Sharma <r-sharma3@ti.com>
>> ---
>>   .../recipes-tisdk/tisdk-uenv/tisdk-uenv.bbappend     |  5 +++++
>>   .../tisdk-uenv/tisdk-uenv/am62pxx-evm/uEnv.txt       | 12 ++++++++++++
>>   2 files changed, 17 insertions(+)
>>   create mode 100644 meta-ti-bsp/recipes-tisdk/tisdk-uenv/tisdk-uenv.bbappend
>>   create mode 100644 meta-ti-bsp/recipes-tisdk/tisdk-uenv/tisdk-uenv/am62pxx-evm/uEnv.txt
>>
>> diff --git a/meta-ti-bsp/recipes-tisdk/tisdk-uenv/tisdk-uenv.bbappend b/meta-ti-bsp/recipes-tisdk/tisdk-uenv/tisdk-uenv.bbappend
>> new file mode 100644
>> index 00000000..bc061b1c
>> --- /dev/null
>> +++ b/meta-ti-bsp/recipes-tisdk/tisdk-uenv/tisdk-uenv.bbappend
>> @@ -0,0 +1,5 @@
>> +FILESEXTRAPATHS:prepend := "${THISDIR}/${PN}/${MACHINE}:"
>> +
>> +SRC_URI:append:am62pxx = "\
>> +    file://uEnv.txt \
>> +"
>> diff --git a/meta-ti-bsp/recipes-tisdk/tisdk-uenv/tisdk-uenv/am62pxx-evm/uEnv.txt b/meta-ti-bsp/recipes-tisdk/tisdk-uenv/tisdk-uenv/am62pxx-evm/uEnv.txt
>> new file mode 100644
>> index 00000000..0ffda50d
>> --- /dev/null
>> +++ b/meta-ti-bsp/recipes-tisdk/tisdk-uenv/tisdk-uenv/am62pxx-evm/uEnv.txt
>> @@ -0,0 +1,12 @@
>> +# This uEnv.txt file can contain additional environment settings that you
>> +# want to set in U-Boot at boot time.  This can be simple variables such
>> +# as the serverip or custom variables.  The format of this file is:
>> +#    variable=value
>> +# NOTE: This file will be evaluated after the bootcmd is run and the
>> +#       bootcmd must be set to load this file if it exists (this is the
>> +#       default on all newer U-Boot images.  This also means that some
>> +#       variables such as bootdelay cannot be changed by this file since
>> +#       it is not evaluated until the bootcmd is run.
>> +
>> +optargs=vt.global_cursor_default=0
>> +name_overlays=ti/k3-am62p5-sk-oldi-simplefb.dtbo
> 
> NAK.  You are modifying a file in meta-ti-bsp that is owned by meta-arago.  The BSP layer should not be changing something from a distro layer.  You should just send a patch to the meta-arago list to change this.
> 
> But I need to ask, why are we trying to change things like this in the uenv file?  Is no better way of doing this in the defconfig for building uboot on this platform?
> 
> 

Agree, don't add anything to the eEnv.txt file, that exists for end
users to make customizations, usually just for quick testing (vs having
to rebuild u-boot to change the default environment).

For us, you should make these kinds of changes in ti-u-boot.

Andrew

> 
> 
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#20530): https://lists.yoctoproject.org/g/meta-ti/message/20530
> Mute This Topic: https://lists.yoctoproject.org/mt/122209407/3619733
> Group Owner: meta-ti+owner@lists.yoctoproject.org
> Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub [afd@ti.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
diff mbox series

Patch

diff --git a/meta-ti-bsp/recipes-tisdk/tisdk-uenv/tisdk-uenv.bbappend b/meta-ti-bsp/recipes-tisdk/tisdk-uenv/tisdk-uenv.bbappend
new file mode 100644
index 00000000..bc061b1c
--- /dev/null
+++ b/meta-ti-bsp/recipes-tisdk/tisdk-uenv/tisdk-uenv.bbappend
@@ -0,0 +1,5 @@ 
+FILESEXTRAPATHS:prepend := "${THISDIR}/${PN}/${MACHINE}:"
+
+SRC_URI:append:am62pxx = "\
+    file://uEnv.txt \
+"
diff --git a/meta-ti-bsp/recipes-tisdk/tisdk-uenv/tisdk-uenv/am62pxx-evm/uEnv.txt b/meta-ti-bsp/recipes-tisdk/tisdk-uenv/tisdk-uenv/am62pxx-evm/uEnv.txt
new file mode 100644
index 00000000..0ffda50d
--- /dev/null
+++ b/meta-ti-bsp/recipes-tisdk/tisdk-uenv/tisdk-uenv/am62pxx-evm/uEnv.txt
@@ -0,0 +1,12 @@ 
+# This uEnv.txt file can contain additional environment settings that you
+# want to set in U-Boot at boot time.  This can be simple variables such
+# as the serverip or custom variables.  The format of this file is:
+#    variable=value
+# NOTE: This file will be evaluated after the bootcmd is run and the
+#       bootcmd must be set to load this file if it exists (this is the
+#       default on all newer U-Boot images.  This also means that some
+#       variables such as bootdelay cannot be changed by this file since
+#       it is not evaluated until the bootcmd is run.
+
+optargs=vt.global_cursor_default=0
+name_overlays=ti/k3-am62p5-sk-oldi-simplefb.dtbo