On Wed, Nov 17, 2010 at 12:30:51PM +0100, repository service wrote:
> Note that the bios rom decode is enabled in
> src/southbridge/via/vt8237r/vt8237r_early_smbus.c
> for the whole 4MB area (even though the comment says 1MB).

I think the enable_rom_decode() function in vt8237r_early_smbus.c
should be dropped entirely, the VT8237R already enables full ROM
decode in src/southbridge/via/vt8237r/bootblock.c. We should probably
enable TINYBOOTBLOCK for VT8237R in general (and thus all boards using
this southbridge) if that's not already the case, then drop
enable_rom_decode().


Uwe.
-- 
http://hermann-uwe.de     | http://sigrok.org
http://randomprojects.org | http://unmaintained-free-software.org

-- 
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to