On Dienstag, 20. September 2016 14:12:45 CEST Linus Lüssing wrote:
> @@ -1435,6 +1448,10 @@ struct batadv_algo_iface_ops {
>   */
>  struct batadv_algo_neigh_ops {
>         void (*hardif_init)(struct batadv_hardif_neigh_node *neigh);
> +       bool (*hardif_no_broadcast)
> +               (struct batadv_hard_iface *if_outgoing,
> +                struct batadv_hardif_neigh_node *hardif_neigh,
> +                bool inverse_metric);
>         int (*cmp)(struct batadv_neigh_node *neigh1,
>                    struct batadv_hard_iface *if_outgoing1,
>                    struct batadv_neigh_node *neigh2,

Can you shorten hardif_neigh a little bit to avoid this slightly confusing
line break after "hardif_no_broadcast)"? Many tools like checkpatch or
kernel-doc go crazy about it :)

Thanks,
	Sven