Re: [PATCH] pinctrl: madera: Fix uninitialized variable issue in madera_mux_set_mux

From: Gustavo A. R. Silva
Date: Wed Oct 10 2018 - 10:58:40 EST


Hi Charles,

> I don't believe this would fully address the issue since if you
> initialise ret to -1 you will still drop into the if statement
> and you will use the reg variable which should also be
> uninitialised on that code path.
>
> Feels like initialising to 0 would probably be better anyway
> since the code path looks like the case were you have n_pins == 0
> and nothing really failed in that case just nothing needed to be
> done.
>

You're right. I got confused with another similar issue I fixed, recently.

I'll send v2 shortly.

Thanks for the feedback.
--
Gustavo