)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1001678,"name":"Nishanth Menon","email":"nm@ti.com","username":"nmenon"},"change_message_id":"5ed7abc0097e2c588af17abd88517ae45edaae69","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Sameer Srivastava \u003cs-srivastava4@ti.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2026-05-13 12:39:24 +0530"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"tcl: added am13e230x target and launchpad cfg"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"- target defines SRAM work area and flash region"},{"line_number":10,"context_line":"- board selects xds110 interface and speed"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"0870fd9b_75e0d818","line":7,"updated":"2026-05-13 12:58:21.000000000","message":"s/added/add","commit_id":"9a54d006289b09c815b2a70a3d691656c9ff6904"},{"author":{"_account_id":1002511,"name":"Sameer Srivastava","email":"sam33r012@gmail.com","username":"s-m33r"},"change_message_id":"6ebf7ccfa44c5845a4b8e09c9545c0f43571e964","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Sameer Srivastava \u003cs-srivastava4@ti.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2026-05-13 12:39:24 +0530"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"tcl: added am13e230x target and launchpad cfg"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"- target defines SRAM work area and flash region"},{"line_number":10,"context_line":"- board selects xds110 interface and speed"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"8a3add0d_62c12aac","line":7,"in_reply_to":"0870fd9b_75e0d818","updated":"2026-06-08 12:21:52.000000000","message":"Done","commit_id":"9a54d006289b09c815b2a70a3d691656c9ff6904"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":1001678,"name":"Nishanth Menon","email":"nm@ti.com","username":"nmenon"},"change_message_id":"9f43d47775c71b125d4642f0600508051694f4a0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"67286a24_f2c1038f","updated":"2026-05-13 12:59:23.000000000","message":"generic comment -\u003e I think the sequence of patches should be:\na) contrib for algo\nb) flash driver\nc) tcl file.\n\ngit rebase -i should allow you to reorder the patches I think.","commit_id":"9a54d006289b09c815b2a70a3d691656c9ff6904"},{"author":{"_account_id":1001678,"name":"Nishanth Menon","email":"nm@ti.com","username":"nmenon"},"change_message_id":"7dfcbf34ad43dc274142950e3c39b1e9d2af0193","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"c785c124_a5aee28c","updated":"2026-06-10 15:08:29.000000000","message":"Minor note: it is best to put the tcl addition as the last patch in the series.","commit_id":"ff40968c664abd17dcec3d336cd3329bf1beafb9"},{"author":{"_account_id":1002511,"name":"Sameer Srivastava","email":"sam33r012@gmail.com","username":"s-m33r"},"change_message_id":"0293736911bbdd8befbafc7766eac5ef7ce849c3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"133829cf_b3b0b449","in_reply_to":"66bbb03a_74dcc15e","updated":"2026-06-15 06:21:44.000000000","message":"Done","commit_id":"ff40968c664abd17dcec3d336cd3329bf1beafb9"},{"author":{"_account_id":1001678,"name":"Nishanth Menon","email":"nm@ti.com","username":"nmenon"},"change_message_id":"d293e30a257664edc00f8962a309ef382cb66806","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":4,"id":"66bbb03a_74dcc15e","in_reply_to":"c785c124_a5aee28c","updated":"2026-06-11 13:14:27.000000000","message":"the tcl references the am13 flash driver (flash bank ... am13 ...) which is not introduced until patch 9527 -\u003e just put the tcl addition as the last patch in the series.","commit_id":"ff40968c664abd17dcec3d336cd3329bf1beafb9"},{"author":{"_account_id":1001678,"name":"Nishanth Menon","email":"nm@ti.com","username":"nmenon"},"change_message_id":"92aceb542edb665a9c8a4a17db71a8b45d820c10","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"d91f1bc7_fddfe2e2","updated":"2026-06-16 16:30:56.000000000","message":"minor","commit_id":"00be3bd32ecaa4c846049314682812eb2bac41d3"},{"author":{"_account_id":1001678,"name":"Nishanth Menon","email":"nm@ti.com","username":"nmenon"},"change_message_id":"8bbe46897ac1315cbf908bed64918ad1061a8353","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"7e624b00_6e045e53","updated":"2026-06-17 14:42:58.000000000","message":"Antonio, Paul, things look good now, I think.","commit_id":"18c9a41babcc25086fc2b43a36974918f41190f7"}],"tcl/board/ti/am13e230x-launchpad.cfg":[{"author":{"_account_id":1000021,"name":"Antonio Borneo","email":"borneo.antonio@gmail.com","username":"borneoa"},"change_message_id":"ce61e34eb72fe2cdebd704cee2d2ee2938297f7e","unresolved":true,"context_lines":[{"line_number":5,"context_line":"source [find interface/xds110.cfg]"},{"line_number":6,"context_line":"adapter speed 10000"},{"line_number":7,"context_line":"source [find target/ti/am13e230x.cfg]"},{"line_number":8,"context_line":"init"},{"line_number":9,"context_line":"reset halt"}],"source_content_type":"text/x-ttcn-cfg","patch_set":1,"id":"9d2e2f3b_a0c6a088","line":9,"range":{"start_line":8,"start_character":1,"end_line":9,"end_character":10},"updated":"2026-05-11 09:42:16.000000000","message":"We do not use to put `init` in the board file. This would prevent adding other configuration commands in the command line of OpenOCD.\nAlso `reset halt` should not be part of the script. When GDB attaches to OpenOCD the `halt` command is executed automatically. If user need it from the telnet console, it should type it.\nPlease remove these last two commands","commit_id":"cb28a05c91e73629d2888ad8af6c30bde05dc69c"},{"author":{"_account_id":1002511,"name":"Sameer Srivastava","email":"sam33r012@gmail.com","username":"s-m33r"},"change_message_id":"ffa2b865bacc3a01d80d4ec90d84ca4ed398a1b4","unresolved":false,"context_lines":[{"line_number":5,"context_line":"source [find interface/xds110.cfg]"},{"line_number":6,"context_line":"adapter speed 10000"},{"line_number":7,"context_line":"source [find target/ti/am13e230x.cfg]"},{"line_number":8,"context_line":"init"},{"line_number":9,"context_line":"reset halt"}],"source_content_type":"text/x-ttcn-cfg","patch_set":1,"id":"5df4186c_06bd302a","line":9,"range":{"start_line":8,"start_character":1,"end_line":9,"end_character":10},"in_reply_to":"9d2e2f3b_a0c6a088","updated":"2026-05-13 07:11:24.000000000","message":"Done","commit_id":"cb28a05c91e73629d2888ad8af6c30bde05dc69c"}],"tcl/target/ti/am13e230x.cfg":[{"author":{"_account_id":1000021,"name":"Antonio Borneo","email":"borneo.antonio@gmail.com","username":"borneoa"},"change_message_id":"ce61e34eb72fe2cdebd704cee2d2ee2938297f7e","unresolved":true,"context_lines":[{"line_number":18,"context_line":"\tset _DAP_TAPID 0x6ba00477"},{"line_number":19,"context_line":"}"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"transport select jtag"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"set _DAP_ID $_DAP_TAPID"},{"line_number":24,"context_line":""}],"source_content_type":"text/x-ttcn-cfg","patch_set":1,"id":"b52961f0_f8e12e78","line":21,"updated":"2026-05-11 09:42:16.000000000","message":"The device supports both JTAG and SWD, as reported in the TRM https://www.ti.com/lit/ug/sprujf2a/sprujf2a.pdf\nSo it\u0027s incorrect to select one mode in the target file.\nDrop this line, add at line 8\n`source [find target/swj-dp.tcl]`\nand replace line 25 with\n`swj_newdap $_CHIPNAME cpu -irlen 4 -expected-id $_DAP_TAPID`\nbut I expect the value of `_DAP_TAPID` would not be correct for SWD, as other devices have different ID for JTAG and SWD, no idea for this specific chip.\n\nThen, you need to add the `transport select` command in the board file `am13e230x-launchpad.cfg`. I don\u0027t know if SWD will work, so you could stick at JTAG.","commit_id":"cb28a05c91e73629d2888ad8af6c30bde05dc69c"},{"author":{"_account_id":1002511,"name":"Sameer Srivastava","email":"sam33r012@gmail.com","username":"s-m33r"},"change_message_id":"c687dd62383a08b087fb2d58f3a37e1f7af7f04a","unresolved":true,"context_lines":[{"line_number":18,"context_line":"\tset _DAP_TAPID 0x6ba00477"},{"line_number":19,"context_line":"}"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"transport select jtag"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"set _DAP_ID $_DAP_TAPID"},{"line_number":24,"context_line":""}],"source_content_type":"text/x-ttcn-cfg","patch_set":1,"id":"cc30e294_cee52bc2","line":21,"in_reply_to":"b52961f0_f8e12e78","updated":"2026-05-13 07:10:53.000000000","message":"Thanks, there is a different ID for SWD that I am now passing as another `-expected-id` to the `swj_newdap` call. Also moved `transport select` to the board file.","commit_id":"cb28a05c91e73629d2888ad8af6c30bde05dc69c"},{"author":{"_account_id":1001678,"name":"Nishanth Menon","email":"nm@ti.com","username":"nmenon"},"change_message_id":"8bbe46897ac1315cbf908bed64918ad1061a8353","unresolved":false,"context_lines":[{"line_number":18,"context_line":"\tset _DAP_TAPID 0x6ba00477"},{"line_number":19,"context_line":"}"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"transport select jtag"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"set _DAP_ID $_DAP_TAPID"},{"line_number":24,"context_line":""}],"source_content_type":"text/x-ttcn-cfg","patch_set":1,"id":"b17ec1f5_b071318e","line":21,"in_reply_to":"cc30e294_cee52bc2","updated":"2026-06-17 14:42:58.000000000","message":"Ack","commit_id":"cb28a05c91e73629d2888ad8af6c30bde05dc69c"},{"author":{"_account_id":1000021,"name":"Antonio Borneo","email":"borneo.antonio@gmail.com","username":"borneoa"},"change_message_id":"ce61e34eb72fe2cdebd704cee2d2ee2938297f7e","unresolved":true,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"transport select jtag"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"set _DAP_ID $_DAP_TAPID"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"jtag newtap $_CHIPNAME cpu -irlen 4 -expected-id $_DAP_TAPID"},{"line_number":26,"context_line":""}],"source_content_type":"text/x-ttcn-cfg","patch_set":1,"id":"d7fcc9a7_559825fe","line":23,"updated":"2026-05-11 09:42:16.000000000","message":"This is not used. You can drop it","commit_id":"cb28a05c91e73629d2888ad8af6c30bde05dc69c"},{"author":{"_account_id":1002511,"name":"Sameer Srivastava","email":"sam33r012@gmail.com","username":"s-m33r"},"change_message_id":"ffa2b865bacc3a01d80d4ec90d84ca4ed398a1b4","unresolved":false,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"transport select jtag"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"set _DAP_ID $_DAP_TAPID"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"jtag newtap $_CHIPNAME cpu -irlen 4 -expected-id $_DAP_TAPID"},{"line_number":26,"context_line":""}],"source_content_type":"text/x-ttcn-cfg","patch_set":1,"id":"0620a385_78248d14","line":23,"in_reply_to":"d7fcc9a7_559825fe","updated":"2026-05-13 07:11:24.000000000","message":"Done","commit_id":"cb28a05c91e73629d2888ad8af6c30bde05dc69c"},{"author":{"_account_id":1001678,"name":"Nishanth Menon","email":"nm@ti.com","username":"nmenon"},"change_message_id":"5ed7abc0097e2c588af17abd88517ae45edaae69","unresolved":true,"context_lines":[{"line_number":22,"context_line":""},{"line_number":23,"context_line":"swj_newdap $_CHIPNAME cpu -irlen 4 \\"},{"line_number":24,"context_line":"\t-expected-id $_DAP_TAPID \\"},{"line_number":25,"context_line":"\t-expected-id 0x6ba02477"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"dap create $_CHIPNAME.dap -chain-position $_CHIPNAME.cpu"},{"line_number":28,"context_line":""}],"source_content_type":"text/x-ttcn-cfg","patch_set":2,"id":"ca47cf7a_1770a030","line":25,"updated":"2026-05-13 12:58:21.000000000","message":"a) shouldn\u0027t this be a variable? something like _CPU_SWD_TAPID or _DAP_SWD_ID\n\n```c\nif { [info exists DAP_SWD_ID] } {   \n    set _DAP_SWD_ID $DAP_SWD_ID\n} else {\n    set _DAP_SWD_ID 0x6ba02477\n}                                   \n```\n\nb) I wasn\u0027t able to grab this info from TRM. but some internal documentation seems to indicate this - could you confirm?","commit_id":"9a54d006289b09c815b2a70a3d691656c9ff6904"},{"author":{"_account_id":1001678,"name":"Nishanth Menon","email":"nm@ti.com","username":"nmenon"},"change_message_id":"c9cb1b1bff85bc636538841d2bd6d57061076ecc","unresolved":false,"context_lines":[{"line_number":22,"context_line":""},{"line_number":23,"context_line":"swj_newdap $_CHIPNAME cpu -irlen 4 \\"},{"line_number":24,"context_line":"\t-expected-id $_DAP_TAPID \\"},{"line_number":25,"context_line":"\t-expected-id 0x6ba02477"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"dap create $_CHIPNAME.dap -chain-position $_CHIPNAME.cpu"},{"line_number":28,"context_line":""}],"source_content_type":"text/x-ttcn-cfg","patch_set":2,"id":"17814e2e_54251cf8","line":25,"in_reply_to":"8cb6078f_50eea4bd","updated":"2026-06-10 15:01:30.000000000","message":"Ack. thanks for filing internal ticket to have it documented.","commit_id":"9a54d006289b09c815b2a70a3d691656c9ff6904"},{"author":{"_account_id":1002511,"name":"Sameer Srivastava","email":"sam33r012@gmail.com","username":"s-m33r"},"change_message_id":"6ebf7ccfa44c5845a4b8e09c9545c0f43571e964","unresolved":true,"context_lines":[{"line_number":22,"context_line":""},{"line_number":23,"context_line":"swj_newdap $_CHIPNAME cpu -irlen 4 \\"},{"line_number":24,"context_line":"\t-expected-id $_DAP_TAPID \\"},{"line_number":25,"context_line":"\t-expected-id 0x6ba02477"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"dap create $_CHIPNAME.dap -chain-position $_CHIPNAME.cpu"},{"line_number":28,"context_line":""}],"source_content_type":"text/x-ttcn-cfg","patch_set":2,"id":"8cb6078f_50eea4bd","line":25,"in_reply_to":"ca47cf7a_1770a030","updated":"2026-06-08 12:21:52.000000000","message":"a) Added DAP_SWD_ID / _DAP_SWD_ID following the same pattern as `mspm0.cfg`.\n\nb) I got these IDs from OpenOCD\u0027s initial output:\n```\n\nInfo tap/device found: 0x6ba00477 (mfg: 0x23b (ARM Ltd), part: 0xba00, ver: 0x6)\n\n```\nbut couldn\u0027t find these in the TRM.","commit_id":"9a54d006289b09c815b2a70a3d691656c9ff6904"},{"author":{"_account_id":1001678,"name":"Nishanth Menon","email":"nm@ti.com","username":"nmenon"},"change_message_id":"5ed7abc0097e2c588af17abd88517ae45edaae69","unresolved":true,"context_lines":[{"line_number":37,"context_line":"if { [info exists WORKAREASIZE] } {"},{"line_number":38,"context_line":"\tset _WORKAREASIZE $WORKAREASIZE"},{"line_number":39,"context_line":"} else {"},{"line_number":40,"context_line":"\tset _WORKAREASIZE 0x18000"},{"line_number":41,"context_line":"}"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"$_TARGETNAME configure -work-area-phys $_WORKAREABASE -work-area-size $_WORKAREASIZE -work-area-backup 0"}],"source_content_type":"text/x-ttcn-cfg","patch_set":2,"id":"5aaae8fa_8f0f0a81","line":40,"updated":"2026-05-13 12:58:21.000000000","message":"M13_ALGO_WORKING_SIZE \u003d 0x3100 in the algo. The OpenOCD convention is to use a modest fixed size like 32 KB rather than a large fraction of SRAM. will 0x8000 suffice - including future proofing?","commit_id":"9a54d006289b09c815b2a70a3d691656c9ff6904"},{"author":{"_account_id":1001678,"name":"Nishanth Menon","email":"nm@ti.com","username":"nmenon"},"change_message_id":"c9cb1b1bff85bc636538841d2bd6d57061076ecc","unresolved":false,"context_lines":[{"line_number":37,"context_line":"if { [info exists WORKAREASIZE] } {"},{"line_number":38,"context_line":"\tset _WORKAREASIZE $WORKAREASIZE"},{"line_number":39,"context_line":"} else {"},{"line_number":40,"context_line":"\tset _WORKAREASIZE 0x18000"},{"line_number":41,"context_line":"}"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"$_TARGETNAME configure -work-area-phys $_WORKAREABASE -work-area-size $_WORKAREASIZE -work-area-backup 0"}],"source_content_type":"text/x-ttcn-cfg","patch_set":2,"id":"1c5ddfd7_2ffce4ab","line":40,"in_reply_to":"25d5be5e_383cccd4","updated":"2026-06-10 15:01:30.000000000","message":"Ack","commit_id":"9a54d006289b09c815b2a70a3d691656c9ff6904"},{"author":{"_account_id":1002511,"name":"Sameer Srivastava","email":"sam33r012@gmail.com","username":"s-m33r"},"change_message_id":"6ebf7ccfa44c5845a4b8e09c9545c0f43571e964","unresolved":true,"context_lines":[{"line_number":37,"context_line":"if { [info exists WORKAREASIZE] } {"},{"line_number":38,"context_line":"\tset _WORKAREASIZE $WORKAREASIZE"},{"line_number":39,"context_line":"} else {"},{"line_number":40,"context_line":"\tset _WORKAREASIZE 0x18000"},{"line_number":41,"context_line":"}"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"$_TARGETNAME configure -work-area-phys $_WORKAREABASE -work-area-size $_WORKAREASIZE -work-area-backup 0"}],"source_content_type":"text/x-ttcn-cfg","patch_set":2,"id":"25d5be5e_383cccd4","line":40,"in_reply_to":"5aaae8fa_8f0f0a81","updated":"2026-06-08 12:21:52.000000000","message":"Yes, that should be sufficient for the current loader with room for growth. Updated `_WORKAREASIZE` to 0x8000.","commit_id":"9a54d006289b09c815b2a70a3d691656c9ff6904"},{"author":{"_account_id":1001678,"name":"Nishanth Menon","email":"nm@ti.com","username":"nmenon"},"change_message_id":"d293e30a257664edc00f8962a309ef382cb66806","unresolved":true,"context_lines":[{"line_number":53,"context_line":"flash bank $_FLASHNAME.nonmain am13 0x60100000 0 0 0 $_TARGETNAME"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"if {![using_hla]} {"},{"line_number":56,"context_line":"      cortex_m reset_config sysresetreq"},{"line_number":57,"context_line":"}"}],"source_content_type":"text/x-ttcn-cfg","patch_set":4,"id":"32bf618f_02473607","line":56,"updated":"2026-06-11 13:14:27.000000000","message":"please use tabs instead of space for indentation.","commit_id":"ff40968c664abd17dcec3d336cd3329bf1beafb9"},{"author":{"_account_id":1002511,"name":"Sameer Srivastava","email":"sam33r012@gmail.com","username":"s-m33r"},"change_message_id":"0293736911bbdd8befbafc7766eac5ef7ce849c3","unresolved":false,"context_lines":[{"line_number":53,"context_line":"flash bank $_FLASHNAME.nonmain am13 0x60100000 0 0 0 $_TARGETNAME"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"if {![using_hla]} {"},{"line_number":56,"context_line":"      cortex_m reset_config sysresetreq"},{"line_number":57,"context_line":"}"}],"source_content_type":"text/x-ttcn-cfg","patch_set":4,"id":"acbc29c7_f0e4f6a9","line":56,"in_reply_to":"32bf618f_02473607","updated":"2026-06-15 06:21:44.000000000","message":"Done","commit_id":"ff40968c664abd17dcec3d336cd3329bf1beafb9"},{"author":{"_account_id":1001678,"name":"Nishanth Menon","email":"nm@ti.com","username":"nmenon"},"change_message_id":"92aceb542edb665a9c8a4a17db71a8b45d820c10","unresolved":true,"context_lines":[{"line_number":4,"context_line":"# Texas Instruments AM13E230X - ARM Cortex-M33 @ 200MHz"},{"line_number":5,"context_line":"#"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"source [find bitsbytes.tcl]"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"if { [info exists CHIPNAME] } {"},{"line_number":10,"context_line":"\tset _CHIPNAME $CHIPNAME"}],"source_content_type":"text/x-ttcn-cfg","patch_set":6,"id":"df9d1ee0_1c0ae95e","line":7,"updated":"2026-06-16 16:30:56.000000000","message":"drop this - this is unused","commit_id":"00be3bd32ecaa4c846049314682812eb2bac41d3"},{"author":{"_account_id":1002511,"name":"Sameer Srivastava","email":"sam33r012@gmail.com","username":"s-m33r"},"change_message_id":"0b9a6df7847d880672c757377916d7968eab249f","unresolved":false,"context_lines":[{"line_number":4,"context_line":"# Texas Instruments AM13E230X - ARM Cortex-M33 @ 200MHz"},{"line_number":5,"context_line":"#"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"source [find bitsbytes.tcl]"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"if { [info exists CHIPNAME] } {"},{"line_number":10,"context_line":"\tset _CHIPNAME $CHIPNAME"}],"source_content_type":"text/x-ttcn-cfg","patch_set":6,"id":"cb159316_2bfacc39","line":7,"in_reply_to":"df9d1ee0_1c0ae95e","updated":"2026-06-17 05:41:10.000000000","message":"Done","commit_id":"00be3bd32ecaa4c846049314682812eb2bac41d3"}]}