[coreboot] Config.lb WTF

Peter Stuge peter at stuge.se
Sun Apr 12 16:23:47 CEST 2009

Carl-Daniel Hailfinger wrote:
> >>> -I$(TOP)/src -I. -nostdinc -nostdlib -fno-builtin -Wall -Os -c -S
> >>> $(MAINBOARD)/$(CACHE_AS_RAM_AUTO_C) -o $@"
> > Since the config tool supports include files, why not exploit that
> > capability to fix this problem.
> And how do you determine which version you have to include? Guess? Hope?
> I'd prefer a systematic solution.
> 1. Unify the code as much as possible.

Didn't you just do this? Or almost? :)

> 2. Explain the remaining differences (and spot the bugs).

Are there any?

> 3. Automatic/scripted conversion to include files.

Sure thing.

> > Longer term, we need a real set of Makefiles a la v3,

I completely agree.

> my 3-step plan is the only viable step forward.

In the short term, agreed. Please go ahead!


More information about the coreboot mailing list