From: Chao-ying Fu <icebergfu@gmail.com>
To: opensbi@lists.infradead.org
Subject: [PATCH] Work with hartid equal to or greater than SBI_HARTMASK_MAX_BITS.
Date: Thu, 9 Jan 2025 15:08:53 -0800 [thread overview]
Message-ID: <20250109230853.2440-1-cfu@mips.com> (raw)
Some platforms use hartid that is equal to greater than SBI_HARTMASK_MAX_BITS,
so we should remove the check.
---
lib/utils/fdt/fdt_domain.c | 3 ---
lib/utils/fdt/fdt_helper.c | 9 ---------
platform/generic/platform.c | 3 ---
3 files changed, 15 deletions(-)
diff --git a/lib/utils/fdt/fdt_domain.c b/lib/utils/fdt/fdt_domain.c
index 4bc7ed8..6bc63c2 100644
--- a/lib/utils/fdt/fdt_domain.c
+++ b/lib/utils/fdt/fdt_domain.c
@@ -471,9 +471,6 @@ static int __fdt_parse_domain(const void *fdt, int domain_offset, void *opaque)
if (err)
continue;
- if (SBI_HARTMASK_MAX_BITS <= val32)
- continue;
-
if (!fdt_node_is_enabled(fdt, cpu_offset))
continue;
diff --git a/lib/utils/fdt/fdt_helper.c b/lib/utils/fdt/fdt_helper.c
index cb350e5..aec08fa 100644
--- a/lib/utils/fdt/fdt_helper.c
+++ b/lib/utils/fdt/fdt_helper.c
@@ -1032,9 +1032,6 @@ int fdt_parse_aclint_node(const void *fdt, int nodeoffset,
if (rc)
continue;
- if (SBI_HARTMASK_MAX_BITS <= hartid)
- continue;
-
if (match_hwirq == hwirq) {
if (hartid < first_hartid)
first_hartid = hartid;
@@ -1097,9 +1094,6 @@ int fdt_parse_plmt_node(const void *fdt, int nodeoffset, unsigned long *plmt_bas
if (rc)
continue;
- if (SBI_HARTMASK_MAX_BITS <= hartid)
- continue;
-
if (hwirq == IRQ_M_TIMER)
hcount++;
}
@@ -1153,9 +1147,6 @@ int fdt_parse_plicsw_node(const void *fdt, int nodeoffset, unsigned long *plicsw
if (rc)
continue;
- if (SBI_HARTMASK_MAX_BITS <= hartid)
- continue;
-
if (hwirq == IRQ_M_SOFT)
hcount++;
}
diff --git a/platform/generic/platform.c b/platform/generic/platform.c
index c03ed88..331ff8e 100644
--- a/platform/generic/platform.c
+++ b/platform/generic/platform.c
@@ -200,9 +200,6 @@ unsigned long fw_platform_init(unsigned long arg0, unsigned long arg1,
if (rc)
continue;
- if (SBI_HARTMASK_MAX_BITS <= hartid)
- continue;
-
if (!fdt_node_is_enabled(fdt, cpu_offset))
continue;
--
2.47.1
next reply other threads:[~2025-01-09 23:08 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-01-09 23:08 Chao-ying Fu [this message]
2025-01-10 0:42 ` [PATCH] Work with hartid equal to or greater than SBI_HARTMASK_MAX_BITS Inochi Amaoto
2025-01-10 1:20 ` Chao-ying Fu
2025-01-10 0:57 ` Jessica Clarke
2025-01-10 1:36 ` Chao-ying Fu
2025-01-15 23:46 ` [PATCH v2] " Raj Vishwanathan
2025-01-16 1:01 ` Xiang W
2025-01-21 1:12 ` [PATCH v3] " Raj Vishwanathan
2025-01-21 3:29 ` Xiang W
2025-01-22 0:42 ` Raj Vishwanathan
2025-02-11 4:51 ` Anup Patel
2025-01-27 20:20 ` [PATCH v3] New configuration CONFIG_SBI_SCRATCH_ALLOC_ALIGNMENT Raj Vishwanathan
2025-01-28 4:36 ` Xiang W
2025-02-11 4:45 ` Anup Patel
2025-03-05 0:31 ` Raj Vishwanathan
2025-03-05 2:01 ` Samuel Holland
2025-03-09 14:44 ` [PATCH v4] Set the scratch allocation to alignment to cacheline size Raj Vishwanathan
2025-03-10 5:26 ` Xiang W
2025-03-13 17:53 ` Raj Vishwanathan
2025-03-14 1:31 ` Xiang W
2025-01-28 21:33 ` [PATCH v3] Check that hartid is within the SBI_HARTMASK_MAX_BITS Raj Vishwanathan
2025-01-29 9:07 ` Xiang W
2025-02-11 22:00 ` [PATCH v3] lib: utils:Check that hartid is valid Raj Vishwanathan
2025-02-12 4:01 ` Anup Patel
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20250109230853.2440-1-cfu@mips.com \
--to=icebergfu@gmail.com \
--cc=opensbi@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox