All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ti-sgx-ddk-um: update SRCREV to pick up Mesa-based EGL/GLES libraries
@ 2019-10-29 13:20 Eric Ruei
  2019-10-29 23:29 ` Denys Dmytriyenko
  2019-10-30 13:22 ` Andrew F. Davis
  0 siblings, 2 replies; 15+ messages in thread
From: Eric Ruei @ 2019-10-29 13:20 UTC (permalink / raw)
  To: meta-ti

This is the initial step toward Mesa-based EGL/GLES libraries which
support all the required EGL 1.5 extensions. We plan to provide a
Mesa-pvr recipe to build Mesa from source and SGX/DDK patches where
ti-sgx-ddk-um shall provide the EGL/GLES plugins only at the next step.

Signed-off-by: Eric Ruei <e-ruei1@ti.com>
---
 recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb b/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
index 7a6f013e..3991d917 100644
--- a/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
+++ b/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
@@ -11,7 +11,7 @@ PR = "r34"
 BRANCH = "ti-img-sgx/thud/${PV}"
 
 SRC_URI = "git://git.ti.com/graphics/omap5-sgx-ddk-um-linux.git;protocol=git;branch=${BRANCH}"
-SRCREV = "87d7e5c1e4db1bab048939c9719059d549c1e8dd"
+SRCREV = "2a2e5bb090ced870d73ed4edbc54793e952cc6d8"
 
 TARGET_PRODUCT_omap-a15 = "jacinto6evm"
 TARGET_PRODUCT_ti33x = "ti335x"
@@ -47,7 +47,9 @@ S = "${WORKDIR}/git"
 
 do_install () {
     oe_runmake install DESTDIR=${D} TARGET_PRODUCT=${TARGET_PRODUCT}
-    ln -sf libGLESv2.so.${PV} ${D}${libdir}/libGLESv2.so.1
+    ln -sf libGLESv2.so ${D}${libdir}/libGLESv2.so.1
+
+    rm -rf ${D}${includedir}/GL
 
     chown -R root:root ${D}
 }
@@ -58,7 +60,7 @@ FILES_${PN} +=  "${includedir}/*"
 FILES_${PN} +=  "${sysconfdir}/*"
 
 PACKAGES =+ "${PN}-plugins"
-FILES_${PN}-plugins = "${libdir}/libsrv_init.so ${libdir}/libsrv_um.so ${libdir}/libglslcompiler.so ${libdir}/libPVRScopeServices.so ${libdir}/libGLESv2.so ${libdir}/libEGL.so ${libdir}/libGLES_CM.so ${libdir}/libpvrDRMWSEGL.so  ${libdir}/libpvrGBMWSEGL.so  ${libdir}/libpvrws_WAYLAND.so"
+FILES_${PN}-plugins = "${libdir}/libsrv_init.so ${libdir}/libsrv_um.so ${libdir}/libglslcompiler.so ${libdir}/libPVRScopeServices.so ${libdir}/libGLESv2.so ${libdir}/libEGL.so ${libdir}/libGLESv1_CM.so ${libdir}/libGLES_CM.so ${libdir}/libGLESv1_PVR_MESA.so ${libdir}/libGLESv2_PVR_MESA.so"
 RDEPENDS_${PN} += "${PN}-plugins"
 
 ALLOW_EMPTY_${PN}-plugins = "1"
-- 
2.17.1



^ permalink raw reply related	[flat|nested] 15+ messages in thread

* Re: [PATCH] ti-sgx-ddk-um: update SRCREV to pick up Mesa-based EGL/GLES libraries
  2019-10-29 13:20 [PATCH] ti-sgx-ddk-um: update SRCREV to pick up Mesa-based EGL/GLES libraries Eric Ruei
@ 2019-10-29 23:29 ` Denys Dmytriyenko
  2019-10-30 13:16   ` Ruei, Eric
  2019-10-30 13:22 ` Andrew F. Davis
  1 sibling, 1 reply; 15+ messages in thread
From: Denys Dmytriyenko @ 2019-10-29 23:29 UTC (permalink / raw)
  To: Eric Ruei; +Cc: meta-ti

On Tue, Oct 29, 2019 at 09:20:20AM -0400, Eric Ruei wrote:
> This is the initial step toward Mesa-based EGL/GLES libraries which
> support all the required EGL 1.5 extensions. We plan to provide a
> Mesa-pvr recipe to build Mesa from source and SGX/DDK patches where
> ti-sgx-ddk-um shall provide the EGL/GLES plugins only at the next step.

Eric,

The new binaries are huge. The compressed package went from 900 KB to 8 MB in 
size and now both AM3 and AM4 won't fit into allocated rootfs size...


> Signed-off-by: Eric Ruei <e-ruei1@ti.com>
> ---
>  recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb | 8 +++++---
>  1 file changed, 5 insertions(+), 3 deletions(-)
> 
> diff --git a/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb b/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
> index 7a6f013e..3991d917 100644
> --- a/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
> +++ b/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
> @@ -11,7 +11,7 @@ PR = "r34"
>  BRANCH = "ti-img-sgx/thud/${PV}"
>  
>  SRC_URI = "git://git.ti.com/graphics/omap5-sgx-ddk-um-linux.git;protocol=git;branch=${BRANCH}"
> -SRCREV = "87d7e5c1e4db1bab048939c9719059d549c1e8dd"
> +SRCREV = "2a2e5bb090ced870d73ed4edbc54793e952cc6d8"
>  
>  TARGET_PRODUCT_omap-a15 = "jacinto6evm"
>  TARGET_PRODUCT_ti33x = "ti335x"
> @@ -47,7 +47,9 @@ S = "${WORKDIR}/git"
>  
>  do_install () {
>      oe_runmake install DESTDIR=${D} TARGET_PRODUCT=${TARGET_PRODUCT}
> -    ln -sf libGLESv2.so.${PV} ${D}${libdir}/libGLESv2.so.1
> +    ln -sf libGLESv2.so ${D}${libdir}/libGLESv2.so.1
> +
> +    rm -rf ${D}${includedir}/GL
>  
>      chown -R root:root ${D}
>  }
> @@ -58,7 +60,7 @@ FILES_${PN} +=  "${includedir}/*"
>  FILES_${PN} +=  "${sysconfdir}/*"
>  
>  PACKAGES =+ "${PN}-plugins"
> -FILES_${PN}-plugins = "${libdir}/libsrv_init.so ${libdir}/libsrv_um.so ${libdir}/libglslcompiler.so ${libdir}/libPVRScopeServices.so ${libdir}/libGLESv2.so ${libdir}/libEGL.so ${libdir}/libGLES_CM.so ${libdir}/libpvrDRMWSEGL.so  ${libdir}/libpvrGBMWSEGL.so  ${libdir}/libpvrws_WAYLAND.so"
> +FILES_${PN}-plugins = "${libdir}/libsrv_init.so ${libdir}/libsrv_um.so ${libdir}/libglslcompiler.so ${libdir}/libPVRScopeServices.so ${libdir}/libGLESv2.so ${libdir}/libEGL.so ${libdir}/libGLESv1_CM.so ${libdir}/libGLES_CM.so ${libdir}/libGLESv1_PVR_MESA.so ${libdir}/libGLESv2_PVR_MESA.so"
>  RDEPENDS_${PN} += "${PN}-plugins"
>  
>  ALLOW_EMPTY_${PN}-plugins = "1"
> -- 
> 2.17.1
> 
> -- 
> _______________________________________________
> meta-ti mailing list
> meta-ti@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-ti


^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [PATCH] ti-sgx-ddk-um: update SRCREV to pick up Mesa-based EGL/GLES libraries
  2019-10-29 23:29 ` Denys Dmytriyenko
@ 2019-10-30 13:16   ` Ruei, Eric
  0 siblings, 0 replies; 15+ messages in thread
From: Ruei, Eric @ 2019-10-30 13:16 UTC (permalink / raw)
  To: Denys Dmytriyenko, Eric Ruei, Andrew F. Davis; +Cc: meta-ti

On 10/29/2019 7:29 PM, Denys Dmytriyenko wrote:
> On Tue, Oct 29, 2019 at 09:20:20AM -0400, Eric Ruei wrote:
>> This is the initial step toward Mesa-based EGL/GLES libraries which
>> support all the required EGL 1.5 extensions. We plan to provide a
>> Mesa-pvr recipe to build Mesa from source and SGX/DDK patches where
>> ti-sgx-ddk-um shall provide the EGL/GLES plugins only at the next step.
> 
> Eric,
> 
> The new binaries are huge. The compressed package went from 900 KB to 8 MB in
> size and now both AM3 and AM4 won't fit into allocated rootfs size...
> 
> 
Denys:

The Mesa-based GLES/EGL is much bigger. The major increase is caused by 
the new file pvr_dri.so (28M) under usr/lib/dri. The LibEGL related 
libraries are also increased from sub-200K to 1.1 M and there is another 
library libglapi.so (900k).

Eric


>> Signed-off-by: Eric Ruei <e-ruei1@ti.com>
>> ---
>>   recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb | 8 +++++---
>>   1 file changed, 5 insertions(+), 3 deletions(-)
>>
>> diff --git a/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb b/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
>> index 7a6f013e..3991d917 100644
>> --- a/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
>> +++ b/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
>> @@ -11,7 +11,7 @@ PR = "r34"
>>   BRANCH = "ti-img-sgx/thud/${PV}"
>>   
>>   SRC_URI = "git://git.ti.com/graphics/omap5-sgx-ddk-um-linux.git;protocol=git;branch=${BRANCH}"
>> -SRCREV = "87d7e5c1e4db1bab048939c9719059d549c1e8dd"
>> +SRCREV = "2a2e5bb090ced870d73ed4edbc54793e952cc6d8"
>>   
>>   TARGET_PRODUCT_omap-a15 = "jacinto6evm"
>>   TARGET_PRODUCT_ti33x = "ti335x"
>> @@ -47,7 +47,9 @@ S = "${WORKDIR}/git"
>>   
>>   do_install () {
>>       oe_runmake install DESTDIR=${D} TARGET_PRODUCT=${TARGET_PRODUCT}
>> -    ln -sf libGLESv2.so.${PV} ${D}${libdir}/libGLESv2.so.1
>> +    ln -sf libGLESv2.so ${D}${libdir}/libGLESv2.so.1
>> +
>> +    rm -rf ${D}${includedir}/GL
>>   
>>       chown -R root:root ${D}
>>   }
>> @@ -58,7 +60,7 @@ FILES_${PN} +=  "${includedir}/*"
>>   FILES_${PN} +=  "${sysconfdir}/*"
>>   
>>   PACKAGES =+ "${PN}-plugins"
>> -FILES_${PN}-plugins = "${libdir}/libsrv_init.so ${libdir}/libsrv_um.so ${libdir}/libglslcompiler.so ${libdir}/libPVRScopeServices.so ${libdir}/libGLESv2.so ${libdir}/libEGL.so ${libdir}/libGLES_CM.so ${libdir}/libpvrDRMWSEGL.so  ${libdir}/libpvrGBMWSEGL.so  ${libdir}/libpvrws_WAYLAND.so"
>> +FILES_${PN}-plugins = "${libdir}/libsrv_init.so ${libdir}/libsrv_um.so ${libdir}/libglslcompiler.so ${libdir}/libPVRScopeServices.so ${libdir}/libGLESv2.so ${libdir}/libEGL.so ${libdir}/libGLESv1_CM.so ${libdir}/libGLES_CM.so ${libdir}/libGLESv1_PVR_MESA.so ${libdir}/libGLESv2_PVR_MESA.so"
>>   RDEPENDS_${PN} += "${PN}-plugins"
>>   
>>   ALLOW_EMPTY_${PN}-plugins = "1"
>> -- 
>> 2.17.1
>>
>> -- 
>> _______________________________________________
>> meta-ti mailing list
>> meta-ti@yoctoproject.org
>> https://lists.yoctoproject.org/listinfo/meta-ti



^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [PATCH] ti-sgx-ddk-um: update SRCREV to pick up Mesa-based EGL/GLES libraries
  2019-10-29 13:20 [PATCH] ti-sgx-ddk-um: update SRCREV to pick up Mesa-based EGL/GLES libraries Eric Ruei
  2019-10-29 23:29 ` Denys Dmytriyenko
