Whilst this if () statement is technically correct, it lacks clarity.
Signed-off-by: Ian Molton <ian@mnementh.co.uk>
Signed-off-by: Arend van Spriel <arend.vanspriel@broadcom.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
        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;
 }