On 30/08/17 19:51, Arend van Spriel wrote: > On 22-08-17 13:25, Ian Molton wrote: >> All the other IO functions are the other way round in this >> driver. Make this one match. > > Sorry for being a nit, but not sure why the commit message starts with a > tab. Not a problem - I don't mind nits being pointed out. Not at all. > Other than that... Will update it in the next (hopefully last!) rev. > Acked-by: Arend van Spriel <arend.vanspriel@xxxxxxxxxxxx> Ta :) >> Signed-off-by: Ian Molton <ian@xxxxxxxxxxxxxx> >> --- >> drivers/net/wireless/broadcom/brcm80211/brcmfmac/bcmsdh.c | 8 ++++---- >> 1 file changed, 4 insertions(+), 4 deletions(-) -Ian