Hey,

I was about to merge the patch when I realized checkpatch is complaining. See 
below for details.


> @@ -618,7 +615,7 @@ int recv_tt_query(struct sk_buff *skb, struct
> hard_iface *recv_if) if (skb_linearize(skb) < 0)
>                               goto out;
> 
> -                     tt_len = tt_query->tt_data * sizeof(struct tt_change);
> +                     tt_len = ntohs(tt_query->tt_data) * sizeof(struct 
> tt_change);

Line is too long.


> @@ -1727,7 +1727,7 @@ void handle_tt_response(struct bat_priv *bat_priv,
> 
>       bat_dbg(DBG_TT, bat_priv,
>               "Received TT_RESPONSE from %pM for ttvn %d t_size: %d [%c]\n",
> -             tt_response->src, tt_response->ttvn, tt_response->tt_data,
> +             tt_response->src, tt_response->ttvn, 
> ntohs(tt_response->tt_data),
>               (tt_response->flags & TT_FULL_TABLE ? 'F' : '.'));
> 
>       /* we should have never asked a backbone gw */
> @@ -1741,7 +1741,7 @@ void handle_tt_response(struct bat_priv *bat_priv,
>       if (tt_response->flags & TT_FULL_TABLE)
>               tt_fill_gtable(bat_priv, tt_response);
>       else
> -             tt_update_changes(bat_priv, orig_node, tt_response->tt_data,
> +             tt_update_changes(bat_priv, orig_node, 
> ntohs(tt_response->tt_data),
>                                 tt_response->ttvn,
>                                 (struct tt_change *)(tt_response + 1));

Both changes are too long as well.

Regards,
Marek

Reply via email to