Commit 5c7f0333 authored by Harvey Harrison's avatar Harvey Harrison Committed by David S. Miller

phonet: sparse annotations of protocol, remove forward declaration

net/phonet/af_phonet.c:38:36: error: marked inline, but without a definition
net/phonet/pep-gprs.c:63:10: warning: incorrect type in return expression (different base types)
net/phonet/pep-gprs.c:63:10:    expected int
net/phonet/pep-gprs.c:63:10:    got restricted __be16 [usertype] <noident>
net/phonet/pep-gprs.c:65:10: warning: incorrect type in return expression (different base types)
net/phonet/pep-gprs.c:65:10:    expected int
net/phonet/pep-gprs.c:65:10:    got restricted __be16 [usertype] <noident>
net/phonet/pep-gprs.c:124:16: warning: incorrect type in assignment (different base types)
net/phonet/pep-gprs.c:124:16:    expected restricted __be16 [usertype] protocol
net/phonet/pep-gprs.c:124:16:    got unsigned short [unsigned] [usertype] protocol
Signed-off-by: default avatarHarvey Harrison <harvey.harrison@gmail.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent ca62059b
...@@ -35,7 +35,11 @@ ...@@ -35,7 +35,11 @@
static struct net_proto_family phonet_proto_family; static struct net_proto_family phonet_proto_family;
static struct phonet_protocol *phonet_proto_get(int protocol); static struct phonet_protocol *phonet_proto_get(int protocol);
static inline void phonet_proto_put(struct phonet_protocol *pp);
static inline void phonet_proto_put(struct phonet_protocol *pp)
{
module_put(pp->prot->owner);
}
/* protocol family functions */ /* protocol family functions */
...@@ -428,11 +432,6 @@ static struct phonet_protocol *phonet_proto_get(int protocol) ...@@ -428,11 +432,6 @@ static struct phonet_protocol *phonet_proto_get(int protocol)
return pp; return pp;
} }
static inline void phonet_proto_put(struct phonet_protocol *pp)
{
module_put(pp->prot->owner);
}
/* Module registration */ /* Module registration */
static int __init phonet_init(void) static int __init phonet_init(void)
{ {
......
...@@ -49,14 +49,14 @@ struct gprs_dev { ...@@ -49,14 +49,14 @@ struct gprs_dev {
unsigned tx_max; unsigned tx_max;
}; };
static int gprs_type_trans(struct sk_buff *skb) static __be16 gprs_type_trans(struct sk_buff *skb)
{ {
const u8 *pvfc; const u8 *pvfc;
u8 buf; u8 buf;
pvfc = skb_header_pointer(skb, 0, 1, &buf); pvfc = skb_header_pointer(skb, 0, 1, &buf);
if (!pvfc) if (!pvfc)
return 0; return htons(0);
/* Look at IP version field */ /* Look at IP version field */
switch (*pvfc >> 4) { switch (*pvfc >> 4) {
case 4: case 4:
...@@ -64,7 +64,7 @@ static int gprs_type_trans(struct sk_buff *skb) ...@@ -64,7 +64,7 @@ static int gprs_type_trans(struct sk_buff *skb)
case 6: case 6:
return htons(ETH_P_IPV6); return htons(ETH_P_IPV6);
} }
return 0; return htons(0);
} }
/* /*
...@@ -84,7 +84,7 @@ static void gprs_state_change(struct sock *sk) ...@@ -84,7 +84,7 @@ static void gprs_state_change(struct sock *sk)
static int gprs_recv(struct gprs_dev *dev, struct sk_buff *skb) static int gprs_recv(struct gprs_dev *dev, struct sk_buff *skb)
{ {
int err = 0; int err = 0;
u16 protocol = gprs_type_trans(skb); __be16 protocol = gprs_type_trans(skb);
if (!protocol) { if (!protocol) {
err = -EINVAL; err = -EINVAL;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment