From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ben Warren Date: Tue, 30 Mar 2010 14:57:07 -0700 Subject: [U-Boot] [PATCH 1/2 v2] net, fec_mxc: only setup the device enetaddr with eeprom value, if ethaddr is not setup In-Reply-To: <20100330203400.8EBE5E73028@gemini.denx.de> References: <4BB238E9.7060609@denx.de> <20100330203400.8EBE5E73028@gemini.denx.de> Message-ID: <4BB273B3.5090506@gmail.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Wolfgang, On 3/30/2010 1:34 PM, Wolfgang Denk wrote: > Dear Heiko Schocher, > > In message<4BB238E9.7060609@denx.de> you wrote: > >> if ethaddr is not setup in the environment, fill the device >> enetaddr with the contents of the eeprom, and only >> the device enetaddr, not the mac address registers! >> >> Tested on the magnesium board. >> >> Signed-off-by: Heiko Schocher >> --- >> - changes since v1 posted here: >> http://lists.denx.de/pipermail/u-boot/2010-March/069192.html >> >> - splitted in two patches as Wolfgang suggested >> > Thanks. Note that it would also have been an excellent idea to put > the responsible custodian on Cc: > > > >> drivers/net/fec_mxc.c | 9 +++++---- >> 1 files changed, 5 insertions(+), 4 deletions(-) >> > Applied, thanks. > > > Ben, this is (as far as I see it) an undisputed bug fix, so I'm > pulling this patch (and only this one from this series of 4) > directly. Hope this is ok with you. > > Sorry for not wading into this conversation earlier. I have issues with this driver, in particular that it isn't truly a 'MULTI' driver. This came to my attention when I noticed that Heiko's changes reference the 'ethaddr' environment variable. Hopefully somebody will fix it properly. In any case, as you say, this fixes a real bug and doesn't make things worse. Good enough, I guess. > > Best regards, > > Wolfgang Denk > > thanks, Ben