[coreboot] [PATCH] Factor out common CAR asm snippets
Kevin O'Connor
kevin at koconnor.net
Sat Oct 2 18:43:23 CEST 2010
On Sat, Oct 02, 2010 at 12:09:31AM +0200, Stefan Reinauer wrote:
> On 10/1/10 11:27 PM, Uwe Hermann wrote:
> > See patch.
> >
> > abuilt-tested.
> >
> >
> > Uwe.
> IMHO the patch makes it really hard to actually see what the code does.
> I think Idwer was recently running into a problem where the use of
> post_code would trash %eax were hard to comprehend. Now we seem have a
> lot more code that comes from somewhere else and touches registers
> without it being obvious to the reader of the code.
>
> Just my 2ct.
Well, lets make it 4cts.
I think the code is easier to read when it is all right there. The
CAR code is doing something very complex - trying to abstract out a
few minor operations I think could further obscure what is occurring.
-Kevin
More information about the coreboot
mailing list