@ 2019-10-30 13:22 ` Andrew F. Davis
  2019-10-30 13:31   ` Ruei, Eric
  1 sibling, 1 reply; 15+ messages in thread
From: Andrew F. Davis @ 2019-10-30 13:22 UTC (permalink / raw)
  To: Eric Ruei, meta-ti

On 10/29/19 9:20 AM, Eric Ruei wrote:
> This is the initial step toward Mesa-based EGL/GLES libraries which
> support all the required EGL 1.5 extensions. We plan to provide a
> Mesa-pvr recipe to build Mesa from source and SGX/DDK patches where
> ti-sgx-ddk-um shall provide the EGL/GLES plugins only at the next step.
> 
> Signed-off-by: Eric Ruei <e-ruei1@ti.com>
> ---
>  recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb | 8 +++++---
>  1 file changed, 5 insertions(+), 3 deletions(-)
> 
> diff --git a/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb b/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
> index 7a6f013e..3991d917 100644
> --- a/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
> +++ b/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
> @@ -11,7 +11,7 @@ PR = "r34"
>  BRANCH = "ti-img-sgx/thud/${PV}"
>  
>  SRC_URI = "git://git.ti.com/graphics/omap5-sgx-ddk-um-linux.git;protocol=git;branch=${BRANCH}"
> -SRCREV = "87d7e5c1e4db1bab048939c9719059d549c1e8dd"
> +SRCREV = "2a2e5bb090ced870d73ed4edbc54793e952cc6d8"
>  
>  TARGET_PRODUCT_omap-a15 = "jacinto6evm"
>  TARGET_PRODUCT_ti33x = "ti335x"
> @@ -47,7 +47,9 @@ S = "${WORKDIR}/git"
>  
>  do_install () {
>      oe_runmake install DESTDIR=${D} TARGET_PRODUCT=${TARGET_PRODUCT}
> -    ln -sf libGLESv2.so.${PV} ${D}${libdir}/libGLESv2.so.1
> +    ln -sf libGLESv2.so ${D}${libdir}/libGLESv2.so.1
> +
> +    rm -rf ${D}${includedir}/GL


Why remove this?


>  
>      chown -R root:root ${D}
>  }
> @@ -58,7 +60,7 @@ FILES_${PN} +=  "${includedir}/*"
>  FILES_${PN} +=  "${sysconfdir}/*"
>  
>  PACKAGES =+ "${PN}-plugins"
> -FILES_${PN}-plugins = "${libdir}/libsrv_init.so ${libdir}/libsrv_um.so ${libdir}/libglslcompiler.so ${libdir}/libPVRScopeServices.so ${libdir}/libGLESv2.so ${libdir}/libEGL.so ${libdir}/libGLES_CM.so ${libdir}/libpvrDRMWSEGL.so  ${libdir}/libpvrGBMWSEGL.so  ${libdir}/libpvrws_WAYLAND.so"
> +FILES_${PN}-plugins = "${libdir}/libsrv_init.so ${libdir}/libsrv_um.so ${libdir}/libglslcompiler.so ${libdir}/libPVRScopeServices.so ${libdir}/libGLESv2.so ${libdir}/libEGL.so ${libdir}/libGLESv1_CM.so ${libdir}/libGLES_CM.so ${libdir}/libGLESv1_PVR_MESA.so ${libdir}/libGLESv2_PVR_MESA.so"
>  RDEPENDS_${PN} += "${PN}-plugins"


The newer binaries after the DDK commit "um: Attempt to load shared
object with version extension automatically" do not need all this plugin
stuff, it can all be dropped.

Andrew


>  
>  ALLOW_EMPTY_${PN}-plugins = "1"
> 


^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [PATCH] ti-sgx-ddk-um: update SRCREV to pick up Mesa-based EGL/GLES libraries
  2019-10-30 13:22 ` Andrew F. Davis
@ 2019-10-30 13:31   ` Ruei, Eric
  2019-10-30 13:36     ` Andrew F. Davis
  0 siblings, 1 reply; 15+ messages in thread
From: Ruei, Eric @ 2019-10-30 13:31 UTC (permalink / raw)
  To: Andrew F. Davis, Eric Ruei, meta-ti

On 10/30/2019 9:22 AM, Andrew F. Davis wrote:
> On 10/29/19 9:20 AM, Eric Ruei wrote:
>> This is the initial step toward Mesa-based EGL/GLES libraries which
>> support all the required EGL 1.5 extensions. We plan to provide a
>> Mesa-pvr recipe to build Mesa from source and SGX/DDK patches where
>> ti-sgx-ddk-um shall provide the EGL/GLES plugins only at the next step.
>>
>> Signed-off-by: Eric Ruei <e-ruei1@ti.com>
>> ---
>>   recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb | 8 +++++---
>>   1 file changed, 5 insertions(+), 3 deletions(-)
>>
>> diff --git a/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb b/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
>> index 7a6f013e..3991d917 100644
>> --- a/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
>> +++ b/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
>> @@ -11,7 +11,7 @@ PR = "r34"
>>   BRANCH = "ti-img-sgx/thud/${PV}"
>>   
>>   SRC_URI = "git://git.ti.com/graphics/omap5-sgx-ddk-um-linux.git;protocol=git;branch=${BRANCH}"
>> -SRCREV = "87d7e5c1e4db1bab048939c9719059d549c1e8dd"
>> +SRCREV = "2a2e5bb090ced870d73ed4edbc54793e952cc6d8"
>>   
>>   TARGET_PRODUCT_omap-a15 = "jacinto6evm"
>>   TARGET_PRODUCT_ti33x = "ti335x"
>> @@ -47,7 +47,9 @@ S = "${WORKDIR}/git"
>>   
>>   do_install () {
>>       oe_runmake install DESTDIR=${D} TARGET_PRODUCT=${TARGET_PRODUCT}
>> -    ln -sf libGLESv2.so.${PV} ${D}${libdir}/libGLESv2.so.1
>> +    ln -sf libGLESv2.so ${D}${libdir}/libGLESv2.so.1
>> +
>> +    rm -rf ${D}${includedir}/GL
> 
> 
> Why remove this?
> 
> 

There is another component provides GL header files.
Denys: how do we resolve this conflict?

Eric



>>   
>>       chown -R root:root ${D}
>>   }
>> @@ -58,7 +60,7 @@ FILES_${PN} +=  "${includedir}/*"
>>   FILES_${PN} +=  "${sysconfdir}/*"
>>   
>>   PACKAGES =+ "${PN}-plugins"
>> -FILES_${PN}-plugins = "${libdir}/libsrv_init.so ${libdir}/libsrv_um.so ${libdir}/libglslcompiler.so ${libdir}/libPVRScopeServices.so ${libdir}/libGLESv2.so ${libdir}/libEGL.so ${libdir}/libGLES_CM.so ${libdir}/libpvrDRMWSEGL.so  ${libdir}/libpvrGBMWSEGL.so  ${libdir}/libpvrws_WAYLAND.so"
>> +FILES_${PN}-plugins = "${libdir}/libsrv_init.so ${libdir}/libsrv_um.so ${libdir}/libglslcompiler.so ${libdir}/libPVRScopeServices.so ${libdir}/libGLESv2.so ${libdir}/libEGL.so ${libdir}/libGLESv1_CM.so ${libdir}/libGLES_CM.so ${libdir}/libGLESv1_PVR_MESA.so ${libdir}/libGLESv2_PVR_MESA.so"
>>   RDEPENDS_${PN} += "${PN}-plugins"
> 
> 
> The newer binaries after the DDK commit "um: Attempt to load shared
> object with version extension automatically" do not need all this plugin
> stuff, it can all be dropped.
> 
> Andrew
> 
> 
>>   
>>   ALLOW_EMPTY_${PN}-plugins = "1"
>>



^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [PATCH] ti-sgx-ddk-um: update SRCREV to pick up Mesa-based EGL/GLES libraries
  2019-10-30 13:31   ` Ruei, Eric
@ 2019-10-30 13:36     ` Andrew F. Davis
  2019-10-30 13:53       ` [EXTERNAL] " Tammana, Gowtham
  0 siblings, 1 reply; 15+ messages in thread
From: Andrew F. Davis @ 2019-10-30 13:36 UTC (permalink / raw)
  To: Ruei, Eric, Eric Ruei, meta-ti

