On Thu, 2010-07-22 at 17:20 +0200, Laurent Pinchart wrote: > > Laurent Pinchart wrote: > > > > ... > > > > > diff --git a/Documentation/media-framework.txt > > > b/Documentation/media-framework.txt index 3acc62b..16c0177 100644 > > > --- a/Documentation/media-framework.txt > > > +++ b/Documentation/media-framework.txt > > > @@ -270,3 +270,137 @@ required, drivers don't need to provide a set_power > > [snip] > > > > +The media_user_pad, media_user_link and media_user_links structure are > > > defined > > > +as > > > > I have a comment on naming. These are user space structures, sure, but > > do we want that fact to be visible in the names of the structures? I > > would just drop the user_ out and make the naming as good as possible in > > user space. That is much harder to change later than naming inside the > > kernel. > > I agree. > > > That change causes a lot of clashes in naming since the equivalent > > kernel structure is there as well. Those could have _k postfix, for > > example, to differentiate them from user space names. I don't really > > have a good suggestion how they should be called. > > Maybe media_k_* ? I'm not very happy with that name either though. What do you think about a single underscore prefix for the kernel structures, used commonly to indicate that a declaration is limited? -- 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
