summaryrefslogtreecommitdiffstats
path: root/net/mac802154
diff options
context:
space:
mode:
authorStefan Schmidt <stefan@osg.samsung.com>2015-06-08 22:06:40 +0200
committerMarcel Holtmann <marcel@holtmann.org>2015-06-09 09:44:23 +0200
commit9a4d3d4ba17c93def2b4dc3126eba30716d15469 (patch)
treee95ba5f010feb0a861a2f99b7e3c686849fbdcde /net/mac802154
parentBluetooth: btusb: Fix secure send command length alignment on Intel 8260 (diff)
downloadlinux-9a4d3d4ba17c93def2b4dc3126eba30716d15469.tar.xz
linux-9a4d3d4ba17c93def2b4dc3126eba30716d15469.zip
mac802154/iface: remove superfluous WARN_ON call in slave_open()
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>
Diffstat (limited to 'net/mac802154')
-rw-r--r--net/mac802154/iface.c1
1 files changed, 0 insertions, 1 deletions
diff --git a/net/mac802154/iface.c b/net/mac802154/iface.c
index 3a67d35d4672..e3d77b07c0e4 100644
--- a/net/mac802154/iface.c
+++ b/net/mac802154/iface.c
@@ -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;
}