Re: [PATCH] staging: ks7010: remove unnecessary parentheses
Greg KH <[email protected]>
| Newsgroups | org.linuxdriverproject.driverdev-devel,dev.linux.lists.linux-staging |
|---|---|
| Message-ID | <[email protected]> |
On Thu, Mar 30, 2023 at 08:48:28PM +0800, Joel Camilo Chang Gonzalez wrote: > Remove redundant parentheses > > Signed-off-by: Joel Camilo Chang Gonzalez <[email protected]> > --- > drivers/staging/ks7010/ks_wlan_net.c | 12 ++++++------ > 1 file changed, 6 insertions(+), 6 deletions(-) > > diff --git a/drivers/staging/ks7010/ks_wlan_net.c b/drivers/staging/ks7010/ks_wlan_net.c > index e03c87f0bfe7..eef1a1e70088 100644 > --- a/drivers/staging/ks7010/ks_wlan_net.c > +++ b/drivers/staging/ks7010/ks_wlan_net.c > @@ -193,14 +193,14 @@ static int ks_wlan_set_freq(struct net_device *dev, > fwrq->freq.m = c + 1; > } > /* Setting by channel number */ > - if ((fwrq->freq.m > 1000) || (fwrq->freq.e > 0)) > + if (fwrq->freq.m > 1000 || fwrq->freq.e > 0) Do you want to have to remember the precidence order between "||" and ">"? I don't, so please don't make this change. thanks, greg k-h _______________________________________________ devel mailing list [email protected] http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel