[U-Boot] [PATCH] ARM: Add support for displaying second ethaddr in 'bdinfo' command

[Detlev Zundel]

Hi Gray,

> On 03/29/2011 10:49 PM, Wolfgang Denk wrote:
>> Dear Gray Remlin,
>>
>> In message<1301433395-25203-1-git-send-email-gryrmln at gmail.com>  you wrote:
>>> Signed-off-by: Gray Remlin<gryrmln at gmail.com>
>>> ---
>>>   common/cmd_bdinfo.c |    3 +++
>>>   1 files changed, 3 insertions(+), 0 deletions(-)
>> Why limit this to eth1addr?  What's the chances that ARM systems may
>> have more than 2 network interfaces?
>>
>> Best regards,
>>
>> Wolfgang Denk
>>
> Good question, I have already asked myself this, so I have stuck with 
> what I do know.
>
> 1. It has had the one ethaddr limit for a (relatively) long time, which 
> it seems no-one else 'required'\'submitted a patch' to change it.
> 2. I only know (with my very limited knowledge) of ARM boards with a 
> maximum of two interfaces as standard.
> 3. Why limit it to six (as in other parts of the source) ?

Because todays CPUs get drowned with six saturated links? ;)

> My answer:
> It is not my place to dictate policy, that is the role of the Project 
> Manager.
> And no, that is not a 'cop-out', it is the only way to avoid anarchy.

If we extend something, it is a good idea to sync to other places in the
sources.

Cheers
  Detlev

-- 
Question    : If you were redesigning UNIX, what would you do differently?
Ken Thompson: I'd spell creat with an e.
--
DENX Software Engineering GmbH,      MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich,  Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-40 Fax: (+49)-8142-66989-80 Email: dzu at denx.de