On 10/30/19 9:31 AM, Ruei, Eric wrote:
> On 10/30/2019 9:22 AM, Andrew F. Davis wrote:
>> On 10/29/19 9:20 AM, Eric Ruei wrote:
>>> This is the initial step toward Mesa-based EGL/GLES libraries which
>>> support all the required EGL 1.5 extensions. We plan to provide a
>>> Mesa-pvr recipe to build Mesa from source and SGX/DDK patches where
>>> ti-sgx-ddk-um shall provide the EGL/GLES plugins only at the next step.
>>>
>>> Signed-off-by: Eric Ruei <e-ruei1@ti.com>
>>> ---
>>>   recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb | 8 +++++---
>>>   1 file changed, 5 insertions(+), 3 deletions(-)
>>>
>>> diff --git a/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
>>> b/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
>>> index 7a6f013e..3991d917 100644
>>> --- a/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
>>> +++ b/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
>>> @@ -11,7 +11,7 @@ PR = "r34"
>>>   BRANCH = "ti-img-sgx/thud/${PV}"
>>>     SRC_URI =
>>> "git://git.ti.com/graphics/omap5-sgx-ddk-um-linux.git;protocol=git;branch=${BRANCH}"
>>>
>>> -SRCREV = "87d7e5c1e4db1bab048939c9719059d549c1e8dd"
>>> +SRCREV = "2a2e5bb090ced870d73ed4edbc54793e952cc6d8"
>>>     TARGET_PRODUCT_omap-a15 = "jacinto6evm"
>>>   TARGET_PRODUCT_ti33x = "ti335x"
>>> @@ -47,7 +47,9 @@ S = "${WORKDIR}/git"
>>>     do_install () {
>>>       oe_runmake install DESTDIR=${D} TARGET_PRODUCT=${TARGET_PRODUCT}
>>> -    ln -sf libGLESv2.so.${PV} ${D}${libdir}/libGLESv2.so.1
>>> +    ln -sf libGLESv2.so ${D}${libdir}/libGLESv2.so.1
>>> +
>>> +    rm -rf ${D}${includedir}/GL
>>
>>
>> Why remove this?
>>
>>
> 
> There is another component provides GL header files.
> Denys: how do we resolve this conflict?
> 


The DSP OpenCL implementation? That package needs fixed, not this one,
the OpenGL implementation (this driver) should provide the GL headers.

Andrew


> Eric
> 
> 
> 
>>>         chown -R root:root ${D}
>>>   }
>>> @@ -58,7 +60,7 @@ FILES_${PN} +=  "${includedir}/*"
>>>   FILES_${PN} +=  "${sysconfdir}/*"
>>>     PACKAGES =+ "${PN}-plugins"
>>> -FILES_${PN}-plugins = "${libdir}/libsrv_init.so
>>> ${libdir}/libsrv_um.so ${libdir}/libglslcompiler.so
>>> ${libdir}/libPVRScopeServices.so ${libdir}/libGLESv2.so
>>> ${libdir}/libEGL.so ${libdir}/libGLES_CM.so
>>> ${libdir}/libpvrDRMWSEGL.so  ${libdir}/libpvrGBMWSEGL.so 
>>> ${libdir}/libpvrws_WAYLAND.so"
>>> +FILES_${PN}-plugins = "${libdir}/libsrv_init.so
>>> ${libdir}/libsrv_um.so ${libdir}/libglslcompiler.so
>>> ${libdir}/libPVRScopeServices.so ${libdir}/libGLESv2.so
>>> ${libdir}/libEGL.so ${libdir}/libGLESv1_CM.so ${libdir}/libGLES_CM.so
>>> ${libdir}/libGLESv1_PVR_MESA.so ${libdir}/libGLESv2_PVR_MESA.so"
>>>   RDEPENDS_${PN} += "${PN}-plugins"
>>
>>
>> The newer binaries after the DDK commit "um: Attempt to load shared
>> object with version extension automatically" do not need all this plugin
>> stuff, it can all be dropped.
>>
>> Andrew
>>
>>
>>>     ALLOW_EMPTY_${PN}-plugins = "1"
>>>
> 


^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [EXTERNAL] Re: [PATCH] ti-sgx-ddk-um: update SRCREV to pick up Mesa-based EGL/GLES libraries
  2019-10-30 13:36     ` Andrew F. Davis
@ 2019-10-30 13:53       ` Tammana, Gowtham
  2019-10-30 14:53         ` Ruei, Eric
  0 siblings, 1 reply; 15+ messages in thread
From: Tammana, Gowtham @ 2019-10-30 13:53 UTC (permalink / raw)
  To: Davis, Andrew, Ruei, Eric, Ruei, Eric, meta-ti



>-----Original Message-----
>From: meta-ti-bounces@yoctoproject.org [mailto:meta-ti-
>bounces@yoctoproject.org] On Behalf Of Davis, Andrew
>Sent: Wednesday, October 30, 2019 8:36 AM
>To: Ruei, Eric; Ruei, Eric; meta-ti@yoctoproject.org
>Subject: [EXTERNAL] Re: [meta-ti] [PATCH] ti-sgx-ddk-um: update SRCREV to pick
>up Mesa-based EGL/GLES libraries
>
>On 10/30/19 9:31 AM, Ruei, Eric wrote:
>> On 10/30/2019 9:22 AM, Andrew F. Davis wrote:
>>> On 10/29/19 9:20 AM, Eric Ruei wrote:
>>>> This is the initial step toward Mesa-based EGL/GLES libraries which
>>>> support all the required EGL 1.5 extensions. We plan to provide a
>>>> Mesa-pvr recipe to build Mesa from source and SGX/DDK patches where
>>>> ti-sgx-ddk-um shall provide the EGL/GLES plugins only at the next step.
>>>>
>>>> Signed-off-by: Eric Ruei <e-ruei1@ti.com>
>>>> ---
>>>>   recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb | 8 +++++---
>>>>   1 file changed, 5 insertions(+), 3 deletions(-)
>>>>
>>>> diff --git a/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
>>>> b/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
>>>> index 7a6f013e..3991d917 100644
>>>> --- a/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
>>>> +++ b/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
>>>> @@ -11,7 +11,7 @@ PR = "r34"
>>>>   BRANCH = "ti-img-sgx/thud/${PV}"
>>>>     SRC_URI =
>>>> "git://git.ti.com/graphics/omap5-sgx-ddk-um-
>linux.git;protocol=git;branch=${BRANCH}"
>>>>
>>>> -SRCREV = "87d7e5c1e4db1bab048939c9719059d549c1e8dd"
>>>> +SRCREV = "2a2e5bb090ced870d73ed4edbc54793e952cc6d8"
>>>>     TARGET_PRODUCT_omap-a15 = "jacinto6evm"
>>>>   TARGET_PRODUCT_ti33x = "ti335x"
>>>> @@ -47,7 +47,9 @@ S = "${WORKDIR}/git"
>>>>     do_install () {
>>>>       oe_runmake install DESTDIR=${D} TARGET_PRODUCT=${TARGET_PRODUCT}
>>>> -    ln -sf libGLESv2.so.${PV} ${D}${libdir}/libGLESv2.so.1
>>>> +    ln -sf libGLESv2.so ${D}${libdir}/libGLESv2.so.1
>>>> +
>>>> +    rm -rf ${D}${includedir}/GL
>>>
>>>
>>> Why remove this?
>>>
>>>
>>
>> There is another component provides GL header files.
>> Denys: how do we resolve this conflict?
>>
>
>
>The DSP OpenCL implementation? That package needs fixed, not this one,
>the OpenGL implementation (this driver) should provide the GL headers.

We don't support desktop GL, they shouldn't come from this package.

Gowtham

>>
>>
>>>>         chown -R root:root ${D}
>>>>   }
>>>> @@ -58,7 +60,7 @@ FILES_${PN} +=  "${includedir}/*"
>>>>   FILES_${PN} +=  "${sysconfdir}/*"
>>>>     PACKAGES =+ "${PN}-plugins"
>>>> -FILES_${PN}-plugins = "${libdir}/libsrv_init.so
>>>> ${libdir}/libsrv_um.so ${libdir}/libglslcompiler.so
>>>> ${libdir}/libPVRScopeServices.so ${libdir}/libGLESv2.so
>>>> ${libdir}/libEGL.so ${libdir}/libGLES_CM.so
>>>> ${libdir}/libpvrDRMWSEGL.so  ${libdir}/libpvrGBMWSEGL.so
>>>> ${libdir}/libpvrws_WAYLAND.so"
>>>> +FILES_${PN}-plugins = "${libdir}/libsrv_init.so
>>>> ${libdir}/libsrv_um.so ${libdir}/libglslcompiler.so
>>>> ${libdir}/libPVRScopeServices.so ${libdir}/libGLESv2.so
>>>> ${libdir}/libEGL.so ${libdir}/libGLESv1_CM.so ${libdir}/libGLES_CM.so
>>>> ${libdir}/libGLESv1_PVR_MESA.so ${libdir}/libGLESv2_PVR_MESA.so"
>>>>   RDEPENDS_${PN} += "${PN}-plugins"
>>>
>>>
>>> The newer binaries after the DDK commit "um: Attempt to load shared
>>> object with version extension automatically" do not need all this plugin
>>> stuff, it can all be dropped.
>>>
>>> Andrew
>>>
>>>
>>>>     ALLOW_EMPTY_${PN}-plugins = "1"
>>>>
>>
>--
>_______________________________________________
>meta-ti mailing list
>meta-ti@yoctoproject.org
>https://lists.yoctoproject.org/listinfo/meta-ti

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [EXTERNAL] Re: [PATCH] ti-sgx-ddk-um: update SRCREV to pick up Mesa-based EGL/GLES libraries
  2019-10-30 13:53       ` [EXTERNAL] " Tammana, Gowtham
@ 2019-10-30 14:53         ` Ruei, Eric
  2019-10-30 14:58           ` Andrew F. Davis
  0 siblings, 1 reply; 15+ messages in thread
From: Ruei, Eric @ 2019-10-30 14:53 UTC (permalink / raw)
  To: Tammana, Gowtham, Davis, Andrew, Ruei, Eric, meta-ti

