All of lore.kernel.org
 help / color / mirror / Atom feed
* [meta-ti][scarthgap][PATCH v3] ti-vendor-kernel-headers: add recipe for TI-only dma-heap.h ioctl
@ 2026-08-28 14:58 Hari Prasath Gujulan Elango
  2026-08-28 15:07 ` PRC Automation
  2026-08-28 17:44 ` Denys Dmytriyenko
  0 siblings, 2 replies; 8+ messages in thread
From: Hari Prasath Gujulan Elango @ 2026-08-28 14:58 UTC (permalink / raw)
  To: reatmon, afd
  Cc: praneeth, denys, meta-ti, s-kapoor, p-gupta5, u-kumar1,
	Hari Prasath Gujulan Elango

The TI vendor kernel's dma-heap.h carries DMA_HEAP_IOCTL_EXPORT in
downstream. The standard header provided by linux-libc-headers' wouldn't
have the custom changes.Copy it straight from the kernel's staged
source tree (STAGING_KERNEL_DIR) into a separate /usr/include/ti/linux
path instead, for userspace code that needs it.

Signed-off-by: Hari Prasath Gujulan Elango <gehariprasath@ti.com>
---
	v1->v2: Address review comments in commit message and recipe description
	v2->v3: Fix the patch subject prefix

 .../ti-vendor-kernel-headers_1.0.bb             | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)
 create mode 100644 meta-ti-bsp/recipes-kernel/ti-vendor-kernel-headers/ti-vendor-kernel-headers_1.0.bb

diff --git a/meta-ti-bsp/recipes-kernel/ti-vendor-kernel-headers/ti-vendor-kernel-headers_1.0.bb b/meta-ti-bsp/recipes-kernel/ti-vendor-kernel-headers/ti-vendor-kernel-headers_1.0.bb
new file mode 100644
index 000000000..5d570689a
--- /dev/null
+++ b/meta-ti-bsp/recipes-kernel/ti-vendor-kernel-headers/ti-vendor-kernel-headers_1.0.bb
@@ -0,0 +1,17 @@
+SUMMARY = "TI vendor kernel uapi headers not present in mainline linux-libc-headers"
+DESCRIPTION = "Copies select UAPI headers straight from the TI vendor kernel source \
+tree into a TI-specific include path (/usr/include/ti/linux), for userspace code \
+that needs them but can't rely on linux-libc-headers carrying TI-only additions."
+LICENSE = "GPL-2.0-only"
+LIC_FILES_CHKSUM = "file://${COMMON_LICENSE_DIR}/GPL-2.0-only;md5=801f80980d171dd6425610833a22dbe6"
+
+PACKAGE_ARCH = "${MACHINE_ARCH}"
+
+do_configure[depends] += "virtual/kernel:do_shared_workdir"
+do_install[depends] += "virtual/kernel:do_shared_workdir"
+
+do_install() {
+    install -d ${D}${includedir}/ti/linux
+    install -m 0644 ${STAGING_KERNEL_DIR}/include/uapi/linux/dma-heap.h ${D}${includedir}/ti/linux/dma-heap.h
+}
+
-- 
2.34.1



^ permalink raw reply related	[flat|nested] 8+ messages in thread

* Re: [meta-ti][scarthgap][PATCH v3] ti-vendor-kernel-headers: add recipe for TI-only dma-heap.h ioctl
  2026-08-28 14:58 [meta-ti][scarthgap][PATCH v3] ti-vendor-kernel-headers: add recipe for TI-only dma-heap.h ioctl Hari Prasath Gujulan Elango
@ 2026-08-28 15:07 ` PRC Automation
  2026-08-28 17:44 ` Denys Dmytriyenko
  1 sibling, 0 replies; 8+ messages in thread
From: PRC Automation @ 2026-08-28 15:07 UTC (permalink / raw)
  To: Hari Prasath Gujulan Elango
  Cc: reatmon, afd, praneeth, denys, meta-ti, s-kapoor, p-gupta5,
	u-kumar1

meta-ti / na / 20260828145834.1304057-1-gehariprasath

PRC Results: PASS

=========================================================
  check-yocto-patches: PASS
