[meta-arago][dunfell][PATCH 0/3] Adding support for am62xx

[meta-arago][dunfell][PATCH 0/3] Adding support for am62xx

[Ryan Eatmon]

These patches add initial recipe support for am62xx.

Krunal Bhargav (1):
  packagegroup-*-graphics: add virtual provider for AM62x GPU driver v2

Nishanth Menon (2):
  ti-tisdk-makefile: adding uboot r5 config variable for am62xx
  busybox: Enable rtcwake

 .../conf/distro/include/arago-prefs.inc              | 12 ++++++------
 .../recipes-core/busybox/busybox/defconfig           |  2 +-
 .../ti-tisdk-makefile/ti-tisdk-makefile_1.0.bb       |  1 +
 3 files changed, 8 insertions(+), 7 deletions(-)

-- 
2.17.1

[meta-arago][dunfell][PATCH 1/3] ti-tisdk-makefile: adding uboot r5 config variable for am62xx

[Ryan Eatmon]

From: Nishanth Menon <nm@ti.com>

lets add support for the latest of TI SoC.

Signed-off-by: Nishanth Menon <nm@ti.com>
Signed-off-by: Ryan Eatmon <reatmon@ti.com>
---
 .../recipes-tisdk/ti-tisdk-makefile/ti-tisdk-makefile_1.0.bb     | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti-tisdk-makefile_1.0.bb b/meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti-tisdk-makefile_1.0.bb
index 7bda8eed..7fb47bbb 100644
--- a/meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti-tisdk-makefile_1.0.bb
+++ b/meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti-tisdk-makefile_1.0.bb
@@ -363,6 +363,7 @@ K3_UBOOT_MACHINE_R5_j7-evm = "j721e_evm_r5_config"
 K3_UBOOT_MACHINE_R5_j7-hs-evm = "j721e_hs_evm_r5_config"
 K3_UBOOT_MACHINE_R5_j7200-evm = "j7200_evm_r5_config"
 K3_UBOOT_MACHINE_R5_j721s2-evm = "j721s2_evm_r5_config"
