[meta-ti][kirkstone][PATCHv2 0/4] Enable vulkan proper

[meta-ti][kirkstone][PATCHv2 0/4] Enable vulkan proper

[rs]

From: Randolph Sapp <rs@ti.com>

Properly enable Vulkan with all WSI extensions enabled. Attempt some
light packaging changes to the graphics umlibs recipe so we can decrease
the number of check we have to skip, and the checks become a little more
targeted.

This version uses a cairo append that conditionally tweaks PACKAGECONFIG
flags based on the GPU driver that is selected. We're using this driver
flag instead of a machine feature because the open source driver will
add a whole new set of preferred configs that will be optimized later.

Randolph Sapp (4):
  ti-img-rogue-umlibs: bump srcrev, add vulkan, update deps
  pvr-mesa: add new vulkan configuration options and wsi
  cairo: resolve opengl version
  ti-img-rogue-umlibs: package firmware separately

 .../recipes-graphics/cairo/cairo_%.bbappend   |  1 +
 .../recipes-graphics/cairo/pvr-cairo.inc      |  5 +++
 .../ti-img-rogue-umlibs_23.1.6404501.bb       | 33 +++++++++++++++----
 .../recipes-graphics/mesa/pvr-mesa.inc        |  6 +++-
 4 files changed, 37 insertions(+), 8 deletions(-)
 create mode 100644 meta-ti-bsp/recipes-graphics/cairo/cairo_%.bbappend
 create mode 100644 meta-ti-bsp/recipes-graphics/cairo/pvr-cairo.inc

-- 
2.40.0

[meta-ti][kirkstone][PATCHv2 1/4] ti-img-rogue-umlibs: bump srcrev, add vulkan, update deps

[rs]

From: Randolph Sapp <rs@ti.com>

Update the source revision for the following upstream changes:
	- Drop the included ICD loader. Use the Mesa/Khronos one
	  instead.
	- Enable all vulkan window system extensions

Separate the vulkan libs into their own package so people can choose to
ignore them if they want, considering the new runtime deps.

Signed-off-by: Randolph Sapp <rs@ti.com>
---
 .../ti-img-rogue-umlibs_23.1.6404501.bb       | 27 ++++++++++++++-----
 1 file changed, 21 insertions(+), 6 deletions(-)

diff --git a/meta-ti-bsp/recipes-graphics/libgles/ti-img-rogue-umlibs_23.1.6404501.bb b/meta-ti-bsp/recipes-graphics/libgles/ti-img-rogue-umlibs_23.1.6404501.bb
index 3ad0cce6..9068d341 100644
--- a/meta-ti-bsp/recipes-graphics/libgles/ti-img-rogue-umlibs_23.1.6404501.bb
+++ b/meta-ti-bsp/recipes-graphics/libgles/ti-img-rogue-umlibs_23.1.6404501.bb
@@ -12,7 +12,7 @@ PR = "r2"
 
 BRANCH = "linuxws/kirkstone/k6.1/${PV}"
 SRC_URI = "git://git.ti.com/git/graphics/ti-img-rogue-umlibs.git;protocol=https;branch=${BRANCH}"
-SRCREV = "49958a8e820a321d22ac3c635bd21d4a3118c006"
+SRCREV = "452b0f50d2984171c81d5a0d3b22198177d919ad"
 S = "${WORKDIR}/git/targetfs/${TARGET_PRODUCT}/${PVR_WS}/${PVR_BUILD}"
 
 TARGET_PRODUCT:j721e = "j721e_linux"
@@ -30,14 +30,29 @@ do_install:append() {
         rmdir ${D}/lib
     fi
     rm -rf "${D}/etc/init.d"
-    rm -rf "${D}/usr/lib/libvulkan.so"
-    rm -rf "${D}/usr/lib/libvulkan.so.0"
-    rm -rf "${D}/usr/lib/libvulkan.so.1"
 }
 
-PACKAGES = "${PN}-tools ${PN}"
+PACKAGES = "${PN}-vulkan ${PN}-tools ${PN}"
+
+FILES:${PN}-vulkan = " \
+    ${datadir}/vulkan \
+    ${libdir}/libVK_IMG.so* \
+"
+RDEPENDS:${PN}-vulkan += " \
+    mesa-vulkan-drivers \
+    libdrm \
+    ti-img-rogue-driver \
+    libx11-xcb \
+    wayland \
+"
+INSANE_SKIP:${PN}-vulkan += " \
+    already-stripped \
+    dev-so \
+"
+
 FILES:${PN}-tools = "${bindir}/"
 RDEPENDS:${PN}-tools = "python3-core"
