diff mbox

[1/1] ti-gfx: bump to version 5_01_00_01

Message ID 52fd38fd.870eec0a.7f7f.13a5@mx.google.com
State Accepted
Commit 83fb9bb738ae17bf2f11ae3e25250c035a8a7dba
Headers show

Commit Message

Marcelo Gutiérrez(UTN/FRH) Feb. 13, 2014, 6:38 p.m. UTC
From: Marcelo Gutierrez <kuyurix@gmail.com>


Signed-off-by: Marcelo Gutierrez <kuyurix@gmail.com>
---
 package/ti-gfx/ti-gfx.mk |    8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

Comments

Spenser Gilliland Feb. 13, 2014, 10:03 p.m. UTC | #1
On Thu, Feb 13, 2014 at 12:38 PM,  <kuyurix@gmail.com> wrote:
> From: Marcelo Gutierrez <kuyurix@gmail.com>
>
>
> Signed-off-by: Marcelo Gutierrez <kuyurix@gmail.com>
> ---
>  package/ti-gfx/ti-gfx.mk |    8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/package/ti-gfx/ti-gfx.mk b/package/ti-gfx/ti-gfx.mk
> index a94f00d..5ea6b92 100644
> --- a/package/ti-gfx/ti-gfx.mk
> +++ b/package/ti-gfx/ti-gfx.mk
> @@ -4,13 +4,13 @@
>  #
>  ################################################################################
>
> -TI_GFX_VERSION = 4_09_00_01
> -TI_GFX_SO_VERSION = 1.9.2188537
> +TI_GFX_VERSION = 5_01_00_01
> +TI_GFX_SO_VERSION = 1.10.2359475
>
>  ifeq ($(BR2_ARM_EABIHF),y)
> -TI_GFX_SOURCE = Graphics_SDK_setuplinux_$(TI_GFX_VERSION)_hardfp_minimal_demos.bin
> +TI_GFX_SOURCE = Graphics_SDK_setuplinux_hardfp_$(TI_GFX_VERSION).bin
>  else
> -TI_GFX_SOURCE = Graphics_SDK_setuplinux_$(TI_GFX_VERSION)_minimal_demos.bin
> +TI_GFX_SOURCE = Graphics_SDK_setuplinux_softfp_$(TI_GFX_VERSION).bin
>  endif
>
>  TI_GFX_SITE = http://downloads.ti.com/dsps/dsps_public_sw/sdo_sb/targetcontent/gfxsdk/$(TI_GFX_VERSION)/exports/
> --
> 1.7.9.5
>
> _______________________________________________
> buildroot mailing list
> buildroot@busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

Marcelo Gutierrez,

Please see the following site.  The 5_01_00_01 version is not a
supported version; it is of alpha quality and has several
deficiencies.

http://tigraphics.blogspot.com/2013/11/4q-2013-linux-graphics-sdk-release.html

Thanks,
Spenser
Marcelo Gutiérrez(UTN/FRH) Feb. 13, 2014, 10:50 p.m. UTC | #2
Hi Spenser,

Please see the following site.  The 5_01_00_01 version is not a
> supported version; it is of alpha quality and has several
> deficiencies.
>
>
> http://tigraphics.blogspot.com/2013/11/4q-2013-linux-graphics-sdk-release.html
>


I've tested Graphics SDK 5_01_00_01 and run great!
This is the release note:
http://processors.wiki.ti.com/index.php/RN_5_01_00_01

The link that you've sent is about 5_00_00_01_alpha version and maybe that
version got several issues.

On the other hand, a feature in this release(5_01_00_01) is " 3.12 kernel
support"  and if you take a look this commit "beaglebone: update to TI 3.12
kernel"

http://git.buildroot.net/buildroot/commit/configs?id=1ca56e4e159cd5dd44cd3aba997396d49f22ecda

The new update to TI 3.12 will not work with the actual version
"TI_GFX_VERSION = 4_09_00_01"

