diff mbox series

[meta-ti,master/scarthgap,RFC] conf: machine: am62xx-unified: Create new common am62xx machine

Message ID 20240808192718.14154-1-reatmon@ti.com
State RFC
Delegated to: Ryan Eatmon
Headers show
Series [meta-ti,master/scarthgap,RFC] conf: machine: am62xx-unified: Create new common am62xx machine | expand

Commit Message

Ryan Eatmon Aug. 8, 2024, 7:27 p.m. UTC
To reduce the number of required builds for testing purposes, create a
common unified am62xx machine target that can produce all of the files
for all of the variant platforms related to am62xx (sip and lp).
Additionally, continue to support building the specific target if
desired.

Signed-off-by: Ryan Eatmon <reatmon@ti.com>
---
 meta-ti-bsp/conf/machine/am62xx-evm-k3r5.conf |  9 +---
 meta-ti-bsp/conf/machine/am62xx-evm.conf      |  5 +--
 .../conf/machine/am62xx-lp-evm-k3r5.conf      |  9 +---
 meta-ti-bsp/conf/machine/am62xx-lp-evm.conf   |  5 +--
 .../conf/machine/am62xx-unified-k3r5.conf     | 20 +++++++++
 meta-ti-bsp/conf/machine/am62xx-unified.conf  | 43 +++++++++++++++++++
 .../conf/machine/am62xxsip-evm-k3r5.conf      |  8 ++--
 meta-ti-bsp/conf/machine/am62xxsip-evm.conf   |  6 +--
 8 files changed, 76 insertions(+), 29 deletions(-)
 create mode 100644 meta-ti-bsp/conf/machine/am62xx-unified-k3r5.conf
 create mode 100644 meta-ti-bsp/conf/machine/am62xx-unified.conf

Comments

Chirag Shilwant Aug. 14, 2024, 6:09 p.m. UTC | #1
Hi Ryan & Denys,
On 09/08/24 00:57, Ryan Eatmon via lists.yoctoproject.org wrote:
> To reduce the number of required builds for testing purposes, create a 
> common unified am62xx machine target that can produce all of the files 
> for all of the variant platforms related to am62xx (sip and lp). 
> Additionally, continue to support
> ZjQcmQRYFpfptBannerStart
> This message was sent from outside of Texas Instruments.
> Do not click links or open attachments unless you recognize the source 
> of this email and know the content is safe. If you wish to report this 
> message to IT Security, please forward the message as an attachment to 
> phishing@list.ti.com
> Report Suspicious
> <https://us-phishalarm-ewt.proofpoint.com/EWT/v1/G3vK!uFdqXRfP9mwbKwZFHJNj_V92vi6mT9DGTpQZeFU0LNi8euAkYQM024IcX-dd54y6XIw8nwcGUnREacyk1hI$> 
>
> ZjQcmQRYFpfptBannerEnd
> To reduce the number of required builds for testing purposes, create a
> common unified am62xx machine target that can produce all of the files
> for all of the variant platforms related to am62xx (sip and lp).
> Additionally, continue to support building the specific target if
> desired.
>
> Signed-off-by: Ryan Eatmon <reatmon@ti.com>



We need to see how do we enable the support of fragment configs with 
UBOOT_CONFIG
as I see atleast ti-u-boot-2024.04 still following the fragment config 
model for few usecases.

I see two approaches,

- Add the support in oe-core to handle u-boot fragment configs

- Extend u-boot-mergeconfig.inc we currently have in meta-ti [1] to 
support UBOOT_CONFIG


Personally, I would prefer the first option if that's acceptable to 
oe-core as well
but I would like to hear your opinions on the same


[1]: 
https://git.ti.com/cgit/arago-project/meta-ti/tree/meta-ti-bsp/recipes-bsp/u-boot/u-boot-mergeconfig.inc?h=scarthgap


---
Chirag



