From: Linus Lüssing <linus.luess...@c0d3.blue>
Date: Fri, 22 May 2015 15:32:37 +0200

> @@ -1786,14 +1784,8 @@ int br_multicast_set_router(struct net_bridge *br, 
> unsigned long val)
>       case 1:
>               br->multicast_router = val;
>               err = 0;
> -             break;
> -
> -     default:
> -             err = -EINVAL;
> -             break;
>       }

Please don't remove the break; statement.

> @@ -1827,14 +1817,8 @@ int br_multicast_set_port_router(struct 
> net_bridge_port *p, unsigned long val)
>                       break;
>  
>               br_multicast_add_router(br, p);
> -             break;
> -
> -     default:
> -             err = -EINVAL;
> -             break;
>       }
>  
> -unlock:
>       spin_unlock(&br->multicast_lock);
>  
>       return err;

Likewise.

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