On Mon, Jul 30, 2018 at 09:51:25AM +0200, jacopo mondi wrote: > Hi Sakari, > > On Mon, Jul 30, 2018 at 10:26:27AM +0300, Sakari Ailus wrote: > > While this isn't a bug, initialise the variable to quash the warning. > > > > Reported-by: Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx> > > Signed-off-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> > > --- > > drivers/media/i2c/mt9v111.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/media/i2c/mt9v111.c b/drivers/media/i2c/mt9v111.c > > index da8f6ab91307..58d5f2224bff 100644 > > --- a/drivers/media/i2c/mt9v111.c > > +++ b/drivers/media/i2c/mt9v111.c > > @@ -884,7 +884,7 @@ static int mt9v111_set_format(struct v4l2_subdev *subdev, > > struct v4l2_mbus_framefmt new_fmt; > > struct v4l2_mbus_framefmt *__fmt; > > unsigned int best_fit = ~0L; > > - unsigned int idx; > > + unsigned int idx = 0; > > unsigned int i; > > > > Thanks for this, but there is already a patch sent on Friday by Jasmin > addressing the warning > > https://patchwork.kernel.org/patch/10547983/ Ah, ok. I'll mark this as rejected then. -- Sakari Ailus sakari.ailus@xxxxxxxxxxxxxxx
