2019-05-30 23:18:48

by Florian Fainelli

Subject: [PATCH] irqchip/bcm7038-l1: Fix build on ARM/ARM64

The driver uses cpu_logical_map[] which is defined under
arch/{arm,arm64}/include/asm/smp_plat.h.

Fixes: 5f7f0317ed28 ("IRQCHIP: Add new driver for BCM7038-style level 1 interrupt controllers")
Signed-off-by: Florian Fainelli <[email protected]>
---
drivers/irqchip/irq-bcm7038-l1.c | 3 +++
1 file changed, 3 insertions(+)

diff --git a/drivers/irqchip/irq-bcm7038-l1.c b/drivers/irqchip/irq-bcm7038-l1.c
index 0acebac1920b..970857a32508 100644
--- a/drivers/irqchip/irq-bcm7038-l1.c
+++ b/drivers/irqchip/irq-bcm7038-l1.c
@@ -30,6 +30,9 @@
#include <linux/types.h>
#include <linux/irqchip.h>
#include <linux/irqchip/chained_irq.h>
+#if defined(CONFIG_ARM) || defined(CONFIG_ARM64)
+#include <asm/smp_plat.h>
+#endif

#define IRQS_PER_WORD 32
#define REG_BYTES_PER_IRQ_WORD (sizeof(u32) * 4)
--
2.17.1

2019-06-07 18:58:55

by Florian Fainelli

Subject: Re: [PATCH] irqchip/bcm7038-l1: Fix build on ARM/ARM64

On 5/30/19 4:16 PM, Florian Fainelli wrote:
> The driver uses cpu_logical_map[] which is defined under
> arch/{arm,arm64}/include/asm/smp_plat.h.
>
> Fixes: 5f7f0317ed28 ("IRQCHIP: Add new driver for BCM7038-style level 1 interrupt controllers")
> Signed-off-by: Florian Fainelli <[email protected]>

Please discard this patch, we won't need it with:

https://www.armlinux.org.uk/developer/patches/viewpatch.php?id=8866/1
and:
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=a02a532c2a6c79a898cd6c430fe3ad011d9aece3

> ---
> drivers/irqchip/irq-bcm7038-l1.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/drivers/irqchip/irq-bcm7038-l1.c b/drivers/irqchip/irq-bcm7038-l1.c
> index 0acebac1920b..970857a32508 100644
> --- a/drivers/irqchip/irq-bcm7038-l1.c
> +++ b/drivers/irqchip/irq-bcm7038-l1.c
> @@ -30,6 +30,9 @@
> #include <linux/types.h>
> #include <linux/irqchip.h>
> #include <linux/irqchip/chained_irq.h>
> +#if defined(CONFIG_ARM) || defined(CONFIG_ARM64)
> +#include <asm/smp_plat.h>
> +#endif
>
> #define IRQS_PER_WORD 32
> #define REG_BYTES_PER_IRQ_WORD (sizeof(u32) * 4)
>


--
Florian