Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 16 Mar 2014 02:41:47 +0000 (UTC)
From:      Adrian Chadd <adrian@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r263224 - head/sys/mips/atheros
Message-ID:  <201403160241.s2G2flQb017755@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: adrian
Date: Sun Mar 16 02:41:47 2014
New Revision: 263224
URL: http://svnweb.freebsd.org/changeset/base/263224

Log:
  Handle the case where both arge0 and arge1 MAC addresses are available via
  'eeprommac'.
  
  The existing driver would just make arge units past 0 take the primary
  MAC and increment it by the unit number, without correct address wrapping.
  That has to be fixed at a later date.
  
  Tested:
  
  * Atheros DB120 reference obard

Modified:
  head/sys/mips/atheros/if_arge.c

Modified: head/sys/mips/atheros/if_arge.c
==============================================================================
--- head/sys/mips/atheros/if_arge.c	Sun Mar 16 02:34:33 2014	(r263223)
+++ head/sys/mips/atheros/if_arge.c	Sun Mar 16 02:41:47 2014	(r263224)
@@ -555,6 +555,7 @@ arge_attach(device_t dev)
 	long			eeprom_mac_addr = 0;
 	int			miicfg = 0;
 	int			readascii = 0;
+	int			local_mac = 0;
 
 	sc = device_get_softc(dev);
 	sc->arge_dev = dev;
@@ -576,6 +577,7 @@ arge_attach(device_t dev)
 	 */
 	 if (resource_long_value(device_get_name(dev), device_get_unit(dev),
 	    "eeprommac", &eeprom_mac_addr) == 0) {
+		local_mac = 1;
 		int i;
 		const char *mac =
 		    (const char *) MIPS_PHYS_TO_KSEG1(eeprom_mac_addr);
@@ -729,7 +731,22 @@ arge_attach(device_t dev)
 		sc->arge_eaddr[4] = (rnd >> 16) & 0xff;
 		sc->arge_eaddr[5] = (rnd >> 8) & 0xff;
 	}
-	if (sc->arge_mac_unit != 0)
+
+	/*
+	 * This is a little hairy and stupid.
+	 *
+	 * For some older boards, the arge1 mac isn't pulled from anywhere.
+	 * It's just assumed the MAC is the base MAC + 1.
+	 *
+	 * For other boards, there's multiple MAC addresses stored in EEPROM.
+	 *
+	 * So, if we did read the eeprommac for this particular interface,
+	 * let's use the address as given.  Otherwise, just add the MAC unit
+	 * counter to it.
+	 *
+	 * XXX TODO: we really should handle MAC byte wraparound!
+	 */
+	if (local_mac == 0 && sc->arge_mac_unit != 0)
 		sc->arge_eaddr[5] +=  sc->arge_mac_unit;
 
 	if (arge_dma_alloc(sc) != 0) {



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201403160241.s2G2flQb017755>