On 10/30/2019 9:53 AM, Tammana, Gowtham wrote:
> 
> 
>> -----Original Message-----
>> From: meta-ti-bounces@yoctoproject.org [mailto:meta-ti-
>> bounces@yoctoproject.org] On Behalf Of Davis, Andrew
>> Sent: Wednesday, October 30, 2019 8:36 AM
>> To: Ruei, Eric; Ruei, Eric; meta-ti@yoctoproject.org
>> Subject: [EXTERNAL] Re: [meta-ti] [PATCH] ti-sgx-ddk-um: update SRCREV to pick
>> up Mesa-based EGL/GLES libraries
>>
>> On 10/30/19 9:31 AM, Ruei, Eric wrote:
>>> On 10/30/2019 9:22 AM, Andrew F. Davis wrote:
>>>> On 10/29/19 9:20 AM, Eric Ruei wrote:
>>>>> This is the initial step toward Mesa-based EGL/GLES libraries which
>>>>> support all the required EGL 1.5 extensions. We plan to provide a
>>>>> Mesa-pvr recipe to build Mesa from source and SGX/DDK patches where
>>>>> ti-sgx-ddk-um shall provide the EGL/GLES plugins only at the next step.
>>>>>
>>>>> Signed-off-by: Eric Ruei <e-ruei1@ti.com>
>>>>> ---
>>>>>    recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb | 8 +++++---
>>>>>    1 file changed, 5 insertions(+), 3 deletions(-)
>>>>>
>>>>> diff --git a/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
>>>>> b/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
>>>>> index 7a6f013e..3991d917 100644
>>>>> --- a/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
>>>>> +++ b/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
>>>>> @@ -11,7 +11,7 @@ PR = "r34"
>>>>>    BRANCH = "ti-img-sgx/thud/${PV}"
>>>>>      SRC_URI =
>>>>> "git://git.ti.com/graphics/omap5-sgx-ddk-um-
>> linux.git;protocol=git;branch=${BRANCH}"
>>>>>
>>>>> -SRCREV = "87d7e5c1e4db1bab048939c9719059d549c1e8dd"
>>>>> +SRCREV = "2a2e5bb090ced870d73ed4edbc54793e952cc6d8"
>>>>>      TARGET_PRODUCT_omap-a15 = "jacinto6evm"
>>>>>    TARGET_PRODUCT_ti33x = "ti335x"
>>>>> @@ -47,7 +47,9 @@ S = "${WORKDIR}/git"
>>>>>      do_install () {
>>>>>        oe_runmake install DESTDIR=${D} TARGET_PRODUCT=${TARGET_PRODUCT}
>>>>> -    ln -sf libGLESv2.so.${PV} ${D}${libdir}/libGLESv2.so.1
>>>>> +    ln -sf libGLESv2.so ${D}${libdir}/libGLESv2.so.1
>>>>> +
>>>>> +    rm -rf ${D}${includedir}/GL
>>>>
>>>>
>>>> Why remove this?
>>>>
>>>>
>>>
>>> There is another component provides GL header files.
>>> Denys: how do we resolve this conflict?
>>>
>>
>>
>> The DSP OpenCL implementation? That package needs fixed, not this one,
>> the OpenGL implementation (this driver) should provide the GL headers.
> 
> We don't support desktop GL, they shouldn't come from this package.
> 
> Gowtham
> 

Andrew:

Do you agree? I can keep the line here tentatively until GL is removed 
from the package itself.

Eric



>>>
>>>
>>>>>          chown -R root:root ${D}
>>>>>    }
>>>>> @@ -58,7 +60,7 @@ FILES_${PN} +=  "${includedir}/*"
>>>>>    FILES_${PN} +=  "${sysconfdir}/*"
>>>>>      PACKAGES =+ "${PN}-plugins"
>>>>> -FILES_${PN}-plugins = "${libdir}/libsrv_init.so
>>>>> ${libdir}/libsrv_um.so ${libdir}/libglslcompiler.so
>>>>> ${libdir}/libPVRScopeServices.so ${libdir}/libGLESv2.so
>>>>> ${libdir}/libEGL.so ${libdir}/libGLES_CM.so
>>>>> ${libdir}/libpvrDRMWSEGL.so  ${libdir}/libpvrGBMWSEGL.so
>>>>> ${libdir}/libpvrws_WAYLAND.so"
>>>>> +FILES_${PN}-plugins = "${libdir}/libsrv_init.so
>>>>> ${libdir}/libsrv_um.so ${libdir}/libglslcompiler.so
>>>>> ${libdir}/libPVRScopeServices.so ${libdir}/libGLESv2.so
>>>>> ${libdir}/libEGL.so ${libdir}/libGLESv1_CM.so ${libdir}/libGLES_CM.so
>>>>> ${libdir}/libGLESv1_PVR_MESA.so ${libdir}/libGLESv2_PVR_MESA.so"
>>>>>    RDEPENDS_${PN} += "${PN}-plugins"
>>>>
>>>>
>>>> The newer binaries after the DDK commit "um: Attempt to load shared
>>>> object with version extension automatically" do not need all this plugin
>>>> stuff, it can all be dropped.
>>>>
>>>> Andrew
>>>>
>>>>
>>>>>      ALLOW_EMPTY_${PN}-plugins = "1"
>>>>>
>>>
>> --
>> _______________________________________________
>> meta-ti mailing list
>> meta-ti@yoctoproject.org
>> https://lists.yoctoproject.org/listinfo/meta-ti



^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [EXTERNAL] Re: [PATCH] ti-sgx-ddk-um: update SRCREV to pick up Mesa-based EGL/GLES libraries
  2019-10-30 14:53         ` Ruei, Eric
@ 2019-10-30 14:58           ` Andrew F. Davis
  2019-10-30 15:06             ` Denys Dmytriyenko
  2019-10-31 12:39             ` Ruei, Eric
  0 siblings, 2 replies; 15+ messages in thread
From: Andrew F. Davis @ 2019-10-30 14:58 UTC (permalink / raw)
  To: Ruei, Eric, Tammana, Gowtham, Ruei, Eric, meta-ti

On 10/30/19 10:53 AM, Ruei, Eric wrote:
> On 10/30/2019 9:53 AM, Tammana, Gowtham wrote:
>>
>>
>>> -----Original Message-----
>>> From: meta-ti-bounces@yoctoproject.org [mailto:meta-ti-
>>> bounces@yoctoproject.org] On Behalf Of Davis, Andrew
>>> Sent: Wednesday, October 30, 2019 8:36 AM
>>> To: Ruei, Eric; Ruei, Eric; meta-ti@yoctoproject.org
>>> Subject: [EXTERNAL] Re: [meta-ti] [PATCH] ti-sgx-ddk-um: update
>>> SRCREV to pick
>>> up Mesa-based EGL/GLES libraries
>>>
>>> On 10/30/19 9:31 AM, Ruei, Eric wrote:
>>>> On 10/30/2019 9:22 AM, Andrew F. Davis wrote:
>>>>> On 10/29/19 9:20 AM, Eric Ruei wrote:
>>>>>> This is the initial step toward Mesa-based EGL/GLES libraries which
>>>>>> support all the required EGL 1.5 extensions. We plan to provide a
>>>>>> Mesa-pvr recipe to build Mesa from source and SGX/DDK patches where
>>>>>> ti-sgx-ddk-um shall provide the EGL/GLES plugins only at the next
>>>>>> step.
>>>>>>
>>>>>> Signed-off-by: Eric Ruei <e-ruei1@ti.com>
>>>>>> ---
>>>>>>    recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb | 8
>>>>>> +++++---
>>>>>>    1 file changed, 5 insertions(+), 3 deletions(-)
>>>>>>
>>>>>> diff --git a/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
>>>>>> b/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
>>>>>> index 7a6f013e..3991d917 100644
>>>>>> --- a/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
>>>>>> +++ b/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
>>>>>> @@ -11,7 +11,7 @@ PR = "r34"
>>>>>>    BRANCH = "ti-img-sgx/thud/${PV}"
>>>>>>      SRC_URI =
>>>>>> "git://git.ti.com/graphics/omap5-sgx-ddk-um-
>>> linux.git;protocol=git;branch=${BRANCH}"
>>>>>>
>>>>>> -SRCREV = "87d7e5c1e4db1bab048939c9719059d549c1e8dd"
>>>>>> +SRCREV = "2a2e5bb090ced870d73ed4edbc54793e952cc6d8"
>>>>>>      TARGET_PRODUCT_omap-a15 = "jacinto6evm"
>>>>>>    TARGET_PRODUCT_ti33x = "ti335x"
>>>>>> @@ -47,7 +47,9 @@ S = "${WORKDIR}/git"
>>>>>>      do_install () {
>>>>>>        oe_runmake install DESTDIR=${D}
>>>>>> TARGET_PRODUCT=${TARGET_PRODUCT}
>>>>>> -    ln -sf libGLESv2.so.${PV} ${D}${libdir}/libGLESv2.so.1
>>>>>> +    ln -sf libGLESv2.so ${D}${libdir}/libGLESv2.so.1
>>>>>> +
>>>>>> +    rm -rf ${D}${includedir}/GL
>>>>>
>>>>>
>>>>> Why remove this?
>>>>>
>>>>>
>>>>
>>>> There is another component provides GL header files.
>>>> Denys: how do we resolve this conflict?
>>>>
>>>
>>>
>>> The DSP OpenCL implementation? That package needs fixed, not this one,
>>> the OpenGL implementation (this driver) should provide the GL headers.
>>
>> We don't support desktop GL, they shouldn't come from this package.
>>
>> Gowtham
>>
> 
> Andrew:
> 
> Do you agree? I can keep the line here tentatively until GL is removed
> from the package itself.
> 


I still believe we should be shipping the GL headers in this package.
But I won't object to removing the headers temporarily using this recipe
until the conflicting ones can be removed from the OpenCL package.

Andrew


> Eric
> 
> 
> 
>>>>
>>>>
>>>>>>          chown -R root:root ${D}
>>>>>>    }
>>>>>> @@ -58,7 +60,7 @@ FILES_${PN} +=  "${includedir}/*"
>>>>>>    FILES_${PN} +=  "${sysconfdir}/*"
>>>>>>      PACKAGES =+ "${PN}-plugins"
>>>>>> -FILES_${PN}-plugins = "${libdir}/libsrv_init.so
>>>>>> ${libdir}/libsrv_um.so ${libdir}/libglslcompiler.so
>>>>>> ${libdir}/libPVRScopeServices.so ${libdir}/libGLESv2.so
>>>>>> ${libdir}/libEGL.so ${libdir}/libGLES_CM.so
>>>>>> ${libdir}/libpvrDRMWSEGL.so  ${libdir}/libpvrGBMWSEGL.so
>>>>>> ${libdir}/libpvrws_WAYLAND.so"
>>>>>> +FILES_${PN}-plugins = "${libdir}/libsrv_init.so
>>>>>> ${libdir}/libsrv_um.so ${libdir}/libglslcompiler.so
>>>>>> ${libdir}/libPVRScopeServices.so ${libdir}/libGLESv2.so
>>>>>> ${libdir}/libEGL.so ${libdir}/libGLESv1_CM.so ${libdir}/libGLES_CM.so
>>>>>> ${libdir}/libGLESv1_PVR_MESA.so ${libdir}/libGLESv2_PVR_MESA.so"
>>>>>>    RDEPENDS_${PN} += "${PN}-plugins"
>>>>>
>>>>>
>>>>> The newer binaries after the DDK commit "um: Attempt to load shared
>>>>> object with version extension automatically" do not need all this
>>>>> plugin
>>>>> stuff, it can all be dropped.
>>>>>
>>>>> Andrew
>>>>>
>>>>>
>>>>>>      ALLOW_EMPTY_${PN}-plugins = "1"
>>>>>>
>>>>
>>> -- 
>>> _______________________________________________
>>> meta-ti mailing list
>>> meta-ti@yoctoproject.org
>>> https://lists.yoctoproject.org/listinfo/meta-ti
> 


