Re: [v4l-utils RFC 0/2] libmediatext library

[Hans de Goede <hdegoede@redhat.com>]

Hi,

On 10/23/2014 10:54 AM, Jacek Anaszewski wrote:
> Hi Hans, Sakari,
> 
> On 10/23/2014 10:19 AM, Hans de Goede wrote:
>> Hi,
>>
>> On 10/22/2014 11:47 AM, Sakari Ailus wrote:
>>> Hi Hans,
>>>
>>> Hans de Goede wrote:
>>>> Hi Sakari,
>>>>
>>>> On 10/21/2014 12:40 PM, Sakari Ailus wrote:
>>>>> Hi,
>>>>>
>>>>> This is a tiny library for parsing text-based media link, V4L2 sub-device
>>>>> format (and selection) configurations as well as controls with limited
>>>>> types.
>>>>
>>>> Hmm, we also have:
>>>>
>>>> [PATCH/RFC v2 1/4] Add a media device configuration file parser.
>>>>
>>>> How do these 2 relate ?
>>>
>>> Jacek is working on a Samsung Exynos libv4l2 plugin, a part of which is not specific to that plugin itself, and thus should be elsewhere (libmediactl, for instance). I didn't know about that effort, and having written something close to that in the past but without finishing it, I posted mine here as well.
>>>
>>> The common subset of functionality is limited to parsing text based link configurations. Most of that is really implemented in libmediactl.
>>
>> Hmm, I guess we need to sort this out then before merging Jacek's plugin.
>>
>> Jacek, have you  looked into using (and if necessary extending) libmediactl
>> inside your plugin ?
> 
> Not yet, temporarily I had to switch to different task, but I will
> look into libmediactl soon to figure out how to use its API
> in my plugin and what parts of my code could be added to it.

Great, I think it is best to wait with merging your plugin to this is
sorted out. Once that is sorted out I think we should be able to merge
it relatively quickly.

Maybe we should merge libmediactl into v4l-utils then ? Rather then
v4l-utils growing an external dependency on it. Sakari ?

Regards,

Hans