All of lore.kernel.org
 help / color / mirror / Atom feed
From: Philip Balister <philip@balister.org>
To: u-boot@lists.denx.de
Subject: [U-Boot] Bogus "Warning: failed to set MAC address" on Gumstix Overo
Date: Thu, 08 Sep 2011 11:08:04 -0400	[thread overview]
Message-ID: <4E68DA54.4000005@balister.org> (raw)
In-Reply-To: <20110908144037.EA969140875B@gemini.denx.de>

On 09/08/2011 10:40 AM, Wolfgang Denk wrote:
> Dear Philip Balister,
>
> In message<4E68C67F.2080002@opensdr.com>  you wrote:
>> When booting master on a Gumstix Overo based system, I get the following
>> messages during ethernet initialization:
>>
>> Net:   smc911x-0
>> Warning: failed to set MAC address
>>
>> (Note that patch [1] is needed to move the Warning to a new line,
>> otherwise it is after the previous line with no space)
>>
>> In this system, the ethernet chip is loading the MAC address from an
>> attached eeprom, so the warning is inaccurate. I've looked at the code
>
> The warning gets printed when eth_write_hwaddr() fails.
>
> What makes you claim that this is not correct?  There _is_ a problem
> when eth_write_hwaddr() returns an error code.

My point is the code shouldn't be trying to write the MAC address, when 
the MAC address is set in hardware. Earlier, (say 2011.6 but I am not 
being scientific here) u-boots did the write thing (IE not print the 
warning). This behavior has changed in the past few months.

Philip

  reply	other threads:[~2011-09-08 15:08 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-09-08 13:43 [U-Boot] Bogus "Warning: failed to set MAC address" on Gumstix Overo Philip Balister
2011-09-08 14:40 ` Wolfgang Denk
2011-09-08 15:08   ` Philip Balister [this message]
2011-09-08 15:17     ` Wolfgang Denk

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4E68DA54.4000005@balister.org \
    --to=philip@balister.org \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.