drivers: intc: plic: use per-instance spinlock

Instead of doing an `irq_lock()`, use per-instance spinlock instead.

Signed-off-by: Yong Cong Sin <[email protected]>

Author: ycsin

drivers/interrupt_controller/intc_plic.c

@@ -103,6 +103,7 @@ struct plic_stats {
 };
 
 struct plic_data {
+	struct k_spinlock lock;
 
 #ifdef CONFIG_PLIC_SHELL_IRQ_COUNT
 	struct plic_stats stats;
@@ -279,11 +280,13 @@ static void plic_irq_enable_set_state(uint32_t irq, bool enable)
  */
 void riscv_plic_irq_enable(uint32_t irq)
 {
-	uint32_t key = irq_lock();
+	const struct device *dev = get_plic_dev_from_irq(irq);
+	struct plic_data *data = dev->data;
+	k_spinlock_key_t key = k_spin_lock(&data->lock);
 
 	plic_irq_enable_set_state(irq, true);
 
-	irq_unlock(key);
+	k_spin_unlock(&data->lock, key);
 }
 
 /**
@@ -298,11 +301,13 @@ void riscv_plic_irq_enable(uint32_t irq)
  */
 void riscv_plic_irq_disable(uint32_t irq)
 {
-	uint32_t key = irq_lock();
+	const struct device *dev = get_plic_dev_from_irq(irq);
+	struct plic_data *data = dev->data;
+	k_spinlock_key_t key = k_spin_lock(&data->lock);
 
 	plic_irq_enable_set_state(irq, false);
 
-	irq_unlock(key);
+	k_spin_unlock(&data->lock, key);
 }
 
 /**
@@ -316,11 +321,12 @@ void riscv_plic_irq_disable(uint32_t irq)
 int riscv_plic_irq_is_enabled(uint32_t irq)
 {
 	const struct device *dev = get_plic_dev_from_irq(irq);
+	struct plic_data *data = dev->data;
 	const uint32_t local_irq = irq_from_level_2(irq);
 	uint32_t bit_position = local_irq & PLIC_REG_MASK;
 	uint32_t en_value;
 	int is_enabled = IS_ENABLED(CONFIG_PLIC_IRQ_AFFINITY) ? 0 : 1;
-	uint32_t key = irq_lock();
+	k_spinlock_key_t key = k_spin_lock(&data->lock);
 
 	for (uint32_t cpu_num = 0; cpu_num < arch_num_cpus(); cpu_num++) {
 		mem_addr_t en_addr =
@@ -334,7 +340,7 @@ int riscv_plic_irq_is_enabled(uint32_t irq)
 		}
 	}
 
-	irq_unlock(key);
+	k_spin_unlock(&data->lock, key);
 
 	return is_enabled;
 }
@@ -413,7 +419,7 @@ const struct device *riscv_plic_get_dev(void)
 int riscv_plic_irq_set_affinity(uint32_t irq, uint32_t cpumask)
 {
 	const struct device *dev = get_plic_dev_from_irq(irq);
-	const struct plic_data *data = dev->data;
+	struct plic_data *data = dev->data;
 	__maybe_unused const struct plic_config *config = dev->config;
 	const uint32_t local_irq = irq_from_level_2(irq);
 
@@ -427,7 +433,7 @@ int riscv_plic_irq_set_affinity(uint32_t irq, uint32_t cpumask)
 		return -EINVAL;
 	}
 
-	uint32_t key = irq_lock();
+	k_spinlock_key_t key = k_spin_lock(&data->lock);
 
 	/* Updated irq_cpumask for next time setting plic enable register */
 	data->irq_cpumask[local_irq] = (plic_cpumask_t)cpumask;
@@ -437,7 +443,7 @@ int riscv_plic_irq_set_affinity(uint32_t irq, uint32_t cpumask)
 		riscv_plic_irq_enable(irq);
 	}
 
-	irq_unlock(key);
+	k_spin_unlock(&data->lock, key);
 
 	return 0;
 }