This patch has the same subject line as the one I previously commented on (modulo the typo in that one). 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. > _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel