Myles wrote:
> Here's your suggestion as a patch.

Hmm ... I'd even go a bit further and also rename the
BOARD_PCENGINES_ALIX2D3 option, change MAINBOARD_PART_NUMBER and rename
the whole directory to keep everything consistent.

What do you think about this (attached)? (Renaming of the alix2d3/
directory to alix2d/ is not included in the patch, please do this
manually before applying.)

Signed-off-by: Jens Rottmann <[email protected]>

Cheers,
Jens

diff -urN rev 5758/src/mainboard/pcengines/Kconfig working 
copy/src/mainboard/pcengines/Kconfig
--- rev 5758/src/mainboard/pcengines/Kconfig    2010-08-30 18:36:51.000000000 
+0200
+++ working copy/src/mainboard/pcengines/Kconfig        2010-08-31 
20:22:06.000000000 +0200
@@ -5,13 +5,13 @@
 
 config BOARD_PCENGINES_ALIX1C
        bool "ALIX.1C"
-config BOARD_PCENGINES_ALIX2D3
-       bool "ALIX.2D3"
+config BOARD_PCENGINES_ALIX2D
+       bool "ALIX.2D2 or 2D3"
 
 endchoice
 
 source "src/mainboard/pcengines/alix1c/Kconfig"
-source "src/mainboard/pcengines/alix2d3/Kconfig"
+source "src/mainboard/pcengines/alix2d/Kconfig"
 
 config MAINBOARD_VENDOR
        string
diff -urN rev 5758/src/mainboard/pcengines/alix2d/Kconfig working 
copy/src/mainboard/pcengines/alix2d/Kconfig
--- rev 5758/src/mainboard/pcengines/alix2d/Kconfig     2010-08-30 
19:53:13.000000000 +0200
+++ working copy/src/mainboard/pcengines/alix2d/Kconfig 2010-08-31 
20:24:28.000000000 +0200
@@ -1,4 +1,4 @@
-if BOARD_PCENGINES_ALIX2D3
+if BOARD_PCENGINES_ALIX2D
 
 config BOARD_SPECIFIC_OPTIONS # dummy
        def_bool y
@@ -14,18 +14,18 @@
 
 config MAINBOARD_DIR
        string
-       default pcengines/alix2d3
+       default pcengines/alix2d
 
 config MAINBOARD_PART_NUMBER
        string
-       default "ALIX.2D3"
+       default "ALIX.2D"
 
 config IRQ_SLOT_COUNT
        int
-       default 6
+       default 7
 
 config RAMBASE
        hex
        default 0x4000
 
-endif # BOARD_PCENGINES_ALIX2D3
+endif # BOARD_PCENGINES_ALIX2D
diff -urN rev 5758/src/mainboard/pcengines/alix2d/irq_tables.c working 
copy/src/mainboard/pcengines/alix2d/irq_tables.c
--- rev 5758/src/mainboard/pcengines/alix2d/irq_tables.c        2010-08-24 
14:58:17.000000000 +0200
+++ working copy/src/mainboard/pcengines/alix2d/irq_tables.c    2010-08-31 
20:26:36.000000000 +0200
@@ -94,7 +94,7 @@
                /* On-board ethernet (Left) */
                {0x00, (0x09 << 3) | 0x0, {{L_PIRQB, M_PIRQB}, {0x00, 0x00}, 
{0x00, 0x00}, {0x00, 0x00}}, 0x0, 0x0},
 
-               /* On-board ethernet (Middle) */
+               /* On-board ethernet (Middle, ALIX.2D3 only) */
                {0x00, (0x0A << 3) | 0x0, {{L_PIRQC, M_PIRQC}, {0x00, 0x00}, 
{0x00, 0x00}, {0x00, 0x00}}, 0x0, 0x0},
 
                /* On-board ethernet (Right) */
@@ -103,6 +103,9 @@
                /* Mini PCI (slot 1) */
                {0x00, (0x0C << 3) | 0x0, {{L_PIRQA, M_PIRQA}, {L_PIRQB, 
M_PIRQB}, {0x00, 0x00}, {0x00, 0x00}}, 0x0, 0x0},
 
+               /* Mini PCI (slot 2, ALIX.2D2 only) */
+               {0x00, (0x0E << 3) | 0x0, {{L_PIRQC, M_PIRQC}, {L_PIRQD, 
M_PIRQD}, {0x00, 0x00}, {0x00, 0x00}}, 0x0, 0x0},
+
                /* Chipset slots -- f.3 wires to B, and f.4 and f.5 wires to D. 
*/
                {0x00, (0x0F << 3) | 0x0, {{L_PIRQA, M_PIRQA}, {L_PIRQB, 
M_PIRQB}, {L_PIRQC, M_PIRQC}, {L_PIRQD, M_PIRQD}}, 0x0, 0x0},
        }
diff -urN rev 5758/src/mainboard/pcengines/alix2d/mainboard.c working 
copy/src/mainboard/pcengines/alix2d/mainboard.c
--- rev 5758/src/mainboard/pcengines/alix2d/mainboard.c 2010-08-24 
14:58:17.000000000 +0200
+++ working copy/src/mainboard/pcengines/alix2d/mainboard.c     2010-08-31 
20:19:37.000000000 +0200
@@ -23,8 +23,8 @@
 
 static void init(struct device *dev)
 {
-       printk(BIOS_DEBUG, "ALIX.2D3 ENTER %s\n", __func__);
-       printk(BIOS_DEBUG, "ALIX.2D3 EXIT %s\n", __func__);
+       printk(BIOS_DEBUG, "ALIX.2D ENTER %s\n", __func__);
+       printk(BIOS_DEBUG, "ALIX.2D EXIT %s\n", __func__);
 }
 
 static void enable_dev(struct device *dev)
@@ -33,7 +33,7 @@
 }
 
 struct chip_operations mainboard_ops = {
-       CHIP_NAME("PC Engines ALIX.2D3 Mainboard")
+       CHIP_NAME("PC Engines ALIX.2D Mainboard")
        .enable_dev = enable_dev,
 };
 
diff -urN rev 5758/src/mainboard/pcengines/alix2d/romstage.c working 
copy/src/mainboard/pcengines/alix2d/romstage.c
--- rev 5758/src/mainboard/pcengines/alix2d/romstage.c  2010-08-24 
14:58:17.000000000 +0200
+++ working copy/src/mainboard/pcengines/alix2d/romstage.c      2010-08-31 
20:26:55.000000000 +0200
@@ -35,7 +35,7 @@
 
 #define SERIAL_DEV PNP_DEV(0x2e, W83627HF_SP1)
 
-/* The ALIX.2D3 has no SMBus; the setup is hard-wired. */
+/* The ALIX.2D has no SMBus; the setup is hard-wired. */
 static void cs5536_enable_smbus(void)
 {
 }
-- 
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to