[coreboot] fix vga for tyan s2881 in lab configuration

Jordan Crouse jordan.crouse at amd.com
Fri Oct 10 23:22:13 CEST 2008


On 10/10/08 17:12 -0400, Ward Vandewege wrote:
> -- 
> Ward Vandewege <ward at fsf.org>
> Free Software Foundation - Senior Systems Administrator

> 
> The Tyan s2881 ships with a 512KB rom chip. For LAB we require an 1MB chip, but
> the VGA rom address is hardcoded assuming the 512KB chip.
> 
> This patch 
> 
> a) bumps up the v2 revision to 3646 which actually enables vga on the s2881
> b) conditionally patches the v2 tree for a 1MB chip when the s2881 is being
> built with the LAB config file.
> 
> Tested on real hardware.
> 
> Signed-off-by: Ward Vandewege <ward at gnu.org>

Acked-by: Jordan Crouse <jordan.crouse at amd.com>
> Index: config/platforms/tyan-s2881.conf
> ===================================================================
> --- config/platforms/tyan-s2881.conf	(revision 245)
> +++ config/platforms/tyan-s2881.conf	(working copy)
> @@ -30,4 +30,4 @@
>  COREBOOT_VENDOR=tyan
>  COREBOOT_BOARD=s2881
>  CBV2_TDIR=s2881
> -CBV2_TAG=3131
> +CBV2_TAG=3646
> Index: packages/coreboot-v2/coreboot.inc
> ===================================================================
> --- packages/coreboot-v2/coreboot.inc	(revision 245)
> +++ packages/coreboot-v2/coreboot.inc	(working copy)
> @@ -3,6 +3,9 @@
>  ifeq ($(CONFIG_USE_LZMA),y)
>  	CBV2_CONFIG=Config-lab.lb
>  	CBV2_PAYLOAD_FILE_EXT=elf.lzma
> +ifeq ($(COREBOOT_BOARD),s2881)
> +	CBV2_PATCHES += $(PACKAGE_DIR)/coreboot-v2/patches/s2881-lab-vga-fix.patch 
> +endif
>  else
>  	CBV2_CONFIG=Config.lb
>  	CBV2_PAYLOAD_FILE_EXT=elf
> @@ -150,6 +153,7 @@
>  
>  $(OUTPUT_DIR)/$(TARGET_ROM): $(CBV2_COMPONENTS)
>  	@ mkdir -p $(OUTPUT_DIR)
> +	@ echo $(BIN_DIR)/construct-rom.sh $(PLATFORM_DOPAD) $(CBV2_COMPONENTS) 
>  	@ $(BIN_DIR)/construct-rom.sh $(PLATFORM_DOPAD) $(CBV2_COMPONENTS) > $@
>  
>  generic-coreboot: $(OUTPUT_DIR)/$(TARGET_ROM)
> Index: packages/coreboot-v2/patches/s2881-lab-vga-fix.patch
> ===================================================================
> --- packages/coreboot-v2/patches/s2881-lab-vga-fix.patch	(revision 0)
> +++ packages/coreboot-v2/patches/s2881-lab-vga-fix.patch	(revision 0)
> @@ -0,0 +1,14 @@
> +Index: src/mainboard/tyan/s2881/Config.lb
> +===================================================================
> +--- a/src/mainboard/tyan/s2881/Config.lb	(revision 3646)
> ++++ a/src/mainboard/tyan/s2881/Config.lb	(working copy)
> +@@ -231,7 +231,8 @@
> +                                                 end
> +                                                 chip drivers/pci/onboard
> +                                                         device pci 6.0 on end
> +-							register "rom_address" = "0xfff80000"
> ++							#register "rom_address" = "0xfff80000" #for 512K
> ++							register "rom_address" = "0xfff00000" #for 1M
> +                                                 end
> + 					end
> + 					device pci 1.0 on

> --
> coreboot mailing list: coreboot at coreboot.org
> http://www.coreboot.org/mailman/listinfo/coreboot


-- 
Jordan Crouse
Systems Software Development Engineer 
Advanced Micro Devices, Inc.





More information about the coreboot mailing list