> I did not saw changes to the mips_m4k_write_memory() and
> mips_m4k_read_memory() (where my byte-flip code lives) in the patches
> you presented. Maybe I did not follow carefully...

I talk about the patch I sent a couple of minutes ago.


>> I am not very happy with my crappy patch code, but if it is confirmed to
>> work we could find a smarter way.
>
> I still did not test it. I can confirm that it worked as I expected
> before your changes, though. But if these changes would help the
> design they should be integrated and then the rest of the code
> corrected, so that the OpenOCD code stays as consistent as possible.

The problem is, that some parts of the code did not work for big endian HOSTs.



>> lead to the NULL pointer dereference in the time of jtag data scan 
>> execution (r is a automatic variable, local to the
>> mips_ejtag_fastdata_scan() function) ?
> Correction, not NULL pointer, but some trash value pointer from the
> no longer valid stack.

No, buf_get_u32 fills r[4]. The initial value does not matter.

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

Reply via email to