Myles Watson wrote: > > > 2009/10/23 Stefan Reinauer <[email protected] > <mailto:[email protected]>> > > See patch > > -- > coresystems GmbH • Brahmsstr. 16 • D-79104 Freiburg i. Br. > Tel.: +49 761 7668825 • Fax: +49 761 7664613 > Email: [email protected] <mailto:[email protected]> • > http://www.coresystems.de/ > Registergericht: Amtsgericht Freiburg • HRB 7656 > Geschäftsführer: Stefan Reinauer • Ust-IdNr.: DE245674866 > > > * refactor x86 smi handler (put all debug stuff in an extra file > smiutil.c) > * lock other CPUs in SMI handler while one CPU is handling an SMI. > Without this > various racing scenarios could happen. > > Signed-off-by: Stefan Reinauer <[email protected] > <mailto:[email protected]>> > > Acked-by: Myles Watson <[email protected] <mailto:[email protected]>> Thanks, r4840.
> Is there some header file that could be included for these defines? > Maybe even config.h? > > > +#ifndef CONFIG_TTYS0_BASE > +#define CONFIG_TTYS0_BASE 0x3f8 > +#endif > > No, but it would make sense... What's the right place for this? include/uart8250.h ? Right now they are defined in ./src/arch/i386/init/crt0.S.lb ./src/console/uart8250_console.c ./src/cpu/amd/model_lx/cache_as_ram.inc ./src/cpu/x86/smm/smiutil.c ./src/pc80/serial.c -- coreboot mailing list: [email protected] http://www.coreboot.org/mailman/listinfo/coreboot

