[LIBERTAS] Don't set NETIF_F_IPV6_CSUM in dev->features

Dan Williams dcbw at redhat.com
Mon Nov 26 09:59:57 EST 2007


On Sat, 2007-11-24 at 19:48 +0000, David Woodhouse wrote:
> I'm not sure why it was doing this, and I'm not sure I _want_ to know
> why. But calling it NETIF_F_DYNALLOC doesn't change the fact that the
> kernel believes it to be NETIF_F_IPV6_CSUM, and that IPv6 communication
> is hence buggered.
> 
> Signed-off-by: David Woodhouse <dwmw2 at infradead.org>

Yeah, really odd.

Acked-by: Dan Williams <dcbw at redhat.com>

> diff --git a/drivers/net/wireless/libertas/main.c b/drivers/net/wireless/libertas/main.c
> index 0a4f202..825970a 100644
> --- a/drivers/net/wireless/libertas/main.c
> +++ b/drivers/net/wireless/libertas/main.c
> @@ -1165,8 +1165,6 @@ lbs_private *lbs_add_card(void *card, struct device *dmdev)
>  #ifdef	WIRELESS_EXT
>  	dev->wireless_handlers = (struct iw_handler_def *)&lbs_handler_def;
>  #endif
> -#define NETIF_F_DYNALLOC 16
> -	dev->features |= NETIF_F_DYNALLOC;
>  	dev->flags |= IFF_BROADCAST | IFF_MULTICAST;
>  	dev->set_multicast_list = lbs_set_multicast_list;
>  
> @@ -1348,8 +1346,6 @@ int lbs_add_mesh(lbs_private *priv, struct device *dev)
>  #ifdef	WIRELESS_EXT
>  	mesh_dev->wireless_handlers = (struct iw_handler_def *)&mesh_handler_def;
>  #endif
> -#define NETIF_F_DYNALLOC 16
> -
>  	/* Register virtual mesh interface */
>  	ret = register_netdev(mesh_dev);
>  	if (ret) {
> 




More information about the libertas-dev mailing list