On Fri, Jul 22, 2016 at 12:09 AM, Andre McCurdy <armccu...@gmail.com> wrote:
> On Thu, Jul 21, 2016 at 8:20 PM, Khem Raj <raj.k...@gmail.com> wrote:
>>
>>> On Jul 21, 2016, at 6:45 PM, Paul Eggleton <paul.eggle...@linux.intel.com>
>>> wrote:
>>>
>>> On Thu, 21 Jul 2016 18:35:25 Khem Raj wrote:
>>>> On Thu, Jul 21, 2016 at 6:32 PM, Paul Eggleton
>>>>
>>>> <paul.eggle...@linux.intel.com> wrote:
>>>>> On Thu, 21 Jul 2016 18:22:32 Khem Raj wrote:
>>>>>>> On Jul 21, 2016, at 5:20 PM, Vijayakumar Badiger
>>>>>>> <vijayakuma...@gmail.com>
>>>>>>> wrote:
>>>>>>>
>>>>>>> The sync driver is already ported in the kernel. I just need to get
>>>>>>> this
>>>>>>> compilation error fixed. I tried adding below change to kernel bb file
>>>>>>> but still I get that same error.
>>>>>>>
>>>>>>> PACKAGES =+ "kernel-headers"
>>>>>>> FILES_kernel-headers = "${KDIR}/usr/include”
>>>>>>
>>>>>> there is a different recipe for headers linux-libc-headers that also
>>>>>> needs
>>>>>> to be patched and then hopefully kernel build system is exporting this
>>>>>> header for userspace automatically and you will be set.
>>>>>
>>>>> I thought we weren't supposed to be encouraging modifying
>>>>> linux-libc-headers - especially as this isn't for the libc - it's for
>>>>> other userspace code...?
>>>>
>>>> Yes thats correct thats why
>>>> I am not asking for submitting this patch upstream.
>>>
>>> Sure, but this question comes up a lot. We ought to be telling people the
>>> correct mechanism for introducing extra headers such as this if it isn't to
>>> modify linux-libc-headers.
>>
>> Thats possible for external kernel modules. This is a kernel patch. There is
>> no other elegant way
>> of solving this. They could write another recipe just to stage this header.
>> but thats equally ugly.
>
> A new recipe isn't needed. You can add something like the following to
> the existing kernel recipe:
>
> sysroot_stage_all_append () {
> mkdir -p ${SYSROOT_DESTDIR}/${includedir}/linux
> install -m644 ${S}/include/linux/sync.h
> ${SYSROOT_DESTDIR}/${includedir}/linux/
> }
I wonder if this will this work with sstate
>
> and then add a dependency on the kernel to the recipe which needs the headers.
>
>
>>>
>>> Cheers,
>>> Paul
>>>
>>> --
>>>
>>> Paul Eggleton
>>> Intel Open Source Technology Centre
>>
>>
>> --
>> _______________________________________________
>> yocto mailing list
>> yocto@yoctoproject.org
>> https://lists.yoctoproject.org/listinfo/yocto
>>
--
_______________________________________________
yocto mailing list
yocto@yoctoproject.org
https://lists.yoctoproject.org/listinfo/yocto
