<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Feb 18, 2015 at 10:52 AM, Julius Werner <span dir="ltr"><<a href="mailto:jwerner@chromium.org" target="_blank">jwerner@chromium.org</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"><span>>> As Patrick already said, compared to the total effort to integrate external<br>
>> sources, the issue of argument order is insignificant. In the time you spent<br>
>> writing this email, you could have found out how to do it with coccinelle, and<br>
>> could have applied it to any number of sources.<br>
><br>
> <a href="http://review.coreboot.org/8483" target="_blank">http://review.coreboot.org/8483</a><br>
<br>
</span>Remember that those other code bases use writel(v, a), not write32(v,<br>
a). Just going half the way by changing the order but not the name<br>
wouldn't be very useful I think.<br></blockquote><div><br></div><div>Yes, fixing the order is far more important.</div><div><br></div><div>I wouldn't even care if we still end up with both write32(a, v) or writel(a, v) in the codebase (or u32 vs. uint32_t), so long the usage is consistent and wrappers are trivial.</div><div><br></div></div>-- <br><div>David Hendricks (dhendrix)<br>Systems Software Engineer, Google Inc.</div>
</div></div>