From: Mike Frysinger <vapier@gentoo.org>
Date: Tue, 27 Apr 2010 14:42:44 -0400
Subject: [U-Boot] [PATCH] Blackfin: bfin_mac: hook up new write_hwaddr
	function
In-Reply-To: <4BD72DE5.9030808@gmail.com>
References: <1272392128-13333-1-git-send-email-vapier@gentoo.org>
	<4BD72DE5.9030808@gmail.com>
Message-ID: <201004271442.45304.vapier@gentoo.org>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On Tuesday 27 April 2010 14:33:09 Ben Warren wrote:
> On 4/27/2010 11:15 AM, Mike Frysinger wrote:
> > -	bfin_EMAC_setup_addr(dev->enetaddr);
> > +	bfin_EMAC_setup_addr(dev);
> 
> Are you sure you still want to program it on every init() call?  There's
> nothing wrong with that, BTW...

that's the correct documented behavior ... otherwise, there's no way of 
syncing the env to the hardware.  this is init(), not initialize().
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
Url : http://lists.denx.de/pipermail/u-boot/attachments/20100427/e6381e33/attachment.pgp