=========================================================
Patches
----------------------------------------
All patches passed



=========================================================
  apply-yocto-patch: PASS
=========================================================
scarthgap
=====================
Summary:
- Patch Series: [meta-ti][scarthgap][PATCH v3] ti-vendor-kernel-headers: add recipe for TI-only dma-heap.h ioctl
- Submitter: From: Hari Prasath Gujulan Elango <gehariprasath@ti.com>
- Date: Date: Fri, 28 Aug 2026 20:28:34 +0530
- Num Patches: 1
- Mailing List (public inbox) Commit SHA: 71b70ca82ff70205a5338cbf7645708eb4172576

Applied to:
- Repository: lcpd-prc-meta-ti
- Base Branch: scarthgap-wip
- Commit Author: LCPD Automation Script <lcpdbld@list.ti.com>
- Commit Subject: CI/CD Auto-Merger: cicd.scarthgap.202608190946
- Commit SHA: ef43a90c91d5a9754b2f28f09debd553fbc6a2d5

Patches
----------------------------------------
All patches applied



=========================================================
  check-yocto-repo: PASS
=========================================================
scarthgap
=====================
PASS



=========================================================
  yocto-check-layers: PASS
=========================================================
scarthgap - PASS
=====================
All checks passed





^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [meta-ti][scarthgap][PATCH v3] ti-vendor-kernel-headers: add recipe for TI-only dma-heap.h ioctl
  2026-08-28 14:58 [meta-ti][scarthgap][PATCH v3] ti-vendor-kernel-headers: add recipe for TI-only dma-heap.h ioctl Hari Prasath Gujulan Elango
  2026-08-28 15:07 ` PRC Automation
@ 2026-08-28 17:44 ` Denys Dmytriyenko
  2026-08-28 20:54   ` Ryan Eatmon
  2026-08-29  8:08   ` Hari Prasath G E
  1 sibling, 2 replies; 8+ messages in thread
From: Denys Dmytriyenko @ 2026-08-28 17:44 UTC (permalink / raw)
  To: gehariprasath
  Cc: reatmon, afd, praneeth, denys, meta-ti, s-kapoor, p-gupta5,
	u-kumar1

On Fri, Aug 28, 2026 at 08:28:34PM +0530, Hari Prasath via lists.yoctoproject.org wrote:
> The TI vendor kernel's dma-heap.h carries DMA_HEAP_IOCTL_EXPORT in
> downstream. The standard header provided by linux-libc-headers' wouldn't
> have the custom changes.Copy it straight from the kernel's staged
> source tree (STAGING_KERNEL_DIR) into a separate /usr/include/ti/linux
> path instead, for userspace code that needs it.
> 
> Signed-off-by: Hari Prasath Gujulan Elango <gehariprasath@ti.com>
> ---
> 	v1->v2: Address review comments in commit message and recipe description
> 	v2->v3: Fix the patch subject prefix
> 
>  .../ti-vendor-kernel-headers_1.0.bb             | 17 +++++++++++++++++
>  1 file changed, 17 insertions(+)
>  create mode 100644 meta-ti-bsp/recipes-kernel/ti-vendor-kernel-headers/ti-vendor-kernel-headers_1.0.bb
> 
> diff --git a/meta-ti-bsp/recipes-kernel/ti-vendor-kernel-headers/ti-vendor-kernel-headers_1.0.bb b/meta-ti-bsp/recipes-kernel/ti-vendor-kernel-headers/ti-vendor-kernel-headers_1.0.bb
> new file mode 100644
> index 000000000..5d570689a
> --- /dev/null
> +++ b/meta-ti-bsp/recipes-kernel/ti-vendor-kernel-headers/ti-vendor-kernel-headers_1.0.bb
> @@ -0,0 +1,17 @@
> +SUMMARY = "TI vendor kernel uapi headers not present in mainline linux-libc-headers"
> +DESCRIPTION = "Copies select UAPI headers straight from the TI vendor kernel source \
> +tree into a TI-specific include path (/usr/include/ti/linux), for userspace code \
> +that needs them but can't rely on linux-libc-headers carrying TI-only additions."
> +LICENSE = "GPL-2.0-only"
> +LIC_FILES_CHKSUM = "file://${COMMON_LICENSE_DIR}/GPL-2.0-only;md5=801f80980d171dd6425610833a22dbe6"
> +
> +PACKAGE_ARCH = "${MACHINE_ARCH}"
> +
> +do_configure[depends] += "virtual/kernel:do_shared_workdir"

