[LinuxBIOS] Config file dependencies

Li-Ta Lo ollie at lanl.gov
Thu Oct 27 22:56:12 CEST 2005


On Thu, 2005-10-27 at 13:45 -0700, Lu, Yinghai wrote:
> No 
> 1. CONFIG_USE_INIT==0 is very useful for debug.

Why??

> 2. CONFIG_USE_INIT==1 is something ppc car using...
> 

AFIAK, the way you are using USE_INIT is totally different from what 
Greg is doing on PPC. I am not an expert on this issue but I believe
the only common thing in these two is the same macro name. Greg, can
you have some comment on this?

> Auto.c and cache_as_ram_auto.c is very different
> 1. cache_as_ram_auto include failover.c

Why do you do that? Why don't you have cache_as_ram_failover.c too?

> 2. later cache_as_ram_auto will have struct sys_info sysinfox, and it
> will include every info include dimm.....and we don't need to keep read
> smbus and pci conf space for....
> 

Any reason it has to be in cache_as_ram_auto only?

> 
> YH
> 
> -----Original Message-----
> From: Li-Ta Lo [mailto:ollie at lanl.gov] 
> Sent: Thursday, October 27, 2005 1:22 PM
> To: Lu, Yinghai
> Cc: Stefan Reinauer; LinuxBIOS
> Subject: RE: [LinuxBIOS] Config file dependencies
> 
> 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?
> 
> > 
> > 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 10:31 AM
> > To: Stefan Reinauer
> > Cc: LinuxBIOS
> > Subject: Re: [LinuxBIOS] Config file dependencies
> > 
> > On Thu, 2005-10-27 at 09:26 +0200, Stefan Reinauer wrote:
> > > Hi,
> > > 
> > > > @@ -130,10 +130,10 @@
> > > >  ##
> > > >  ## enable CACHE_AS_RAM specifics
> > > >  ##
> > > > -default USE_DCACHE_RAM=1
> > > > +default USE_DCACHE_RAM=0
> > > >  default DCACHE_RAM_BASE=0xcf000
> > > >  default DCACHE_RAM_SIZE=0x1000
> > > > -default CONFIG_USE_INIT=1
> > > > +default CONFIG_USE_INIT=0
> > > 
> > > 
> > > The config files have a number of dependencies like the above.
> > > USE_DCACHE_RAM needs CONFIG_USE_INIT otherwise it won't work.
> > > 
> > 
> > That's something I try to figure out. If DCACHE_RAM needs USE_INIT,
> > why we still have
> > 
> > #if CONFIG_USE_INIT == 0
> > #include "lib/memcpy.c"
> > #endif
> > 
> > in the src/mainboard/tyan/s2895/cache_as_ram_auto.c ?
> > 
> > > Should we put this dependency in another place? Keeping such 
> > > things in mind should be part of the LinuxBIOS configuration, 
> > > not the LinuxBIOS (mainboard) developer.. 
> > > 
> > > Stefan
> > > 
> > > 
> > -- 
> > Li-Ta Lo <ollie at lanl.gov>
> > Los Alamos National Lab
> > 
> > 
-- 
Li-Ta Lo <ollie at lanl.gov>
Los Alamos National Lab





More information about the coreboot mailing list