On 11/6/24 12:51 PM, Denys Dmytriyenko wrote:
On Wed, Nov 06, 2024 at 12:38:22PM -0600, Andrew Davis via 
lists.yoctoproject.org wrote:
On 11/1/24 5:23 PM, Mendez, Judith via lists.yoctoproject.org wrote:
Add hwspinlock recipe to build out of tree module hwspinlock test.

Signed-off-by: Judith Mendez <[email protected]>
---
  .../recipes-core/packagegroups/ti-test.bb     |   1 +
  .../hwspinlocktest/files/LICENSE              |  29 ++
  .../hwspinlocktest/files/Makefile             |  19 ++
  .../files/omap_hwspinlock_test.c              | 251 ++++++++++++++++++

Why are we adding the entire source of this project into meta-arago? Is this
not the upstream: https://github.com/sumananna/omap-hwspinlock-test ?

You should point to that and fetch from there like normal recipes.

FYI, this has been asked and answered already:
https://lists.yoctoproject.org/g/meta-arago/topic/109054399#msg15583


Ah, didn't see any hint to this not being the first version of this series..

Copying the answer here to respond inline,

Apparently we do not want to pull from personal repos

Since when? We do that for lots of recipes.

there is a JIRA req to move
the repo to ti.com, but the target date is for 11.0. Since we
want to add the test asap, the idea is to add to yocto and update
the recipe later when the repo has been moved.

If you really want to use a ti.com based source repo, then do that first.
If setting up the repo clone is not a priority, then this test can't be that
important either. Do this in the right order. Copying the whole source
into the layer is hacky.

Andrew


  .../hwspinlocktest/hwspinlocktest_0.1.bb      |  18 ++
  5 files changed, 318 insertions(+)
  create mode 100644 meta-arago-test/recipes-kernel/hwspinlocktest/files/LICENSE
  create mode 100644 
meta-arago-test/recipes-kernel/hwspinlocktest/files/Makefile
  create mode 100644 
meta-arago-test/recipes-kernel/hwspinlocktest/files/omap_hwspinlock_test.c
  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 85398eaf..98fd0f56 100644