> ---
>   meta-ti-bsp/conf/machine/am62xx-evm-k3r5.conf |  9 +---
>   meta-ti-bsp/conf/machine/am62xx-evm.conf      |  5 +--
>   .../conf/machine/am62xx-lp-evm-k3r5.conf      |  9 +---
>   meta-ti-bsp/conf/machine/am62xx-lp-evm.conf   |  5 +--
>   .../conf/machine/am62xx-unified-k3r5.conf     | 20 +++++++++
>   meta-ti-bsp/conf/machine/am62xx-unified.conf  | 43 +++++++++++++++++++
>   .../conf/machine/am62xxsip-evm-k3r5.conf      |  8 ++--
>   meta-ti-bsp/conf/machine/am62xxsip-evm.conf   |  6 +--
>   8 files changed, 76 insertions(+), 29 deletions(-)
>   create mode 100644 meta-ti-bsp/conf/machine/am62xx-unified-k3r5.conf
>   create mode 100644 meta-ti-bsp/conf/machine/am62xx-unified.conf
>
> diff --git a/meta-ti-bsp/conf/machine/am62xx-evm-k3r5.conf b/meta-ti-bsp/conf/machine/am62xx-evm-k3r5.conf
> index 548369ca..3a819835 100644
> --- a/meta-ti-bsp/conf/machine/am62xx-evm-k3r5.conf
> +++ b/meta-ti-bsp/conf/machine/am62xx-evm-k3r5.conf
> @@ -2,11 +2,6 @@
>   #@NAME: AM62XX EVM (R5F)
>   #@DESCRIPTION: Machine configuration for the TI AM62xx EVM (R5F core)
>   
> -require conf/machine/include/k3r5.inc
> -require conf/machine/include/ti-extras.inc
> +require conf/machine/am62xx-unified-k3r5.conf
>   
> -SYSFW_SOC = "am62x"
> -SYSFW_CONFIG = "evm"
> -SYSFW_SUFFIX = "hs-fs"
> -
> -UBOOT_MACHINE = "am62x_evm_r5_defconfig"
> +UBOOT_CONFIG = "evm"
> diff --git a/meta-ti-bsp/conf/machine/am62xx-evm.conf b/meta-ti-bsp/conf/machine/am62xx-evm.conf
> index 31de4a14..8c80cb1f 100644
> --- a/meta-ti-bsp/conf/machine/am62xx-evm.conf
> +++ b/meta-ti-bsp/conf/machine/am62xx-evm.conf
> @@ -2,8 +2,7 @@
>   #@NAME: AM62XX EVM
>   #@DESCRIPTION: Machine configuration for the TI AM62XX EVM
>   
> -require conf/machine/include/am62xx.inc
> -require conf/machine/include/ti-extras.inc
> +require conf/machine/am62xx-unified.conf
>   
>   KERNEL_DEVICETREE_PREFIX = " \
>       ti/k3-am625 \
> @@ -30,4 +29,4 @@ KERNEL_DEVICETREE = " \
>       ti/k3-am62x-sk-hdmi-audio.dtbo \
>   "
>   
> -UBOOT_MACHINE = "am62x_evm_a53_defconfig"
> +UBOOT_CONFIG = "evm"
> diff --git a/meta-ti-bsp/conf/machine/am62xx-lp-evm-k3r5.conf b/meta-ti-bsp/conf/machine/am62xx-lp-evm-k3r5.conf
> index 52b69a72..fb760e99 100644
> --- a/meta-ti-bsp/conf/machine/am62xx-lp-evm-k3r5.conf
> +++ b/meta-ti-bsp/conf/machine/am62xx-lp-evm-k3r5.conf
> @@ -2,11 +2,6 @@
>   #@NAME: AM62XX LP GP EVM (R5F)
>   #@DESCRIPTION: Machine configuration for the TI AM62xx LP GP EVM (R5F core)
>   
> -require conf/machine/include/k3r5.inc
> -require conf/machine/include/ti-extras.inc
> +require conf/machine/am62xx-unified-k3r5.conf
>   
> -SYSFW_SOC = "am62x"
> -SYSFW_CONFIG = "evm"
> -SYSFW_SUFFIX = "hs-fs"
> -
> -UBOOT_MACHINE = "am62x_lpsk_r5_defconfig"
> +UBOOT_CONFIG = "lpsk"
> diff --git a/meta-ti-bsp/conf/machine/am62xx-lp-evm.conf b/meta-ti-bsp/conf/machine/am62xx-lp-evm.conf
> index e9271f5b..f7d89d68 100644
> --- a/meta-ti-bsp/conf/machine/am62xx-lp-evm.conf
> +++ b/meta-ti-bsp/conf/machine/am62xx-lp-evm.conf
> @@ -2,8 +2,7 @@
>   #@NAME: AM62XX LP EVM
>   #@DESCRIPTION: Machine configuration for the TI AM62XX LP EVM
>   
> -require conf/machine/include/am62xx.inc
> -require conf/machine/include/ti-extras.inc
> +require conf/machine/am62xx-unified.conf
>   
>   KERNEL_DEVICETREE_PREFIX = " \
>       ti/k3-am62-lp \
> @@ -19,4 +18,4 @@ KERNEL_DEVICETREE = " \
>       ti/k3-am62x-sk-hdmi-audio.dtbo \
>   "
>   
> -UBOOT_MACHINE = "am62x_lpsk_a53_defconfig"
> +UBOOT_CONFIG = "lpsk"
> diff --git a/meta-ti-bsp/conf/machine/am62xx-unified-k3r5.conf b/meta-ti-bsp/conf/machine/am62xx-unified-k3r5.conf
> new file mode 100644
> index 00000000..7d65e6af
> --- /dev/null
> +++ b/meta-ti-bsp/conf/machine/am62xx-unified-k3r5.conf
> @@ -0,0 +1,20 @@
> +#@TYPE: Machine
> +#@NAME: AM62XX EVM (R5F)
> +#@DESCRIPTION: Machine configuration for the TI AM62xx EVM (R5F core)
> +
> +require conf/machine/include/k3r5.inc
> +require conf/machine/include/ti-extras.inc
> +
> +SYSFW_SOC = "am62x"
> +SYSFW_CONFIG = "evm"
> +SYSFW_SUFFIX = "hs-fs"
> +
> +UBOOT_MACHINE = ""
> +# Last config in the list is default
> +UBOOT_CONFIG ??= "sip lpsk evm"
> +
> +UBOOT_CONFIG:bsp-ti-6_1 ??= "evm"
> +
> +UBOOT_CONFIG[sip] = "am62xsip_evm_r5_defconfig"
> +UBOOT_CONFIG[lpsk] = "am62x_lpsk_r5_defconfig"
> +UBOOT_CONFIG[evm] = "am62x_evm_r5_defconfig"
> diff --git a/meta-ti-bsp/conf/machine/am62xx-unified.conf b/meta-ti-bsp/conf/machine/am62xx-unified.conf
> new file mode 100644
> index 00000000..5b03239f
> --- /dev/null
> +++ b/meta-ti-bsp/conf/machine/am62xx-unified.conf
> @@ -0,0 +1,43 @@
> +#@TYPE: Machine
> +#@NAME: AM62XX Unified EVM
> +#@DESCRIPTION: Machine configuration for the TI AM62XX EVM and all of its variations
> +
> +require conf/machine/include/am62xx.inc
> +require conf/machine/include/ti-extras.inc
> +
> +KERNEL_DEVICETREE_PREFIX = " \
> +    ti/k3-am62-lp \
> +    ti/k3-am625 \
> +    ti/k3-am62x-sk \
> +"
> +
> +KERNEL_DEVICETREE = " \
> +    ti/k3-am62-lp-sk.dtb \
> +    ti/k3-am625-beagleplay-csi2-ov5640.dtbo \
> +    ti/k3-am625-beagleplay-csi2-tevi-ov5640.dtbo \
> +    ti/k3-am625-beagleplay.dtb \
> +    ti/k3-am625-phyboard-lyra-rdk.dtb \
> +    ti/k3-am625-sk.dtb \
> +    ti/k3-am625-verdin-nonwifi-dahlia.dtb \
> +    ti/k3-am625-verdin-nonwifi-dev.dtb \
> +    ti/k3-am625-verdin-nonwifi-mallow.dtb \
> +    ti/k3-am625-verdin-nonwifi-yavia.dtb \
> +    ti/k3-am625-verdin-wifi-dahlia.dtb \
> +    ti/k3-am625-verdin-wifi-dev.dtb \
> +    ti/k3-am625-verdin-wifi-mallow.dtb \
> +    ti/k3-am625-verdin-wifi-yavia.dtb \
> +    ti/k3-am62x-sk-csi2-imx219.dtbo \
> +    ti/k3-am62x-sk-csi2-ov5640.dtbo \
> +    ti/k3-am62x-sk-csi2-tevi-ov5640.dtbo \
> +    ti/k3-am62x-sk-hdmi-audio.dtbo \
> +"
> +
> +UBOOT_MACHINE = ""
> +# Last config in the list is default
> +UBOOT_CONFIG ??= "sip lpsk evm"
> +
> +UBOOT_CONFIG:bsp-ti-6_1 ??= "lpsk evm"
> +
> +UBOOT_CONFIG[sip] = "am62xsip_evm_a53_defconfig"
> +UBOOT_CONFIG[lpsk] = "am62x_lpsk_a53_defconfig"
> +UBOOT_CONFIG[evm] = "am62x_evm_a53_defconfig"
> diff --git a/meta-ti-bsp/conf/machine/am62xxsip-evm-k3r5.conf b/meta-ti-bsp/conf/machine/am62xxsip-evm-k3r5.conf
> index 55bc530b..ab538ff3 100644
> --- a/meta-ti-bsp/conf/machine/am62xxsip-evm-k3r5.conf
> +++ b/meta-ti-bsp/conf/machine/am62xxsip-evm-k3r5.conf
> @@ -2,13 +2,11 @@
>   #@NAME: AM62XX SIP EVM (R5F)
>   #@DESCRIPTION: Machine configuration for the TI AM62xx SIP EVM (R5F core)
>   
> -require conf/machine/include/k3r5.inc
> +require conf/machine/am62xx-unified-k3r5.conf
>   
> -SYSFW_SOC = "am62x"
> -SYSFW_CONFIG = "evm"
> -SYSFW_SUFFIX = "hs-fs"
> +UBOOT_CONFIG = "sip"
>   
> -UBOOT_MACHINE = "am62xsip_evm_r5_defconfig"
> +UBOOT_CONFIG:bsp-ti-6_1 = ""
>   
>   UBOOT_MACHINE:bsp-ti-6_1 = "am62x_evm_r5_defconfig"
>   
> diff --git a/meta-ti-bsp/conf/machine/am62xxsip-evm.conf b/meta-ti-bsp/conf/machine/am62xxsip-evm.conf
> index 8104329e..6b179b80 100644
> --- a/meta-ti-bsp/conf/machine/am62xxsip-evm.conf
> +++ b/meta-ti-bsp/conf/machine/am62xxsip-evm.conf
> @@ -2,7 +2,7 @@
>   #@NAME: AM62XX SIP EVM
>   #@DESCRIPTION: Machine configuration for the TI AM62XX SIP EVM
>   
> -require conf/machine/include/am62xx.inc
> +require conf/machine/am62xx-unified.conf
>   
>   KERNEL_DEVICETREE_PREFIX = " \
>       ti/k3-am625 \
> @@ -13,6 +13,4 @@ KERNEL_DEVICETREE = " \
>       ti/k3-am625-sk.dtb \
>   "
>   
> -UBOOT_MACHINE = "am62xsip_evm_a53_defconfig"
> -
> -UBOOT_MACHINE:bsp-ti-6_1 = "am62x_evm_a53_defconfig"
> +UBOOT_CONFIG = "sip"
> -- 
> 2.17.1
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#17931): https://urldefense.com/v3/__https://lists.yoctoproject.org/g/meta-ti/message/17931__;!!G3vK!U-r2MVyHaQGOtRj3aNhTjsDUYyItxHnzUGRG-G5JLIPNsN8lJvMKVSLNpaAab2D6lT8SjjhTdHlsGIgYCUbpGqaVKNaNnb30MQ$
> Mute This Topic: https://urldefense.com/v3/__https://lists.yoctoproject.org/mt/107796202/7030289__;!!G3vK!U-r2MVyHaQGOtRj3aNhTjsDUYyItxHnzUGRG-G5JLIPNsN8lJvMKVSLNpaAab2D6lT8SjjhTdHlsGIgYCUbpGqaVKNZDezDSJQ$
> Group Owner: meta-ti+owner@lists.yoctoproject.org
> Unsubscribe: https://urldefense.com/v3/__https://lists.yoctoproject.org/g/meta-ti/unsub__;!!G3vK!U-r2MVyHaQGOtRj3aNhTjsDUYyItxHnzUGRG-G5JLIPNsN8lJvMKVSLNpaAab2D6lT8SjjhTdHlsGIgYCUbpGqaVKNZtDaB-gQ$  [c-shilwant@ti.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
Ryan Eatmon Aug. 14, 2024, 7:15 p.m. UTC | #2
On 8/14/2024 1:09 PM, Chirag Shilwant wrote:
> Hi Ryan & Denys,
> On 09/08/24 00:57, Ryan Eatmon via lists.yoctoproject.org wrote:
>> To reduce the number of required builds for testing purposes, create a 
>> common unified am62xx machine target that can produce all of the files 
>> for all of the variant platforms related to am62xx (sip and lp). 
>> Additionally, continue to support
>> ZjQcmQRYFpfptBannerStart
>> This message was sent from outside of Texas Instruments.
>> Do not click links or open attachments unless you recognize the source 
>> of this email and know the content is safe. If you wish to report this 
>> message to IT Security, please forward the message as an attachment to 
>> phishing@list.ti.com
>> Report Suspicious
>> <https://us-phishalarm-ewt.proofpoint.com/EWT/v1/G3vK!uFdqXRfP9mwbKwZFHJNj_V92vi6mT9DGTpQZeFU0LNi8euAkYQM024IcX-dd54y6XIw8nwcGUnREacyk1hI$>
>> ZjQcmQRYFpfptBannerEnd
>> To reduce the number of required builds for testing purposes, create a
>> common unified am62xx machine target that can produce all of the files
>> for all of the variant platforms related to am62xx (sip and lp).
>> Additionally, continue to support building the specific target if
>> desired.
>>
>> Signed-off-by: Ryan Eatmon <reatmon@ti.com>
> 
> 
> 
> We need to see how do we enable the support of fragment configs with 
> UBOOT_CONFIG
> as I see atleast ti-u-boot-2024.04 still following the fragment config 
> model for few usecases.

I've been working on this topic all day today.  I'll probably have an 
additional RFC version tomorrow once I'm done testing it.

Once we are happy with everything we can decide if we want to push the 
solution to oe-core, or keep it in meta-ti.


