Hello,

the correct fix should be:

((uint32_t*)buffer)[i+1] = ((uint32_t*)buffer_x)[i1];

This function merge buffer_y and buffer_x into buffer.


Regards,

Mathias


Am 15.03.2011 15:08, schrieb Øyvind Harboe:
> found by inspection, not confirmed.
> 
> Signed-off-by: Øyvind Harboe <[email protected]>
> ---
>  src/target/dsp563xx.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/src/target/dsp563xx.c b/src/target/dsp563xx.c
> index cb2025e..4969ed5 100644
> --- a/src/target/dsp563xx.c
> +++ b/src/target/dsp563xx.c
> @@ -1467,7 +1467,7 @@ static int dsp563xx_read_memory(struct target *target, 
> int mem_type, uint32_t ad
>       for(i=0,i1=0;i<count;i+=2,i1++)
>       {
>               ((uint32_t*)buffer)[i] = ((uint32_t*)buffer_y)[i1];
> -             ((uint32_t*)buffer)[i] = ((uint32_t*)buffer_x)[i1];
> +             ((uint32_t*)buffer)[i] = ((uint32_t*)buffer_x)[i1+1];
>       }
>  
>       free(buffer_y);

_______________________________________________
Openocd-development mailing list
[email protected]
https://lists.berlios.de/mailman/listinfo/openocd-development

Reply via email to