[PATCH] early_mtrr_init is broken, comment it out

[Carl-Daniel Hailfinger]

early_mtrr_init() nukes all MTRRs including those which we use for CAR.
That means if it ever gets called while CAR is active, our stack will be
dropped and mayhem will ensue.
Besides that, it just replicated stage0.S functionality which already
nukes all MTRRs before enabling CAR.

We could move XIP ROM caching from stage0.S to early_mtrr_init(). It
would probably slow down booting a bit, but the amount of asm code would
be reduced.

Thanks to Corey for telling me that early_mtrr_init() broke booting for him.

Signed-off-by: Carl-Daniel Hailfinger

Index: corebootv3-kill_early_mtrr_init/arch/x86/stage1_mtrr.c
===================================================================
--- corebootv3-kill_early_mtrr_init/arch/x86/stage1_mtrr.c	(Revision 1066)
+++ corebootv3-kill_early_mtrr_init/arch/x86/stage1_mtrr.c	(Arbeitskopie)
@@ -67,6 +67,11 @@
    enable_cache();
 }

+/* early_mtrr_init was only usable for ROMCC compiled code. It nukes CAR.
+ * The only remaining purpose would be to enable ROM caching here instead of
+ * in stage0.S and even that is debatable.
+ */
+#if 0
 /* the fixed and variable MTTRs are power-up with random values,
  * clear them to MTRR_TYPE_UNCACHEABLE for safty.
  */
@@ -130,6 +135,7 @@
    do_early_mtrr_init(mtrr_msrs);
    enable_cache();
 }
+#endif

int early_mtrr_init_detected(void)
 {

[ron minnich]

Acked-by: Ronald G. Minnich

[Carl-Daniel Hailfinger]

On 10.12.2008 19:17, ron minnich wrote:
> Acked-by: Ronald G. Minnich

Thanks, r1067.

Regards,
Carl-Daniel

[Peter Stuge]

Carl-Daniel Hailfinger wrote:
> The only remaining purpose would be to enable ROM caching here
> instead of in stage0.S and even that is debatable.
> */
>
> +#if 0

In that case might as well remove the function. If you'd like to do
that, it is:

Acked-by: Peter Stuge