]> git.kernelconcepts.de Git - karo-tx-linux.git/commit
atl1: relax eeprom mac address error check
authorRadu Cristescu <advantis@gmx.net>
Fri, 20 Jun 2008 01:27:55 +0000 (20:27 -0500)
committerGreg Kroah-Hartman <gregkh@suse.de>
Tue, 24 Jun 2008 21:08:28 +0000 (14:08 -0700)
commit2e77edd8f4b9ab4ef6ee4ba3fdc6cf50a972cce1
tree899eb2066d6cccfa01f5e3ff6cbb4812ea5363d0
parente1bcbd8de4501336d6ff3760ef75fb6232bfe625
atl1: relax eeprom mac address error check

upstream commit: 58c7821c4264a7ddd6f0c31c5caaf393b3897f10

The atl1 driver tries to determine the MAC address thusly:

- If an EEPROM exists, read the MAC address from EEPROM and
  validate it.
- If an EEPROM doesn't exist, try to read a MAC address from
  SPI flash.
- If that fails, try to read a MAC address directly from the
  MAC Station Address register.
- If that fails, assign a random MAC address provided by the
  kernel.

We now have a report of a system fitted with an EEPROM containing all
zeros where we expect the MAC address to be, and we currently handle
this as an error condition.  Turns out, on this system the BIOS writes
a valid MAC address to the NIC's MAC Station Address register, but we
never try to read it because we return an error when we find the all-
zeros address in EEPROM.

This patch relaxes the error check and continues looking for a MAC
address even if it finds an illegal one in EEPROM.

http://ubuntuforums.org/showthread.php?t=562617

[jacliburn@bellsouth.net: backport to 2.6.25.7]

Signed-off-by: Radu Cristescu <advantis@gmx.net>
Signed-off-by: Jay Cliburn <jacliburn@bellsouth.net>
Signed-off-by: Jeff Garzik <jgarzik@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/net/atl1/atl1_hw.c