All of lore.kernel.org
 help / color / mirror / Atom feed
* [master/scarthgap][PATCH] hwspinlocktest: Update hwspinlock recipe remote repo url
@ 2025-09-11 21:06 Judith Mendez
  2025-09-11 21:10 ` PRC Automation
                   ` (2 more replies)
  0 siblings, 3 replies; 5+ messages in thread
From: Judith Mendez @ 2025-09-11 21:06 UTC (permalink / raw)
  To: Judith Mendez, Ryan Eatmon
  Cc: Chirag Shilwant, Denys Dmytriyenko, Praneeth Bajjuri, meta-arago

The hwspinlock module is still not moved to git.ti.com. So a new
fork has been created to add support for am62sip platform.

Move now to new forked repo and remove patches that have already been
integrated to the new forked repo master branch.

Signed-off-by: Judith Mendez <jm@ti.com>
---
 ...test-Remove-hwspin_lock_get_id-calls.patch | 69 -------------------
 .../0002-Add-support-for-AM62-SoCs.patch      | 31 ---------
 .../hwspinlocktest/hwspinlocktest_0.1.bb      |  6 +-
 3 files changed, 2 insertions(+), 104 deletions(-)
 delete mode 100644 meta-arago-test/recipes-kernel/hwspinlocktest/files/0001-omap_hwspinlock_test-Remove-hwspin_lock_get_id-calls.patch
 delete mode 100644 meta-arago-test/recipes-kernel/hwspinlocktest/files/0002-Add-support-for-AM62-SoCs.patch

