On 2014/2/12 15:45, Denis Kirjanov wrote: > remove useless if check from register_netdevice() > > Signed-off-by: Denis Kirjanov <kda@xxxxxxxxxxxxxxxxx> > --- > net/core/dev.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/net/core/dev.c b/net/core/dev.c > index 4ad1b78..4be9a37 100644 > --- a/net/core/dev.c > +++ b/net/core/dev.c > @@ -5876,8 +5876,7 @@ int register_netdevice(struct net_device *dev) > if (dev->netdev_ops->ndo_init) { > ret = dev->netdev_ops->ndo_init(dev); > if (ret) { > - if (ret > 0) > - ret = -EIO; > + ret = -EIO; pls use checkpatch.pl to check the code. Ding > goto out; > } > } > -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html