--- a/meta-arago-test/recipes-core/packagegroups/ti-test.bb
+++ b/meta-arago-test/recipes-core/packagegroups/ti-test.bb
@@ -18,6 +18,7 @@ TI_TEST_BASE = "\
      evtest \
      fio \
      git \
+    hwspinlocktest \
      hdparm \
      i2c-tools \
      iozone3 \
diff --git a/meta-arago-test/recipes-kernel/hwspinlocktest/files/LICENSE 
b/meta-arago-test/recipes-kernel/hwspinlocktest/files/LICENSE
new file mode 100644
index 00000000..f5351a25
--- /dev/null
+++ b/meta-arago-test/recipes-kernel/hwspinlocktest/files/LICENSE
@@ -0,0 +1,29 @@
+BSD 3-Clause License
+
+Copyright (c) 2019, Suman Anna
+All rights reserved.
+
+Redistribution and use in source and binary forms, with or without
+modification, are permitted provided that the following conditions are met:
+
+1. Redistributions of source code must retain the above copyright notice, this
+   list of conditions and the following disclaimer.
+
+2. Redistributions in binary form must reproduce the above copyright notice,
+   this list of conditions and the following disclaimer in the documentation
+   and/or other materials provided with the distribution.
+
+3. Neither the name of the copyright holder nor the names of its
+   contributors may be used to endorse or promote products derived from
+   this software without specific prior written permission.
+
+THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
+AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
+IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
+DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE LIABLE
+FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
+DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR
+SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER
+CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY,
+OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
diff --git a/meta-arago-test/recipes-kernel/hwspinlocktest/files/Makefile 
b/meta-arago-test/recipes-kernel/hwspinlocktest/files/Makefile
new file mode 100644
index 00000000..d8fe76dc
--- /dev/null
+++ b/meta-arago-test/recipes-kernel/hwspinlocktest/files/Makefile
@@ -0,0 +1,19 @@
+# SPDX-License-Identifier: (GPL-2.0 OR BSD-3-Clause)
+#
+# TI OMAP HwSpinlock Unit Test
+#
+
+obj-m := omap_hwspinlock_test.o
+
+SRC := $(shell pwd)
+
+all:
+       $(MAKE) -C $(KERNEL_SRC) M=$(SRC)
+
+modules_install:
+       $(MAKE) -C $(KERNEL_SRC) M=$(SRC) modules_install
+
+clean:
+       rm -f *.o *~ core .depend .*.cmd *.ko *.mod.c
+       rm -f Module.markers Module.symvers modules.order
+       rm -rf .tmp_versions Modules.symvers
diff --git 
a/meta-arago-test/recipes-kernel/hwspinlocktest/files/omap_hwspinlock_test.c 
b/meta-arago-test/recipes-kernel/hwspinlocktest/files/omap_hwspinlock_test.c
new file mode 100644
index 00000000..b1801389
--- /dev/null
+++ b/meta-arago-test/recipes-kernel/hwspinlocktest/files/omap_hwspinlock_test.c
@@ -0,0 +1,251 @@
+// SPDX-License-Identifier: (GPL-2.0 OR BSD-3-Clause)
+/*
+ * OMAP hardware spinlock test driver
+ *
+ * Copyright (C) 2014-2021 Texas Instruments Incorporated - https://www.ti.com
+ *     Suman Anna <[email protected]>
+ */
+
+#include <linux/kernel.h>
+#include <linux/module.h>
+#include <linux/init.h>
+#include <linux/err.h>
+#include <linux/sched.h>
+#include <linux/platform_device.h>
+#include <linux/of.h>
+#include <linux/of_device.h>
+
+#include <linux/hwspinlock.h>
+
+/* load-time options */
+static int count = 2;
+module_param(count, int, 0444);
+
+struct hwspinlock_data {
+       const char *compatible;
+       const unsigned int max_locks;
+};
+
+static int hwspin_lock_test(struct hwspinlock *hwlock)
+{
+       int i;
+       int ret;
+
+       pr_err("\nTesting lock %d\n", hwspin_lock_get_id(hwlock));
+       for (i = 0; i < count; i++) {
+               ret = hwspin_trylock(hwlock);
+               if (ret) {
+                       pr_err("%s: Initial lock failed\n", __func__);
+                       return -EFAULT;
+               }
+               pr_err("trylock #1 status[%d] = %d\n", i, ret);
+
+               /* Verify lock actually works - re-acquiring it should fail */
+               ret = hwspin_trylock(hwlock);
+               pr_err("trylock #2 status[%d] = %d\n", i, ret);
+               if (!ret) {
+                       /* Keep locks balanced even in failure cases */
+                       hwspin_unlock(hwlock);
+                       hwspin_unlock(hwlock);
+                       pr_err("%s: Recursive lock succeeded unexpectedly\n",
+                              __func__);
+                       return -EFAULT;
+               }
+
+               /* Verify unlock by re-acquiring the lock after releasing it */
+               hwspin_unlock(hwlock);
+               ret = hwspin_trylock(hwlock);
+               pr_err("trylock after unlock status[%d] = %d\n", i, ret);
+               if (ret) {
+                       pr_err("%s: Unlock failed\n", __func__);
+                       return -EINVAL;
+               }
+
+               hwspin_unlock(hwlock);
+       }
+
+       return 0;
+}
+
+static int hwspin_lock_test_all_locks(unsigned int max_locks)
+{
+       int i;
+       int ret = 0, ret1 = 0;
+       struct hwspinlock *hwlock = NULL;
+
+       pr_err("Testing %d locks\n", max_locks);
+       for (i = 0; i < max_locks; i++) {
+               hwlock = hwspin_lock_request_specific(i);
+               if (!hwlock) {
+                       pr_err("request lock %d failed\n", i);
+                       ret = -EIO;
+                       continue;
+               }
+
+               ret1 = hwspin_lock_test(hwlock);
+               if (ret1) {
+                       pr_err("hwspinlock tests failed on lock %d\n", i);
+                       ret = ret1;
+                       goto free_lock;
+               }
+
+free_lock:
+               ret1 = hwspin_lock_free(hwlock);
+               if (ret1) {
+                       pr_err("hwspin_lock_free failed on lock %d\n", i);
+                       ret = ret1;
+               }
+       }
+
+       return ret;
+}
+
+static const struct of_device_id omap_hwspinlock_test_of_match[] = {
+       { .compatible = "ti,omap-hwspinlock-test",   },
+       { .compatible = "ti,omap4-hwspinlock-test",  },
+       { .compatible = "ti,omap5-hwspinlock-test",  },
+       { .compatible = "ti,dra7-hwspinlock-test",   },
+       { .compatible = "ti,am33xx-hwspinlock-test", },
+       { .compatible = "ti,am43xx-hwspinlock-test", },
+       { .compatible = "ti,am654-hwspinlock-test",  },
+       { /* end */ },
+};
+
+static int hwspin_lock_test_all_phandle_locks(unsigned int max_locks)
+{
+       struct device_node *np = NULL;
+       struct hwspinlock *hwlock = NULL;
+       int ret = 0, ret1 = 0;
+       unsigned int i;
+       int num_locks;
+       int hwlock_id;
+
+       np = of_find_matching_node_and_match(NULL,
+                                            omap_hwspinlock_test_of_match,
+                                            NULL);
+       if (!np) {
+               pr_err("\nNo test node provided\n");
+               return 0;
+       }
+
+       num_locks = of_count_phandle_with_args(np, "hwlocks", "#hwlock-cells");
+       pr_err("Number of phandles = %d max_locks = %d\n",
+              num_locks, max_locks);
+
+       for (i = 0; i < num_locks; i++) {
+               hwlock_id = of_hwspin_lock_get_id(np, i);
+               if (hwlock_id < 0) {
+                       pr_err("unable to get hwlock_id : %d\n", hwlock_id);
+                       ret = -EINVAL;
+                       continue;
+               };
+
+               hwlock = hwspin_lock_request_specific(hwlock_id);
+               if (!hwlock) {
+                       pr_err("unable to get hwlock\n");
+                       ret = -EINVAL;
+                       continue;
+               }
+
+               ret1 = hwspin_lock_test(hwlock);
+               if (ret1) {
+                       pr_err("hwspinlock test failed on DT lock %d, ret = 
%d\n",
+                              hwspin_lock_get_id(hwlock), ret1);
+                       ret = ret1;
+               }
+
+               ret1 = hwspin_lock_free(hwlock);
+               if (ret1) {
+                       pr_err("hwspin_lock_free failed on lock %d\n",
+                              hwspin_lock_get_id(hwlock));
+                       ret = ret1;
+               }
+       }
+
+       return ret;
+}
+
+static
+unsigned int omap_hwspinlock_get_locks(const struct hwspinlock_data *data)
+{
+       unsigned int locks = 0;
+
+       while (data->compatible) {
+               if (of_machine_is_compatible(data->compatible)) {
+                       locks = data->max_locks;
+                       break;
+               }
+               data++;
+       }
+
+       return locks;
+}
+
+static const struct of_device_id omap_hwspinlock_of_match[] = {
+       { .compatible = "ti,omap4-hwspinlock", },
+       { .compatible = "ti,am654-hwspinlock", },
+       { .compatible = "ti,am64-hwspinlock",  },
+       { /* end */ },
+};
+
+static const struct hwspinlock_data soc_data[] = {
+       { "ti,omap4", 32, },
+       { "ti,omap5", 32, },
+       { "ti,dra7",  256, },
+       { "ti,am33xx",        128, },
+       { "ti,am43",  128, },
+       { "ti,am654", 256, },
+       { "ti,j721e", 256, },
+       { "ti,j7200", 256, },
+       { "ti,am642", 256, },
+       { "ti,j721s2",        256, },
+       { /* sentinel */ },
+};
+
+static int omap_hwspinlock_test_init(void)
+{
+       struct device_node *np;
+       unsigned int max_locks;
+       int ret;
+
+       pr_err("\n** HwSpinLock Unit Test Module initiated **\n");
+
+       max_locks = omap_hwspinlock_get_locks(soc_data);
+       if (!max_locks) {
+               pr_err("\nNot a compatible platform\n");
+               return -ENODEV;
+       }
+
+       np = of_find_matching_node_and_match(NULL, omap_hwspinlock_of_match,
+                                            NULL);
+       if (!np || !of_device_is_available(np)) {
+               pr_err("\nNo HwSpinlock node provided/enabled\n");
+               return -ENODEV;
+       }
+
+       pr_err("\n***** Begin - Test All Locks ****\n");
+       ret = hwspin_lock_test_all_locks(max_locks);
+       if (ret)
+               pr_err("hwspin_lock_test_all_locks failed, ret = %d\n", ret);
+       pr_err("\n***** End - Test All Locks ****\n");
+
+       pr_err("\n***** Begin - Test All pHandle Locks ****\n");
+       ret = hwspin_lock_test_all_phandle_locks(max_locks);
+       if (ret)
+               pr_err("hwspin_lock_test_all_locks failed, ret = %d\n", ret);
+       pr_err("\n***** End - Test All pHandle Locks ****\n");
+
+       return 0;
+}
+
+static void omap_hwspinlock_test_exit(void)
+{
+       pr_err("\n** HwSpinLock Unit Test Module finished **\n");
+}
+
+module_init(omap_hwspinlock_test_init);
+module_exit(omap_hwspinlock_test_exit);
+
+MODULE_LICENSE("Dual BSD/GPL");
+MODULE_DESCRIPTION("Hardware spinlock Test driver for TI SoCs");
+MODULE_AUTHOR("Suman Anna <[email protected]>");
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..2edd93f6
--- /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://LICENSE;md5=bfa02c83df161e37647ee23a2c7eacd4"
+
+inherit module
+
+SRC_URI = "file://Makefile \
+           file://omap_hwspinlock_test.c \
+           file://LICENSE \
+          "
+
+S = "${WORKDIR}"
+
+# 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"


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#15598): 
https://lists.yoctoproject.org/g/meta-arago/message/15598
Mute This Topic: https://lists.yoctoproject.org/mt/109343942/21656
Group Owner: [email protected]
Unsubscribe: https://lists.yoctoproject.org/g/meta-arago/unsub 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to