[LinuxBIOS] [PATCH] Common code base for Advantech PCM-5820, ASI MB_5BLMP, BCOM WinNET100
Peter Stuge
peter at stuge.se
Tue Oct 23 16:39:00 CEST 2007
On Fri, Oct 19, 2007 at 01:51:28AM +0200, Uwe Hermann wrote:
> Comments welcome, Uwe.
On Fri, Oct 19, 2007 at 09:33:21PM +0200, Uwe Hermann wrote:
> > That sounds not right. Is there other way to refactor?
>
> I can't think of a better way to do it, but I'm open to
> suggestions.
I think this is too much work for v2.
> The #elif's are a cheap price when we can get rid of all the
> duplicated files we had before.
For v2, I think I'm with Ron. Duplicated code and files isn't that
bad. Yes, it's dirty, but rather than cleaning it up piece by piece
I'd like to work on v3.
> There are just 2-3 places where you have to add your board in the
> #elif.
>
> On the long run (in v3 maybe)
How long is "long" ?
> we could make the framework a bit more generic, to allow to plug in
> different Super I/Os etc., without having to #ifdef all of them...
Isn't this the case already?
Bottom line:
I would much rather see this effort go into v3 and a bonus is that
many of the boards supported by v2 would be working also with v3.
And:
#ifdef/#elif is really ugly. At least do #include instead?
//Peter
More information about the coreboot
mailing list