[meta-arago,master/kirkstone/scarthgap] Add hwspinlocktest

Add hwspinlock recipe to build out of tree module hwspinlock test.

Signed-off-by: Judith Mendez <jm@ti.com>
---
 .../recipes-core/packagegroups/ti-test.bb     |  1 +
 .../0001-Add-support-for-AM62-SoCs.patch      | 31 +++++++++++++
 ...Fix-Makefile-for-to-build-with-yocto.patch | 46 +++++++++++++++++++
 .../hwspinlocktest/hwspinlocktest_0.1.bb      | 18 ++++++++
 4 files changed, 96 insertions(+)
 create mode 100644 meta-arago-test/recipes-kernel/hwspinlocktest/files/0001-Add-support-for-AM62-SoCs.patch
 create mode 100644 meta-arago-test/recipes-kernel/hwspinlocktest/files/0001-Fix-Makefile-for-to-build-with-yocto.patch
 create mode 100644 meta-arago-test/recipes-kernel/hwspinlocktest/hwspinlocktest_0.1.bb

Hi all,

Forgot to put v2 here, will resend the patch..

~ Judith

On 11/18/24 5:36 PM, Judith Mendez wrote:
> Add hwspinlock recipe to build out of tree module hwspinlock test.
> 
> Signed-off-by: Judith Mendez <jm@ti.com>
> ---
>   .../recipes-core/packagegroups/ti-test.bb     |  1 +
>   .../0001-Add-support-for-AM62-SoCs.patch      | 31 +++++++++++++
>   ...Fix-Makefile-for-to-build-with-yocto.patch | 46 +++++++++++++++++++
>   .../hwspinlocktest/hwspinlocktest_0.1.bb      | 18 ++++++++
>   4 files changed, 96 insertions(+)
>   create mode 100644 meta-arago-test/recipes-kernel/hwspinlocktest/files/0001-Add-support-for-AM62-SoCs.patch
>   create mode 100644 meta-arago-test/recipes-kernel/hwspinlocktest/files/0001-Fix-Makefile-for-to-build-with-yocto.patch
>   create mode 100644 meta-arago-test/recipes-kernel/hwspinlocktest/hwspinlocktest_0.1.bb
> 
> diff --git a/meta-arago-test/recipes-core/packagegroups/ti-test.bb b/meta-arago-test/recipes-core/packagegroups/ti-test.bb
> index 695bed5c..91ebec44 100644
> --- a/meta-arago-test/recipes-core/packagegroups/ti-test.bb
> +++ b/meta-arago-test/recipes-core/packagegroups/ti-test.bb
> @@ -17,6 +17,7 @@ TI_TEST_BASE = "\
>       evtest \
>       fio \
>       git \
> +    hwspinlocktest \
>       hdparm \
>       i2c-tools \
>       iozone3 \
> diff --git a/meta-arago-test/recipes-kernel/hwspinlocktest/files/0001-Add-support-for-AM62-SoCs.patch b/meta-arago-test/recipes-kernel/hwspinlocktest/files/0001-Add-support-for-AM62-SoCs.patch
> new file mode 100644
> index 00000000..b2b99814
> --- /dev/null
> +++ b/meta-arago-test/recipes-kernel/hwspinlocktest/files/0001-Add-support-for-AM62-SoCs.patch
> @@ -0,0 +1,31 @@
> +From 2d8d169898fb6615dd5100c4ac4e307c54678153 Mon Sep 17 00:00:00 2001
> +From: Judith Mendez <jm@ti.com>
> +Date: Mon, 18 Nov 2024 14:57:45 -0600
> +Subject: [PATCH] Add support for AM62* SoCs
> +
> +The K3 AM62* SoCs have a hwspinlock IP in MAIN domain.
> +Extend the test to add the desired SoC compatible so that
> +the test can be exercised on these new SoC families.
> +
> +Signed-off-by: Judith Mendez <jm@ti.com>
> +---
> + omap_hwspinlock_test.c | 3 +++
> + 1 file changed, 3 insertions(+)
> +
> +diff --git a/omap_hwspinlock_test.c b/omap_hwspinlock_test.c
> +index b180138..46fcc08 100644
> +--- a/omap_hwspinlock_test.c
> ++++ b/omap_hwspinlock_test.c
> +@@ -199,6 +199,9 @@ static const struct hwspinlock_data soc_data[] = {
> + 	{ "ti,j7200",	256, },
> + 	{ "ti,am642",	256, },
> + 	{ "ti,j721s2",	256, },
> ++	{ "ti,am625",	256, },
> ++	{ "ti,am62a7",	256, },
> ++	{ "ti,am62p5",	256, },
> + 	{ /* sentinel */ },
> + };
> +
> +--
> +2.47.0
> +
> diff --git a/meta-arago-test/recipes-kernel/hwspinlocktest/files/0001-Fix-Makefile-for-to-build-with-yocto.patch b/meta-arago-test/recipes-kernel/hwspinlocktest/files/0001-Fix-Makefile-for-to-build-with-yocto.patch
> new file mode 100644
> index 00000000..ac362b2c
> --- /dev/null
> +++ b/meta-arago-test/recipes-kernel/hwspinlocktest/files/0001-Fix-Makefile-for-to-build-with-yocto.patch
> @@ -0,0 +1,46 @@
> +From 3e1a3e3b9455f4cd40f7cc86a8eaff80206f77ce Mon Sep 17 00:00:00 2001
> +From: Judith Mendez <jm@ti.com>
> +Date: Mon, 18 Nov 2024 10:26:55 -0600
> +Subject: [PATCH] Fix Makefile for to build with yocto
> +
> +Fix Makefile that builds hwspinlocktest out-of-tree-module
> +for yocto recipie to be able to build the module.
> +
> +Signed-off-by: Judith Mendez <jm@ti.com>
> +---
> + Makefile | 18 ++++++++++--------
> + 1 file changed, 10 insertions(+), 8 deletions(-)
> +
> +diff --git a/Makefile b/Makefile
> +index 23ee629..d8fe76d 100644
> +--- a/Makefile
> ++++ b/Makefile
> +@@ -3,15 +3,17 @@
> + # TI OMAP HwSpinlock Unit Test
> + #
> +
> +-obj-m = omap_hwspinlock_test.o
> ++obj-m := omap_hwspinlock_test.o
> ++
> ++SRC := $(shell pwd)
> +
> + all:
> +-ifeq ($(KERNELDIR),)
> +-	@echo "Error: KERNELDIR not set, exiting..."
> +-	@echo "Eg: make ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf- KERNELDIR=<linux-kernel rootdir>"
> +-	@exit 1
> +-endif
> +-	make ${MAKE_OPTS} -C $(KERNELDIR) M=$(PWD) modules
> ++	$(MAKE) -C $(KERNEL_SRC) M=$(SRC)
> ++
> ++modules_install:
> ++	$(MAKE) -C $(KERNEL_SRC) M=$(SRC) modules_install
> +
> + clean:
> +-	$(RM) -r *.o *.ko *.mod* *.dwo .*.dwo .*.cmd *.symvers modules.order
> ++	rm -f *.o *~ core .depend .*.cmd *.ko *.mod.c
> ++	rm -f Module.markers Module.symvers modules.order
> ++	rm -rf .tmp_versions Modules.symvers
> +--
> +2.47.0
> +
> diff --git a/meta-arago-test/recipes-kernel/hwspinlocktest/hwspinlocktest_0.1.bb b/meta-arago-test/recipes-kernel/hwspinlocktest/hwspinlocktest_0.1.bb
> new file mode 100644
> index 00000000..6de7bbb5
> --- /dev/null
> +++ b/meta-arago-test/recipes-kernel/hwspinlocktest/hwspinlocktest_0.1.bb
> @@ -0,0 +1,18 @@
> +SUMMARY = "Build hwspinlock test as an external Linux kernel module"
> +DESCRIPTION = "${SUMMARY}"
> +LICENSE = "GPL-2.0-only | BSD-3-Clause"
> +LIC_FILES_CHKSUM = "file://${WORKDIR}/git/LICENSE;md5=bfa02c83df161e37647ee23a2c7eacd4"
> +
> +inherit module
> +
> +SRC_URI = "git://github.com/sumananna/omap-hwspinlock-test;protocol=https;branch=master \
> +		   file://0001-Fix-Makefile-for-to-build-with-yocto.patch \
> +		   file://0001-Add-support-for-AM62-SoCs.patch "
> +SRCREV = "20f15e79a3a0197e06328c252d23aec225de21ed"
> +
> +S = "${WORKDIR}/git"
> +
> +# The inherit of module.bbclass will automatically name module packages with
> +# "kernel-module-" prefix as required by the oe-core build environment.
> +
> +RPROVIDES:${PN} += "kernel-module-hwspinlocktest"