[LinuxBIOS] r401 - in LinuxBIOSv3/northbridge: amd/geodelx intel/i440bxemulation

Stefan Reinauer stepan at coresystems.de
Fri Jun 29 00:03:19 CEST 2007


* Uwe Hermann <uwe at hermann-uwe.de> [070628 23:10]:
> On Thu, Jun 28, 2007 at 07:12:37PM +0200, svn at openbios.org wrote:
> > +#include <types.h>
> > +#include <lib.h>
> > +#include <console.h>
> > +#include <post_code.h>
> > +#include <device/device.h>
> > +#include <device/pci.h>
> > +#include <string.h>
> > +#include <msr.h>
> > +#include <io.h>
> > +#include <cpu.h>
> > +#include <amd_geodelx.h>
> 
> The list of includes we need in every file is getting pretty large
> slowly... Should we introduce a linuxbios.h file which includes
> all "core" headers such as types.h, string.h, io.h and others?
> This would reduce the number of includes a lot.
 
I get the impression that we can drop quite some of them. 

Having a general linuxbios.h might increase compile time quite a bit.
Can you do some measures and/or send a patch?

> > +	__asm__ __volatile__("FINIT\n");
> 
> Should be a function in some header?

What does it do?

-- 
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/




More information about the coreboot mailing list