There is one il_set_tx_power() call in this function without
return value check. Print error message and return error code
on failure just like the other il_set_tx_power() call.

Signed-off-by: Dinghao Liu <dinghao....@zju.edu.cn>
---
 drivers/net/wireless/intel/iwlegacy/4965.c | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/drivers/net/wireless/intel/iwlegacy/4965.c 
b/drivers/net/wireless/intel/iwlegacy/4965.c
index 9fa556486511..3235b8be1894 100644
--- a/drivers/net/wireless/intel/iwlegacy/4965.c
+++ b/drivers/net/wireless/intel/iwlegacy/4965.c
@@ -1361,7 +1361,11 @@ il4965_commit_rxon(struct il_priv *il)
                 * We do not commit tx power settings while channel changing,
                 * do it now if tx power changed.
                 */
-               il_set_tx_power(il, il->tx_power_next, false);
+               ret = il_set_tx_power(il, il->tx_power_next, false);
+               if (ret) {
+                       IL_ERR("Error sending TX power (%d)\n", ret);
+                       return ret;
+               }
                return 0;
        }
 
-- 
2.17.1

Reply via email to