Re: [PATCH] irqchip/sifive-plic: Mark two global variables __ro_after_init

[Marc Zyngier]

On Tue, 30 Mar 2021 02:09:11 +0800, Jisheng Zhang wrote:
> All of these two are never modified after init, so they can be
>  __ro_after_init.

Applied to irq/irqchip-next, thanks!

[1/1] irqchip/sifive-plic: Mark two global variables __ro_after_init
  commit: e03b7c1bcbfad6f27b4682f638b98627c4e416ba

Cheers,

M.
-- 
Without deviation from the norm, progress is not possible.

[PATCH] irqchip/sifive-plic: Mark two global variables __ro_after_init

[Jisheng Zhang]

From: Jisheng Zhang 

All of these two are never modified after init, so they can be
 __ro_after_init.

Signed-off-by: Jisheng Zhang 
---
 drivers/irqchip/irq-sifive-plic.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/irqchip/irq-sifive-plic.c 
b/drivers/irqchip/irq-sifive-plic.c
index 6f432d2a5ceb..97d4d04b0a80 100644
--- a/drivers/irqchip/irq-sifive-plic.c
+++ b/drivers/irqchip/irq-sifive-plic.c
@@ -77,8 +77,8 @@ struct plic_handler {
void __iomem*enable_base;
struct plic_priv*priv;
 };
-static int plic_parent_irq;
-static bool plic_cpuhp_setup_done;
+static int plic_parent_irq __ro_after_init;
+static bool plic_cpuhp_setup_done __ro_after_init;
 static DEFINE_PER_CPU(struct plic_handler, plic_handlers);
 
 static inline void plic_toggle(struct plic_handler *handler,
-- 
2.31.0