[meta-freescale] [PATCH 50/52] imx-vpu-hantro: Update to 1.7.0

[Gary Bisson]

Hi Tom,

On Fri, Aug 24, 2018 at 9:45 PM, Tom Hochstein <tom.hochstein at nxp.com>
wrote:

>
>
> > -----Original Message-----
> > From: Tom Hochstein
> > Sent: Tuesday, July 31, 2018 2:44 PM
>
>
> > > -----Original Message-----
> > > From: Gary Bisson [mailto:gary.bisson at boundarydevices.com]
> > > 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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.yoctoproject.org/pipermail/meta-freescale/attachments/20180827/33db1eb2/attachment.html>