Hi Mauro, On Fri, Aug 14, 2015 at 11:56:37AM -0300, Mauro Carvalho Chehab wrote: > Those are the initial patches from my previous series of MC changes. > > The first patch removes an unused parameter when creating links. > > The next 5 patches warrant that all object types (entities, pads and > links) will have an unique ID, as agreed at the MC workshop. > > They prepare for the addition of the media interfaces and interface > links. Having looked the set through, I don't think the patches in the set are strictly necessary for adding media interfaces. Again, I need to stress I'd very much prefer to keep things simple in order to get support for media interfaces in soon, as I understand your intention is as well. We could make things more dynamic later on, and represent associations using links --- if there's a use case for that. I don't as such object the patchset, but my question is: where will this all lead to? I'd like to see that, or at least some more, before finally acking the patches. I sense these should be closely related to supporting the property API rather than media interfaces (or DVB), but unfortunately I won't have time to work on the property API for the following ~ three weeks. struct media_interface could be pointed to from entities using a statically allocated array of pointers, a bit like links (except that they're not pointers). I think we'd get quite far with this already while making much fewer changes to the framework. One thing that wasn't discussed at length in the meeting, but which I understood was generally agreed on, was DMA engines as entities (vs. having a pad for the sake of the interface in the video node entity, which is ugly). IMHO a sound foundation is important for the proposed changes. Just my one euro cent --- got some left from Italy. :-) -- Kind 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
