From: Thomas F Herbert <thomasfherb...@gmail.com>
Date: Sun, 26 Jul 2015 10:52:34 -0400

> Signed-off-by: Thomas F Herbert <thomasfherb...@gmail.com>
> ---
>  include/linux/if_vlan.h | 17 +++++++++++++++++
>  1 file changed, 17 insertions(+)
> 
> diff --git a/include/linux/if_vlan.h b/include/linux/if_vlan.h
> index 67ce5bd..88d1be4 100644
> --- a/include/linux/if_vlan.h
> +++ b/include/linux/if_vlan.h
> @@ -627,6 +627,23 @@ static inline netdev_features_t 
> vlan_features_check(const struct sk_buff *skb,
>  
>       return features;
>  }
> +/**
> + * eth_type_vlan - check for valid vlan ether type.
> + * @ethertype: ether type to check
> + *
> + * Returns true if the ether type is a vlan ether type.
> + */
> +static inline bool eth_type_vlan(__be16 ethertype)
> +{
> +     switch (ethertype) {
> +     case (htons(ETH_P_8021Q)):
> +             return true;
> +     case (htons(ETH_P_8021AD)):
> +             return true;

Please don't put unnecessary parenthesis around case statement values.

Thanks.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to