[Buildroot] [PATCH 10/10] gst1-imx: bump to version 0.12.0

Gary Bisson gary.bisson at boundarydevices.com
Mon Feb 1 22:49:27 UTC 2016


Thomas, All,

On Mon, Feb 1, 2016 at 11:27 PM, Thomas Petazzoni
<thomas.petazzoni at free-electrons.com> wrote:
> Hello,
>
> On Mon, 1 Feb 2016 22:20:36 +0100, Gary Bisson wrote:
>
>> > So it no longer depends on the GPU stuff ?
>>
>> No, you can build this package without the GPU backend which implies
>> the GPU sinks won't be built. The end goal is for i.MX7 which doesn't
>> have any GPU, right now I have to include the GPU binaries just to
>> build the pxp/v4l plugins.
>
> ACK, makes sense.
>
>> >> -     depends on BR2_PACKAGE_LIBFSLVPUWRAP
>> >>       select BR2_PACKAGE_GST1_PLUGINS_BASE
>> >> +     select BR2_PACKAGE_GST1_IMX_IPU_PLUGIN
>> >> +     select BR2_PACKAGE_GST1_IMX_PXP_PLUGIN
>> >
>> > This is weird. If you "select" these options here, it means that there
>> > is no way to disable those options. So why are they options in the
>> > first place ?
>>
>> I just wanted to make it explicit that the package will at least build
>> those two plugins. Then leaving it up to the user to select whichever
>> plugin he wants. There actually is no option to disable plugins from
>> the packages, it's all a question of dependency. As soon as the i.MX
>> linux kernel is built, PXP and IPU will be. As soon as the GPU
>> libraries are includes, GPU sink plugins will be built.
>
> Hum, then it is not good, because it means that even if you disable the
> GPU sink plugin options, but still have the GPU libraries enabled, the
> GPU sink plugins will be installed on your target. This is very
> confusing.

Yes, I understand, is it really a big deal? There's going to be more
plugins than expected in the rootfs, it's free!

> I think you should remove the sub-options, and then simply expand the
> Config.in help text of the main option to say:
>
>  - The IPU and PXP plugins are always built.
>  - The GPU sink plugin is built when ... is enabled.
>  - The ... plugin is built when ... is enabled.

This would be ever more confusing in my opinion. It means that when
you select gstreamer-imx you have no idea of what is going to be
built. Then when you realize you need the graphics libraries you have
to browse to select it yourself.

Wouldn't it be possible to force the option value when the IMX_GPU_VIV
package is selected? I guess that would bring a circular dependency
but at least someone wouldn't be able to remove the option without
removing the graphics binaries.

Regards,
Gary



More information about the buildroot mailing list