On Tue, Mar 25, 2008 at 3:04 AM, Stefan Roese <[EMAIL PROTECTED]> wrote:
> On Saturday 22 March 2008, Ben Warren wrote:
>
>  <snip>
>
>
>  > >  All I'm asking for is to make sure that this can be configured  in  a
>  > >  board  specific  way.  If  there  is a zensible default setting which
>  > >  covers most cases without need for board-specific stuff  that's  just
>  > >  all the better.
>  >
>  > How about something like this:
>  >
>  > #ifdef CONFIG_ETH_INIT_DEFAULT
>  >         CFG_ETH_INIT_DEFAULT(bis);
>  > #else
>  >         board_eth_init(bis);
>  > #endif
>
>  Using Markus's idea, why not use a cpu (platform) specific *and* a board
>  specific init function, both with an empty weak alias in the common eth.c
>  code:
>
>         cpu_eth_init(bis);
>         board_eth_init(bis);
>

Sorry I missed that.  Excellent idea.

regards,
Ben

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Reply via email to