[LinuxBIOS] Config file dependencies

[Stefan Reinauer]

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.

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

[Ronald G Minnich]

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

well, that's a good point. One way to do this is to make the default 
value of CONFIG_USE_INIT in Options.lb be this:
CONFIG_USE_INIT=USE_DCACHE_RAM

That ought to do it.

ron

[Li-Ta Lo]

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