diff mbox series

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

Message ID 20260309130109.52886-2-a-garg7@ti.com
State New
Headers show
Series Update PCIe Endpoint Testing Application | expand

Commit Message

Aksh Garg March 9, 2026, 1:01 p.m. UTC 
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 <a-garg7@ti.com>
---
 .../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

Comments

Ryan Eatmon March 9, 2026, 1:46 p.m. UTC | #1 
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 <a-garg7@ti.com>
> ---
>   .../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}
> -}
diff mbox series

Patch

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 \
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 \
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
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}
-}