diff options
author | Stefan Schmidt <stefan@osg.samsung.com> | 2015-06-08 22:06:40 +0200 |
---|---|---|
committer | Marcel Holtmann <marcel@holtmann.org> | 2015-06-09 09:44:23 +0200 |
commit | 9a4d3d4ba17c93def2b4dc3126eba30716d15469 (patch) | |
tree | e95ba5f010feb0a861a2f99b7e3c686849fbdcde /net/mac802154 | |
parent | e66890a96abbb746e1229c3067471be36dc49b34 (diff) | |
download | lwn-9a4d3d4ba17c93def2b4dc3126eba30716d15469.tar.gz lwn-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.c | 1 |
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; } |