[coreboot] [PATCH] v3: Improve comments in Core2Duo code

Carl-Daniel Hailfinger c-d.hailfinger.devel.2006 at gmx.net
Wed Nov 26 16:51:56 CET 2008


This patch is purely adding comments. I have verified that the compiled
code is bit-for-bit identical.
r1059 was a bit overzealous with comment removal.

If anybody complains that this breaks his setup, I need
build/arch/x86/intel/core2/stage0.o from a working and from a patched
build. If you're seeing differences, your toolchain is broken.

Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>

Index: arch/x86/intel/core2/stage0.S
===================================================================
--- arch/x86/intel/core2/stage0.S	(Revision 1059)
+++ arch/x86/intel/core2/stage0.S	(Arbeitskopie)
@@ -159,9 +159,13 @@
 	movl	$(CACHE_AS_RAM_BASE + CACHE_AS_RAM_SIZE - 4), %eax
 	movl	%eax, %esp
 
-	/* Restore the BIST result */
+	/* Restore the BIST result. */
 	movl	%ebp, %eax
+
+	/* We need to set ebp? No need. */
 	movl	%esp, %ebp
+
+	/* First parameter: bist */
 	pushl	%eax
 
 #if 0
@@ -170,6 +174,7 @@
 #endif
 
 	call	stage1_phase1
+	/* We will not go back. */
 
 	port80_post(0x2f)
 error:


-- 
http://www.hailfinger.org/

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: linuxbios3_core2duo_car_readd_comments.diff
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20081126/1adcd4f6/attachment.ksh>


More information about the coreboot mailing list