^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [EXTERNAL] Re: [PATCH] ti-sgx-ddk-um: update SRCREV to pick up Mesa-based EGL/GLES libraries
  2019-10-30 14:58           ` Andrew F. Davis
@ 2019-10-30 15:06             ` Denys Dmytriyenko
  2019-10-30 15:14               ` Ruei, Eric
  2019-10-30 15:14               ` Andrew F. Davis
  2019-10-31 12:39             ` Ruei, Eric
  1 sibling, 2 replies; 15+ messages in thread
From: Denys Dmytriyenko @ 2019-10-30 15:06 UTC (permalink / raw)
  To: Andrew F. Davis; +Cc: Ruei, Eric, meta-ti

On Wed, Oct 30, 2019 at 10:58:31AM -0400, Andrew F. Davis wrote:
> On 10/30/19 10:53 AM, Ruei, Eric wrote:
> > On 10/30/2019 9:53 AM, Tammana, Gowtham wrote:
> >>
> >>
> >>> -----Original Message-----
> >>> From: meta-ti-bounces@yoctoproject.org [mailto:meta-ti-
> >>> bounces@yoctoproject.org] On Behalf Of Davis, Andrew
> >>> Sent: Wednesday, October 30, 2019 8:36 AM
> >>> To: Ruei, Eric; Ruei, Eric; meta-ti@yoctoproject.org
> >>> Subject: [EXTERNAL] Re: [meta-ti] [PATCH] ti-sgx-ddk-um: update
> >>> SRCREV to pick
> >>> up Mesa-based EGL/GLES libraries
> >>>
> >>> On 10/30/19 9:31 AM, Ruei, Eric wrote:
> >>>> On 10/30/2019 9:22 AM, Andrew F. Davis wrote:
> >>>>> On 10/29/19 9:20 AM, Eric Ruei wrote:
> >>>>>> This is the initial step toward Mesa-based EGL/GLES libraries which
> >>>>>> support all the required EGL 1.5 extensions. We plan to provide a
> >>>>>> Mesa-pvr recipe to build Mesa from source and SGX/DDK patches where
> >>>>>> ti-sgx-ddk-um shall provide the EGL/GLES plugins only at the next
> >>>>>> step.
> >>>>>>
> >>>>>> Signed-off-by: Eric Ruei <e-ruei1@ti.com>
> >>>>>> ---
> >>>>>>    recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb | 8
> >>>>>> +++++---
> >>>>>>    1 file changed, 5 insertions(+), 3 deletions(-)
> >>>>>>
> >>>>>> diff --git a/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
> >>>>>> b/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
> >>>>>> index 7a6f013e..3991d917 100644
> >>>>>> --- a/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
> >>>>>> +++ b/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
> >>>>>> @@ -11,7 +11,7 @@ PR = "r34"
> >>>>>>    BRANCH = "ti-img-sgx/thud/${PV}"
> >>>>>>      SRC_URI =
> >>>>>> "git://git.ti.com/graphics/omap5-sgx-ddk-um-
> >>> linux.git;protocol=git;branch=${BRANCH}"
> >>>>>>
> >>>>>> -SRCREV = "87d7e5c1e4db1bab048939c9719059d549c1e8dd"
> >>>>>> +SRCREV = "2a2e5bb090ced870d73ed4edbc54793e952cc6d8"
> >>>>>>      TARGET_PRODUCT_omap-a15 = "jacinto6evm"
> >>>>>>    TARGET_PRODUCT_ti33x = "ti335x"
> >>>>>> @@ -47,7 +47,9 @@ S = "${WORKDIR}/git"
> >>>>>>      do_install () {
> >>>>>>        oe_runmake install DESTDIR=${D}
> >>>>>> TARGET_PRODUCT=${TARGET_PRODUCT}
> >>>>>> -    ln -sf libGLESv2.so.${PV} ${D}${libdir}/libGLESv2.so.1
> >>>>>> +    ln -sf libGLESv2.so ${D}${libdir}/libGLESv2.so.1
> >>>>>> +
> >>>>>> +    rm -rf ${D}${includedir}/GL
> >>>>>
> >>>>>
> >>>>> Why remove this?
> >>>>>
> >>>>>
> >>>>
> >>>> There is another component provides GL header files.
> >>>> Denys: how do we resolve this conflict?
> >>>>
> >>>
> >>>
> >>> The DSP OpenCL implementation? That package needs fixed, not this one,
> >>> the OpenGL implementation (this driver) should provide the GL headers.
> >>
> >> We don't support desktop GL, they shouldn't come from this package.
> >>
> >> Gowtham
> >>
> > 
> > Andrew:
> > 
> > Do you agree? I can keep the line here tentatively until GL is removed
> > from the package itself.
> > 
> 
> 
> I still believe we should be shipping the GL headers in this package.
> But I won't object to removing the headers temporarily using this recipe
> until the conflicting ones can be removed from the OpenCL package.

Previously DDK only provided headers in these dirs: EGL, GLES, GLES2, KHR, gbm.

And OpenCL required GL headers, hence there was a "hacky" package created 
specifically for that:
http://arago-project.org/git/?p=meta-arago.git;a=blob;f=meta-arago-extras/recipes-ti/ocl/ocl-gl-headers_git.bb;hb=HEAD

If DDK now properly provides GL headers, the other package can be dropped.

But the question remains - should DDK actually provide GL headers, even though 
it doesn't provide full support for it?

-- 
Denys


> >>>>
> >>>>
> >>>>>>          chown -R root:root ${D}
> >>>>>>    }
> >>>>>> @@ -58,7 +60,7 @@ FILES_${PN} +=  "${includedir}/*"
> >>>>>>    FILES_${PN} +=  "${sysconfdir}/*"
> >>>>>>      PACKAGES =+ "${PN}-plugins"
> >>>>>> -FILES_${PN}-plugins = "${libdir}/libsrv_init.so
> >>>>>> ${libdir}/libsrv_um.so ${libdir}/libglslcompiler.so
> >>>>>> ${libdir}/libPVRScopeServices.so ${libdir}/libGLESv2.so
> >>>>>> ${libdir}/libEGL.so ${libdir}/libGLES_CM.so
> >>>>>> ${libdir}/libpvrDRMWSEGL.so  ${libdir}/libpvrGBMWSEGL.so
> >>>>>> ${libdir}/libpvrws_WAYLAND.so"
> >>>>>> +FILES_${PN}-plugins = "${libdir}/libsrv_init.so
> >>>>>> ${libdir}/libsrv_um.so ${libdir}/libglslcompiler.so
> >>>>>> ${libdir}/libPVRScopeServices.so ${libdir}/libGLESv2.so
> >>>>>> ${libdir}/libEGL.so ${libdir}/libGLESv1_CM.so ${libdir}/libGLES_CM.so
> >>>>>> ${libdir}/libGLESv1_PVR_MESA.so ${libdir}/libGLESv2_PVR_MESA.so"
> >>>>>>    RDEPENDS_${PN} += "${PN}-plugins"
> >>>>>
> >>>>>
> >>>>> The newer binaries after the DDK commit "um: Attempt to load shared
> >>>>> object with version extension automatically" do not need all this
> >>>>> plugin
> >>>>> stuff, it can all be dropped.
> >>>>>
> >>>>> Andrew
> >>>>>
> >>>>>
> >>>>>>      ALLOW_EMPTY_${PN}-plugins = "1"
> >>>>>>
> >>>>
> >>> -- 
> >>> _______________________________________________
> >>> meta-ti mailing list
> >>> meta-ti@yoctoproject.org
> >>> https://lists.yoctoproject.org/listinfo/meta-ti
> > 
> -- 
> _______________________________________________
> meta-ti mailing list
> meta-ti@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-ti


^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [EXTERNAL] Re: [PATCH] ti-sgx-ddk-um: update SRCREV to pick up Mesa-based EGL/GLES libraries
  2019-10-30 15:06             ` Denys Dmytriyenko
