In article <[EMAIL PROTECTED]> (at Thu, 17 May 2007 18:53:03 +0200), Oliver 
Hartkopp <[EMAIL PROTECTED]> says:

> +static int ipv6_hwtype(struct net_device *dev)
> +{
> +    if ((dev->type == ARPHRD_ETHER) ||
> +        (dev->type == ARPHRD_LOOPBACK) ||
> +#if defined(CONFIG_IPV6_SIT) || defined(CONFIG_IPV6_SIT_MODULE)
> +        (dev->type == ARPHRD_SIT) ||
> +#endif
> +        (dev->type == ARPHRD_TUNNEL6) ||
> +        (dev->type == ARPHRD_FDDI) ||
> +        (dev->type == ARPHRD_IEEE802_TR) ||
> +        (dev->type == ARPHRD_ARCNET) ||
> +        (dev->type == ARPHRD_INFINIBAND))
> +        return 1;
> +
> +    return 0;
> +}
> +

Please remove #if, or please provide for other
dependencies as well (e.g., CONFIG_IPV6_TUNNEL etc.)

Otherwise, I would agree.

-- 
YOSHIFUJI Hideaki @ USAGI Project  <[EMAIL PROTECTED]>
GPG-FP  : 9022 65EB 1ECF 3AD1 0BDF  80D8 4807 F894 E062 0EEA
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to