[U-Boot] [PATCH 01/27 v2] Blackfin: bfin_mac: force board_get_enetaddr() usage

[Mike Frysinger]

On Thursday 29 January 2009 05:45:23 Wolfgang Denk wrote:
> In message <1233212358-27956-1-git-send-email-vapier at gentoo.org> you wrote:
> > Since the on-chip MAC does not have an eeprom or similar interface, force
> > all Blackfin boards that use this to define their own
> > board_get_enetaddr() function.
>
> I still think this belongs into board specific code, not into a
> generic driver.

the driver isnt really generic.  it's specific to the hardware that exists 
inside of Blackfin chips.  the hardware has no support at all for storing the 
MAC in eeprom or any other place.  thus it forces boards to declare some 
method for finding the MAC since there is literally no way for the hardware 
itself to do it.

what you propose wouldnt have been possible before, but with Ben's work it 
should be easy to move to board_eth_init().

as for generic code, i'd point out that lib_ppc/board.c is a much worse 
offender in this regard ...
-mike