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

On 09/03/26 19:16, Ryan Eatmon wrote:
> 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.
> 

I missed the fact that master branch can support 6.12 kernel build.
Thank you for your feedback. I will post a v2 series with the required
changes.

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