>>>>> "ladis" == Ladislav Michl <ladis@linux-mips.org> writes: Hi ladis> + unsigned long size = ip22_eeprom_read(&sgimc->eeprom, 17); Defining somewhere what 17 mean could be a good idea :) Except for that looks like a good gain, you remove a lot fo code :) Later, Juan. -- In theory, practice and theory are the same, but in practice they are different -- Larry McVoy