[coreboot] [PATCH]fix various vgabios.c copies in light of CBFS

Stefan Reinauer stepan at coresystems.de
Sat Oct 3 17:29:42 CEST 2009


Patrick Georgi wrote:
> Hi,
>
> there were a couple of copies of vgabios.c that looked for the option
> rom image at fixed addresses (usually begin of flash), which is not the
> supported way of doing things anymore.
>
> As this patch removes the non-CBFS capability, I'd commit it right after
> the removal of non-CBFS support this evening (in ~3 hours), as non-CBFS
> support isn't necessary anymore at that point.
>
> Of course, the duplication should be eliminated, as well as (some of)
> the rom_address entries in Config.lb and devicetree.cb for the boards.
>
> It might also be worthwhile to investigate if src/devices/pci_rom.c
> couldn't be used for the option rom loading. Just using dev->rom_address
> didn't work on my board, but this fix was good enough for me for now.
>
> The change is the same in all vgabios.c files, it's boot tested on cx700
> (via/vt8454c), via/epia-m and artecgroup/dbe61 are build tested.
> Furthermore,
> for cn400, via/epia-n built,
> for cn700, via/epia-cn built,
> for vx800, via/epia-m700 built
>
> Signed-off-by: Patrick Georgi <patrick.georgi at coresystems.de>
>   
Acked-by: Stefan Reinauer <stepan at coresystems.de>

-- 
coresystems GmbH • Brahmsstr. 16 • D-79104 Freiburg i. Br.
      Tel.: +49 761 7668825 • Fax: +49 761 7664613
Email: info at coresystems.dehttp://www.coresystems.de/
Registergericht: Amtsgericht Freiburg • HRB 7656
Geschäftsführer: Stefan Reinauer • Ust-IdNr.: DE245674866





More information about the coreboot mailing list