Regards,
Marcelo
Peter Kümmel Feb. 14, 2014, 9:57 a.m. UTC | #3

Peter Kümmel Feb. 14, 2014, 11:03 a.m. UTC | #4

Marcelo Gutiérrez(UTN/FRH) Feb. 14, 2014, 4:02 p.m. UTC | #5
Hi all,

I'm using hardfloat:

$readelf -A output/target/bin/busybox
Attribute Section: aeabi
File Attributes
  Tag_CPU_name: "7-A"
  Tag_CPU_arch: v7
  Tag_CPU_arch_profile: Application
  Tag_ARM_ISA_use: Yes
  Tag_THUMB_ISA_use: Thumb-2
  Tag_FP_arch: VFPv3
  Tag_Advanced_SIMD_arch: NEONv1
  Tag_ABI_PCS_wchar_t: 4
  Tag_ABI_FP_rounding: Needed
  Tag_ABI_FP_denormal: Needed
  Tag_ABI_FP_exceptions: Needed
  Tag_ABI_FP_number_model: IEEE 754
  Tag_ABI_align_needed: 8-byte
  Tag_ABI_align_preserved: 8-byte, except leaf SP
  Tag_ABI_enum_size: int
  Tag_ABI_HardFP_use: SP and DP
  Tag_ABI_VFP_args: VFP registers
  Tag_CPU_unaligned_access: v6
  Tag_Virtualization_use: TrustZone

$readelf -A pvrsrvctl
Attribute Section: aeabi
File Attributes
  Tag_CPU_name: "7-A"
  Tag_CPU_arch: v7
  Tag_CPU_arch_profile: Application
  Tag_ARM_ISA_use: Yes
  Tag_THUMB_ISA_use: Thumb-2
  Tag_FP_arch: VFPv3-D16
  Tag_ABI_PCS_wchar_t: 4
  Tag_ABI_FP_denormal: Needed
  Tag_ABI_FP_exceptions: Needed
  Tag_ABI_FP_number_model: IEEE 754
  Tag_ABI_align_needed: 8-byte
  Tag_ABI_align_preserved: 8-byte, except leaf SP
  Tag_ABI_enum_size: int
  Tag_ABI_HardFP_use: SP and DP
  Tag_ABI_VFP_args: VFP registers
  Tag_CPU_unaligned_access: v6
  Tag_DIV_use: Not allowed

I tested 5_01_00_01 version on BBB.

