Skip to content

vulkan-loader: iMX8M Mini doesn't support vulkan#1249

Merged
otavio merged 3 commits intoFreescale:masterfrom
quaresmajose:vulkan-loader
Oct 4, 2022
Merged

vulkan-loader: iMX8M Mini doesn't support vulkan#1249
otavio merged 3 commits intoFreescale:masterfrom
quaresmajose:vulkan-loader

Conversation

@quaresmajose
Copy link
Copy Markdown
Member

@quaresmajose quaresmajose commented Oct 3, 2022

Vulkan is disabled on meta-freescale for 8M Mini since ee92b5a, so add the same logic to not recommends libvulkan-imx on this machines.

  • imx-base.inc: Introduce imxvulkan override

All iMX8 modules with the imxgpu override support Vulkan with a
single exception for the iMX8M Mini that uses the mx8mm override.

  • vulkan-loader: use the imxvulkan override in recommends

With this the libvulkan-imx is recommends only when the machine
supports Vulkan.

  • imx-gpu-viv: use the imxvulkan override for libvulkan

Signed-off-by: Jose Quaresma [email protected]

Comment thread recipes-graphics/vulkan/vulkan-loader_%.bbappend Outdated
@thochstein
Copy link
Copy Markdown
Contributor

Note, there's a bigger issue here that we don't support the latest version of Vulkan. To get this working we also need to downgrade to 1.2.182.

@otavio
Copy link
Copy Markdown
Member

otavio commented Oct 3, 2022

The downgrade is for master only, right?

@thochstein
Copy link
Copy Markdown
Contributor

It's for kirkstone as well.

quaresmajose added a commit to quaresmajose/meta-lmp that referenced this pull request Oct 4, 2022
Vulkan is disabled on meta-freescale for 8M Mini since ee92b5a,
so add the same logic to not recommends libvulkan-imx on this machines.
Freescale/meta-freescale@ee92b5a

This append is version specific because it's a workaround until
upstream found and merge a solution then we can drop this patch.
Freescale/meta-freescale#1249

This also fix:
| ERROR: Nothing RPROVIDES 'libvulkan-imx' (but /srv/oe/build/conf/../../layers/openembedded-core/meta/recipes-graphics/vulkan/vulkan-loader_1.3.216.0.bb RDEPENDS on or otherwise requires it)
| ERROR: Required build target 'lmp-base-console-image' has no buildable providers.
| Missing or unbuildable dependency chain was: ['lmp-base-console-image', 'weston', 'virtual/libgles2', 'mesa', 'vulkan-loader', 'libvulkan-imx']

Signed-off-by: Jose Quaresma <[email protected]>
@otavio otavio self-requested a review October 4, 2022 11:55
@thochstein
Copy link
Copy Markdown
Contributor

I'm working on the downgrade.

All iMX8 modules with the imxgpu override support Vulkan with a
single exception for the iMX8M Mini that uses the mx8mm override.

Vulkan is disabled on meta-freescale for 8M Mini since ee92b5a,
so we create the imxvulkan override to use when Vulkan is supported.

Sugested-by: Otavio Salvador <[email protected]>
Signed-off-by: Jose Quaresma <[email protected]>
@quaresmajose quaresmajose changed the title vulkan-loader: mx8mm-nxp-bsp doesn't support vulkan vulkan-loader: iMX8M Mini doesn't support vulkan Oct 4, 2022
Comment thread recipes-graphics/vulkan/vulkan-loader_%.bbappend Outdated
With this the libvulkan-imx is recommends only when the machine
supports Vulkan.

Signed-off-by: Jose Quaresma <[email protected]>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Oct 4, 2022

Successfully created backport PR #1250 for kirkstone.

Copy link
Copy Markdown
Contributor

@thochstein thochstein left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good. I can always follow up if mesa-vulkan-drivers should be removed.

@quaresmajose quaresmajose deleted the vulkan-loader branch December 6, 2022 11:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants