Hi Mauro, On Fri, Aug 14, 2015 at 11:56:42AM -0300, Mauro Carvalho Chehab wrote: > @@ -448,6 +456,7 @@ EXPORT_SYMBOL_GPL(media_device_register_entity); > */ > void media_device_unregister_entity(struct media_entity *entity) > { > + int i; I'd declare temporary variables as last. > struct media_device *mdev = entity->parent; > > if (mdev == NULL) -- Regards, Sakari Ailus e-mail: sakari.ailus@xxxxxx XMPP: sailus@xxxxxxxxxxxxxx -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html