On 08/08/17 12:19, Arend van Spriel wrote:

>>  #define brcmf_sdiod_func0_rb(sdiodev, addr, r) \
>> -    sdio_f0_readb((sdiodev)->func[0], (addr), (r))
>> +    sdio_f0_readb((sdiodev)->func[1], (addr), (r))
> There is no reason to keep these any longer as these do not provide any
> functionality over the core sdio function unless you consider the
> sdiodev dereference.

I tend to agree, although they're fairly readable anyway. I was trying
to keep the changes small and incremental.

I'll knock up a patch and see how it looks with them converted to the
actual functions when I get back from hoiliday.


Reply via email to