)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1001678,"name":"Nishanth Menon","email":"nm@ti.com","username":"nmenon"},"change_message_id":"ade4b22ec384c06da58437643a8473b6da1f53fa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"0b19d801_ddb2f7de","updated":"2021-10-04 15:11:52.000000000","message":"New patch as part of cleanup in series #2","commit_id":"02554428ce0239c4404e5fce0707f723d4e966e6"},{"author":{"_account_id":1001678,"name":"Nishanth Menon","email":"nm@ti.com","username":"nmenon"},"change_message_id":"e9cddab468a5aa0aefbc0be9935b11a9dd55fed2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"fe8dd153_6f0a5492","updated":"2022-03-01 12:35:25.000000000","message":"Thanks for catching this.","commit_id":"ea3677b4def3ed419467f3eeed39f6448991e5fe"}],"tcl/target/ti_k3.cfg":[{"author":{"_account_id":1000021,"name":"Antonio Borneo","email":"borneo.antonio@gmail.com","username":"borneoa"},"change_message_id":"f12fb9c3b8911fe1a7eef3ed17e4574ec2c20658","unresolved":true,"context_lines":[{"line_number":222,"context_line":"\t\tset _r5_name \"main1_r5\""},{"line_number":223,"context_line":"\t\tset _r5_core_id [expr {$_core - $_main1_base_core_id}]"},{"line_number":224,"context_line":"\t}"},{"line_number":225,"context_line":"\tcti create $_CTINAME.$_r5_name.$_r5_core_id -dap $_CHIPNAME.dap -ap-num 1 \\"},{"line_number":226,"context_line":"\t\t-baseaddr [lindex $R5_CTIBASE $_core]"},{"line_number":227,"context_line":""},{"line_number":228,"context_line":"\t# inactive core examination will fail - wait till startup of additional core"}],"source_content_type":"text/x-ttcn-cfg","patch_set":3,"id":"42de82c0_106a5712","line":225,"updated":"2022-03-01 00:11:07.000000000","message":"It look to me overly complicated and unreadable.\nProposal: together with \"set R5_DBGBASE {...}\" above, why not adding also:\nset R5_NAMES {mcu_5.0 mcu_5.1 main0_r5.0 main0_r5.1 main_r5.0 main1_r5.1}\nand here you simply extract [lint $R5_names $_core] ?\nI think this will let you drop the variable XX_base_core_id and XX_r5_cores.","commit_id":"ea3677b4def3ed419467f3eeed39f6448991e5fe"},{"author":{"_account_id":1001678,"name":"Nishanth Menon","email":"nm@ti.com","username":"nmenon"},"change_message_id":"e9cddab468a5aa0aefbc0be9935b11a9dd55fed2","unresolved":false,"context_lines":[{"line_number":222,"context_line":"\t\tset _r5_name \"main1_r5\""},{"line_number":223,"context_line":"\t\tset _r5_core_id [expr {$_core - $_main1_base_core_id}]"},{"line_number":224,"context_line":"\t}"},{"line_number":225,"context_line":"\tcti create $_CTINAME.$_r5_name.$_r5_core_id -dap $_CHIPNAME.dap -ap-num 1 \\"},{"line_number":226,"context_line":"\t\t-baseaddr [lindex $R5_CTIBASE $_core]"},{"line_number":227,"context_line":""},{"line_number":228,"context_line":"\t# inactive core examination will fail - wait till startup of additional core"}],"source_content_type":"text/x-ttcn-cfg","patch_set":3,"id":"2f09ac15_de953c79","line":225,"in_reply_to":"42de82c0_106a5712","updated":"2022-03-01 12:35:25.000000000","message":"Yes indeed. thanks.. cleaned up.","commit_id":"ea3677b4def3ed419467f3eeed39f6448991e5fe"}]}
