Subject: Re: CVS commit: src/sys/arch/evbmips
To: Izumi Tsutsui <>
From: Garrett D'Amore <>
List: port-evbmips
Date: 02/23/2006 12:34:44
Izumi Tsutsui wrote:
> In article <>
> wrote:
>> We have a sanity check already -- the link will fail due to multiple
>> symbol definitions. :-)
> Well, that doesn't indicate what is wrong exactly.
Not exactly, but pretty darn close. :-)
>> I am generally *not* fond of #ifdef jungles, and your proposal creates
>> two of them.
>> The other problem with this is that I can imagine having a board
>> defintion that actually supports a board *family* (where the type of
>> board is run-time detectable via a register), and then what do you do? 
> Maybe MI board_info() can call board specific xxx_board_info()
> funtcions? Some of examples are ofppc/ofppc/platform.c and
> pmax/pmax/sysconf.c etc.
I am okay with that.  But at this point I don't really see much benefit
in it.  The big switch table just adds a thin layer of code.

If you want the big #ifdef table, we can do it, and rename the
board_info routines to xxx_board_info or somesuch.

    -- Garrett

Garrett D'Amore, Principal Software Engineer
Tadpole Computer / Computing Technologies Division,
General Dynamics C4 Systems
Phone: 951 325-2134  Fax: 951 325-2191