diff mbox series

[meta-ti,scarthgap/master,2/3] recipes-graphics : port gc320 libs to work on kernel 6.12

Message ID 20250212201830.2090247-3-a-christidis@ti.com
State Superseded
Delegated to: Ryan Eatmon
Headers show
Series Enabling the gc320 driver for am57xx-evm | expand

Commit Message

Antonios Christidis Feb. 12, 2025, 8:18 p.m. UTC
From: Antonios Christidis <a0507488@uda0867391-2.dhcp.ti.com>

The user mode libraries needed recompiling with the scarthgap revision of gcc to
remove all the warning about 32bit API usage during the linking stage, New SRCREV
and branch includes this changes. Versions of the libraries remain the same.

Also, create custom package split logic. Old logic would place the symbolic link
to the new recompiled packages, in a package that would not be included in the
SDK. So, new logic makes sure the symbolic link is included in a package that
get included in the rootfs.

Signed-Off-By: Antonios Christidis <a-christidis@ti.com>
---
 .../recipes-graphics/libgal/ti-gc320-libs_5.0.11.p7.bb | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

Comments

Denys Dmytriyenko Feb. 15, 2025, 12:14 a.m. UTC | #1
On Wed, Feb 12, 2025 at 02:18:29PM -0600, Antonios Christidis via lists.yoctoproject.org wrote:
> From: Antonios Christidis <a0507488@uda0867391-2.dhcp.ti.com>

You might want to check your configuration to fix the email address.


> The user mode libraries needed recompiling with the scarthgap revision of gcc to
> remove all the warning about 32bit API usage during the linking stage, New SRCREV
> and branch includes this changes. Versions of the libraries remain the same.
> 
> Also, create custom package split logic. Old logic would place the symbolic link
> to the new recompiled packages, in a package that would not be included in the
> SDK. So, new logic makes sure the symbolic link is included in a package that
> get included in the rootfs.
> 
> Signed-Off-By: Antonios Christidis <a-christidis@ti.com>
> ---
>  .../recipes-graphics/libgal/ti-gc320-libs_5.0.11.p7.bb | 10 +++++++---
>  1 file changed, 7 insertions(+), 3 deletions(-)
> 
> diff --git a/meta-ti-bsp/recipes-graphics/libgal/ti-gc320-libs_5.0.11.p7.bb b/meta-ti-bsp/recipes-graphics/libgal/ti-gc320-libs_5.0.11.p7.bb
> index 807f421c..02b44355 100644
> --- a/meta-ti-bsp/recipes-graphics/libgal/ti-gc320-libs_5.0.11.p7.bb
> +++ b/meta-ti-bsp/recipes-graphics/libgal/ti-gc320-libs_5.0.11.p7.bb
> @@ -11,10 +11,10 @@ COMPATIBLE_MACHINE = "omap-a15"
>  
>  CLEANBROKEN = "1"
>  
> -BRANCH = "ti-${PV}"
> +BRANCH = "ti-${PV}-scarthgap"
>  
>  SRC_URI = "git://git.ti.com/git/graphics/ti-gc320-libs.git;protocol=https;branch=${BRANCH}"
> -SRCREV = "c0afab259de59909cfe74c01f3f7fbaa147f94b5"
> +SRCREV = "85c175e8425c33dda6e272aeb45afe0f69a79096"
>  
>  RRECOMMENDS:${PN} = "ti-gc320-driver"
>  
> @@ -41,6 +41,10 @@ EXTRA_OEMAKE += "DESTDIR=${D} TARGET_PRODUCT=${TARGET_PRODUCT} LIBDIR=${libdir}"
>  do_install() {
>      oe_runmake install
>      chown -R root:root ${D}
> +    mkdir lib
>  }
>  
> -INSANE_SKIP:${PN} += "ldflags"
> +PACKAGES = "${PN}"
> +FILES:${PN} = "${libdir} /usr/include"

This is incorrect, please don't do that. Header files and .so symlinks that 
you mentioned in the commit message should go into the -dev package.


> +INSANE_SKIP:${PN} += "ldflags already-stripped dev-so"
> -- 
> 2.34.1
diff mbox series

Patch

diff --git a/meta-ti-bsp/recipes-graphics/libgal/ti-gc320-libs_5.0.11.p7.bb b/meta-ti-bsp/recipes-graphics/libgal/ti-gc320-libs_5.0.11.p7.bb
index 807f421c..02b44355 100644
--- a/meta-ti-bsp/recipes-graphics/libgal/ti-gc320-libs_5.0.11.p7.bb
+++ b/meta-ti-bsp/recipes-graphics/libgal/ti-gc320-libs_5.0.11.p7.bb
@@ -11,10 +11,10 @@  COMPATIBLE_MACHINE = "omap-a15"
 
 CLEANBROKEN = "1"
 
-BRANCH = "ti-${PV}"
+BRANCH = "ti-${PV}-scarthgap"
 
 SRC_URI = "git://git.ti.com/git/graphics/ti-gc320-libs.git;protocol=https;branch=${BRANCH}"
-SRCREV = "c0afab259de59909cfe74c01f3f7fbaa147f94b5"
+SRCREV = "85c175e8425c33dda6e272aeb45afe0f69a79096"
 
 RRECOMMENDS:${PN} = "ti-gc320-driver"
 
@@ -41,6 +41,10 @@  EXTRA_OEMAKE += "DESTDIR=${D} TARGET_PRODUCT=${TARGET_PRODUCT} LIBDIR=${libdir}"
 do_install() {
     oe_runmake install
     chown -R root:root ${D}
+    mkdir lib
 }
 
-INSANE_SKIP:${PN} += "ldflags"
+PACKAGES = "${PN}"
+FILES:${PN} = "${libdir} /usr/include"
+
+INSANE_SKIP:${PN} += "ldflags already-stripped dev-so"