lkml.org 
[lkml]   [2020]   [May]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 2/2] irqchip: loongson-*: Fix COMPILE_TEST
On Sat, 30 May 2020 13:11:13 +0100,
Jiaxun Yang <jiaxun.yang@flygoat.com> wrote:
>
> spurious_interrupt helper only exists on MIPS and x86,
> so define a dummy function on other architectures to fix
> COMPILE_TEST.
>
> Reported-by: kbuild test robot <lkp@intel.com>
> Signed-off-by: Jiaxun Yang <jiaxun.yang@flygoat.com>
> ---
> drivers/irqchip/irq-loongson-htpic.c | 4 ++++
> drivers/irqchip/irq-loongson-htvec.c | 4 ++++
> drivers/irqchip/irq-loongson-liointc.c | 4 ++++
> 3 files changed, 12 insertions(+)
>
> diff --git a/drivers/irqchip/irq-loongson-htpic.c b/drivers/irqchip/irq-loongson-htpic.c
> index dd018c22ea83..7f38fdb2cb43 100644
> --- a/drivers/irqchip/irq-loongson-htpic.c
> +++ b/drivers/irqchip/irq-loongson-htpic.c
> @@ -26,6 +26,10 @@ struct loongson_htpic {
>
> static struct loongson_htpic *htpic;
>
> +#if !defined(CONFIG_MIPS) && !defined(CONFIG_X86)
> +static inline void spurious_interrupt(void) { }
> +#endif

No, that's really horrible, and it doesn't scale. Dropping the
COMPILE_TEST dependency is cleaner, and we should look into having a
common framework across architectures to account for spurious
interrupts.

Not to mention that the HTPIC is not compiled with COMPILE_TEST anyway.

> +
> static void htpic_irq_dispatch(struct irq_desc *desc)
> {
> struct loongson_htpic *priv = irq_desc_get_handler_data(desc);
> diff --git a/drivers/irqchip/irq-loongson-htvec.c b/drivers/irqchip/irq-loongson-htvec.c
> index 1ece9337c78d..03c3973d7525 100644
> --- a/drivers/irqchip/irq-loongson-htvec.c
> +++ b/drivers/irqchip/irq-loongson-htvec.c
> @@ -33,6 +33,10 @@ struct htvec {
> raw_spinlock_t htvec_lock;
> };
>
> +#if !defined(CONFIG_MIPS) && !defined(CONFIG_X86)
> +static inline void spurious_interrupt(void) { }
> +#endif

I've already addressed this by dropping COMPILE_TEST.

> +
> static void htvec_irq_dispatch(struct irq_desc *desc)
> {
> int i;
> diff --git a/drivers/irqchip/irq-loongson-liointc.c b/drivers/irqchip/irq-loongson-liointc.c
> index 63b61474a0cc..e31744e2d800 100644
> --- a/drivers/irqchip/irq-loongson-liointc.c
> +++ b/drivers/irqchip/irq-loongson-liointc.c
> @@ -46,6 +46,10 @@ struct liointc_priv {
> bool has_lpc_irq_errata;
> };
>
> +#if !defined(CONFIG_MIPS) && !defined(CONFIG_X86)
> +static inline void spurious_interrupt(void) { }
> +#endif

Neither is this one.

M.

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

\
 
 \ /
  Last update: 2020-05-30 15:11    [W:0.089 / U:5.984 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site