Re: [meta-arago][master][PATCH 1/2] ti-test: Remove the pcitest application

Ryan Eatmon <[email protected]> Mon, 9 Mar 2026 08:46:25 -0500
Newsgroups org.yoctoproject.lists.meta-arago
Message-ID <[email protected]>
We can't just remove this globally.  We need to base this on the bsp 
that is being built.

If you build the 6.12 kernel with master then you will not get a tool 
that works and will try and include a tool that will not work.

We will need to support this until the 6.12 kernel rolls out of support 
in master.

More comments inline

On 3/9/2026 8:01 AM, Aksh Garg wrote:
> The 'pcitest' application, used to test the PCIe endpoint functionality,
> is obsolete on the 6.18 kernel. Hence, remove the pcitest application
> from packagegroups.
> 
> Signed-off-by: Aksh Garg <[email protected]>
> ---
>   .../packagegroup-arago-bootstrap.bb            |  1 -
>   .../recipes-core/packagegroups/ti-test.bb      |  1 -
>   .../recipes-kernel/pcitest/pcitest_5.10.bb     | 18 ------------------
>   3 files changed, 20 deletions(-)
>   delete mode 100644 meta-arago-test/recipes-kernel/pcitest/pcitest_5.10.bb
> 
> diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-bootstrap.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-bootstrap.bb
> index e7a28eb80575..35c54ce7b176 100644
> --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-bootstrap.bb
> +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-bootstrap.bb
> @@ -11,7 +11,6 @@ UTILS = " \
>   	 evtest \
>   	 memtester \
>   	 nbench-byte \
> -	 pcitest \
>   	 serialcheck \
>   	 yavta \
>   	 hdparm \

UTILS:remove:bsp-ti-6_18 = "pcitest"


> diff --git a/meta-arago-test/recipes-core/packagegroups/ti-test.bb b/meta-arago-test/recipes-core/packagegroups/ti-test.bb
> index 65cf7f1ea614..25d8cce5d615 100644
> --- a/meta-arago-test/recipes-core/packagegroups/ti-test.bb
> +++ b/meta-arago-test/recipes-core/packagegroups/ti-test.bb
> @@ -37,7 +37,6 @@ TI_TEST_BASE = "\
>       nbench-byte \
>       openntpd \
>       ${@"optee-test" if d.getVar('OPTEEMACHINE') else ""} \
> -    pcitest \
>       pciutils \
>       perf \
>       powertop \

TI_TEST_BASE:remove:bsp-ti-6_18 = "pcitest"


> diff --git a/meta-arago-test/recipes-kernel/pcitest/pcitest_5.10.bb b/meta-arago-test/recipes-kernel/pcitest/pcitest_5.10.bb
> deleted file mode 100644

You cannot remove the recipe since it is still needed for 6.12 support.


> index 9ab0dbedf4ba..000000000000
> --- a/meta-arago-test/recipes-kernel/pcitest/pcitest_5.10.bb
> +++ /dev/null
> @@ -1,18 +0,0 @@
> -SUMMARY = "Linux Kernel PCI test"
> -LICENSE = "GPL-2.0-only"
> -LIC_FILES_CHKSUM = "file://COPYING;md5=6bc538ed5bd9a7fc9398086aedcd7e46"
> -
> -BRANCH = "ti-linux-5.10.y"
> -SRCREV = "73aa709ca10103b61fba3a07471dbb4dcb56db45"
> -SRC_URI = "git://git.ti.com/git/ti-linux-kernel/ti-linux-kernel.git;protocol=https;branch=${BRANCH}"
> -
> -do_compile () {
> -	cd ${S}/tools/pci
> -	${CC} ${CFLAGS} ${LDFLAGS} -o pcitest pcitest.c
> -}
> -
> -do_install () {
> -	install -d ${D}${bindir}
> -	install -m 0755 ${S}/tools/pci/pcitest ${D}${bindir}
> -	install -m 0755 ${S}/tools/pci/pcitest.sh ${D}${bindir}
> -}

-- 
Ryan Eatmon                [email protected]
-----------------------------------------
Texas Instruments, Inc.  -  LCPD  -  MGTS