Why do you need this dependency for do_configure? Moreover, since you don't 
have do_configure nor do_compile, you might want to mark them as "noexec"


> +do_install[depends] += "virtual/kernel:do_shared_workdir"

How do you ensure that virtual/kernel is the TI vendor kernel? If it's not, 
you'll be copying the original unmodified header into the ti/linux location.


> +
> +do_install() {
> +    install -d ${D}${includedir}/ti/linux
> +    install -m 0644 ${STAGING_KERNEL_DIR}/include/uapi/linux/dma-heap.h ${D}${includedir}/ti/linux/dma-heap.h
> +}
> +
> -- 
> 2.34.1


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [meta-ti][scarthgap][PATCH v3] ti-vendor-kernel-headers: add recipe for TI-only dma-heap.h ioctl
  2026-08-28 17:44 ` Denys Dmytriyenko
@ 2026-08-28 20:54   ` Ryan Eatmon
  2026-08-29 20:04     ` Denys Dmytriyenko
       [not found]     ` <18D05F5A209FB29A.873310@lists.yoctoproject.org>
  2026-08-29  8:08   ` Hari Prasath G E
  1 sibling, 2 replies; 8+ messages in thread
From: Ryan Eatmon @ 2026-08-28 20:54 UTC (permalink / raw)
  To: Denys Dmytriyenko, gehariprasath
  Cc: afd, praneeth, denys, meta-ti, s-kapoor, p-gupta5, u-kumar1



On 8/28/2026 12:44 PM, Denys Dmytriyenko wrote:
> On Fri, Aug 28, 2026 at 08:28:34PM +0530, Hari Prasath via lists.yoctoproject.org wrote:
>> The TI vendor kernel's dma-heap.h carries DMA_HEAP_IOCTL_EXPORT in
>> downstream. The standard header provided by linux-libc-headers' wouldn't
>> have the custom changes.Copy it straight from the kernel's staged
>> source tree (STAGING_KERNEL_DIR) into a separate /usr/include/ti/linux
>> path instead, for userspace code that needs it.
>>
>> Signed-off-by: Hari Prasath Gujulan Elango <gehariprasath@ti.com>
>> ---
>> 	v1->v2: Address review comments in commit message and recipe description
>> 	v2->v3: Fix the patch subject prefix
>>
>>   .../ti-vendor-kernel-headers_1.0.bb             | 17 +++++++++++++++++
>>   1 file changed, 17 insertions(+)
>>   create mode 100644 meta-ti-bsp/recipes-kernel/ti-vendor-kernel-headers/ti-vendor-kernel-headers_1.0.bb
>>
>> diff --git a/meta-ti-bsp/recipes-kernel/ti-vendor-kernel-headers/ti-vendor-kernel-headers_1.0.bb b/meta-ti-bsp/recipes-kernel/ti-vendor-kernel-headers/ti-vendor-kernel-headers_1.0.bb
>> new file mode 100644
>> index 000000000..5d570689a
>> --- /dev/null
>> +++ b/meta-ti-bsp/recipes-kernel/ti-vendor-kernel-headers/ti-vendor-kernel-headers_1.0.bb
>> @@ -0,0 +1,17 @@
>> +SUMMARY = "TI vendor kernel uapi headers not present in mainline linux-libc-headers"
>> +DESCRIPTION = "Copies select UAPI headers straight from the TI vendor kernel source \
>> +tree into a TI-specific include path (/usr/include/ti/linux), for userspace code \
>> +that needs them but can't rely on linux-libc-headers carrying TI-only additions."
>> +LICENSE = "GPL-2.0-only"
>> +LIC_FILES_CHKSUM = "file://${COMMON_LICENSE_DIR}/GPL-2.0-only;md5=801f80980d171dd6425610833a22dbe6"
>> +
>> +PACKAGE_ARCH = "${MACHINE_ARCH}"
>> +
>> +do_configure[depends] += "virtual/kernel:do_shared_workdir"
> 
> Why do you need this dependency for do_configure? Moreover, since you don't
> have do_configure nor do_compile, you might want to mark them as "noexec"
> 
> 
>> +do_install[depends] += "virtual/kernel:do_shared_workdir"
> 
> How do you ensure that virtual/kernel is the TI vendor kernel? If it's not,
> you'll be copying the original unmodified header into the ti/linux location.


Should this just be part of the linux-ti-staging recipes?  That way we 
can correctly only include it in the versions that have this file?


>> +
>> +do_install() {
>> +    install -d ${D}${includedir}/ti/linux
>> +    install -m 0644 ${STAGING_KERNEL_DIR}/include/uapi/linux/dma-heap.h ${D}${includedir}/ti/linux/dma-heap.h
>> +}
>> +
>> -- 
>> 2.34.1

-- 
Ryan Eatmon                reatmon@ti.com
-----------------------------------------
Texas Instruments, Inc.  -  LCPD  -  MGTS


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [meta-ti][scarthgap][PATCH v3] ti-vendor-kernel-headers: add recipe for TI-only dma-heap.h ioctl
  2026-08-28 17:44 ` Denys Dmytriyenko
  2026-08-28 20:54   ` Ryan Eatmon
@ 2026-08-29  8:08   ` Hari Prasath G E
  1 sibling, 0 replies; 8+ messages in thread
