On Fri, Jul 8, 2011 at 12:35 PM, Øyvind Harboe <oyvind.har...@zylin.com> wrote:
> On Fri, Jul 8, 2011 at 12:31 PM, Drasko DRASKOVIC
> <drasko.drasko...@gmail.com> wrote:
>> On Fri, Jul 8, 2011 at 12:14 PM, Øyvind Harboe <oyvind.har...@zylin.com> 
>> wrote:
>>>> There is no particular need to cast this into uint8_t* and this can be
>>>> kept as a void*. Would that suppress the warnings ?
>>>
>>> It does look like this code is using uint8_t * in lieu of void *...
>>
>> Why ? It is just an address of 1-byte placeholder to which you would
>> like to copy byte read from the memory.
>
> It's not about what you do, it is about telling the compiler as much
> as possible about the types so that it can check for problems.
>
> I don't know the code too well to comment about which types
> should be used. Especially with big/small endian...

I know this code, and I can change it. I just do not see any casting
to unaligned access by visual inspection, and I can not reproduce the
problem. So I do not know where to start changing - everything looks
OK to me here.

BR,
Drasko
_______________________________________________
Openocd-development mailing list
Openocd-development@lists.berlios.de
https://lists.berlios.de/mailman/listinfo/openocd-development

Reply via email to