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/ > mutex_lock(&mt9v111->stream_mutex); > -- > 2.11.0 >
Attachment:
signature.asc
Description: PGP signature
