Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753171AbbFMKwl (ORCPT ); Sat, 13 Jun 2015 06:52:41 -0400 Received: from www.osadl.org ([62.245.132.105]:60799 "EHLO www.osadl.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750853AbbFMKwd (ORCPT ); Sat, 13 Jun 2015 06:52:33 -0400 From: Nicholas Mc Guire To: Larry Finger Cc: Chaoming Li , Kalle Valo , "John W. Linville" , Jiri Kosina , Randy Dunlap , Masanari Iida , linux-wireless@vger.kernel.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [BUG ?] rtlwifi: rtl8723be: condition with no effect Date: Sat, 13 Jun 2015 12:43:31 +0200 Message-Id: <1434192211-5855-1-git-send-email-hofrat@osadl.org> X-Mailer: git-send-email 1.7.10.4 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1103 Lines: 26 From: Nicholas Mc Guire scanning for trivial bug-patters with coccinelle spatches returned: ./drivers/net/wireless/rtlwifi/rtl8723be/dm.c:886 WARNING: condition with no effect (if branch == else) Added in 'commit a619d1abe20c ("rtlwifi: rtl8723be: Add new driver")' drivers/net/wireless/rtlwifi/rtl8723be/dm.c - line numbers from 4.1-rc7 886 if (thermalvalue > rtlefuse->eeprom_thermalmeter) 887 rtl8723be_dm_tx_power_track_set_power(hw, BBSWING, 0, 888 index_for_channel); 889 else 890 rtl8723be_dm_tx_power_track_set_power(hw, BBSWING, 0, 891 index_for_channel); Can't figure out what the intent of this condition is but it currently has no effect as if == else and this most likely is not the intent. thx! hofrat -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/