diff --git a/meta-arago-test/recipes-kernel/hwspinlocktest/files/0001-omap_hwspinlock_test-Remove-hwspin_lock_get_id-calls.patch b/meta-arago-test/recipes-kernel/hwspinlocktest/files/0001-omap_hwspinlock_test-Remove-hwspin_lock_get_id-calls.patch
deleted file mode 100644
index d4448397..00000000
--- a/meta-arago-test/recipes-kernel/hwspinlocktest/files/0001-omap_hwspinlock_test-Remove-hwspin_lock_get_id-calls.patch
+++ /dev/null
@@ -1,69 +0,0 @@
-From fb986ed355cd488da64da9eb94d0f260bb329d5b Mon Sep 17 00:00:00 2001
-From: Ryan Eatmon <reatmon@ti.com>
-Date: Fri, 11 Apr 2025 13:49:11 -0500
-Subject: [PATCH] omap_hwspinlock_test: Remove hwspin_lock_get_id() calls
-
-The hwspin_lock_get_id() function was removed in the most recent
-kernel [1].  We can safely stop using this function even for older kernels,
-so go ahead and take these changes.
-
-[1] https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/?id=fec04edb74126f21ac628c7be763c97deb49f69d
-
-Upstream-Status: Inactive-Upstream [private repo]
-
-Signed-off-by: Ryan Eatmon <reatmon@ti.com>
----
- omap_hwspinlock_test.c | 12 ++++++------
- 1 file changed, 6 insertions(+), 6 deletions(-)
-
-diff --git a/omap_hwspinlock_test.c b/omap_hwspinlock_test.c
-index b180138..f3e0912 100644
---- a/omap_hwspinlock_test.c
-+++ b/omap_hwspinlock_test.c
-@@ -26,12 +26,12 @@ struct hwspinlock_data {
-	const unsigned int max_locks;
- };
-
--static int hwspin_lock_test(struct hwspinlock *hwlock)
-+static int hwspin_lock_test(struct hwspinlock *hwlock, int lock_num)
- {
-	int i;
-	int ret;
-
--	pr_err("\nTesting lock %d\n", hwspin_lock_get_id(hwlock));
-+	pr_err("\nTesting lock %d\n", lock_num);
-	for (i = 0; i < count; i++) {
-		ret = hwspin_trylock(hwlock);
-		if (ret) {
-@@ -82,7 +82,7 @@ static int hwspin_lock_test_all_locks(unsigned int max_locks)
-			continue;
-		}
-
--		ret1 = hwspin_lock_test(hwlock);
-+		ret1 = hwspin_lock_test(hwlock, i);
-		if (ret1) {
-			pr_err("hwspinlock tests failed on lock %d\n", i);
-			ret = ret1;
-@@ -147,17 +147,17 @@ static int hwspin_lock_test_all_phandle_locks(unsigned int max_locks)
-			continue;
-		}
-
--		ret1 = hwspin_lock_test(hwlock);
-+		ret1 = hwspin_lock_test(hwlock, i);
-		if (ret1) {
-			pr_err("hwspinlock test failed on DT lock %d, ret = %d\n",
--			       hwspin_lock_get_id(hwlock), ret1);
-+			       hwlock_id, 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));
-+			       hwlock_id);
-			ret = ret1;
-		}
-	}
---
-2.17.1
diff --git a/meta-arago-test/recipes-kernel/hwspinlocktest/files/0002-Add-support-for-AM62-SoCs.patch b/meta-arago-test/recipes-kernel/hwspinlocktest/files/0002-Add-support-for-AM62-SoCs.patch
deleted file mode 100644
index 30d9886a..00000000
--- a/meta-arago-test/recipes-kernel/hwspinlocktest/files/0002-Add-support-for-AM62-SoCs.patch
+++ /dev/null
@@ -1,31 +0,0 @@
-From 5ea20dff1aba059cc49e11031f731a422f3fd114 Mon Sep 17 00:00:00 2001
-From: Judith Mendez <jm@ti.com>
-Date: Mon, 18 Nov 2024 14:57:45 -0600
-Subject: [PATCH 2/2] 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.
-
-Upstream-Status: Inactive-Upstream [private repo]
-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/hwspinlocktest_0.1.bb b/meta-arago-test/recipes-kernel/hwspinlocktest/hwspinlocktest_0.1.bb
index 454f0131..50892379 100644
--- a/meta-arago-test/recipes-kernel/hwspinlocktest/hwspinlocktest_0.1.bb
+++ b/meta-arago-test/recipes-kernel/hwspinlocktest/hwspinlocktest_0.1.bb
@@ -6,13 +6,11 @@ LIC_FILES_CHKSUM = "file://LICENSE;md5=bfa02c83df161e37647ee23a2c7eacd4"
 inherit module
 
 SRC_URI = "\
-    git://github.com/sumananna/omap-hwspinlock-test;protocol=https;branch=master \
+    git://github.com/jmenti/omap-hwspinlock-test-jm;protocol=https;branch=master \
     file://0001-Fix-Makefile-for-to-build-with-yocto.patch \
-    file://0002-Add-support-for-AM62-SoCs.patch \
-    file://0001-omap_hwspinlock_test-Remove-hwspin_lock_get_id-calls.patch \
 "
 
-SRCREV = "20f15e79a3a0197e06328c252d23aec225de21ed"
+SRCREV = "125cf8e0e874d3ccb314c542331f01a719f1538a"
 
 # The inherit of module.bbclass will automatically name module packages with
 # "kernel-module-" prefix as required by the oe-core build environment.
-- 
2.51.0



^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [master/scarthgap][PATCH] hwspinlocktest: Update hwspinlock recipe remote repo url
  2025-09-11 21:06 [master/scarthgap][PATCH] hwspinlocktest: Update hwspinlock recipe remote repo url Judith Mendez
