Add CONFIG_VGA_ROM_RUN to dbm690t and pistachio, otherwise the
authorZheng Bao <zheng.bao@amd.com>
Fri, 16 Oct 2009 07:44:04 +0000 (07:44 +0000)
committerZheng Bao <Zheng.Bao@amd.com>
Fri, 16 Oct 2009 07:44:04 +0000 (07:44 +0000)
VGA ROM can not run. After make, run
> ./cbfs/cbfstool ./coreboot.rom add ../vga_bios.rom   pci1002,791f.rom  optionrom
to make the final image with vga bios.

The macro vga_rom_address is out-of-date when CBFS starts play its role. it also should
be eliminated from rs690/chip.h as below. But it will cause building error on other board, which I
cant make test on.

##    Index: src/southbridge/amd/rs690/chip.h
##    ===================================================================
##    --- src/southbridge/amd/rs690/chip.h (revision 4782)
##    +++ src/southbridge/amd/rs690/chip.h (working copy)
##    @@ -23,7 +23,6 @@
##     /* Member variables are defined in Config.lb. */
##     struct southbridge_amd_rs690_config
##     {
##    - u32 vga_rom_address; /* The location that the VGA rom has been appened. */
##      u8 gpp_configuration; /* The configuration of General Purpose Port, A/B/C/D/E. */
##      u8 port_enable; /* Which port is enabled? GFX(2,3), GPP(4,5,6,7) */
##      u8 gfx_dev2_dev3; /* for GFX Core initialization REFCLK_SEL */
##

Don't apply above patch about rs690/chip.h before every board has been fixed.

Signed-off-by: Zheng Bao <zheng.bao@amd.com>
Acked-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006@gmx.net>
git-svn-id: svn://svn.coreboot.org/coreboot/trunk@4783 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1

src/mainboard/amd/dbm690t/Config.lb
src/mainboard/amd/dbm690t/Options.lb
src/mainboard/amd/dbm690t/devicetree.cb
src/mainboard/amd/pistachio/Config.lb
src/mainboard/amd/pistachio/Options.lb
src/mainboard/amd/pistachio/devicetree.cb
src/southbridge/amd/rs690/rs690_gfx.c
targets/amd/dbm690t/Config.lb
targets/amd/pistachio/Config.lb

index 68255eee8c8bfb17f86ba47501f66799b64b8fe5..98ff2dd0f79eb5494d97e90052e743c5bb934a6a 100644 (file)
@@ -136,7 +136,6 @@ config chip.h
 #The variables belong to mainboard are defined here.
 
 #Define gpp_configuration,     A=0, B=1, C=2, D=3, E=4(default)
-#Define vga_rom_address = 0xfff0000
 #Define port_enable, (bit map): GFX(2,3), GPP(4,5,6,7)
 #Define gfx_dev2_dev3, 0: a link will never be established on Dev2 or Dev3,
 #                                         1: the system allows a PCIE link to be established on Dev2 or Dev3.
@@ -170,7 +169,6 @@ chip northbridge/amd/amdk8/root_complex
                                        device pci 6.0 on end # PCIE P2P bridge 0x7916
                                        device pci 7.0 on end # PCIE P2P bridge 0x7917
                                        device pci 8.0 off end # NB/SB Link P2P bridge
-                                       register "vga_rom_address" = "0xfff00000"
                                        register "gpp_configuration" = "4"
                                        register "port_enable" = "0xfc"
                                        register "gfx_dev2_dev3" = "1"
index 92378ebb10575891de8ad9883be9081d0e1ce73a..d5d54f985ec9e60202130bcaf54ac7cf6e62ef75 100644 (file)
@@ -73,6 +73,7 @@ uses HOSTCC
 uses CONFIG_OBJCOPY
 uses CONFIG_CONSOLE_VGA
 uses CONFIG_PCI_ROM_RUN
+uses CONFIG_VGA_ROM_RUN
 uses CONFIG_HW_MEM_HOLE_SIZEK
 uses CONFIG_HT_CHAIN_UNITID_BASE
 uses CONFIG_HT_CHAIN_END_UNITID_BASE
