Carl-Daniel Hailfinger wrote: > >>> action "$(CC) $(DISTRO_CFLAGS) $(CFLAGS) $(CPPFLAGS) $(DEBUG_CFLAGS) > >>> -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! //Peter -- coreboot mailing list: [email protected] http://www.coreboot.org/mailman/listinfo/coreboot
