lib: utils/irqchip: plic: Fix the off-by-one error in plic_context_init()
authorBin Meng <bmeng@tinylab.org>
Sun, 11 Dec 2022 06:54:22 +0000 (14:54 +0800)
committerAnup Patel <anup@brainfault.org>
Sat, 17 Dec 2022 03:30:35 +0000 (09:00 +0530)
The number of interrupt enable register in words was once correct,
but was wrongly changed to have an off-by-one error since
commit 8c362e7d065e ("lib: irqchip/plic: Factor out a context init function").

Fixes: 8c362e7d065e ("lib: irqchip/plic: Factor out a context init function")
Signed-off-by: Bin Meng <bmeng@tinylab.org>
Reviewed-by: Anup Patel <anup@brainfault.org>
Reviewed-by: Samuel Holland <samuel@sholland.org>
lib/utils/irqchip/plic.c

index c5c1879..5b87d3b 100644 (file)
@@ -121,7 +121,7 @@ int plic_context_init(const struct plic_data *plic, int context_id,
        if (!plic || context_id < 0)
                return SBI_EINVAL;
 
-       ie_words = (plic->num_src + 31) / 32;
+       ie_words = plic->num_src / 32 + 1;
        ie_value = enable ? 0xffffffffU : 0U;
 
        for (u32 i = 0; i < ie_words; i++)