Re: [meta-fsl-arm][PATCH] gstreamer1.0-plugins-imx: add i.MX7 support

[Carlos Rafael Giani <dv@pseudoterminal.org>]

Actually, I have started to modify the build scripts to be able to 
enable/disable individual components by using packageconfigs, so the 
timing is nice :)
Also because some people might not need/want the uniaudio decoders for 
example.

I'll see if this can be added quickly. If so, then I'll do a 0.12.1 
release with the updated recipe. I'll let you know.


On 04/04/2016 03:04 PM, Gary Bisson wrote:
> Otavio, All,
>
> On Mon, Apr 4, 2016 at 1:04 PM, Otavio Salvador
> <otavio.salvador@ossystems.com.br> wrote:
>> On Sat, Apr 2, 2016 at 12:58 PM, Gary Bisson
>> <gary.bisson@boundarydevices.com> wrote:
>>> Modify the dependencies to match the platform features. By default
>>> the build will generate the following plugins for all the mx6/mx7
>>> platforms:
>>> - imxpxpvideosink
>>> - imxpxpvideotransform
>>> - imxipuvideotransform
>>> - imxipuvideosink
>>> - imxipucompositor
>>> - imxv4l2videosrc
>>> - imxuniaudiodec
>>> - imxmp3audioenc
>>>
>>> Then all the platforms with GPU support (6QDL, 6SL, 6SX) will also have:
>>> - imxeglvivsink
>>> - imxg2dvideosink
>>> - imxg2dvideotransform
>>> - imxg2dcompositor
>>> Note that only the G2D plugins can work on 6SL.
>>>
>>> Finally, platforms with VPU support (6QDL) will also have:
>>> - imxvpudec
>>> - imxvpuenc_h263
>>> - imxvpuenc_h264
>>> - imxvpuenc_mpeg4
>>> - imxvpuenc_mjpeg
>>>
>>> The pxp and v4l2 plugins have been tested on an i.MX7 Nitrogen7
>>> platform from Boundary Devices.
>>>
>>> Signed-off-by: Gary Bisson <gary.bisson@boundarydevices.com>
>> I do support this change, however it needs to use PACKAGECONFIG flags
>> and also include 'configure' options which enable/disable the
>> features. The features cannot be enabled depending on library
>> detection as it can result in non-deterministic builds.
> I agree this would be the best solution. I think gstreamer-imx doesn't
> offer this kind of configuration options. Not sure how complicated it
> is to implement, I'll let Carlos give his feedback as I'm not familiar
> with waf.
>
> However, if it takes time to implement, I think this patch could still
> be useful as-is for now. Especially since the current recipe already
> relies on on library detection to enable features. Also, this patch
> fixes the dependencies of platforms without VPU such as the i.MX6SX
> and i.MX6SL. How does the build behaves right now if
> gstreamer1.0-plugins-imx is selected for an i.MX6SX platform? I'm not
> sure why it would result in non-deterministic builds, would it be any
> different than the current recipe to that regard?
>
> Regards,
> Gary