-RRECOMMENDS:${PN} += "${PN}-tools"
+
+RRECOMMENDS:${PN} += "${PN}-vulkan ${PN}-tools"
 
 INSANE_SKIP:${PN} += "ldflags arch already-stripped dev-so"
-- 
2.40.0

[meta-ti][kirkstone][PATCHv2 2/4] pvr-mesa: add new vulkan configuration options and wsi

[rs]

From: Randolph Sapp <rs@ti.com>

Add the required vulkan configuration options and the new PVR WSI file to
the corresponding package.

Also bump the PV to match the version of Mesa actually being shipped.

Signed-off-by: Randolph Sapp <rs@ti.com>
---
 meta-ti-bsp/recipes-graphics/mesa/pvr-mesa.inc | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/meta-ti-bsp/recipes-graphics/mesa/pvr-mesa.inc b/meta-ti-bsp/recipes-graphics/mesa/pvr-mesa.inc
index d938f41c..caf94156 100644
--- a/meta-ti-bsp/recipes-graphics/mesa/pvr-mesa.inc
+++ b/meta-ti-bsp/recipes-graphics/mesa/pvr-mesa.inc
@@ -38,14 +38,18 @@ PACKAGECONFIG:remove = "xvmc"
 PACKAGECONFIG[xvmc] = ""
 
 PACKAGE_ARCH = "${MACHINE_ARCH}"
-PV = "22.0.3+pvr"
+PV = "22.3.5+pvr"
 
 GALLIUMDRIVERS:append = "${@bb.utils.contains('PACKAGECONFIG', 'pvr', ',pvr', '', d)}"
 GALLIUMDRIVERS:append = "${@bb.utils.contains('PACKAGECONFIG', 'sgx', ',sgx', '', d)}"
 
+VULKAN_DRIVERS:append = "${@bb.utils.contains('PACKAGECONFIG', 'pvr', ',pvr', '', d)}"
+
 do_install:append () {
     # remove pvr custom pkgconfig
     rm -rf ${D}${datadir}/pkgconfig
 }
 
+FILES:mesa-vulkan-drivers += " ${libdir}/libpvr_mesa_wsi.so"
+
 RRECOMMENDS:mesa-megadriver:append:class-target = " ${@d.getVar('PREFERRED_PROVIDER_virtual/gpudriver')}"
-- 
2.40.0

[meta-ti][kirkstone][PATCHv2 3/4] cairo: resolve opengl version

[rs]

From: Randolph Sapp <rs@ti.com>

Cairo can either be built for use with OpenGL or OpenGLES. By default it
assumes it can use OpenGL if X11 is provided as a distro feature. This
assumption is not true when using the proprietary SGX / Rogue drivers.

Vulkan libraries need some X libs at runtime now, so we need to manually
override the PACKAGECONFIG for cairo to use OpenGLES.

Signed-off-by: Randolph Sapp <rs@ti.com>
---
 meta-ti-bsp/recipes-graphics/cairo/cairo_%.bbappend | 1 +
 meta-ti-bsp/recipes-graphics/cairo/pvr-cairo.inc    | 5 +++++
 2 files changed, 6 insertions(+)
 create mode 100644 meta-ti-bsp/recipes-graphics/cairo/cairo_%.bbappend
 create mode 100644 meta-ti-bsp/recipes-graphics/cairo/pvr-cairo.inc

diff --git a/meta-ti-bsp/recipes-graphics/cairo/cairo_%.bbappend b/meta-ti-bsp/recipes-graphics/cairo/cairo_%.bbappend
new file mode 100644
index 00000000..459d8d09
--- /dev/null
+++ b/meta-ti-bsp/recipes-graphics/cairo/cairo_%.bbappend
@@ -0,0 +1 @@
+require ${@bb.utils.contains_any('PREFERRED_PROVIDER_virtual/gpudriver', 'ti-img-rogue-driver ti-sgx-ddk-km', 'pvr-cairo.inc', '', d)}
diff --git a/meta-ti-bsp/recipes-graphics/cairo/pvr-cairo.inc b/meta-ti-bsp/recipes-graphics/cairo/pvr-cairo.inc
new file mode 100644
index 00000000..b41f1a01
--- /dev/null
+++ b/meta-ti-bsp/recipes-graphics/cairo/pvr-cairo.inc
@@ -0,0 +1,5 @@
+# This is required since cairo expects the opengl distro feature to refer to
+# the desktop version of OpenGL. Imagination drivers only provide OpenGLES.
+
+PACKAGECONFIG:remove = "opengl"
+PACKAGECONFIG:append = " glesv2"
-- 
2.40.0