@ 2025-09-11 21:10 ` PRC Automation
  2025-09-11 21:13 ` [meta-arago] " Denys Dmytriyenko
  2025-09-12  1:42 ` Bryan Brattlof
  2 siblings, 0 replies; 5+ messages in thread
From: PRC Automation @ 2025-09-11 21:10 UTC (permalink / raw)
  To: Judith Mendez
  Cc: Ryan Eatmon, Chirag Shilwant, Denys Dmytriyenko, Praneeth Bajjuri,
	meta-arago

meta-arago / na / 20250911210645.2142573-1-jm

PRC Results: FAIL

=========================================================
  check-yocto-patches: PASS
=========================================================
Patches
----------------------------------------
All patches passed



=========================================================
  apply-yocto-patch: FAIL
=========================================================
master
=====================
Summary:
- Patch Series: [master/scarthgap][PATCH] hwspinlocktest: Update hwspinlock recipe remote repo url
- Submitter: From: Judith Mendez <jm@ti.com>
-From: Ryan Eatmon <reatmon@ti.com>
-From: Judith Mendez <jm@ti.com>
- Date: Date: Thu, 11 Sep 2025 16:06:45 -0500
-Date: Fri, 11 Apr 2025 13:49:11 -0500
-Date: Mon, 18 Nov 2024 14:57:45 -0600
- Num Patches: 1
- Mailing List (public inbox) Commit SHA: f31e15e8203a40c39a6a13448a4416fea74746bd

Applied to:
- Repository: lcpd-prc-meta-arago
- Base Branch: master-wip
- Commit Author: Thorsten Lannynd <t-lannynd@ti.com>
- Commit Subject: chromium: Replace /dev/video* with Chromium naming convention
- Commit SHA: e3ad867e65837c5892885b41510d09699b5a4b62

Patches
----------------------------------------
All patches applied

scarthgap
=====================
Summary:
- Patch Series: [master/scarthgap][PATCH] hwspinlocktest: Update hwspinlock recipe remote repo url
- Submitter: From: Judith Mendez <jm@ti.com>
-From: Ryan Eatmon <reatmon@ti.com>
-From: Judith Mendez <jm@ti.com>
- Date: Date: Thu, 11 Sep 2025 16:06:45 -0500
-Date: Fri, 11 Apr 2025 13:49:11 -0500
-Date: Mon, 18 Nov 2024 14:57:45 -0600
- Num Patches: 1
- Mailing List (public inbox) Commit SHA: f31e15e8203a40c39a6a13448a4416fea74746bd

Applied to:
- Repository: lcpd-prc-meta-arago
- Base Branch: scarthgap
- Commit Author: LCPD Automation Script <lcpdbld@list.ti.com>
- Commit Subject: CI/CD Auto-Merger: cicd.scarthgap.202509111148
- Commit SHA: 67c473bc6d88a2cce13b2e8c36fcb79cb0015339

Patches
----------------------------------------
FAIL - [master/scarthgap][PATCH] hwspinlocktest: Update hwspinlock recipe remote repo url
    error: patch failed: meta-arago-test/recipes-kernel/hwspinlocktest/hwspinlocktest_0.1.bb:6
    error: meta-arago-test/recipes-kernel/hwspinlocktest/hwspinlocktest_0.1.bb: patch does not apply
    Applying: hwspinlocktest: Update hwspinlock recipe remote repo url
    Patch failed at 0001 hwspinlocktest: Update hwspinlock recipe remote repo url





^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [meta-arago] [master/scarthgap][PATCH] hwspinlocktest: Update hwspinlock recipe remote repo url
  2025-09-11 21:06 [master/scarthgap][PATCH] hwspinlocktest: Update hwspinlock recipe remote repo url Judith Mendez
  2025-09-11 21:10 ` PRC Automation
