[Buildroot] [PATCH 06/16] xdriver_xf86-video-imx-viv: bump to version 6.2.2.p0

[Gary Bisson]

Hi Thomas,

On Wed, Jan 3, 2018 at 9:50 PM, Thomas Petazzoni
<thomas.petazzoni at free-electrons.com> wrote:
> Hello,
>
> On Wed,  3 Jan 2018 19:38:32 +0100, Gary Bisson wrote:
>> The following xorg.conf was used in order to force the use of the
>> vivante module:
>> https://github.com/Freescale/meta-freescale/blob/master/recipes-graphics/xorg-xserver/xserver-xf86-config/mx6/xorg.conf
>>
>> Note that the X server must be started with the "noreset" option to
>> avoid crashes when closing X apps.
>> Tested with:
>>  # X -noreset &
>>  # cd /usr/share/examples/viv_samples/vdk/
>>  # DISPLAY=:0 ./tutorial7
>>
>> Cc: Jérôme Pouiller <jezz at sysmic.org>
>> Cc: Bernd Kuhls <bernd.kuhls at t-online.de>
>> Signed-off-by: Gary Bisson <gary.bisson at boundarydevices.com>
>> ---
>>  ...001-Stop-using-Git-to-write-local-version.patch | 90 ----------------------
>
> Explaining in the commit log why the patch is removed would be nice.

Will do, the patch has been merged upstream basically:
https://source.codeaurora.org/external/imx/xf86-video-imx-vivante/commit/?h=rel_imx_4.9.x_1.0.0_ga&id=70ebd67c6b0d2df3640d91fc67e65a90ddcd70bd

>> diff --git a/package/x11r7/xdriver_xf86-video-imx-viv/Config.in b/package/x11r7/xdriver_xf86-video-imx-viv/Config.in
>> index c3746a100f..5a0257305c 100644
>> --- a/package/x11r7/xdriver_xf86-video-imx-viv/Config.in
>> +++ b/package/x11r7/xdriver_xf86-video-imx-viv/Config.in
>> @@ -1,6 +1,7 @@
>>  config BR2_PACKAGE_XDRIVER_XF86_VIDEO_IMX_VIV
>>       bool "xf86-video-imx-viv"
>>       depends on BR2_PACKAGE_IMX_GPU_VIV_OUTPUT_X11
>> +     depends on BR2_PACKAGE_IMX_GPU_G2D
>
> Perhaps a "select BR2_PACKAGE_IMX_GPU_G2D" would be better.

Well, this part of the i.MX packages is messy, you can't select much
without having recursive dependency.

Here if I try your suggestion I get:
package/x11r7/xdriver_xf86-video-imx-viv/Config.in:1:error: recursive
dependency detected!
package/x11r7/xdriver_xf86-video-imx-viv/Config.in:1: symbol
BR2_PACKAGE_XDRIVER_XF86_VIDEO_IMX_VIV depends on
BR2_PACKAGE_IMX_GPU_VIV_OUTPUT_X11
package/freescale-imx/imx-gpu-viv/Config.in:34: symbol
BR2_PACKAGE_IMX_GPU_VIV_OUTPUT_X11 is part of choice <choice>
package/freescale-imx/imx-gpu-viv/Config.in:27: choice <choice>
contains symbol <choice>
package/freescale-imx/imx-gpu-viv/Config.in:27: choice <choice>
contains symbol BR2_PACKAGE_IMX_GPU_VIV
package/freescale-imx/imx-gpu-viv/Config.in:5: symbol
BR2_PACKAGE_IMX_GPU_VIV is selected by BR2_PACKAGE_IMX_GPU_G2D
package/freescale-imx/imx-gpu-g2d/Config.in:5: symbol
BR2_PACKAGE_IMX_GPU_G2D is selected by
BR2_PACKAGE_XDRIVER_XF86_VIDEO_IMX_VIV

So using depends is the easy path, let me know if you'd like more
investigation on that.

Also, since G2D will be provided by a different package for the i.MX8,
my plan is to later have a virtual/g2d and depend on it.

Then depending on the platform it will either be imx-gpu-g2d or imx-dpu-g2d.

Regards,
Gary