Re: [meta-ti][dunfell][PATCH v2] ti-graphics: gpu enable and move all platforms to ddk 1.15

[Denys Dmytriyenko <denis@denix.org>]

On Sat, Apr 30, 2022 at 02:02:27PM -0400, Denys Dmytriyenko wrote:
> Darren,
> 
> DDK 1.15 (kernel driver) build fails for am62x using GCC 11:
> 
> | .../build/tmp-default-glibc/work/am62xx_evm-oe-linux/ti-img-rogue-driver/1.15.6133109-r0a/git/binary_am62_linux_wayland_release/target_aarch64/kbuild/services/server/devices/rogue/rgxinit.c:1345:36: error: implicit conversion from 'enum <anonymous>' to 'IMG_BOOL' {aka 'enum tag_img_bool'} [-Werror=enum-conversion]
> |  1345 |                         bEnableAPM = false;
> |       |                                    ^
> 
> Can you please take a look? Thanks.

Never mind, I have a fix.


> 
> -- 
> Denys
> 
> 
> On Wed, Mar 23, 2022 at 02:37:07PM -0500, Darren Etheridge wrote:
> > Enable the GPU for am62xx and j721s2 and use IMG DDK 1.15
> > 
> > Migrate Imagination DDK 1.13 to DDK 1.15 for J721e
> > 
> > Signed-off-by: Darren Etheridge <detheridge@ti.com>
> > ---
> > 
> > No changes in the content of the patch, however I regenerated it with the 
> > similarity threshold reduced as requested by Denys.
> > 
> >  conf/machine/am62xx-evm.conf                  |  2 ++
> >  conf/machine/j721s2-evm.conf                  |  2 ++
> >  ...bb => ti-img-rogue-driver_1.15.6133109.bb} | 20 +++++++++++--------
> >  ...bb => ti-img-rogue-umlibs_1.15.6133109.bb} | 13 ++++++------
> >  4 files changed, 23 insertions(+), 14 deletions(-)
> >  rename recipes-bsp/powervr-drivers/{ti-img-rogue-driver_1.13.5776728.bb => ti-img-rogue-driver_1.15.6133109.bb} (48%)
> >  rename recipes-graphics/libgles/{ti-img-rogue-umlibs_1.13.5776728.bb => ti-img-rogue-umlibs_1.15.6133109.bb} (84%)
> > 
> > diff --git a/conf/machine/am62xx-evm.conf b/conf/machine/am62xx-evm.conf
> > index 600abb58..086e40e4 100644
> > --- a/conf/machine/am62xx-evm.conf
> > +++ b/conf/machine/am62xx-evm.conf
> > @@ -4,6 +4,8 @@
> >  
> >  require conf/machine/include/am62xx.inc
> >  
> > +MACHINE_FEATURES += "gpu"
> > +
> >  KERNEL_DEVICETREE = " \
> >      ti/k3-am625-sk.dtb \
> >      ti/k3-am625-sk-lpmdemo.dtb \
> > diff --git a/conf/machine/j721s2-evm.conf b/conf/machine/j721s2-evm.conf
> > index 82fd33da..e5ed0b73 100644
> > --- a/conf/machine/j721s2-evm.conf
> > +++ b/conf/machine/j721s2-evm.conf
> > @@ -4,6 +4,8 @@
> >  
> >  require conf/machine/include/j7.inc
> >  
> > +MACHINE_FEATURES += "gpu"
> > +
> >  SERIAL_CONSOLES = "115200;ttyS10"
> >  SERIAL_CONSOLES_CHECK = "${SERIAL_CONSOLES}"
> >  TFA_K3_USART = "0x8"
> > diff --git a/recipes-bsp/powervr-drivers/ti-img-rogue-driver_1.13.5776728.bb b/recipes-bsp/powervr-drivers/ti-img-rogue-driver_1.15.6133109.bb
> > similarity index 48%
> > rename from recipes-bsp/powervr-drivers/ti-img-rogue-driver_1.13.5776728.bb
> > rename to recipes-bsp/powervr-drivers/ti-img-rogue-driver_1.15.6133109.bb
> > index a05de0f2..fbff6c51 100644
> > --- a/recipes-bsp/powervr-drivers/ti-img-rogue-driver_1.13.5776728.bb
> > +++ b/recipes-bsp/powervr-drivers/ti-img-rogue-driver_1.15.6133109.bb
> > @@ -7,17 +7,17 @@ inherit module features_check
> >  
> >  REQUIRED_MACHINE_FEATURES = "gpu"
> >  
> > -MACHINE_KERNEL_PR_append = "b"
> > +MACHINE_KERNEL_PR_append = "a"
> >  PR = "${MACHINE_KERNEL_PR}"
> >  
> >  PACKAGE_ARCH = "${MACHINE_ARCH}"
> > -COMPATIBLE_MACHINE = "j7"
> > +COMPATIBLE_MACHINE = "j7-evm|j721s2-evm|am62xx"
> >  
> >  DEPENDS = "virtual/kernel"
> >  
> >  PROVIDES = "virtual/gpudriver"
> >  
> > -BRANCH = "1.13-5776728/linux-k5.10"
> > +BRANCH = "linuxws/dunfell/k5.10/${PV}"
> >  
> >  SRC_URI = " \
> >      git://git.ti.com/graphics/ti-img-rogue-driver.git;branch=${BRANCH} \
> > @@ -26,15 +26,19 @@ SRC_URI = " \
> >  
> >  S = "${WORKDIR}/git"
> >  
> > -SRCREV = "35a25875ae8738f82c7cabc6b077ef992b0cca84"
> > +SRCREV = "ee0674adccac16f5b2f7cb8d5d05948706080cb5"
> >  
> > -PVR_SOC = "j721e_linux"
> > -PVR_BVNC = "22.104.208.318"
> > +TARGET_PRODUCT_j7-evm = "j721e_linux"
> > +TARGET_PRODUCT_j721s2-evm = "j721s2_linux"
> > +TARGET_PRODUCT_am62xx = "am62_linux"
> > +TARGET_BVNC_j7-evm = "22.104.208.318"
> > +TARGET_BVNC_j721s2-evm = "36.53.104.796"
> > +TARGET_BVNC_am62xx = "33.15.11.3"
> >  PVR_BUILD = "release"
> >  PVR_WS = "wayland"
> >  
> > -EXTRA_OEMAKE += 'KERNELDIR="${STAGING_KERNEL_DIR}" RGX_BVNC=${PVR_BVNC} BUILD=${PVR_BUILD} PVR_BUILD_DIR=${PVR_SOC} WINDOW_SYSTEM=${PVR_WS}'
> > +EXTRA_OEMAKE += 'KERNELDIR="${STAGING_KERNEL_DIR}" RGX_BVNC=${TARGET_BVNC} BUILD=${PVR_BUILD} PVR_BUILD_DIR=${TARGET_PRODUCT} WINDOW_SYSTEM=${PVR_WS}'
> >  
> >  do_install() {
> > -    make -C ${STAGING_KERNEL_DIR} M=${B}/binary_${PVR_SOC}_${PVR_WS}_${PVR_BUILD}/target_aarch64/kbuild INSTALL_MOD_PATH=${D}${root_prefix} PREFIX=${STAGING_DIR_HOST} modules_install
> > +    make -C ${STAGING_KERNEL_DIR} M=${B}/binary_${TARGET_PRODUCT}_${PVR_WS}_${PVR_BUILD}/target_aarch64/kbuild INSTALL_MOD_PATH=${D}${root_prefix} PREFIX=${STAGING_DIR_HOST} modules_install
> >  }
> > diff --git a/recipes-graphics/libgles/ti-img-rogue-umlibs_1.13.5776728.bb b/recipes-graphics/libgles/ti-img-rogue-umlibs_1.15.6133109.bb
> > similarity index 84%
> > rename from recipes-graphics/libgles/ti-img-rogue-umlibs_1.13.5776728.bb
> > rename to recipes-graphics/libgles/ti-img-rogue-umlibs_1.15.6133109.bb
> > index 5194015e..e00dce99 100644
> > --- a/recipes-graphics/libgles/ti-img-rogue-umlibs_1.13.5776728.bb
> > +++ b/recipes-graphics/libgles/ti-img-rogue-umlibs_1.15.6133109.bb
> > @@ -8,17 +8,18 @@ inherit features_check
> >  REQUIRED_MACHINE_FEATURES = "gpu"
> >  
> >  PACKAGE_ARCH = "${MACHINE_ARCH}"
> > -COMPATIBLE_MACHINE = "j7"
> > +COMPATIBLE_MACHINE = "j7-evm|j721s2-evm|am62xx"
> >  
> > -PR = "r3"
> > +PR = "r1"
> >  
> >  BRANCH = "linuxws/dunfell/k5.10/${PV}"
> >  
> >  SRC_URI = "git://git.ti.com/graphics/ti-img-rogue-umlibs.git;branch=${BRANCH}"
> > +SRCREV = "47c3ffc44f2881397d45c8a2b5dfa7d6c58b79fb"
> >  
> > -SRCREV = "5ef38353c38d921a65ea6df75c7b6ce710b66034"
> > -
> > -PVR_SOC ?= "j721e_linux"
> > +TARGET_PRODUCT_j7-evm = "j721e_linux"
> > +TARGET_PRODUCT_j721s2-evm = "j721s2_linux"
> > +TARGET_PRODUCT_am62xx = "am62_linux"
> >  PVR_BUILD ?= "release"
> >  PVR_WS = "wayland"
> >  
> > @@ -50,7 +51,7 @@ RRECOMMENDS_${PN} += "ti-img-rogue-driver"
> >  S = "${WORKDIR}/git"
> >  
> >  do_install () {
> > -    oe_runmake install DESTDIR=${D} TARGET_PRODUCT=${PVR_SOC} BUILD=${PVR_BUILD} WINDOW_SYSTEM=${PVR_WS}
> > +    oe_runmake install DESTDIR=${D} TARGET_PRODUCT=${TARGET_PRODUCT} BUILD=${PVR_BUILD} WINDOW_SYSTEM=${PVR_WS}
> >      chown -R root:root ${D}
> >  }
> >  
> > -- 
> > 2.17.1
> > 

> 
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#14637): https://lists.yoctoproject.org/g/meta-ti/message/14637
> Mute This Topic: https://lists.yoctoproject.org/mt/89983769/3617104
> Group Owner: meta-ti+owner@lists.yoctoproject.org
> Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub [denis@denix.org]
> -=-=-=-=-=-=-=-=-=-=-=-
> 


-- 
Regards,
Denys Dmytriyenko <denis@denix.org>
PGP: 0x420902729A92C964 - https://denix.org/0x420902729A92C964
Fingerprint: 25FC E4A5 8A72 2F69 1186  6D76 4209 0272 9A92 C964