[RFC V2 PATCH 3/8] ARM: kirkwood: nsa310: cleanup includes and unneeded code
Jason Cooper
jason at lakedaemon.net
Sat Jan 26 15:22:08 EST 2013
On Sun, Jan 27, 2013 at 12:08:44AM +0400, Sergei Shtylyov wrote:
> Hello.
>
> On 25-01-2013 9:32, Jason Cooper wrote:
>
> >Signed-off-by: Jason Cooper <jason at lakedaemon.net>
> >---
> > arch/arm/mach-kirkwood/board-nsa310.c | 9 +--------
> > 1 file changed, 1 insertion(+), 8 deletions(-)
>
> >diff --git a/arch/arm/mach-kirkwood/board-nsa310.c b/arch/arm/mach-kirkwood/board-nsa310.c
> >index 1bd328d..0b99533 100644
> >--- a/arch/arm/mach-kirkwood/board-nsa310.c
> >+++ b/arch/arm/mach-kirkwood/board-nsa310.c
> >@@ -10,11 +10,8 @@
> >
> > #include <linux/kernel.h>
> > #include <linux/init.h>
> >-#include <linux/i2c.h>
> >-
> >-#include <asm/mach-types.h>
> >-#include <asm/mach/arch.h>
> > #include <mach/kirkwood.h>
> >+#include <linux/of.h>
>
> You also add an #include and don't mention it anywhere. Or is
> that part of the cleanup?
After removing the unneeded linux/i2c.h, linux/of.h was needed for
of_machine_is_compatible(). i2c.h had included of.h.
thanks for the review,
Jason.
More information about the linux-arm-kernel
mailing list