[LinuxBIOS] Config file dependencies

Lu, Yinghai yinghai.lu at amd.com
Thu Oct 27 22:49:15 CEST 2005


__ASSEMBLY__ already be used.

__PRE_SDRAM__ maybe good, but if we define USE_DCACHE = 1, 
__ROMCC__is equal to __PRE_SDRAM___ it mean code will be compiled by gcc
instead of romcc.

YH

-----Original Message-----
From: linuxbios-bounces at openbios.org
[mailto:linuxbios-bounces at openbios.org] On Behalf Of Li-Ta Lo
Sent: Thursday, October 27, 2005 1:30 PM
To: Lu, Yinghai
Cc: LinuxBIOS
Subject: RE: [LinuxBIOS] Config file dependencies

On Thu, 2005-10-27 at 14:21 -0600, Li-Ta Lo wrote:
> On Thu, 2005-10-27 at 11:11 -0700, Lu, Yinghai wrote:
> > If CONFIG_USE_INIT == 0, the cache_as_ram_auto.c will be compiled to
> > auto.inc. ---> part of crt0.s---> part of linuxbios_rom
> > 
> > If CONFIG_USE_INIF == 1, the cache_as_ram_auto.c will be compiled to
> > auto.o and be linked with memcpy.o....and auto.o and memcpy.o --->
> > init.o
> > And init.o ---> part of linuxbios_rom...
> > 
> 
> Is there any reason we need two different ways to do it? If both of
them
> are as good, can we drop one and support just the other?
> 
> BTW, after reading your patch, I found there is very little difference

> between auto.c and cache_as_ram_auto.c. Most of the differences are in
> the included files, can we merge auto.c and cache_as_ram_auto.c?
> 

Talking about merging auto.c and cache_as_ram_auto.c, you are using 
#define __ROMCC__ to choose different definitions of device_t. I don't
thing it is the right way to do it. Instead, we should use another
macro for this (__ASSEMBLY__ or __PRE_SDRAM__)?

-- 
Li-Ta Lo <ollie at lanl.gov>
Los Alamos National Lab


-- 
LinuxBIOS mailing list
LinuxBIOS at openbios.org
http://www.openbios.org/mailman/listinfo/linuxbios






More information about the coreboot mailing list