diff options
author | Karolina Drobnik <karolinadrobnik@gmail.com> | 2021-11-15 13:55:43 +0300 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2021-11-17 15:55:32 +0300 |
commit | cfbfa0d3c35f7a8a77d57711dc13c30f0caf771a (patch) | |
tree | f07706dc1991f77eaef57137da68c0e8071c4d0f /drivers/staging | |
parent | 6a143ec198a6434fc67b35299d4aad087bae4966 (diff) | |
download | linux-cfbfa0d3c35f7a8a77d57711dc13c30f0caf771a.tar.xz |
staging: vt6655: Introduce `idx` variable
Add a local variable to store `MISCFIFO_SYNDATA_IDX` offset.
This change helps in shortening the lines in `rf.c` that
are deemed too long by checkpatch.pl.
Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Karolina Drobnik <karolinadrobnik@gmail.com>
Link: https://lore.kernel.org/r/596ae9bc980379860b5b1785b3ecc88e18121103.1636729147.git.karolinadrobnik@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging')
-rw-r--r-- | drivers/staging/vt6655/rf.c | 17 |
1 files changed, 9 insertions, 8 deletions
diff --git a/drivers/staging/vt6655/rf.c b/drivers/staging/vt6655/rf.c index aa90890c5c7a..d57fab3f10fb 100644 --- a/drivers/staging/vt6655/rf.c +++ b/drivers/staging/vt6655/rf.c @@ -683,6 +683,7 @@ bool RFvWriteWakeProgSyn(struct vnt_private *priv, unsigned char rf_type, int i; unsigned char init_count = 0; unsigned char sleep_count = 0; + unsigned short idx = MISCFIFO_SYNDATA_IDX; VNSvOutPortW(iobase + MAC_REG_MISCFFNDEX, 0); switch (rf_type) { @@ -699,11 +700,11 @@ bool RFvWriteWakeProgSyn(struct vnt_private *priv, unsigned char rf_type, return false; for (i = 0; i < CB_AL2230_INIT_SEQ; i++) - MACvSetMISCFifo(priv, MISCFIFO_SYNDATA_IDX + i, al2230_init_table[i]); + MACvSetMISCFifo(priv, idx + i, al2230_init_table[i]); - MACvSetMISCFifo(priv, MISCFIFO_SYNDATA_IDX + i, al2230_channel_table0[channel - 1]); + MACvSetMISCFifo(priv, idx + i, al2230_channel_table0[channel - 1]); i++; - MACvSetMISCFifo(priv, MISCFIFO_SYNDATA_IDX + i, al2230_channel_table1[channel - 1]); + MACvSetMISCFifo(priv, idx + i, al2230_channel_table1[channel - 1]); break; /* Need to check, PLLON need to be low for channel setting */ @@ -716,17 +717,17 @@ bool RFvWriteWakeProgSyn(struct vnt_private *priv, unsigned char rf_type, if (channel <= CB_MAX_CHANNEL_24G) { for (i = 0; i < CB_AL7230_INIT_SEQ; i++) - MACvSetMISCFifo(priv, MISCFIFO_SYNDATA_IDX + i, al7230_init_table[i]); + MACvSetMISCFifo(priv, idx + i, al7230_init_table[i]); } else { for (i = 0; i < CB_AL7230_INIT_SEQ; i++) - MACvSetMISCFifo(priv, MISCFIFO_SYNDATA_IDX + i, al7230_init_table_a_mode[i]); + MACvSetMISCFifo(priv, idx + i, al7230_init_table_a_mode[i]); } - MACvSetMISCFifo(priv, MISCFIFO_SYNDATA_IDX + i, al7230_channel_table0[channel - 1]); + MACvSetMISCFifo(priv, idx + i, al7230_channel_table0[channel - 1]); i++; - MACvSetMISCFifo(priv, MISCFIFO_SYNDATA_IDX + i, al7230_channel_table1[channel - 1]); + MACvSetMISCFifo(priv, idx + i, al7230_channel_table1[channel - 1]); i++; - MACvSetMISCFifo(priv, MISCFIFO_SYNDATA_IDX + i, al7230_channel_table2[channel - 1]); + MACvSetMISCFifo(priv, idx + i, al7230_channel_table2[channel - 1]); break; case RF_NOTHING: |