From: Hari Prasath G E @ 2026-08-29  8:08 UTC (permalink / raw)
  To: Denys Dmytriyenko
  Cc: reatmon, afd, praneeth, denys, meta-ti, s-kapoor, p-gupta5,
	u-kumar1

Hello Denys,

On 8/28/2026 11:14 PM, Denys Dmytriyenko wrote:
> On Fri, Aug 28, 2026 at 08:28:34PM +0530, Hari Prasath via lists.yoctoproject.org wrote:
>> The TI vendor kernel's dma-heap.h carries DMA_HEAP_IOCTL_EXPORT in
>> downstream. The standard header provided by linux-libc-headers' wouldn't
>> have the custom changes.Copy it straight from the kernel's staged
>> source tree (STAGING_KERNEL_DIR) into a separate /usr/include/ti/linux
>> path instead, for userspace code that needs it.
>>
>> Signed-off-by: Hari Prasath Gujulan Elango <gehariprasath@ti.com>
>> ---
>> 	v1->v2: Address review comments in commit message and recipe description
>> 	v2->v3: Fix the patch subject prefix
>>
>>   .../ti-vendor-kernel-headers_1.0.bb             | 17 +++++++++++++++++
>>   1 file changed, 17 insertions(+)
>>   create mode 100644 meta-ti-bsp/recipes-kernel/ti-vendor-kernel-headers/ti-vendor-kernel-headers_1.0.bb
>>
>> diff --git a/meta-ti-bsp/recipes-kernel/ti-vendor-kernel-headers/ti-vendor-kernel-headers_1.0.bb b/meta-ti-bsp/recipes-kernel/ti-vendor-kernel-headers/ti-vendor-kernel-headers_1.0.bb
>> new file mode 100644
>> index 000000000..5d570689a
>> --- /dev/null
>> +++ b/meta-ti-bsp/recipes-kernel/ti-vendor-kernel-headers/ti-vendor-kernel-headers_1.0.bb
>> @@ -0,0 +1,17 @@
>> +SUMMARY = "TI vendor kernel uapi headers not present in mainline linux-libc-headers"
>> +DESCRIPTION = "Copies select UAPI headers straight from the TI vendor kernel source \
>> +tree into a TI-specific include path (/usr/include/ti/linux), for userspace code \
>> +that needs them but can't rely on linux-libc-headers carrying TI-only additions."
>> +LICENSE = "GPL-2.0-only"
>> +LIC_FILES_CHKSUM = "file://${COMMON_LICENSE_DIR}/GPL-2.0-only;md5=801f80980d171dd6425610833a22dbe6"
>> +
>> +PACKAGE_ARCH = "${MACHINE_ARCH}"
>> +
>> +do_configure[depends] += "virtual/kernel:do_shared_workdir"
> 
> Why do you need this dependency for do_configure? Moreover, since you don't
> have do_configure nor do_compile, you might want to mark them as "noexec"
> 
> 