[meta-ti][kirkstone][PATCHv2 4/4] ti-img-rogue-umlibs: package firmware separately

[rs]

From: Randolph Sapp <rs@ti.com>

Split the firmware into it's own package so we can make the insane
checks a little more paranoid again.

Signed-off-by: Randolph Sapp <rs@ti.com>
---
 .../libgles/ti-img-rogue-umlibs_23.1.6404501.bb           | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/meta-ti-bsp/recipes-graphics/libgles/ti-img-rogue-umlibs_23.1.6404501.bb b/meta-ti-bsp/recipes-graphics/libgles/ti-img-rogue-umlibs_23.1.6404501.bb
index 9068d341..c33f8ae0 100644
--- a/meta-ti-bsp/recipes-graphics/libgles/ti-img-rogue-umlibs_23.1.6404501.bb
+++ b/meta-ti-bsp/recipes-graphics/libgles/ti-img-rogue-umlibs_23.1.6404501.bb
@@ -32,7 +32,7 @@ do_install:append() {
     rm -rf "${D}/etc/init.d"
 }
 
-PACKAGES = "${PN}-vulkan ${PN}-tools ${PN}"
+PACKAGES = "${PN}-vulkan ${PN}-tools ${PN}-firmware ${PN}"
 
 FILES:${PN}-vulkan = " \
     ${datadir}/vulkan \
@@ -53,6 +53,10 @@ INSANE_SKIP:${PN}-vulkan += " \
 FILES:${PN}-tools = "${bindir}/"
 RDEPENDS:${PN}-tools = "python3-core"
 
+FILES:${PN}-firmware = "${base_libdir}/firmware/*"
+INSANE_SKIP:${PN}-firmware += "arch"
+
 RRECOMMENDS:${PN} += "${PN}-vulkan ${PN}-tools"
+RDEPENDS:${PN} += " ${PN}-firmware"
 
-INSANE_SKIP:${PN} += "ldflags arch already-stripped dev-so"
+INSANE_SKIP:${PN} += "already-stripped dev-so"
-- 
2.40.0

Re: [meta-ti][kirkstone][PATCHv2 0/4] Enable vulkan proper

[Denys Dmytriyenko]

On Thu, Apr 27, 2023 at 04:07:42PM -0500, rs@ti.com wrote:
> From: Randolph Sapp <rs@ti.com>
> 
> Properly enable Vulkan with all WSI extensions enabled. Attempt some
> light packaging changes to the graphics umlibs recipe so we can decrease
> the number of check we have to skip, and the checks become a little more
> targeted.
> 
> This version uses a cairo append that conditionally tweaks PACKAGECONFIG
> flags based on the GPU driver that is selected. We're using this driver
> flag instead of a machine feature because the open source driver will
> add a whole new set of preferred configs that will be optimized later.

LGTM. Is it kirkstone-specific? Any plans for master?


> Randolph Sapp (4):
>   ti-img-rogue-umlibs: bump srcrev, add vulkan, update deps
>   pvr-mesa: add new vulkan configuration options and wsi
>   cairo: resolve opengl version
>   ti-img-rogue-umlibs: package firmware separately
> 
>  .../recipes-graphics/cairo/cairo_%.bbappend   |  1 +
>  .../recipes-graphics/cairo/pvr-cairo.inc      |  5 +++
>  .../ti-img-rogue-umlibs_23.1.6404501.bb       | 33 +++++++++++++++----
>  .../recipes-graphics/mesa/pvr-mesa.inc        |  6 +++-
>  4 files changed, 37 insertions(+), 8 deletions(-)
>  create mode 100644 meta-ti-bsp/recipes-graphics/cairo/cairo_%.bbappend
>  create mode 100644 meta-ti-bsp/recipes-graphics/cairo/pvr-cairo.inc

Re: [EXTERNAL] Re: [meta-ti][kirkstone][PATCHv2 0/4] Enable vulkan proper

[Randolph Sapp]

