All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] [PATCH] mpc5200: Correct return value of memcpy function
@ 2017-01-24 12:47 thomas.graziadei at omicronenergy.com
  2017-01-24 13:23 ` Anatolij Gustschin
  2017-02-01 16:13 ` york sun
  0 siblings, 2 replies; 6+ messages in thread
From: thomas.graziadei at omicronenergy.com @ 2017-01-24 12:47 UTC (permalink / raw)
  To: u-boot

From: Mark Marshall <Mark.Marshall@omicron.at>

The memcpy() function returns a pointer to trg.

Signed-off-by: Mark Marshall <Mark.Marshall@omicron.at>
Reviewed-by: Thomas Graziadei <thomas.graziadei@omicronenergy.com>
---
 arch/powerpc/lib/memcpy_mpc5200.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/powerpc/lib/memcpy_mpc5200.c b/arch/powerpc/lib/memcpy_mpc5200.c
index 75a3ef9..7e5a005 100644
--- a/arch/powerpc/lib/memcpy_mpc5200.c
+++ b/arch/powerpc/lib/memcpy_mpc5200.c
@@ -31,7 +31,7 @@ void *memcpy(void *trg, const void *src, size_t len)
 	extern void* __memcpy(void *, const void *, size_t);
 	char *s = (char *)src;
 	char *t = (char *)trg;