@@ -103,8 +104,6 @@ default CONFIG_ROM_SIZE=524288
 ##
 ## CONFIG_FALLBACK_SIZE is the amount of the ROM the complete fallback image will use
 ##
-#default CONFIG_FALLBACK_SIZE = CONFIG_ROM_IMAGE_SIZE
-#256K
 default CONFIG_FALLBACK_SIZE = CONFIG_ROM_IMAGE_SIZE
 
 ##
@@ -160,6 +159,7 @@ default CONFIG_HW_MEM_HOLE_SIZEK=0x100000
 #VGA Console
 default CONFIG_CONSOLE_VGA=1
 default CONFIG_PCI_ROM_RUN=1
+default CONFIG_VGA_ROM_RUN=1
 
 # BTDC: Only one HT device on Herring.
 #HT Unit ID offset
index 870503789f5c5cd6fb83532ba3eba48443ee5490..2e89c13aaf9b61e6ac0ce501770f37bec22143c1 100644 (file)
@@ -1,5 +1,4 @@
 #Define gpp_configuration,     A=0, B=1, C=2, D=3, E=4(default)
-#Define vga_rom_address = 0xfff0000
 #Define port_enable, (bit map): GFX(2,3), GPP(4,5,6,7)
 #Define gfx_dev2_dev3, 0: a link will never be established on Dev2 or Dev3,
 #                      1: the system allows a PCIE link to be established on Dev2 or Dev3.
@@ -33,7 +32,6 @@ chip northbridge/amd/amdk8/root_complex
                                        device pci 6.0 on end # PCIE P2P bridge 0x7916
                                        device pci 7.0 on end # PCIE P2P bridge 0x7917
                                        device pci 8.0 off end # NB/SB Link P2P bridge
-                                       register "vga_rom_address" = "0xfff00000"
                                        register "gpp_configuration" = "4"
                                        register "port_enable" = "0xfc"
                                        register "gfx_dev2_dev3" = "1"
index e0458d36098cc66709aa487e112786d02dc32fed..b0f82b21c789b0b8dfefcea93a830c24d606a889 100644 (file)
@@ -136,7 +136,6 @@ config chip.h
 #The variables belong to mainboard are defined here.
 
 #Define gpp_configuration,     A=0, B=1, C=2, D=3, E=4(default)
-#Define vga_rom_address = 0xfff0000
 #Define port_enable, (bit map): GFX(2,3), GPP(4,5,6,7)
 #Define gfx_dev2_dev3, 0: a link will never be established on Dev2 or Dev3,
 #                                         1: the system allows a PCIE link to be established on Dev2 or Dev3.
@@ -171,7 +170,6 @@ chip northbridge/amd/amdk8/root_complex
                                        device pci 6.0 on end # PCIE P2P bridge 0x7916
                                        device pci 7.0 on end # PCIE P2P bridge 0x7917
                                        device pci 8.0 off end # NB/SB Link P2P bridge
-                                       register "vga_rom_address" = "0xfff00000"
                                        register "gpp_configuration" = "4"
                                        register "port_enable" = "0xfc"
                                        register "gfx_dev2_dev3" = "1"
index 0082151c7de671307472b43579ef013026f8e8f9..922edf5210faae3aa18f4af637b3479f6133379d 100644 (file)
@@ -73,6 +73,7 @@ uses HOSTCC
 uses CONFIG_OBJCOPY
 uses CONFIG_CONSOLE_VGA
 uses CONFIG_PCI_ROM_RUN
+uses CONFIG_VGA_ROM_RUN
 uses CONFIG_HW_MEM_HOLE_SIZEK
 uses CONFIG_HT_CHAIN_UNITID_BASE
 uses CONFIG_HT_CHAIN_END_UNITID_BASE
@@ -158,6 +159,7 @@ default CONFIG_HW_MEM_HOLE_SIZEK=0x100000
 #VGA Console
 default CONFIG_CONSOLE_VGA=1
 default CONFIG_PCI_ROM_RUN=1
+default CONFIG_VGA_ROM_RUN=1
 
 # BTDC: Only one HT device on Herring.
 #HT Unit ID offset