On 4/27/23 16:50, Denys Dmytriyenko wrote:
> On Thu, Apr 27, 2023 at 04:07:42PM -0500, rs@ti.com wrote:
>> From: Randolph Sapp <rs@ti.com>
>>
>> Properly enable Vulkan with all WSI extensions enabled. Attempt some
>> light packaging changes to the graphics umlibs recipe so we can decrease
>> the number of check we have to skip, and the checks become a little more
>> targeted.
>>
>> This version uses a cairo append that conditionally tweaks PACKAGECONFIG
>> flags based on the GPU driver that is selected. We're using this driver
>> flag instead of a machine feature because the open source driver will
>> add a whole new set of preferred configs that will be optimized later.
> 
> LGTM. Is it kirkstone-specific? Any plans for master?

Yeah, unfortunately this is kirkstone specific for now. Master version 
soon. Just wanted to get this and the meta-arago patches sorted and 
merged first.

> 
>> Randolph Sapp (4):
>>    ti-img-rogue-umlibs: bump srcrev, add vulkan, update deps
>>    pvr-mesa: add new vulkan configuration options and wsi
>>    cairo: resolve opengl version
>>    ti-img-rogue-umlibs: package firmware separately
>>
>>   .../recipes-graphics/cairo/cairo_%.bbappend   |  1 +
>>   .../recipes-graphics/cairo/pvr-cairo.inc      |  5 +++
>>   .../ti-img-rogue-umlibs_23.1.6404501.bb       | 33 +++++++++++++++----
>>   .../recipes-graphics/mesa/pvr-mesa.inc        |  6 +++-
>>   4 files changed, 37 insertions(+), 8 deletions(-)
>>   create mode 100644 meta-ti-bsp/recipes-graphics/cairo/cairo_%.bbappend
>>   create mode 100644 meta-ti-bsp/recipes-graphics/cairo/pvr-cairo.inc

Re: [EXTERNAL] Re: [meta-ti][kirkstone][PATCHv2 0/4] Enable vulkan proper

[Randolph Sapp]

On 4/27/23 16:50, Denys Dmytriyenko wrote:
> On Thu, Apr 27, 2023 at 04:07:42PM -0500, rs@ti.com wrote:
>> From: Randolph Sapp <rs@ti.com>
>>
>> Properly enable Vulkan with all WSI extensions enabled. Attempt some
>> light packaging changes to the graphics umlibs recipe so we can decrease
>> the number of check we have to skip, and the checks become a little more
>> targeted.
>>
>> This version uses a cairo append that conditionally tweaks PACKAGECONFIG
>> flags based on the GPU driver that is selected. We're using this driver
>> flag instead of a machine feature because the open source driver will
>> add a whole new set of preferred configs that will be optimized later.
> 
> LGTM. Is it kirkstone-specific? Any plans for master?

Actually, disregard that last response. It'll apply cleanly to master if 
you also cherry-pick the following patches from kirkstone:
c994d48021664d328ced52829542a3b1b86c4e02
2cc7895ebfea1df4afc2229f9762f129946b0b06
b86fb13fb9638635bf957156833bccc10ddf5393

I must have incorrectly assumed those wouldn't apply to master when I 
submitted them originally. My bad, they apply fine.

> 
>> Randolph Sapp (4):
>>    ti-img-rogue-umlibs: bump srcrev, add vulkan, update deps
>>    pvr-mesa: add new vulkan configuration options and wsi
>>    cairo: resolve opengl version
>>    ti-img-rogue-umlibs: package firmware separately
>>
>>   .../recipes-graphics/cairo/cairo_%.bbappend   |  1 +
>>   .../recipes-graphics/cairo/pvr-cairo.inc      |  5 +++
>>   .../ti-img-rogue-umlibs_23.1.6404501.bb       | 33 +++++++++++++++----
>>   .../recipes-graphics/mesa/pvr-mesa.inc        |  6 +++-
>>   4 files changed, 37 insertions(+), 8 deletions(-)
>>   create mode 100644 meta-ti-bsp/recipes-graphics/cairo/cairo_%.bbappend
>>   create mode 100644 meta-ti-bsp/recipes-graphics/cairo/pvr-cairo.inc

Re: [meta-ti][kirkstone][PATCHv2 0/4] Enable vulkan proper

[Denys Dmytriyenko]

