* [PATCH] include/linux/icmpv6.h - remove duplicate include
@ 2008-03-09 23:55 Joe Perches
2008-03-24 5:18 ` David Miller
0 siblings, 1 reply; 4+ messages in thread
From: Joe Perches @ 2008-03-09 23:55 UTC (permalink / raw)
To: Hideaki YOSHIFUJI, netdev; +Cc: LKML
Remove duplicate #include <linux/skbuff.h>
Combine #ifdef __KERNEL__ blocks
Neaten function prototypes
Signed-off-by: Joe Perches <joe@perches.com>
include/linux/icmpv6.h | 32 +++++++++++---------------------
1 files changed, 11 insertions(+), 21 deletions(-)
diff --git a/include/linux/icmpv6.h b/include/linux/icmpv6.h
index 7c5e981..5185ffa 100644
--- a/include/linux/icmpv6.h
+++ b/include/linux/icmpv6.h
@@ -75,15 +75,6 @@ struct icmp6hdr {
#define icmp6_router_pref icmp6_dataun.u_nd_ra.router_pref
};
-#ifdef __KERNEL__
-#include <linux/skbuff.h>
-
-static inline struct icmp6hdr *icmp6_hdr(const struct sk_buff *skb)
-{
- return (struct icmp6hdr *)skb_transport_header(skb);
-}
-#endif
-
#define ICMPV6_ROUTER_PREF_LOW 0x3
#define ICMPV6_ROUTER_PREF_MEDIUM 0x0
#define ICMPV6_ROUTER_PREF_HIGH 0x1
@@ -166,22 +157,21 @@ struct icmp6_filter {
#define MLD2_ALL_MCR_INIT { { { 0xff,0x02,0,0,0,0,0,0,0,0,0,0,0,0,0,0x16 } } }
#ifdef __KERNEL__
-
-#include <linux/netdevice.h>
#include <linux/skbuff.h>
+#include <linux/netdevice.h>
+static inline struct icmp6hdr *icmp6_hdr(const struct sk_buff *skb)
+{
+ return (struct icmp6hdr *)skb_transport_header(skb);
+}
-extern void icmpv6_send(struct sk_buff *skb,
- int type, int code,
- __u32 info,
- struct net_device *dev);
+extern void icmpv6_send(struct sk_buff *skb, int type, int code, __u32 info,
+ struct net_device *dev);
-extern int icmpv6_init(struct net_proto_family *ops);
-extern int icmpv6_err_convert(int type, int code,
- int *err);
-extern void icmpv6_cleanup(void);
-extern void icmpv6_param_prob(struct sk_buff *skb,
- int code, int pos);
+extern int icmpv6_init(struct net_proto_family *ops);
+extern int icmpv6_err_convert(int type, int code, int *err);
+extern void icmpv6_cleanup(void);
+extern void icmpv6_param_prob(struct sk_buff *skb, int code, int pos);
#endif
#endif
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] include/linux/icmpv6.h - remove duplicate include
2008-03-09 23:55 [PATCH] include/linux/icmpv6.h - remove duplicate include Joe Perches
@ 2008-03-24 5:18 ` David Miller
2008-03-24 17:46 ` [PATCH 1/2] include/linux/icmpv6.h - Remove duplicate include, combine #ifdef __KERNEL__ sections Joe Perches
2008-03-24 17:46 ` Subject: [PATCH 2/2] include/linux/icmpv6.h - neaten prototypes Joe Perches
0 siblings, 2 replies; 4+ messages in thread
From: David Miller @ 2008-03-24 5:18 UTC (permalink / raw)
To: joe; +Cc: yoshfuji, netdev, linux-kernel
From: Joe Perches <joe@perches.com>
Date: Sun, 09 Mar 2008 16:55:28 -0700
> Remove duplicate #include <linux/skbuff.h>
> Combine #ifdef __KERNEL__ blocks
> Neaten function prototypes
>
> Signed-off-by: Joe Perches <joe@perches.com>
Does not apply to net-2.6.26, dropped.
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH 1/2] include/linux/icmpv6.h - Remove duplicate include, combine #ifdef __KERNEL__ sections
2008-03-24 5:18 ` David Miller
@ 2008-03-24 17:46 ` Joe Perches
2008-03-24 17:46 ` Subject: [PATCH 2/2] include/linux/icmpv6.h - neaten prototypes Joe Perches
1 sibling, 0 replies; 4+ messages in thread
From: Joe Perches @ 2008-03-24 17:46 UTC (permalink / raw)
To: David Miller; +Cc: yoshfuji, netdev, linux-kernel
On Sun, 2008-03-23 at 22:18 -0700, David Miller wrote:
> Does not apply to net-2.6.26, dropped.
Signed-off-by: Joe Perches <joe@perches.com>
---
include/linux/icmpv6.h | 13 ++++---------
1 files changed, 4 insertions(+), 9 deletions(-)
diff --git a/include/linux/icmpv6.h b/include/linux/icmpv6.h
index 0306744..c1a30c7 100644
--- a/include/linux/icmpv6.h
+++ b/include/linux/icmpv6.h
@@ -75,15 +75,6 @@ struct icmp6hdr {
#define icmp6_router_pref icmp6_dataun.u_nd_ra.router_pref
};
-#ifdef __KERNEL__
-#include <linux/skbuff.h>
-
-static inline struct icmp6hdr *icmp6_hdr(const struct sk_buff *skb)
-{
- return (struct icmp6hdr *)skb_transport_header(skb);
-}
-#endif
-
#define ICMPV6_ROUTER_PREF_LOW 0x3
#define ICMPV6_ROUTER_PREF_MEDIUM 0x0
#define ICMPV6_ROUTER_PREF_HIGH 0x1
@@ -170,6 +161,10 @@ struct icmp6_filter {
#include <linux/netdevice.h>
#include <linux/skbuff.h>
+static inline struct icmp6hdr *icmp6_hdr(const struct sk_buff *skb)
+{
+ return (struct icmp6hdr *)skb_transport_header(skb);
+}
extern void icmpv6_send(struct sk_buff *skb,
int type, int code,
--
1.5.4.rc2
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Subject: [PATCH 2/2] include/linux/icmpv6.h - neaten prototypes
2008-03-24 5:18 ` David Miller
2008-03-24 17:46 ` [PATCH 1/2] include/linux/icmpv6.h - Remove duplicate include, combine #ifdef __KERNEL__ sections Joe Perches
@ 2008-03-24 17:46 ` Joe Perches
1 sibling, 0 replies; 4+ messages in thread
From: Joe Perches @ 2008-03-24 17:46 UTC (permalink / raw)
To: David Miller; +Cc: yoshfuji, netdev, linux-kernel
On Sun, 2008-03-23 at 22:18 -0700, David Miller wrote:
> Does not apply to net-2.6.26, dropped.
Signed-off-by: Joe Perches <joe@perches.com>
---
include/linux/icmpv6.h | 27 ++++++++++-----------------
1 files changed, 10 insertions(+), 17 deletions(-)
diff --git a/include/linux/icmpv6.h b/include/linux/icmpv6.h
index c1a30c7..5f39689 100644
--- a/include/linux/icmpv6.h
+++ b/include/linux/icmpv6.h
@@ -166,26 +166,19 @@ static inline struct icmp6hdr *icmp6_hdr(const struct sk_buff *skb)
return (struct icmp6hdr *)skb_transport_header(skb);
}
-extern void icmpv6_send(struct sk_buff *skb,
- int type, int code,
- __u32 info,
- struct net_device *dev);
-
-extern int icmpv6_init(void);
-extern int icmpv6_err_convert(int type, int code,
- int *err);
-extern void icmpv6_cleanup(void);
-extern void icmpv6_param_prob(struct sk_buff *skb,
- int code, int pos);
+extern void icmpv6_send(struct sk_buff *skb, int type, int code, __u32 info,
+ struct net_device *dev);
+
+extern int icmpv6_init(void);
+extern int icmpv6_err_convert(int type, int code, int *err);
+extern void icmpv6_cleanup(void);
+extern void icmpv6_param_prob(struct sk_buff *skb, int code, int pos);
struct flowi;
struct in6_addr;
-extern void icmpv6_flow_init(struct sock *sk,
- struct flowi *fl,
- u8 type,
- const struct in6_addr *saddr,
- const struct in6_addr *daddr,
- int oif);
+extern void icmpv6_flow_init(struct sock *sk, struct flowi *fl, u8 type,
+ const struct in6_addr *saddr,
+ const struct in6_addr *daddr, int oif);
#endif
#endif
--
1.5.4.rc2
^ permalink raw reply related [flat|nested] 4+ messages in thread
end of thread, other threads:[~2008-03-24 17:47 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-03-09 23:55 [PATCH] include/linux/icmpv6.h - remove duplicate include Joe Perches
2008-03-24 5:18 ` David Miller
2008-03-24 17:46 ` [PATCH 1/2] include/linux/icmpv6.h - Remove duplicate include, combine #ifdef __KERNEL__ sections Joe Perches
2008-03-24 17:46 ` Subject: [PATCH 2/2] include/linux/icmpv6.h - neaten prototypes Joe Perches
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.