On Monday 30 March 2009, Kevin Hilman wrote: > What I didn't like was the at24 stuff in the new "common" code. > However, as I think about it, once I move to the new memory_accessor > series (which I will push shortly) the common code only has the > memory_accessor struct, not the at24_iface struct so that should be > fine. > > IOW, once you rebase this on top of the memory_accessor series, it > should be fine. For you it should mostly be s/at24_iface/memory_accessor/
I also disliked using a fixed offset, in particular one that forced use of an EEPROM with at least 32KB storage. You could pass the offset as the "context" used by that setup routine, and probably save an instruction while lifting those needless constraints: the Ethernet address could be stored anywhere, in persistent storage of any size. - Dave _______________________________________________ Davinci-linux-open-source mailing list [email protected] http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source
