In message <[EMAIL PROTECTED]> you wrote: > Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <[EMAIL PROTECTED]> > --- > examples/eepro100_eeprom.c | 5 ++++- > 1 files changed, 4 insertions(+), 1 deletions(-) > > diff --git a/examples/eepro100_eeprom.c b/examples/eepro100_eeprom.c > index 2b15d05..5f4eb78 100644 > --- a/examples/eepro100_eeprom.c > +++ b/examples/eepro100_eeprom.c > @@ -80,11 +80,13 @@ static inline short inw(long addr) > static inline void *memcpy(void *dst, const void *src, unsigned int len) > { > char *ret = dst; > + > while (len-- > 0) { > *ret++ = *((char *)src); > src++; > } > - return (void *)ret; > + > + return (void *)dst;
While technically correct, this is bogus. We have a variable ret, but we don't return it. And we have a variable dst, but we don't use it as destination pointer. Please change the *ret++ = *((char *)src); into *dst++ = *((char *)src); and leave all the rest. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: [EMAIL PROTECTED] On my planet, to rest is to rest -- to cease using energy. To me, it is quite illogical to run up and down on green grass, using energy, instead of saving it. -- Spock, "Shore Leave", stardate 3025.2 ------------------------------------------------------------------------- This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ _______________________________________________ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users