Re: [PATCH 08/34] brcmfmac: Fix uninitialised variable

2017-08-19 Thread Ian Molton
On 07/08/17 12:26, Arend van Spriel wrote: > On 7/26/2017 10:25 PM, Ian Molton wrote: >> Unlikely to be a problem, but brcmf_sdiod_regrb() is >> not symmetric with brcmf_sdiod_regrl() in this regard. > > You are pretty keen on symmetry, but you could also remove the > initialization in

Re: [PATCH 08/34] brcmfmac: Fix uninitialised variable

2017-08-07 Thread Arend van Spriel
On 7/26/2017 10:25 PM, Ian Molton wrote: Unlikely to be a problem, but brcmf_sdiod_regrb() is not symmetric with brcmf_sdiod_regrl() in this regard. You are pretty keen on symmetry, but you could also remove the initialization in brcmf_sdiod_regrl(). As long as no -Wunitialized pops