On Mon, Mar 14, 2011 at 15:33, Øyvind Harboe <[email protected]> wrote:
> Signed-off-by: Øyvind Harboe <[email protected]>
> ---
> src/target/dsp563xx.c | 12 ++++++------
> 1 files changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/src/target/dsp563xx.c b/src/target/dsp563xx.c
> index cb2025e..19f793e 100644
> --- a/src/target/dsp563xx.c
> +++ b/src/target/dsp563xx.c
...
> @@ -1466,8 +1466,8 @@ 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];
> + buf_set_u32(buffer + i*sizeof(uint32_t), 0, 32,
> buf_get_u32(buffer_y+i1*sizeof(uint32_t), 0, 32));
> + buf_set_u32(buffer + i*sizeof(uint32_t), 0, 32,
> buf_get_u32(buffer_x+i1*sizeof(uint32_t), 0, 32));
Was this ever correct? It looks like the 'x' line should have (i+1), like below.
> }
>
> free(buffer_y);
> @@ -1587,8 +1587,8 @@ static int dsp563xx_write_memory(struct target *target,
> int mem_type, uint32_t a
>
> for(i=0,i1=0;i<count;i+=2,i1++)
> {
> - ((uint32_t*)buffer_y)[i1] = ((uint32_t*)buffer)[i];
> - ((uint32_t*)buffer_x)[i1] = ((uint32_t*)buffer)[i+1];
> + buf_set_u32(buffer_y + i1*sizeof(uint32_t), 0, 32,
> buf_get_u32(buffer+i*sizeof(uint32_t), 0, 32));
> + buf_set_u32(buffer_x + i1*sizeof(uint32_t), 0, 32,
> buf_get_u32(buffer+(i+1)*sizeof(uint32_t), 0, 32));
> }
>
> err =
> dsp563xx_write_memory_core(target,MEM_Y,address,size,count/2,buffer_y);
--
Domen
_______________________________________________
Openocd-development mailing list
[email protected]
https://lists.berlios.de/mailman/listinfo/openocd-development
