<div dir="ltr">Well, I say it again: it matters not. I don't care how we do it, some ox is going to get gored. <div>The order is much less important than consistency. And ox are tasty. So let's gore one.</div><div><br></div><div>The problem  for me is not that I have an opinion, but that people I respect so highly have</div><div>different opinions. How can very smart people have different opinions? This can't be :-)<br><br>Somebody make a decision and go for it. Just make sure it's type safe so we can catch<div>it when I get it backwards.<br><div><br></div><div>Sorry I have been absent, somehow I got unsubscribed and did not notice for a while ...</div><div><br></div><div>I think it's probably more important to get sparse working for coreboot than this</div><div>order of arguments thing anyway ...</div><div><br></div><div>ron</div></div></div></div><br><div class="gmail_quote">On Wed Feb 18 2015 at 8:58:23 AM Alexandru Gagniuc <<a href="mailto:mr.nuke.me@gmail.com">mr.nuke.me@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Wednesday, February 18, 2015 08:23:08 AM Vadim Bendebury wrote:<br>
> On Wed, Feb 18, 2015 at 7:16 AM, Aaron Durbin via coreboot<br>
><br>
> > As I have noted on <a href="http://review.coreboot.org/#/c/7924/" target="_blank">http://review.coreboot.org/#/<u></u>c/7924/</a> it's very<br>
> > short sighted to go this route. In assembling a coreboot stack (which<br>
> > includes libpayload and the payload itself) the code usually comes<br>
> > from different software systems. Those include libpayload, linux<br>
> > kernel, u-boot, etc. They all have the write(val, addr) semantics. I<br>
> > see no good reason to artificially erect an ever present barrier for<br>
> > integrating code into a coreboot system.<br>
><br>
> This is a great reason to keep those accessors, IMO it tramps other<br>
> considerations voiced on this thread. Let's be consistent with other<br>
> software systems.<br>
><br>
Following that reasoning, we should switch to EFI style so we can better<br>
integrate vendorcode:<br>
<br>
 read32 -> LibVendorMemRead(..., AccessWidth32, ...)<br>
<br>
Doing what other software stacks do without a real reason, and just because<br>
it's trendy...<br>
<br>
Alex<br>
<br>
--<br>
coreboot mailing list: <a href="mailto:coreboot@coreboot.org" target="_blank">coreboot@coreboot.org</a><br>
<a href="http://www.coreboot.org/mailman/listinfo/coreboot" target="_blank">http://www.coreboot.org/<u></u>mailman/listinfo/coreboot</a><br>
</blockquote></div>