)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1000687,"name":"Tomas Vanek","display_name":"Tomas Vanek","email":"vanekt@fbl.cz","username":"vanekt"},"change_message_id":"9c02e8026196806eaed2c2e8a89b9178a2a59047","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c314a851_18d622b4","updated":"2025-11-23 06:20:46.000000000","message":"Thanks for the series, Daniel.\n\nJust a remainder that the new ap gateway option should be documented in openocd.texi","commit_id":"c210d2e2de12e3cebef8c3bede3d343a9877322d"},{"author":{"_account_id":1000021,"name":"Antonio Borneo","email":"borneo.antonio@gmail.com","username":"borneoa"},"change_message_id":"a8403a0be1f6229c4fbcd8a9a156bdf87078ea54","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"8f6fe153_748eefef","in_reply_to":"3eb9483c_e0373b23","updated":"2025-11-24 08:24:43.000000000","message":"Thanks for the series, Daniel. Really clean code!\n\nI think the flag `-ap-num-gateway` does not scale for multiple nesting. Do we need something different or do you have already a proposal?\n\nI prefer not merging the series as is to prevent changing several commands and their documentation to add `-ap-num-gateway` and then rework them again and deprecate the intermediate command syntax.\n\nI have just started reworking the adiv5 code to put in a single \"opaque\" struct the config parameters of an AP. Only few functions will handle the internals of the struct, while the rest of OpenOCD code will use it as a black box to get a handler to the AP. This would simplify changing later the internal representation by adding new information, e.g. the gateway or directly a chain of AP for the multiple nesting.\n\nI hit the code in `arm_tpiu_swo.c` that contains more than 200 lines deprecated that I plan to drop after the tag v1.0.0.\nThese lines contain too much direct access to the AP config parameters and make the rework much heavier.\nFor the moment I have dropped them as a first step for the rework, but this means merging such code after v1.0.0.\nI will complete the rework and then I will see if it would make sense to change current `arm_tpiu_swo.c` too, allowing a merge before v1.0.0.\nNevertheless v1.0.0 should land quite soon!","commit_id":"c210d2e2de12e3cebef8c3bede3d343a9877322d"},{"author":{"_account_id":1001661,"name":"Daniel Goehring","email":"dgoehrin@os.amperecomputing.com","username":"dgoehrin"},"change_message_id":"fcde032e5c901f2a2ba300cec57bb538fd529f0d","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"3eb9483c_e0373b23","in_reply_to":"c314a851_18d622b4","updated":"2025-11-24 02:59:32.000000000","message":"I\u0027ll update openocd.texi on the next patch update.","commit_id":"c210d2e2de12e3cebef8c3bede3d343a9877322d"}]}
