* [PATCH 1/8] net/ipv6/sit.c: Checkpatch cleanup
@ 2012-04-01 17:49 Eldad Zack
2012-04-01 17:49 ` [PATCH 2/8] net/ipv6/addrconf_core.c: " Eldad Zack
` (7 more replies)
0 siblings, 8 replies; 16+ messages in thread
From: Eldad Zack @ 2012-04-01 17:49 UTC (permalink / raw)
To: David S. Miller, Alexey Kuznetsov, James Morris,
Hideaki YOSHIFUJI, Patrick McHardy
Cc: netdev, linux-kernel, Eldad Zack
sit.c:118: ERROR: "foo * bar" should be "foo *bar"
sit.c:694: ERROR: "(foo*)" should be "(foo *)"
sit.c:724: ERROR: "(foo*)" should be "(foo *)"
Signed-off-by: Eldad Zack <eldad@fogrefinery.com>
---
net/ipv6/sit.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/net/ipv6/sit.c b/net/ipv6/sit.c
index c4ffd17..f9608db 100644
--- a/net/ipv6/sit.c
+++ b/net/ipv6/sit.c
@@ -115,7 +115,7 @@ static struct net_device_stats *ipip6_get_stats(struct net_device *dev)
/*
* Must be invoked with rcu_read_lock
*/
-static struct ip_tunnel * ipip6_tunnel_lookup(struct net *net,
+static struct ip_tunnel *ipip6_tunnel_lookup(struct net *net,
struct net_device *dev, __be32 remote, __be32 local)
{
unsigned int h0 = HASH(remote);
@@ -691,7 +691,7 @@ static netdev_tx_t ipip6_tunnel_xmit(struct sk_buff *skb,
goto tx_error;
}
- addr6 = (const struct in6_addr*)&neigh->primary_key;
+ addr6 = (const struct in6_addr *)&neigh->primary_key;
addr_type = ipv6_addr_type(addr6);
if ((addr_type & IPV6_ADDR_UNICAST) &&
@@ -721,7 +721,7 @@ static netdev_tx_t ipip6_tunnel_xmit(struct sk_buff *skb,
goto tx_error;
}
- addr6 = (const struct in6_addr*)&neigh->primary_key;
+ addr6 = (const struct in6_addr *)&neigh->primary_key;
addr_type = ipv6_addr_type(addr6);
if (addr_type == IPV6_ADDR_ANY) {
--
1.7.9.5
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH 2/8] net/ipv6/addrconf_core.c: Checkpatch cleanup
2012-04-01 17:49 [PATCH 1/8] net/ipv6/sit.c: Checkpatch cleanup Eldad Zack
@ 2012-04-01 17:49 ` Eldad Zack
2012-04-02 3:07 ` David Miller
2012-04-01 17:49 ` [PATCH 3/8] net/ipv6/datagram.c: Checkpatch cleanups Eldad Zack
` (6 subsequent siblings)
7 siblings, 1 reply; 16+ messages in thread
From: Eldad Zack @ 2012-04-01 17:49 UTC (permalink / raw)
To: David S. Miller, Alexey Kuznetsov, James Morris,
Hideaki YOSHIFUJI, Patrick McHardy
Cc: netdev, linux-kernel, Eldad Zack
addrconf_core.c:13: ERROR: space required before the open parenthesis '('
Signed-off-by: Eldad Zack <eldad@fogrefinery.com>
---
net/ipv6/addrconf_core.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/net/ipv6/addrconf_core.c b/net/ipv6/addrconf_core.c
index 399287e..7981bde 100644
--- a/net/ipv6/addrconf_core.c
+++ b/net/ipv6/addrconf_core.c
@@ -10,7 +10,7 @@
static inline unsigned ipv6_addr_scope2type(unsigned scope)
{
- switch(scope) {
+ switch (scope) {
case IPV6_ADDR_SCOPE_NODELOCAL:
return (IPV6_ADDR_SCOPE_TYPE(IPV6_ADDR_SCOPE_NODELOCAL) |
IPV6_ADDR_LOOPBACK);
--
1.7.9.5
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH 3/8] net/ipv6/datagram.c: Checkpatch cleanups
2012-04-01 17:49 [PATCH 1/8] net/ipv6/sit.c: Checkpatch cleanup Eldad Zack
2012-04-01 17:49 ` [PATCH 2/8] net/ipv6/addrconf_core.c: " Eldad Zack
@ 2012-04-01 17:49 ` Eldad Zack
2012-04-02 3:07 ` David Miller
2012-04-01 17:49 ` [PATCH 4/8] net/ipv6/exthdrs.c: " Eldad Zack
` (5 subsequent siblings)
7 siblings, 1 reply; 16+ messages in thread
From: Eldad Zack @ 2012-04-01 17:49 UTC (permalink / raw)
To: David S. Miller, Alexey Kuznetsov, James Morris,
Hideaki YOSHIFUJI, Patrick McHardy
Cc: netdev, linux-kernel, Eldad Zack
datagram.c:101: ERROR: "(foo*)" should be "(foo *)"
datagram.c:521: ERROR: space required before the open parenthesis '('
datagram.c:830: WARNING: braces {} are not necessary for single statement blocks
datagram.c:849: WARNING: braces {} are not necessary for single statement blocks
Signed-off-by: Eldad Zack <eldad@fogrefinery.com>
---
net/ipv6/datagram.c | 10 ++++------
1 file changed, 4 insertions(+), 6 deletions(-)
diff --git a/net/ipv6/datagram.c b/net/ipv6/datagram.c
index 76832c8..f6210d6 100644
--- a/net/ipv6/datagram.c
+++ b/net/ipv6/datagram.c
@@ -98,7 +98,7 @@ int ip6_datagram_connect(struct sock *sk, struct sockaddr *uaddr, int addr_len)
sin.sin_port = usin->sin6_port;
err = ip4_datagram_connect(sk,
- (struct sockaddr*) &sin,
+ (struct sockaddr *) &sin,
sizeof(sin));
ipv4_connected:
@@ -518,7 +518,7 @@ int datagram_recv_ctl(struct sock *sk, struct msghdr *msg, struct sk_buff *skb)
unsigned len;
u8 *ptr = nh + off;
- switch(nexthdr) {
+ switch (nexthdr) {
case IPPROTO_DSTOPTS:
nexthdr = ptr[0];
len = (ptr[1] + 1) << 3;
@@ -827,9 +827,8 @@ int datagram_send_ctl(struct net *net, struct sock *sk,
int tc;
err = -EINVAL;
- if (cmsg->cmsg_len != CMSG_LEN(sizeof(int))) {
+ if (cmsg->cmsg_len != CMSG_LEN(sizeof(int)))
goto exit_f;
- }
tc = *(int *)CMSG_DATA(cmsg);
if (tc < -1 || tc > 0xff)
@@ -846,9 +845,8 @@ int datagram_send_ctl(struct net *net, struct sock *sk,
int df;
err = -EINVAL;
- if (cmsg->cmsg_len != CMSG_LEN(sizeof(int))) {
+ if (cmsg->cmsg_len != CMSG_LEN(sizeof(int)))
goto exit_f;
- }
df = *(int *)CMSG_DATA(cmsg);
if (df < 0 || df > 1)
--
1.7.9.5
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH 4/8] net/ipv6/exthdrs.c: Checkpatch cleanups
2012-04-01 17:49 [PATCH 1/8] net/ipv6/sit.c: Checkpatch cleanup Eldad Zack
2012-04-01 17:49 ` [PATCH 2/8] net/ipv6/addrconf_core.c: " Eldad Zack
2012-04-01 17:49 ` [PATCH 3/8] net/ipv6/datagram.c: Checkpatch cleanups Eldad Zack
@ 2012-04-01 17:49 ` Eldad Zack
2012-04-02 3:07 ` David Miller
2012-04-01 17:49 ` [PATCH 5/8] net/ipv6/exthdrs_core.c: " Eldad Zack
` (4 subsequent siblings)
7 siblings, 1 reply; 16+ messages in thread
From: Eldad Zack @ 2012-04-01 17:49 UTC (permalink / raw)
To: David S. Miller, Alexey Kuznetsov, James Morris,
Hideaki YOSHIFUJI, Patrick McHardy
Cc: netdev, linux-kernel, Eldad Zack
exthdrs.c:726: WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable
exthdrs.c:741: ERROR: "(foo*)" should be "(foo *)"
exthdrs.c:741: ERROR: "(foo*)" should be "(foo *)"
exthdrs.c:744: ERROR: "(foo**)" should be "(foo **)"
exthdrs.c:746: ERROR: "(foo**)" should be "(foo **)"
exthdrs.c:748: ERROR: "(foo**)" should be "(foo **)"
exthdrs.c:750: ERROR: "(foo**)" should be "(foo **)"
exthdrs.c:755: WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable
exthdrs.c:896: WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable
Signed-off-by: Eldad Zack <eldad@fogrefinery.com>
---
net/ipv6/exthdrs.c | 13 +++++--------
1 file changed, 5 insertions(+), 8 deletions(-)
diff --git a/net/ipv6/exthdrs.c b/net/ipv6/exthdrs.c
index 3d641b6..c486b8e 100644
--- a/net/ipv6/exthdrs.c
+++ b/net/ipv6/exthdrs.c
@@ -722,7 +722,6 @@ void ipv6_push_nfrag_opts(struct sk_buff *skb, struct ipv6_txoptions *opt,
if (opt->hopopt)
ipv6_push_exthdr(skb, proto, NEXTHDR_HOP, opt->hopopt);
}
-
EXPORT_SYMBOL(ipv6_push_nfrag_opts);
void ipv6_push_frag_opts(struct sk_buff *skb, struct ipv6_txoptions *opt, u8 *proto)
@@ -738,20 +737,19 @@ ipv6_dup_options(struct sock *sk, struct ipv6_txoptions *opt)
opt2 = sock_kmalloc(sk, opt->tot_len, GFP_ATOMIC);
if (opt2) {
- long dif = (char*)opt2 - (char*)opt;
+ long dif = (char *)opt2 - (char *)opt;
memcpy(opt2, opt, opt->tot_len);
if (opt2->hopopt)
- *((char**)&opt2->hopopt) += dif;
+ *((char **)&opt2->hopopt) += dif;
if (opt2->dst0opt)
- *((char**)&opt2->dst0opt) += dif;
+ *((char **)&opt2->dst0opt) += dif;
if (opt2->dst1opt)
- *((char**)&opt2->dst1opt) += dif;
+ *((char **)&opt2->dst1opt) += dif;
if (opt2->srcrt)
- *((char**)&opt2->srcrt) += dif;
+ *((char **)&opt2->srcrt) += dif;
}
return opt2;
}
-
EXPORT_SYMBOL_GPL(ipv6_dup_options);
static int ipv6_renew_option(void *ohdr,
@@ -892,5 +890,4 @@ struct in6_addr *fl6_update_dst(struct flowi6 *fl6,
fl6->daddr = *((struct rt0_hdr *)opt->srcrt)->addr;
return orig;
}
-
EXPORT_SYMBOL_GPL(fl6_update_dst);
--
1.7.9.5
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH 5/8] net/ipv6/exthdrs_core.c: Checkpatch cleanups
2012-04-01 17:49 [PATCH 1/8] net/ipv6/sit.c: Checkpatch cleanup Eldad Zack
` (2 preceding siblings ...)
2012-04-01 17:49 ` [PATCH 4/8] net/ipv6/exthdrs.c: " Eldad Zack
@ 2012-04-01 17:49 ` Eldad Zack
2012-04-02 3:07 ` David Miller
2012-04-01 17:49 ` [PATCH 6/8] net/ipv6/fib6_rules.c: Checkpatch cleanup Eldad Zack
` (3 subsequent siblings)
7 siblings, 1 reply; 16+ messages in thread
From: Eldad Zack @ 2012-04-01 17:49 UTC (permalink / raw)
To: David S. Miller, Alexey Kuznetsov, James Morris,
Hideaki YOSHIFUJI, Patrick McHardy
Cc: netdev, linux-kernel, Eldad Zack
exthdrs_core.c:113: WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable
exthdrs_core.c:114: WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable
Signed-off-by: Eldad Zack <eldad@fogrefinery.com>
---
net/ipv6/exthdrs_core.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/net/ipv6/exthdrs_core.c b/net/ipv6/exthdrs_core.c
index 72957f4..7b1a884 100644
--- a/net/ipv6/exthdrs_core.c
+++ b/net/ipv6/exthdrs_core.c
@@ -21,6 +21,7 @@ int ipv6_ext_hdr(u8 nexthdr)
(nexthdr == NEXTHDR_NONE) ||
(nexthdr == NEXTHDR_DEST);
}
+EXPORT_SYMBOL(ipv6_ext_hdr);
/*
* Skip any extension headers. This is used by the ICMP module.
@@ -109,6 +110,4 @@ int ipv6_skip_exthdr(const struct sk_buff *skb, int start, u8 *nexthdrp,
*nexthdrp = nexthdr;
return start;
}
-
-EXPORT_SYMBOL(ipv6_ext_hdr);
EXPORT_SYMBOL(ipv6_skip_exthdr);
--
1.7.9.5
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH 6/8] net/ipv6/fib6_rules.c: Checkpatch cleanup
2012-04-01 17:49 [PATCH 1/8] net/ipv6/sit.c: Checkpatch cleanup Eldad Zack
` (3 preceding siblings ...)
2012-04-01 17:49 ` [PATCH 5/8] net/ipv6/exthdrs_core.c: " Eldad Zack
@ 2012-04-01 17:49 ` Eldad Zack
2012-04-02 3:07 ` David Miller
2012-04-01 17:49 ` [PATCH 7/8] net/ipv6/icmp.c: Checkpatch cleanups Eldad Zack
` (2 subsequent siblings)
7 siblings, 1 reply; 16+ messages in thread
From: Eldad Zack @ 2012-04-01 17:49 UTC (permalink / raw)
To: David S. Miller, Alexey Kuznetsov, James Morris,
Hideaki YOSHIFUJI, Patrick McHardy
Cc: netdev, linux-kernel, Eldad Zack
fib6_rules.c:26: ERROR: open brace '{' following struct go on the same line
Signed-off-by: Eldad Zack <eldad@fogrefinery.com>
---
net/ipv6/fib6_rules.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/net/ipv6/fib6_rules.c b/net/ipv6/fib6_rules.c
index b6c5731..da75c67 100644
--- a/net/ipv6/fib6_rules.c
+++ b/net/ipv6/fib6_rules.c
@@ -22,8 +22,7 @@
#include <net/ip6_route.h>
#include <net/netlink.h>
-struct fib6_rule
-{
+struct fib6_rule {
struct fib_rule common;
struct rt6key src;
struct rt6key dst;
--
1.7.9.5
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH 7/8] net/ipv6/icmp.c: Checkpatch cleanups
2012-04-01 17:49 [PATCH 1/8] net/ipv6/sit.c: Checkpatch cleanup Eldad Zack
` (4 preceding siblings ...)
2012-04-01 17:49 ` [PATCH 6/8] net/ipv6/fib6_rules.c: Checkpatch cleanup Eldad Zack
@ 2012-04-01 17:49 ` Eldad Zack
2012-04-02 3:07 ` David Miller
2012-04-01 17:49 ` [PATCH 8/8] net/ipv6/addrconf.c: " Eldad Zack
2012-04-02 3:07 ` [PATCH 1/8] net/ipv6/sit.c: Checkpatch cleanup David Miller
7 siblings, 1 reply; 16+ messages in thread
From: Eldad Zack @ 2012-04-01 17:49 UTC (permalink / raw)
To: David S. Miller, Alexey Kuznetsov, James Morris,
Hideaki YOSHIFUJI, Patrick McHardy
Cc: netdev, linux-kernel, Eldad Zack
icmp.c:501: ERROR: "(foo*)" should be "(foo *)"
icmp.c:582: ERROR: "(foo*)" should be "(foo *)"
icmp.c:954: WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable
Signed-off-by: Eldad Zack <eldad@fogrefinery.com>
---
net/ipv6/icmp.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/net/ipv6/icmp.c b/net/ipv6/icmp.c
index 27ac95a..cc079d8 100644
--- a/net/ipv6/icmp.c
+++ b/net/ipv6/icmp.c
@@ -498,7 +498,7 @@ void icmpv6_send(struct sk_buff *skb, u8 type, u8 code, __u32 info)
err = ip6_append_data(sk, icmpv6_getfrag, &msg,
len + sizeof(struct icmp6hdr),
sizeof(struct icmp6hdr), hlimit,
- np->tclass, NULL, &fl6, (struct rt6_info*)dst,
+ np->tclass, NULL, &fl6, (struct rt6_info *)dst,
MSG_DONTWAIT, np->dontfrag);
if (err) {
ICMP6_INC_STATS_BH(net, idev, ICMP6_MIB_OUTERRORS);
@@ -579,7 +579,7 @@ static void icmpv6_echo_reply(struct sk_buff *skb)
err = ip6_append_data(sk, icmpv6_getfrag, &msg, skb->len + sizeof(struct icmp6hdr),
sizeof(struct icmp6hdr), hlimit, np->tclass, NULL, &fl6,
- (struct rt6_info*)dst, MSG_DONTWAIT,
+ (struct rt6_info *)dst, MSG_DONTWAIT,
np->dontfrag);
if (err) {
@@ -950,7 +950,6 @@ int icmpv6_err_convert(u8 type, u8 code, int *err)
return fatal;
}
-
EXPORT_SYMBOL(icmpv6_err_convert);
#ifdef CONFIG_SYSCTL
--
1.7.9.5
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH 8/8] net/ipv6/addrconf.c: Checkpatch cleanups
2012-04-01 17:49 [PATCH 1/8] net/ipv6/sit.c: Checkpatch cleanup Eldad Zack
` (5 preceding siblings ...)
2012-04-01 17:49 ` [PATCH 7/8] net/ipv6/icmp.c: Checkpatch cleanups Eldad Zack
@ 2012-04-01 17:49 ` Eldad Zack
2012-04-02 3:07 ` David Miller
2012-04-02 3:07 ` [PATCH 1/8] net/ipv6/sit.c: Checkpatch cleanup David Miller
7 siblings, 1 reply; 16+ messages in thread
From: Eldad Zack @ 2012-04-01 17:49 UTC (permalink / raw)
To: David S. Miller, Alexey Kuznetsov, James Morris,
Hideaki YOSHIFUJI, Patrick McHardy
Cc: netdev, linux-kernel, Eldad Zack
net/ipv6/addrconf.c:340: WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable
net/ipv6/addrconf.c:342: ERROR: "foo * bar" should be "foo *bar"
net/ipv6/addrconf.c:444: ERROR: "foo * bar" should be "foo *bar"
net/ipv6/addrconf.c:1337: WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable
net/ipv6/addrconf.c:1526: ERROR: "(foo*)" should be "(foo *)"
net/ipv6/addrconf.c:1671: ERROR: open brace '{' following function declarations go on the next line
net/ipv6/addrconf.c:1914: ERROR: "foo * bar" should be "foo *bar"
net/ipv6/addrconf.c:2368: ERROR: "foo * bar" should be "foo *bar"
net/ipv6/addrconf.c:2370: ERROR: "foo * bar" should be "foo *bar"
net/ipv6/addrconf.c:2416: ERROR: "foo * bar" should be "foo *bar"
net/ipv6/addrconf.c:2437: ERROR: "foo * bar" should be "foo *bar"
net/ipv6/addrconf.c:2573: ERROR: "foo * bar" should be "foo *bar"
net/ipv6/addrconf.c:3797: ERROR: "foo* bar" should be "foo *bar"
Signed-off-by: Eldad Zack <eldad@fogrefinery.com>
---
net/ipv6/addrconf.c | 25 ++++++++++++-------------
1 file changed, 12 insertions(+), 13 deletions(-)
diff --git a/net/ipv6/addrconf.c b/net/ipv6/addrconf.c
index 6a3bb60..0031f05 100644
--- a/net/ipv6/addrconf.c
+++ b/net/ipv6/addrconf.c
@@ -336,10 +336,9 @@ void in6_dev_finish_destroy(struct inet6_dev *idev)
snmp6_free_dev(idev);
kfree_rcu(idev, rcu);
}
-
EXPORT_SYMBOL(in6_dev_finish_destroy);
-static struct inet6_dev * ipv6_add_dev(struct net_device *dev)
+static struct inet6_dev *ipv6_add_dev(struct net_device *dev)
{
struct inet6_dev *ndev;
@@ -441,7 +440,7 @@ static struct inet6_dev * ipv6_add_dev(struct net_device *dev)
return ndev;
}
-static struct inet6_dev * ipv6_find_idev(struct net_device *dev)
+static struct inet6_dev *ipv6_find_idev(struct net_device *dev)
{
struct inet6_dev *idev;
@@ -1333,7 +1332,6 @@ int ipv6_chk_prefix(const struct in6_addr *addr, struct net_device *dev)
rcu_read_unlock();
return onlink;
}
-
EXPORT_SYMBOL(ipv6_chk_prefix);
struct inet6_ifaddr *ipv6_get_ifaddr(struct net *net, const struct in6_addr *addr,
@@ -1523,7 +1521,7 @@ static int addrconf_ifid_arcnet(u8 *eui, struct net_device *dev)
if (dev->addr_len != ARCNET_ALEN)
return -1;
memset(eui, 0, 7);
- eui[7] = *(u8*)dev->dev_addr;
+ eui[7] = *(u8 *)dev->dev_addr;
return 0;
}
@@ -1668,7 +1666,8 @@ out:
in6_dev_put(idev);
}
-static int __ipv6_try_regen_rndid(struct inet6_dev *idev, struct in6_addr *tmpaddr) {
+static int __ipv6_try_regen_rndid(struct inet6_dev *idev, struct in6_addr *tmpaddr)
+{
int ret = 0;
if (tmpaddr && memcmp(idev->rndid, &tmpaddr->s6_addr[8], 8) == 0)
@@ -1911,7 +1910,7 @@ void addrconf_prefix_rcv(struct net_device *dev, u8 *opt, int len, bool sllao)
/* Try to figure out our local address for this prefix */
if (pinfo->autoconf && in6_dev->cnf.autoconf) {
- struct inet6_ifaddr * ifp;
+ struct inet6_ifaddr *ifp;
struct in6_addr addr;
int create = 0, update_lft = 0;
@@ -2365,9 +2364,9 @@ static void sit_add_v4_addrs(struct inet6_dev *idev)
}
for_each_netdev(net, dev) {
- struct in_device * in_dev = __in_dev_get_rtnl(dev);
+ struct in_device *in_dev = __in_dev_get_rtnl(dev);
if (in_dev && (dev->flags & IFF_UP)) {
- struct in_ifaddr * ifa;
+ struct in_ifaddr *ifa;
int flag = scope;
@@ -2413,7 +2412,7 @@ static void init_loopback(struct net_device *dev)
static void addrconf_add_linklocal(struct inet6_dev *idev, const struct in6_addr *addr)
{
- struct inet6_ifaddr * ifp;
+ struct inet6_ifaddr *ifp;
u32 addr_flags = IFA_F_PERMANENT;
#ifdef CONFIG_IPV6_OPTIMISTIC_DAD
@@ -2434,7 +2433,7 @@ static void addrconf_add_linklocal(struct inet6_dev *idev, const struct in6_addr
static void addrconf_dev_config(struct net_device *dev)
{
struct in6_addr addr;
- struct inet6_dev * idev;
+ struct inet6_dev *idev;
ASSERT_RTNL();
@@ -2570,7 +2569,7 @@ static void addrconf_ip6_tnl_config(struct net_device *dev)
}
static int addrconf_notify(struct notifier_block *this, unsigned long event,
- void * data)
+ void *data)
{
struct net_device *dev = (struct net_device *) data;
struct inet6_dev *idev = __in6_dev_get(dev);
@@ -3794,7 +3793,7 @@ static int inet6_dump_ifacaddr(struct sk_buff *skb, struct netlink_callback *cb)
return inet6_dump_addr(skb, cb, type);
}
-static int inet6_rtm_getaddr(struct sk_buff *in_skb, struct nlmsghdr* nlh,
+static int inet6_rtm_getaddr(struct sk_buff *in_skb, struct nlmsghdr *nlh,
void *arg)
{
struct net *net = sock_net(in_skb->sk);
--
1.7.9.5
^ permalink raw reply related [flat|nested] 16+ messages in thread
* Re: [PATCH 1/8] net/ipv6/sit.c: Checkpatch cleanup
2012-04-01 17:49 [PATCH 1/8] net/ipv6/sit.c: Checkpatch cleanup Eldad Zack
` (6 preceding siblings ...)
2012-04-01 17:49 ` [PATCH 8/8] net/ipv6/addrconf.c: " Eldad Zack
@ 2012-04-02 3:07 ` David Miller
7 siblings, 0 replies; 16+ messages in thread
From: David Miller @ 2012-04-02 3:07 UTC (permalink / raw)
To: eldad; +Cc: kuznet, jmorris, yoshfuji, kaber, netdev, linux-kernel
From: Eldad Zack <eldad@fogrefinery.com>
Date: Sun, 1 Apr 2012 19:49:01 +0200
> sit.c:118: ERROR: "foo * bar" should be "foo *bar"
> sit.c:694: ERROR: "(foo*)" should be "(foo *)"
> sit.c:724: ERROR: "(foo*)" should be "(foo *)"
>
> Signed-off-by: Eldad Zack <eldad@fogrefinery.com>
Applied.
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH 2/8] net/ipv6/addrconf_core.c: Checkpatch cleanup
2012-04-01 17:49 ` [PATCH 2/8] net/ipv6/addrconf_core.c: " Eldad Zack
@ 2012-04-02 3:07 ` David Miller
0 siblings, 0 replies; 16+ messages in thread
From: David Miller @ 2012-04-02 3:07 UTC (permalink / raw)
To: eldad; +Cc: kuznet, jmorris, yoshfuji, kaber, netdev, linux-kernel
From: Eldad Zack <eldad@fogrefinery.com>
Date: Sun, 1 Apr 2012 19:49:02 +0200
> addrconf_core.c:13: ERROR: space required before the open parenthesis '('
>
> Signed-off-by: Eldad Zack <eldad@fogrefinery.com>
Applied.
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH 3/8] net/ipv6/datagram.c: Checkpatch cleanups
2012-04-01 17:49 ` [PATCH 3/8] net/ipv6/datagram.c: Checkpatch cleanups Eldad Zack
@ 2012-04-02 3:07 ` David Miller
0 siblings, 0 replies; 16+ messages in thread
From: David Miller @ 2012-04-02 3:07 UTC (permalink / raw)
To: eldad; +Cc: kuznet, jmorris, yoshfuji, kaber, netdev, linux-kernel
From: Eldad Zack <eldad@fogrefinery.com>
Date: Sun, 1 Apr 2012 19:49:03 +0200
> datagram.c:101: ERROR: "(foo*)" should be "(foo *)"
> datagram.c:521: ERROR: space required before the open parenthesis '('
> datagram.c:830: WARNING: braces {} are not necessary for single statement blocks
> datagram.c:849: WARNING: braces {} are not necessary for single statement blocks
>
> Signed-off-by: Eldad Zack <eldad@fogrefinery.com>
Applied.
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH 4/8] net/ipv6/exthdrs.c: Checkpatch cleanups
2012-04-01 17:49 ` [PATCH 4/8] net/ipv6/exthdrs.c: " Eldad Zack
@ 2012-04-02 3:07 ` David Miller
0 siblings, 0 replies; 16+ messages in thread
From: David Miller @ 2012-04-02 3:07 UTC (permalink / raw)
To: eldad; +Cc: kuznet, jmorris, yoshfuji, kaber, netdev, linux-kernel
From: Eldad Zack <eldad@fogrefinery.com>
Date: Sun, 1 Apr 2012 19:49:04 +0200
> exthdrs.c:726: WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable
> exthdrs.c:741: ERROR: "(foo*)" should be "(foo *)"
> exthdrs.c:741: ERROR: "(foo*)" should be "(foo *)"
> exthdrs.c:744: ERROR: "(foo**)" should be "(foo **)"
> exthdrs.c:746: ERROR: "(foo**)" should be "(foo **)"
> exthdrs.c:748: ERROR: "(foo**)" should be "(foo **)"
> exthdrs.c:750: ERROR: "(foo**)" should be "(foo **)"
> exthdrs.c:755: WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable
> exthdrs.c:896: WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable
>
> Signed-off-by: Eldad Zack <eldad@fogrefinery.com>
Applied.
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH 5/8] net/ipv6/exthdrs_core.c: Checkpatch cleanups
2012-04-01 17:49 ` [PATCH 5/8] net/ipv6/exthdrs_core.c: " Eldad Zack
@ 2012-04-02 3:07 ` David Miller
0 siblings, 0 replies; 16+ messages in thread
From: David Miller @ 2012-04-02 3:07 UTC (permalink / raw)
To: eldad; +Cc: kuznet, jmorris, yoshfuji, kaber, netdev, linux-kernel
From: Eldad Zack <eldad@fogrefinery.com>
Date: Sun, 1 Apr 2012 19:49:05 +0200
> exthdrs_core.c:113: WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable
> exthdrs_core.c:114: WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable
>
> Signed-off-by: Eldad Zack <eldad@fogrefinery.com>
Applied.
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH 6/8] net/ipv6/fib6_rules.c: Checkpatch cleanup
2012-04-01 17:49 ` [PATCH 6/8] net/ipv6/fib6_rules.c: Checkpatch cleanup Eldad Zack
@ 2012-04-02 3:07 ` David Miller
0 siblings, 0 replies; 16+ messages in thread
From: David Miller @ 2012-04-02 3:07 UTC (permalink / raw)
To: eldad; +Cc: kuznet, jmorris, yoshfuji, kaber, netdev, linux-kernel
From: Eldad Zack <eldad@fogrefinery.com>
Date: Sun, 1 Apr 2012 19:49:06 +0200
> fib6_rules.c:26: ERROR: open brace '{' following struct go on the same line
>
> Signed-off-by: Eldad Zack <eldad@fogrefinery.com>
Applied.
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH 7/8] net/ipv6/icmp.c: Checkpatch cleanups
2012-04-01 17:49 ` [PATCH 7/8] net/ipv6/icmp.c: Checkpatch cleanups Eldad Zack
@ 2012-04-02 3:07 ` David Miller
0 siblings, 0 replies; 16+ messages in thread
From: David Miller @ 2012-04-02 3:07 UTC (permalink / raw)
To: eldad; +Cc: kuznet, jmorris, yoshfuji, kaber, netdev, linux-kernel
From: Eldad Zack <eldad@fogrefinery.com>
Date: Sun, 1 Apr 2012 19:49:07 +0200
> icmp.c:501: ERROR: "(foo*)" should be "(foo *)"
> icmp.c:582: ERROR: "(foo*)" should be "(foo *)"
> icmp.c:954: WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable
>
> Signed-off-by: Eldad Zack <eldad@fogrefinery.com>
Applied.
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH 8/8] net/ipv6/addrconf.c: Checkpatch cleanups
2012-04-01 17:49 ` [PATCH 8/8] net/ipv6/addrconf.c: " Eldad Zack
@ 2012-04-02 3:07 ` David Miller
0 siblings, 0 replies; 16+ messages in thread
From: David Miller @ 2012-04-02 3:07 UTC (permalink / raw)
To: eldad; +Cc: kuznet, jmorris, yoshfuji, kaber, netdev, linux-kernel
From: Eldad Zack <eldad@fogrefinery.com>
Date: Sun, 1 Apr 2012 19:49:08 +0200
> net/ipv6/addrconf.c:340: WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable
> net/ipv6/addrconf.c:342: ERROR: "foo * bar" should be "foo *bar"
> net/ipv6/addrconf.c:444: ERROR: "foo * bar" should be "foo *bar"
> net/ipv6/addrconf.c:1337: WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable
> net/ipv6/addrconf.c:1526: ERROR: "(foo*)" should be "(foo *)"
> net/ipv6/addrconf.c:1671: ERROR: open brace '{' following function declarations go on the next line
> net/ipv6/addrconf.c:1914: ERROR: "foo * bar" should be "foo *bar"
> net/ipv6/addrconf.c:2368: ERROR: "foo * bar" should be "foo *bar"
> net/ipv6/addrconf.c:2370: ERROR: "foo * bar" should be "foo *bar"
> net/ipv6/addrconf.c:2416: ERROR: "foo * bar" should be "foo *bar"
> net/ipv6/addrconf.c:2437: ERROR: "foo * bar" should be "foo *bar"
> net/ipv6/addrconf.c:2573: ERROR: "foo * bar" should be "foo *bar"
> net/ipv6/addrconf.c:3797: ERROR: "foo* bar" should be "foo *bar"
>
> Signed-off-by: Eldad Zack <eldad@fogrefinery.com>
Applied.
^ permalink raw reply [flat|nested] 16+ messages in thread
end of thread, other threads:[~2012-04-02 3:10 UTC | newest]
Thread overview: 16+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-04-01 17:49 [PATCH 1/8] net/ipv6/sit.c: Checkpatch cleanup Eldad Zack
2012-04-01 17:49 ` [PATCH 2/8] net/ipv6/addrconf_core.c: " Eldad Zack
2012-04-02 3:07 ` David Miller
2012-04-01 17:49 ` [PATCH 3/8] net/ipv6/datagram.c: Checkpatch cleanups Eldad Zack
2012-04-02 3:07 ` David Miller
2012-04-01 17:49 ` [PATCH 4/8] net/ipv6/exthdrs.c: " Eldad Zack
2012-04-02 3:07 ` David Miller
2012-04-01 17:49 ` [PATCH 5/8] net/ipv6/exthdrs_core.c: " Eldad Zack
2012-04-02 3:07 ` David Miller
2012-04-01 17:49 ` [PATCH 6/8] net/ipv6/fib6_rules.c: Checkpatch cleanup Eldad Zack
2012-04-02 3:07 ` David Miller
2012-04-01 17:49 ` [PATCH 7/8] net/ipv6/icmp.c: Checkpatch cleanups Eldad Zack
2012-04-02 3:07 ` David Miller
2012-04-01 17:49 ` [PATCH 8/8] net/ipv6/addrconf.c: " Eldad Zack
2012-04-02 3:07 ` David Miller
2012-04-02 3:07 ` [PATCH 1/8] net/ipv6/sit.c: Checkpatch cleanup David Miller
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).