]> git.baikalelectronics.ru Git - kernel.git/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)
commitcfc51f99869d0e0d79762d769f49a94a27d0d6e0
treeb46be5e8e200beba0d687695401ce2bf20b01496
parent270364e12c34118568f0c3d3b434df656bdfa8a3
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