[meta-ti] [PATCH] ti-sgx-ddk-um: update SRCREV to pick up Mesa-based EGL/GLES libraries

[Andrew F. Davis]

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 at 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"
>>>
>