> @@ -270,7 +270,7 @@ EXPORT_SYMBOL_GPL(eeprom_93cx6_readb); > * @eeprom: Pointer to eeprom structure > * @byte: Index from where we should start reading > * @data: target pointer where the information will have to be stored > - * @words: Number of bytes that should be read. > + * @bytes: Number of bytes that should be read. Now we have 'byte' and 'bytes' here as arguments which is confusing. I think renaming 'words' into 'num_bytes' would be even better.