[U-Boot-Users] [PATCH-1/2] LAN91C111

From: Woodruff, Richard <r-woodruff2@ti.com>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] [PATCH-1/2] LAN91C111
Date: Mon, 3 Nov 2003 06:13:07 -0600	[thread overview]
Message-ID: <71555548814716479478431542AA5F8A058F82@dlee2k98.ent.ti.com> (raw)

I've been using the 91111 for a while now.  I've not had any real problems
with it.  As the environment I'm in has some 30 odd boards, I've never used
the hard coded mac address.  

Regards,

Richard W.

-----Original Message-----
From: u-boot-users-admin@lists.sourceforge.net
[mailto:u-boot-users-admin at lists.sourceforge.net] On Behalf Of Stephan Linz
Sent: Monday, November 03, 2003 5:02 AM
To: u-boot-users at lists.sourceforge.net
Subject: [U-Boot-Users] [PATCH-1/2] LAN91C111

Hi all,

in context of one of my emails last week:
http://sourceforge.net/mailarchive/forum.php?thread_id=3364771&forum_id=1289
8

I'm wondering about how many people are using the LAN91C111 driver without 
significant problems. There is (just was :-) a fat signed/unsigned bug in 
drivers/smc91111.c which is significant at NIOS plattforms when you are
using 
MAC addresses with address elements above 0x7f. Example: 00:07:ed:0a:a4:7b 
goes over to 00:07:ed:ff:a4:ff inside the chip. Simple solution:

	static char smc_mac_addr[] = { . . . };

have to be:

	static char unsigned smc_mac_addr[6] = { . . . };

In attatchment you will find my smc91111_fix patch. It includes some other 
minor fixes I've made. This patch has been working successful at NIOS
Stratix 
and Cyclone boards.

Please try out and patch CVS tree.

Best Regards,
-- 
Mit freundlichen Gruessen
Stephan Linz

======================================================================
Stephan Linz
Softwareentwicklung

MAZeT GmbH                   Email: mailto:linz at mazet.de
G?schwitzer Str. 32          Tel. : (3641) 2809-55
D-07745 Jena                 Fax  : (3641) 2809-12

Besuchen Sie bitte unsere Web-Seiten: http://www.MAZeT.de
======================================================================