Re: [PATCH] pinctrl: core: Set ret to 0 when group is skipped

2021-03-15 Thread Linus Walleij
On Fri, Mar 12, 2021 at 8:31 AM Michal Simek wrote: > Static analyzer tool found that the ret variable is not initialized but > code expects ret value >=0 when pinconf is skipped in the first pinmux > loop. The same expectation is for pinmux in a pinconf loop. > That's why initialize ret to 0 to

Re: [PATCH] pinctrl: core: Set ret to 0 when group is skipped

2021-03-12 Thread Colin Ian King
On 12/03/2021 07:31, Michal Simek wrote: > Static analyzer tool found that the ret variable is not initialized but > code expects ret value >=0 when pinconf is skipped in the first pinmux > loop. The same expectation is for pinmux in a pinconf loop. > That's why initialize ret to 0 to avoid

[PATCH] pinctrl: core: Set ret to 0 when group is skipped

2021-03-11 Thread Michal Simek
Static analyzer tool found that the ret variable is not initialized but code expects ret value >=0 when pinconf is skipped in the first pinmux loop. The same expectation is for pinmux in a pinconf loop. That's why initialize ret to 0 to avoid uninitialized ret value in first loop or reusing ret