On 07/08/17 12:26, Arend van Spriel wrote: >> + 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... > >> return err; >> > > ...and kept this one. > >> } > Good catch. There's a couple of others - I'll break them out. That latter one is clearly an error on my part. -Ian
