)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1000687,"name":"Tomas Vanek","display_name":"Tomas Vanek","email":"vanekt@fbl.cz","username":"vanekt"},"change_message_id":"975d8e51d043ae8c87f1560a94955acef35c6851","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c41f398e_6af50c70","updated":"2025-02-12 11:07:20.000000000","message":"Do you intend to drop `adapter name` command later?\nSeems me an unnecessary duplicity to have two commands to get the selected adapter name.","commit_id":"eae861923221528c51a2bb45072c99f281c2fe57"},{"author":{"_account_id":1000021,"name":"Antonio Borneo","email":"borneo.antonio@gmail.com","username":"borneoa"},"change_message_id":"d490f2ed887980f3c2990baa01b6dce9df7ed5fd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"cb078b01_e05104a4","updated":"2025-02-12 12:44:20.000000000","message":"Thanks for the review","commit_id":"eae861923221528c51a2bb45072c99f281c2fe57"},{"author":{"_account_id":1000687,"name":"Tomas Vanek","display_name":"Tomas Vanek","email":"vanekt@fbl.cz","username":"vanekt"},"change_message_id":"435536ae645a22ab904313d42786ef6b43929cd6","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"9ddc5149_730754db","in_reply_to":"3a172d95_86c4b294","updated":"2025-02-12 13:40:19.000000000","message":"Honestly my only preference is to avoid unnecessary changes of the existing command interface and extending the deprecation lists.\n\nOpenOCD has several `names` comands (`cti names`, `dap names`, `target names`, `tpiu names`) versus `targets`, `flash list` and `flash banks` for listing of configuration created entities.\nFor compiled-in drivers we have `adapter list` and `nand drivers`.\nWe get current selections by `target current`, `adapter name` and many setting commands w/o parameter. Did I miss something?\nWell little bit mess IMO. I understand your motivation however I don\u0027t think that replacing `adapter name` by `adapter driver` would make much difference.","commit_id":"eae861923221528c51a2bb45072c99f281c2fe57"},{"author":{"_account_id":1000021,"name":"Antonio Borneo","email":"borneo.antonio@gmail.com","username":"borneoa"},"change_message_id":"fe5a10d0d142d4501336e276f4b08696fdf3065c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"0853b87e_b7347097","in_reply_to":"9ddc5149_730754db","updated":"2025-02-12 14:28:52.000000000","message":"Fine for me.\nLet\u0027s drop this. I will rework 8679.","commit_id":"eae861923221528c51a2bb45072c99f281c2fe57"},{"author":{"_account_id":1000021,"name":"Antonio Borneo","email":"borneo.antonio@gmail.com","username":"borneoa"},"change_message_id":"d490f2ed887980f3c2990baa01b6dce9df7ed5fd","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"3a172d95_86c4b294","in_reply_to":"c41f398e_6af50c70","updated":"2025-02-12 12:44:20.000000000","message":"Looks like in this series I have missed the existence of command `adapter name`.\nI have extended here `adapter driver` to use it in the last patch 8679 of this series to get if hla was already selected.\n\nWe can either:\n- drop this patch and modify 8679 to use `adapter name` (minor rebase required too), or\n- keep this series as is, drop `adapter name` and change the 3 uses in `tcl/target/psoc4.cfg` and `tcl/target/ti_k3.cfg`. But we need to change `adapter driver` to be `COMMAND_ANY` for using it in `psoc4.cfg`.\n\nI don\u0027t have strong preferences as both will work for this series.\nAs I wrote in this commit message, it could be inline with OpenOCD style to have `adapter driver` returning the name.\nI can add the drop of `adapter name` at the end of this series","commit_id":"eae861923221528c51a2bb45072c99f281c2fe57"}]}
