[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID:
<ZQ4PR01MB1202FC81C8EFCC9F1D5ED8CCF2EB2@ZQ4PR01MB1202.CHNPR01.prod.partner.outlook.cn>
Date: Tue, 14 Oct 2025 01:05:38 +0000
From: Changhuang Liang <changhuang.liang@...rfivetech.com>
To: Johan Hovold <johan@...nel.org>, Thomas Gleixner <tglx@...utronix.de>
CC: Florian Fainelli <florian.fainelli@...adcom.com>, Shawn Guo
<shawnguo@...nel.org>, Sascha Hauer <s.hauer@...gutronix.de>, Fabio Estevam
<festevam@...il.com>, Claudiu Beznea <claudiu.beznea@...on.dev>, Neil
Armstrong <neil.armstrong@...aro.org>, Kevin Hilman <khilman@...libre.com>,
Jerome Brunet <jbrunet@...libre.com>, Martin Blumenstingl
<martin.blumenstingl@...glemail.com>, Geert Uytterhoeven
<geert+renesas@...der.be>, Magnus Damm <magnus.damm@...il.com>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject:
回复: [PATCH 08/11] irqchip/starfive-jh8100: Fix section mismatch
Hi, Johan
Thanks for the patch.
> Platform drivers can be probed after their init sections have been discarded so
> the irqchip init callback must not live in init.
>
> Fixes: e4e535036173 ("irqchip: Add StarFive external interrupt controller")
> Cc: Changhuang Liang <changhuang.liang@...rfivetech.com>
> Signed-off-by: Johan Hovold <johan@...nel.org>
Reviewed-by: Changhuang Liang <changhuang.liang@...rfivetech.com>
> ---
> drivers/irqchip/irq-starfive-jh8100-intc.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/irqchip/irq-starfive-jh8100-intc.c
> b/drivers/irqchip/irq-starfive-jh8100-intc.c
> index 2460798ec158..117f2c651ebd 100644
> --- a/drivers/irqchip/irq-starfive-jh8100-intc.c
> +++ b/drivers/irqchip/irq-starfive-jh8100-intc.c
> @@ -114,8 +114,7 @@ static void starfive_intc_irq_handler(struct irq_desc
> *desc)
> chained_irq_exit(chip, desc);
> }
>
> -static int __init starfive_intc_init(struct device_node *intc,
> - struct device_node *parent)
> +static int starfive_intc_init(struct device_node *intc, struct
> +device_node *parent)
> {
> struct starfive_irq_chip *irqc;
> struct reset_control *rst;
> --
> 2.49.1
Powered by blists - more mailing lists