Patch set updated for coreboot: da2d0ed AMD CIMx SB800 boards: platform_cfg.h: Integrate Kconfig SATA Mode choice

[Paul Menzel]

Paul Menzel ([e-mail]) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/2610

-gerrit

commit da2d0edc212bf461c86c01eaaed18f3667ec8631
Author: Paul Menzel 
Date:   Sun Mar 3 10:56:15 2013 +0100

AMD CIMx SB800 boards: platform_cfg.h: Integrate Kconfig SATA Mode choice

Currently for Advansus A785E-I, ASRock E350M1 and ASUS M5A88-V
    despite what is chosen in Kconfig »Chipset« menu item,

$ more .config
        […]
        # CONFIG_ENABLE_IDE_COMBINED_MODE is not set
        CONFIG_IDE_COMBINED_MODE=0x1
        # CONFIG_SB800_SATA_IDE is not set
        CONFIG_SB800_SATA_AHCI=y
        # CONFIG_SB800_SATA_RAID is not set
        CONFIG_SB800_SATA_MODE=0x2
        […]

the SATA controller is put into IDE mode.

$ lspci -nn | grep SATA
        00:11.0 SATA controller [0106]: Advanced Micro Devices [AMD] nee ATI SB7x0/SB8x0/SB9x0 SATA Controller [IDE mode] [1002:4390] (rev 40)

Commit »sb800: Add sata ahci/raid mode kconfig option«
    (d4a0e7d0) [1] added the options above to configure the mode
    using Kconfig and some SB800 boards were adapted already. For
    example commit »persimmon: sb800 sata mode configure update«
    (1386fa74) [2] did so for AMD Persimmon.

Doing the same by assigning the Kconfig variable to the value in
    `platform_cfg.h` integrates this with the three remaining boards
    listed above.

The patch is successfully tested with the ASRock E350M1.

$ lspci -nn | grep SATA
        00:11.0 SATA controller [0106]: Advanced Micro Devices [AMD] nee ATI SB7x0/SB8x0/SB9x0 SATA Controller [AHCI mode] [1002:4391] (rev 40)

[1] http://review.coreboot.org/225
    [2] http://review.coreboot.org/227

Change-Id: I227257e2c8f04f18c27ff00fe62d42e372de67e4
    Signed-off-by: Paul Menzel 
---
 src/mainboard/advansus/a785e-i/platform_cfg.h | 2 +-
 src/mainboard/asrock/e350m1/platform_cfg.h    | 2 +-
 src/mainboard/asus/m5a88-v/platform_cfg.h     | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/mainboard/advansus/a785e-i/platform_cfg.h b/src/mainboard/advansus/a785e-i/platform_cfg.h
index 9f70dcc..f1eb9f5 100644
--- a/src/mainboard/advansus/a785e-i/platform_cfg.h
+++ b/src/mainboard/advansus/a785e-i/platform_cfg.h
@@ -101,7 +101,7 @@
  * @brief INCHIP Sata Controller Mode
  *   NOTE: DO NOT ALLOW SATA & IDE use same mode
  */
-#define SATA_MODE			NATIVE_IDE_MODE
+#define SATA_MODE			CONFIG_SB800_SATA_MODE

/**
  * @brief INCHIP Sata IDE Controller Mode
diff --git a/src/mainboard/asrock/e350m1/platform_cfg.h b/src/mainboard/asrock/e350m1/platform_cfg.h
index 389e7db..24a66dd 100644
--- a/src/mainboard/asrock/e350m1/platform_cfg.h
+++ b/src/mainboard/asrock/e350m1/platform_cfg.h
@@ -103,7 +103,7 @@
  * @brief INCHIP Sata Controller Mode
  *   NOTE: DO NOT ALLOW SATA & IDE use same mode
  */
-#define SATA_MODE			NATIVE_IDE_MODE
+#define SATA_MODE			CONFIG_SB800_SATA_MODE

/**
  * @brief INCHIP Sata IDE Controller Mode
diff --git a/src/mainboard/asus/m5a88-v/platform_cfg.h b/src/mainboard/asus/m5a88-v/platform_cfg.h
index 9f70dcc..f1eb9f5 100644
--- a/src/mainboard/asus/m5a88-v/platform_cfg.h
+++ b/src/mainboard/asus/m5a88-v/platform_cfg.h
@@ -101,7 +101,7 @@
  * @brief INCHIP Sata Controller Mode
  *   NOTE: DO NOT ALLOW SATA & IDE use same mode
  */
-#define SATA_MODE			NATIVE_IDE_MODE
+#define SATA_MODE			CONFIG_SB800_SATA_MODE

/**
  * @brief INCHIP Sata IDE Controller Mode