lkml.org 
[lkml]   [2022]   [Jun]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] net: wireless/broadcom: fix possible condition with no effect
On Sun, 26 Jun 2022 13:03:57 +0200
Sebastian Gottschall <s.gottschall@newmedia-net.de> wrote:

> Am 25.06.2022 um 21:29 schrieb Praghadeesh T K S:
> > Fix a coccinelle warning by removing condition with no possible
> > effect
> >
> > Signed-off-by: Praghadeesh T K S <praghadeeshthevendria@gmail.com>
> > ---
> > drivers/net/wireless/broadcom/b43/xmit.c | 7 +------
> > 1 file changed, 1 insertion(+), 6 deletions(-)
> >
> > diff --git a/drivers/net/wireless/broadcom/b43/xmit.c
> > b/drivers/net/wireless/broadcom/b43/xmit.c index 7651b1b..667a74b
> > 100644 --- a/drivers/net/wireless/broadcom/b43/xmit.c
> > +++ b/drivers/net/wireless/broadcom/b43/xmit.c
> > @@ -169,12 +169,7 @@ static u16 b43_generate_tx_phy_ctl1(struct
> > b43_wldev *dev, u8 bitrate) const struct b43_phy *phy = &dev->phy;
> > const struct b43_tx_legacy_rate_phy_ctl_entry *e;
> > u16 control = 0;
> > - u16 bw;
> > -
> > - if (phy->type == B43_PHYTYPE_LP)
> > - bw = B43_TXH_PHY1_BW_20;
> > - else /* FIXME */
> > - bw = B43_TXH_PHY1_BW_20;
> > + u16 bw = B43_TXH_PHY1_BW_20;
> Can you check if this is a possible register typo?


I think use of this name was intentional.
The code is marked as being incomplete with /* FIXME */

This change removes the FIXME. Which is bad. It doesn't improve the
code. It reduces code quality by removing the
incompleteness documentation.

Please leave the code as-is.

--
Michael Büsch
https://bues.ch/
[unhandled content-type:application/pgp-signature]
\
 
 \ /
  Last update: 2022-06-26 14:00    [W:0.093 / U:0.148 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site