lkml.org 
[lkml]   [2006]   [Feb]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] remove duplicate #includes
Jean Delvare <khali@linux-fr.org> wrote:
>
> Hi Herbert,
>
> > recently I stumbled over a few files which #include the
> > same .h file twice -- sometimes even in the immediately
> > following line. so I thought I'd look into that to reduce
> > the amount of duplicate includes in the kernel ...
> > (...)
> > diff -NurpP --minimal linux-2.6.16-rc2/drivers/macintosh/therm_pm72.c linux-2.6.16-rc2-mpf/drivers/macintosh/therm_pm72.c
> > --- linux-2.6.16-rc2/drivers/macintosh/therm_pm72.c 2006-02-07 11:52:31 +0100
> > +++ linux-2.6.16-rc2-mpf/drivers/macintosh/therm_pm72.c 2006-02-13 02:07:58 +0100
> > @@ -104,7 +104,6 @@
> > #include <linux/kernel.h>
> > #include <linux/delay.h>
> > #include <linux/sched.h>
> > -#include <linux/i2c.h>
> > #include <linux/slab.h>
> > #include <linux/init.h>
> > #include <linux/spinlock.h>
>
> This one was already taken care of in a different patch:
>
> http://www.kernel.org/pub/linux/kernel/people/akpm/patches/2.6/2.6.16-rc3/2.6.16-rc3-mm1/broken-out/macintosh-cleanup-the-use-of-i2c-headers.patch
>
> So please exclude this part from your patch so as to avoid collisions.
>

Is OK. I'll stage the big cleanup patches like this after everyone else's
patches, so all that is left in this patch is stuff which doesn't intersect
with anyone else's work.

That's one of the advantages of having everyone's development trees all in
one place. (And people who run development trees which aren't in -mm lose).
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2006-02-14 22:25    [W:0.148 / U:0.256 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site