[LinuxBIOS] r381 - in LinuxBIOSv3/mainboard: . adl adl/msm800sev amd/norwich emulation/qemu-x86
Peter Stuge
peter at stuge.se
Thu Jun 28 01:01:35 CEST 2007
On Wed, Jun 27, 2007 at 10:38:27PM +0200, svn at openbios.org wrote:
> +++ LinuxBIOSv3/mainboard/adl/msm800sev/initram.c 2007-06-27 20:38:27 UTC (rev 381)
[..]
> +int main(void)
> +{
> + void done_cache_as_ram_main(void);
> + void w83627hf_enable_serial(u8 dev, u8 serial, u16 iobase);
> + post_code(POST_START_OF_MAIN);
> +
> + system_preinit();
> + geodelx_msr_init();
> +
> + cs5536_early_setup();
> +
> + /* NOTE: must do this AFTER the early_setup!
> + * it is counting on some early MSR setup
> + * for cs5536
> + */
> + cs5536_disable_internal_uart();
Shouldn't it go into _early_setup() then?
> + w83627hf_enable_serial(0x2e, 0x30, 0x3f8);
"Magic" values - shouldn't at least 0x3f8 be a CONFIG_ value?
> + /* Switch from Cache as RAM to real RAM */
> + printk(BIOS_SPEW, "Before wbinvd\n");
> + __asm__("wbinvd\n");
> + printk(BIOS_SPEW, "After wbinvd\n");
Isn't there already a function for disabling CAR?
//Peter
More information about the coreboot
mailing list