]> Pileus Git - ~andy/linux/commit
staging: csr: Fix obvious logic error in staging/csr
authorTobias Polzer <tobias.polzer@fau.de>
Thu, 13 Jun 2013 21:00:35 +0000 (23:00 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 17 Jun 2013 21:48:12 +0000 (14:48 -0700)
commit0a58857707c8bc07dcca39a4a6b2c88e37ff2b01
treeb46be5e8e200beba0d687695401ce2bf20b01496
parent126898b0a26a7e1b01a5f90a19601495094e2055
staging: csr: Fix obvious logic error in staging/csr

In uf_process_ma_packet_req() (staging/csr/unifi_pdu_processing.c), the
test for
(staInfo->timSet != CSR_WIFI_TIM_RESET) || (staInfo->timSet !=
CSR_WIFI_TIM_RESETTING)
obviously makes no sense and causes GCC to emit a warning if
-Wlogical-op is enabled.
As suggested in the referenced bug report, || was replaced by &&.

References: https://bugzilla.kernel.org/show_bug.cgi?id=46571
Reported-By: David Binderman <dcb314@hotmail.com>
Signed-off-by: Tobias Polzer <tobias.polzer@fau.de>
Signed-off-by: Dominik Paulus <dominik.paulus@fau.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/csr/unifi_pdu_processing.c