)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1000021,"name":"Antonio Borneo","email":"borneo.antonio@gmail.com","username":"borneoa"},"change_message_id":"aef40d711523507faf56d6379a97f9d7889c9598","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e57135ff_2b2cb03c","updated":"2023-09-09 12:55:40.000000000","message":"Thanks for this patch series.\njust one minor comment","commit_id":"adb430cc80473b3b5486e5b26fef9f004764d414"},{"author":{"_account_id":1002010,"name":"Bryan Brattlof","email":"hello@bryanbrattlof.com","username":"bryanbrattlof","status":"Texas Instruments"},"change_message_id":"7a054274555e8da7f6bf7470f2e39d5bd8d1ced0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"2ca6142b_0e0ccd6b","updated":"2023-09-09 01:53:38.000000000","message":"Was able to step/load/and read from the a53s and the wkup r5! thanks for sending this 😊","commit_id":"adb430cc80473b3b5486e5b26fef9f004764d414"},{"author":{"_account_id":1001678,"name":"Nishanth Menon","email":"nm@ti.com","username":"nmenon"},"change_message_id":"2564aae35d39c851ab3d742da7772858035d756d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"582da429_b08dacdb","in_reply_to":"e57135ff_2b2cb03c","updated":"2023-09-11 13:09:40.000000000","message":"Thank you for the review. fixed.","commit_id":"adb430cc80473b3b5486e5b26fef9f004764d414"},{"author":{"_account_id":1000021,"name":"Antonio Borneo","email":"borneo.antonio@gmail.com","username":"borneoa"},"change_message_id":"74e605bf1b71aa6df2721c00770e5887414f5612","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"8d4e9fc6_b899ede5","updated":"2023-09-17 14:19:20.000000000","message":"Thanks","commit_id":"1ef9cca6d0a180eef75861bbba53b213e72693b9"}],"tcl/target/ti_k3.cfg":[{"author":{"_account_id":1000021,"name":"Antonio Borneo","email":"borneo.antonio@gmail.com","username":"borneoa"},"change_message_id":"aef40d711523507faf56d6379a97f9d7889c9598","unresolved":true,"context_lines":[{"line_number":154,"context_line":"\t\tset _sysctrl_ap_unlock_offsets {0xf0 0x78}"},{"line_number":155,"context_line":""},{"line_number":156,"context_line":"\t\t# Overrides for am62p"},{"line_number":157,"context_line":"\t\tif { 0 \u003d\u003d [string compare $_soc am62p] } {"},{"line_number":158,"context_line":"\t\t\tset _K3_DAP_TAPID 0x0bb9d02f"},{"line_number":159,"context_line":"\t\t\tset R5_NAMES {wkup0_r5.0 mcu0_r5.0}"},{"line_number":160,"context_line":"\t\t}"}],"source_content_type":"text/x-ttcn-cfg","patch_set":1,"id":"ac19e160_27dca9d3","line":157,"updated":"2023-09-09 12:55:40.000000000","message":"I think it\u0027s more readable as:\nif { $_soc \u003d\u003d \"am62p\" } {\nor\nif { \"$_soc\" \u003d\u003d \"am62p\" } {\nIt\u0027s important to have the constant string \"am62p\" within double quotes (or within {}, as {am62p} )","commit_id":"adb430cc80473b3b5486e5b26fef9f004764d414"},{"author":{"_account_id":1001678,"name":"Nishanth Menon","email":"nm@ti.com","username":"nmenon"},"change_message_id":"2564aae35d39c851ab3d742da7772858035d756d","unresolved":false,"context_lines":[{"line_number":154,"context_line":"\t\tset _sysctrl_ap_unlock_offsets {0xf0 0x78}"},{"line_number":155,"context_line":""},{"line_number":156,"context_line":"\t\t# Overrides for am62p"},{"line_number":157,"context_line":"\t\tif { 0 \u003d\u003d [string compare $_soc am62p] } {"},{"line_number":158,"context_line":"\t\t\tset _K3_DAP_TAPID 0x0bb9d02f"},{"line_number":159,"context_line":"\t\t\tset R5_NAMES {wkup0_r5.0 mcu0_r5.0}"},{"line_number":160,"context_line":"\t\t}"}],"source_content_type":"text/x-ttcn-cfg","patch_set":1,"id":"fc631903_c98fa1a5","line":157,"in_reply_to":"ac19e160_27dca9d3","updated":"2023-09-11 13:09:40.000000000","message":"Ack","commit_id":"adb430cc80473b3b5486e5b26fef9f004764d414"},{"author":{"_account_id":1000021,"name":"Antonio Borneo","email":"borneo.antonio@gmail.com","username":"borneoa"},"change_message_id":"74e605bf1b71aa6df2721c00770e5887414f5612","unresolved":false,"context_lines":[{"line_number":154,"context_line":"\t\tset _sysctrl_ap_unlock_offsets {0xf0 0x78}"},{"line_number":155,"context_line":""},{"line_number":156,"context_line":"\t\t# Overrides for am62p"},{"line_number":157,"context_line":"\t\tif { \"$_soc\" \u003d\u003d \"am62p\" } {"},{"line_number":158,"context_line":"\t\t\tset _K3_DAP_TAPID 0x0bb9d02f"},{"line_number":159,"context_line":"\t\t\tset R5_NAMES {wkup0_r5.0 mcu0_r5.0}"},{"line_number":160,"context_line":"\t\t}"}],"source_content_type":"text/x-ttcn-cfg","patch_set":2,"id":"c256054c_6125c140","line":157,"updated":"2023-09-17 14:19:20.000000000","message":"there is no need for quotes around $_soc in TCL, but this way can be even more readable. No problem!","commit_id":"1ef9cca6d0a180eef75861bbba53b213e72693b9"}]}