@ 2025-09-11 21:13 ` Denys Dmytriyenko
  2025-09-11 22:31   ` Judith Mendez
  2025-09-12  1:42 ` Bryan Brattlof
  2 siblings, 1 reply; 5+ messages in thread
From: Denys Dmytriyenko @ 2025-09-11 21:13 UTC (permalink / raw)
  To: jm; +Cc: Ryan Eatmon, Chirag Shilwant, Praneeth Bajjuri, meta-arago

On Thu, Sep 11, 2025 at 04:06:45PM -0500, Mendez, Judith via lists.yoctoproject.org wrote:
> The hwspinlock module is still not moved to git.ti.com. So a new
> fork has been created to add support for am62sip platform.
> 
> Move now to new forked repo and remove patches that have already been
> integrated to the new forked repo master branch.

Why not move it to https://github.com/TexasInstruments/ instead of another 
personal repo?


> Signed-off-by: Judith Mendez <jm@ti.com>
> ---
>  ...test-Remove-hwspin_lock_get_id-calls.patch | 69 -------------------
>  .../0002-Add-support-for-AM62-SoCs.patch      | 31 ---------
>  .../hwspinlocktest/hwspinlocktest_0.1.bb      |  6 +-
>  3 files changed, 2 insertions(+), 104 deletions(-)
>  delete mode 100644 meta-arago-test/recipes-kernel/hwspinlocktest/files/0001-omap_hwspinlock_test-Remove-hwspin_lock_get_id-calls.patch
>  delete mode 100644 meta-arago-test/recipes-kernel/hwspinlocktest/files/0002-Add-support-for-AM62-SoCs.patch
> 
> diff --git a/meta-arago-test/recipes-kernel/hwspinlocktest/files/0001-omap_hwspinlock_test-Remove-hwspin_lock_get_id-calls.patch b/meta-arago-test/recipes-kernel/hwspinlocktest/files/0001-omap_hwspinlock_test-Remove-hwspin_lock_get_id-calls.patch
> deleted file mode 100644
> index d4448397..00000000
> --- a/meta-arago-test/recipes-kernel/hwspinlocktest/files/0001-omap_hwspinlock_test-Remove-hwspin_lock_get_id-calls.patch
> +++ /dev/null
> @@ -1,69 +0,0 @@
> -From fb986ed355cd488da64da9eb94d0f260bb329d5b Mon Sep 17 00:00:00 2001
> -From: Ryan Eatmon <reatmon@ti.com>
> -Date: Fri, 11 Apr 2025 13:49:11 -0500
> -Subject: [PATCH] omap_hwspinlock_test: Remove hwspin_lock_get_id() calls
> -
> -The hwspin_lock_get_id() function was removed in the most recent
> -kernel [1].  We can safely stop using this function even for older kernels,
> -so go ahead and take these changes.
> -
> -[1] https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/?id=fec04edb74126f21ac628c7be763c97deb49f69d
> -
> -Upstream-Status: Inactive-Upstream [private repo]
> -
> -Signed-off-by: Ryan Eatmon <reatmon@ti.com>
> ----
> - omap_hwspinlock_test.c | 12 ++++++------
> - 1 file changed, 6 insertions(+), 6 deletions(-)
> -
> -diff --git a/omap_hwspinlock_test.c b/omap_hwspinlock_test.c
> -index b180138..f3e0912 100644
> ---- a/omap_hwspinlock_test.c
> -+++ b/omap_hwspinlock_test.c
> -@@ -26,12 +26,12 @@ struct hwspinlock_data {
> -	const unsigned int max_locks;
> - };
> -
> --static int hwspin_lock_test(struct hwspinlock *hwlock)
> -+static int hwspin_lock_test(struct hwspinlock *hwlock, int lock_num)
> - {
> -	int i;
> -	int ret;
> -
> --	pr_err("\nTesting lock %d\n", hwspin_lock_get_id(hwlock));
> -+	pr_err("\nTesting lock %d\n", lock_num);
> -	for (i = 0; i < count; i++) {
> -		ret = hwspin_trylock(hwlock);
> -		if (ret) {
> -@@ -82,7 +82,7 @@ static int hwspin_lock_test_all_locks(unsigned int max_locks)
> -			continue;
> -		}
> -
> --		ret1 = hwspin_lock_test(hwlock);
> -+		ret1 = hwspin_lock_test(hwlock, i);
> -		if (ret1) {
> -			pr_err("hwspinlock tests failed on lock %d\n", i);
> -			ret = ret1;
> -@@ -147,17 +147,17 @@ static int hwspin_lock_test_all_phandle_locks(unsigned int max_locks)
> -			continue;
> -		}
> -
> --		ret1 = hwspin_lock_test(hwlock);
> -+		ret1 = hwspin_lock_test(hwlock, i);
> -		if (ret1) {
> -			pr_err("hwspinlock test failed on DT lock %d, ret = %d\n",
> --			       hwspin_lock_get_id(hwlock), ret1);
> -+			       hwlock_id, 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));
> -+			       hwlock_id);
> -			ret = ret1;
> -		}
> -	}
> ---
> -2.17.1
> diff --git a/meta-arago-test/recipes-kernel/hwspinlocktest/files/0002-Add-support-for-AM62-SoCs.patch b/meta-arago-test/recipes-kernel/hwspinlocktest/files/0002-Add-support-for-AM62-SoCs.patch
> deleted file mode 100644
> index 30d9886a..00000000
> --- a/meta-arago-test/recipes-kernel/hwspinlocktest/files/0002-Add-support-for-AM62-SoCs.patch
> +++ /dev/null
> @@ -1,31 +0,0 @@
> -From 5ea20dff1aba059cc49e11031f731a422f3fd114 Mon Sep 17 00:00:00 2001
> -From: Judith Mendez <jm@ti.com>
> -Date: Mon, 18 Nov 2024 14:57:45 -0600
> -Subject: [PATCH 2/2] 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.
> -
> -Upstream-Status: Inactive-Upstream [private repo]
> -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/hwspinlocktest_0.1.bb b/meta-arago-test/recipes-kernel/hwspinlocktest/hwspinlocktest_0.1.bb
> index 454f0131..50892379 100644
> --- a/meta-arago-test/recipes-kernel/hwspinlocktest/hwspinlocktest_0.1.bb
> +++ b/meta-arago-test/recipes-kernel/hwspinlocktest/hwspinlocktest_0.1.bb
> @@ -6,13 +6,11 @@ LIC_FILES_CHKSUM = "file://LICENSE;md5=bfa02c83df161e37647ee23a2c7eacd4"
>  inherit module
>  
>  SRC_URI = "\
> -    git://github.com/sumananna/omap-hwspinlock-test;protocol=https;branch=master \
> +    git://github.com/jmenti/omap-hwspinlock-test-jm;protocol=https;branch=master \
>      file://0001-Fix-Makefile-for-to-build-with-yocto.patch \
> -    file://0002-Add-support-for-AM62-SoCs.patch \
> -    file://0001-omap_hwspinlock_test-Remove-hwspin_lock_get_id-calls.patch \
>  "
>  
> -SRCREV = "20f15e79a3a0197e06328c252d23aec225de21ed"
> +SRCREV = "125cf8e0e874d3ccb314c542331f01a719f1538a"
>  
>  # The inherit of module.bbclass will automatically name module packages with
>  # "kernel-module-" prefix as required by the oe-core build environment.
> -- 
> 2.51.0


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [meta-arago] [master/scarthgap][PATCH] hwspinlocktest: Update hwspinlock recipe remote repo url
  2025-09-11 21:13 ` [meta-arago] " Denys Dmytriyenko
