Activity for Martin Hierholzer
-
Martin Hierholzer modified a comment on discussion Open Discussion
Hi, I am not sure how to report bugs properly, the ticket system does not allow to create new issues here... I (resp. LSAN) have found a small memory leak: rpc_uaddr2taddr_af() is allocating, depending on the address type, the buffer ret->buf, but when called e.g. via pmap_set(), only the outer buffer ls freed via "free(na)". There should be a "free(na->buf)" before... Cheers, Martin
-
Hi, I am not sure how to report bugs properly, the ticket system does not allow to create new issues here... I (resp. asan) have found a small memory leak: rpc_uaddr2taddr_af() is allocating, depending on the address type, the buffer ret->buf, but when called e.g. via pmap_set(), only the outer buffer ls freed via "free(na)". There should be a "free(na->buf)" before... Cheers, Martin
-
I submitted the patch to Gerrit: https://review.openocd.org/c/openocd/+/7015 I hope I got your comments right :-)
-
Thanks for the fast reply! I will submit this to Gerrit once I have addressed your first comments. About the REVID: The interface switch was done between Revision 1.x and 2.x. This is described in this NXP document: https://www.nxp.com.cn/docs/en/application-note/AN4445.pdf My MK60 chip has the part number MK60DN512ZVMC10 and REVID = 7. The very first revisions 1.0 to 1.2 were not even called MK60 but PK60 but without a "Z" in the part number. The "Z" in the mask set obviously does not correspond...
-
Thanks for the fast reply! I will submit this to Gerrit once I have addressed your first comments. About the REVID: The interface switch was done between Revision 1.x and 2.x. This is described in this NXP document: https://www.nxp.com.cn/docs/en/application-note/AN4445.pdf My MK60 chip has the part number MK60DN512ZVMC10 and REVID = 7. The very first revisions 1.0 to 1.2 were not even called MK60 but PK60 but without a "Z" in the part number. The "Z" in the mask set obviously does not correspond...
-
I am having the same problem. I created a patch to fix this: https://github.com/mhier/openocd/commit/521f516ec6874b5f9da379247015530fb5a122a4 This has been tested with a MK60DN512ZVMC10. Due to lack of hardware, I cannot test this with revision 2.x chips (i.e. without the Z). Also I am not using the VLPR mode, so I did not test whether the switch to normal run mode works.
1