diff options
| author | Karolina Drobnik <karolinadrobnik@gmail.com> | 2021-11-15 10:55:48 +0000 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2021-11-17 13:55:32 +0100 |
| commit | 8026ee384a283bde5c7e3342049abb5abc890d4c (patch) | |
| tree | e5b0be89e6b250db4ab9a30c702fa3dd5d1769ef /drivers/staging/vt6655 | |
| parent | 6a141baa801b911b8ec61a4b19a4807e11e299b7 (diff) | |
| download | linux-8026ee384a283bde5c7e3342049abb5abc890d4c.tar.gz linux-8026ee384a283bde5c7e3342049abb5abc890d4c.tar.bz2 linux-8026ee384a283bde5c7e3342049abb5abc890d4c.zip | |
staging: vt6655: Delete bogus check for `init_count` in AL2230
Remove an unnecessary check in `rf_write_wake_prog_syn` in `RF_AL2230S`
switch case. This `if` conditional will never be true as `init_count` is
equal to 17 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/29c758ba58b10bc1da7e864b3a4f377147fc4428.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 d4c18d9bfc67..4bfadd85c82c 100644 --- a/drivers/staging/vt6655/rf.c +++ b/drivers/staging/vt6655/rf.c @@ -697,8 +697,6 @@ bool rf_write_wake_prog_syn(struct vnt_private *priv, unsigned char rf_type, /* Init Reg + Channel Reg (2) */ init_count = CB_AL2230_INIT_SEQ + 2; sleep_count = 0; - if (init_count > (MISCFIFO_SYNDATASIZE - sleep_count)) - return false; for (i = 0; i < CB_AL2230_INIT_SEQ; i++) MACvSetMISCFifo(priv, idx++, al2230_init_table[i]); |