mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
synced 2026-05-14 00:39:30 -04:00
staging: vt6656: Use BIT() macro instead of bit shift operator
Use the BIT() macro instead of the bit left shift operator. So the code is more clear. It's safe to remove the casting to u16 type because the value obtained never exceeds 16 bits. So the casting is unnecessary. Signed-off-by: Oscar Carter <oscar.carter@gmx.com> Link: https://lore.kernel.org/r/20200414153849.5785-3-oscar.carter@gmx.com Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
committed by
Greg Kroah-Hartman
parent
f823cce279
commit
91fdbcd909
@@ -228,7 +228,7 @@ void vnt_update_top_rates(struct vnt_private *priv)
|
||||
|
||||
/*Determines the highest basic rate.*/
|
||||
for (i = RATE_54M; i >= RATE_6M; i--) {
|
||||
if (priv->basic_rates & (u16)(1 << i)) {
|
||||
if (priv->basic_rates & BIT(i)) {
|
||||
top_ofdm = i;
|
||||
break;
|
||||
}
|
||||
@@ -237,7 +237,7 @@ void vnt_update_top_rates(struct vnt_private *priv)
|
||||
priv->top_ofdm_basic_rate = top_ofdm;
|
||||
|
||||
for (i = RATE_11M;; i--) {
|
||||
if (priv->basic_rates & (u16)(1 << i)) {
|
||||
if (priv->basic_rates & BIT(i)) {
|
||||
top_cck = i;
|
||||
break;
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user