iwlwifi: remove late null-check and duplicate bug_on

These pieces of code appear to be useless. The BUG_ON is also performed in
iwl_send_cmd_async.

Serious karma would be needed to enter iwl_ativate_qos with a null priv.
I had a deja vu when patching this, but for the life of me I couldn't track
down a similar patch.

Signed-off-by: Ian Schram <ischram@telenet.be>
Signed-off-by: Tomas Winkler <tomas.winkler@intel.com>
Signed-off-by: Zhu Yi <yi.zhu@intel.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>

2 files changed, 0 insertions, 14 deletions

diff --git a/drivers/net/wireless/iwlwifi/iwl3945-base.c b/drivers/net/wireless/iwlwifi/iwl3945-base.c
index 5041368a9a20..c6606e1af51d 100644
--- a/drivers/net/wireless/iwlwifi/iwl3945-base.c
+++ b/drivers/net/wireless/iwlwifi/iwl3945-base.c
@@ -808,10 +808,6 @@ out:
 
 int iwl_send_cmd(struct iwl_priv *priv, struct iwl_host_cmd *cmd)
 {
-        /* A command can not be asynchronous AND expect an SKB to be set. */
-        BUG_ON((cmd->meta.flags & CMD_ASYNC) &&
-               (cmd->meta.flags & CMD_WANT_SKB));
-
         if (cmd->meta.flags & CMD_ASYNC)
                 return iwl_send_cmd_async(priv, cmd);
 
@@ -1968,9 +1964,6 @@ static void iwl_activate_qos(struct iwl_priv *priv, u8 force)
 {
         unsigned long flags;
 
-        if (priv == NULL)
-                return;
-
         if (test_bit(STATUS_EXIT_PENDING, &priv->status))
                 return;
 
diff --git a/drivers/net/wireless/iwlwifi/iwl4965-base.c b/drivers/net/wireless/iwlwifi/iwl4965-base.c
index 076c5518211b..fa11eb95699b 100644
--- a/drivers/net/wireless/iwlwifi/iwl4965-base.c
+++ b/drivers/net/wireless/iwlwifi/iwl4965-base.c
@@ -806,10 +806,6 @@ out:
 
 int iwl_send_cmd(struct iwl_priv *priv, struct iwl_host_cmd *cmd)
 {
-        /* A command can not be asynchronous AND expect an SKB to be set. */
-        BUG_ON((cmd->meta.flags & CMD_ASYNC) &&
-               (cmd->meta.flags & CMD_WANT_SKB));
-
         if (cmd->meta.flags & CMD_ASYNC)
                 return iwl_send_cmd_async(priv, cmd);
 
@@ -2040,9 +2036,6 @@ static void iwl_activate_qos(struct iwl_priv *priv, u8 force)
 {
         unsigned long flags;
 
-        if (priv == NULL)
-                return;
-
         if (test_bit(STATUS_EXIT_PENDING, &priv->status))
                 return;