On Thu, Apr 27, 2023 at 06:58:32PM -0500, Randolph Sapp via lists.yoctoproject.org wrote:
> On 4/27/23 16:50, Denys Dmytriyenko wrote:
> >On Thu, Apr 27, 2023 at 04:07:42PM -0500, rs@ti.com wrote:
> >>From: Randolph Sapp <rs@ti.com>
> >>
> >>Properly enable Vulkan with all WSI extensions enabled. Attempt some
> >>light packaging changes to the graphics umlibs recipe so we can decrease
> >>the number of check we have to skip, and the checks become a little more
> >>targeted.
> >>
> >>This version uses a cairo append that conditionally tweaks PACKAGECONFIG
> >>flags based on the GPU driver that is selected. We're using this driver
> >>flag instead of a machine feature because the open source driver will
> >>add a whole new set of preferred configs that will be optimized later.
> >
> >LGTM. Is it kirkstone-specific? Any plans for master?
> 
> Actually, disregard that last response. It'll apply cleanly to
> master if you also cherry-pick the following patches from kirkstone:
> c994d48021664d328ced52829542a3b1b86c4e02
> 2cc7895ebfea1df4afc2229f9762f129946b0b06
> b86fb13fb9638635bf957156833bccc10ddf5393

Can you please be more specific - none of these 3 commits exist in meta-ti nor 
meta-arago...


> I must have incorrectly assumed those wouldn't apply to master when
> I submitted them originally. My bad, they apply fine.
> 
> >
> >>Randolph Sapp (4):
> >>   ti-img-rogue-umlibs: bump srcrev, add vulkan, update deps
> >>   pvr-mesa: add new vulkan configuration options and wsi
> >>   cairo: resolve opengl version
> >>   ti-img-rogue-umlibs: package firmware separately
> >>
> >>  .../recipes-graphics/cairo/cairo_%.bbappend   |  1 +
> >>  .../recipes-graphics/cairo/pvr-cairo.inc      |  5 +++
> >>  .../ti-img-rogue-umlibs_23.1.6404501.bb       | 33 +++++++++++++++----
> >>  .../recipes-graphics/mesa/pvr-mesa.inc        |  6 +++-
> >>  4 files changed, 37 insertions(+), 8 deletions(-)
> >>  create mode 100644 meta-ti-bsp/recipes-graphics/cairo/cairo_%.bbappend
> >>  create mode 100644 meta-ti-bsp/recipes-graphics/cairo/pvr-cairo.inc

Re: [EXTERNAL] Re: [meta-ti][kirkstone][PATCHv2 0/4] Enable vulkan proper

[Randolph Sapp]

On 4/27/23 20:42, Denys Dmytriyenko wrote:
> On Thu, Apr 27, 2023 at 06:58:32PM -0500, Randolph Sapp via lists.yoctoproject.org wrote:
>> On 4/27/23 16:50, Denys Dmytriyenko wrote:
>>> On Thu, Apr 27, 2023 at 04:07:42PM -0500, rs@ti.com wrote:
>>>> From: Randolph Sapp <rs@ti.com>
>>>>
>>>> Properly enable Vulkan with all WSI extensions enabled. Attempt some
>>>> light packaging changes to the graphics umlibs recipe so we can decrease
>>>> the number of check we have to skip, and the checks become a little more
>>>> targeted.
>>>>
>>>> This version uses a cairo append that conditionally tweaks PACKAGECONFIG
>>>> flags based on the GPU driver that is selected. We're using this driver
>>>> flag instead of a machine feature because the open source driver will
>>>> add a whole new set of preferred configs that will be optimized later.
>>>
>>> LGTM. Is it kirkstone-specific? Any plans for master?
>>
>> Actually, disregard that last response. It'll apply cleanly to
>> master if you also cherry-pick the following patches from kirkstone:
>> c994d48021664d328ced52829542a3b1b86c4e02
>> 2cc7895ebfea1df4afc2229f9762f129946b0b06
>> b86fb13fb9638635bf957156833bccc10ddf5393
> 
> Can you please be more specific - none of these 3 commits exist in meta-ti nor
> meta-arago...
> 

Right, my bad. I meant this patch set: 
https://lists.yoctoproject.org/g/meta-ti/message/16397

Or these commits from kirkstone:
16a8078a43eef297449a04d6861fd9811bfeb052
b9581a704648cb3ed70c6bcb28d5a1a89b08534d
c8ba8aaade90101f06164aec1b393d7c8b3aa201

I don't know where I got those previous hashes. Must have done something 
goofy.