>[   15.008605] CPU: 0    Tainted: G        W    (3.0.50-vallon #1)

Your kernel version is 3.0.50 and that could be the issue.

The release notes contain this instruction:
>Pre-requisites
>
>Kernel(Linux PSP) versions supported or used for validation
>
>Only the following devices have been tested & details below -
>
>AM335x & AM43x -
>
>Tree: git://git.ti.com/ti-linux-kernel/ti-linux-kernel.git
>Branch: ti-linux-3.12.y
>commit - dfea149efb32485f17bb7440eb326549c8f647e2, tag - ti2013.12.00
>
>NOTE-
>1.Ensure the reset controller driver is enabled in kernel menuconfig if not enabled by default as below-
>CONFIG_ARCH_HAS_RESET_CONTROLLER=y
>CONFIG_RESET_CONTROLLER=y
>CONFIG_RESET_TI=y

Regards,
Marcelo
Peter Kümmel Feb. 14, 2014, 4:11 p.m. UTC | #6
> 
> Your kernel version is 3.0.50 and that could be the issue.

Indeed, seems my kernel is too old.

Thanks for the explanation.
Peter

> 
> The release notes contain this instruction:
> >Pre-requisites
> >
> >Kernel(Linux PSP) versions supported or used for validation
> >
> >Only the following devices have been tested & details below -
> >
> >AM335x & AM43x -
> >
> >Tree: git://git.ti.com/ti-linux-kernel/ti-linux-kernel.git
> >Branch: ti-linux-3.12.y
> >commit - dfea149efb32485f17bb7440eb326549c8f647e2, tag - ti2013.12.00
> >
> >NOTE-
> >1.Ensure the reset controller driver is enabled in kernel menuconfig if not enabled by default as below-
> >CONFIG_ARCH_HAS_RESET_CONTROLLER=y
> >CONFIG_RESET_CONTROLLER=y
> >CONFIG_RESET_TI=y
> 
> Regards,
> Marcelo
>
Marcelo Gutiérrez(UTN/FRH) Feb. 14, 2014, 4:22 p.m. UTC | #7
You're welcome. ;-)

Regards,
Marcelo
Yann E. MORIN Feb. 23, 2014, 7:31 p.m. UTC | #8
Marcelo, All,

On 2014-02-13 18:38 +0000, kuyurix@gmail.com spake thusly:
> From: Marcelo Gutierrez <kuyurix@gmail.com>
> 
> Signed-off-by: Marcelo Gutierrez <kuyurix@gmail.com>

Reviewed-by: "Yann E. MORIN" <yann.morin.1998@free.fr>

Notes:

  - The version pointed to by Spencer was 5.00.x, and was a beta
    version, and only contained the hardfp variant. The version in
    this patch is 5.01.x, is not marked as beta, and contains both
    the hardfp and softfp variants.

  - With 4.10.x and before, the *_$(TI_GFX_VERSION).bin archive would
    contain a huge load of stuff we did not need, and was roughly
    1.7GiB big. But with 5.01.x, this file is now back to a more
    acceptable size, roughly 140MiB (although it still contains way
    much more than we need).

Regards,
Yann E. MORIN.

> ---
>  package/ti-gfx/ti-gfx.mk |    8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/package/ti-gfx/ti-gfx.mk b/package/ti-gfx/ti-gfx.mk
> index a94f00d..5ea6b92 100644
> --- a/package/ti-gfx/ti-gfx.mk
> +++ b/package/ti-gfx/ti-gfx.mk
> @@ -4,13 +4,13 @@
>  #
>  ################################################################################
>  
> -TI_GFX_VERSION = 4_09_00_01
> -TI_GFX_SO_VERSION = 1.9.2188537
> +TI_GFX_VERSION = 5_01_00_01
> +TI_GFX_SO_VERSION = 1.10.2359475
>  
>  ifeq ($(BR2_ARM_EABIHF),y)
> -TI_GFX_SOURCE = Graphics_SDK_setuplinux_$(TI_GFX_VERSION)_hardfp_minimal_demos.bin
> +TI_GFX_SOURCE = Graphics_SDK_setuplinux_hardfp_$(TI_GFX_VERSION).bin
>  else
> -TI_GFX_SOURCE = Graphics_SDK_setuplinux_$(TI_GFX_VERSION)_minimal_demos.bin
> +TI_GFX_SOURCE = Graphics_SDK_setuplinux_softfp_$(TI_GFX_VERSION).bin
>  endif
>  
>  TI_GFX_SITE = http://downloads.ti.com/dsps/dsps_public_sw/sdo_sb/targetcontent/gfxsdk/$(TI_GFX_VERSION)/exports/
> -- 
> 1.7.9.5
> 
> _______________________________________________
> buildroot mailing list
> buildroot@busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
Peter Korsgaard Feb. 23, 2014, 7:56 p.m. UTC | #9
>>>>> "Yann" == Yann E MORIN <yann.morin.1998@free.fr> writes:

 > Marcelo, All,
 > On 2014-02-13 18:38 +0000, kuyurix@gmail.com spake thusly:
 >> From: Marcelo Gutierrez <kuyurix@gmail.com>
 >> 
 >> Signed-off-by: Marcelo Gutierrez <kuyurix@gmail.com>

 > Reviewed-by: "Yann E. MORIN" <yann.morin.1998@free.fr>

 > Notes:

 >   - The version pointed to by Spencer was 5.00.x, and was a beta
 >     version, and only contained the hardfp variant. The version in
 >     this patch is 5.01.x, is not marked as beta, and contains both
 >     the hardfp and softfp variants.

 >   - With 4.10.x and before, the *_$(TI_GFX_VERSION).bin archive would
 >     contain a huge load of stuff we did not need, and was roughly
 >     1.7GiB big. But with 5.01.x, this file is now back to a more
 >     acceptable size, roughly 140MiB (although it still contains way
 >     much more than we need).

Sounds good. Committed to next, thanks both.
Peter Korsgaard Feb. 26, 2014, 1:31 p.m. UTC | #10
>>>>> "Peter" == Peter Korsgaard <jacmet@uclibc.org> writes:

>>>>> "Yann" == Yann E MORIN <yann.morin.1998@free.fr> writes:
 >> Marcelo, All,
 >> On 2014-02-13 18:38 +0000, kuyurix@gmail.com spake thusly:
 >>> From: Marcelo Gutierrez <kuyurix@gmail.com>
 >>> 
 >>> Signed-off-by: Marcelo Gutierrez <kuyurix@gmail.com>

 >> Reviewed-by: "Yann E. MORIN" <yann.morin.1998@free.fr>

 >> Notes:

 >> - The version pointed to by Spencer was 5.00.x, and was a beta
 >> version, and only contained the hardfp variant. The version in
 >> this patch is 5.01.x, is not marked as beta, and contains both
 >> the hardfp and softfp variants.

 >> - With 4.10.x and before, the *_$(TI_GFX_VERSION).bin archive would
 >> contain a huge load of stuff we did not need, and was roughly
 >> 1.7GiB big. But with 5.01.x, this file is now back to a more
 >> acceptable size, roughly 140MiB (although it still contains way
 >> much more than we need).

 > Sounds good. Committed to next, thanks both.

Ehh, but the newclkapi patch no longer applies:

Applying ti-gfx-newclkapi.patch using patch: 
patching file GFX_Linux_KM/services4/system/omap3630/sysutils_linux.c
Hunk #1 FAILED at 153.
Hunk #2 FAILED at 167.
Hunk #3 FAILED at 233.
Hunk #4 FAILED at 246.
Hunk #5 succeeded at 540 (offset 166 lines).
Hunk #6 FAILED at 444.
Hunk #7 FAILED at 490.
6 out of 7 hunks FAILED -- saving rejects to file GFX_Linux_KM/services4/system/omap3630/sysutils_linux.c.rej
Patch failed!  Please fix ti-gfx-newclkapi.patch!
make: *** [/home/peko/source/buildroot/output/build/ti-gfx-5_01_00_01/.stamp_patched] Error 1

Could you please send a patch to fix it?

I also see that 5_01_01_01 is out:

http://tigraphics.blogspot.be/2014/02/1q-2014-linux-graphics-sdk-release.html
Marcelo Gutiérrez(UTN/FRH) Feb. 27, 2014, 8:08 p.m. UTC | #11
Hi Peter, all

Ehh, but the newclkapi patch no longer applies:
>
> Applying ti-gfx-newclkapi.patch using patch:
> patching file GFX_Linux_KM/services4/system/omap3630/sysutils_linux.c
> Hunk #1 FAILED at 153.
> Hunk #2 FAILED at 167.
> Hunk #3 FAILED at 233.
> Hunk #4 FAILED at 246.
> Hunk #5 succeeded at 540 (offset 166 lines).
> Hunk #6 FAILED at 444.
> Hunk #7 FAILED at 490.
> 6 out of 7 hunks FAILED -- saving rejects to file
> GFX_Linux_KM/services4/system/omap3630/sysutils_linux.c.rej
> Patch failed!  Please fix ti-gfx-newclkapi.patch!
> make: *** [/home/peko/source/buildroot/
> output/build/ti-gfx-5_01_00_01/.stamp_patched] Error 1
>
> Could you please send a patch to fix it?
>

I wonder if we need to apply that patch?

All I can say that SGX version works great on BBB(ti-kernel 3.12 using
beaglebone_defconfig)

Regards,
Marcelo




2014-02-26 10:31 GMT-03:00 Peter Korsgaard <peter@korsgaard.com>:

> >>>>> "Peter" == Peter Korsgaard <jacmet@uclibc.org> writes:
>
> >>>>> "Yann" == Yann E MORIN <yann.morin.1998@free.fr> writes:
>  >> Marcelo, All,
>  >> On 2014-02-13 18:38 +0000, kuyurix@gmail.com spake thusly:
>  >>> From: Marcelo Gutierrez <kuyurix@gmail.com>
>  >>>
>  >>> Signed-off-by: Marcelo Gutierrez <kuyurix@gmail.com>
>
>  >> Reviewed-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
>
>  >> Notes:
>
>  >> - The version pointed to by Spencer was 5.00.x, and was a beta
>  >> version, and only contained the hardfp variant. The version in
>  >> this patch is 5.01.x, is not marked as beta, and contains both
>  >> the hardfp and softfp variants.
>
>  >> - With 4.10.x and before, the *_$(TI_GFX_VERSION).bin archive would
>  >> contain a huge load of stuff we did not need, and was roughly
>  >> 1.7GiB big. But with 5.01.x, this file is now back to a more
>  >> acceptable size, roughly 140MiB (although it still contains way
>  >> much more than we need).
>
>  > Sounds good. Committed to next, thanks both.
>
> Ehh, but the newclkapi patch no longer applies:
>
> Applying ti-gfx-newclkapi.patch using patch:
> patching file GFX_Linux_KM/services4/system/omap3630/sysutils_linux.c
> Hunk #1 FAILED at 153.
> Hunk #2 FAILED at 167.
> Hunk #3 FAILED at 233.
> Hunk #4 FAILED at 246.
> Hunk #5 succeeded at 540 (offset 166 lines).
> Hunk #6 FAILED at 444.
> Hunk #7 FAILED at 490.
> 6 out of 7 hunks FAILED -- saving rejects to file
> GFX_Linux_KM/services4/system/omap3630/sysutils_linux.c.rej
> Patch failed!  Please fix ti-gfx-newclkapi.patch!
> make: ***
> [/home/peko/source/buildroot/output/build/ti-gfx-5_01_00_01/.stamp_patched]
> Error 1
>
> Could you please send a patch to fix it?
>
> I also see that 5_01_01_01 is out:
>
>
> http://tigraphics.blogspot.be/2014/02/1q-2014-linux-graphics-sdk-release.html
>
> --
> Bye, Peter Korsgaard
>
diff mbox

Patch

diff --git a/package/ti-gfx/ti-gfx.mk b/package/ti-gfx/ti-gfx.mk
index a94f00d..5ea6b92 100644
--- a/package/ti-gfx/ti-gfx.mk
+++ b/package/ti-gfx/ti-gfx.mk
@@ -4,13 +4,13 @@ 
 #
 ################################################################################
 
-TI_GFX_VERSION = 4_09_00_01
-TI_GFX_SO_VERSION = 1.9.2188537
+TI_GFX_VERSION = 5_01_00_01
+TI_GFX_SO_VERSION = 1.10.2359475
 
 ifeq ($(BR2_ARM_EABIHF),y)
-TI_GFX_SOURCE = Graphics_SDK_setuplinux_$(TI_GFX_VERSION)_hardfp_minimal_demos.bin
+TI_GFX_SOURCE = Graphics_SDK_setuplinux_hardfp_$(TI_GFX_VERSION).bin
 else
-TI_GFX_SOURCE = Graphics_SDK_setuplinux_$(TI_GFX_VERSION)_minimal_demos.bin
+TI_GFX_SOURCE = Graphics_SDK_setuplinux_softfp_$(TI_GFX_VERSION).bin
 endif
 
 TI_GFX_SITE = http://downloads.ti.com/dsps/dsps_public_sw/sdo_sb/targetcontent/gfxsdk/$(TI_GFX_VERSION)/exports/