> I see two approaches,
> 
> - Add the support in oe-core to handle u-boot fragment configs
> 
> - Extend u-boot-mergeconfig.inc we currently have in meta-ti [1] to 
> support UBOOT_CONFIG
> 
> 
> Personally, I would prefer the first option if that's acceptable to 
> oe-core as well
> but I would like to hear your opinions on the same
> 
> 
> [1]: 
> https://git.ti.com/cgit/arago-project/meta-ti/tree/meta-ti-bsp/recipes-bsp/u-boot/u-boot-mergeconfig.inc?h=scarthgap
> 
> 
> ---
> Chirag
> 
> 
> 
>> ---
>>   meta-ti-bsp/conf/machine/am62xx-evm-k3r5.conf |  9 +---
>>   meta-ti-bsp/conf/machine/am62xx-evm.conf      |  5 +--
>>   .../conf/machine/am62xx-lp-evm-k3r5.conf      |  9 +---
>>   meta-ti-bsp/conf/machine/am62xx-lp-evm.conf   |  5 +--
>>   .../conf/machine/am62xx-unified-k3r5.conf     | 20 +++++++++
>>   meta-ti-bsp/conf/machine/am62xx-unified.conf  | 43 +++++++++++++++++++
>>   .../conf/machine/am62xxsip-evm-k3r5.conf      |  8 ++--
>>   meta-ti-bsp/conf/machine/am62xxsip-evm.conf   |  6 +--
>>   8 files changed, 76 insertions(+), 29 deletions(-)
>>   create mode 100644 meta-ti-bsp/conf/machine/am62xx-unified-k3r5.conf
>>   create mode 100644 meta-ti-bsp/conf/machine/am62xx-unified.conf
>>
>> diff --git a/meta-ti-bsp/conf/machine/am62xx-evm-k3r5.conf 
>> b/meta-ti-bsp/conf/machine/am62xx-evm-k3r5.conf
>> index 548369ca..3a819835 100644
>> --- a/meta-ti-bsp/conf/machine/am62xx-evm-k3r5.conf
>> +++ b/meta-ti-bsp/conf/machine/am62xx-evm-k3r5.conf
>> @@ -2,11 +2,6 @@
>>   #@NAME: AM62XX EVM (R5F)
>>   #@DESCRIPTION: Machine configuration for the TI AM62xx EVM (R5F core)
>> -require conf/machine/include/k3r5.inc
>> -require conf/machine/include/ti-extras.inc
>> +require conf/machine/am62xx-unified-k3r5.conf
>> -SYSFW_SOC = "am62x"
>> -SYSFW_CONFIG = "evm"
>> -SYSFW_SUFFIX = "hs-fs"
>> -
>> -UBOOT_MACHINE = "am62x_evm_r5_defconfig"
>> +UBOOT_CONFIG = "evm"
>> diff --git a/meta-ti-bsp/conf/machine/am62xx-evm.conf 
>> b/meta-ti-bsp/conf/machine/am62xx-evm.conf
>> index 31de4a14..8c80cb1f 100644
>> --- a/meta-ti-bsp/conf/machine/am62xx-evm.conf
>> +++ b/meta-ti-bsp/conf/machine/am62xx-evm.conf
>> @@ -2,8 +2,7 @@
>>   #@NAME: AM62XX EVM
>>   #@DESCRIPTION: Machine configuration for the TI AM62XX EVM
>> -require conf/machine/include/am62xx.inc
>> -require conf/machine/include/ti-extras.inc
>> +require conf/machine/am62xx-unified.conf
>>   KERNEL_DEVICETREE_PREFIX = " \
>>       ti/k3-am625 \
>> @@ -30,4 +29,4 @@ KERNEL_DEVICETREE = " \
>>       ti/k3-am62x-sk-hdmi-audio.dtbo \
>>   "
>> -UBOOT_MACHINE = "am62x_evm_a53_defconfig"
>> +UBOOT_CONFIG = "evm"
>> diff --git a/meta-ti-bsp/conf/machine/am62xx-lp-evm-k3r5.conf 
>> b/meta-ti-bsp/conf/machine/am62xx-lp-evm-k3r5.conf
>> index 52b69a72..fb760e99 100644
>> --- a/meta-ti-bsp/conf/machine/am62xx-lp-evm-k3r5.conf
>> +++ b/meta-ti-bsp/conf/machine/am62xx-lp-evm-k3r5.conf
>> @@ -2,11 +2,6 @@
>>   #@NAME: AM62XX LP GP EVM (R5F)
>>   #@DESCRIPTION: Machine configuration for the TI AM62xx LP GP EVM 
>> (R5F core)
>> -require conf/machine/include/k3r5.inc
>> -require conf/machine/include/ti-extras.inc
>> +require conf/machine/am62xx-unified-k3r5.conf
>> -SYSFW_SOC = "am62x"
>> -SYSFW_CONFIG = "evm"
>> -SYSFW_SUFFIX = "hs-fs"
>> -
>> -UBOOT_MACHINE = "am62x_lpsk_r5_defconfig"
>> +UBOOT_CONFIG = "lpsk"
>> diff --git a/meta-ti-bsp/conf/machine/am62xx-lp-evm.conf 
>> b/meta-ti-bsp/conf/machine/am62xx-lp-evm.conf
>> index e9271f5b..f7d89d68 100644
>> --- a/meta-ti-bsp/conf/machine/am62xx-lp-evm.conf
>> +++ b/meta-ti-bsp/conf/machine/am62xx-lp-evm.conf
>> @@ -2,8 +2,7 @@
>>   #@NAME: AM62XX LP EVM
>>   #@DESCRIPTION: Machine configuration for the TI AM62XX LP EVM
>> -require conf/machine/include/am62xx.inc
>> -require conf/machine/include/ti-extras.inc
>> +require conf/machine/am62xx-unified.conf
>>   KERNEL_DEVICETREE_PREFIX = " \
>>       ti/k3-am62-lp \
>> @@ -19,4 +18,4 @@ KERNEL_DEVICETREE = " \
>>       ti/k3-am62x-sk-hdmi-audio.dtbo \
>>   "
>> -UBOOT_MACHINE = "am62x_lpsk_a53_defconfig"
>> +UBOOT_CONFIG = "lpsk"
>> diff --git a/meta-ti-bsp/conf/machine/am62xx-unified-k3r5.conf 
>> b/meta-ti-bsp/conf/machine/am62xx-unified-k3r5.conf
>> new file mode 100644
>> index 00000000..7d65e6af
>> --- /dev/null
>> +++ b/meta-ti-bsp/conf/machine/am62xx-unified-k3r5.conf
>> @@ -0,0 +1,20 @@
>> +#@TYPE: Machine
>> +#@NAME: AM62XX EVM (R5F)
>> +#@DESCRIPTION: Machine configuration for the TI AM62xx EVM (R5F core)
>> +
>> +require conf/machine/include/k3r5.inc
>> +require conf/machine/include/ti-extras.inc
>> +
>> +SYSFW_SOC = "am62x"
>> +SYSFW_CONFIG = "evm"
>> +SYSFW_SUFFIX = "hs-fs"
>> +
>> +UBOOT_MACHINE = ""
>> +# Last config in the list is default
>> +UBOOT_CONFIG ??= "sip lpsk evm"
>> +
>> +UBOOT_CONFIG:bsp-ti-6_1 ??= "evm"
>> +
>> +UBOOT_CONFIG[sip] = "am62xsip_evm_r5_defconfig"
>> +UBOOT_CONFIG[lpsk] = "am62x_lpsk_r5_defconfig"
>> +UBOOT_CONFIG[evm] = "am62x_evm_r5_defconfig"
>> diff --git a/meta-ti-bsp/conf/machine/am62xx-unified.conf 
>> b/meta-ti-bsp/conf/machine/am62xx-unified.conf
>> new file mode 100644
>> index 00000000..5b03239f
>> --- /dev/null
>> +++ b/meta-ti-bsp/conf/machine/am62xx-unified.conf
>> @@ -0,0 +1,43 @@
>> +#@TYPE: Machine
>> +#@NAME: AM62XX Unified EVM
>> +#@DESCRIPTION: Machine configuration for the TI AM62XX EVM and all of 
>> its variations
>> +
>> +require conf/machine/include/am62xx.inc
>> +require conf/machine/include/ti-extras.inc
>> +
>> +KERNEL_DEVICETREE_PREFIX = " \
>> +    ti/k3-am62-lp \
>> +    ti/k3-am625 \
>> +    ti/k3-am62x-sk \
>> +"
>> +
>> +KERNEL_DEVICETREE = " \
>> +    ti/k3-am62-lp-sk.dtb \
>> +    ti/k3-am625-beagleplay-csi2-ov5640.dtbo \
>> +    ti/k3-am625-beagleplay-csi2-tevi-ov5640.dtbo \
>> +    ti/k3-am625-beagleplay.dtb \
>> +    ti/k3-am625-phyboard-lyra-rdk.dtb \
>> +    ti/k3-am625-sk.dtb \
>> +    ti/k3-am625-verdin-nonwifi-dahlia.dtb \
>> +    ti/k3-am625-verdin-nonwifi-dev.dtb \
>> +    ti/k3-am625-verdin-nonwifi-mallow.dtb \
>> +    ti/k3-am625-verdin-nonwifi-yavia.dtb \
>> +    ti/k3-am625-verdin-wifi-dahlia.dtb \
>> +    ti/k3-am625-verdin-wifi-dev.dtb \
>> +    ti/k3-am625-verdin-wifi-mallow.dtb \
>> +    ti/k3-am625-verdin-wifi-yavia.dtb \
>> +    ti/k3-am62x-sk-csi2-imx219.dtbo \
>> +    ti/k3-am62x-sk-csi2-ov5640.dtbo \
>> +    ti/k3-am62x-sk-csi2-tevi-ov5640.dtbo \
>> +    ti/k3-am62x-sk-hdmi-audio.dtbo \
>> +"
>> +
>> +UBOOT_MACHINE = ""
>> +# Last config in the list is default
>> +UBOOT_CONFIG ??= "sip lpsk evm"
>> +
>> +UBOOT_CONFIG:bsp-ti-6_1 ??= "lpsk evm"
>> +
>> +UBOOT_CONFIG[sip] = "am62xsip_evm_a53_defconfig"
>> +UBOOT_CONFIG[lpsk] = "am62x_lpsk_a53_defconfig"
>> +UBOOT_CONFIG[evm] = "am62x_evm_a53_defconfig"
>> diff --git a/meta-ti-bsp/conf/machine/am62xxsip-evm-k3r5.conf 
>> b/meta-ti-bsp/conf/machine/am62xxsip-evm-k3r5.conf
>> index 55bc530b..ab538ff3 100644
>> --- a/meta-ti-bsp/conf/machine/am62xxsip-evm-k3r5.conf
>> +++ b/meta-ti-bsp/conf/machine/am62xxsip-evm-k3r5.conf
>> @@ -2,13 +2,11 @@
>>   #@NAME: AM62XX SIP EVM (R5F)
>>   #@DESCRIPTION: Machine configuration for the TI AM62xx SIP EVM (R5F 
>> core)
>> -require conf/machine/include/k3r5.inc
>> +require conf/machine/am62xx-unified-k3r5.conf
>> -SYSFW_SOC = "am62x"
>> -SYSFW_CONFIG = "evm"
>> -SYSFW_SUFFIX = "hs-fs"
>> +UBOOT_CONFIG = "sip"
>> -UBOOT_MACHINE = "am62xsip_evm_r5_defconfig"
>> +UBOOT_CONFIG:bsp-ti-6_1 = ""
>>   UBOOT_MACHINE:bsp-ti-6_1 = "am62x_evm_r5_defconfig"
>> diff --git a/meta-ti-bsp/conf/machine/am62xxsip-evm.conf 
>> b/meta-ti-bsp/conf/machine/am62xxsip-evm.conf
>> index 8104329e..6b179b80 100644
>> --- a/meta-ti-bsp/conf/machine/am62xxsip-evm.conf
>> +++ b/meta-ti-bsp/conf/machine/am62xxsip-evm.conf
>> @@ -2,7 +2,7 @@
>>   #@NAME: AM62XX SIP EVM
>>   #@DESCRIPTION: Machine configuration for the TI AM62XX SIP EVM
>> -require conf/machine/include/am62xx.inc
>> +require conf/machine/am62xx-unified.conf
>>   KERNEL_DEVICETREE_PREFIX = " \
>>       ti/k3-am625 \
>> @@ -13,6 +13,4 @@ KERNEL_DEVICETREE = " \
>>       ti/k3-am625-sk.dtb \
>>   "
>> -UBOOT_MACHINE = "am62xsip_evm_a53_defconfig"
>> -
>> -UBOOT_MACHINE:bsp-ti-6_1 = "am62x_evm_a53_defconfig"
>> +UBOOT_CONFIG = "sip"
>> -- 
>> 2.17.1
>>
>>
>> -=-=-=-=-=-=-=-=-=-=-=-
>> Links: You receive all messages sent to this group.
>> View/Reply Online (#17931): 
>> https://urldefense.com/v3/__https://lists.yoctoproject.org/g/meta-ti/message/17931__;!!G3vK!U-r2MVyHaQGOtRj3aNhTjsDUYyItxHnzUGRG-G5JLIPNsN8lJvMKVSLNpaAab2D6lT8SjjhTdHlsGIgYCUbpGqaVKNaNnb30MQ$
>> Mute This Topic: 
>> https://urldefense.com/v3/__https://lists.yoctoproject.org/mt/107796202/7030289__;!!G3vK!U-r2MVyHaQGOtRj3aNhTjsDUYyItxHnzUGRG-G5JLIPNsN8lJvMKVSLNpaAab2D6lT8SjjhTdHlsGIgYCUbpGqaVKNZDezDSJQ$
>> Group Owner: meta-ti+owner@lists.yoctoproject.org
>> Unsubscribe: 
>> https://urldefense.com/v3/__https://lists.yoctoproject.org/g/meta-ti/unsub__;!!G3vK!U-r2MVyHaQGOtRj3aNhTjsDUYyItxHnzUGRG-G5JLIPNsN8lJvMKVSLNpaAab2D6lT8SjjhTdHlsGIgYCUbpGqaVKNZtDaB-gQ$  [c-shilwant@ti.com]
>> -=-=-=-=-=-=-=-=-=-=-=-
>>
Ryan Eatmon Aug. 14, 2024, 7:23 p.m. UTC | #3
On 8/14/2024 2:15 PM, Ryan Eatmon via lists.yoctoproject.org wrote:
> 
> 
> On 8/14/2024 1:09 PM, Chirag Shilwant wrote:
>> Hi Ryan & Denys,
>> On 09/08/24 00:57, Ryan Eatmon via lists.yoctoproject.org wrote:
>>> To reduce the number of required builds for testing purposes, create 
>>> a common unified am62xx machine target that can produce all of the 
>>> files for all of the variant platforms related to am62xx (sip and 
>>> lp). Additionally, continue to support
>>> ZjQcmQRYFpfptBannerStart
>>> This message was sent from outside of Texas Instruments.
>>> Do not click links or open attachments unless you recognize the 
>>> source of this email and know the content is safe. If you wish to 
>>> report this message to IT Security, please forward the message as an 
>>> attachment to phishing@list.ti.com
>>> Report Suspicious
>>> <https://us-phishalarm-ewt.proofpoint.com/EWT/v1/G3vK!uFdqXRfP9mwbKwZFHJNj_V92vi6mT9DGTpQZeFU0LNi8euAkYQM024IcX-dd54y6XIw8nwcGUnREacyk1hI$>
>>> ZjQcmQRYFpfptBannerEnd
>>> To reduce the number of required builds for testing purposes, create a
>>> common unified am62xx machine target that can produce all of the files
>>> for all of the variant platforms related to am62xx (sip and lp).
>>> Additionally, continue to support building the specific target if
>>> desired.
>>>
>>> Signed-off-by: Ryan Eatmon <reatmon@ti.com>
>>
>>
>>
>> We need to see how do we enable the support of fragment configs with 
>> UBOOT_CONFIG
>> as I see atleast ti-u-boot-2024.04 still following the fragment config 
>> model for few usecases.
> 
> I've been working on this topic all day today.  I'll probably have an 
> additional RFC version tomorrow once I'm done testing it.
> 
> Once we are happy with everything we can decide if we want to push the 
> solution to oe-core, or keep it in meta-ti.

But to be fair, there are no fragments in the am62xx family.  There were 
back under the 6.1 bsp, but under 6.6 the fragment it is gone.

> 
>> I see two approaches,
>>
>> - Add the support in oe-core to handle u-boot fragment configs
>>
>> - Extend u-boot-mergeconfig.inc we currently have in meta-ti [1] to 
>> support UBOOT_CONFIG
>>
>>
>> Personally, I would prefer the first option if that's acceptable to 
>> oe-core as well
>> but I would like to hear your opinions on the same
>>
>>
>> [1]: 
>> https://git.ti.com/cgit/arago-project/meta-ti/tree/meta-ti-bsp/recipes-bsp/u-boot/u-boot-mergeconfig.inc?h=scarthgap
>>
>>
>> ---
>> Chirag
>>
>>
>>
>>> ---
>>>   meta-ti-bsp/conf/machine/am62xx-evm-k3r5.conf |  9 +---
>>>   meta-ti-bsp/conf/machine/am62xx-evm.conf      |  5 +--
>>>   .../conf/machine/am62xx-lp-evm-k3r5.conf      |  9 +---
>>>   meta-ti-bsp/conf/machine/am62xx-lp-evm.conf   |  5 +--
>>>   .../conf/machine/am62xx-unified-k3r5.conf     | 20 +++++++++
>>>   meta-ti-bsp/conf/machine/am62xx-unified.conf  | 43 +++++++++++++++++++
>>>   .../conf/machine/am62xxsip-evm-k3r5.conf      |  8 ++--
>>>   meta-ti-bsp/conf/machine/am62xxsip-evm.conf   |  6 +--
>>>   8 files changed, 76 insertions(+), 29 deletions(-)
>>>   create mode 100644 meta-ti-bsp/conf/machine/am62xx-unified-k3r5.conf
>>>   create mode 100644 meta-ti-bsp/conf/machine/am62xx-unified.conf
>>>
>>> diff --git a/meta-ti-bsp/conf/machine/am62xx-evm-k3r5.conf 
>>> b/meta-ti-bsp/conf/machine/am62xx-evm-k3r5.conf
>>> index 548369ca..3a819835 100644
>>> --- a/meta-ti-bsp/conf/machine/am62xx-evm-k3r5.conf
>>> +++ b/meta-ti-bsp/conf/machine/am62xx-evm-k3r5.conf
>>> @@ -2,11 +2,6 @@
>>>   #@NAME: AM62XX EVM (R5F)
>>>   #@DESCRIPTION: Machine configuration for the TI AM62xx EVM (R5F core)
>>> -require conf/machine/include/k3r5.inc
>>> -require conf/machine/include/ti-extras.inc
>>> +require conf/machine/am62xx-unified-k3r5.conf
>>> -SYSFW_SOC = "am62x"
>>> -SYSFW_CONFIG = "evm"
>>> -SYSFW_SUFFIX = "hs-fs"
>>> -
>>> -UBOOT_MACHINE = "am62x_evm_r5_defconfig"
>>> +UBOOT_CONFIG = "evm"
>>> diff --git a/meta-ti-bsp/conf/machine/am62xx-evm.conf 
>>> b/meta-ti-bsp/conf/machine/am62xx-evm.conf
>>> index 31de4a14..8c80cb1f 100644
>>> --- a/meta-ti-bsp/conf/machine/am62xx-evm.conf
>>> +++ b/meta-ti-bsp/conf/machine/am62xx-evm.conf
>>> @@ -2,8 +2,7 @@
>>>   #@NAME: AM62XX EVM
>>>   #@DESCRIPTION: Machine configuration for the TI AM62XX EVM
>>> -require conf/machine/include/am62xx.inc
>>> -require conf/machine/include/ti-extras.inc
>>> +require conf/machine/am62xx-unified.conf
>>>   KERNEL_DEVICETREE_PREFIX = " \
>>>       ti/k3-am625 \
>>> @@ -30,4 +29,4 @@ KERNEL_DEVICETREE = " \
>>>       ti/k3-am62x-sk-hdmi-audio.dtbo \
>>>   "
>>> -UBOOT_MACHINE = "am62x_evm_a53_defconfig"
>>> +UBOOT_CONFIG = "evm"
>>> diff --git a/meta-ti-bsp/conf/machine/am62xx-lp-evm-k3r5.conf 
>>> b/meta-ti-bsp/conf/machine/am62xx-lp-evm-k3r5.conf
>>> index 52b69a72..fb760e99 100644
>>> --- a/meta-ti-bsp/conf/machine/am62xx-lp-evm-k3r5.conf
>>> +++ b/meta-ti-bsp/conf/machine/am62xx-lp-evm-k3r5.conf
>>> @@ -2,11 +2,6 @@
>>>   #@NAME: AM62XX LP GP EVM (R5F)
>>>   #@DESCRIPTION: Machine configuration for the TI AM62xx LP GP EVM 
>>> (R5F core)
>>> -require conf/machine/include/k3r5.inc
>>> -require conf/machine/include/ti-extras.inc
>>> +require conf/machine/am62xx-unified-k3r5.conf
>>> -SYSFW_SOC = "am62x"
>>> -SYSFW_CONFIG = "evm"
>>> -SYSFW_SUFFIX = "hs-fs"
>>> -
>>> -UBOOT_MACHINE = "am62x_lpsk_r5_defconfig"
>>> +UBOOT_CONFIG = "lpsk"
>>> diff --git a/meta-ti-bsp/conf/machine/am62xx-lp-evm.conf 
>>> b/meta-ti-bsp/conf/machine/am62xx-lp-evm.conf
>>> index e9271f5b..f7d89d68 100644
>>> --- a/meta-ti-bsp/conf/machine/am62xx-lp-evm.conf
>>> +++ b/meta-ti-bsp/conf/machine/am62xx-lp-evm.conf
>>> @@ -2,8 +2,7 @@
>>>   #@NAME: AM62XX LP EVM
>>>   #@DESCRIPTION: Machine configuration for the TI AM62XX LP EVM
>>> -require conf/machine/include/am62xx.inc
>>> -require conf/machine/include/ti-extras.inc
>>> +require conf/machine/am62xx-unified.conf
>>>   KERNEL_DEVICETREE_PREFIX = " \
>>>       ti/k3-am62-lp \
>>> @@ -19,4 +18,4 @@ KERNEL_DEVICETREE = " \
>>>       ti/k3-am62x-sk-hdmi-audio.dtbo \
>>>   "
>>> -UBOOT_MACHINE = "am62x_lpsk_a53_defconfig"
>>> +UBOOT_CONFIG = "lpsk"
>>> diff --git a/meta-ti-bsp/conf/machine/am62xx-unified-k3r5.conf 
>>> b/meta-ti-bsp/conf/machine/am62xx-unified-k3r5.conf
>>> new file mode 100644
>>> index 00000000..7d65e6af
>>> --- /dev/null
>>> +++ b/meta-ti-bsp/conf/machine/am62xx-unified-k3r5.conf
>>> @@ -0,0 +1,20 @@
>>> +#@TYPE: Machine
>>> +#@NAME: AM62XX EVM (R5F)
>>> +#@DESCRIPTION: Machine configuration for the TI AM62xx EVM (R5F core)
>>> +
>>> +require conf/machine/include/k3r5.inc
>>> +require conf/machine/include/ti-extras.inc
>>> +
>>> +SYSFW_SOC = "am62x"
>>> +SYSFW_CONFIG = "evm"
>>> +SYSFW_SUFFIX = "hs-fs"
>>> +
>>> +UBOOT_MACHINE = ""
>>> +# Last config in the list is default
>>> +UBOOT_CONFIG ??= "sip lpsk evm"
>>> +
>>> +UBOOT_CONFIG:bsp-ti-6_1 ??= "evm"
>>> +
>>> +UBOOT_CONFIG[sip] = "am62xsip_evm_r5_defconfig"
>>> +UBOOT_CONFIG[lpsk] = "am62x_lpsk_r5_defconfig"
>>> +UBOOT_CONFIG[evm] = "am62x_evm_r5_defconfig"
>>> diff --git a/meta-ti-bsp/conf/machine/am62xx-unified.conf 
>>> b/meta-ti-bsp/conf/machine/am62xx-unified.conf
>>> new file mode 100644
>>> index 00000000..5b03239f
>>> --- /dev/null
>>> +++ b/meta-ti-bsp/conf/machine/am62xx-unified.conf
>>> @@ -0,0 +1,43 @@
>>> +#@TYPE: Machine
>>> +#@NAME: AM62XX Unified EVM
>>> +#@DESCRIPTION: Machine configuration for the TI AM62XX EVM and all 
>>> of its variations
>>> +
>>> +require conf/machine/include/am62xx.inc
>>> +require conf/machine/include/ti-extras.inc
>>> +
>>> +KERNEL_DEVICETREE_PREFIX = " \
>>> +    ti/k3-am62-lp \
>>> +    ti/k3-am625 \
>>> +    ti/k3-am62x-sk \
>>> +"
>>> +
>>> +KERNEL_DEVICETREE = " \
>>> +    ti/k3-am62-lp-sk.dtb \
>>> +    ti/k3-am625-beagleplay-csi2-ov5640.dtbo \
>>> +    ti/k3-am625-beagleplay-csi2-tevi-ov5640.dtbo \
>>> +    ti/k3-am625-beagleplay.dtb \
>>> +    ti/k3-am625-phyboard-lyra-rdk.dtb \
>>> +    ti/k3-am625-sk.dtb \
>>> +    ti/k3-am625-verdin-nonwifi-dahlia.dtb \
>>> +    ti/k3-am625-verdin-nonwifi-dev.dtb \
>>> +    ti/k3-am625-verdin-nonwifi-mallow.dtb \
>>> +    ti/k3-am625-verdin-nonwifi-yavia.dtb \
>>> +    ti/k3-am625-verdin-wifi-dahlia.dtb \
>>> +    ti/k3-am625-verdin-wifi-dev.dtb \
>>> +    ti/k3-am625-verdin-wifi-mallow.dtb \
>>> +    ti/k3-am625-verdin-wifi-yavia.dtb \
>>> +    ti/k3-am62x-sk-csi2-imx219.dtbo \
>>> +    ti/k3-am62x-sk-csi2-ov5640.dtbo \
>>> +    ti/k3-am62x-sk-csi2-tevi-ov5640.dtbo \
>>> +    ti/k3-am62x-sk-hdmi-audio.dtbo \
>>> +"
>>> +
>>> +UBOOT_MACHINE = ""
>>> +# Last config in the list is default
>>> +UBOOT_CONFIG ??= "sip lpsk evm"
>>> +
>>> +UBOOT_CONFIG:bsp-ti-6_1 ??= "lpsk evm"
>>> +
>>> +UBOOT_CONFIG[sip] = "am62xsip_evm_a53_defconfig"
>>> +UBOOT_CONFIG[lpsk] = "am62x_lpsk_a53_defconfig"
>>> +UBOOT_CONFIG[evm] = "am62x_evm_a53_defconfig"
>>> diff --git a/meta-ti-bsp/conf/machine/am62xxsip-evm-k3r5.conf 
>>> b/meta-ti-bsp/conf/machine/am62xxsip-evm-k3r5.conf
>>> index 55bc530b..ab538ff3 100644
>>> --- a/meta-ti-bsp/conf/machine/am62xxsip-evm-k3r5.conf
>>> +++ b/meta-ti-bsp/conf/machine/am62xxsip-evm-k3r5.conf
>>> @@ -2,13 +2,11 @@
>>>   #@NAME: AM62XX SIP EVM (R5F)
>>>   #@DESCRIPTION: Machine configuration for the TI AM62xx SIP EVM (R5F 
>>> core)
>>> -require conf/machine/include/k3r5.inc
>>> +require conf/machine/am62xx-unified-k3r5.conf
>>> -SYSFW_SOC = "am62x"
>>> -SYSFW_CONFIG = "evm"
>>> -SYSFW_SUFFIX = "hs-fs"
>>> +UBOOT_CONFIG = "sip"
>>> -UBOOT_MACHINE = "am62xsip_evm_r5_defconfig"
>>> +UBOOT_CONFIG:bsp-ti-6_1 = ""
>>>   UBOOT_MACHINE:bsp-ti-6_1 = "am62x_evm_r5_defconfig"
>>> diff --git a/meta-ti-bsp/conf/machine/am62xxsip-evm.conf 
>>> b/meta-ti-bsp/conf/machine/am62xxsip-evm.conf
>>> index 8104329e..6b179b80 100644
>>> --- a/meta-ti-bsp/conf/machine/am62xxsip-evm.conf
>>> +++ b/meta-ti-bsp/conf/machine/am62xxsip-evm.conf
>>> @@ -2,7 +2,7 @@
>>>   #@NAME: AM62XX SIP EVM
>>>   #@DESCRIPTION: Machine configuration for the TI AM62XX SIP EVM
>>> -require conf/machine/include/am62xx.inc
>>> +require conf/machine/am62xx-unified.conf
>>>   KERNEL_DEVICETREE_PREFIX = " \
>>>       ti/k3-am625 \
>>> @@ -13,6 +13,4 @@ KERNEL_DEVICETREE = " \
>>>       ti/k3-am625-sk.dtb \
>>>   "
>>> -UBOOT_MACHINE = "am62xsip_evm_a53_defconfig"
>>> -
>>> -UBOOT_MACHINE:bsp-ti-6_1 = "am62x_evm_a53_defconfig"
>>> +UBOOT_CONFIG = "sip"
>>> -- 
>>> 2.17.1
>>>
>>>
>>>
>>>
> 
> 
> 
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#17937): https://lists.yoctoproject.org/g/meta-ti/message/17937
> Mute This Topic: https://lists.yoctoproject.org/mt/107796202/6551054
> Group Owner: meta-ti+owner@lists.yoctoproject.org
> Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub [reatmon@ti.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
Chirag Shilwant Aug. 14, 2024, 9:15 p.m. UTC | #4
On 15/08/24 00:53, Ryan Eatmon wrote:
>
>
> On 8/14/2024 2:15 PM, Ryan Eatmon via lists.yoctoproject.org wrote:
>>
>>
>> On 8/14/2024 1:09 PM, Chirag Shilwant wrote:
>>> Hi Ryan & Denys,
>>> On 09/08/24 00:57, Ryan Eatmon via lists.yoctoproject.org wrote:
>>>> To reduce the number of required builds for testing purposes, 
>>>> create a common unified am62xx machine target that can produce all 
>>>> of the files for all of the variant platforms related to am62xx 
>>>> (sip and lp). Additionally, continue to support
>>>> ZjQcmQRYFpfptBannerStart
>>>> This message was sent from outside of Texas Instruments.
>>>> Do not click links or open attachments unless you recognize the 
>>>> source of this email and know the content is safe. If you wish to 
>>>> report this message to IT Security, please forward the message as 
>>>> an attachment to phishing@list.ti.com
>>>> Report Suspicious
>>>> <https://us-phishalarm-ewt.proofpoint.com/EWT/v1/G3vK!uFdqXRfP9mwbKwZFHJNj_V92vi6mT9DGTpQZeFU0LNi8euAkYQM024IcX-dd54y6XIw8nwcGUnREacyk1hI$> 
>>>>
>>>> ZjQcmQRYFpfptBannerEnd
>>>> To reduce the number of required builds for testing purposes, create a
>>>> common unified am62xx machine target that can produce all of the files
>>>> for all of the variant platforms related to am62xx (sip and lp).
>>>> Additionally, continue to support building the specific target if
>>>> desired.
>>>>
>>>> Signed-off-by: Ryan Eatmon <reatmon@ti.com>
>>>
>>>
>>>
>>> We need to see how do we enable the support of fragment configs with 
>>> UBOOT_CONFIG
>>> as I see atleast ti-u-boot-2024.04 still following the fragment 
>>> config model for few usecases.
>>
>> I've been working on this topic all day today.  I'll probably have an 
>> additional RFC version tomorrow once I'm done testing it.
>>
>> Once we are happy with everything we can decide if we want to push 
>> the solution to oe-core, or keep it in meta-ti.
>
> But to be fair, there are no fragments in the am62xx family. There 
> were back under the 6.1 bsp, but under 6.6 the fragment it is gone.



 From Yocto pov that's true but again not for ti-u-boot-2024.04 perspective.
We still have many fragment configs in u-boot tree for am62xx family.
For Instance - am62x_a53_splashscreen.config, am62x_a53_usbdfu.config, 
etc [1]

Though they currently aren't in default machine configurations of meta-ti
for am62xx family but we never know what happens in future.
Also, not supporting u-boot fragments via meta-ti would eventually need a
signup from baseport team to never use it. Which means the 6.1 bsp flow,
where 62sip, 62lp had fragment model shouldn't be prefered by ti-u-boot
in future.


Praneeth/ Vignesh,

Correct me if I'm wrong but AFAIR, fragments where added in 6.1 bsp based
on upstream feedbacks & were removed from 6.6 bsp for the same reason.
So, we can't rule out the possibility of that fragments being added back
to default flow in future LTS.


[1]: 
https://git.ti.com/cgit/ti-u-boot/ti-u-boot/tree/configs?h=ti-u-boot-2024.04

---
Chirag



>
>
>>
>>> I see two approaches,
>>>
>>> - Add the support in oe-core to handle u-boot fragment configs
>>>
>>> - Extend u-boot-mergeconfig.inc we currently have in meta-ti [1] to 
>>> support UBOOT_CONFIG
>>>
>>>
>>> Personally, I would prefer the first option if that's acceptable to 
>>> oe-core as well
>>> but I would like to hear your opinions on the same
>>>
>>>
>>> [1]: 
>>> https://git.ti.com/cgit/arago-project/meta-ti/tree/meta-ti-bsp/recipes-bsp/u-boot/u-boot-mergeconfig.inc?h=scarthgap
>>>
>>>
>>> ---
>>> Chirag
>>>
>>>
>>>
>>>> ---
>>>>   meta-ti-bsp/conf/machine/am62xx-evm-k3r5.conf |  9 +---
>>>>   meta-ti-bsp/conf/machine/am62xx-evm.conf      |  5 +--
>>>>   .../conf/machine/am62xx-lp-evm-k3r5.conf      |  9 +---
>>>>   meta-ti-bsp/conf/machine/am62xx-lp-evm.conf   |  5 +--
>>>>   .../conf/machine/am62xx-unified-k3r5.conf     | 20 +++++++++
>>>>   meta-ti-bsp/conf/machine/am62xx-unified.conf  | 43 
>>>> +++++++++++++++++++
>>>>   .../conf/machine/am62xxsip-evm-k3r5.conf      |  8 ++--
>>>>   meta-ti-bsp/conf/machine/am62xxsip-evm.conf   |  6 +--
>>>>   8 files changed, 76 insertions(+), 29 deletions(-)
>>>>   create mode 100644 meta-ti-bsp/conf/machine/am62xx-unified-k3r5.conf
>>>>   create mode 100644 meta-ti-bsp/conf/machine/am62xx-unified.conf
>>>>
>>>> diff --git a/meta-ti-bsp/conf/machine/am62xx-evm-k3r5.conf 
>>>> b/meta-ti-bsp/conf/machine/am62xx-evm-k3r5.conf
>>>> index 548369ca..3a819835 100644
>>>> --- a/meta-ti-bsp/conf/machine/am62xx-evm-k3r5.conf
>>>> +++ b/meta-ti-bsp/conf/machine/am62xx-evm-k3r5.conf
>>>> @@ -2,11 +2,6 @@
>>>>   #@NAME: AM62XX EVM (R5F)
>>>>   #@DESCRIPTION: Machine configuration for the TI AM62xx EVM (R5F 
>>>> core)
>>>> -require conf/machine/include/k3r5.inc
>>>> -require conf/machine/include/ti-extras.inc
>>>> +require conf/machine/am62xx-unified-k3r5.conf
>>>> -SYSFW_SOC = "am62x"
>>>> -SYSFW_CONFIG = "evm"
>>>> -SYSFW_SUFFIX = "hs-fs"
>>>> -
>>>> -UBOOT_MACHINE = "am62x_evm_r5_defconfig"
>>>> +UBOOT_CONFIG = "evm"
>>>> diff --git a/meta-ti-bsp/conf/machine/am62xx-evm.conf 
>>>> b/meta-ti-bsp/conf/machine/am62xx-evm.conf
>>>> index 31de4a14..8c80cb1f 100644
>>>> --- a/meta-ti-bsp/conf/machine/am62xx-evm.conf
>>>> +++ b/meta-ti-bsp/conf/machine/am62xx-evm.conf
>>>> @@ -2,8 +2,7 @@
>>>>   #@NAME: AM62XX EVM
>>>>   #@DESCRIPTION: Machine configuration for the TI AM62XX EVM
>>>> -require conf/machine/include/am62xx.inc
>>>> -require conf/machine/include/ti-extras.inc
>>>> +require conf/machine/am62xx-unified.conf
>>>>   KERNEL_DEVICETREE_PREFIX = " \
>>>>       ti/k3-am625 \
>>>> @@ -30,4 +29,4 @@ KERNEL_DEVICETREE = " \
>>>>       ti/k3-am62x-sk-hdmi-audio.dtbo \
>>>>   "
>>>> -UBOOT_MACHINE = "am62x_evm_a53_defconfig"
>>>> +UBOOT_CONFIG = "evm"
>>>> diff --git a/meta-ti-bsp/conf/machine/am62xx-lp-evm-k3r5.conf 
>>>> b/meta-ti-bsp/conf/machine/am62xx-lp-evm-k3r5.conf
>>>> index 52b69a72..fb760e99 100644
>>>> --- a/meta-ti-bsp/conf/machine/am62xx-lp-evm-k3r5.conf
>>>> +++ b/meta-ti-bsp/conf/machine/am62xx-lp-evm-k3r5.conf
>>>> @@ -2,11 +2,6 @@
>>>>   #@NAME: AM62XX LP GP EVM (R5F)
>>>>   #@DESCRIPTION: Machine configuration for the TI AM62xx LP GP EVM 
>>>> (R5F core)
>>>> -require conf/machine/include/k3r5.inc
>>>> -require conf/machine/include/ti-extras.inc
>>>> +require conf/machine/am62xx-unified-k3r5.conf
>>>> -SYSFW_SOC = "am62x"
>>>> -SYSFW_CONFIG = "evm"
>>>> -SYSFW_SUFFIX = "hs-fs"
>>>> -
>>>> -UBOOT_MACHINE = "am62x_lpsk_r5_defconfig"
>>>> +UBOOT_CONFIG = "lpsk"
>>>> diff --git a/meta-ti-bsp/conf/machine/am62xx-lp-evm.conf 
>>>> b/meta-ti-bsp/conf/machine/am62xx-lp-evm.conf
>>>> index e9271f5b..f7d89d68 100644
>>>> --- a/meta-ti-bsp/conf/machine/am62xx-lp-evm.conf
>>>> +++ b/meta-ti-bsp/conf/machine/am62xx-lp-evm.conf
>>>> @@ -2,8 +2,7 @@
>>>>   #@NAME: AM62XX LP EVM
>>>>   #@DESCRIPTION: Machine configuration for the TI AM62XX LP EVM
>>>> -require conf/machine/include/am62xx.inc
>>>> -require conf/machine/include/ti-extras.inc
>>>> +require conf/machine/am62xx-unified.conf
>>>>   KERNEL_DEVICETREE_PREFIX = " \
>>>>       ti/k3-am62-lp \
>>>> @@ -19,4 +18,4 @@ KERNEL_DEVICETREE = " \
>>>>       ti/k3-am62x-sk-hdmi-audio.dtbo \
>>>>   "
>>>> -UBOOT_MACHINE = "am62x_lpsk_a53_defconfig"
>>>> +UBOOT_CONFIG = "lpsk"
>>>> diff --git a/meta-ti-bsp/conf/machine/am62xx-unified-k3r5.conf 
>>>> b/meta-ti-bsp/conf/machine/am62xx-unified-k3r5.conf
>>>> new file mode 100644
>>>> index 00000000..7d65e6af
>>>> --- /dev/null
>>>> +++ b/meta-ti-bsp/conf/machine/am62xx-unified-k3r5.conf
>>>> @@ -0,0 +1,20 @@
>>>> +#@TYPE: Machine
>>>> +#@NAME: AM62XX EVM (R5F)
>>>> +#@DESCRIPTION: Machine configuration for the TI AM62xx EVM (R5F core)
>>>> +
>>>> +require conf/machine/include/k3r5.inc
>>>> +require conf/machine/include/ti-extras.inc
>>>> +
>>>> +SYSFW_SOC = "am62x"
>>>> +SYSFW_CONFIG = "evm"
>>>> +SYSFW_SUFFIX = "hs-fs"
>>>> +
>>>> +UBOOT_MACHINE = ""
>>>> +# Last config in the list is default
>>>> +UBOOT_CONFIG ??= "sip lpsk evm"
>>>> +
>>>> +UBOOT_CONFIG:bsp-ti-6_1 ??= "evm"
>>>> +
>>>> +UBOOT_CONFIG[sip] = "am62xsip_evm_r5_defconfig"
>>>> +UBOOT_CONFIG[lpsk] = "am62x_lpsk_r5_defconfig"
>>>> +UBOOT_CONFIG[evm] = "am62x_evm_r5_defconfig"
>>>> diff --git a/meta-ti-bsp/conf/machine/am62xx-unified.conf 
>>>> b/meta-ti-bsp/conf/machine/am62xx-unified.conf
>>>> new file mode 100644
>>>> index 00000000..5b03239f
>>>> --- /dev/null
>>>> +++ b/meta-ti-bsp/conf/machine/am62xx-unified.conf
>>>> @@ -0,0 +1,43 @@
>>>> +#@TYPE: Machine
>>>> +#@NAME: AM62XX Unified EVM
>>>> +#@DESCRIPTION: Machine configuration for the TI AM62XX EVM and all 
>>>> of its variations
>>>> +
>>>> +require conf/machine/include/am62xx.inc
>>>> +require conf/machine/include/ti-extras.inc
>>>> +
>>>> +KERNEL_DEVICETREE_PREFIX = " \
>>>> +    ti/k3-am62-lp \
>>>> +    ti/k3-am625 \
>>>> +    ti/k3-am62x-sk \
>>>> +"
>>>> +
>>>> +KERNEL_DEVICETREE = " \
>>>> +    ti/k3-am62-lp-sk.dtb \
>>>> +    ti/k3-am625-beagleplay-csi2-ov5640.dtbo \
>>>> +    ti/k3-am625-beagleplay-csi2-tevi-ov5640.dtbo \
>>>> +    ti/k3-am625-beagleplay.dtb \
>>>> +    ti/k3-am625-phyboard-lyra-rdk.dtb \
>>>> +    ti/k3-am625-sk.dtb \
>>>> +    ti/k3-am625-verdin-nonwifi-dahlia.dtb \
>>>> +    ti/k3-am625-verdin-nonwifi-dev.dtb \
>>>> +    ti/k3-am625-verdin-nonwifi-mallow.dtb \
>>>> +    ti/k3-am625-verdin-nonwifi-yavia.dtb \
>>>> +    ti/k3-am625-verdin-wifi-dahlia.dtb \
>>>> +    ti/k3-am625-verdin-wifi-dev.dtb \
>>>> +    ti/k3-am625-verdin-wifi-mallow.dtb \
>>>> +    ti/k3-am625-verdin-wifi-yavia.dtb \
>>>> +    ti/k3-am62x-sk-csi2-imx219.dtbo \
>>>> +    ti/k3-am62x-sk-csi2-ov5640.dtbo \
>>>> +    ti/k3-am62x-sk-csi2-tevi-ov5640.dtbo \
>>>> +    ti/k3-am62x-sk-hdmi-audio.dtbo \
>>>> +"
>>>> +
>>>> +UBOOT_MACHINE = ""
>>>> +# Last config in the list is default
>>>> +UBOOT_CONFIG ??= "sip lpsk evm"
>>>> +
>>>> +UBOOT_CONFIG:bsp-ti-6_1 ??= "lpsk evm"
>>>> +
>>>> +UBOOT_CONFIG[sip] = "am62xsip_evm_a53_defconfig"
>>>> +UBOOT_CONFIG[lpsk] = "am62x_lpsk_a53_defconfig"
>>>> +UBOOT_CONFIG[evm] = "am62x_evm_a53_defconfig"
>>>> diff --git a/meta-ti-bsp/conf/machine/am62xxsip-evm-k3r5.conf 
>>>> b/meta-ti-bsp/conf/machine/am62xxsip-evm-k3r5.conf
>>>> index 55bc530b..ab538ff3 100644
>>>> --- a/meta-ti-bsp/conf/machine/am62xxsip-evm-k3r5.conf
>>>> +++ b/meta-ti-bsp/conf/machine/am62xxsip-evm-k3r5.conf
>>>> @@ -2,13 +2,11 @@
>>>>   #@NAME: AM62XX SIP EVM (R5F)
>>>>   #@DESCRIPTION: Machine configuration for the TI AM62xx SIP EVM 
>>>> (R5F core)
>>>> -require conf/machine/include/k3r5.inc
>>>> +require conf/machine/am62xx-unified-k3r5.conf
>>>> -SYSFW_SOC = "am62x"
>>>> -SYSFW_CONFIG = "evm"
>>>> -SYSFW_SUFFIX = "hs-fs"
>>>> +UBOOT_CONFIG = "sip"
>>>> -UBOOT_MACHINE = "am62xsip_evm_r5_defconfig"
>>>> +UBOOT_CONFIG:bsp-ti-6_1 = ""
>>>>   UBOOT_MACHINE:bsp-ti-6_1 = "am62x_evm_r5_defconfig"
>>>> diff --git a/meta-ti-bsp/conf/machine/am62xxsip-evm.conf 
>>>> b/meta-ti-bsp/conf/machine/am62xxsip-evm.conf
>>>> index 8104329e..6b179b80 100644
>>>> --- a/meta-ti-bsp/conf/machine/am62xxsip-evm.conf
>>>> +++ b/meta-ti-bsp/conf/machine/am62xxsip-evm.conf
>>>> @@ -2,7 +2,7 @@
>>>>   #@NAME: AM62XX SIP EVM
>>>>   #@DESCRIPTION: Machine configuration for the TI AM62XX SIP EVM
>>>> -require conf/machine/include/am62xx.inc
>>>> +require conf/machine/am62xx-unified.conf
>>>>   KERNEL_DEVICETREE_PREFIX = " \
>>>>       ti/k3-am625 \
>>>> @@ -13,6 +13,4 @@ KERNEL_DEVICETREE = " \
>>>>       ti/k3-am625-sk.dtb \
>>>>   "
>>>> -UBOOT_MACHINE = "am62xsip_evm_a53_defconfig"
>>>> -
>>>> -UBOOT_MACHINE:bsp-ti-6_1 = "am62x_evm_a53_defconfig"
>>>> +UBOOT_CONFIG = "sip"
>>>> -- 
>>>> 2.17.1
>>>>
>>>>
>>>>
>>>>
>>
>>
>>
>> -=-=-=-=-=-=-=-=-=-=-=-
>> Links: You receive all messages sent to this group.
>> View/Reply Online (#17937): 
>> https://lists.yoctoproject.org/g/meta-ti/message/17937
>> Mute This Topic: https://lists.yoctoproject.org/mt/107796202/6551054
>> Group Owner: meta-ti+owner@lists.yoctoproject.org
>> Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub 
>> [reatmon@ti.com]
>> -=-=-=-=-=-=-=-=-=-=-=-
>>
>
diff mbox series

Patch

diff --git a/meta-ti-bsp/conf/machine/am62xx-evm-k3r5.conf b/meta-ti-bsp/conf/machine/am62xx-evm-k3r5.conf
index 548369ca..3a819835 100644
--- a/meta-ti-bsp/conf/machine/am62xx-evm-k3r5.conf
+++ b/meta-ti-bsp/conf/machine/am62xx-evm-k3r5.conf
@@ -2,11 +2,6 @@ 
 #@NAME: AM62XX EVM (R5F)
 #@DESCRIPTION: Machine configuration for the TI AM62xx EVM (R5F core)
 
-require conf/machine/include/k3r5.inc
-require conf/machine/include/ti-extras.inc
+require conf/machine/am62xx-unified-k3r5.conf
 
-SYSFW_SOC = "am62x"
-SYSFW_CONFIG = "evm"
-SYSFW_SUFFIX = "hs-fs"
-
-UBOOT_MACHINE = "am62x_evm_r5_defconfig"
+UBOOT_CONFIG = "evm"
diff --git a/meta-ti-bsp/conf/machine/am62xx-evm.conf b/meta-ti-bsp/conf/machine/am62xx-evm.conf
index 31de4a14..8c80cb1f 100644
--- a/meta-ti-bsp/conf/machine/am62xx-evm.conf
+++ b/meta-ti-bsp/conf/machine/am62xx-evm.conf
@@ -2,8 +2,7 @@ 
 #@NAME: AM62XX EVM
 #@DESCRIPTION: Machine configuration for the TI AM62XX EVM
 
-require conf/machine/include/am62xx.inc
-require conf/machine/include/ti-extras.inc
+require conf/machine/am62xx-unified.conf
 
 KERNEL_DEVICETREE_PREFIX = " \
     ti/k3-am625 \
@@ -30,4 +29,4 @@  KERNEL_DEVICETREE = " \
     ti/k3-am62x-sk-hdmi-audio.dtbo \
 "
 
-UBOOT_MACHINE = "am62x_evm_a53_defconfig"
+UBOOT_CONFIG = "evm"
diff --git a/meta-ti-bsp/conf/machine/am62xx-lp-evm-k3r5.conf b/meta-ti-bsp/conf/machine/am62xx-lp-evm-k3r5.conf
index 52b69a72..fb760e99 100644
--- a/meta-ti-bsp/conf/machine/am62xx-lp-evm-k3r5.conf
+++ b/meta-ti-bsp/conf/machine/am62xx-lp-evm-k3r5.conf
@@ -2,11 +2,6 @@ 
 #@NAME: AM62XX LP GP EVM (R5F)
 #@DESCRIPTION: Machine configuration for the TI AM62xx LP GP EVM (R5F core)
 
-require conf/machine/include/k3r5.inc
-require conf/machine/include/ti-extras.inc
+require conf/machine/am62xx-unified-k3r5.conf
 
-SYSFW_SOC = "am62x"
-SYSFW_CONFIG = "evm"
-SYSFW_SUFFIX = "hs-fs"
-
-UBOOT_MACHINE = "am62x_lpsk_r5_defconfig"
+UBOOT_CONFIG = "lpsk"
diff --git a/meta-ti-bsp/conf/machine/am62xx-lp-evm.conf b/meta-ti-bsp/conf/machine/am62xx-lp-evm.conf
index e9271f5b..f7d89d68 100644
--- a/meta-ti-bsp/conf/machine/am62xx-lp-evm.conf
+++ b/meta-ti-bsp/conf/machine/am62xx-lp-evm.conf
@@ -2,8 +2,7 @@ 
 #@NAME: AM62XX LP EVM
 #@DESCRIPTION: Machine configuration for the TI AM62XX LP EVM
 
-require conf/machine/include/am62xx.inc
-require conf/machine/include/ti-extras.inc
+require conf/machine/am62xx-unified.conf
 
 KERNEL_DEVICETREE_PREFIX = " \
     ti/k3-am62-lp \
@@ -19,4 +18,4 @@  KERNEL_DEVICETREE = " \
     ti/k3-am62x-sk-hdmi-audio.dtbo \
 "
 
-UBOOT_MACHINE = "am62x_lpsk_a53_defconfig"
+UBOOT_CONFIG = "lpsk"
diff --git a/meta-ti-bsp/conf/machine/am62xx-unified-k3r5.conf b/meta-ti-bsp/conf/machine/am62xx-unified-k3r5.conf
new file mode 100644
index 00000000..7d65e6af
--- /dev/null
+++ b/meta-ti-bsp/conf/machine/am62xx-unified-k3r5.conf
@@ -0,0 +1,20 @@ 
+#@TYPE: Machine
+#@NAME: AM62XX EVM (R5F)
+#@DESCRIPTION: Machine configuration for the TI AM62xx EVM (R5F core)
+
+require conf/machine/include/k3r5.inc
+require conf/machine/include/ti-extras.inc
+
+SYSFW_SOC = "am62x"
+SYSFW_CONFIG = "evm"
+SYSFW_SUFFIX = "hs-fs"
+
+UBOOT_MACHINE = ""
+# Last config in the list is default
+UBOOT_CONFIG ??= "sip lpsk evm"
+
+UBOOT_CONFIG:bsp-ti-6_1 ??= "evm"
+
+UBOOT_CONFIG[sip] = "am62xsip_evm_r5_defconfig"
+UBOOT_CONFIG[lpsk] = "am62x_lpsk_r5_defconfig"
+UBOOT_CONFIG[evm] = "am62x_evm_r5_defconfig"
diff --git a/meta-ti-bsp/conf/machine/am62xx-unified.conf b/meta-ti-bsp/conf/machine/am62xx-unified.conf
new file mode 100644
index 00000000..5b03239f
--- /dev/null
+++ b/meta-ti-bsp/conf/machine/am62xx-unified.conf
@@ -0,0 +1,43 @@ 
+#@TYPE: Machine
+#@NAME: AM62XX Unified EVM
+#@DESCRIPTION: Machine configuration for the TI AM62XX EVM and all of its variations
+
+require conf/machine/include/am62xx.inc
+require conf/machine/include/ti-extras.inc
+
+KERNEL_DEVICETREE_PREFIX = " \
+    ti/k3-am62-lp \
+    ti/k3-am625 \
+    ti/k3-am62x-sk \
+"
+
+KERNEL_DEVICETREE = " \
+    ti/k3-am62-lp-sk.dtb \
+    ti/k3-am625-beagleplay-csi2-ov5640.dtbo \
+    ti/k3-am625-beagleplay-csi2-tevi-ov5640.dtbo \
+    ti/k3-am625-beagleplay.dtb \
+    ti/k3-am625-phyboard-lyra-rdk.dtb \
+    ti/k3-am625-sk.dtb \
+    ti/k3-am625-verdin-nonwifi-dahlia.dtb \
+    ti/k3-am625-verdin-nonwifi-dev.dtb \
+    ti/k3-am625-verdin-nonwifi-mallow.dtb \
+    ti/k3-am625-verdin-nonwifi-yavia.dtb \
+    ti/k3-am625-verdin-wifi-dahlia.dtb \
+    ti/k3-am625-verdin-wifi-dev.dtb \
+    ti/k3-am625-verdin-wifi-mallow.dtb \
+    ti/k3-am625-verdin-wifi-yavia.dtb \
+    ti/k3-am62x-sk-csi2-imx219.dtbo \
+    ti/k3-am62x-sk-csi2-ov5640.dtbo \
+    ti/k3-am62x-sk-csi2-tevi-ov5640.dtbo \
+    ti/k3-am62x-sk-hdmi-audio.dtbo \
+"
+
+UBOOT_MACHINE = ""
+# Last config in the list is default
+UBOOT_CONFIG ??= "sip lpsk evm"
+
+UBOOT_CONFIG:bsp-ti-6_1 ??= "lpsk evm"
+
+UBOOT_CONFIG[sip] = "am62xsip_evm_a53_defconfig"
+UBOOT_CONFIG[lpsk] = "am62x_lpsk_a53_defconfig"
+UBOOT_CONFIG[evm] = "am62x_evm_a53_defconfig"
diff --git a/meta-ti-bsp/conf/machine/am62xxsip-evm-k3r5.conf b/meta-ti-bsp/conf/machine/am62xxsip-evm-k3r5.conf
index 55bc530b..ab538ff3 100644
--- a/meta-ti-bsp/conf/machine/am62xxsip-evm-k3r5.conf
+++ b/meta-ti-bsp/conf/machine/am62xxsip-evm-k3r5.conf
@@ -2,13 +2,11 @@ 
 #@NAME: AM62XX SIP EVM (R5F)
 #@DESCRIPTION: Machine configuration for the TI AM62xx SIP EVM (R5F core)
 
-require conf/machine/include/k3r5.inc
+require conf/machine/am62xx-unified-k3r5.conf
 
-SYSFW_SOC = "am62x"
-SYSFW_CONFIG = "evm"
-SYSFW_SUFFIX = "hs-fs"
+UBOOT_CONFIG = "sip"
 
-UBOOT_MACHINE = "am62xsip_evm_r5_defconfig"
+UBOOT_CONFIG:bsp-ti-6_1 = ""
 
 UBOOT_MACHINE:bsp-ti-6_1 = "am62x_evm_r5_defconfig"
 
diff --git a/meta-ti-bsp/conf/machine/am62xxsip-evm.conf b/meta-ti-bsp/conf/machine/am62xxsip-evm.conf
index 8104329e..6b179b80 100644
--- a/meta-ti-bsp/conf/machine/am62xxsip-evm.conf
+++ b/meta-ti-bsp/conf/machine/am62xxsip-evm.conf
@@ -2,7 +2,7 @@ 
 #@NAME: AM62XX SIP EVM
 #@DESCRIPTION: Machine configuration for the TI AM62XX SIP EVM
 
-require conf/machine/include/am62xx.inc
+require conf/machine/am62xx-unified.conf
 
 KERNEL_DEVICETREE_PREFIX = " \
     ti/k3-am625 \
@@ -13,6 +13,4 @@  KERNEL_DEVICETREE = " \
     ti/k3-am625-sk.dtb \
 "
 
-UBOOT_MACHINE = "am62xsip_evm_a53_defconfig"
-
-UBOOT_MACHINE:bsp-ti-6_1 = "am62x_evm_a53_defconfig"
+UBOOT_CONFIG = "sip"