index 802c000a2ab65102f7a8ae889b1cc4263ac2057c..ec16256b2645306963180743b73f9f44a9a290bd 100644 (file)
@@ -1,5 +1,4 @@
 #Define gpp_configuration,     A=0, B=1, C=2, D=3, E=4(default)
-#Define vga_rom_address = 0xfff0000
 #Define port_enable, (bit map): GFX(2,3), GPP(4,5,6,7)
 #Define gfx_dev2_dev3, 0: a link will never be established on Dev2 or Dev3,
 #                      1: the system allows a PCIE link to be established on Dev2 or Dev3.
@@ -34,7 +33,6 @@ chip northbridge/amd/amdk8/root_complex
                                        device pci 6.0 on end # PCIE P2P bridge 0x7916
                                        device pci 7.0 on end # PCIE P2P bridge 0x7917
                                        device pci 8.0 off end # NB/SB Link P2P bridge
-                                       register "vga_rom_address" = "0xfff00000"
                                        register "gpp_configuration" = "4"
                                        register "port_enable" = "0xfc"
                                        register "gfx_dev2_dev3" = "1"
index bddb825863e4b1abe710c93af848b8faf499e8a5..3199f087f38e0222527a65e510bf2b70e73e6cd1 100644 (file)
@@ -77,13 +77,9 @@ static void internal_gfx_pci_dev_init(struct device *dev)
            (struct southbridge_amd_rs690_config *)dev->chip_info;
        deviceid = pci_read_config16(dev, PCI_DEVICE_ID);
        vendorid = pci_read_config16(dev, PCI_VENDOR_ID);
-       printk_info("internal_gfx_pci_dev_init device=%x, vendor=%x, vga_rom_address=0x%x.\n",
-            deviceid, vendorid, cfg->vga_rom_address);
+       printk_info("internal_gfx_pci_dev_init device=%x, vendor=%x.\n",
+            deviceid, vendorid);
 
-#if 0 /* I think these should be done in Config.lb. Please check it. */
-       dev->on_mainboard = 1;
-       dev->rom_address = cfg->vga_rom_address;        /* 0xfff00000; */
-#endif
        pci_dev_init(dev);
 
        /* clk ind */
index 80c9db639dad055e6e85c6ac6cd935c66d54bb7c..e3f0ab3e94f3e84c785f120e3036360a55fcdba2 100644 (file)
@@ -4,14 +4,14 @@ target dbm690t
 mainboard amd/dbm690t
 
 romimage "normal"
-       option CONFIG_ROM_SIZE = 1024*1024 - 55808
+       option CONFIG_ROM_SIZE = 1024*1024
        option CONFIG_USE_FALLBACK_IMAGE=0
        option CONFIG_ROM_IMAGE_SIZE=0x20000
        option CONFIG_XIP_ROM_SIZE=0x20000
        payload ../payload.elf
 end
 
-romimage "fallback" 
+romimage "fallback"
        option CONFIG_USE_FALLBACK_IMAGE=1
        option CONFIG_ROM_IMAGE_SIZE=0x20000
        option CONFIG_XIP_ROM_SIZE=0x20000
index e2e7939ac72502465f9a79f2ebacd1b331e616b3..5d6ca9b759a9ab75091efe172af984f44ec5a945 100644 (file)
@@ -4,14 +4,14 @@ target pistachio
 mainboard amd/pistachio
 
 romimage "normal"
-       option CONFIG_ROM_SIZE = 1024*1024 - 55808
+       option CONFIG_ROM_SIZE = 1024*1024
        option CONFIG_USE_FALLBACK_IMAGE=0
        option CONFIG_ROM_IMAGE_SIZE=0x20000
        option CONFIG_XIP_ROM_SIZE=0x20000
        payload ../payload.elf
 end
 
-romimage "fallback" 
+romimage "fallback"
        option CONFIG_USE_FALLBACK_IMAGE=1
        option CONFIG_ROM_IMAGE_SIZE=0x20000
        option CONFIG_XIP_ROM_SIZE=0x20000