)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1000160,"name":"Paul Fertser","email":"fercerpav@gmail.com","username":"pfertser"},"change_message_id":"0ca26e7ef72d0c0d6c03cb2bf5b990febf71b9bc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d57119c4_091295ff","updated":"2026-05-13 21:51:06.000000000","message":"Adapter speed is normally specified by the end user because it depends on quality of wiring used etc when an external debug adapter is attached.","commit_id":"b5ee5e807d18c2d242d9f7f0faf65caa82b76bc1"},{"author":{"_account_id":1000160,"name":"Paul Fertser","email":"fercerpav@gmail.com","username":"pfertser"},"change_message_id":"4a2bece30c2c6d6034803117a9260a6fa3e97af9","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"71c2dbc1_ae95415c","updated":"2026-05-13 21:49:00.000000000","message":"Hello Mark,\n\nThank you for the patch.\n\nThis doesn\u0027t define the flash bank for the on-board additional memory chip, is support for that planned? Why do you consider having this board file is better than just sourcing the target file directly?\n\nAnd why did Analog add USB UART converter but didn\u0027t just put there an additional small MCU with DAPLink to provide integrated debug (/and/ UART), do you know? :)","commit_id":"b5ee5e807d18c2d242d9f7f0faf65caa82b76bc1"},{"author":{"_account_id":1002482,"name":"Mark O\u0027Donovan","username":"shiftee"},"change_message_id":"4e0e75789c94dc8bf3e257f7d2bbb12078942dd7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"28135e89_e63d4fd6","updated":"2026-05-13 21:33:44.000000000","message":"Hi Paul,\nWould you be able to take a look at this at some point?\nThanks","commit_id":"b5ee5e807d18c2d242d9f7f0faf65caa82b76bc1"},{"author":{"_account_id":1002482,"name":"Mark O\u0027Donovan","username":"shiftee"},"change_message_id":"9f29dfae70a5caccb4eb51954b20608286ec79db","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e47833ab_b796e4df","updated":"2026-05-14 11:25:20.000000000","message":"It sounds like this board file should not exist unless the on-board memory chip is added.\n\nNote: There are over 200 references each in the boards dir configuring hardware reset and adapter speed so it is not entirely obvious that these things are not recommended in many cases.\n\nIf I want to drop the commit, how do I do so?\n\nAlso, if you could look at the other two tiny commits in the relation chain it would be appreciated.\nThanks","commit_id":"b5ee5e807d18c2d242d9f7f0faf65caa82b76bc1"},{"author":{"_account_id":1000160,"name":"Paul Fertser","email":"fercerpav@gmail.com","username":"pfertser"},"change_message_id":"f4d9b0169b1243ecc3c288d55e0b8d369bce04b6","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"992492c9_43e544e9","in_reply_to":"66a65945_9cefd80d","updated":"2026-05-14 12:53:05.000000000","message":"\u003e The warnings in question are generated by OpenOCD. The phrases \"will try\", and \"reasonable communication speed\" imply something is wrong.\n\nOr at least suboptimal, yes :)\n \n\u003e Warn : An adapter speed is not selected in the init scripts. OpenOCD will try to run the adapter at very low speed (100 kHz).\n\nThat\u0027s correct, when an external adapter is used an end user is supposed to specify approriate speed explicitly because it depends on the wiring.\n \n\u003e Warn : [max32xxx.cpu] Only resetting the Cortex-M core, use a reset-init event handler to reset any peripherals or configure hardware srst support.\n\nDoes this target really NOT support SYSRESETREQ? Because if support is there in hardware then `cortex_m reset_config sysresetreq` should be specified in the target config (it\u0027s not on by default because back then when Cortex-M was new too many MCU manufacturers neglected to implement it).","commit_id":"b5ee5e807d18c2d242d9f7f0faf65caa82b76bc1"},{"author":{"_account_id":1002482,"name":"Mark O\u0027Donovan","username":"shiftee"},"change_message_id":"9f29dfae70a5caccb4eb51954b20608286ec79db","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d601526c_c41601e6","in_reply_to":"71c2dbc1_ae95415c","updated":"2026-05-14 11:25:20.000000000","message":"Hi Paul,\nI was not planning on adding the on-board memory, but may find time to look at it.\n\nI added the board file out of an assumption that it was the correct thing to do in order to silence the adapter speed and hardware reset warnings.\nBut it sounds like it can be removed unless the on-board flash is added.\n\nI asked around but failed to find anyone who was involved with making those boards.\nIn my experience many engineers prefer to use their expensive \"professional\" debuggers, so it might be down to that.","commit_id":"b5ee5e807d18c2d242d9f7f0faf65caa82b76bc1"},{"author":{"_account_id":1002482,"name":"Mark O\u0027Donovan","username":"shiftee"},"change_message_id":"d4243503999eb77434bcdea7b69e508b3ef6806b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ff35aa98_7192873f","in_reply_to":"992492c9_43e544e9","updated":"2026-05-14 21:18:23.000000000","message":"Another thing I didn\u0027t know, thanks.\nThe board functionality still works and the warning is gone.\nShould it be gated by if {![using_hla]} {  ?","commit_id":"b5ee5e807d18c2d242d9f7f0faf65caa82b76bc1"},{"author":{"_account_id":1002482,"name":"Mark O\u0027Donovan","username":"shiftee"},"change_message_id":"345084fe8a0a90fa60552a5ee19d163d22d7f65b","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"66a65945_9cefd80d","in_reply_to":"b3cdcfa4_7c49b573","updated":"2026-05-14 11:56:13.000000000","message":"The warnings in question are generated by OpenOCD. The phrases \"will try\", and \"reasonable communication speed\" imply something is wrong.\n\nWarn : An adapter speed is not selected in the init scripts. OpenOCD will try to run the adapter at very low speed (100 kHz).\nWarn : To remove this warnings and achieve reasonable communication speed with the target, set \"adapter speed\" or \"jtag_rclk\" in the init scripts.\n\nWarn : [max32xxx.cpu] Only resetting the Cortex-M core, use a reset-init event handler to reset any peripherals or configure hardware srst support.","commit_id":"b5ee5e807d18c2d242d9f7f0faf65caa82b76bc1"},{"author":{"_account_id":1000160,"name":"Paul Fertser","email":"fercerpav@gmail.com","username":"pfertser"},"change_message_id":"506f0b928290edb37f8d47469f68f5de3a315efe","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b3cdcfa4_7c49b573","in_reply_to":"d601526c_c41601e6","updated":"2026-05-14 11:45:36.000000000","message":"\u003e I added the board file out of an assumption that it was the correct thing to do in order to silence the adapter speed and hardware reset warnings.\n\nI wonder what is the hardware reset warning exactly, what it is caused by, can you cite it please?\n\n\u003e I asked around but failed to find anyone who was involved with making those boards.\n\u003e In my experience many engineers prefer to use their expensive \"professional\" debuggers, so it might be down to that.\n\nThat\u0027s why most evaluation boards have both integrated OCD adapter and a standard connector (or at least some way) for an external one, and for a demo board the price of a cheap additional MCU vs price of a USB UART is negligible.","commit_id":"b5ee5e807d18c2d242d9f7f0faf65caa82b76bc1"},{"author":{"_account_id":1000160,"name":"Paul Fertser","email":"fercerpav@gmail.com","username":"pfertser"},"change_message_id":"506f0b928290edb37f8d47469f68f5de3a315efe","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"06568cb5_66b3147c","in_reply_to":"e47833ab_b796e4df","updated":"2026-05-14 11:45:36.000000000","message":"\u003e It sounds like this board file should not exist unless the on-board memory chip is added.\n\nProbably so, yes, unless we see the added value in the discussion here eventually :)\n\n\u003e Note: There are over 200 references each in the boards dir configuring hardware reset and adapter speed so it is not entirely obvious that these things are not recommended in many cases.\n\nWhen there\u0027s an adapter integrated on board then the reset connections and speed capabilities of the data link are known and fixed, so that\u0027s why you see so many hits.\n\n\u003e If I want to drop the commit, how do I do so?\n\nThere\u0027s an abandon button, but there\u0027s no hurry for that, let\u0027s talk more about the reset configuration first.\n\n\u003e Also, if you could look at the other two tiny commits in the relation chain it would be appreciated.\n\nI will soon, np.","commit_id":"b5ee5e807d18c2d242d9f7f0faf65caa82b76bc1"}],"tcl/board/max32666evkit.cfg":[{"author":{"_account_id":1000160,"name":"Paul Fertser","email":"fercerpav@gmail.com","username":"pfertser"},"change_message_id":"4a2bece30c2c6d6034803117a9260a6fa3e97af9","unresolved":true,"context_lines":[{"line_number":4,"context_line":"adapter speed 2000"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"# Set the reset pin configuration"},{"line_number":7,"context_line":"reset_config srst_only"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"source [find target/max32666.cfg]"}],"source_content_type":"text/x-ttcn-cfg","patch_set":1,"id":"5ee21e2c_71fdb40b","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":3},"updated":"2026-05-13 21:49:00.000000000","message":"It\u0027s an external adapter, so what if the user connects just jumper wires without SRST, will it not work at all reasonably? Then the requirement should be documented in a comment I guess.","commit_id":"b5ee5e807d18c2d242d9f7f0faf65caa82b76bc1"},{"author":{"_account_id":1000160,"name":"Paul Fertser","email":"fercerpav@gmail.com","username":"pfertser"},"change_message_id":"506f0b928290edb37f8d47469f68f5de3a315efe","unresolved":false,"context_lines":[{"line_number":4,"context_line":"adapter speed 2000"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"# Set the reset pin configuration"},{"line_number":7,"context_line":"reset_config srst_only"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"source [find target/max32666.cfg]"}],"source_content_type":"text/x-ttcn-cfg","patch_set":1,"id":"150df544_9c82d626","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":3},"in_reply_to":"1a772e1a_872581c0","updated":"2026-05-14 11:45:36.000000000","message":"\u003e I had been operating under the assumption that if a board connected SRST to the DAP then it made sense to configure it here.\n\u003e But as always, what a user can do, a user will do :)\n\nA development board meant to be used with an external adapter but not connecting SRST is a much rarer sight than a user with three jumper wires attaching an old st-link clone. So basically the current custom is to only specify reset config when it\u0027s fixed in hardware (in case of an on-board debugger) or when it can\u0027t work any other way (but then a clarification is added, and also such a config usually belongs to target cfg file if it\u0027s a target rather than the board quirk).","commit_id":"b5ee5e807d18c2d242d9f7f0faf65caa82b76bc1"},{"author":{"_account_id":1002482,"name":"Mark O\u0027Donovan","username":"shiftee"},"change_message_id":"9f29dfae70a5caccb4eb51954b20608286ec79db","unresolved":false,"context_lines":[{"line_number":4,"context_line":"adapter speed 2000"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"# Set the reset pin configuration"},{"line_number":7,"context_line":"reset_config srst_only"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"source [find target/max32666.cfg]"}],"source_content_type":"text/x-ttcn-cfg","patch_set":1,"id":"1a772e1a_872581c0","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":3},"in_reply_to":"5ee21e2c_71fdb40b","updated":"2026-05-14 11:25:20.000000000","message":"I had been operating under the assumption that if a board connected SRST to the DAP then it made sense to configure it here.\nBut as always, what a user can do, a user will do :)","commit_id":"b5ee5e807d18c2d242d9f7f0faf65caa82b76bc1"}]}