Thanks for catching this, will fix in v4. do_install() is where the 
vendor tree dependency needs to be,not in do_configure(),will drop it 
and mark it  as a noexec.I see  the cpsw9g-eth-fw_git.bb uses the same 
method.

>> +do_install[depends] += "virtual/kernel:do_shared_workdir"
> 
> How do you ensure that virtual/kernel is the TI vendor kernel? If it's not,
> you'll be copying the original unmodified header into the ti/linux location.
> 
> 

okay so virtual/kernel can resolve to linux-ti-mainline or linux-bb.org 
or anything set by BSP_KERNEL_PROVIDER defined in per machine conf ? I 
don't have a complete understanding of how these are organized in our 
Yocto project layers. Let me check how to restrict to only the TI vendor 
kernel where this header file change is patched.

Regards,
Hari

>> +
>> +do_install() {
>> +    install -d ${D}${includedir}/ti/linux
>> +    install -m 0644 ${STAGING_KERNEL_DIR}/include/uapi/linux/dma-heap.h ${D}${includedir}/ti/linux/dma-heap.h
>> +}
>> +
>> -- 
>> 2.34.1



^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [meta-ti][scarthgap][PATCH v3] ti-vendor-kernel-headers: add recipe for TI-only dma-heap.h ioctl
  2026-08-28 20:54   ` Ryan Eatmon
@ 2026-08-29 20:04     ` Denys Dmytriyenko
  2026-08-30 16:28       ` Hari Prasath G E
       [not found]     ` <18D05F5A209FB29A.873310@lists.yoctoproject.org>
  1 sibling, 1 reply; 8+ messages in thread
From: Denys Dmytriyenko @ 2026-08-29 20:04 UTC (permalink / raw)
  To: Ryan Eatmon
  Cc: gehariprasath, afd, praneeth, denys, meta-ti, s-kapoor, p-gupta5,
	u-kumar1

On Fri, Aug 28, 2026 at 03:54:00PM -0500, Ryan Eatmon wrote:
> 
> 
> On 8/28/2026 12:44 PM, Denys Dmytriyenko wrote:
> >On Fri, Aug 28, 2026 at 08:28:34PM +0530, Hari Prasath via lists.yoctoproject.org wrote:
> >>The TI vendor kernel's dma-heap.h carries DMA_HEAP_IOCTL_EXPORT in
> >>downstream. The standard header provided by linux-libc-headers' wouldn't
> >>have the custom changes.Copy it straight from the kernel's staged
> >>source tree (STAGING_KERNEL_DIR) into a separate /usr/include/ti/linux
> >>path instead, for userspace code that needs it.
> >>
> >>Signed-off-by: Hari Prasath Gujulan Elango <gehariprasath@ti.com>
> >>---
> >>	v1->v2: Address review comments in commit message and recipe description
> >>	v2->v3: Fix the patch subject prefix
> >>
> >>  .../ti-vendor-kernel-headers_1.0.bb             | 17 +++++++++++++++++
> >>  1 file changed, 17 insertions(+)
> >>  create mode 100644 meta-ti-bsp/recipes-kernel/ti-vendor-kernel-headers/ti-vendor-kernel-headers_1.0.bb
> >>
> >>diff --git a/meta-ti-bsp/recipes-kernel/ti-vendor-kernel-headers/ti-vendor-kernel-headers_1.0.bb b/meta-ti-bsp/recipes-kernel/ti-vendor-kernel-headers/ti-vendor-kernel-headers_1.0.bb
> >>new file mode 100644
> >>index 000000000..5d570689a
> >>--- /dev/null
> >>+++ b/meta-ti-bsp/recipes-kernel/ti-vendor-kernel-headers/ti-vendor-kernel-headers_1.0.bb
> >>@@ -0,0 +1,17 @@
> >>+SUMMARY = "TI vendor kernel uapi headers not present in mainline linux-libc-headers"
> >>+DESCRIPTION = "Copies select UAPI headers straight from the TI vendor kernel source \
> >>+tree into a TI-specific include path (/usr/include/ti/linux), for userspace code \
> >>+that needs them but can't rely on linux-libc-headers carrying TI-only additions."
> >>+LICENSE = "GPL-2.0-only"
> >>+LIC_FILES_CHKSUM = "file://${COMMON_LICENSE_DIR}/GPL-2.0-only;md5=801f80980d171dd6425610833a22dbe6"
> >>+
> >>+PACKAGE_ARCH = "${MACHINE_ARCH}"
> >>+
> >>+do_configure[depends] += "virtual/kernel:do_shared_workdir"
> >
> >Why do you need this dependency for do_configure? Moreover, since you don't
> >have do_configure nor do_compile, you might want to mark them as "noexec"
> >
> >
> >>+do_install[depends] += "virtual/kernel:do_shared_workdir"
> >
> >How do you ensure that virtual/kernel is the TI vendor kernel? If it's not,
> >you'll be copying the original unmodified header into the ti/linux location.
> 
> 
> Should this just be part of the linux-ti-staging recipes?  That way
> we can correctly only include it in the versions that have this
> file?