@ 2019-10-30 15:14               ` Ruei, Eric
  2019-10-30 16:00                 ` Denys Dmytriyenko
  2019-10-30 15:14               ` Andrew F. Davis
  1 sibling, 1 reply; 15+ messages in thread
From: Ruei, Eric @ 2019-10-30 15:14 UTC (permalink / raw)
  To: Denys Dmytriyenko, Andrew F. Davis; +Cc: meta-ti

Hi, guys:

Is there anything that we do regarding the large package size (1M to 8M)?

Best regards,

Eric

On 10/30/2019 11:06 AM, Denys Dmytriyenko wrote:
> On Wed, Oct 30, 2019 at 10:58:31AM -0400, Andrew F. Davis wrote:
>> On 10/30/19 10:53 AM, Ruei, Eric wrote:
>>> On 10/30/2019 9:53 AM, Tammana, Gowtham wrote:
>>>>
>>>>
>>>>> -----Original Message-----
>>>>> From: meta-ti-bounces@yoctoproject.org [mailto:meta-ti-
>>>>> bounces@yoctoproject.org] On Behalf Of Davis, Andrew
>>>>> Sent: Wednesday, October 30, 2019 8:36 AM
>>>>> To: Ruei, Eric; Ruei, Eric; meta-ti@yoctoproject.org
>>>>> Subject: [EXTERNAL] Re: [meta-ti] [PATCH] ti-sgx-ddk-um: update
>>>>> SRCREV to pick
>>>>> up Mesa-based EGL/GLES libraries
>>>>>
>>>>> On 10/30/19 9:31 AM, Ruei, Eric wrote:
>>>>>> On 10/30/2019 9:22 AM, Andrew F. Davis wrote:
>>>>>>> On 10/29/19 9:20 AM, Eric Ruei wrote:
>>>>>>>> This is the initial step toward Mesa-based EGL/GLES libraries which
>>>>>>>> support all the required EGL 1.5 extensions. We plan to provide a
>>>>>>>> Mesa-pvr recipe to build Mesa from source and SGX/DDK patches where
>>>>>>>> ti-sgx-ddk-um shall provide the EGL/GLES plugins only at the next
>>>>>>>> step.
>>>>>>>>
>>>>>>>> Signed-off-by: Eric Ruei <e-ruei1@ti.com>
>>>>>>>> ---
>>>>>>>>     recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb | 8
>>>>>>>> +++++---
>>>>>>>>     1 file changed, 5 insertions(+), 3 deletions(-)
>>>>>>>>
>>>>>>>> diff --git a/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
>>>>>>>> b/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
>>>>>>>> index 7a6f013e..3991d917 100644
>>>>>>>> --- a/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
>>>>>>>> +++ b/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
>>>>>>>> @@ -11,7 +11,7 @@ PR = "r34"
>>>>>>>>     BRANCH = "ti-img-sgx/thud/${PV}"
>>>>>>>>       SRC_URI =
>>>>>>>> "git://git.ti.com/graphics/omap5-sgx-ddk-um-
>>>>> linux.git;protocol=git;branch=${BRANCH}"
>>>>>>>>
>>>>>>>> -SRCREV = "87d7e5c1e4db1bab048939c9719059d549c1e8dd"
>>>>>>>> +SRCREV = "2a2e5bb090ced870d73ed4edbc54793e952cc6d8"
>>>>>>>>       TARGET_PRODUCT_omap-a15 = "jacinto6evm"
>>>>>>>>     TARGET_PRODUCT_ti33x = "ti335x"
>>>>>>>> @@ -47,7 +47,9 @@ S = "${WORKDIR}/git"
>>>>>>>>       do_install () {
>>>>>>>>         oe_runmake install DESTDIR=${D}
>>>>>>>> TARGET_PRODUCT=${TARGET_PRODUCT}
>>>>>>>> -    ln -sf libGLESv2.so.${PV} ${D}${libdir}/libGLESv2.so.1
>>>>>>>> +    ln -sf libGLESv2.so ${D}${libdir}/libGLESv2.so.1
>>>>>>>> +
>>>>>>>> +    rm -rf ${D}${includedir}/GL
>>>>>>>
>>>>>>>
>>>>>>> Why remove this?
>>>>>>>
>>>>>>>
>>>>>>
>>>>>> There is another component provides GL header files.
>>>>>> Denys: how do we resolve this conflict?
>>>>>>
>>>>>
>>>>>
>>>>> The DSP OpenCL implementation? That package needs fixed, not this one,
>>>>> the OpenGL implementation (this driver) should provide the GL headers.
>>>>
>>>> We don't support desktop GL, they shouldn't come from this package.
>>>>
>>>> Gowtham
>>>>
>>>
>>> Andrew:
>>>
>>> Do you agree? I can keep the line here tentatively until GL is removed
>>> from the package itself.
>>>
>>
>>
>> I still believe we should be shipping the GL headers in this package.
>> But I won't object to removing the headers temporarily using this recipe
>> until the conflicting ones can be removed from the OpenCL package.
> 
> Previously DDK only provided headers in these dirs: EGL, GLES, GLES2, KHR, gbm.
> 
> And OpenCL required GL headers, hence there was a "hacky" package created
> specifically for that:
> http://arago-project.org/git/?p=meta-arago.git;a=blob;f=meta-arago-extras/recipes-ti/ocl/ocl-gl-headers_git.bb;hb=HEAD
> 
> If DDK now properly provides GL headers, the other package can be dropped.
> 
> But the question remains - should DDK actually provide GL headers, even though
> it doesn't provide full support for it?
> 



^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [EXTERNAL] Re: [PATCH] ti-sgx-ddk-um: update SRCREV to pick up Mesa-based EGL/GLES libraries
  2019-10-30 15:06             ` Denys Dmytriyenko
  2019-10-30 15:14               ` Ruei, Eric
@ 2019-10-30 15:14               ` Andrew F. Davis
  1 sibling, 0 replies; 15+ messages in thread
From: Andrew F. Davis @ 2019-10-30 15:14 UTC (permalink / raw)
  To: Denys Dmytriyenko; +Cc: Ruei, Eric, meta-ti

