summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAnup Patel <apatel@ventanamicro.com>2023-04-20 10:14:43 +0530
committerAnup Patel <anup@brainfault.org>2023-06-05 16:51:07 +0530
commit1df52fa7e8db2c23a921014404bd472d20ca42e8 (patch)
treecd414e79c6ad2be2051cd9d8ed4ee3995a4860b5
parentb3594ac1d10ef6c7269c6381a1e826b73b06aca0 (diff)
downloadopensbi-1df52fa7e8db2c23a921014404bd472d20ca42e8.tar.gz
opensbi-1df52fa7e8db2c23a921014404bd472d20ca42e8.zip
lib: utils/irqchip: Don't check hartid in imsic_update_hartid_table()
The imsic_map_hartid_to_data() already checks hartid before using so we don't need to check in imsic_update_hartid_table(). Signed-off-by: Anup Patel <apatel@ventanamicro.com> Reviewed-by: Andrew Jones <ajones@ventanamicro.com>
-rw-r--r--lib/utils/irqchip/fdt_irqchip_imsic.c3
1 files changed, 0 insertions, 3 deletions
diff --git a/lib/utils/irqchip/fdt_irqchip_imsic.c b/lib/utils/irqchip/fdt_irqchip_imsic.c
index 590b2022..d032ac8f 100644
--- a/lib/utils/irqchip/fdt_irqchip_imsic.c
+++ b/lib/utils/irqchip/fdt_irqchip_imsic.c
@@ -12,7 +12,6 @@
#include <sbi/riscv_asm.h>
#include <sbi/sbi_error.h>
#include <sbi/sbi_heap.h>
-#include <sbi/sbi_hartmask.h>
#include <sbi_utils/fdt/fdt_helper.h>
#include <sbi_utils/irqchip/fdt_irqchip.h>
#include <sbi_utils/irqchip/imsic.h>
@@ -44,8 +43,6 @@ static int irqchip_imsic_update_hartid_table(void *fdt, int nodeoff,
err = fdt_parse_hart_id(fdt, cpu_offset, &hartid);
if (err)
return SBI_EINVAL;
- if (SBI_HARTMASK_MAX_BITS <= hartid)
- return SBI_EINVAL;
switch (hwirq) {
case IRQ_M_EXT: