<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, May 1, 2017 at 11:29 PM, Eric Anholt <span dir="ltr"><<a href="mailto:eric@anholt.net" target="_blank">eric@anholt.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="gmail-">Eric Anholt <<a href="mailto:eric@anholt.net">eric@anholt.net</a>> writes:<br>
<br>
> I wrote this code with reference to pixman, though I've only decided to<br>
> cover Linux (what I'm testing) and Android (seems obvious enough).  Linux<br>
> has getauxval() as a cleaner interface to the /proc entry, but it's more<br>
> glibc-specific and I didn't want to add detection for that.<br>
><br>
> This will be used to enable NEON at runtime on ARMv6 builds of vc4.<br>
><br>
> v2: Actually initialize the temp vars in the Android path (noticed by<br>
>     daniels)<br>
> v3: Actually pull in the cpufeatures library (change by robher).<br>
>     Use O_CLOEXEC.  Break out of the loop when we find our feature.<br>
> v4: Drop VFP code, which was confused about what it was detecting and not<br>
>     actually used yet.<br>
<br>
</span>Ping on review of this one.  Clearly several people have looked at it,<br>
so if I don't hear something soon I'll probably push.<br></blockquote><div><br>Yeah I commented on earlier version of this. FWIW:<br><br></div></div>Reviewed-by: Grazvydas Ignotas <<a href="mailto:notasas@gmail.com">notasas@gmail.com</a>><br><br></div></div>