From: Ian Molton <i...@mnementh.co.uk>

Whilst this if () statement is technically correct, it lacks clarity.

Signed-off-by: Ian Molton <i...@mnementh.co.uk>
Signed-off-by: Arend van Spriel <arend.vanspr...@broadcom.com>
---
 drivers/net/wireless/broadcom/brcm80211/brcmfmac/bcmsdh.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/bcmsdh.c 
b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/bcmsdh.c
index bff4610..17cdc13 100644
--- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/bcmsdh.c
+++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/bcmsdh.c
@@ -746,16 +746,17 @@ int brcmf_sdiod_send_pkt(struct brcmf_sdio_dev *sdiodev,
        if (err)
                return err;
 
-       if (pktq->qlen == 1 || !sdiodev->sg_support)
+       if (pktq->qlen == 1 || !sdiodev->sg_support) {
                skb_queue_walk(pktq, skb) {
                        err = brcmf_sdiod_buff_write(sdiodev, SDIO_FUNC_2,
                                                     addr, skb);
                        if (err)
                                break;
                }
-       else
+       } else {
                err = brcmf_sdiod_sglist_rw(sdiodev, SDIO_FUNC_2, true, addr,
                                            pktq);
+       }
 
        return err;
 }
-- 
1.9.1

Reply via email to