[oe] Patches to add support for TI Linux Graphics SDK release 04.04.00.02
Srinivas, Prathap
msprathap at ti.com
Thu Nov 17 10:43:31 UTC 2011
The comments provided have been addressed.
Please find the updated patches attached.
Thanks,
Prathap.
-----Original Message-----
From: openembedded-devel-bounces at lists.openembedded.org [mailto:openembedded-devel-bounces at lists.openembedded.org] On Behalf Of Koen Kooi
Sent: Wednesday, October 12, 2011 4:28 PM
To: openembedded-devel at lists.openembedded.org
Subject: Re: [oe] Patches to add support for TI Linux Graphics SDK release 04.04.00.02
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Op 12-10-11 12:42, Srinivas, Prathap schreef:
> Thanks for the comments. Please find my responses inline.
>> diff --git a/recipes/powervr-drivers/libgles-omap3_4.04.00.02.bb
>> b/recipes/powervr-drivers/libgles-omap3_4.04.00.02.bb new file mode
>> 100644 index 0000000..c5062c1 --- /dev/null +++
>> b/recipes/powervr-drivers/libgles-omap3_4.04.00.02.bb @@ -0,0 +1,29 @@
>> +BINLOCATION_omap3 = "${S}/gfx_rel_es3.x" +BINLOCATION_ti816x =
>> "${S}/gfx_rel_es6.x" + +ES2LOCATION = "${S}/gfx_rel_es2.x" +ES3LOCATION
>> = "${S}/gfx_rel_es3.x" +ES5LOCATION = "${S}/gfx_rel_es5.x" +ES6LOCATION
>> = "${S}/gfx_rel_es6.x"
>
> Since the above is being repeated in each recipe, shouldn't it move to
> the .inc? - I am just following in the way of already existing recipes.
> Existing recipes have this. If it still needs to be addressed, let me
> know.
It needs to be addressed, otherwise I wouldn't have mentioned it
>
>> --- /dev/null +++
>> b/recipes/powervr-drivers/omap3-sgx-modules_1.6.16.4117.bb
>
>> +MACHINE_KERNEL_PR_append = "i"
>
> Start at 'a' please, or remove it - I am just following in the way of
> already existing recipes. Existing recipes have this way only. If it
> still needs to be addressed, let me know.
It needs to be addressed, otherwise I wouldn't have mentioned it
>> +TI_PLATFORM_omap3 = "omap3630" +TI_PLATFORM_ti816x = "ti81xx" +
>> +MODULESLOCATION_omap3 = "dc_omapfb3_linux" +MODULESLOCATION_ti816x =
>> "dc_ti81xx_linux"
>
> Move to .inc? - I am just following in the way of already existing
> recipes. Existing recipes have this way only. If it still needs to be
> addressed, let me know.
It needs to be addressed, otherwise I wouldn't have mentioned it
>> diff --git a/recipes/powervr-drivers/libgles-omap3.inc
>> b/recipes/powervr-drivers/libgles-omap3.inc index 36c311d..e605681
>> 100644 --- a/recipes/powervr-drivers/libgles-omap3.inc +++
>> b/recipes/powervr-drivers/libgles-omap3.inc @@ -1,7 +1,7 @@ DESCRIPTION
>> = "libGLES for the omap3" LICENSE = "proprietary-binary"
>
>> -PR = "r15" +PR = "r16"
>
>> COMPATIBLE_MACHINE = "(omap3|ti816x)"
>
>> @@ -35,6 +35,13 @@ do_configure() { for i in $(find ${S} -name "*.h")
>> $(find ${S} -name "*.c") $(find ${S} -name "Make*") ; do chmod 0644 $i
>> done + + for drifile in $(find ${S} -name "libsrv_um_dri.so"); do + if
>> [ "$drifile" != "" ] ; then + dir=$(dirname
>> ${drifile}) + rm -rf ${dir}/libsrv_um_dri.so +
>> fi + done
>
> What does this try to fix exactly? - This fix is for the issue seen with
> opkg install of libgles-omap3_4.04.00.02-r16.6_armv7a.ipk on target. With
> Graphics SDK release 04.04.00.02, there are 2 versions of the library
> libsrv_um.so - one is libsrv_um.so and other is libsrv_um_dri.so (For
> Xorg driver). What we observed was in presence of libsrv_um_dri.so , the
> links created under /usr/lib were all by default linking to
> libsrv_um_dri.so only. Due to this demos used to fail. Now this fix makes
> sure that only 1 appropriate version of the library libsrv_um.so is
> present and hence links created will now be proper.
=
Please put that explanation in both the commit message and as a comment
about the for loop so people won't say "I am just following in the way of
already existing recipes" when this issues has been properly fixed.
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.5 (Darwin)
Comment: GPGTools - http://gpgtools.org
iD8DBQFOlXKkMkyGM64RGpERAvROAKCCDqURb233i2eJ2mxdCNU2DoyhEACfXQmr
jB2nZhTMFSiQY0LWiRxYedE=
=1KC2
-----END PGP SIGNATURE-----
_______________________________________________
Openembedded-devel mailing list
Openembedded-devel at lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0003-libgles-omap3.inc-Fix-for-library-linking-isssue-wit.patch
Type: application/octet-stream
Size: 2351 bytes
Desc: 0003-libgles-omap3.inc-Fix-for-library-linking-isssue-wit.patch
URL: <http://lists.openembedded.org/pipermail/openembedded-devel/attachments/20111117/37187480/attachment-0006.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-kernel.bbclass-stage-additional-directories.patch
Type: application/octet-stream
Size: 1384 bytes
Desc: 0001-kernel.bbclass-stage-additional-directories.patch
URL: <http://lists.openembedded.org/pipermail/openembedded-devel/attachments/20111117/37187480/attachment-0007.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-libgles-omap3_4.04.00.02-added-support-for-graphics-.patch
Type: application/octet-stream
Size: 4903 bytes
Desc: 0002-libgles-omap3_4.04.00.02-added-support-for-graphics-.patch
URL: <http://lists.openembedded.org/pipermail/openembedded-devel/attachments/20111117/37187480/attachment-0008.obj>
More information about the Openembedded-devel
mailing list