> 
>> I must have incorrectly assumed those wouldn't apply to master when
>> I submitted them originally. My bad, they apply fine.
>>
>>>
>>>> Randolph Sapp (4):
>>>>    ti-img-rogue-umlibs: bump srcrev, add vulkan, update deps
>>>>    pvr-mesa: add new vulkan configuration options and wsi
>>>>    cairo: resolve opengl version
>>>>    ti-img-rogue-umlibs: package firmware separately
>>>>
>>>>   .../recipes-graphics/cairo/cairo_%.bbappend   |  1 +
>>>>   .../recipes-graphics/cairo/pvr-cairo.inc      |  5 +++
>>>>   .../ti-img-rogue-umlibs_23.1.6404501.bb       | 33 +++++++++++++++----
>>>>   .../recipes-graphics/mesa/pvr-mesa.inc        |  6 +++-
>>>>   4 files changed, 37 insertions(+), 8 deletions(-)
>>>>   create mode 100644 meta-ti-bsp/recipes-graphics/cairo/cairo_%.bbappend
>>>>   create mode 100644 meta-ti-bsp/recipes-graphics/cairo/pvr-cairo.inc

Re: [EXTERNAL] Re: [meta-ti][kirkstone][PATCHv2 0/4] Enable vulkan proper

[Denys Dmytriyenko]

On Fri, Apr 28, 2023 at 11:30:06AM -0500, Randolph Sapp wrote:
> On 4/27/23 20:42, Denys Dmytriyenko wrote:
> >On Thu, Apr 27, 2023 at 06:58:32PM -0500, Randolph Sapp via lists.yoctoproject.org wrote:
> >>On 4/27/23 16:50, Denys Dmytriyenko wrote:
> >>>On Thu, Apr 27, 2023 at 04:07:42PM -0500, rs@ti.com wrote:
> >>>>From: Randolph Sapp <rs@ti.com>
> >>>>
> >>>>Properly enable Vulkan with all WSI extensions enabled. Attempt some
> >>>>light packaging changes to the graphics umlibs recipe so we can decrease
> >>>>the number of check we have to skip, and the checks become a little more
> >>>>targeted.
> >>>>
> >>>>This version uses a cairo append that conditionally tweaks PACKAGECONFIG
> >>>>flags based on the GPU driver that is selected. We're using this driver
> >>>>flag instead of a machine feature because the open source driver will
> >>>>add a whole new set of preferred configs that will be optimized later.
> >>>
> >>>LGTM. Is it kirkstone-specific? Any plans for master?
> >>
> >>Actually, disregard that last response. It'll apply cleanly to
> >>master if you also cherry-pick the following patches from kirkstone:
> >>c994d48021664d328ced52829542a3b1b86c4e02
> >>2cc7895ebfea1df4afc2229f9762f129946b0b06
> >>b86fb13fb9638635bf957156833bccc10ddf5393
> >
> >Can you please be more specific - none of these 3 commits exist in meta-ti nor
> >meta-arago...
> >
> 
> Right, my bad. I meant this patch set:
> https://lists.yoctoproject.org/g/meta-ti/message/16397
> 
> Or these commits from kirkstone:
> 16a8078a43eef297449a04d6861fd9811bfeb052
> b9581a704648cb3ed70c6bcb28d5a1a89b08534d
> c8ba8aaade90101f06164aec1b393d7c8b3aa201

Thanks. The original patches for these commits were marked specifically for 
kirkstone (same as this series) and I guess the assumption was that they won't 
apply to master and were skipped.

I just cherry-picked them for master and can confirm everything applies and 
builds cleanly - I just re-sent them to the list just in case. Once they are 
in, this Vulkan series can also be applied to master.


> I don't know where I got those previous hashes. Must have done
> something goofy.
> 
> 
> >
> >>I must have incorrectly assumed those wouldn't apply to master when
> >>I submitted them originally. My bad, they apply fine.
> >>
> >>>
> >>>>Randolph Sapp (4):
> >>>>   ti-img-rogue-umlibs: bump srcrev, add vulkan, update deps
> >>>>   pvr-mesa: add new vulkan configuration options and wsi
> >>>>   cairo: resolve opengl version
> >>>>   ti-img-rogue-umlibs: package firmware separately
> >>>>
> >>>>  .../recipes-graphics/cairo/cairo_%.bbappend   |  1 +
> >>>>  .../recipes-graphics/cairo/pvr-cairo.inc      |  5 +++
> >>>>  .../ti-img-rogue-umlibs_23.1.6404501.bb       | 33 +++++++++++++++----
> >>>>  .../recipes-graphics/mesa/pvr-mesa.inc        |  6 +++-
> >>>>  4 files changed, 37 insertions(+), 8 deletions(-)
> >>>>  create mode 100644 meta-ti-bsp/recipes-graphics/cairo/cairo_%.bbappend

