On Mon, 2011-09-05 at 16:57 +0200, Laurent Pinchart wrote: > > > This will break binary compatibility if an application creates a struct > > > media_device instances itself. On the other hand applications are not > > > supposed to do that. > > > > > > As the struct udev pointer is only used internally, what about passing it > > > around between functions explicitly instead ? > > > > That we will break the API in media_close(). > > Might be I am a blind, but I can't see the way how to do both 1) don't > > provide static global variable and 2) don't break the API/ABI. > > What about passing the udev pointer explictly to media_enum_entities() (which > is static), and calling media_udev_close() in media_open() after the > media_enum_entities() call ? I sent the patch series that incorporates your last comments. -- Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> Intel Finland Oy -- 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