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
