On Thu, Nov 02, 2017 at 04:51:40AM +0200, Laurent Pinchart wrote: > Hi Mauro, > > Thank you for the patch. > > On Wednesday, 1 November 2017 23:05:45 EET Mauro Carvalho Chehab wrote: > > Smatch reports this warning: > > drivers/media/v4l2-core/v4l2-async.c:597 v4l2_async_register_subdev() > > error: uninitialized symbol 'ret'. > > > > However, there's nothing wrong there. So, just shut up the > > warning. > > Nothing wrong, really ? ret does seem to be used uninitialized when the > function returns at the very last line. There's another ret defined in a block under this one; removing that is the correct fix. I wonder why GCC didn't complain about that to begin with... usually it does. -- Sakari Ailus e-mail: sakari.ailus@xxxxxx