On 10/30/19 11:06 AM, Denys Dmytriyenko wrote:
> On Wed, Oct 30, 2019 at 10:58:31AM -0400, Andrew F. Davis wrote:
>> On 10/30/19 10:53 AM, Ruei, Eric wrote:
>>> On 10/30/2019 9:53 AM, Tammana, Gowtham wrote:
>>>>
>>>>
>>>>> -----Original Message-----
>>>>> From: meta-ti-bounces@yoctoproject.org [mailto:meta-ti-
>>>>> bounces@yoctoproject.org] On Behalf Of Davis, Andrew
>>>>> Sent: Wednesday, October 30, 2019 8:36 AM
>>>>> To: Ruei, Eric; Ruei, Eric; meta-ti@yoctoproject.org
>>>>> Subject: [EXTERNAL] Re: [meta-ti] [PATCH] ti-sgx-ddk-um: update
>>>>> SRCREV to pick
>>>>> up Mesa-based EGL/GLES libraries
>>>>>
>>>>> On 10/30/19 9:31 AM, Ruei, Eric wrote:
>>>>>> On 10/30/2019 9:22 AM, Andrew F. Davis wrote:
>>>>>>> On 10/29/19 9:20 AM, Eric Ruei wrote:
>>>>>>>> This is the initial step toward Mesa-based EGL/GLES libraries which
>>>>>>>> support all the required EGL 1.5 extensions. We plan to provide a
>>>>>>>> Mesa-pvr recipe to build Mesa from source and SGX/DDK patches where
>>>>>>>> ti-sgx-ddk-um shall provide the EGL/GLES plugins only at the next
>>>>>>>> step.
>>>>>>>>
>>>>>>>> Signed-off-by: Eric Ruei <e-ruei1@ti.com>
>>>>>>>> ---
>>>>>>>>    recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb | 8
>>>>>>>> +++++---
>>>>>>>>    1 file changed, 5 insertions(+), 3 deletions(-)
>>>>>>>>
>>>>>>>> diff --git a/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
>>>>>>>> b/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
>>>>>>>> index 7a6f013e..3991d917 100644
>>>>>>>> --- a/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
>>>>>>>> +++ b/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
>>>>>>>> @@ -11,7 +11,7 @@ PR = "r34"
>>>>>>>>    BRANCH = "ti-img-sgx/thud/${PV}"
>>>>>>>>      SRC_URI =
>>>>>>>> "git://git.ti.com/graphics/omap5-sgx-ddk-um-
>>>>> linux.git;protocol=git;branch=${BRANCH}"
>>>>>>>>
>>>>>>>> -SRCREV = "87d7e5c1e4db1bab048939c9719059d549c1e8dd"
>>>>>>>> +SRCREV = "2a2e5bb090ced870d73ed4edbc54793e952cc6d8"
>>>>>>>>      TARGET_PRODUCT_omap-a15 = "jacinto6evm"
>>>>>>>>    TARGET_PRODUCT_ti33x = "ti335x"
>>>>>>>> @@ -47,7 +47,9 @@ S = "${WORKDIR}/git"
>>>>>>>>      do_install () {
>>>>>>>>        oe_runmake install DESTDIR=${D}
>>>>>>>> TARGET_PRODUCT=${TARGET_PRODUCT}
>>>>>>>> -    ln -sf libGLESv2.so.${PV} ${D}${libdir}/libGLESv2.so.1
>>>>>>>> +    ln -sf libGLESv2.so ${D}${libdir}/libGLESv2.so.1
>>>>>>>> +
>>>>>>>> +    rm -rf ${D}${includedir}/GL
>>>>>>>
>>>>>>>
>>>>>>> Why remove this?
>>>>>>>
>>>>>>>
>>>>>>
>>>>>> There is another component provides GL header files.
>>>>>> Denys: how do we resolve this conflict?
>>>>>>
>>>>>
>>>>>
>>>>> The DSP OpenCL implementation? That package needs fixed, not this one,
>>>>> the OpenGL implementation (this driver) should provide the GL headers.
>>>>
>>>> We don't support desktop GL, they shouldn't come from this package.
>>>>
>>>> Gowtham
>>>>
>>>
>>> Andrew:
>>>
>>> Do you agree? I can keep the line here tentatively until GL is removed
>>> from the package itself.
>>>
>>
>>
>> I still believe we should be shipping the GL headers in this package.
>> But I won't object to removing the headers temporarily using this recipe
>> until the conflicting ones can be removed from the OpenCL package.
> 
> Previously DDK only provided headers in these dirs: EGL, GLES, GLES2, KHR, gbm.
> 
> And OpenCL required GL headers, hence there was a "hacky" package created 
> specifically for that:
> http://arago-project.org/git/?p=meta-arago.git;a=blob;f=meta-arago-extras/recipes-ti/ocl/ocl-gl-headers_git.bb;hb=HEAD
> 
> If DDK now properly provides GL headers, the other package can be dropped.
> 
> But the question remains - should DDK actually provide GL headers, even though 
> it doesn't provide full support for it?
> 

The GL headers are the stock headers provided by Kronos that every GL
implementation should provide, they often contain definitions for
unsupported features (even the GLES folders have unsupported extension
definitions). The exact set of usable features can be determined at
runtime using various provided methods.

Basically it is up to the EGL layer to report back what the underlying
implementation supports, it should not be based on the content/age of
the headers at compile time. Without these headers valid/usable programs
may not compile on our platform.

Andrew


^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [EXTERNAL] Re: [PATCH] ti-sgx-ddk-um: update SRCREV to pick up Mesa-based EGL/GLES libraries
  2019-10-30 15:14               ` Ruei, Eric
@ 2019-10-30 16:00                 ` Denys Dmytriyenko
  0 siblings, 0 replies; 15+ messages in thread
From: Denys Dmytriyenko @ 2019-10-30 16:00 UTC (permalink / raw)
  To: Ruei, Eric; +Cc: meta-ti

On Wed, Oct 30, 2019 at 11:14:34AM -0400, Ruei, Eric wrote:
> Hi, guys:
> 
> Is there anything that we do regarding the large package size (1M to 8M)?

All the standard libs in /usr/lib get stripped properly.
But there's a new /usr/lib/dri/pvr_dri.so that is 28 MB even stripped. Is it 
supposed to be this large?

-- 
Denys


> Best regards,
> 
> Eric
> 
> On 10/30/2019 11:06 AM, Denys Dmytriyenko wrote:
> >On Wed, Oct 30, 2019 at 10:58:31AM -0400, Andrew F. Davis wrote:
> >>On 10/30/19 10:53 AM, Ruei, Eric wrote:
> >>>On 10/30/2019 9:53 AM, Tammana, Gowtham wrote:
> >>>>
> >>>>
> >>>>>-----Original Message-----
> >>>>>From: meta-ti-bounces@yoctoproject.org [mailto:meta-ti-
> >>>>>bounces@yoctoproject.org] On Behalf Of Davis, Andrew
> >>>>>Sent: Wednesday, October 30, 2019 8:36 AM
> >>>>>To: Ruei, Eric; Ruei, Eric; meta-ti@yoctoproject.org
> >>>>>Subject: [EXTERNAL] Re: [meta-ti] [PATCH] ti-sgx-ddk-um: update
> >>>>>SRCREV to pick
> >>>>>up Mesa-based EGL/GLES libraries
> >>>>>
> >>>>>On 10/30/19 9:31 AM, Ruei, Eric wrote:
> >>>>>>On 10/30/2019 9:22 AM, Andrew F. Davis wrote:
> >>>>>>>On 10/29/19 9:20 AM, Eric Ruei wrote:
> >>>>>>>>This is the initial step toward Mesa-based EGL/GLES libraries which
> >>>>>>>>support all the required EGL 1.5 extensions. We plan to provide a
> >>>>>>>>Mesa-pvr recipe to build Mesa from source and SGX/DDK patches where
> >>>>>>>>ti-sgx-ddk-um shall provide the EGL/GLES plugins only at the next
> >>>>>>>>step.
> >>>>>>>>
> >>>>>>>>Signed-off-by: Eric Ruei <e-ruei1@ti.com>
> >>>>>>>>---
> >>>>>>>>    recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb | 8
> >>>>>>>>+++++---
> >>>>>>>>    1 file changed, 5 insertions(+), 3 deletions(-)
> >>>>>>>>
> >>>>>>>>diff --git a/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
> >>>>>>>>b/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
> >>>>>>>>index 7a6f013e..3991d917 100644
> >>>>>>>>--- a/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
> >>>>>>>>+++ b/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
> >>>>>>>>@@ -11,7 +11,7 @@ PR = "r34"
> >>>>>>>>    BRANCH = "ti-img-sgx/thud/${PV}"
> >>>>>>>>      SRC_URI =
> >>>>>>>>"git://git.ti.com/graphics/omap5-sgx-ddk-um-
> >>>>>linux.git;protocol=git;branch=${BRANCH}"
> >>>>>>>>
> >>>>>>>>-SRCREV = "87d7e5c1e4db1bab048939c9719059d549c1e8dd"
> >>>>>>>>+SRCREV = "2a2e5bb090ced870d73ed4edbc54793e952cc6d8"
> >>>>>>>>      TARGET_PRODUCT_omap-a15 = "jacinto6evm"
> >>>>>>>>    TARGET_PRODUCT_ti33x = "ti335x"
> >>>>>>>>@@ -47,7 +47,9 @@ S = "${WORKDIR}/git"
> >>>>>>>>      do_install () {
> >>>>>>>>        oe_runmake install DESTDIR=${D}
> >>>>>>>>TARGET_PRODUCT=${TARGET_PRODUCT}
> >>>>>>>>-    ln -sf libGLESv2.so.${PV} ${D}${libdir}/libGLESv2.so.1
> >>>>>>>>+    ln -sf libGLESv2.so ${D}${libdir}/libGLESv2.so.1
> >>>>>>>>+
> >>>>>>>>+    rm -rf ${D}${includedir}/GL
> >>>>>>>
> >>>>>>>
> >>>>>>>Why remove this?
> >>>>>>>
> >>>>>>>
> >>>>>>
> >>>>>>There is another component provides GL header files.
> >>>>>>Denys: how do we resolve this conflict?
> >>>>>>
> >>>>>
> >>>>>
> >>>>>The DSP OpenCL implementation? That package needs fixed, not this one,
> >>>>>the OpenGL implementation (this driver) should provide the GL headers.
> >>>>
> >>>>We don't support desktop GL, they shouldn't come from this package.
> >>>>
> >>>>Gowtham
> >>>>
> >>>
> >>>Andrew:
> >>>
> >>>Do you agree? I can keep the line here tentatively until GL is removed
> >>>from the package itself.
> >>>
> >>
> >>
> >>I still believe we should be shipping the GL headers in this package.
> >>But I won't object to removing the headers temporarily using this recipe
> >>until the conflicting ones can be removed from the OpenCL package.
> >
> >Previously DDK only provided headers in these dirs: EGL, GLES, GLES2, KHR, gbm.
> >
> >And OpenCL required GL headers, hence there was a "hacky" package created
> >specifically for that:
> >http://arago-project.org/git/?p=meta-arago.git;a=blob;f=meta-arago-extras/recipes-ti/ocl/ocl-gl-headers_git.bb;hb=HEAD
> >
> >If DDK now properly provides GL headers, the other package can be dropped.
> >
> >But the question remains - should DDK actually provide GL headers, even though
> >it doesn't provide full support for it?
> >
> 


^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [EXTERNAL] Re: [PATCH] ti-sgx-ddk-um: update SRCREV to pick up Mesa-based EGL/GLES libraries
  2019-10-30 14:58           ` Andrew F. Davis
  2019-10-30 15:06             ` Denys Dmytriyenko
@ 2019-10-31 12:39             ` Ruei, Eric
  2019-11-04 15:28               ` Andrew F. Davis
  1 sibling, 1 reply; 15+ messages in thread
From: Ruei, Eric @ 2019-10-31 12:39 UTC (permalink / raw)
  To: Andrew F. Davis, Tammana, Gowtham, Ruei, Eric, meta-ti

Hi, Andrew, Gowtham and Denys:

What is our conclusion (solution) for the package size increment?

Best regards,

Eric

On 10/30/2019 10:58 AM, Andrew F. Davis wrote:
> On 10/30/19 10:53 AM, Ruei, Eric wrote:
>> On 10/30/2019 9:53 AM, Tammana, Gowtham wrote:
>>>
>>>
>>>> -----Original Message-----
>>>> From: meta-ti-bounces@yoctoproject.org [mailto:meta-ti-
>>>> bounces@yoctoproject.org] On Behalf Of Davis, Andrew
>>>> Sent: Wednesday, October 30, 2019 8:36 AM
>>>> To: Ruei, Eric; Ruei, Eric; meta-ti@yoctoproject.org
>>>> Subject: [EXTERNAL] Re: [meta-ti] [PATCH] ti-sgx-ddk-um: update
>>>> SRCREV to pick
>>>> up Mesa-based EGL/GLES libraries
>>>>
>>>> On 10/30/19 9:31 AM, Ruei, Eric wrote:
>>>>> On 10/30/2019 9:22 AM, Andrew F. Davis wrote:
>>>>>> On 10/29/19 9:20 AM, Eric Ruei wrote:
>>>>>>> This is the initial step toward Mesa-based EGL/GLES libraries which
>>>>>>> support all the required EGL 1.5 extensions. We plan to provide a
>>>>>>> Mesa-pvr recipe to build Mesa from source and SGX/DDK patches where
>>>>>>> ti-sgx-ddk-um shall provide the EGL/GLES plugins only at the next
>>>>>>> step.
>>>>>>>
>>>>>>> Signed-off-by: Eric Ruei <e-ruei1@ti.com>
>>>>>>> ---
>>>>>>>     recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb | 8
>>>>>>> +++++---
>>>>>>>     1 file changed, 5 insertions(+), 3 deletions(-)
>>>>>>>
>>>>>>> diff --git a/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
>>>>>>> b/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
>>>>>>> index 7a6f013e..3991d917 100644
>>>>>>> --- a/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
>>>>>>> +++ b/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
>>>>>>> @@ -11,7 +11,7 @@ PR = "r34"
>>>>>>>     BRANCH = "ti-img-sgx/thud/${PV}"
>>>>>>>       SRC_URI =
>>>>>>> "git://git.ti.com/graphics/omap5-sgx-ddk-um-
>>>> linux.git;protocol=git;branch=${BRANCH}"
>>>>>>>
>>>>>>> -SRCREV = "87d7e5c1e4db1bab048939c9719059d549c1e8dd"
>>>>>>> +SRCREV = "2a2e5bb090ced870d73ed4edbc54793e952cc6d8"
>>>>>>>       TARGET_PRODUCT_omap-a15 = "jacinto6evm"
>>>>>>>     TARGET_PRODUCT_ti33x = "ti335x"
>>>>>>> @@ -47,7 +47,9 @@ S = "${WORKDIR}/git"
>>>>>>>       do_install () {
>>>>>>>         oe_runmake install DESTDIR=${D}
>>>>>>> TARGET_PRODUCT=${TARGET_PRODUCT}
>>>>>>> -    ln -sf libGLESv2.so.${PV} ${D}${libdir}/libGLESv2.so.1
>>>>>>> +    ln -sf libGLESv2.so ${D}${libdir}/libGLESv2.so.1
>>>>>>> +
>>>>>>> +    rm -rf ${D}${includedir}/GL
>>>>>>
>>>>>>
>>>>>> Why remove this?
>>>>>>
>>>>>>
>>>>>
>>>>> There is another component provides GL header files.
>>>>> Denys: how do we resolve this conflict?
>>>>>
>>>>
>>>>
>>>> The DSP OpenCL implementation? That package needs fixed, not this one,
>>>> the OpenGL implementation (this driver) should provide the GL headers.
>>>
>>> We don't support desktop GL, they shouldn't come from this package.
>>>
>>> Gowtham
>>>
>>
>> Andrew:
>>
>> Do you agree? I can keep the line here tentatively until GL is removed
>> from the package itself.
>>
> 
> 
> I still believe we should be shipping the GL headers in this package.
> But I won't object to removing the headers temporarily using this recipe
> until the conflicting ones can be removed from the OpenCL package.
> 
> Andrew
> 
> 
>> Eric
>>
>>
>>
>>>>>
>>>>>
>>>>>>>           chown -R root:root ${D}
>>>>>>>     }
>>>>>>> @@ -58,7 +60,7 @@ FILES_${PN} +=  "${includedir}/*"
>>>>>>>     FILES_${PN} +=  "${sysconfdir}/*"
>>>>>>>       PACKAGES =+ "${PN}-plugins"
>>>>>>> -FILES_${PN}-plugins = "${libdir}/libsrv_init.so
>>>>>>> ${libdir}/libsrv_um.so ${libdir}/libglslcompiler.so
>>>>>>> ${libdir}/libPVRScopeServices.so ${libdir}/libGLESv2.so
>>>>>>> ${libdir}/libEGL.so ${libdir}/libGLES_CM.so
>>>>>>> ${libdir}/libpvrDRMWSEGL.so  ${libdir}/libpvrGBMWSEGL.so
>>>>>>> ${libdir}/libpvrws_WAYLAND.so"
>>>>>>> +FILES_${PN}-plugins = "${libdir}/libsrv_init.so
>>>>>>> ${libdir}/libsrv_um.so ${libdir}/libglslcompiler.so
>>>>>>> ${libdir}/libPVRScopeServices.so ${libdir}/libGLESv2.so
>>>>>>> ${libdir}/libEGL.so ${libdir}/libGLESv1_CM.so ${libdir}/libGLES_CM.so
>>>>>>> ${libdir}/libGLESv1_PVR_MESA.so ${libdir}/libGLESv2_PVR_MESA.so"
>>>>>>>     RDEPENDS_${PN} += "${PN}-plugins"
>>>>>>
>>>>>>
>>>>>> The newer binaries after the DDK commit "um: Attempt to load shared
>>>>>> object with version extension automatically" do not need all this
>>>>>> plugin
>>>>>> stuff, it can all be dropped.
>>>>>>
>>>>>> Andrew
>>>>>>
>>>>>>
>>>>>>>       ALLOW_EMPTY_${PN}-plugins = "1"
>>>>>>>
>>>>>
>>>> -- 
>>>> _______________________________________________
>>>> meta-ti mailing list
>>>> meta-ti@yoctoproject.org
>>>> https://lists.yoctoproject.org/listinfo/meta-ti
>>



^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [EXTERNAL] Re: [PATCH] ti-sgx-ddk-um: update SRCREV to pick up Mesa-based EGL/GLES libraries
  2019-10-31 12:39             ` Ruei, Eric
@ 2019-11-04 15:28               ` Andrew F. Davis
  0 siblings, 0 replies; 15+ messages in thread
From: Andrew F. Davis @ 2019-11-04 15:28 UTC (permalink / raw)
  To: Ruei, Eric, Tammana, Gowtham, Ruei, Eric, meta-ti

On 10/31/19 8:39 AM, Ruei, Eric wrote:
> Hi, Andrew, Gowtham and Denys:
> 
> What is our conclusion (solution) for the package size increment?
> 


As said, the Mesa part is a debug build (this is intentional, we really
do want all the symbols and debug strings while enabling this new
feature, or any issues will be much harder to solve). If we do this for
another cycle then I can do a release build then. But the hopeful plan
is to have Mesa built as its own recipe (mesa-pvr) which will mean this
wont be a problem next time (2020lts).

Andrew


> Best regards,
> 
> Eric
> 
> On 10/30/2019 10:58 AM, Andrew F. Davis wrote:
>> On 10/30/19 10:53 AM, Ruei, Eric wrote:
>>> On 10/30/2019 9:53 AM, Tammana, Gowtham wrote:
>>>>
>>>>
>>>>> -----Original Message-----
>>>>> From: meta-ti-bounces@yoctoproject.org [mailto:meta-ti-
>>>>> bounces@yoctoproject.org] On Behalf Of Davis, Andrew
>>>>> Sent: Wednesday, October 30, 2019 8:36 AM
>>>>> To: Ruei, Eric; Ruei, Eric; meta-ti@yoctoproject.org
>>>>> Subject: [EXTERNAL] Re: [meta-ti] [PATCH] ti-sgx-ddk-um: update
>>>>> SRCREV to pick
>>>>> up Mesa-based EGL/GLES libraries
>>>>>
>>>>> On 10/30/19 9:31 AM, Ruei, Eric wrote:
>>>>>> On 10/30/2019 9:22 AM, Andrew F. Davis wrote:
>>>>>>> On 10/29/19 9:20 AM, Eric Ruei wrote:
>>>>>>>> This is the initial step toward Mesa-based EGL/GLES libraries which
>>>>>>>> support all the required EGL 1.5 extensions. We plan to provide a
>>>>>>>> Mesa-pvr recipe to build Mesa from source and SGX/DDK patches where
>>>>>>>> ti-sgx-ddk-um shall provide the EGL/GLES plugins only at the next
>>>>>>>> step.
>>>>>>>>
>>>>>>>> Signed-off-by: Eric Ruei <e-ruei1@ti.com>
>>>>>>>> ---
>>>>>>>>     recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb | 8
>>>>>>>> +++++---
>>>>>>>>     1 file changed, 5 insertions(+), 3 deletions(-)
>>>>>>>>
>>>>>>>> diff --git a/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
>>>>>>>> b/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
>>>>>>>> index 7a6f013e..3991d917 100644
>>>>>>>> --- a/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
>>>>>>>> +++ b/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb
>>>>>>>> @@ -11,7 +11,7 @@ PR = "r34"
>>>>>>>>     BRANCH = "ti-img-sgx/thud/${PV}"
>>>>>>>>       SRC_URI =
>>>>>>>> "git://git.ti.com/graphics/omap5-sgx-ddk-um-
>>>>> linux.git;protocol=git;branch=${BRANCH}"
>>>>>>>>
>>>>>>>> -SRCREV = "87d7e5c1e4db1bab048939c9719059d549c1e8dd"
>>>>>>>> +SRCREV = "2a2e5bb090ced870d73ed4edbc54793e952cc6d8"
>>>>>>>>       TARGET_PRODUCT_omap-a15 = "jacinto6evm"
>>>>>>>>     TARGET_PRODUCT_ti33x = "ti335x"
>>>>>>>> @@ -47,7 +47,9 @@ S = "${WORKDIR}/git"
>>>>>>>>       do_install () {
>>>>>>>>         oe_runmake install DESTDIR=${D}
>>>>>>>> TARGET_PRODUCT=${TARGET_PRODUCT}
>>>>>>>> -    ln -sf libGLESv2.so.${PV} ${D}${libdir}/libGLESv2.so.1
>>>>>>>> +    ln -sf libGLESv2.so ${D}${libdir}/libGLESv2.so.1
>>>>>>>> +
>>>>>>>> +    rm -rf ${D}${includedir}/GL
>>>>>>>
>>>>>>>
>>>>>>> Why remove this?
>>>>>>>
>>>>>>>
>>>>>>
>>>>>> There is another component provides GL header files.
>>>>>> Denys: how do we resolve this conflict?
>>>>>>
>>>>>
>>>>>
>>>>> The DSP OpenCL implementation? That package needs fixed, not this one,
>>>>> the OpenGL implementation (this driver) should provide the GL headers.
>>>>
>>>> We don't support desktop GL, they shouldn't come from this package.
>>>>
>>>> Gowtham
>>>>
>>>
>>> Andrew:
>>>
>>> Do you agree? I can keep the line here tentatively until GL is removed
>>> from the package itself.
>>>
>>
>>
>> I still believe we should be shipping the GL headers in this package.
>> But I won't object to removing the headers temporarily using this recipe
>> until the conflicting ones can be removed from the OpenCL package.
>>
>> Andrew
>>
>>
>>> Eric
>>>
>>>
>>>
>>>>>>
>>>>>>
>>>>>>>>           chown -R root:root ${D}
>>>>>>>>     }
>>>>>>>> @@ -58,7 +60,7 @@ FILES_${PN} +=  "${includedir}/*"
>>>>>>>>     FILES_${PN} +=  "${sysconfdir}/*"
>>>>>>>>       PACKAGES =+ "${PN}-plugins"
>>>>>>>> -FILES_${PN}-plugins = "${libdir}/libsrv_init.so
>>>>>>>> ${libdir}/libsrv_um.so ${libdir}/libglslcompiler.so
>>>>>>>> ${libdir}/libPVRScopeServices.so ${libdir}/libGLESv2.so
>>>>>>>> ${libdir}/libEGL.so ${libdir}/libGLES_CM.so
>>>>>>>> ${libdir}/libpvrDRMWSEGL.so  ${libdir}/libpvrGBMWSEGL.so
>>>>>>>> ${libdir}/libpvrws_WAYLAND.so"
>>>>>>>> +FILES_${PN}-plugins = "${libdir}/libsrv_init.so
>>>>>>>> ${libdir}/libsrv_um.so ${libdir}/libglslcompiler.so
>>>>>>>> ${libdir}/libPVRScopeServices.so ${libdir}/libGLESv2.so
>>>>>>>> ${libdir}/libEGL.so ${libdir}/libGLESv1_CM.so
>>>>>>>> ${libdir}/libGLES_CM.so
>>>>>>>> ${libdir}/libGLESv1_PVR_MESA.so ${libdir}/libGLESv2_PVR_MESA.so"
>>>>>>>>     RDEPENDS_${PN} += "${PN}-plugins"
>>>>>>>
>>>>>>>
>>>>>>> The newer binaries after the DDK commit "um: Attempt to load shared
>>>>>>> object with version extension automatically" do not need all this
>>>>>>> plugin
>>>>>>> stuff, it can all be dropped.
>>>>>>>
>>>>>>> Andrew
>>>>>>>
>>>>>>>
>>>>>>>>       ALLOW_EMPTY_${PN}-plugins = "1"
>>>>>>>>
>>>>>>
>>>>> -- 
>>>>> _______________________________________________
>>>>> meta-ti mailing list
>>>>> meta-ti@yoctoproject.org
>>>>> https://lists.yoctoproject.org/listinfo/meta-ti
>>>
> 


^ permalink raw reply	[flat|nested] 15+ messages in thread

end of thread, other threads:[~2019-11-04 15:28 UTC | newest]

Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-10-29 13:20 [PATCH] ti-sgx-ddk-um: update SRCREV to pick up Mesa-based EGL/GLES libraries Eric Ruei
2019-10-29 23:29 ` Denys Dmytriyenko
2019-10-30 13:16   ` Ruei, Eric
2019-10-30 13:22 ` Andrew F. Davis
2019-10-30 13:31   ` Ruei, Eric
2019-10-30 13:36     ` Andrew F. Davis
2019-10-30 13:53       ` [EXTERNAL] " Tammana, Gowtham
2019-10-30 14:53         ` Ruei, Eric
2019-10-30 14:58           ` Andrew F. Davis
2019-10-30 15:06             ` Denys Dmytriyenko
2019-10-30 15:14               ` Ruei, Eric
2019-10-30 16:00                 ` Denys Dmytriyenko
2019-10-30 15:14               ` Andrew F. Davis
2019-10-31 12:39             ` Ruei, Eric
2019-11-04 15:28               ` Andrew F. Davis

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.