+K3_UBOOT_MACHINE_R5_am62xx-evm = "am62xx_evm_r5_config"
 
 do_install_append_k3() {
     cat >> ${D}/Rules.make << __EOF__
-- 
2.17.1

Re: [meta-arago][dunfell][PATCH 1/3] ti-tisdk-makefile: adding uboot r5 config variable for am62xx

[Nishanth Menon]

On 14:35-20220322, Ryan Eatmon via lists.yoctoproject.org wrote:
> From: Nishanth Menon <nm@ti.com>
> 
> lets add support for the latest of TI SoC.
> 
> Signed-off-by: Nishanth Menon <nm@ti.com>
> Signed-off-by: Ryan Eatmon <reatmon@ti.com>
> ---
>  .../recipes-tisdk/ti-tisdk-makefile/ti-tisdk-makefile_1.0.bb     | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti-tisdk-makefile_1.0.bb b/meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti-tisdk-makefile_1.0.bb
> index 7bda8eed..7fb47bbb 100644
> --- a/meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti-tisdk-makefile_1.0.bb
> +++ b/meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti-tisdk-makefile_1.0.bb
> @@ -363,6 +363,7 @@ K3_UBOOT_MACHINE_R5_j7-evm = "j721e_evm_r5_config"
>  K3_UBOOT_MACHINE_R5_j7-hs-evm = "j721e_hs_evm_r5_config"
>  K3_UBOOT_MACHINE_R5_j7200-evm = "j7200_evm_r5_config"
>  K3_UBOOT_MACHINE_R5_j721s2-evm = "j721s2_evm_r5_config"
> +K3_UBOOT_MACHINE_R5_am62xx-evm = "am62xx_evm_r5_config"
>  
>  do_install_append_k3() {
>      cat >> ${D}/Rules.make << __EOF__
> -- 
> 2.17.1


We probably also will need update to
recipes-core/packagegroups/packagegroup-arago-tisdk-addons-sdk-host.bb
and recipes-core/packagegroups/packagegroup-arago-tisdk-addons.bb


same update is needed in ti-tisdk-makefile
PRU_ICSS_INSTALL_TARGET_am64xx ..

-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D

[meta-arago][dunfell][PATCH 2/3] packagegroup-*-graphics: add virtual provider for AM62x GPU driver v2

[Ryan Eatmon]

From: Krunal Bhargav <k-bhargav@ti.com>

Signed-off-by: Krunal Bhargav <k-bhargav@ti.com>
Signed-off-by: Ryan Eatmon <reatmon@ti.com>
---
 .../conf/distro/include/arago-prefs.inc              | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/meta-arago-distro/conf/distro/include/arago-prefs.inc b/meta-arago-distro/conf/distro/include/arago-prefs.inc
index a356c76c..7bc2ba52 100644
--- a/meta-arago-distro/conf/distro/include/arago-prefs.inc
+++ b/meta-arago-distro/conf/distro/include/arago-prefs.inc
@@ -14,12 +14,12 @@ PREFERRED_PROVIDER_virtual/libgles2 = "ti-sgx-ddk-um"
 PREFERRED_PROVIDER_virtual/libgbm = "ti-sgx-ddk-um"
 PREFERRED_PROVIDER_virtual/gpudriver = "ti-sgx-ddk-km"
 
-# J7 has a Rogue GPU core
-PREFERRED_PROVIDER_virtual/egl_j7 = "ti-img-rogue-umlibs"
-PREFERRED_PROVIDER_virtual/libgles1_j7 = "ti-img-rogue-umlibs"
-PREFERRED_PROVIDER_virtual/libgles2_j7 = "ti-img-rogue-umlibs"
-PREFERRED_PROVIDER_virtual/libgbm_j7 = "ti-img-rogue-umlibs"
-PREFERRED_PROVIDER_virtual/gpudriver_j7 = "ti-img-rogue-driver"
+# K3 TI Platforms has a Rogue GPU core
+PREFERRED_PROVIDER_virtual/egl = "ti-img-rogue-umlibs"
+PREFERRED_PROVIDER_virtual/libgles1 = "ti-img-rogue-umlibs"
+PREFERRED_PROVIDER_virtual/libgles2 = "ti-img-rogue-umlibs"
+PREFERRED_PROVIDER_virtual/libgbm = "ti-img-rogue-umlibs"
+PREFERRED_PROVIDER_virtual/gpudriver = "ti-img-rogue-driver"
 
 # This is required when using keystone devices with graphical support
 # such as Qt5
-- 
2.17.1

Re: [meta-arago][dunfell][PATCH 2/3] packagegroup-*-graphics: add virtual provider for AM62x GPU driver v2

[Nishanth Menon]

On 14:35-20220322, Ryan Eatmon via lists.yoctoproject.org wrote:
> From: Krunal Bhargav <k-bhargav@ti.com>
> 
> Signed-off-by: Krunal Bhargav <k-bhargav@ti.com>
> Signed-off-by: Ryan Eatmon <reatmon@ti.com>
> ---
>  .../conf/distro/include/arago-prefs.inc              | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/meta-arago-distro/conf/distro/include/arago-prefs.inc b/meta-arago-distro/conf/distro/include/arago-prefs.inc
> index a356c76c..7bc2ba52 100644
> --- a/meta-arago-distro/conf/distro/include/arago-prefs.inc
> +++ b/meta-arago-distro/conf/distro/include/arago-prefs.inc
> @@ -14,12 +14,12 @@ PREFERRED_PROVIDER_virtual/libgles2 = "ti-sgx-ddk-um"
>  PREFERRED_PROVIDER_virtual/libgbm = "ti-sgx-ddk-um"
>  PREFERRED_PROVIDER_virtual/gpudriver = "ti-sgx-ddk-km"
>  
> -# J7 has a Rogue GPU core
> -PREFERRED_PROVIDER_virtual/egl_j7 = "ti-img-rogue-umlibs"
> -PREFERRED_PROVIDER_virtual/libgles1_j7 = "ti-img-rogue-umlibs"
> -PREFERRED_PROVIDER_virtual/libgles2_j7 = "ti-img-rogue-umlibs"
> -PREFERRED_PROVIDER_virtual/libgbm_j7 = "ti-img-rogue-umlibs"
> -PREFERRED_PROVIDER_virtual/gpudriver_j7 = "ti-img-rogue-driver"
> +# K3 TI Platforms has a Rogue GPU core
> +PREFERRED_PROVIDER_virtual/egl = "ti-img-rogue-umlibs"
> +PREFERRED_PROVIDER_virtual/libgles1 = "ti-img-rogue-umlibs"
> +PREFERRED_PROVIDER_virtual/libgles2 = "ti-img-rogue-umlibs"
> +PREFERRED_PROVIDER_virtual/libgbm = "ti-img-rogue-umlibs"
> +PREFERRED_PROVIDER_virtual/gpudriver = "ti-img-rogue-driver"

How does this apply to k3 and leave non-k3 alone? Also, am65x for example is
sgx.

>  
>  # This is required when using keystone devices with graphical support
>  # such as Qt5
> -- 
> 2.17.1
> 
> 
> 
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#13711): https://lists.yoctoproject.org/g/meta-arago/message/13711
> Mute This Topic: https://lists.yoctoproject.org/mt/89960456/5513541
> Group Owner: meta-arago+owner@lists.yoctoproject.org
> Unsubscribe: https://lists.yoctoproject.org/g/meta-arago/unsub [nm@ti.com]
> -=-=-=-=-=-=-=-=-=-=-=-
> 
> 

-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D

Re: [meta-arago][dunfell][PATCH 2/3] packagegroup-*-graphics: add virtual provider for AM62x GPU driver v2

[Etheridge, Darren]

On 3/22/2022 3:49 PM, Nishanth Menon via lists.yoctoproject.org wrote:
> On 14:35-20220322, Ryan Eatmon via lists.yoctoproject.org wrote:
>> From: Krunal Bhargav <k-bhargav@ti.com>
>>
>> Signed-off-by: Krunal Bhargav <k-bhargav@ti.com>
>> Signed-off-by: Ryan Eatmon <reatmon@ti.com>
>> ---
>>   .../conf/distro/include/arago-prefs.inc              | 12 ++++++------
>>   1 file changed, 6 insertions(+), 6 deletions(-)
>>
>> diff --git a/meta-arago-distro/conf/distro/include/arago-prefs.inc b/meta-arago-distro/conf/distro/include/arago-prefs.inc
>> index a356c76c..7bc2ba52 100644
>> --- a/meta-arago-distro/conf/distro/include/arago-prefs.inc
>> +++ b/meta-arago-distro/conf/distro/include/arago-prefs.inc
>> @@ -14,12 +14,12 @@ PREFERRED_PROVIDER_virtual/libgles2 = "ti-sgx-ddk-um"
>>   PREFERRED_PROVIDER_virtual/libgbm = "ti-sgx-ddk-um"
>>   PREFERRED_PROVIDER_virtual/gpudriver = "ti-sgx-ddk-km"
>>   
>> -# J7 has a Rogue GPU core
>> -PREFERRED_PROVIDER_virtual/egl_j7 = "ti-img-rogue-umlibs"
>> -PREFERRED_PROVIDER_virtual/libgles1_j7 = "ti-img-rogue-umlibs"
>> -PREFERRED_PROVIDER_virtual/libgles2_j7 = "ti-img-rogue-umlibs"
>> -PREFERRED_PROVIDER_virtual/libgbm_j7 = "ti-img-rogue-umlibs"
>> -PREFERRED_PROVIDER_virtual/gpudriver_j7 = "ti-img-rogue-driver"
>> +# K3 TI Platforms has a Rogue GPU core
>> +PREFERRED_PROVIDER_virtual/egl = "ti-img-rogue-umlibs"
>> +PREFERRED_PROVIDER_virtual/libgles1 = "ti-img-rogue-umlibs"
>> +PREFERRED_PROVIDER_virtual/libgles2 = "ti-img-rogue-umlibs"
>> +PREFERRED_PROVIDER_virtual/libgbm = "ti-img-rogue-umlibs"
>> +PREFERRED_PROVIDER_virtual/gpudriver = "ti-img-rogue-driver"
> 
> How does this apply to k3 and leave non-k3 alone? Also, am65x for example is
> sgx.
> 

That is a very good point.  This will mess up SGX platforms.   I wish we 
had k3_ as the prefix for the name here for all the k3 platforms.

So I guess the only solution is leave all the _j7 as they were

and add:
+PREFERRED_PROVIDER_virtual/egl_am62 = "ti-img-rogue-umlibs"
+PREFERRED_PROVIDER_virtual/libgles1_am62 = "ti-img-rogue-umlibs"
+PREFERRED_PROVIDER_virtual/libgles2_am62 = "ti-img-rogue-umlibs"
+PREFERRED_PROVIDER_virtual/libgbm_am62 = "ti-img-rogue-umlibs"
+PREFERRED_PROVIDER_virtual/gpudriver_am62 = "ti-img-rogue-driver"


Is there anything cleverer?

>>   
>>   # This is required when using keystone devices with graphical support
>>   # such as Qt5
>> -- 
>> 2.17.1
>>
>>
>>
>>
>>
>>
> 

Re: [meta-arago][dunfell][PATCH 2/3] packagegroup-*-graphics: add virtual provider for AM62x GPU driver v2

[Denys Dmytriyenko]

On Tue, Mar 22, 2022 at 03:58:04PM -0500, Etheridge, Darren via lists.yoctoproject.org wrote:
> 
> 
> On 3/22/2022 3:49 PM, Nishanth Menon via lists.yoctoproject.org wrote:
> >On 14:35-20220322, Ryan Eatmon via lists.yoctoproject.org wrote:
> >>From: Krunal Bhargav <k-bhargav@ti.com>
> >>
> >>Signed-off-by: Krunal Bhargav <k-bhargav@ti.com>
> >>Signed-off-by: Ryan Eatmon <reatmon@ti.com>
> >>---
> >>  .../conf/distro/include/arago-prefs.inc              | 12 ++++++------
> >>  1 file changed, 6 insertions(+), 6 deletions(-)
> >>
> >>diff --git a/meta-arago-distro/conf/distro/include/arago-prefs.inc b/meta-arago-distro/conf/distro/include/arago-prefs.inc
> >>index a356c76c..7bc2ba52 100644
> >>--- a/meta-arago-distro/conf/distro/include/arago-prefs.inc
> >>+++ b/meta-arago-distro/conf/distro/include/arago-prefs.inc
> >>@@ -14,12 +14,12 @@ PREFERRED_PROVIDER_virtual/libgles2 = "ti-sgx-ddk-um"
> >>  PREFERRED_PROVIDER_virtual/libgbm = "ti-sgx-ddk-um"
> >>  PREFERRED_PROVIDER_virtual/gpudriver = "ti-sgx-ddk-km"
> >>-# J7 has a Rogue GPU core
> >>-PREFERRED_PROVIDER_virtual/egl_j7 = "ti-img-rogue-umlibs"
> >>-PREFERRED_PROVIDER_virtual/libgles1_j7 = "ti-img-rogue-umlibs"
> >>-PREFERRED_PROVIDER_virtual/libgles2_j7 = "ti-img-rogue-umlibs"
> >>-PREFERRED_PROVIDER_virtual/libgbm_j7 = "ti-img-rogue-umlibs"
> >>-PREFERRED_PROVIDER_virtual/gpudriver_j7 = "ti-img-rogue-driver"
> >>+# K3 TI Platforms has a Rogue GPU core
> >>+PREFERRED_PROVIDER_virtual/egl = "ti-img-rogue-umlibs"
> >>+PREFERRED_PROVIDER_virtual/libgles1 = "ti-img-rogue-umlibs"
> >>+PREFERRED_PROVIDER_virtual/libgles2 = "ti-img-rogue-umlibs"
> >>+PREFERRED_PROVIDER_virtual/libgbm = "ti-img-rogue-umlibs"
> >>+PREFERRED_PROVIDER_virtual/gpudriver = "ti-img-rogue-driver"
> >
> >How does this apply to k3 and leave non-k3 alone? Also, am65x for example is
> >sgx.
> >
> 
> That is a very good point.  This will mess up SGX platforms.   I
> wish we had k3_ as the prefix for the name here for all the k3
> platforms.

There is "k3" defined and you can do _k3 override to address all K3 platforms, 
but again, some are SGX and some are RGX...


> So I guess the only solution is leave all the _j7 as they were
> 
> and add:
> +PREFERRED_PROVIDER_virtual/egl_am62 = "ti-img-rogue-umlibs"
> +PREFERRED_PROVIDER_virtual/libgles1_am62 = "ti-img-rogue-umlibs"
> +PREFERRED_PROVIDER_virtual/libgles2_am62 = "ti-img-rogue-umlibs"
> +PREFERRED_PROVIDER_virtual/libgbm_am62 = "ti-img-rogue-umlibs"
> +PREFERRED_PROVIDER_virtual/gpudriver_am62 = "ti-img-rogue-driver"

And there's no "am62" defined, but "am62xx" :)


> Is there anything cleverer?
> 
> >>  # This is required when using keystone devices with graphical support
> >>  # such as Qt5
> >>-- 
> >>2.17.1

-- 
Regards,
Denys Dmytriyenko <denis@denix.org>
PGP: 0x420902729A92C964 - https://denix.org/0x420902729A92C964
Fingerprint: 25FC E4A5 8A72 2F69 1186  6D76 4209 0272 9A92 C964

Re: [meta-arago][dunfell][PATCH 2/3] packagegroup-*-graphics: add virtual provider for AM62x GPU driver v2

[Etheridge, Darren]

On 3/22/2022 5:13 PM, Denys Dmytriyenko wrote:
> On Tue, Mar 22, 2022 at 03:58:04PM -0500, Etheridge, Darren via lists.yoctoproject.org wrote:
>>
>>
>> On 3/22/2022 3:49 PM, Nishanth Menon via lists.yoctoproject.org wrote:
>>> On 14:35-20220322, Ryan Eatmon via lists.yoctoproject.org wrote:
>>>> From: Krunal Bhargav <k-bhargav@ti.com>
>>>>
>>>> Signed-off-by: Krunal Bhargav <k-bhargav@ti.com>
>>>> Signed-off-by: Ryan Eatmon <reatmon@ti.com>
>>>> ---
>>>>   .../conf/distro/include/arago-prefs.inc              | 12 ++++++------
>>>>   1 file changed, 6 insertions(+), 6 deletions(-)
>>>>
>>>> diff --git a/meta-arago-distro/conf/distro/include/arago-prefs.inc b/meta-arago-distro/conf/distro/include/arago-prefs.inc
>>>> index a356c76c..7bc2ba52 100644
>>>> --- a/meta-arago-distro/conf/distro/include/arago-prefs.inc
>>>> +++ b/meta-arago-distro/conf/distro/include/arago-prefs.inc
>>>> @@ -14,12 +14,12 @@ PREFERRED_PROVIDER_virtual/libgles2 = "ti-sgx-ddk-um"
>>>>   PREFERRED_PROVIDER_virtual/libgbm = "ti-sgx-ddk-um"
>>>>   PREFERRED_PROVIDER_virtual/gpudriver = "ti-sgx-ddk-km"
>>>> -# J7 has a Rogue GPU core
>>>> -PREFERRED_PROVIDER_virtual/egl_j7 = "ti-img-rogue-umlibs"
>>>> -PREFERRED_PROVIDER_virtual/libgles1_j7 = "ti-img-rogue-umlibs"
>>>> -PREFERRED_PROVIDER_virtual/libgles2_j7 = "ti-img-rogue-umlibs"
>>>> -PREFERRED_PROVIDER_virtual/libgbm_j7 = "ti-img-rogue-umlibs"
>>>> -PREFERRED_PROVIDER_virtual/gpudriver_j7 = "ti-img-rogue-driver"
>>>> +# K3 TI Platforms has a Rogue GPU core
>>>> +PREFERRED_PROVIDER_virtual/egl = "ti-img-rogue-umlibs"
>>>> +PREFERRED_PROVIDER_virtual/libgles1 = "ti-img-rogue-umlibs"
>>>> +PREFERRED_PROVIDER_virtual/libgles2 = "ti-img-rogue-umlibs"
>>>> +PREFERRED_PROVIDER_virtual/libgbm = "ti-img-rogue-umlibs"
>>>> +PREFERRED_PROVIDER_virtual/gpudriver = "ti-img-rogue-driver"
>>>
>>> How does this apply to k3 and leave non-k3 alone? Also, am65x for example is
>>> sgx.
>>>
>>
>> That is a very good point.  This will mess up SGX platforms.   I
>> wish we had k3_ as the prefix for the name here for all the k3
>> platforms.
> 
> There is "k3" defined and you can do _k3 override to address all K3 platforms,
> but again, some are SGX and some are RGX...
> 
> 
>> So I guess the only solution is leave all the _j7 as they were
>>
>> and add:
>> +PREFERRED_PROVIDER_virtual/egl_am62 = "ti-img-rogue-umlibs"
>> +PREFERRED_PROVIDER_virtual/libgles1_am62 = "ti-img-rogue-umlibs"
>> +PREFERRED_PROVIDER_virtual/libgles2_am62 = "ti-img-rogue-umlibs"
>> +PREFERRED_PROVIDER_virtual/libgbm_am62 = "ti-img-rogue-umlibs"
>> +PREFERRED_PROVIDER_virtual/gpudriver_am62 = "ti-img-rogue-driver"
> 
> And there's no "am62" defined, but "am62xx" :)
> 

Isn't it a partial pattern match?   If it is not the _j7 won't work for 
j721s2 either.

> 
>> Is there anything cleverer?
>>
>>>>   # This is required when using keystone devices with graphical support
>>>>   # such as Qt5
>>>> -- 
>>>> 2.17.1
> 

Re: [meta-arago][dunfell][PATCH 2/3] packagegroup-*-graphics: add virtual provider for AM62x GPU driver v2

[Denys Dmytriyenko]

On Tue, Mar 22, 2022 at 05:16:20PM -0500, Etheridge, Darren wrote:
> 
> 
> On 3/22/2022 5:13 PM, Denys Dmytriyenko wrote:
> >On Tue, Mar 22, 2022 at 03:58:04PM -0500, Etheridge, Darren via lists.yoctoproject.org wrote:
> >>
> >>
> >>On 3/22/2022 3:49 PM, Nishanth Menon via lists.yoctoproject.org wrote:
> >>>On 14:35-20220322, Ryan Eatmon via lists.yoctoproject.org wrote:
> >>>>From: Krunal Bhargav <k-bhargav@ti.com>
> >>>>
> >>>>Signed-off-by: Krunal Bhargav <k-bhargav@ti.com>
> >>>>Signed-off-by: Ryan Eatmon <reatmon@ti.com>
> >>>>---
> >>>>  .../conf/distro/include/arago-prefs.inc              | 12 ++++++------
> >>>>  1 file changed, 6 insertions(+), 6 deletions(-)
> >>>>
> >>>>diff --git a/meta-arago-distro/conf/distro/include/arago-prefs.inc b/meta-arago-distro/conf/distro/include/arago-prefs.inc
> >>>>index a356c76c..7bc2ba52 100644
> >>>>--- a/meta-arago-distro/conf/distro/include/arago-prefs.inc
> >>>>+++ b/meta-arago-distro/conf/distro/include/arago-prefs.inc
> >>>>@@ -14,12 +14,12 @@ PREFERRED_PROVIDER_virtual/libgles2 = "ti-sgx-ddk-um"
> >>>>  PREFERRED_PROVIDER_virtual/libgbm = "ti-sgx-ddk-um"
> >>>>  PREFERRED_PROVIDER_virtual/gpudriver = "ti-sgx-ddk-km"
> >>>>-# J7 has a Rogue GPU core
> >>>>-PREFERRED_PROVIDER_virtual/egl_j7 = "ti-img-rogue-umlibs"
> >>>>-PREFERRED_PROVIDER_virtual/libgles1_j7 = "ti-img-rogue-umlibs"
> >>>>-PREFERRED_PROVIDER_virtual/libgles2_j7 = "ti-img-rogue-umlibs"
> >>>>-PREFERRED_PROVIDER_virtual/libgbm_j7 = "ti-img-rogue-umlibs"
> >>>>-PREFERRED_PROVIDER_virtual/gpudriver_j7 = "ti-img-rogue-driver"
> >>>>+# K3 TI Platforms has a Rogue GPU core
> >>>>+PREFERRED_PROVIDER_virtual/egl = "ti-img-rogue-umlibs"
> >>>>+PREFERRED_PROVIDER_virtual/libgles1 = "ti-img-rogue-umlibs"
> >>>>+PREFERRED_PROVIDER_virtual/libgles2 = "ti-img-rogue-umlibs"
> >>>>+PREFERRED_PROVIDER_virtual/libgbm = "ti-img-rogue-umlibs"
> >>>>+PREFERRED_PROVIDER_virtual/gpudriver = "ti-img-rogue-driver"
> >>>
> >>>How does this apply to k3 and leave non-k3 alone? Also, am65x for example is
> >>>sgx.
> >>>
> >>
> >>That is a very good point.  This will mess up SGX platforms.   I
> >>wish we had k3_ as the prefix for the name here for all the k3
> >>platforms.
> >
> >There is "k3" defined and you can do _k3 override to address all K3 platforms,
> >but again, some are SGX and some are RGX...
> >
> >
> >>So I guess the only solution is leave all the _j7 as they were
> >>
> >>and add:
> >>+PREFERRED_PROVIDER_virtual/egl_am62 = "ti-img-rogue-umlibs"
> >>+PREFERRED_PROVIDER_virtual/libgles1_am62 = "ti-img-rogue-umlibs"
> >>+PREFERRED_PROVIDER_virtual/libgles2_am62 = "ti-img-rogue-umlibs"
> >>+PREFERRED_PROVIDER_virtual/libgbm_am62 = "ti-img-rogue-umlibs"
> >>+PREFERRED_PROVIDER_virtual/gpudriver_am62 = "ti-img-rogue-driver"
> >
> >And there's no "am62" defined, but "am62xx" :)
> >
> 
> Isn't it a partial pattern match?   If it is not the _j7 won't work
> for j721s2 either.

No, it's not a partial pattern match. Overrides use exact matches only, so 
those have to be defined. Specifically these overrides come from SOC_FAMILY 
variable and there are definitions for "k3", "j7", but there's no "j721s2" 
SOC_FAMILY defined, so you cannot use _j721s2, nor _j7 will match it.
https://git.yoctoproject.org/meta-ti/tree/conf/machine/include?h=dunfell

-- 
Regards,
Denys Dmytriyenko <denis@denix.org>
PGP: 0x420902729A92C964 - https://denix.org/0x420902729A92C964
Fingerprint: 25FC E4A5 8A72 2F69 1186  6D76 4209 0272 9A92 C964

Re: [meta-arago][dunfell][PATCH 2/3] packagegroup-*-graphics: add virtual provider for AM62x GPU driver v2

[Etheridge, Darren]

On 3/22/2022 5:27 PM, Denys Dmytriyenko wrote:
> On Tue, Mar 22, 2022 at 05:16:20PM -0500, Etheridge, Darren wrote:
>>
>>
>> On 3/22/2022 5:13 PM, Denys Dmytriyenko wrote:
>>> On Tue, Mar 22, 2022 at 03:58:04PM -0500, Etheridge, Darren via lists.yoctoproject.org wrote:
>>>>
>>>>
>>>> On 3/22/2022 3:49 PM, Nishanth Menon via lists.yoctoproject.org wrote:
>>>>> On 14:35-20220322, Ryan Eatmon via lists.yoctoproject.org wrote:
>>>>>> From: Krunal Bhargav <k-bhargav@ti.com>
>>>>>>
>>>>>> Signed-off-by: Krunal Bhargav <k-bhargav@ti.com>
>>>>>> Signed-off-by: Ryan Eatmon <reatmon@ti.com>
>>>>>> ---
>>>>>>   .../conf/distro/include/arago-prefs.inc              | 12 ++++++------
>>>>>>   1 file changed, 6 insertions(+), 6 deletions(-)
>>>>>>
>>>>>> diff --git a/meta-arago-distro/conf/distro/include/arago-prefs.inc b/meta-arago-distro/conf/distro/include/arago-prefs.inc
>>>>>> index a356c76c..7bc2ba52 100644
>>>>>> --- a/meta-arago-distro/conf/distro/include/arago-prefs.inc
>>>>>> +++ b/meta-arago-distro/conf/distro/include/arago-prefs.inc
>>>>>> @@ -14,12 +14,12 @@ PREFERRED_PROVIDER_virtual/libgles2 = "ti-sgx-ddk-um"
>>>>>>   PREFERRED_PROVIDER_virtual/libgbm = "ti-sgx-ddk-um"
>>>>>>   PREFERRED_PROVIDER_virtual/gpudriver = "ti-sgx-ddk-km"
>>>>>> -# J7 has a Rogue GPU core
>>>>>> -PREFERRED_PROVIDER_virtual/egl_j7 = "ti-img-rogue-umlibs"
>>>>>> -PREFERRED_PROVIDER_virtual/libgles1_j7 = "ti-img-rogue-umlibs"
>>>>>> -PREFERRED_PROVIDER_virtual/libgles2_j7 = "ti-img-rogue-umlibs"
>>>>>> -PREFERRED_PROVIDER_virtual/libgbm_j7 = "ti-img-rogue-umlibs"
>>>>>> -PREFERRED_PROVIDER_virtual/gpudriver_j7 = "ti-img-rogue-driver"
>>>>>> +# K3 TI Platforms has a Rogue GPU core
>>>>>> +PREFERRED_PROVIDER_virtual/egl = "ti-img-rogue-umlibs"
>>>>>> +PREFERRED_PROVIDER_virtual/libgles1 = "ti-img-rogue-umlibs"
>>>>>> +PREFERRED_PROVIDER_virtual/libgles2 = "ti-img-rogue-umlibs"
>>>>>> +PREFERRED_PROVIDER_virtual/libgbm = "ti-img-rogue-umlibs"
>>>>>> +PREFERRED_PROVIDER_virtual/gpudriver = "ti-img-rogue-driver"
>>>>>
>>>>> How does this apply to k3 and leave non-k3 alone? Also, am65x for example is
>>>>> sgx.
>>>>>
>>>>
>>>> That is a very good point.  This will mess up SGX platforms.   I
>>>> wish we had k3_ as the prefix for the name here for all the k3
>>>> platforms.
>>>
>>> There is "k3" defined and you can do _k3 override to address all K3 platforms,
>>> but again, some are SGX and some are RGX...
>>>
>>>
>>>> So I guess the only solution is leave all the _j7 as they were
>>>>
>>>> and add:
>>>> +PREFERRED_PROVIDER_virtual/egl_am62 = "ti-img-rogue-umlibs"
>>>> +PREFERRED_PROVIDER_virtual/libgles1_am62 = "ti-img-rogue-umlibs"
>>>> +PREFERRED_PROVIDER_virtual/libgles2_am62 = "ti-img-rogue-umlibs"
>>>> +PREFERRED_PROVIDER_virtual/libgbm_am62 = "ti-img-rogue-umlibs"
>>>> +PREFERRED_PROVIDER_virtual/gpudriver_am62 = "ti-img-rogue-driver"
>>>
>>> And there's no "am62" defined, but "am62xx" :)
>>>
>>
>> Isn't it a partial pattern match?   If it is not the _j7 won't work
>> for j721s2 either.
> 
> No, it's not a partial pattern match. Overrides use exact matches only, so
> those have to be defined. Specifically these overrides come from SOC_FAMILY
> variable and there are definitions for "k3", "j7", but there's no "j721s2"
> SOC_FAMILY defined, so you cannot use _j721s2, nor _j7 will match it.
> https://git.yoctoproject.org/meta-ti/tree/conf/machine/include?h=dunfell
> 

Thanks for the explanation Denys,  I was making totally wrong assumption 
on how that was being derived.

So we could actually use _k3 and get am62xx-evm, j721s2-evm and j7-evm 
with a single override.  All k3's use the IMG Rogue driver if they have 
a GPU.

Re: [meta-arago][dunfell][PATCH 2/3] packagegroup-*-graphics: add virtual provider for AM62x GPU driver v2

[Denys Dmytriyenko]

On Tue, Mar 22, 2022 at 05:44:39PM -0500, Etheridge, Darren wrote:
> 
> 
> On 3/22/2022 5:27 PM, Denys Dmytriyenko wrote:
> >On Tue, Mar 22, 2022 at 05:16:20PM -0500, Etheridge, Darren wrote:
> >>
> >>
> >>On 3/22/2022 5:13 PM, Denys Dmytriyenko wrote:
> >>>On Tue, Mar 22, 2022 at 03:58:04PM -0500, Etheridge, Darren via lists.yoctoproject.org wrote:
> >>>>
> >>>>
> >>>>On 3/22/2022 3:49 PM, Nishanth Menon via lists.yoctoproject.org wrote:
> >>>>>On 14:35-20220322, Ryan Eatmon via lists.yoctoproject.org wrote:
> >>>>>>From: Krunal Bhargav <k-bhargav@ti.com>
> >>>>>>
> >>>>>>Signed-off-by: Krunal Bhargav <k-bhargav@ti.com>
> >>>>>>Signed-off-by: Ryan Eatmon <reatmon@ti.com>
> >>>>>>---
> >>>>>>  .../conf/distro/include/arago-prefs.inc              | 12 ++++++------
> >>>>>>  1 file changed, 6 insertions(+), 6 deletions(-)
> >>>>>>
> >>>>>>diff --git a/meta-arago-distro/conf/distro/include/arago-prefs.inc b/meta-arago-distro/conf/distro/include/arago-prefs.inc
> >>>>>>index a356c76c..7bc2ba52 100644
> >>>>>>--- a/meta-arago-distro/conf/distro/include/arago-prefs.inc
> >>>>>>+++ b/meta-arago-distro/conf/distro/include/arago-prefs.inc
> >>>>>>@@ -14,12 +14,12 @@ PREFERRED_PROVIDER_virtual/libgles2 = "ti-sgx-ddk-um"
> >>>>>>  PREFERRED_PROVIDER_virtual/libgbm = "ti-sgx-ddk-um"
> >>>>>>  PREFERRED_PROVIDER_virtual/gpudriver = "ti-sgx-ddk-km"
> >>>>>>-# J7 has a Rogue GPU core
> >>>>>>-PREFERRED_PROVIDER_virtual/egl_j7 = "ti-img-rogue-umlibs"
> >>>>>>-PREFERRED_PROVIDER_virtual/libgles1_j7 = "ti-img-rogue-umlibs"
> >>>>>>-PREFERRED_PROVIDER_virtual/libgles2_j7 = "ti-img-rogue-umlibs"
> >>>>>>-PREFERRED_PROVIDER_virtual/libgbm_j7 = "ti-img-rogue-umlibs"
> >>>>>>-PREFERRED_PROVIDER_virtual/gpudriver_j7 = "ti-img-rogue-driver"
> >>>>>>+# K3 TI Platforms has a Rogue GPU core
> >>>>>>+PREFERRED_PROVIDER_virtual/egl = "ti-img-rogue-umlibs"
> >>>>>>+PREFERRED_PROVIDER_virtual/libgles1 = "ti-img-rogue-umlibs"
> >>>>>>+PREFERRED_PROVIDER_virtual/libgles2 = "ti-img-rogue-umlibs"
> >>>>>>+PREFERRED_PROVIDER_virtual/libgbm = "ti-img-rogue-umlibs"
> >>>>>>+PREFERRED_PROVIDER_virtual/gpudriver = "ti-img-rogue-driver"
> >>>>>
> >>>>>How does this apply to k3 and leave non-k3 alone? Also, am65x for example is
> >>>>>sgx.
> >>>>>
> >>>>
> >>>>That is a very good point.  This will mess up SGX platforms.   I
> >>>>wish we had k3_ as the prefix for the name here for all the k3
> >>>>platforms.
> >>>
> >>>There is "k3" defined and you can do _k3 override to address all K3 platforms,
> >>>but again, some are SGX and some are RGX...
> >>>
> >>>
> >>>>So I guess the only solution is leave all the _j7 as they were
> >>>>
> >>>>and add:
> >>>>+PREFERRED_PROVIDER_virtual/egl_am62 = "ti-img-rogue-umlibs"
> >>>>+PREFERRED_PROVIDER_virtual/libgles1_am62 = "ti-img-rogue-umlibs"
> >>>>+PREFERRED_PROVIDER_virtual/libgles2_am62 = "ti-img-rogue-umlibs"
> >>>>+PREFERRED_PROVIDER_virtual/libgbm_am62 = "ti-img-rogue-umlibs"
> >>>>+PREFERRED_PROVIDER_virtual/gpudriver_am62 = "ti-img-rogue-driver"
> >>>
> >>>And there's no "am62" defined, but "am62xx" :)
> >>>
> >>
> >>Isn't it a partial pattern match?   If it is not the _j7 won't work
> >>for j721s2 either.
> >
> >No, it's not a partial pattern match. Overrides use exact matches only, so
> >those have to be defined. Specifically these overrides come from SOC_FAMILY
> >variable and there are definitions for "k3", "j7", but there's no "j721s2"
> >SOC_FAMILY defined, so you cannot use _j721s2, nor _j7 will match it.
> >https://git.yoctoproject.org/meta-ti/tree/conf/machine/include?h=dunfell
> >
> 
> Thanks for the explanation Denys,  I was making totally wrong
> assumption on how that was being derived.
> 
> So we could actually use _k3 and get am62xx-evm, j721s2-evm and
> j7-evm with a single override.  All k3's use the IMG Rogue driver if
> they have a GPU.

What about am65xx? It is SGX.
And I assume am64xx and j7200 have no GPU.
If am65xx is the only exception in K3 family, then we could flip the 
compatibility setting.

-- 
Denys

[meta-arago][dunfell][PATCH 3/3] busybox: Enable rtcwake

[Ryan Eatmon]

From: Nishanth Menon <nm@ti.com>

This is esp useful for testing low power states with a minimal system.

Signed-off-by: Nishanth Menon <nm@ti.com>
Signed-off-by: Ryan Eatmon <reatmon@ti.com>
---
 meta-arago-distro/recipes-core/busybox/busybox/defconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta-arago-distro/recipes-core/busybox/busybox/defconfig b/meta-arago-distro/recipes-core/busybox/busybox/defconfig
index 65470bc7..351b80d4 100644
--- a/meta-arago-distro/recipes-core/busybox/busybox/defconfig
+++ b/meta-arago-distro/recipes-core/busybox/busybox/defconfig
@@ -583,7 +583,7 @@ CONFIG_PIVOT_ROOT=y
 CONFIG_RDATE=y
 # CONFIG_RDEV is not set
 # CONFIG_READPROFILE is not set
-# CONFIG_RTCWAKE is not set
+CONFIG_RTCWAKE=y
 # CONFIG_SCRIPT is not set
 # CONFIG_SCRIPTREPLAY is not set
 # CONFIG_SETARCH is not set
-- 
2.17.1

Re: [meta-arago][dunfell][PATCH 3/3] busybox: Enable rtcwake

[Nishanth Menon]

On 14:35-20220322, Ryan Eatmon via lists.yoctoproject.org wrote:
> From: Nishanth Menon <nm@ti.com>
> 
> This is esp useful for testing low power states with a minimal system.
> 
> Signed-off-by: Nishanth Menon <nm@ti.com>
> Signed-off-by: Ryan Eatmon <reatmon@ti.com>
> ---
>  meta-arago-distro/recipes-core/busybox/busybox/defconfig | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/meta-arago-distro/recipes-core/busybox/busybox/defconfig b/meta-arago-distro/recipes-core/busybox/busybox/defconfig
> index 65470bc7..351b80d4 100644
> --- a/meta-arago-distro/recipes-core/busybox/busybox/defconfig
> +++ b/meta-arago-distro/recipes-core/busybox/busybox/defconfig
> @@ -583,7 +583,7 @@ CONFIG_PIVOT_ROOT=y
>  CONFIG_RDATE=y
>  # CONFIG_RDEV is not set
>  # CONFIG_READPROFILE is not set
> -# CONFIG_RTCWAKE is not set
> +CONFIG_RTCWAKE=y
>  # CONFIG_SCRIPT is not set
>  # CONFIG_SCRIPTREPLAY is not set
>  # CONFIG_SETARCH is not set

OR we could use base-image cpio
..

-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D

Re: [meta-arago][dunfell][PATCH 3/3] busybox: Enable rtcwake

[Ryan Eatmon]

On 3/22/2022 15:51, Nishanth Menon wrote:
> On 14:35-20220322, Ryan Eatmon via lists.yoctoproject.org wrote:
>> From: Nishanth Menon <nm@ti.com>
>>
>> This is esp useful for testing low power states with a minimal system.
>>
>> Signed-off-by: Nishanth Menon <nm@ti.com>
>> Signed-off-by: Ryan Eatmon <reatmon@ti.com>
>> ---
>>   meta-arago-distro/recipes-core/busybox/busybox/defconfig | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/meta-arago-distro/recipes-core/busybox/busybox/defconfig b/meta-arago-distro/recipes-core/busybox/busybox/defconfig
>> index 65470bc7..351b80d4 100644
>> --- a/meta-arago-distro/recipes-core/busybox/busybox/defconfig
>> +++ b/meta-arago-distro/recipes-core/busybox/busybox/defconfig
>> @@ -583,7 +583,7 @@ CONFIG_PIVOT_ROOT=y
>>   CONFIG_RDATE=y
>>   # CONFIG_RDEV is not set
>>   # CONFIG_READPROFILE is not set
>> -# CONFIG_RTCWAKE is not set
>> +CONFIG_RTCWAKE=y
>>   # CONFIG_SCRIPT is not set
>>   # CONFIG_SCRIPTREPLAY is not set
>>   # CONFIG_SETARCH is not set
> 
> OR we could use base-image cpio
> ..
> 

Are you saying we don't need this patch and do it it a different way?  I 
can drop this patch for now...


-- 
Ryan Eatmon
reatmon@ti.com

Re: [meta-arago][dunfell][PATCH 3/3] busybox: Enable rtcwake

[Nishanth Menon]

On 14:10-20220323, Ryan Eatmon wrote:
> 
> 
> On 3/22/2022 15:51, Nishanth Menon wrote:
> > On 14:35-20220322, Ryan Eatmon via lists.yoctoproject.org wrote:
> > > From: Nishanth Menon <nm@ti.com>
> > > 
> > > This is esp useful for testing low power states with a minimal system.
> > > 
> > > Signed-off-by: Nishanth Menon <nm@ti.com>
> > > Signed-off-by: Ryan Eatmon <reatmon@ti.com>
> > > ---
> > >   meta-arago-distro/recipes-core/busybox/busybox/defconfig | 2 +-
> > >   1 file changed, 1 insertion(+), 1 deletion(-)
> > > 
> > > diff --git a/meta-arago-distro/recipes-core/busybox/busybox/defconfig b/meta-arago-distro/recipes-core/busybox/busybox/defconfig
> > > index 65470bc7..351b80d4 100644
> > > --- a/meta-arago-distro/recipes-core/busybox/busybox/defconfig
> > > +++ b/meta-arago-distro/recipes-core/busybox/busybox/defconfig
> > > @@ -583,7 +583,7 @@ CONFIG_PIVOT_ROOT=y
> > >   CONFIG_RDATE=y
> > >   # CONFIG_RDEV is not set
> > >   # CONFIG_READPROFILE is not set
> > > -# CONFIG_RTCWAKE is not set
> > > +CONFIG_RTCWAKE=y
> > >   # CONFIG_SCRIPT is not set
> > >   # CONFIG_SCRIPTREPLAY is not set
> > >   # CONFIG_SETARCH is not set
> > 
> > OR we could use base-image cpio
> > ..
> > 
> 
> Are you saying we don't need this patch and do it it a different way?  I can
> drop this patch for now...


for lpm, we could use baseimage cpio itself instead of tiny-image.

-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D

Re: [meta-arago][dunfell][PATCH 0/3] Adding support for am62xx

[PRC Automation]

meta-arago / na / 20220322193554.24404-1-reatmon

PRC Results: FAIL

=========================================================
  check-yocto-patches: PASS
=========================================================
Patches
----------------------------------------
All patches passed



=========================================================
  apply-yocto-patch: FAIL
=========================================================
dunfell
=====================
Summary:
- Patch Series: [meta-arago][dunfell][PATCH 0/3] Adding support for am62xx
- Submitter: From: Ryan Eatmon <reatmon@ti.com>
- Date: Date: Tue, 22 Mar 2022 14:35:51 -0500
- Num Patches: 3
- Mailing List (public inbox) Commit SHA: 62c45955c11f5703f31475e6545e759b115eba62

Applied to:
- Repository: lcpd-prc-meta-arago
- Base Branch: dunfell
- Commit Author: Randolph Sapp <rs@ti.com>
- Commit Subject: meta-arago-distro: linux-firmware: remove cadence firmware
- Commit SHA: 86d19ced20934c8383c8a39f88d97c72e7e929ee

Patches
----------------------------------------
FAIL - [meta-arago][dunfell][PATCH 1/3] ti-tisdk-makefile: adding uboot r5 config variable for am62xx
    error: patch failed: meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti-tisdk-makefile_1.0.bb:363
    error: meta-arago-distro/recipes-tisdk/ti-tisdk-makefile/ti-tisdk-makefile_1.0.bb: patch does not apply
    Applying: ti-tisdk-makefile: adding uboot r5 config variable for am62xx
    Patch failed at 0001 ti-tisdk-makefile: adding uboot r5 config variable for am62xx