Yeah, probably the easiest and cleanest approach...


> >>+
> >>+do_install() {
> >>+    install -d ${D}${includedir}/ti/linux
> >>+    install -m 0644 ${STAGING_KERNEL_DIR}/include/uapi/linux/dma-heap.h ${D}${includedir}/ti/linux/dma-heap.h
> >>+}
> >>+
> >>-- 
> >>2.34.1


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [meta-ti][scarthgap][PATCH v3] ti-vendor-kernel-headers: add recipe for TI-only dma-heap.h ioctl
       [not found]     ` <18D05F5A209FB29A.873310@lists.yoctoproject.org>
@ 2026-08-29 20:18       ` Denys Dmytriyenko
  0 siblings, 0 replies; 8+ messages in thread
From: Denys Dmytriyenko @ 2026-08-29 20:18 UTC (permalink / raw)
  To: Ryan Eatmon
  Cc: gehariprasath, afd, praneeth, denys, meta-ti, s-kapoor, p-gupta5,
	u-kumar1

On Sat, Aug 29, 2026 at 04:04:40PM -0400, Denys Dmytriyenko wrote:
> On Fri, Aug 28, 2026 at 03:54:00PM -0500, Ryan Eatmon wrote:
> > 
> > 
> > On 8/28/2026 12:44 PM, Denys Dmytriyenko wrote:
> > >On Fri, Aug 28, 2026 at 08:28:34PM +0530, Hari Prasath via lists.yoctoproject.org wrote:
> > >>The TI vendor kernel's dma-heap.h carries DMA_HEAP_IOCTL_EXPORT in
> > >>downstream. The standard header provided by linux-libc-headers' wouldn't
> > >>have the custom changes.Copy it straight from the kernel's staged
> > >>source tree (STAGING_KERNEL_DIR) into a separate /usr/include/ti/linux
> > >>path instead, for userspace code that needs it.
> > >>
> > >>Signed-off-by: Hari Prasath Gujulan Elango <gehariprasath@ti.com>
> > >>---
> > >>	v1->v2: Address review comments in commit message and recipe description
> > >>	v2->v3: Fix the patch subject prefix
> > >>
> > >>  .../ti-vendor-kernel-headers_1.0.bb             | 17 +++++++++++++++++
> > >>  1 file changed, 17 insertions(+)
> > >>  create mode 100644 meta-ti-bsp/recipes-kernel/ti-vendor-kernel-headers/ti-vendor-kernel-headers_1.0.bb
> > >>
> > >>diff --git a/meta-ti-bsp/recipes-kernel/ti-vendor-kernel-headers/ti-vendor-kernel-headers_1.0.bb b/meta-ti-bsp/recipes-kernel/ti-vendor-kernel-headers/ti-vendor-kernel-headers_1.0.bb
> > >>new file mode 100644
> > >>index 000000000..5d570689a
> > >>--- /dev/null
> > >>+++ b/meta-ti-bsp/recipes-kernel/ti-vendor-kernel-headers/ti-vendor-kernel-headers_1.0.bb
> > >>@@ -0,0 +1,17 @@
> > >>+SUMMARY = "TI vendor kernel uapi headers not present in mainline linux-libc-headers"
> > >>+DESCRIPTION = "Copies select UAPI headers straight from the TI vendor kernel source \
> > >>+tree into a TI-specific include path (/usr/include/ti/linux), for userspace code \
> > >>+that needs them but can't rely on linux-libc-headers carrying TI-only additions."
> > >>+LICENSE = "GPL-2.0-only"
> > >>+LIC_FILES_CHKSUM = "file://${COMMON_LICENSE_DIR}/GPL-2.0-only;md5=801f80980d171dd6425610833a22dbe6"
> > >>+
> > >>+PACKAGE_ARCH = "${MACHINE_ARCH}"
> > >>+
> > >>+do_configure[depends] += "virtual/kernel:do_shared_workdir"
> > >
> > >Why do you need this dependency for do_configure? Moreover, since you don't
> > >have do_configure nor do_compile, you might want to mark them as "noexec"
> > >
> > >
> > >>+do_install[depends] += "virtual/kernel:do_shared_workdir"
> > >
> > >How do you ensure that virtual/kernel is the TI vendor kernel? If it's not,
> > >you'll be copying the original unmodified header into the ti/linux location.
> > 
> > 
> > Should this just be part of the linux-ti-staging recipes?  That way
> > we can correctly only include it in the versions that have this
> > file?
> 
> Yeah, probably the easiest and cleanest approach...

