[LinuxBIOS] [PATCH] Common code base for Advantech PCM-5820, ASI MB_5BLMP, BCOM WinNET100

Uwe Hermann uwe at hermann-uwe.de
Fri Oct 19 21:33:21 CEST 2007


On Fri, Oct 19, 2007 at 12:21:00PM -0700, Kenji Noguchi wrote:
> Hi Uwe
> 
> I just applied your patch and looked at the common/gx1-base/*.
> How do I add my board to the code tree?
> 
> It seems board specific codes which used to be in independent
> files are now placed in #ifdef chain.  Should I add another #elif...?

Yes, basically.

And you need a symlink from

 src/mainboard/XXXX/YYYYY -> src/mainboard/common/gx1-base


> 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.
The #elif's are a cheap price when we can get rid of all the duplicated
files we had before.

There are just 2-3 places where you have to add your board in the #elif.

On the long run (in v3 maybe) 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...


Uwe.
-- 
http://www.hermann-uwe.de  | http://www.holsham-traders.de
http://www.crazy-hacks.org | http://www.unmaintained-free-software.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20071019/a0ec2665/attachment.sig>


More information about the coreboot mailing list