diff options
| author | Karolina Drobnik <karolinadrobnik@gmail.com> | 2021-11-15 10:55:49 +0000 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2021-11-17 13:55:32 +0100 |
| commit | 944f0f697acd674ed34cac284cb2d93a714e3e9d (patch) | |
| tree | 6623f97dc771eff5fa1ca03d1fad2092a1872b31 /drivers/staging/vt6655 | |
| parent | 8026ee384a283bde5c7e3342049abb5abc890d4c (diff) | |
| download | linux-944f0f697acd674ed34cac284cb2d93a714e3e9d.tar.bz2 | |
staging: vt6655: Delete bogus check for `init_count` in AL7230
Remove an unnecessary check in `rf_write_wake_prog_syn` in `RF_AIROHA7230`
switch case. This `if` conditional will never be true as `init_count` is
equal to 18 and can't be bigger than `MISCFIFO_SYNDATASIZE - 0`, which
is equal to 21.
Suggested-by: Mike Rapoport <mike.rapoport@gmail.com>
Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Karolina Drobnik <karolinadrobnik@gmail.com>
Link: https://lore.kernel.org/r/e10d2278975116e03a699260f2259f9ab87501a5.1636729147.git.karolinadrobnik@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/vt6655')
| -rw-r--r-- | drivers/staging/vt6655/rf.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/staging/vt6655/rf.c b/drivers/staging/vt6655/rf.c index 4bfadd85c82c..8513fc565b1a 100644 --- a/drivers/staging/vt6655/rf.c +++ b/drivers/staging/vt6655/rf.c @@ -710,8 +710,6 @@ bool rf_write_wake_prog_syn(struct vnt_private *priv, unsigned char rf_type, /* Init Reg + Channel Reg (3) */ init_count = CB_AL7230_INIT_SEQ + 3; sleep_count = 0; - if (init_count > (MISCFIFO_SYNDATASIZE - sleep_count)) - return false; init_table = (channel <= CB_MAX_CHANNEL_24G) ? al7230_init_table : al7230_init_table_a_mode; |