BTW, just realized these patches are all marked as scarthgap ONLY. Should 
probably go the whole circle master -> wrynose -> scarthgap...


> > >>+
> > >>+do_install() {
> > >>+    install -d ${D}${includedir}/ti/linux
> > >>+    install -m 0644 ${STAGING_KERNEL_DIR}/include/uapi/linux/dma-heap.h ${D}${includedir}/ti/linux/dma-heap.h
> > >>+}
> > >>+
> > >>-- 
> > >>2.34.1


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [meta-ti][scarthgap][PATCH v3] ti-vendor-kernel-headers: add recipe for TI-only dma-heap.h ioctl
  2026-08-29 20:04     ` Denys Dmytriyenko
@ 2026-08-30 16:28       ` Hari Prasath G E
  0 siblings, 0 replies; 8+ messages in thread
From: Hari Prasath G E @ 2026-08-30 16:28 UTC (permalink / raw)
  To: Denys Dmytriyenko, Ryan Eatmon
  Cc: afd, praneeth, denys, meta-ti, s-kapoor, p-gupta5, u-kumar1

On 8/30/2026 1:34 AM, Denys Dmytriyenko wrote:
> On Fri, Aug 28, 2026 at 03:54:00PM -0500, Ryan Eatmon wrote:
>>
>>
>> On 8/28/2026 12:44 PM, Denys Dmytriyenko wrote:
>>> On Fri, Aug 28, 2026 at 08:28:34PM +0530, Hari Prasath via lists.yoctoproject.org wrote:
>>>> The TI vendor kernel's dma-heap.h carries DMA_HEAP_IOCTL_EXPORT in
>>>> downstream. The standard header provided by linux-libc-headers' wouldn't
>>>> have the custom changes.Copy it straight from the kernel's staged
>>>> source tree (STAGING_KERNEL_DIR) into a separate /usr/include/ti/linux
>>>> path instead, for userspace code that needs it.
>>>>
>>>> Signed-off-by: Hari Prasath Gujulan Elango <gehariprasath@ti.com>
>>>> ---
>>>> 	v1->v2: Address review comments in commit message and recipe description
>>>> 	v2->v3: Fix the patch subject prefix
>>>>
>>>>   .../ti-vendor-kernel-headers_1.0.bb             | 17 +++++++++++++++++
>>>>   1 file changed, 17 insertions(+)
>>>>   create mode 100644 meta-ti-bsp/recipes-kernel/ti-vendor-kernel-headers/ti-vendor-kernel-headers_1.0.bb
>>>>
>>>> diff --git a/meta-ti-bsp/recipes-kernel/ti-vendor-kernel-headers/ti-vendor-kernel-headers_1.0.bb b/meta-ti-bsp/recipes-kernel/ti-vendor-kernel-headers/ti-vendor-kernel-headers_1.0.bb
>>>> new file mode 100644
>>>> index 000000000..5d570689a
>>>> --- /dev/null
>>>> +++ b/meta-ti-bsp/recipes-kernel/ti-vendor-kernel-headers/ti-vendor-kernel-headers_1.0.bb
>>>> @@ -0,0 +1,17 @@
>>>> +SUMMARY = "TI vendor kernel uapi headers not present in mainline linux-libc-headers"
>>>> +DESCRIPTION = "Copies select UAPI headers straight from the TI vendor kernel source \
>>>> +tree into a TI-specific include path (/usr/include/ti/linux), for userspace code \
>>>> +that needs them but can't rely on linux-libc-headers carrying TI-only additions."
>>>> +LICENSE = "GPL-2.0-only"
>>>> +LIC_FILES_CHKSUM = "file://${COMMON_LICENSE_DIR}/GPL-2.0-only;md5=801f80980d171dd6425610833a22dbe6"
>>>> +
>>>> +PACKAGE_ARCH = "${MACHINE_ARCH}"
>>>> +
>>>> +do_configure[depends] += "virtual/kernel:do_shared_workdir"
>>>
>>> Why do you need this dependency for do_configure? Moreover, since you don't
>>> have do_configure nor do_compile, you might want to mark them as "noexec"
>>>
>>>
>>>> +do_install[depends] += "virtual/kernel:do_shared_workdir"
>>>
>>> How do you ensure that virtual/kernel is the TI vendor kernel? If it's not,
>>> you'll be copying the original unmodified header into the ti/linux location.
>>
>>
>> Should this just be part of the linux-ti-staging recipes?  That way
>> we can correctly only include it in the versions that have this
>> file?
> 
> Yeah, probably the easiest and cleanest approach...
> 
> 

