> headers ecsv/netlink > -------------------- > > diff --git a/net/batman-adv/bat_v.c b/net/batman-adv/bat_v.c > index 25fd4be..c5e6db5 100644 > --- a/net/batman-adv/bat_v.c > +++ b/net/batman-adv/bat_v.c > @@ -19,9 +19,9 @@ > #include "main.h" > > #include <linux/bug.h> > +#include <linux/cache.h> > #include <linux/errno.h> > #include <linux/if_ether.h> > -#include <linux/cache.h> > #include <linux/init.h> > #include <linux/jiffies.h> > #include <linux/netdevice.h>
So this is i think a false positive. Linus moved cache.h according the git blame. Nothing to do with my patch. > diff --git a/net/batman-adv/netlink.c b/net/batman-adv/netlink.c > index 949a048..e0fd914 100644 > --- a/net/batman-adv/netlink.c > +++ b/net/batman-adv/netlink.c > @@ -36,10 +36,10 @@ > #include <net/sock.h> > #include <uapi/linux/batman_adv.h> > > -#include "originator.h" > #include "bridge_loop_avoidance.h" > #include "gateway_client.h" > #include "hard-interface.h" > +#include "originator.h" > #include "soft-interface.h" > #include "translation-table.h" This one is interesting. I copied this from your rebased_split_netlink/v4-0008-batman-adv-netlink-add-originator-and-neighbor-ta.patch I did consider changing the order, but since you wrote it that way, it must be O.K. > diff --git a/net/batman-adv/netlink.h b/net/batman-adv/netlink.h > index ef861db..fac7d91 100644 > --- a/net/batman-adv/netlink.h > +++ b/net/batman-adv/netlink.h > @@ -18,12 +18,9 @@ > #ifndef _NET_BATMAN_ADV_NETLINK_H_ > #define _NET_BATMAN_ADV_NETLINK_H_ > > #include "main.h" > > -#include <linux/compiler.h> > -#include <linux/genetlink.h> > #include <net/genetlink.h> > -#include <net/netlink.h> > > struct nlmsghdr; Another false positive. These are from: 9492a25881c75ab9910328ff27834707c83df124 which has your Signed-off-by. So overall, nothing useful in this report. Andrew