Re: [EXTERNAL] Re: [EXTERNAL] Re: [meta-ti][kirkstone][PATCHv2 0/4] Enable vulkan proper

[Randolph Sapp]

On 5/1/23 22:46, Denys Dmytriyenko wrote:
> On Fri, Apr 28, 2023 at 11:30:06AM -0500, Randolph Sapp wrote:
>> On 4/27/23 20:42, Denys Dmytriyenko wrote:
>>> On Thu, Apr 27, 2023 at 06:58:32PM -0500, Randolph Sapp via lists.yoctoproject.org wrote:
>>>> On 4/27/23 16:50, Denys Dmytriyenko wrote:
>>>>> On Thu, Apr 27, 2023 at 04:07:42PM -0500, rs@ti.com wrote:
>>>>>> From: Randolph Sapp <rs@ti.com>
>>>>>>
>>>>>> Properly enable Vulkan with all WSI extensions enabled. Attempt some
>>>>>> light packaging changes to the graphics umlibs recipe so we can decrease
>>>>>> the number of check we have to skip, and the checks become a little more
>>>>>> targeted.
>>>>>>
>>>>>> This version uses a cairo append that conditionally tweaks PACKAGECONFIG
>>>>>> flags based on the GPU driver that is selected. We're using this driver
>>>>>> flag instead of a machine feature because the open source driver will
>>>>>> add a whole new set of preferred configs that will be optimized later.
>>>>>
>>>>> LGTM. Is it kirkstone-specific? Any plans for master?
>>>>
>>>> Actually, disregard that last response. It'll apply cleanly to
>>>> master if you also cherry-pick the following patches from kirkstone:
>>>> c994d48021664d328ced52829542a3b1b86c4e02
>>>> 2cc7895ebfea1df4afc2229f9762f129946b0b06
>>>> b86fb13fb9638635bf957156833bccc10ddf5393
>>>
>>> Can you please be more specific - none of these 3 commits exist in meta-ti nor
>>> meta-arago...
>>>
>>
>> Right, my bad. I meant this patch set:
>> https://lists.yoctoproject.org/g/meta-ti/message/16397
>>
>> Or these commits from kirkstone:
>> 16a8078a43eef297449a04d6861fd9811bfeb052
>> b9581a704648cb3ed70c6bcb28d5a1a89b08534d
>> c8ba8aaade90101f06164aec1b393d7c8b3aa201
> 
> Thanks. The original patches for these commits were marked specifically for
> kirkstone (same as this series) and I guess the assumption was that they won't
> apply to master and were skipped.
> 
> I just cherry-picked them for master and can confirm everything applies and
> builds cleanly - I just re-sent them to the list just in case. Once they are
> in, this Vulkan series can also be applied to master.
> 

Excellent, thanks Denys!

> 
>> I don't know where I got those previous hashes. Must have done
>> something goofy.
>>
>>
>>>
>>>> I must have incorrectly assumed those wouldn't apply to master when
>>>> I submitted them originally. My bad, they apply fine.
>>>>
>>>>>
>>>>>> Randolph Sapp (4):
>>>>>>    ti-img-rogue-umlibs: bump srcrev, add vulkan, update deps
>>>>>>    pvr-mesa: add new vulkan configuration options and wsi
>>>>>>    cairo: resolve opengl version
>>>>>>    ti-img-rogue-umlibs: package firmware separately
>>>>>>
>>>>>>   .../recipes-graphics/cairo/cairo_%.bbappend   |  1 +
>>>>>>   .../recipes-graphics/cairo/pvr-cairo.inc      |  5 +++
>>>>>>   .../ti-img-rogue-umlibs_23.1.6404501.bb       | 33 +++++++++++++++----
>>>>>>   .../recipes-graphics/mesa/pvr-mesa.inc        |  6 +++-
>>>>>>   4 files changed, 37 insertions(+), 8 deletions(-)
>>>>>>   create mode 100644 meta-ti-bsp/recipes-graphics/cairo/cairo_%.bbappend