I dropped this patch and instead I added a simple do_install:append() to 
ti-linux-staging_6.12 recipe.The header doesn't get installed into the 
rootfs, but I could see it landing in the kernel-dev's ${D} output 
correctly 
(arago-tmp-default-glibc/work/j784s4_evm-oe-linux/linux-ti-staging/6.12.57+git/image/usr/include/ti/linux/). 
It doesn't get installed in the rootfs since kernel-dev package is a 
build-time-only package.

  Since the only consumer of this header is in meta-edgeai, it makes 
more sense to keep the whole thing self-contained there instead of 
modifying linux-ti-staging upstream.A linux-ti-staging_6.12.bbappend in 
that layer can install the header and split it into its own package, and 
the consuming recipe can RDEPENDS on that package directly. That keeps 
meta-ti-bsp untouched, ties the header to the recipe that actually needs 
it, and avoids shipping it into every image that happens to build this 
kernel.

Saurabh,Puyush let us know your thoughts.

Regards,
Hari

>>>> +
>>>> +do_install() {
>>>> +    install -d ${D}${includedir}/ti/linux
>>>> +    install -m 0644 ${STAGING_KERNEL_DIR}/include/uapi/linux/dma-heap.h ${D}${includedir}/ti/linux/dma-heap.h
>>>> +}
>>>> +
>>>> -- 
>>>> 2.34.1



^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2026-08-30 16:28 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-08-28 14:58 [meta-ti][scarthgap][PATCH v3] ti-vendor-kernel-headers: add recipe for TI-only dma-heap.h ioctl Hari Prasath Gujulan Elango
2026-08-28 15:07 ` PRC Automation
2026-08-28 17:44 ` Denys Dmytriyenko
2026-08-28 20:54   ` Ryan Eatmon
2026-08-29 20:04     ` Denys Dmytriyenko
2026-08-30 16:28       ` Hari Prasath G E
     [not found]     ` <18D05F5A209FB29A.873310@lists.yoctoproject.org>
2026-08-29 20:18       ` Denys Dmytriyenko
2026-08-29  8:08   ` Hari Prasath G E

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.