i945 boards: Drop disabled ram_check() calls
This code would not get enabled just by flipping the options in menuconfig, also ramcheck() no longer test the range like the parameters would imply. We should add non-destructive ram_check() on S3 resume path to verify memory controller configuration has been properly recovered. Change-Id: Ie4675c4770146c4312cdfbc81afa19f243f90ee4 Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com> Reviewed-on: http://review.coreboot.org/6027 Tested-by: build bot (Jenkins) Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net> Reviewed-by: Idwer Vollering <vidwer@gmail.com> Reviewed-by: Patrick Georgi <patrick@georgi-clan.de>
This commit is contained in:
parent
bd4553bb4c
commit
206f37043e
|
@ -340,21 +340,6 @@ void main(unsigned long bist)
|
|||
/* Initialize the internal PCIe links before we go into stage2 */
|
||||
i945_late_initialization();
|
||||
|
||||
#if !CONFIG_HAVE_ACPI_RESUME
|
||||
/* When doing resume, we must not overwrite RAM */
|
||||
#if CONFIG_DEBUG_RAM_SETUP
|
||||
sdram_dump_mchbar_registers();
|
||||
|
||||
{
|
||||
/* This will not work if TSEG is in place! */
|
||||
u32 tom = pci_read_config32(PCI_DEV(0,2,0), 0x5c);
|
||||
|
||||
printk(BIOS_DEBUG, "TOM: 0x%08x\n", tom);
|
||||
ram_check(0x00000000, 0x000a0000);
|
||||
ram_check(0x00100000, tom);
|
||||
}
|
||||
#endif
|
||||
#endif
|
||||
MCHBAR16(SSKPD) = 0xCAFE;
|
||||
|
||||
cbmem_was_initted = !cbmem_recovery(boot_mode==2);
|
||||
|
|
|
@ -291,22 +291,6 @@ void main(unsigned long bist)
|
|||
/* Initialize the internal PCIe links before we go into stage2 */
|
||||
i945_late_initialization();
|
||||
|
||||
#if !CONFIG_HAVE_ACPI_RESUME
|
||||
#if CONFIG_DEFAULT_CONSOLE_LOGLEVEL > 8
|
||||
#if CONFIG_DEBUG_RAM_SETUP
|
||||
sdram_dump_mchbar_registers();
|
||||
#endif
|
||||
|
||||
{
|
||||
/* This will not work if TSEG is in place! */
|
||||
u32 tom = pci_read_config32(PCI_DEV(0,2,0), 0x5c);
|
||||
|
||||
printk(BIOS_DEBUG, "TOM: 0x%08x\n", tom);
|
||||
ram_check(0x00000000, 0x000a0000);
|
||||
//ram_check(0x00100000, tom);
|
||||
}
|
||||
#endif
|
||||
#endif
|
||||
|
||||
quick_ram_check();
|
||||
|
||||
|
|
|
@ -251,23 +251,6 @@ void main(unsigned long bist)
|
|||
/* Initialize the internal PCIe links before we go into stage2 */
|
||||
i945_late_initialization();
|
||||
|
||||
#if !CONFIG_HAVE_ACPI_RESUME
|
||||
#if CONFIG_DEFAULT_CONSOLE_LOGLEVEL > 8
|
||||
#if CONFIG_DEBUG_RAM_SETUP
|
||||
sdram_dump_mchbar_registers();
|
||||
#endif
|
||||
|
||||
{
|
||||
/* This will not work if TSEG is in place! */
|
||||
u32 tom = pci_read_config32(PCI_DEV(0,2,0), 0x5c);
|
||||
|
||||
printk(BIOS_DEBUG, "TOM: 0x%08x\n", tom);
|
||||
ram_check(0x00000000, 0x000a0000);
|
||||
//ram_check(0x00100000, tom);
|
||||
}
|
||||
#endif
|
||||
#endif
|
||||
|
||||
MCHBAR16(SSKPD) = 0xCAFE;
|
||||
|
||||
cbmem_was_initted = !cbmem_recovery(boot_mode==2);
|
||||
|
|
|
@ -402,23 +402,6 @@ void main(unsigned long bist)
|
|||
/* Initialize the internal PCIe links before we go into stage2 */
|
||||
i945_late_initialization();
|
||||
|
||||
#if !CONFIG_HAVE_ACPI_RESUME
|
||||
#if CONFIG_DEFAULT_CONSOLE_LOGLEVEL > 8
|
||||
#if CONFIG_DEBUG_RAM_SETUP
|
||||
sdram_dump_mchbar_registers();
|
||||
#endif
|
||||
|
||||
{
|
||||
/* This will not work if TSEG is in place! */
|
||||
u32 tom = pci_read_config32(PCI_DEV(0,2,0), 0x5c);
|
||||
|
||||
printk(BIOS_DEBUG, "TOM: 0x%08x\n", tom);
|
||||
ram_check(0x00000000, 0x000a0000);
|
||||
//ram_check(0x00100000, tom);
|
||||
}
|
||||
#endif
|
||||
#endif
|
||||
|
||||
quick_ram_check();
|
||||
|
||||
MCHBAR16(SSKPD) = 0xCAFE;
|
||||
|
|
|
@ -302,23 +302,6 @@ void main(unsigned long bist)
|
|||
/* Initialize the internal PCIe links before we go into stage2 */
|
||||
i945_late_initialization();
|
||||
|
||||
#if !CONFIG_HAVE_ACPI_RESUME
|
||||
#if CONFIG_DEFAULT_CONSOLE_LOGLEVEL > 8
|
||||
#if CONFIG_DEBUG_RAM_SETUP
|
||||
sdram_dump_mchbar_registers();
|
||||
|
||||
{
|
||||
/* This will not work if TSEG is in place! */
|
||||
u32 tom = pci_read_config32(PCI_DEV(0,2,0), 0x5c);
|
||||
|
||||
printk(BIOS_DEBUG, "TOM: 0x%08x\n", tom);
|
||||
ram_check(0x00000000, 0x000a0000);
|
||||
ram_check(0x00100000, tom);
|
||||
}
|
||||
#endif
|
||||
#endif
|
||||
#endif
|
||||
|
||||
MCHBAR16(SSKPD) = 0xCAFE;
|
||||
|
||||
cbmem_was_initted = !cbmem_recovery(boot_mode==2);
|
||||
|
|
|
@ -304,23 +304,6 @@ void main(unsigned long bist)
|
|||
/* Initialize the internal PCIe links before we go into stage2 */
|
||||
i945_late_initialization();
|
||||
|
||||
#if !CONFIG_HAVE_ACPI_RESUME
|
||||
#if CONFIG_DEFAULT_CONSOLE_LOGLEVEL > 8
|
||||
#if CONFIG_DEBUG_RAM_SETUP
|
||||
sdram_dump_mchbar_registers();
|
||||
|
||||
{
|
||||
/* This will not work if TSEG is in place! */
|
||||
u32 tom = pci_read_config32(PCI_DEV(0, 2, 0), 0x5c);
|
||||
|
||||
printk(BIOS_DEBUG, "TOM: 0x%08x\n", tom);
|
||||
ram_check(0x00000000, 0x000a0000);
|
||||
ram_check(0x00100000, tom);
|
||||
}
|
||||
#endif
|
||||
#endif
|
||||
#endif
|
||||
|
||||
MCHBAR16(SSKPD) = 0xCAFE;
|
||||
|
||||
cbmem_was_initted = !cbmem_recovery(boot_mode==2);
|
||||
|
|
|
@ -327,23 +327,6 @@ void main(unsigned long bist)
|
|||
/* Initialize the internal PCIe links before we go into stage2 */
|
||||
i945_late_initialization();
|
||||
|
||||
#if !CONFIG_HAVE_ACPI_RESUME
|
||||
#if CONFIG_DEFAULT_CONSOLE_LOGLEVEL > 8
|
||||
#if CONFIG_DEBUG_RAM_SETUP
|
||||
sdram_dump_mchbar_registers();
|
||||
|
||||
{
|
||||
/* This will not work if TSEG is in place! */
|
||||
u32 tom = pci_read_config32(PCI_DEV(0,2,0), 0x5c);
|
||||
|
||||
printk(BIOS_DEBUG, "TOM: 0x%08x\n", tom);
|
||||
ram_check(0x00000000, 0x000a0000);
|
||||
ram_check(0x00100000, tom);
|
||||
}
|
||||
#endif
|
||||
#endif
|
||||
#endif
|
||||
|
||||
MCHBAR16(SSKPD) = 0xCAFE;
|
||||
|
||||
cbmem_was_initted = !cbmem_recovery(boot_mode==2);
|
||||
|
|
Loading…
Reference in New Issue