[lkml]   [2008]   [Jun]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    Patch in this message
    Subject[patch 1/5] atl1: relax eeprom mac address error check
    2.6.25-stable review patch.  If anyone has any objections, please let us know.


    From: Radu Cristescu <>

    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

    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.

    [ backport to]

    Signed-off-by: Radu Cristescu <>
    Signed-off-by: Jay Cliburn <>
    Signed-off-by: Jeff Garzik <>
    Signed-off-by: Greg Kroah-Hartman <>

    drivers/net/atl1/atl1_hw.c | 1 -
    1 file changed, 1 deletion(-)

    --- a/drivers/net/atl1/atl1_hw.c
    +++ b/drivers/net/atl1/atl1_hw.c
    @@ -250,7 +250,6 @@ static int atl1_get_permanent_address(st
    memcpy(hw->perm_mac_addr, eth_addr, ETH_ALEN);
    return 0;
    - return 1;

    /* see if SPI FLAGS exist ? */

     \ /
      Last update: 2008-06-22 21:09    [W:0.021 / U:31.776 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site