Hi,
  I've been running 'sparse' on the kernel and it spotted that in the patch:
'ath5k: Use generic eeprom read from common ath_bus_opts struct.' 

the function
bool ath5k_pci_eeprom_read(struct ath_common *common, u32 offset, u16 *data)

is declared 'bool' yet it returns 0, -EIO, or -ETIMEDOUT ; sparse
is saying that the -EIO/-ETIMEDOUT's will get truncated - so you
probably aren't getting what's expected.

Dave


-- 
 -----Open up your eyes, open up your mind, open up your code -------   
/ Dr. David Alan Gilbert    |       Running GNU/Linux       | Happy  \ 
\ gro.gilbert @ treblig.org |                               | In Hex /
 \ _________________________|_____ http://www.treblig.org   |_______/
_______________________________________________
ath5k-devel mailing list
ath5k-devel@lists.ath5k.org
https://lists.ath5k.org/mailman/listinfo/ath5k-devel

Reply via email to