[B.A.T.M.A.N.] [PATCHv2] batman-adv: (style) fix for switched vid-ifiindex parameter order

[B.A.T.M.A.N.] [PATCHv2] batman-adv: (style) fix for switched vid-ifiindex parameter order

[Linus Lüssing]

batadv_tt_local_add() first takes the vid and then the interface index
parameter. One of these calls had the order switched.

However in this case it is not a functional, but just a style fix
because at the moment BATADV_NO_FLAGS is equal to BATADV_NULL_IFINDEX.

Introduced by 580d7919f19e4f73ccd79e82a532af2bdc638042
("batman-adv: add the VLAN ID attribute to the TT entry")

Acked-by: Antonio Quartulli <ordex@autistici.org>
Signed-off-by: Linus Lüssing <linus.luessing@web.de>
---
v2: Added "Introduced by..." to commit message

 main.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/main.c b/main.c
index f2f1137..a7ad6cb 100644
--- a/main.c
+++ b/main.c
@@ -128,7 +128,7 @@ int batadv_mesh_init(struct net_device *soft_iface)
 		goto err;
 
 	batadv_tt_local_add(soft_iface, soft_iface->dev_addr,
-			    BATADV_NULL_IFINDEX, BATADV_NO_FLAGS);
+			    BATADV_NO_FLAGS, BATADV_NULL_IFINDEX);
 
 	ret = batadv_bla_init(bat_priv);
 	if (ret < 0)
-- 
1.7.10.4

Re: [B.A.T.M.A.N.] [PATCHv2] batman-adv: (style) fix for switched vid-ifiindex parameter order

[Marek Lindner]

On Saturday, June 15, 2013 02:04:09 Linus Lüssing wrote:
> batadv_tt_local_add() first takes the vid and then the interface index
> parameter. One of these calls had the order switched.
> 
> However in this case it is not a functional, but just a style fix
> because at the moment BATADV_NO_FLAGS is equal to BATADV_NULL_IFINDEX.
> 
> Introduced by 580d7919f19e4f73ccd79e82a532af2bdc638042
> ("batman-adv: add the VLAN ID attribute to the TT entry")
> 
> Acked-by: Antonio Quartulli <ordex@autistici.org>
> Signed-off-by: Linus Lüssing <linus.luessing@web.de>
> ---
> v2: Added "Introduced by..." to commit message
> 
>  main.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Applied in revision cc14598.

Thanks,
Marek