From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from bu3sch.de ([62.75.166.246]:41406 "EHLO vs166246.vserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753261AbZBOQsJ (ORCPT ); Sun, 15 Feb 2009 11:48:09 -0500 From: Michael Buesch To: Roel Kluin Subject: Re: [PATCH] B43: misplaced parentheses? Date: Sun, 15 Feb 2009 17:46:44 +0100 Cc: stefano.brivio@polimi.it, linux-wireless@vger.kernel.org, Andrew Morton References: <499844E7.1040307@gmail.com> In-Reply-To: <499844E7.1040307@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Message-Id: <200902151746.44754.mb@bu3sch.de> (sfid-20090215_174813_591417_C8D4FDCF) Sender: linux-wireless-owner@vger.kernel.org List-ID: On Sunday 15 February 2009 17:37:59 Roel Kluin wrote: > I think below is what was intended? otherwise we could as well have written: > > b43_radio_write16(dev, txctl_reg, > (b43_radio_read16(dev, txctl_reg) & ~txctl_value) > || (rfatt->with_padmix) ? txctl_value : 0); > > ^^--- Note: boolean or Uhm, no? Can you please explain why you think this is equal? Does | have precedence over ?: > > please review. > -------------------------->8------------------8<--------------------------- > Fix misplaced parentheses > > Signed-off-by: Roel Kluin > --- > diff --git a/drivers/net/wireless/b43/lo.c b/drivers/net/wireless/b43/lo.c > index 6a18a14..88ed75f 100644 > --- a/drivers/net/wireless/b43/lo.c > +++ b/drivers/net/wireless/b43/lo.c > @@ -783,7 +783,7 @@ struct b43_lo_calib * b43_calibrate_lo_setting(struct b43_wldev *dev, > | rfatt->att); > b43_radio_write16(dev, txctl_reg, > (b43_radio_read16(dev, txctl_reg) & ~txctl_value) > - | (rfatt->with_padmix) ? txctl_value : 0); > + | (rfatt->with_padmix ? txctl_value : 0)); > > max_rx_gain = rfatt->att * 2; > max_rx_gain += bbatt->att / 2; > -- > To unsubscribe from this list: send the line "unsubscribe linux-wireless" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > > -- Greetings, Michael.