diff options
author | Steve Glendinning <steve.glendinning@smsc.com> | 2009-02-15 23:55:01 +0100 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2009-02-20 09:38:51 +0100 |
commit | 196b7e1b9cca9e187bb61fa7d60f04f4ab2c0592 (patch) | |
tree | 50ba17d2078c3cd7465bfe1cafb6fc720ef83f63 /drivers/net/smsc9420.c | |
parent | sundance: missing parentheses? (diff) | |
download | linux-196b7e1b9cca9e187bb61fa7d60f04f4ab2c0592.tar.xz linux-196b7e1b9cca9e187bb61fa7d60f04f4ab2c0592.zip |
smsc9420: handle magic field of ethtool_eeprom
ethtool.h says the driver should set the magic field in get_eeprom and
verify it in set_eeprom. This patch adds this functionality using an
arbitary driver-specific magic value constant (0x9420).
Signed-off-by: Steve Glendinning <steve.glendinning@smsc.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/smsc9420.c')
-rw-r--r-- | drivers/net/smsc9420.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/drivers/net/smsc9420.c b/drivers/net/smsc9420.c index 83938e1953b0..4e15ae068b3f 100644 --- a/drivers/net/smsc9420.c +++ b/drivers/net/smsc9420.c @@ -413,6 +413,7 @@ static int smsc9420_ethtool_get_eeprom(struct net_device *dev, } memcpy(data, &eeprom_data[eeprom->offset], len); + eeprom->magic = SMSC9420_EEPROM_MAGIC; eeprom->len = len; return 0; } @@ -423,6 +424,9 @@ static int smsc9420_ethtool_set_eeprom(struct net_device *dev, struct smsc9420_pdata *pd = netdev_priv(dev); int ret; + if (eeprom->magic != SMSC9420_EEPROM_MAGIC) + return -EINVAL; + smsc9420_eeprom_enable_access(pd); smsc9420_eeprom_send_cmd(pd, E2P_CMD_EPC_CMD_EWEN_); ret = smsc9420_eeprom_write_location(pd, eeprom->offset, *data); |