[coreboot] r3762 - trunk/coreboot-v2/src/mainboard/tyan/s2895

svn at coreboot.org svn at coreboot.org
Thu Nov 20 20:26:17 CET 2008


Author: stepan
Date: 2008-11-20 20:26:16 +0100 (Thu, 20 Nov 2008)
New Revision: 3762

Modified:
   trunk/coreboot-v2/src/mainboard/tyan/s2895/cache_as_ram_auto.c
Log:
OK, people, watch this.

This is a school book example of why trivial indent patches just suck
big time.

This error was introduced by a trivial self-acked indent patch and was
never detected (because of a missing Config-abuild.lb)

So, indenting the code for no reason can make it a lot worse (read:
break it) instead of improving it. 

I ask everyone to keep this in mind when going on indent-frenzy again.

Signed-off-by: Stefan Reinauer <stepan at coresystems.de>
Acked-by: Stefan Reinauer <stepan at coresystems.de>



Modified: trunk/coreboot-v2/src/mainboard/tyan/s2895/cache_as_ram_auto.c
===================================================================
--- trunk/coreboot-v2/src/mainboard/tyan/s2895/cache_as_ram_auto.c	2008-11-19 13:42:14 UTC (rev 3761)
+++ trunk/coreboot-v2/src/mainboard/tyan/s2895/cache_as_ram_auto.c	2008-11-20 19:26:16 UTC (rev 3762)
@@ -222,9 +222,8 @@
 	#if USE_FALLBACK_IMAGE == 1
 	failover_process(bist, cpu_init_detectedx);
 	#endif
-#endif
 	real_main(bist, cpu_init_detectedx);
-
+#endif
 }
 
 #if USE_FAILOVER_IMAGE==0





More information about the coreboot mailing list