[LinuxBIOS] A8N5X port

Uwe Hermann uwe at hermann-uwe.de
Fri Sep 28 16:02:56 CEST 2007


On Thu, Sep 27, 2007 at 08:31:16PM +0200, Stefan Reinauer wrote:
> * Uwe Hermann <uwe at hermann-uwe.de> [070927 19:14]:
> > No duplicated code whatsoever. Sure, this may not be too elegant
> > (there's some room for improvements, though), but it's definately a
> > _lot_ better than duplicating all those files.
>  
> Another question: If that code needs to be duplicated for each of those
> boards, is it generic (NB, SB, SUPERIO) code?

Parts of it, maybe. Didn't look closer, yet. Common component code should be
moved into the respective directory/files, of course. But there'll still
be many files/parts which are not common to the chipsets but rather
common to some (very similar) mainboards. That can (and should IMO) be
handled with a method similar to the one in my patch.

In this case, we could even rename the A8N-E directory to 'a8n-base' or
something, and make a8n-e, a8n5x, a8ne_fm_s be symlinks with only minor
differences in targets/*.


> > +if CONFIG_BOARD_A8N_E
> > +
> 
> ouch.. this is really nasty ;-) 

Yeah, not exactly elegant.

I didn't figure out how to do something like 

  if BOARD_NAME == "A8N-E"
    ...
  end

in Config.lb, that would improve the code (number of variables) a bit.


> And it will potentially make it hard for us in v3.. 

On the contrary, I think with kconfig we can handle this sort
of things even more easily and elegantly.

We even already have MAINBOARD_NAME etc. there, and those macros can
be used in Makefiles as well as C code, so no problem at all.


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/20070928/498381af/attachment.sig>


More information about the coreboot mailing list