[U-Boot] [PATCH 3/3] net: switch device init to initcall

Jean-Christophe PLAGNIOL-VILLARD plagnioj at jcrosoft.com
Sat May 23 18:36:00 CEST 2009


On 08:41 Sat 23 May     , Ben Warren wrote:
>    Jean-Christophe,
> 
>    On Sat, May 23, 2009 at 8:08 AM, Jean-Christophe PLAGNIOL-VILLARD
>    <plagnioj at jcrosoft.com> wrote:
> 
>      apply to at91sam9263ek
> 
>      this will result to reduce the size of 2212 bytes
> 
>      Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj at jcrosoft.com>
>      ---
>       board/atmel/at91sam9263ek/at91sam9263ek.c |   12 ++++++------
>       include/netdev.h                          |   12 ------------
>       net/eth.c                                 |   16 ----------------
>       3 files changed, 6 insertions(+), 34 deletions(-)
> 
>    <snip>
> 
>      -       /* Try board-specific initialization first.  If it fails or
>      isn't
>      -        * present, try the cpu-specific initialization */
>      -       if (board_eth_init(bis) < 0)
>      -               cpu_eth_init(bis);
> 
>    Good idea, but your implementation is flawed.  It's critically important
>    that the prioritization of board ethernet initialization over CPU ethernet
>    initialization be preserved.  Your idea doesn't handle that (at least that
>    I can see).
with this only patch yes, but I'm preparing a full implementation that will
If I understand correctly the board_eth_init is unsed to overwrite the default
cpu_eth_init by passing different parameter

with a device/driver model this will be handle correctly

Best Regards,
J.


More information about the U-Boot mailing list