<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
  <meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
</head>
<body bgcolor="#ffffff" text="#000000">
On 22.04.2009 18:30 Uhr, Stefan Reinauer wrote:
<blockquote cite="mid:49EF4637.3030509@coresystems.de" type="cite">
  <meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
On 22.04.2009 18:00 Uhr, Peter Stuge wrote:
  <blockquote cite="mid:20090422160029.12113.qmail@stuge.se" type="cite"></blockquote>
</blockquote>
<blockquote cite="mid:49EF4637.3030509@coresystems.de" type="cite">
  <blockquote cite="mid:20090422160029.12113.qmail@stuge.se" type="cite">
    <blockquote type="cite">
      <pre wrap="">+#if MEM_TRAIN_SEQ != 0
+       #error "So far it works on AMD and MEM_TRAIN_SEQ == 0"
+#endif
+
+#if _RAMBASE < 0x1F00000
+       #error "For ACPI RESUME you need to have _RAMBASE at least 31MB"
+       #error "Chipset support (S3_NVRAM_EARLY and ACPI_IS_WAKEUP_EARLY functions and memory ctrl)"
+       #error "And coreboot memory reserved in mainboard.c"
+#endif
+#endif
    </pre>
    </blockquote>
+#if 0<br>
    <pre wrap=""><!---->
Why disable these build time checks?


//Peter

  </pre>
  </blockquote>
Because they don't belong in i386 generic code either.. They're
mainboard or chipset or CPU specific limitations. Many chipsets have no
variable named MEM_TRAIN_SEQ<br>
</blockquote>
For a better explanation: The code should go away or go to the right
place. I have a lot less bellyache if they're in arch/i386/boot/acpi
but that's not their final location.  Maybe the checks should go away
completely. Or to the AMD northbridge directory? <br>
<br>
But before I deleted it, I  left it in, commented out. One thing for
sure, their place is NOT hardwaremain.c<br>
<br>
<pre class="moz-signature" cols="72">-- 
coresystems GmbH • Brahmsstr. 16 • D-79104 Freiburg i. Br.
      Tel.: +49 761 7668825 • Fax: +49 761 7664613
Email: <a class="moz-txt-link-abbreviated" href="mailto:info@coresystems.de">info@coresystems.de</a>  • <a class="moz-txt-link-freetext" href="http://www.coresystems.de/">http://www.coresystems.de/</a>
Registergericht: Amtsgericht Freiburg • HRB 7656
Geschäftsführer: Stefan Reinauer • Ust-IdNr.: DE245674866
</pre>
</body>
</html>