Hi Jacek, On Fri, Oct 17, 2014 at 04:54:39PM +0200, Jacek Anaszewski wrote: > This patch adds a parser for a media device configuration > file. The parser expects the configuration file containing > links end v4l2-controls definitions as described in the > header file being added. The links describe connections > between media entities and v4l2-controls define the target > sub-devices for particular user controls related ioctl calls. > > Signed-off-by: Jacek Anaszewski <j.anaszewski@xxxxxxxxxxx> > Acked-by: Kyungmin Park <kyungmin.park@xxxxxxxxxxx> > Cc: Mauro Carvalho Chehab <mchehab@xxxxxxxxxxxxxxx> > Cc: Hans Verkuil <hans.verkuil@xxxxxxxxx> > --- > lib/include/libv4l2-media-conf-parser.h | 148 +++++++++++ > lib/libv4l2/libv4l2-media-conf-parser.c | 441 +++++++++++++++++++++++++++++++ > 2 files changed, 589 insertions(+) > create mode 100644 lib/include/libv4l2-media-conf-parser.h > create mode 100644 lib/libv4l2/libv4l2-media-conf-parser.c > > diff --git a/lib/include/libv4l2-media-conf-parser.h b/lib/include/libv4l2-media-conf-parser.h > new file mode 100644 > index 0000000..b2dba3a > --- /dev/null > +++ b/lib/include/libv4l2-media-conf-parser.h > @@ -0,0 +1,148 @@ > +/* > + * Parser of media device configuration file. > + * > + * Copyright (c) 2014 Samsung Electronics Co., Ltd. > + * http://www.samsung.com > + * > + * Author: Jacek Anaszewski <j.anaszewski@xxxxxxxxxxx> > + * > + * The configuration file has to comply with following format: > + * > + * Link description entry format: > + * > + * link { > + * <TAB>source_entity: <entity_name><LF> > + * <TAB>source_pad: <pad_id><LF> > + * <TAB>sink_entity: <entity_name><LF> > + * <TAB>sink_pad: <pad_id><LF> > + * } Could you use the existing libmediactl format? The parser exists as well. As a matter of fact, I have a few patches to make it easier to user in a library. libmediactl appears to be located under utils/media-ctl. Perhaps it's be better placed under lib. Cc Laurent. > + * The V4L2 control group format: > + * > + * v4l2-controls { > + * <TAB><control1_name>: <entity_name><LF> > + * <TAB><control2_name>: <entity_name><LF> > + * ... > + * <TAB><controlN_name>: <entity_name><LF> > + * } I didn't know you were working on this. I have a small library which does essentially the same. The implementation is incomplete, that's why I hadn't posted it to the list. We could perhaps discuss this a little bit tomorrow. When would you be available, in case you are? What would you think of using a little bit more condensed format for this, similar to that of libmediactl? -- 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
