On Sunday, April 21, 2013 00:17:32 Martin Hundebøll wrote:
@@ -592,9 +591,9 @@ static bool batadv_dat_send_data(struct batadv_priv *bat_priv, goto free_orig;
tmp_skb = pskb_copy(skb, GFP_ATOMIC);
if (!batadv_unicast_4addr_prepare_skb(bat_priv, tmp_skb,cand[i].orig_node,packet_subtype)) {
if (!batadv_send_skb_4addr_prepare_unicast(bat_priv, tmp_skb,cand[i].orig_node,packet_subtype)) {
I think you mangled the "unicast_4addr" name a little. Afaik "4addr" was meant to follow "unicast" to diffirentiate between "unicast" and "unicast_4addr". However, I'd have no problem if you simply dropped the "unicast" part. For example: batadv_send_skb_4addr_prepare()
@@ -977,10 +976,10 @@ bool batadv_dat_snoop_incoming_arp_request(struct batadv_priv *bat_priv, * that a node not using the 4addr packet format doesn't support it. */ if (hdr_size == sizeof(struct batadv_unicast_4addr_packet))
err = batadv_unicast_4addr_send_skb(bat_priv, skb_new,
err = batadv_send_skb_4addr_unicast(bat_priv, skb_new, BATADV_P_DAT_CACHE_REPLY);
If you decide to skip the "unicast" part above feel free to drop it here too.
+/**
- batadv_send_generic_unicast_skb - send an skb as unicast
- @bat_priv: the bat priv with all the soft interface information
- @skb: payload to send
- @packet_type: the batman unicast packet type to use
- @packet_subtype: the batman packet subtype. It is ignored if packet_type
not BATADV_UNICAT_4ADDR
- Returns 1 in case of error or 0 otherwise
- */
Multi-line kernel doc explanations should be indented by one space. If the function returns 0 or 1 only it should be made boolean or the return value should use a readable define.
Cheers, Marek