2021-12-14 08:48:10

by Peng Fan (OSS)

[permalink] [raw]
Subject: [PATCH] irqchip/irq-imx-gpcv2: mark imx_gpcv2_instance with __ro_after_init

From: Peng Fan <[email protected]>

imx_gpcv2_instance will not be updated after init, so mark it with
__ro_after_init.

Signed-off-by: Peng Fan <[email protected]>
---
drivers/irqchip/irq-imx-gpcv2.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/irqchip/irq-imx-gpcv2.c b/drivers/irqchip/irq-imx-gpcv2.c
index 5b5a365dbd5e..b9c22f764b4d 100644
--- a/drivers/irqchip/irq-imx-gpcv2.c
+++ b/drivers/irqchip/irq-imx-gpcv2.c
@@ -26,7 +26,7 @@ struct gpcv2_irqchip_data {
u32 cpu2wakeup;
};

-static struct gpcv2_irqchip_data *imx_gpcv2_instance;
+static struct gpcv2_irqchip_data *imx_gpcv2_instance __ro_after_init;

static void __iomem *gpcv2_idx_to_reg(struct gpcv2_irqchip_data *cd, int i)
{
--
2.25.1



2021-12-14 09:33:44

by Lucas Stach

[permalink] [raw]
Subject: Re: [PATCH] irqchip/irq-imx-gpcv2: mark imx_gpcv2_instance with __ro_after_init

Am Dienstag, dem 14.12.2021 um 16:47 +0800 schrieb Peng Fan (OSS):
> From: Peng Fan <[email protected]>
>
> imx_gpcv2_instance will not be updated after init, so mark it with
> __ro_after_init.
>
> Signed-off-by: Peng Fan <[email protected]>

Reviewed-by: Lucas Stach <[email protected]>

> ---
> drivers/irqchip/irq-imx-gpcv2.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/irqchip/irq-imx-gpcv2.c b/drivers/irqchip/irq-imx-gpcv2.c
> index 5b5a365dbd5e..b9c22f764b4d 100644
> --- a/drivers/irqchip/irq-imx-gpcv2.c
> +++ b/drivers/irqchip/irq-imx-gpcv2.c
> @@ -26,7 +26,7 @@ struct gpcv2_irqchip_data {
> u32 cpu2wakeup;
> };
>
> -static struct gpcv2_irqchip_data *imx_gpcv2_instance;
> +static struct gpcv2_irqchip_data *imx_gpcv2_instance __ro_after_init;
>
> static void __iomem *gpcv2_idx_to_reg(struct gpcv2_irqchip_data *cd, int i)
> {



Subject: [irqchip: irq/irqchip-next] irqchip/imx-gpcv2: Mark imx_gpcv2_instance with __ro_after_init

The following commit has been merged into the irq/irqchip-next branch of irqchip:

Commit-ID: 29e525cc825e33ea0da2ac0fbf951e418fc28c69
Gitweb: https://git.kernel.org/pub/scm/linux/kernel/git/maz/arm-platforms/29e525cc825e33ea0da2ac0fbf951e418fc28c69
Author: Peng Fan <[email protected]>
AuthorDate: Tue, 14 Dec 2021 16:47:11 +08:00
Committer: Marc Zyngier <[email protected]>
CommitterDate: Thu, 16 Dec 2021 15:19:43

irqchip/imx-gpcv2: Mark imx_gpcv2_instance with __ro_after_init

imx_gpcv2_instance will not be updated after init, so mark it with
__ro_after_init.

Signed-off-by: Peng Fan <[email protected]>
Reviewed-by: Lucas Stach <[email protected]>
Signed-off-by: Marc Zyngier <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
---
drivers/irqchip/irq-imx-gpcv2.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/irqchip/irq-imx-gpcv2.c b/drivers/irqchip/irq-imx-gpcv2.c
index 5b5a365..b9c22f7 100644
--- a/drivers/irqchip/irq-imx-gpcv2.c
+++ b/drivers/irqchip/irq-imx-gpcv2.c
@@ -26,7 +26,7 @@ struct gpcv2_irqchip_data {
u32 cpu2wakeup;
};

-static struct gpcv2_irqchip_data *imx_gpcv2_instance;
+static struct gpcv2_irqchip_data *imx_gpcv2_instance __ro_after_init;

static void __iomem *gpcv2_idx_to_reg(struct gpcv2_irqchip_data *cd, int i)
{