-	void *dest = (void *)src;
+	void *dest = (void *)trg;
 
 	/*
 	 * Check is source address is in flash:
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [U-Boot] [PATCH] mpc5200: Correct return value of memcpy function
  2017-01-24 12:47 [U-Boot] [PATCH] mpc5200: Correct return value of memcpy function thomas.graziadei at omicronenergy.com
@ 2017-01-24 13:23 ` Anatolij Gustschin
  2017-02-01 16:13 ` york sun
  1 sibling, 0 replies; 6+ messages in thread
From: Anatolij Gustschin @ 2017-01-24 13:23 UTC (permalink / raw)
  To: u-boot

On Tue, 24 Jan 2017 13:47:31 +0100
thomas.graziadei at omicronenergy.com thomas.graziadei at omicronenergy.com wrote:

> From: Mark Marshall <Mark.Marshall@omicron.at>
> 
> The memcpy() function returns a pointer to trg.
> 
> Signed-off-by: Mark Marshall <Mark.Marshall@omicron.at>
> Reviewed-by: Thomas Graziadei <thomas.graziadei@omicronenergy.com>

Reviewed-by: Anatolij Gustschin <agust@denx.de>

--
Anatolij

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [U-Boot] [PATCH] mpc5200: Correct return value of memcpy function
  2017-01-24 12:47 [U-Boot] [PATCH] mpc5200: Correct return value of memcpy function thomas.graziadei at omicronenergy.com
  2017-01-24 13:23 ` Anatolij Gustschin
@ 2017-02-01 16:13 ` york sun
  1 sibling, 0 replies; 6+ messages in thread
From: york sun @ 2017-02-01 16:13 UTC (permalink / raw)
  To: u-boot

On 01/24/2017 05:41 AM, thomas.graziadei at omicronenergy.com wrote:
> From: Mark Marshall <Mark.Marshall@omicron.at>
>
> The memcpy() function returns a pointer to trg.
>
> Signed-off-by: Mark Marshall <Mark.Marshall@omicron.at>
> Reviewed-by: Thomas Graziadei <thomas.graziadei@omicronenergy.com>
> ---

Applied to u-boot-mpc85xx master, awaiting upstream. Thanks.

York

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [U-Boot] [PATCH] mpc5200: Correct return value of memcpy function
  2017-02-07 16:59 ` york sun
@ 2017-02-08  7:40   ` Thomas Graziadei
  0 siblings, 0 replies; 6+ messages in thread
From: Thomas Graziadei @ 2017-02-08  7:40 UTC (permalink / raw)
  To: u-boot

On 02/07/2017 05:59 PM, york sun wrote:
> On 01/24/2017 05:41 AM, thomas.graziadei at omicronenergy.com wrote:
>> From: Mark Marshall <Mark.Marshall@omicron.at>
>>
>> The memcpy() function returns a pointer to trg.
>>
>> Signed-off-by: Mark Marshall <Mark.Marshall@omicron.at>
>> Reviewed-by: Thomas Graziadei <thomas.graziadei@omicronenergy.com>
>> ---
>>   arch/powerpc/lib/memcpy_mpc5200.c | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/arch/powerpc/lib/memcpy_mpc5200.c b/arch/powerpc/lib/memcpy_mpc5200.c
>> index 75a3ef9..7e5a005 100644
>> --- a/arch/powerpc/lib/memcpy_mpc5200.c
>> +++ b/arch/powerpc/lib/memcpy_mpc5200.c
>> @@ -31,7 +31,7 @@ void *memcpy(void *trg, const void *src, size_t len)
>>   	extern void* __memcpy(void *, const void *, size_t);
>>   	char *s = (char *)src;
>>   	char *t = (char *)trg;
>> -	void *dest = (void *)src;
>> +	void *dest = (void *)trg;
>>
>>   	/*
>>   	 * Check is source address is in flash:
>>
> Looks like this is a duplicated patch.
Yes this is a duplicate.
>
> York
thanks,
Thomas

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [U-Boot] [PATCH] mpc5200: Correct return value of memcpy function
  2017-01-24 12:43 thomas.graziadei at omicronenergy.com
@ 2017-02-07 16:59 ` york sun
  2017-02-08  7:40   ` Thomas Graziadei
  0 siblings, 1 reply; 6+ messages in thread
From: york sun @ 2017-02-07 16:59 UTC (permalink / raw)
  To: u-boot

On 01/24/2017 05:41 AM, thomas.graziadei at omicronenergy.com wrote:
> From: Mark Marshall <Mark.Marshall@omicron.at>
>
> The memcpy() function returns a pointer to trg.
>
> Signed-off-by: Mark Marshall <Mark.Marshall@omicron.at>
> Reviewed-by: Thomas Graziadei <thomas.graziadei@omicronenergy.com>
> ---
>  arch/powerpc/lib/memcpy_mpc5200.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/powerpc/lib/memcpy_mpc5200.c b/arch/powerpc/lib/memcpy_mpc5200.c
> index 75a3ef9..7e5a005 100644
> --- a/arch/powerpc/lib/memcpy_mpc5200.c
> +++ b/arch/powerpc/lib/memcpy_mpc5200.c
> @@ -31,7 +31,7 @@ void *memcpy(void *trg, const void *src, size_t len)
>  	extern void* __memcpy(void *, const void *, size_t);
>  	char *s = (char *)src;
>  	char *t = (char *)trg;
> -	void *dest = (void *)src;
> +	void *dest = (void *)trg;
>
>  	/*
>  	 * Check is source address is in flash:
>

Looks like this is a duplicated patch.

York

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [U-Boot] [PATCH] mpc5200: Correct return value of memcpy function
@ 2017-01-24 12:43 thomas.graziadei at omicronenergy.com
  2017-02-07 16:59 ` york sun
  0 siblings, 1 reply; 6+ messages in thread
From: thomas.graziadei at omicronenergy.com @ 2017-01-24 12:43 UTC (permalink / raw)
  To: u-boot

From: Mark Marshall <Mark.Marshall@omicron.at>

The memcpy() function returns a pointer to trg.

Signed-off-by: Mark Marshall <Mark.Marshall@omicron.at>
Reviewed-by: Thomas Graziadei <thomas.graziadei@omicronenergy.com>
---
 arch/powerpc/lib/memcpy_mpc5200.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/powerpc/lib/memcpy_mpc5200.c b/arch/powerpc/lib/memcpy_mpc5200.c
index 75a3ef9..7e5a005 100644
--- a/arch/powerpc/lib/memcpy_mpc5200.c
+++ b/arch/powerpc/lib/memcpy_mpc5200.c
@@ -31,7 +31,7 @@ void *memcpy(void *trg, const void *src, size_t len)
 	extern void* __memcpy(void *, const void *, size_t);
 	char *s = (char *)src;
 	char *t = (char *)trg;
-	void *dest = (void *)src;
+	void *dest = (void *)trg;
 
 	/*
 	 * Check is source address is in flash:
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2017-02-08  7:40 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-01-24 12:47 [U-Boot] [PATCH] mpc5200: Correct return value of memcpy function thomas.graziadei at omicronenergy.com
2017-01-24 13:23 ` Anatolij Gustschin
2017-02-01 16:13 ` york sun
  -- strict thread matches above, loose matches on Subject: below --
2017-01-24 12:43 thomas.graziadei at omicronenergy.com
2017-02-07 16:59 ` york sun
2017-02-08  7:40   ` Thomas Graziadei

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.