[coreboot] r1057 - in coreboot-v3/arch/x86: . intel/core2
Carl-Daniel Hailfinger
c-d.hailfinger.devel.2006 at gmx.net
Wed Nov 26 03:21:45 CET 2008
On 26.11.2008 02:43, Stefan Reinauer wrote:
> svn at coreboot.org wrote:
>
>> Author: hailfinger
>> Date: 2008-11-26 00:39:31 +0100 (Wed, 26 Nov 2008)
>> New Revision: 1057
>>
>> Modified:
>> coreboot-v3/arch/x86/intel/core2/stage0.S
>> coreboot-v3/arch/x86/stage1.c
>> Log:
>> The Core2Duo CAR code did set up the stack incorrectly. In combination
>> with a wrong calling convention of stage1_phase1() this caused stage1 to
>> assume BIST had failed.
>>
>> Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>
>> Acked-by: Ronald G. Minnich <rminnich at gmail.com>
>>
>>
>>
>
> Can you please back this out again? With this code nothing seems to work
> anymore.
>
I kept the added comments and the two initial pushl $0. Committed in r1058.
If it still fails with svn HEAD, feel free to revert the additional
pushl $0. If the code does fail after that, the breakage was introduced
before my commit (or gcc/gas are malfunctioning).
Regards,
Carl-Daniel
--
http://www.hailfinger.org/
More information about the coreboot
mailing list