Re: [PATCH] [MIPS] IP22: Small cleanups

From: Ralf Baechle
Date: Sat Oct 25 2008 - 10:28:15 EST


On Sat, Oct 25, 2008 at 01:46:56AM +0300, Dmitri Vorobiev wrote:

> diff --git a/arch/mips/sgi-ip22/ip22-int.c b/arch/mips/sgi-ip22/ip22-int.c
> index f6d9bf4..5b9b4f3 100644
> --- a/arch/mips/sgi-ip22/ip22-int.c
> +++ b/arch/mips/sgi-ip22/ip22-int.c
> @@ -16,6 +16,7 @@
> #include <linux/sched.h>
> #include <linux/interrupt.h>
> #include <linux/irq.h>
> +#include <linux/time.h>
>
> #include <asm/mipsregs.h>
> #include <asm/addrspace.h>
> @@ -23,7 +24,6 @@
> #include <asm/sgi/ioc.h>
> #include <asm/sgi/hpc3.h>
> #include <asm/sgi/ip22.h>
> -#include <asm/time.h>

<linux/time.h> does not include <asm/time.h>. This sort of stuff happens
if you believe checkpatch.pl. You only got away because the header isn't
needed anyway. I'll apply the patch with this bit dropped.

Ralf

PS: Suspicion breeds confidence. At least in case of checkpatch ;-)
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/