rt2x00: rt2800: convert read_eeprom functions to return an int value

Both the rtt2x00usb_eeprom_read and the ioremap
functions are allowed to fail, however their
return values are not checked in the read_eeprom
functions in the rt2800{pci,usb} drivers.

The patch adds the missing checks, and converts
all read_eeprom functions to return an int value,
so the error values can be propagated up to the
'rt2800_validate_eeprom' function.

Signed-off-by: Gabor Juhos <juhosg@openwrt.org>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
This commit is contained in:
Gabor Juhos 2012-12-29 14:51:51 +01:00 committed by John W. Linville
parent e1b97c9bc5
commit a02308e931
4 changed files with 43 additions and 21 deletions

View File

@ -4671,12 +4671,14 @@ static void rt2800_efuse_read(struct rt2x00_dev *rt2x00dev, unsigned int i)
mutex_unlock(&rt2x00dev->csr_mutex); mutex_unlock(&rt2x00dev->csr_mutex);
} }
void rt2800_read_eeprom_efuse(struct rt2x00_dev *rt2x00dev) int rt2800_read_eeprom_efuse(struct rt2x00_dev *rt2x00dev)
{ {
unsigned int i; unsigned int i;
for (i = 0; i < EEPROM_SIZE / sizeof(u16); i += 8) for (i = 0; i < EEPROM_SIZE / sizeof(u16); i += 8)
rt2800_efuse_read(rt2x00dev, i); rt2800_efuse_read(rt2x00dev, i);
return 0;
} }
EXPORT_SYMBOL_GPL(rt2800_read_eeprom_efuse); EXPORT_SYMBOL_GPL(rt2800_read_eeprom_efuse);
@ -4686,11 +4688,14 @@ static int rt2800_validate_eeprom(struct rt2x00_dev *rt2x00dev)
u16 word; u16 word;
u8 *mac; u8 *mac;
u8 default_lna_gain; u8 default_lna_gain;
int retval;
/* /*
* Read the EEPROM. * Read the EEPROM.
*/ */
rt2800_read_eeprom(rt2x00dev); retval = rt2800_read_eeprom(rt2x00dev);
if (retval)
return retval;
/* /*
* Start validation of the data that has been read. * Start validation of the data that has been read.

View File

@ -43,7 +43,7 @@ struct rt2800_ops {
const unsigned int offset, const unsigned int offset,
const struct rt2x00_field32 field, u32 *reg); const struct rt2x00_field32 field, u32 *reg);
void (*read_eeprom)(struct rt2x00_dev *rt2x00dev); int (*read_eeprom)(struct rt2x00_dev *rt2x00dev);
bool (*hwcrypt_disabled)(struct rt2x00_dev *rt2x00dev); bool (*hwcrypt_disabled)(struct rt2x00_dev *rt2x00dev);
int (*drv_write_firmware)(struct rt2x00_dev *rt2x00dev, int (*drv_write_firmware)(struct rt2x00_dev *rt2x00dev,
@ -117,11 +117,11 @@ static inline int rt2800_regbusy_read(struct rt2x00_dev *rt2x00dev,
return rt2800ops->regbusy_read(rt2x00dev, offset, field, reg); return rt2800ops->regbusy_read(rt2x00dev, offset, field, reg);
} }
static inline void rt2800_read_eeprom(struct rt2x00_dev *rt2x00dev) static inline int rt2800_read_eeprom(struct rt2x00_dev *rt2x00dev)
{ {
const struct rt2800_ops *rt2800ops = rt2x00dev->ops->drv; const struct rt2800_ops *rt2800ops = rt2x00dev->ops->drv;
rt2800ops->read_eeprom(rt2x00dev); return rt2800ops->read_eeprom(rt2x00dev);
} }
static inline bool rt2800_hwcrypt_disabled(struct rt2x00_dev *rt2x00dev) static inline bool rt2800_hwcrypt_disabled(struct rt2x00_dev *rt2x00dev)
@ -207,7 +207,7 @@ int rt2800_enable_radio(struct rt2x00_dev *rt2x00dev);
void rt2800_disable_radio(struct rt2x00_dev *rt2x00dev); void rt2800_disable_radio(struct rt2x00_dev *rt2x00dev);
int rt2800_efuse_detect(struct rt2x00_dev *rt2x00dev); int rt2800_efuse_detect(struct rt2x00_dev *rt2x00dev);
void rt2800_read_eeprom_efuse(struct rt2x00_dev *rt2x00dev); int rt2800_read_eeprom_efuse(struct rt2x00_dev *rt2x00dev);
int rt2800_probe_hw(struct rt2x00_dev *rt2x00dev); int rt2800_probe_hw(struct rt2x00_dev *rt2x00dev);

View File

@ -90,17 +90,22 @@ static void rt2800pci_mcu_status(struct rt2x00_dev *rt2x00dev, const u8 token)
} }
#if defined(CONFIG_RALINK_RT288X) || defined(CONFIG_RALINK_RT305X) #if defined(CONFIG_RALINK_RT288X) || defined(CONFIG_RALINK_RT305X)
static void rt2800pci_read_eeprom_soc(struct rt2x00_dev *rt2x00dev) static int rt2800pci_read_eeprom_soc(struct rt2x00_dev *rt2x00dev)
{ {
void __iomem *base_addr = ioremap(0x1F040000, EEPROM_SIZE); void __iomem *base_addr = ioremap(0x1F040000, EEPROM_SIZE);
if (!base_addr)
return -ENOMEM;
memcpy_fromio(rt2x00dev->eeprom, base_addr, EEPROM_SIZE); memcpy_fromio(rt2x00dev->eeprom, base_addr, EEPROM_SIZE);
iounmap(base_addr); iounmap(base_addr);
return 0;
} }
#else #else
static inline void rt2800pci_read_eeprom_soc(struct rt2x00_dev *rt2x00dev) static inline int rt2800pci_read_eeprom_soc(struct rt2x00_dev *rt2x00dev)
{ {
return -ENOMEM;
} }
#endif /* CONFIG_RALINK_RT288X || CONFIG_RALINK_RT305X */ #endif /* CONFIG_RALINK_RT288X || CONFIG_RALINK_RT305X */
@ -135,7 +140,7 @@ static void rt2800pci_eepromregister_write(struct eeprom_93cx6 *eeprom)
rt2x00pci_register_write(rt2x00dev, E2PROM_CSR, reg); rt2x00pci_register_write(rt2x00dev, E2PROM_CSR, reg);
} }
static void rt2800pci_read_eeprom_pci(struct rt2x00_dev *rt2x00dev) static int rt2800pci_read_eeprom_pci(struct rt2x00_dev *rt2x00dev)
{ {
struct eeprom_93cx6 eeprom; struct eeprom_93cx6 eeprom;
u32 reg; u32 reg;
@ -164,6 +169,8 @@ static void rt2800pci_read_eeprom_pci(struct rt2x00_dev *rt2x00dev)
eeprom_93cx6_multiread(&eeprom, EEPROM_BASE, rt2x00dev->eeprom, eeprom_93cx6_multiread(&eeprom, EEPROM_BASE, rt2x00dev->eeprom,
EEPROM_SIZE / sizeof(u16)); EEPROM_SIZE / sizeof(u16));
return 0;
} }
static int rt2800pci_efuse_detect(struct rt2x00_dev *rt2x00dev) static int rt2800pci_efuse_detect(struct rt2x00_dev *rt2x00dev)
@ -171,13 +178,14 @@ static int rt2800pci_efuse_detect(struct rt2x00_dev *rt2x00dev)
return rt2800_efuse_detect(rt2x00dev); return rt2800_efuse_detect(rt2x00dev);
} }
static inline void rt2800pci_read_eeprom_efuse(struct rt2x00_dev *rt2x00dev) static inline int rt2800pci_read_eeprom_efuse(struct rt2x00_dev *rt2x00dev)
{ {
rt2800_read_eeprom_efuse(rt2x00dev); return rt2800_read_eeprom_efuse(rt2x00dev);
} }
#else #else
static inline void rt2800pci_read_eeprom_pci(struct rt2x00_dev *rt2x00dev) static inline int rt2800pci_read_eeprom_pci(struct rt2x00_dev *rt2x00dev)
{ {
return -EOPNOTSUPP;
} }
static inline int rt2800pci_efuse_detect(struct rt2x00_dev *rt2x00dev) static inline int rt2800pci_efuse_detect(struct rt2x00_dev *rt2x00dev)
@ -185,8 +193,9 @@ static inline int rt2800pci_efuse_detect(struct rt2x00_dev *rt2x00dev)
return 0; return 0;
} }
static inline void rt2800pci_read_eeprom_efuse(struct rt2x00_dev *rt2x00dev) static inline int rt2800pci_read_eeprom_efuse(struct rt2x00_dev *rt2x00dev)
{ {
return -EOPNOTSUPP;
} }
#endif /* CONFIG_PCI */ #endif /* CONFIG_PCI */
@ -970,14 +979,18 @@ static irqreturn_t rt2800pci_interrupt(int irq, void *dev_instance)
/* /*
* Device probe functions. * Device probe functions.
*/ */
static void rt2800pci_read_eeprom(struct rt2x00_dev *rt2x00dev) static int rt2800pci_read_eeprom(struct rt2x00_dev *rt2x00dev)
{ {
int retval;
if (rt2x00_is_soc(rt2x00dev)) if (rt2x00_is_soc(rt2x00dev))
rt2800pci_read_eeprom_soc(rt2x00dev); retval = rt2800pci_read_eeprom_soc(rt2x00dev);
else if (rt2800pci_efuse_detect(rt2x00dev)) else if (rt2800pci_efuse_detect(rt2x00dev))
rt2800pci_read_eeprom_efuse(rt2x00dev); retval = rt2800pci_read_eeprom_efuse(rt2x00dev);
else else
rt2800pci_read_eeprom_pci(rt2x00dev); retval = rt2800pci_read_eeprom_pci(rt2x00dev);
return retval;
} }
static const struct ieee80211_ops rt2800pci_mac80211_ops = { static const struct ieee80211_ops rt2800pci_mac80211_ops = {

View File

@ -735,13 +735,17 @@ static void rt2800usb_fill_rxdone(struct queue_entry *entry,
/* /*
* Device probe functions. * Device probe functions.
*/ */
static void rt2800usb_read_eeprom(struct rt2x00_dev *rt2x00dev) static int rt2800usb_read_eeprom(struct rt2x00_dev *rt2x00dev)
{ {
int retval;
if (rt2800_efuse_detect(rt2x00dev)) if (rt2800_efuse_detect(rt2x00dev))
rt2800_read_eeprom_efuse(rt2x00dev); retval = rt2800_read_eeprom_efuse(rt2x00dev);
else else
rt2x00usb_eeprom_read(rt2x00dev, rt2x00dev->eeprom, retval = rt2x00usb_eeprom_read(rt2x00dev, rt2x00dev->eeprom,
EEPROM_SIZE); EEPROM_SIZE);
return retval;
} }
static int rt2800usb_probe_hw(struct rt2x00_dev *rt2x00dev) static int rt2800usb_probe_hw(struct rt2x00_dev *rt2x00dev)