On Sat, Jul 30, 2011 at 4:26 AM, Mahr, Stefan <stefan.m...@sphairon.com> wrote:
> Hi Drasko.
>
>> Drasko DRASKOVIC (1):
>>      mips32 : Fixed memory byte access
>
> Your patch fixes the broken byte access, but not the big endian host issue.

Hi Stefan,
this is true. But my intention was not to go in changing all these
functions, especially since you volunteered to do it yourself :)).
Besides, I have an impression that you know what you are doing.

I was just trying to remove obviously blocking and annoying bug
quickly and not introducing any regression.


>Since both problems are tied together, I would prefer a more common solution.
Solution is more common, but the commit history is not clearer. You
are fixing several bugs in one patch...

> Attached patch hopefully fixes both issues. It should also fix alignment 
> warning that Øyvind had reported. Unfortunately I can't test it for the time 
> being.

Looks OK to me.


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

Reply via email to