2023-07-28 11:28:34

by Varshini Rajendran

Subject: [PATCH v3 42/50] irqchip/atmel-aic5: Add support for sam9x7 aic

From: Hari Prasath <[email protected]>

Add support for the Advanced interrupt controller(AIC) chip in the sam9x7.

Signed-off-by: Hari Prasath <[email protected]>
Signed-off-by: Varshini Rajendran <[email protected]>
---
drivers/irqchip/irq-atmel-aic5.c | 10 ++++++++++
1 file changed, 10 insertions(+)

diff --git a/drivers/irqchip/irq-atmel-aic5.c b/drivers/irqchip/irq-atmel-aic5.c
index 145535bd7560..bab11900f3ef 100644
--- a/drivers/irqchip/irq-atmel-aic5.c
+++ b/drivers/irqchip/irq-atmel-aic5.c
@@ -320,6 +320,7 @@ static const struct of_device_id aic5_irq_fixups[] __initconst = {
{ .compatible = "atmel,sama5d3", .data = sama5d3_aic_irq_fixup },
{ .compatible = "atmel,sama5d4", .data = sama5d3_aic_irq_fixup },
{ .compatible = "microchip,sam9x60", .data = sam9x60_aic_irq_fixup },
+ { .compatible = "microchip,sam9x7", .data = sam9x60_aic_irq_fixup },
{ /* sentinel */ },
};

@@ -406,3 +407,12 @@ static int __init sam9x60_aic5_of_init(struct device_node *node,
return aic5_of_init(node, parent, NR_SAM9X60_IRQS);
}
IRQCHIP_DECLARE(sam9x60_aic5, "microchip,sam9x60-aic", sam9x60_aic5_of_init);
+
+#define NR_SAM9X7_IRQS 70
+
+static int __init sam9x7_aic5_of_init(struct device_node *node,
+ struct device_node *parent)
+{
+ return aic5_of_init(node, parent, NR_SAM9X7_IRQS);
+}
+IRQCHIP_DECLARE(sam9x7_aic5, "microchip,sam9x7-aic", sam9x7_aic5_of_init);
--
2.25.1

2023-07-28 11:32:58

by Marc Zyngier

Subject: Re: [PATCH v3 42/50] irqchip/atmel-aic5: Add support for sam9x7 aic

On 2023-07-28 11:30, Varshini Rajendran wrote:
> From: Hari Prasath <[email protected]>
>
> Add support for the Advanced interrupt controller(AIC) chip in the
> sam9x7.
>
> Signed-off-by: Hari Prasath <[email protected]>
> Signed-off-by: Varshini Rajendran <[email protected]>
> ---
> drivers/irqchip/irq-atmel-aic5.c | 10 ++++++++++
> 1 file changed, 10 insertions(+)
>
> diff --git a/drivers/irqchip/irq-atmel-aic5.c
> b/drivers/irqchip/irq-atmel-aic5.c
> index 145535bd7560..bab11900f3ef 100644
> --- a/drivers/irqchip/irq-atmel-aic5.c
> +++ b/drivers/irqchip/irq-atmel-aic5.c
> @@ -320,6 +320,7 @@ static const struct of_device_id aic5_irq_fixups[]
> __initconst = {
> { .compatible = "atmel,sama5d3", .data = sama5d3_aic_irq_fixup },
> { .compatible = "atmel,sama5d4", .data = sama5d3_aic_irq_fixup },
> { .compatible = "microchip,sam9x60", .data = sam9x60_aic_irq_fixup },
> + { .compatible = "microchip,sam9x7", .data = sam9x60_aic_irq_fixup },
> { /* sentinel */ },
> };
>
> @@ -406,3 +407,12 @@ static int __init sam9x60_aic5_of_init(struct
> device_node *node,
> return aic5_of_init(node, parent, NR_SAM9X60_IRQS);
> }
> IRQCHIP_DECLARE(sam9x60_aic5, "microchip,sam9x60-aic",
> sam9x60_aic5_of_init);
> +
> +#define NR_SAM9X7_IRQS 70

One can only wonder why this stuff doesn't come from the device tree,
given that there is no other meaningful difference between this
version of the IP and the previous ones...

M.

> +
> +static int __init sam9x7_aic5_of_init(struct device_node *node,
> + struct device_node *parent)
> +{
> + return aic5_of_init(node, parent, NR_SAM9X7_IRQS);
> +}
> +IRQCHIP_DECLARE(sam9x7_aic5, "microchip,sam9x7-aic",
> sam9x7_aic5_of_init);

--
Jazz is not dead. It just smells funny...