mac802154/iface: remove superfluous WARN_ON call in slave_open()
authorStefan Schmidt <stefan@osg.samsung.com>
Mon, 8 Jun 2015 20:06:40 +0000 (22:06 +0200)
committerMarcel Holtmann <marcel@holtmann.org>
Tue, 9 Jun 2015 07:44:23 +0000 (09:44 +0200)
This call was used before we aligned our code with the wireless code base. We
are wanted to handle this in the err: code path. Which would actually not work
because the WARN_ON() macro would reset the res value to 0 and thus we would
never hit err:. Removing it makes the code do what we actually intend.

Signed-off-by: Stefan Schmidt <stefan@osg.samsung.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
net/mac802154/iface.c

index 3a67d35d46722b9b9179be7df547f8351b3e46ca..e3d77b07c0e4a4924be35b803dcaf6981ec5f789 100644 (file)
@@ -147,7 +147,6 @@ static int mac802154_slave_open(struct net_device *dev)
 
        if (!local->open_count) {
                res = drv_start(local);
-               WARN_ON(res);
                if (res)
                        goto err;
        }