[coreboot] [PATCH] Factor out common CAR asm snippets
wt at penguintechs.org
Sun Oct 3 01:51:31 CEST 2010
On Sat, Oct 2, 2010 at 4:07 PM, Kevin O'Connor <kevin at koconnor.net> wrote:
> I think the three lines of assembler is easier to understand than
> "enable_l2_cache". Assembler isn't C - the macros defined aren't free
> abstractions. (In particular, it's not clear they clobber %eax.)
I definitely agree that clobbering eax is not apparent from the
macros. Maybe we need a documented calling convention that says the
eax register can be destroyed at this point in the coreboot code?
> I think the bit definitions, msr addresses, port numbers, and special
> addresses should use definitions. For an example of this from
> seabios, see:
This seems reasonable.
More information about the coreboot