<div dir="ltr"><div style>Tim, you are right.</div><div style><br></div><div style>If I got the location right, than the linux kernel has just a single function for all the vendors [1]. James, was this the place you where looking at?</div>
<div style>As Linux seems to give us those flags without checking for the vendor, the correct solution seems in fact to remove all vendor checks under the</div><div style>__arm__ && __linux__ condition.</div><div style>
<br></div><div style>I attached an updated patch? Would that work for you?</div><div style><br></div><div style>Cheers,</div><div style>Tobi</div><div style><br></div><div style>[1] <a href="http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/arch/arm/kernel/setup.c#n868">http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/arch/arm/kernel/setup.c#n868</a></div>
<div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jun 10, 2013 at 8:41 PM, Tim Northover <span dir="ltr"><<a href="mailto:t.p.northover@gmail.com" target="_blank">t.p.northover@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="im">>> It looks like the feature strings will be the same - and in fact we<br>

>> grabbed them straight from the Linux kernel source which returns the same<br>
>> strings for ARM-like CPUs (QC or ARM).<br>
><br>
> I was unsure if the feature strings will be the same. However, I just looked<br>
> into the Linux code and it seems it always emits the same feature strings<br>
> for both targets. Here an updated patch.<br>
<br>
</div>So why are we checking the implementor at all? This whole code is<br>
under #ifdef "__arm__" and "__linux__". Looking at Wikipedia there are<br>
more ARM implementors to come, and I would be very surprised if Linux<br>
picked a different set of feature names for them. (More surprised than<br>
if they broke our code by renaming a feature, for example).<br>
<br>
Cheers.<br>
<span class=""><font color="#888888"><br>
Tim.<br>
</font></span></blockquote></div><br></div></div>