[B.A.T.M.A.N.] [PATCHv2 next 1/2] batman-adv: use VLAN_VID_MASK when printing the vid

[B.A.T.M.A.N.] [PATCHv2 next 1/2] batman-adv: use VLAN_VID_MASK when printing the vid

[Antonio Quartulli]

Introduced in 454b849c7de18a7b44eecebe889f5f3149d24780
("batman-adv: print the VID properly")

Signed-off-by: Antonio Quartulli <ordex@autistici.org>
---
 main.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/main.h b/main.h
index d575436..8bcffa5 100644
--- a/main.h
+++ b/main.h
@@ -175,7 +175,7 @@ enum batadv_vlan_flags {
 /* mask needed to extract the vlan ID (12bits) from a 16bits variable */
 #define BATADV_VID_MASK	0x0FFF
 #define BATADV_PRINT_VID(vid) (vid & BATADV_VLAN_HAS_TAG ? \
-			       (int)(vid & BATADV_VID_MASK) : -1)
+			       (int)(vid & VLAN_VID_MASK) : -1)
 
 extern char batadv_routing_algo[];
 extern struct list_head batadv_hardif_list;
-- 
1.8.1.5

[B.A.T.M.A.N.] [PATCHv2 next 2/2] batman-adv: use VLAN_VID_MASK instead of BATADV_VID_MASK

[Antonio Quartulli]

VLAN_VID_MASK is already diefined in linux/if_vlan.h and
therefore does not need to be privately re-defined

Introduced in 0b1da1765fdb00ca5d53bc95c9abc70dfc9aae5b
("batman-adv: change VID semantic in the BLA code")

Signed-off-by: Antonio Quartulli <ordex@autistici.org>
---
 bridge_loop_avoidance.c | 2 +-
 main.h                  | 2 --
 2 files changed, 1 insertion(+), 3 deletions(-)

diff --git a/bridge_loop_avoidance.c b/bridge_loop_avoidance.c
index e95cf4b..e9d8e0b 100644
--- a/bridge_loop_avoidance.c
+++ b/bridge_loop_avoidance.c
@@ -344,7 +344,7 @@ static void batadv_bla_send_claim(struct batadv_priv *bat_priv, uint8_t *mac,
 
 	if (vid & BATADV_VLAN_HAS_TAG)
 		skb = vlan_insert_tag(skb, htons(ETH_P_8021Q),
-				      vid & BATADV_VID_MASK);
+				      vid & VLAN_VID_MASK);
 
 	skb_reset_mac_header(skb);
 	skb->protocol = eth_type_trans(skb, soft_iface);
diff --git a/main.h b/main.h
index 8bcffa5..834b1a0 100644
--- a/main.h
+++ b/main.h
@@ -172,8 +172,6 @@ enum batadv_vlan_flags {
 	BATADV_VLAN_HAS_TAG	= BIT(15),
 };
 
-/* mask needed to extract the vlan ID (12bits) from a 16bits variable */
-#define BATADV_VID_MASK	0x0FFF
 #define BATADV_PRINT_VID(vid) (vid & BATADV_VLAN_HAS_TAG ? \
 			       (int)(vid & VLAN_VID_MASK) : -1)
 
-- 
1.8.1.5

Re: [B.A.T.M.A.N.] [PATCHv2 next 2/2] batman-adv: use VLAN_VID_MASK instead of BATADV_VID_MASK

[Marek Lindner]

On Friday, May 10, 2013 00:48:17 Antonio Quartulli wrote:
> VLAN_VID_MASK is already diefined in linux/if_vlan.h and
> therefore does not need to be privately re-defined
> 
> Introduced in 0b1da1765fdb00ca5d53bc95c9abc70dfc9aae5b
> ("batman-adv: change VID semantic in the BLA code")
> 
> Signed-off-by: Antonio Quartulli <ordex@autistici.org>
> ---
>  bridge_loop_avoidance.c | 2 +-
>  main.h                  | 2 --
>  2 files changed, 1 insertion(+), 3 deletions(-)

Applied in revision 64fc7ed.

Thanks,
Marek

Re: [B.A.T.M.A.N.] [PATCHv2 next 1/2] batman-adv: use VLAN_VID_MASK when printing the vid

[Marek Lindner]

On Friday, May 10, 2013 00:48:16 Antonio Quartulli wrote:
> Introduced in 454b849c7de18a7b44eecebe889f5f3149d24780
> ("batman-adv: print the VID properly")
> 
> Signed-off-by: Antonio Quartulli <ordex@autistici.org>
> ---
>  main.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Applied in revision 78c836c.

Thanks,
Marek