On 07/08/17 18:55, Ian Molton wrote: > On 07/08/17 12:26, Arend van Spriel wrote: >> On 7/26/2017 10:25 PM, Ian Molton wrote: >>> Primarily this patch removes: >>> >>> brcmf_sdiod_f0_writeb() >>> brcmf_sdiod_reg_write() >>> brcmf_sdiod_reg_read() >> >> Having [patch 30/34] "brcmfmac: Correctly handle accesses to SDIO func0" >> before this patch could make this look cleaner. > > This is an artifact of how I came to the realisation the code was using > the obsoleted functions - it could be reordered, but it'd probably get > messy... I gave it a try, and it got messy - I'd prefer to leave these patches in their current ordering if thats OK. -Ian