[U-Boot] [PATCH] Ethernet: let user know if there is no valid ethernet address

Wolfgang Denk wd at denx.de
Sat Jul 12 09:46:08 CEST 2014


Dear Pavel,

In message <20140711203007.GA1298 at amd.pavel.ucw.cz> you wrote:
>
> > > Ok, would it be acceptable to change 'count not set' message to 'could
> > > not set or invalid address' and print the address as well?
> > 
> > Print the address - if it is not set?  I think we should provide useful
> > error messages.  Either the address has not been set, then we should
> > say so, or it is invalid, then we should say that.
> 
> Well, it may be unset (00:00:...) or it may be invalid (b2:a3:...).

The address being not set at all, and it being set to
00:00:00:00:00:00, are two totally different cases, In the former,
it's, well, not set, and in the latter it is set to an invalid value.

> I think I do not understand you correctly. Yes, we should provide
> useful error messages, and current one is untrue and confusing.
> 
> Can you suggest a patch or messages you'd like to see?

If not set at all:

	"ERROR: eth?addr not set"

If set to an illegal value:

	"ERROR: eth?addr=XX:XX:XX:XX:XX:XX illegal value"

Best regards,

Wolfgang Denk

-- 
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
Plans break down. You cannot plan the future. Only presumptuous fools
plan. The wise man _steers_.        - Terry Pratchett, _Making_Money_


More information about the U-Boot mailing list