Skip to contentSkip to conversation

Bogus "Warning: failed to set MAC address" on Gumstix Overo

U-Boot Philip Balister 3 participants 4 messages Sep 8, 2011

lore
Share & tools
lore
#0 lore raw

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 some, and it is not clear to me the best way to suppress this message for the case where the hardware configures its MAC address on reset.

Philip

[1] http://permalink.gmane.org/gmane.comp.boot-loaders.u-boot/107639

#1 lore raw

Dear Philip Balister,

Quoted 11 lines · In message <[email protected]> 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.

Best regards,

Wolfgang Denk

Signature · 5 lines
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Unser Kopf ist rund, damit das Denken die Richtung wechseln kann.
                                                   -- Francis Picabia
#2 lore raw
Quoted 12 lines · Wolfgang Denk

Dear Philip Balister,

Quoted 9 lines · In message<[email protected]> 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)

Quoted 7 lines
Quoted 2 lines

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

#3 lore raw

Dear Philip Balister,

Quoted 5 lines · In message <[email protected]> you wrote:

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.

Yes, we started unifying the code. Please see doc/README.drivers.eth

What is the problem with running eth_write_hwaddr() on this system? _This_ should be fixed.

Best regards,

Wolfgang Denk

Signature · 5 lines
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Pain is a thing of the mind.  The mind can be controlled.
	-- Spock, "Operation -- Annihilate!" stardate 3287.2