On 07/08/17 12:26, Arend van Spriel wrote:

quoted

+        err = brcmf_sdiod_buff_write(sdiodev, SDIO_FUNC_2, addr, mypkt);

      brcmu_pkt_buf_free_skb(mypkt);
+

You are keen on sprinkling whitespace here and there. Could you please
use separate patches for that as much as possible. Not sure why you
added this one...

quoted

      return err;

...and kept this one.

quoted

  }

Good catch. There's a couple of others - I'll break them out. That
latter one is clearly an error on my part.

-Ian