Hi Tom, On Fri, Aug 24, 2018 at 9:45 PM, Tom Hochstein <[email protected]> wrote:
> > > > -----Original Message----- > > From: Tom Hochstein > > Sent: Tuesday, July 31, 2018 2:44 PM > > > > > -----Original Message----- > > > From: Gary Bisson [mailto:[email protected]] > > > Sent: Tuesday, July 31, 2018 1:54 AM > > > > > > recipes-bsp/imx-vpu-hantro/{imx-vpu-hantro-1.6.0 => > > > > imx-vpu-hantro}/0001-Fix-ion.h-header-inclusion-to-be-standard.patch > > > > (71%) > > > > > > Thanks for porting the patch over. Any chance to get it in the > > > upstream repo in the future? > > > > Thanks for pointing this out. I will get the process started now. > > I have misgivings about this patch. The problem is we have packages that > use ion.h that are applicable to both Android and Linux. Having ion.h live > in two different locations means we have to complicate the packages. > > Isn't it better to just adopt the Android model and install > /usr/include/linux/ion.h in our linux-imx kernel? > Android doesn't actually use ion.h from the kernel but relies on the one present in the AOSP already (original kernel-headers or bionic libc). If you want we can have something like: #ifdef ANDROID #include <linux/ion.h> #include <linux/mxc_ion.h> else #include <uapi/ion.h> #endif That would cover both cases but I understand what you are saying. I just wish the staging headers would be installed normally in the first place. Regards, Gary
-- _______________________________________________ meta-freescale mailing list [email protected] https://lists.yoctoproject.org/listinfo/meta-freescale
