On Wednesday, February 18, 2015 09:16:07 AM Aaron Durbin wrote:
As I have noted on http://review.coreboot.org/#/c/7924/ it's very short sighted to go this route. In assembling a coreboot stack (which includes libpayload and the payload itself) the code usually comes from different software systems. Those include libpayload, linux kernel, u-boot, etc. They all have the write(val, addr) semantics. I see no good reason to artificially erect an ever present barrier for integrating code into a coreboot system.
As Patrick already said, compared to the total effort to integrate external sources, the issue of argument order is insignificant. In the time you spent writing this email, you could have found out how to do it with coccinelle, and could have applied it to any number of sources.
Alex, you've clearly stated your opinion you've not justified a reason for keeping the barrier.
If you think that something as simple as this is a barrier, then you're likely just copypasting code. In that case, I do want a barrier to protect you from yourself, and from putting up code that was no reviewed in its entirety. Really, it's not a barrier.
Alex