diff options
author | Jérôme Pouiller <jerome.pouiller@silabs.com> | 2019-10-08 09:43:01 +0000 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2019-10-08 14:44:56 +0200 |
commit | b5be2aa3b3ac153f056f93406c520dc74a1cebbe (patch) | |
tree | 97bd40357c828e475d64c31e63432bb05d4f7781 /drivers/staging/wfx/scan.c | |
parent | staging: wfx: fix copy_{to,from}_user() usage (diff) | |
download | linux-dev-b5be2aa3b3ac153f056f93406c520dc74a1cebbe.tar.xz linux-dev-b5be2aa3b3ac153f056f93406c520dc74a1cebbe.zip |
staging: wfx: drop calls to BUG_ON()
Most of calls to BUG_ON() could replaced by WARN().
By the way, this patch also try to favor WARN() (that include a comment
about the problem) instead of WARN_ON().
Reported-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: Jérôme Pouiller <jerome.pouiller@silabs.com>
Link: https://lore.kernel.org/r/20191008094232.10014-7-Jerome.Pouiller@silabs.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/wfx/scan.c')
-rw-r--r-- | drivers/staging/wfx/scan.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/staging/wfx/scan.c b/drivers/staging/wfx/scan.c index ea5001c915f6..cba735c1e73c 100644 --- a/drivers/staging/wfx/scan.c +++ b/drivers/staging/wfx/scan.c @@ -107,7 +107,7 @@ int wfx_hw_scan(struct ieee80211_hw *hw, wfx_tx_lock_flush(wdev); - BUG_ON(wvif->scan.req); + WARN(wvif->scan.req, "unexpected concurrent scan"); wvif->scan.req = req; wvif->scan.n_ssids = 0; wvif->scan.status = 0; |