@ 2025-09-11 22:31   ` Judith Mendez
  0 siblings, 0 replies; 5+ messages in thread
From: Judith Mendez @ 2025-09-11 22:31 UTC (permalink / raw)
  To: Denys Dmytriyenko
  Cc: Ryan Eatmon, Chirag Shilwant, Praneeth Bajjuri, meta-arago

Hi Denys,

On 9/11/25 4:13 PM, Denys Dmytriyenko wrote:
> On Thu, Sep 11, 2025 at 04:06:45PM -0500, Mendez, Judith via lists.yoctoproject.org wrote:
>> The hwspinlock module is still not moved to git.ti.com. So a new
>> fork has been created to add support for am62sip platform.
>>
>> Move now to new forked repo and remove patches that have already been
>> integrated to the new forked repo master branch.
> 
> Why not move it to https://github.com/TexasInstruments/ instead of another
> personal repo?

I think we have found a way to move ti ti.com. Will send a v2 with that
url instead.

~ Judith

> 
> 
>> Signed-off-by: Judith Mendez <jm@ti.com>
>> ---
>>   ...test-Remove-hwspin_lock_get_id-calls.patch | 69 -------------------
>>   .../0002-Add-support-for-AM62-SoCs.patch      | 31 ---------
>>   .../hwspinlocktest/hwspinlocktest_0.1.bb      |  6 +-
>>   3 files changed, 2 insertions(+), 104 deletions(-)
>>   delete mode 100644 meta-arago-test/recipes-kernel/hwspinlocktest/files/0001-omap_hwspinlock_test-Remove-hwspin_lock_get_id-calls.patch
>>   delete mode 100644 meta-arago-test/recipes-kernel/hwspinlocktest/files/0002-Add-support-for-AM62-SoCs.patch
>>
>> diff --git a/meta-arago-test/recipes-kernel/hwspinlocktest/files/0001-omap_hwspinlock_test-Remove-hwspin_lock_get_id-calls.patch b/meta-arago-test/recipes-kernel/hwspinlocktest/files/0001-omap_hwspinlock_test-Remove-hwspin_lock_get_id-calls.patch
>> deleted file mode 100644
>> index d4448397..00000000
>> --- a/meta-arago-test/recipes-kernel/hwspinlocktest/files/0001-omap_hwspinlock_test-Remove-hwspin_lock_get_id-calls.patch
>> +++ /dev/null
>> @@ -1,69 +0,0 @@
>> -From fb986ed355cd488da64da9eb94d0f260bb329d5b Mon Sep 17 00:00:00 2001
>> -From: Ryan Eatmon <reatmon@ti.com>
>> -Date: Fri, 11 Apr 2025 13:49:11 -0500
>> -Subject: [PATCH] omap_hwspinlock_test: Remove hwspin_lock_get_id() calls
>> -
>> -The hwspin_lock_get_id() function was removed in the most recent
>> -kernel [1].  We can safely stop using this function even for older kernels,
>> -so go ahead and take these changes.
>> -
>> -[1] https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/?id=fec04edb74126f21ac628c7be763c97deb49f69d
>> -
>> -Upstream-Status: Inactive-Upstream [private repo]
>> -
>> -Signed-off-by: Ryan Eatmon <reatmon@ti.com>
>> ----
>> - omap_hwspinlock_test.c | 12 ++++++------
>> - 1 file changed, 6 insertions(+), 6 deletions(-)
>> -
>> -diff --git a/omap_hwspinlock_test.c b/omap_hwspinlock_test.c
>> -index b180138..f3e0912 100644
>> ---- a/omap_hwspinlock_test.c
>> -+++ b/omap_hwspinlock_test.c
>> -@@ -26,12 +26,12 @@ struct hwspinlock_data {
>> -	const unsigned int max_locks;
>> - };
>> -
>> --static int hwspin_lock_test(struct hwspinlock *hwlock)
>> -+static int hwspin_lock_test(struct hwspinlock *hwlock, int lock_num)
>> - {
>> -	int i;
>> -	int ret;
>> -
>> --	pr_err("\nTesting lock %d\n", hwspin_lock_get_id(hwlock));
>> -+	pr_err("\nTesting lock %d\n", lock_num);
>> -	for (i = 0; i < count; i++) {
>> -		ret = hwspin_trylock(hwlock);
>> -		if (ret) {
>> -@@ -82,7 +82,7 @@ static int hwspin_lock_test_all_locks(unsigned int max_locks)
>> -			continue;
>> -		}
>> -
>> --		ret1 = hwspin_lock_test(hwlock);
>> -+		ret1 = hwspin_lock_test(hwlock, i);
>> -		if (ret1) {
>> -			pr_err("hwspinlock tests failed on lock %d\n", i);
>> -			ret = ret1;
>> -@@ -147,17 +147,17 @@ static int hwspin_lock_test_all_phandle_locks(unsigned int max_locks)
>> -			continue;
>> -		}
>> -
>> --		ret1 = hwspin_lock_test(hwlock);
>> -+		ret1 = hwspin_lock_test(hwlock, i);
>> -		if (ret1) {
>> -			pr_err("hwspinlock test failed on DT lock %d, ret = %d\n",
>> --			       hwspin_lock_get_id(hwlock), ret1);
>> -+			       hwlock_id, 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));
>> -+			       hwlock_id);
>> -			ret = ret1;
>> -		}
>> -	}
>> ---
>> -2.17.1
>> diff --git a/meta-arago-test/recipes-kernel/hwspinlocktest/files/0002-Add-support-for-AM62-SoCs.patch b/meta-arago-test/recipes-kernel/hwspinlocktest/files/0002-Add-support-for-AM62-SoCs.patch
>> deleted file mode 100644
>> index 30d9886a..00000000
>> --- a/meta-arago-test/recipes-kernel/hwspinlocktest/files/0002-Add-support-for-AM62-SoCs.patch
>> +++ /dev/null
>> @@ -1,31 +0,0 @@
>> -From 5ea20dff1aba059cc49e11031f731a422f3fd114 Mon Sep 17 00:00:00 2001
>> -From: Judith Mendez <jm@ti.com>
>> -Date: Mon, 18 Nov 2024 14:57:45 -0600
>> -Subject: [PATCH 2/2] 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.
>> -
>> -Upstream-Status: Inactive-Upstream [private repo]
>> -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/hwspinlocktest_0.1.bb b/meta-arago-test/recipes-kernel/hwspinlocktest/hwspinlocktest_0.1.bb
>> index 454f0131..50892379 100644
>> --- a/meta-arago-test/recipes-kernel/hwspinlocktest/hwspinlocktest_0.1.bb
>> +++ b/meta-arago-test/recipes-kernel/hwspinlocktest/hwspinlocktest_0.1.bb
>> @@ -6,13 +6,11 @@ LIC_FILES_CHKSUM = "file://LICENSE;md5=bfa02c83df161e37647ee23a2c7eacd4"
>>   inherit module
>>   
>>   SRC_URI = "\
>> -    git://github.com/sumananna/omap-hwspinlock-test;protocol=https;branch=master \
>> +    git://github.com/jmenti/omap-hwspinlock-test-jm;protocol=https;branch=master \
>>       file://0001-Fix-Makefile-for-to-build-with-yocto.patch \
>> -    file://0002-Add-support-for-AM62-SoCs.patch \
>> -    file://0001-omap_hwspinlock_test-Remove-hwspin_lock_get_id-calls.patch \
>>   "
>>   
>> -SRCREV = "20f15e79a3a0197e06328c252d23aec225de21ed"
>> +SRCREV = "125cf8e0e874d3ccb314c542331f01a719f1538a"
>>   
>>   # The inherit of module.bbclass will automatically name module packages with
>>   # "kernel-module-" prefix as required by the oe-core build environment.
>> -- 
>> 2.51.0



^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [master/scarthgap][PATCH] hwspinlocktest: Update hwspinlock recipe remote repo url
  2025-09-11 21:06 [master/scarthgap][PATCH] hwspinlocktest: Update hwspinlock recipe remote repo url Judith Mendez
  2025-09-11 21:10 ` PRC Automation
  2025-09-11 21:13 ` [meta-arago] " Denys Dmytriyenko
@ 2025-09-12  1:42 ` Bryan Brattlof
  2 siblings, 0 replies; 5+ messages in thread
From: Bryan Brattlof @ 2025-09-12  1:42 UTC (permalink / raw)
  To: Judith Mendez
  Cc: Ryan Eatmon, Chirag Shilwant, Denys Dmytriyenko, Praneeth Bajjuri,
	meta-arago

On September 11, 2025 thus sayeth Judith Mendez:
> The hwspinlock module is still not moved to git.ti.com. So a new
> fork has been created to add support for am62sip platform.
> 
> Move now to new forked repo and remove patches that have already been
> integrated to the new forked repo master branch.
> 

Why are we marking our own testing recipe as inactive and patching it. 
This test looks like a prime candidate for a kselftest port rather than 
punting this problem for each baseline forever.

~Bryan


^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2025-09-12  1:42 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-09-11 21:06 [master/scarthgap][PATCH] hwspinlocktest: Update hwspinlock recipe remote repo url Judith Mendez
2025-09-11 21:10 ` PRC Automation
2025-09-11 21:13 ` [meta-arago] " Denys Dmytriyenko
2025-09-11 22:31   ` Judith Mendez
2025-09-12  1:42 ` Bryan Brattlof

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.