Hi Mauro, On Fri, Aug 14, 2015 at 11:56:39AM -0300, Mauro Carvalho Chehab wrote: ... > diff --git a/include/media/media-entity.h b/include/media/media-entity.h > index 8b21a4d920d9..478d5cd56be9 100644 > --- a/include/media/media-entity.h > +++ b/include/media/media-entity.h > @@ -126,6 +126,8 @@ struct media_entity_graph { > int top; > }; > > +#define entity_id(entity) ((entity)->id) > + > int media_entity_init(struct media_entity *entity, u16 num_pads, > struct media_pad *pads); > void media_entity_cleanup(struct media_entity *entity); media-entity.h is a pretty widely included header file. Perhaps we should think about the naming a bit. All the other names in the header begin with media (or __media); I'd very much prefer not changing that pattern. -- 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