Re: [Outreachy kernel] [PATCH] staging: wlan-ng: Remove unnecessary parentheses

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On Mon, 16 Oct 2017, Frank A. Cancio Bello wrote:

> On Mon, Oct 16, 2017 at 10:18:14PM +0200, Julia Lawall wrote:
> > This patch has the same subject line as the one I previously commented on
> > (modulo the typo in that one).
> >
>
> Thanks Julia for your quick response. Before work on your suggestions I have a question regarding the above comment:
>
> Do you want that I merge this two patches to avoid two patches with the
> same subject line? Or they are OK as two with the same subject line?

No, this is not OK.  Either merge the patches or find a different subject
line.  If there is some reason why you chose to put one file in one patch
or the other then that may suggest what the subject line should be.

julia

> I
> split the changes into two patches to avoid big patches and the subject
> line come as the same because I'm doing the same in each of them ;)
>
> > On Mon, 16 Oct 2017, Frank A. Cancio Bello wrote:
> >
> > > checkpatch message: 'CHECK: Unnecessary parentheses around'.
> >
> > Again, say what and why you have done.
> >
> > julia
> >
> > > Signed-off-by: Frank A. Cancio Bello <frank@xxxxxxxxxxxxxxxxxxxxxx>
> > > ---
> > > FYI:
> > > 	* I read 'Ignore checkpatch.pl if it complains about parens around boolean expressions or ternary conditionals in return values, like this: return ((depth > 1) ? (depth - 1) : depth);'. All the fixes in this patch are not in 'return values', so I hope there are OK ;)
> > >
> > > drivers/staging/wlan-ng/cfg80211.c     | 10 +++++-----
> > >  drivers/staging/wlan-ng/hfa384x_usb.c  | 18 +++++++++---------
> > >  drivers/staging/wlan-ng/p80211conv.c   |  6 +++---
> > >  drivers/staging/wlan-ng/p80211netdev.c |  4 ++--
> > >  4 files changed, 19 insertions(+), 19 deletions(-)
> > >
> > > diff --git a/drivers/staging/wlan-ng/cfg80211.c b/drivers/staging/wlan-ng/cfg80211.c
> > > index 178f6f5..03279aa 100644
> > > --- a/drivers/staging/wlan-ng/cfg80211.c
> > > +++ b/drivers/staging/wlan-ng/cfg80211.c
> > > @@ -265,7 +265,7 @@ static int prism2_get_station(struct wiphy *wiphy, struct net_device *dev,
> > >
> > >  	memset(sinfo, 0, sizeof(*sinfo));
> > >
> > > -	if (!wlandev || (wlandev->msdstate != WLAN_MSD_RUNNING))
> > > +	if (!wlandev || wlandev->msdstate != WLAN_MSD_RUNNING)
> > >  		return -EOPNOTSUPP;
> > >
> > >  	/* build request message */
> > > @@ -367,8 +367,8 @@ static int prism2_scan(struct wiphy *wiphy,
> > >  		msg2.bssindex.data = i;
> > >
> > >  		result = p80211req_dorequest(wlandev, (u8 *)&msg2);
> > > -		if ((result != 0) ||
> > > -		    (msg2.resultcode.data != P80211ENUM_resultcode_success)) {
> > > +		if (result != 0 ||
> > > +		    msg2.resultcode.data != P80211ENUM_resultcode_success) {
> > >  			break;
> > >  		}
> > >
> > > @@ -475,8 +475,8 @@ static int prism2_connect(struct wiphy *wiphy, struct net_device *dev,
> > >  	}
> > >
> > >  	/* Set the authorization */
> > > -	if ((sme->auth_type == NL80211_AUTHTYPE_OPEN_SYSTEM) ||
> > > -	    ((sme->auth_type == NL80211_AUTHTYPE_AUTOMATIC) && !is_wep))
> > > +	if (sme->auth_type == NL80211_AUTHTYPE_OPEN_SYSTEM ||
> > > +	    sme->auth_type == NL80211_AUTHTYPE_AUTOMATIC && !is_wep)
> > >  		msg_join.authtype.data = P80211ENUM_authalg_opensystem;
> > >  	else if ((sme->auth_type == NL80211_AUTHTYPE_SHARED_KEY) ||
> > >  		 ((sme->auth_type == NL80211_AUTHTYPE_AUTOMATIC) && is_wep))
> > > diff --git a/drivers/staging/wlan-ng/hfa384x_usb.c b/drivers/staging/wlan-ng/hfa384x_usb.c
> > > index d1e8218..b5fdfd8 100644
> > > --- a/drivers/staging/wlan-ng/hfa384x_usb.c
> > > +++ b/drivers/staging/wlan-ng/hfa384x_usb.c
> > > @@ -1781,9 +1781,9 @@ int hfa384x_drvr_enable(struct hfa384x *hw, u16 macport)
> > >  {
> > >  	int result = 0;
> > >
> > > -	if ((!hw->isap && macport != 0) ||
> > > -	    (hw->isap && !(macport <= HFA384x_PORTID_MAX)) ||
> > > -	    (hw->port_enabled[macport])) {
> > > +	if (!hw->isap && macport != 0 ||
> > > +	    hw->isap && !(macport <= HFA384x_PORTID_MAX) ||
> > > +	    hw->port_enabled[macport]) {
> > >  		result = -EINVAL;
> > >  	} else {
> > >  		result = hfa384x_cmd_enable(hw, macport);
> > > @@ -2465,7 +2465,7 @@ int hfa384x_drvr_start(struct hfa384x *hw)
> > >  		netdev_err(hw->wlandev->netdev, "Cannot get bulk in endpoint status.\n");
> > >  		goto done;
> > >  	}
> > > -	if ((status == 1) && usb_clear_halt(hw->usb, hw->endp_in))
> > > +	if (status == 1 && usb_clear_halt(hw->usb, hw->endp_in))
> > >  		netdev_err(hw->wlandev->netdev, "Failed to reset bulk in endpoint.\n");
> > >
> > >  	result =
> > > @@ -2474,7 +2474,7 @@ int hfa384x_drvr_start(struct hfa384x *hw)
> > >  		netdev_err(hw->wlandev->netdev, "Cannot get bulk out endpoint status.\n");
> > >  		goto done;
> > >  	}
> > > -	if ((status == 1) && usb_clear_halt(hw->usb, hw->endp_out))
> > > +	if (status == 1 && usb_clear_halt(hw->usb, hw->endp_out))
> > >  		netdev_err(hw->wlandev->netdev, "Failed to reset bulk out endpoint.\n");
> > >
> > >  	/* Synchronous unlink, in case we're trying to restart the driver */
> > > @@ -3054,7 +3054,7 @@ static void hfa384x_usbin_callback(struct urb *urb)
> > >  		goto exit;
> > >
> > >  	skb = hw->rx_urb_skb;
> > > -	if (!skb || (skb->data != urb->transfer_buffer)) {
> > > +	if (!skb || skb->data != urb->transfer_buffer) {
> > >  		WARN_ON(1);
> > >  		return;
> > >  	}
> > > @@ -3501,8 +3501,8 @@ static void hfa384x_int_rxmonitor(struct wlandevice *wlandev,
> > >  		return;
> > >
> > >  	/* only prepend the prism header if in the right mode */
> > > -	if ((wlandev->netdev->type == ARPHRD_IEEE80211_PRISM) &&
> > > -	    (hw->sniffhdr != 0)) {
> > > +	if (wlandev->netdev->type == ARPHRD_IEEE80211_PRISM &&
> > > +	    hw->sniffhdr != 0) {
> > >  		struct p80211_caphdr *caphdr;
> > >  		/* The NEW header format! */
> > >  		datap = skb_put(skb, sizeof(struct p80211_caphdr));
> > > @@ -3772,7 +3772,7 @@ static void hfa384x_ctlxout_callback(struct urb *urb)
> > >
> > >  	spin_unlock_irqrestore(&hw->ctlxq.lock, flags);
> > >
> > > -	if (!timer_ok && (hw->resp_timer_done == 0)) {
> > > +	if (!timer_ok && hw->resp_timer_done == 0) {
> > >  		spin_lock_irqsave(&hw->ctlxq.lock, flags);
> > >  		goto delresp;
> > >  	}
> > > diff --git a/drivers/staging/wlan-ng/p80211conv.c b/drivers/staging/wlan-ng/p80211conv.c
> > > index c1b6d42..bc63eb2 100644
> > > --- a/drivers/staging/wlan-ng/p80211conv.c
> > > +++ b/drivers/staging/wlan-ng/p80211conv.c
> > > @@ -356,10 +356,10 @@ int skb_p80211_to_ether(struct wlandevice *wlandev, u32 ethconv,
> > >  		sizeof(struct wlan_llc));
> > >
> > >  	/* Test for the various encodings */
> > > -	if ((payload_length >= sizeof(struct wlan_ethhdr)) &&
> > > +	if (payload_length >= sizeof(struct wlan_ethhdr) &&
> > >  	    (e_llc->dsap != 0xaa || e_llc->ssap != 0xaa) &&
> > > -	    ((!ether_addr_equal_unaligned(daddr, e_hdr->daddr)) ||
> > > -	     (!ether_addr_equal_unaligned(saddr, e_hdr->saddr)))) {
> > > +	    (!ether_addr_equal_unaligned(daddr, e_hdr->daddr) ||
> > > +	     !ether_addr_equal_unaligned(saddr, e_hdr->saddr))) {
> > >  		pr_debug("802.3 ENCAP len: %d\n", payload_length);
> > >  		/* 802.3 Encapsulated */
> > >  		/* Test for an overlength frame */
> > > diff --git a/drivers/staging/wlan-ng/p80211netdev.c b/drivers/staging/wlan-ng/p80211netdev.c
> > > index 0f50365..ad11a3f 100644
> > > --- a/drivers/staging/wlan-ng/p80211netdev.c
> > > +++ b/drivers/staging/wlan-ng/p80211netdev.c
> > > @@ -427,7 +427,7 @@ static int p80211knetdev_hard_start_xmit(struct sk_buff *skb,
> > >
> > >  failed:
> > >  	/* Free up the WEP buffer if it's not the same as the skb */
> > > -	if ((p80211_wep.data) && (p80211_wep.data != skb->data))
> > > +	if (p80211_wep.data && p80211_wep.data != skb->data)
> > >  		kzfree(p80211_wep.data);
> > >
> > >  	/* we always free the skb here, never in a lower level. */
> > > @@ -491,7 +491,7 @@ static int p80211netdev_ethtool(struct wlandevice *wlandev,
> > >  		edata.cmd = ethcmd;
> > >
> > >  		if (wlandev->linkstatus &&
> > > -		    (wlandev->macmode != WLAN_MACMODE_NONE)) {
> > > +		    wlandev->macmode != WLAN_MACMODE_NONE) {
> > >  			edata.data = 1;
> > >  		} else {
> > >  			edata.data = 0;
> > > --
> > > 2.7.4
> > >
> > > --
> > > You received this message because you are subscribed to the Google Groups "outreachy-kernel" group.
> > > To unsubscribe from this group and stop receiving emails from it, send an email to outreachy-kernel+unsubscribe@xxxxxxxxxxxxxxxx.
> > > To post to this group, send email to outreachy-kernel@xxxxxxxxxxxxxxxx.
> > > To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/20171016182614.GA3341%40ubuntu-server-1604.
> > > For more options, visit https://groups.google.com/d/optout.
> > >
>
> --
> You received this message because you are subscribed to the Google Groups "outreachy-kernel" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to outreachy-kernel+unsubscribe@xxxxxxxxxxxxxxxx.
> To post to this group, send email to outreachy-kernel@xxxxxxxxxxxxxxxx.
> To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/20171016214556.GA2215%40ubuntu-server-1604.
> For more options, visit https://groups.google.com/d/optout.
>
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel



[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux