On Fri, Jul 22, 2016 at 6:52 AM, Khem Raj <raj.k...@gmail.com> wrote: > 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
This version won't. It should be possible to fix that, but it starts to turn into a more extensive change. As Paul mentions, it might be better to support this kind of thing officially (via kernel.bbclass) since allowing a machine specific kernel to install custom header files into sysroot for use by user space recipes is a relatively common situation. >> >> 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