[meta-freescale] [PATCH 27/32] imx-gpu-viv: Add i.MX 8 support

[Tom Hochstein]

Thanks Gary, Otavio, I'm working on it.

Tom

-----Original Message-----
From: Otavio Salvador [mailto:otavio.salvador at ossystems.com.br] 
Sent: Tuesday, June 26, 2018 11:50 AM
To: Gary Bisson <gary.bisson at boundarydevices.com>; Tom Hochstein <tom.hochstein at nxp.com>
Cc: Otavio Salvador <otavio at ossystems.com.br>; meta-freescale Mailing List <meta-freescale at yoctoproject.org>
Subject: Re: [meta-freescale] [PATCH 27/32] imx-gpu-viv: Add i.MX 8 support

Hello,

On Tue, Jun 26, 2018 at 5:12 AM, Gary Bisson <gary.bisson at boundarydevices.com> wrote:
> Hi Otavio, Tom,
>
> On Mon, Jun 25, 2018 at 04:30:27PM -0300, Otavio Salvador wrote:
>> From: Tom Hochstein <tom.hochstein at nxp.com>
>>
>> Signed-off-by: Tom Hochstein <tom.hochstein at nxp.com>
>> Signed-off-by: Otavio Salvador <otavio at ossystems.com.br>
>> ---
>>
>>  .../imx-gpu-viv/imx-gpu-viv-6.inc             | 37 ++++++++++++++++---
>>  .../imx-gpu-viv_6.2.4.p1.2-aarch64.bb         |  6 +++
>>  2 files changed, 37 insertions(+), 6 deletions(-)  create mode 
>> 100644 recipes-graphics/imx-gpu-viv/imx-gpu-viv_6.2.4.p1.2-aarch64.bb
>>
>> diff --git a/recipes-graphics/imx-gpu-viv/imx-gpu-viv-6.inc 
>> b/recipes-graphics/imx-gpu-viv/imx-gpu-viv-6.inc
>> index 6cc8cbb1..d5023bdf 100644
>> --- a/recipes-graphics/imx-gpu-viv/imx-gpu-viv-6.inc
>> +++ b/recipes-graphics/imx-gpu-viv/imx-gpu-viv-6.inc
>> @@ -12,11 +12,11 @@ DEPENDS += " \
>>         bb.utils.contains('DISTRO_FEATURES',     'x11', 'virtual/libx11 libxdamage libxext libxfixes', \
>>                                                         '', d), d)} \  
>> "
>> -DEPENDS += "libdrm"
>>  DEPENDS += " \
>>      ${@bb.utils.contains('DISTRO_FEATURES',     'x11', 'libxcb libxshmfence libxxf86vm', \
>>                                                         '', d)} \  "
>> +DEPENDS_append_mx8 = " libdrm wayland"
>>
>>  # imx-gpu-viv does not provide everything it needs to for 
>> virtual/libgl  # on x11 backend or on Wayland backend with XWayland support.
>> @@ -29,6 +29,8 @@ EXTRA_PROVIDES_append_imxgpu3d = " \
>>      virtual/libgles1 \
>>      virtual/libgles2 \
>>  "
>> +EXTRA_PROVIDES_append_mx8 = " \
>> +    virtual/libopenvx \
>
> Isn't OpenVx only available on i.MX8QM? Same I think it'd be better 
> not to expose openvx for imx8mq since it's not supported.
> https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww
> .nxp.com%2Fproducts%2Fprocessors-and-microcontrollers%2Fapplications-p
> rocessors%2Fi.mx-applications-processors%2Fi.mx-8-processors%3AIMX8-SE
> RIES&data=02%7C01%7Ctom.hochstein%40nxp.com%7Cc51bc7955e3d489ee1e408d5
> db84dafb%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C6366562860254918
> 12&sdata=Qcx0y%2FoTW3nEg0oqywfLwhE9iKPd7MqjpGPfAMrMseA%3D&reserved=0
>
> Also, it seems the i.MX8M Mini only supports OpenGL ES 2.0, not 3.1 
> and that Vulkan isn't supported either.
>
> I know it complexifies the recipe but it still should matches what the 
> HW is capable of, otherwise users will be lost.
>
> To be honest I'd leave i.MX8M Mini out of the picture for now since 
> only the i.MX8MQ is publicly available.

Agreed. Could you guys prepare a patch on top of this to merge?

I am intending to finish the upgrade of SDK and leave imx8 for a second step so we can work in a smaller patchset.

-- 
Otavio Salvador                             O.S. Systems
https://emea01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fwww.ossystems.com.br&data=02%7C01%7Ctom.hochstein%40nxp.com%7Cc51bc7955e3d489ee1e408d5db84dafb%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C636656286025491812&sdata=vihvMQD5HgsRMFh%2BxHoNa1kF1yCLHSh9pbbcmx%2FqBlU%3D&reserved=0        https://emea01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fcode.ossystems.com.br&data=02%7C01%7Ctom.hochstein%40nxp.com%7Cc51bc7955e3d489ee1e408d5db84dafb%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C636656286025491812&sdata=Awvsh11RXVt903NsRdBVYeq2Cr213BWbhtzZvRaGrTY%3D&reserved=0
Mobile: +55 (53) 9 9981-7854          Mobile: +1 (347) 903-9750