)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1000021,"name":"Antonio Borneo","email":"borneo.antonio@gmail.com","username":"borneoa"},"change_message_id":"22617aa963f29a2844535c52c92a57077c645ec6","unresolved":true,"context_lines":[{"line_number":9,"context_line":"simplify ipdbg commands:"},{"line_number":10,"context_line":"$hub_name ipdbg start -\u003e $hub_name start"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"ipdbg/fix: \"Deprecated commands with spaces require quotes\"."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I5a56df2dbf2a5710442242727fa9384e5541ed53"},{"line_number":15,"context_line":"Signed-off-by: Daniel Anselmi \u003cdanselmi@gmx.ch\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"91c3e086_f07e0a96","line":12,"updated":"2025-08-20 13:15:23.000000000","message":"This line looks like a copy/paste from the title of a commit message.\nPlease reformat it, e.g. as:\n```\nWhile there, handle the multi-word deprecated commands\nby adding quotes.\n```","commit_id":"c948488b5f23d71afe94839f581d91d3fd10e8d9"},{"author":{"_account_id":1001810,"name":"Daniel Anselmi","email":"danselmi@gmx.ch","username":"danselmi"},"change_message_id":"58dc4724aed5f7c8dcd6349f52b74909ed077c51","unresolved":false,"context_lines":[{"line_number":9,"context_line":"simplify ipdbg commands:"},{"line_number":10,"context_line":"$hub_name ipdbg start -\u003e $hub_name start"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"ipdbg/fix: \"Deprecated commands with spaces require quotes\"."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I5a56df2dbf2a5710442242727fa9384e5541ed53"},{"line_number":15,"context_line":"Signed-off-by: Daniel Anselmi \u003cdanselmi@gmx.ch\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"6eadcd46_41dfefc3","line":12,"in_reply_to":"91c3e086_f07e0a96","updated":"2025-08-22 21:12:56.000000000","message":"Done","commit_id":"c948488b5f23d71afe94839f581d91d3fd10e8d9"},{"author":{"_account_id":1000021,"name":"Antonio Borneo","email":"borneo.antonio@gmail.com","username":"borneoa"},"change_message_id":"de32b5fc0f1b591cde9e276052ec19acd4cd116b","unresolved":true,"context_lines":[{"line_number":10,"context_line":"$hub_name ipdbg start -\u003e $hub_name start"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"ipdbg/fix: While there, handle the multi-word deprecated commands"},{"line_number":13,"context_line":"by adding quotes. (fix ticket #434)"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I5a56df2dbf2a5710442242727fa9384e5541ed53"},{"line_number":16,"context_line":"Signed-off-by: Daniel Anselmi \u003cdanselmi@gmx.ch\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"884e998e_755c7d1f","line":13,"updated":"2025-10-18 12:37:45.000000000","message":"move the ticket reference below. See the other comment","commit_id":"841cb4475244bea1c92430f52a8160d3b1816166"},{"author":{"_account_id":1001810,"name":"Daniel Anselmi","email":"danselmi@gmx.ch","username":"danselmi"},"change_message_id":"48e4f33209c38cf7613e9b225b2c77a193b00839","unresolved":false,"context_lines":[{"line_number":10,"context_line":"$hub_name ipdbg start -\u003e $hub_name start"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"ipdbg/fix: While there, handle the multi-word deprecated commands"},{"line_number":13,"context_line":"by adding quotes. (fix ticket #434)"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I5a56df2dbf2a5710442242727fa9384e5541ed53"},{"line_number":16,"context_line":"Signed-off-by: Daniel Anselmi \u003cdanselmi@gmx.ch\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"5e54fdb3_44281d95","line":13,"in_reply_to":"884e998e_755c7d1f","updated":"2025-10-21 20:44:09.000000000","message":"Done","commit_id":"841cb4475244bea1c92430f52a8160d3b1816166"},{"author":{"_account_id":1000021,"name":"Antonio Borneo","email":"borneo.antonio@gmail.com","username":"borneoa"},"change_message_id":"de32b5fc0f1b591cde9e276052ec19acd4cd116b","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":6,"id":"b5b5927b_a81349d0","line":17,"updated":"2025-10-18 12:37:45.000000000","message":"Here add the lines:\nFixes: https://sourceforge.net/p/openocd/tickets/434/\nReported-by: Evgeniy Naydanov \u003cevgeniy.naydanov@syntacore.com\u003e","commit_id":"841cb4475244bea1c92430f52a8160d3b1816166"},{"author":{"_account_id":1001810,"name":"Daniel Anselmi","email":"danselmi@gmx.ch","username":"danselmi"},"change_message_id":"48e4f33209c38cf7613e9b225b2c77a193b00839","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"db9e6a77_c0fd45c6","line":17,"in_reply_to":"b5b5927b_a81349d0","updated":"2025-10-21 20:44:09.000000000","message":"Done","commit_id":"841cb4475244bea1c92430f52a8160d3b1816166"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":1001810,"name":"Daniel Anselmi","email":"danselmi@gmx.ch","username":"danselmi"},"change_message_id":"496d7a35e11c3d90da24d8d1e6f8b84ff2fd838e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"93b870b5_e734c294","updated":"2025-08-18 21:08:25.000000000","message":"Thanks Marc for reviewing.\n\nI re-added the top level command `ipdbg`.\n\nRemember: after merge we can close ticket #434.","commit_id":"b96219c15c50cac63a6a9f797c2d7052f1a776cd"},{"author":{"_account_id":1001810,"name":"Daniel Anselmi","email":"danselmi@gmx.ch","username":"danselmi"},"change_message_id":"58dc4724aed5f7c8dcd6349f52b74909ed077c51","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"fd42b0f3_a92c10d2","updated":"2025-08-22 21:12:56.000000000","message":"Thank you guys for spending the time to review!","commit_id":"c948488b5f23d71afe94839f581d91d3fd10e8d9"},{"author":{"_account_id":1000853,"name":"Marc Schink","display_name":"Marc Schink","email":"dev@zapb.de","username":"zapb"},"change_message_id":"32446faa0a43b8495e6cb8066bf66c39afb095b7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"0ad02082_e930c1ee","updated":"2025-08-21 19:40:04.000000000","message":"Thanks Daniel!\n\nIs there a simple example project to test ipdbg? I\u0027m aware of your GitHub repo (https://github.com/IPDBG/) but I cannot find an easy out-of-the-box example.","commit_id":"c948488b5f23d71afe94839f581d91d3fd10e8d9"},{"author":{"_account_id":1000853,"name":"Marc Schink","display_name":"Marc Schink","email":"dev@zapb.de","username":"zapb"},"change_message_id":"e86e08ae027d62651b79ab11ef6924aa74e78117","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"3baed68c_7d5d0bfa","in_reply_to":"00268e19_18e598d9","updated":"2025-08-28 11:31:46.000000000","message":"Thanks, `ghdl` is linked to `ghdl-mcode` by default on my system. Building works, but now I have the problem that `sigrok-cli` does not receive any data:\n\nThe client connection is dropped without receiving any samples while `sigrok-cli` is waiting for data.\n\n```\nInfo : Initializing remote_bitbang driver\nInfo : Connecting to 127.0.0.1:3421\nInfo : remote_bitbang driver initialized\nInfo : Note: The adapter \"remote_bitbang\" doesn\u0027t support configurable speed\nInfo : JTAG tap: ipdbg_adapter.tap tap/device found: 0xf0f0f0f1 (mfg: 0x078 (TRW), part: 0x0f0f, ver: 0xf)\nWarn : gdb services need one or more targets defined\nInfo : starting ipdbg service on port 4242 for tool 0\nInfo : Listening on port 4242 for ipdbg connections\nInfo : starting ipdbg service on port 4243 for tool 1\nInfo : Listening on port 4243 for ipdbg connections\nInfo : starting ipdbg service on port 4244 for tool 2\nInfo : Listening on port 4244 for ipdbg connections\nInfo : starting ipdbg service on port 4245 for tool 3\nInfo : Listening on port 4245 for ipdbg connections\nInfo : Listening on port 9000 for tcl connections\nInfo : Listening on port 9001 for telnet connections\nInfo : accepting \u0027ipdbg\u0027 connection on tcp/4242\nInfo : Flow control enabled on IPDBG JTAG Hub: 0x40\nInfo : IPDBG start_polling\nInfo : New IPDBG Connection\nInfo : rejected \u0027ipdbg\u0027 connection, no more connections allowed\nInfo : Closed IPDBG Connection\nInfo : IPDBG stop_polling\nInfo : dropped \u0027ipdbg\u0027 connection\n```\n\n`sigrok-cli --driver\u003dipdbg-la:conn\u003dtcp-raw/127.0.0.1/4242 --channels \"CH0,CH1\" --samples 1 --output-format csv`\n\nRequesting device information from seems to work fine:\n\n```\nsigrok-cli --driver\u003dipdbg-la:conn\u003dtcp-raw/127.0.0.1/4242 --show\nDriver functions:\n    Logic analyzer\nScan options:\n    conn\nipdbg-la - ipdbg.org IPDBG LA v1.0 with 18 channels: CH0 CH1 CH2 CH3 CH4 CH5 CH6 CH7 CH8 CH9 CH10 CH11 CH12 CH13 CH14 CH15 CH16 CH17\n```\n\nAny idea?","commit_id":"c948488b5f23d71afe94839f581d91d3fd10e8d9"},{"author":{"_account_id":1001810,"name":"Daniel Anselmi","email":"danselmi@gmx.ch","username":"danselmi"},"change_message_id":"58dc4724aed5f7c8dcd6349f52b74909ed077c51","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"5b4afe34_8ba357a9","in_reply_to":"0ad02082_e930c1ee","updated":"2025-08-22 21:12:56.000000000","message":"Sure, simplest is to use IPDBG/sw/CoSim.\nHowever it has some prerequisites:\nGHDL (VHDL simulator), OpenOCD with remote_bitbang support and sigrok/PulseView (Analyzer GUI). Sigrok supports IPDBG\u0027s analyzer out of the box.\n\nOr do you have a specific FPGA board in mind?","commit_id":"c948488b5f23d71afe94839f581d91d3fd10e8d9"},{"author":{"_account_id":1001810,"name":"Daniel Anselmi","email":"danselmi@gmx.ch","username":"danselmi"},"change_message_id":"3bed489f2eac7429698c7966464618b640bbc7d6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"00268e19_18e598d9","in_reply_to":"10faf773_e0762f99","updated":"2025-08-28 07:59:04.000000000","message":"You need ghdl with either the gcc or the llvm backend. You are using the mcode backend which doesn\u0027t generate binaries (which we need to link against).","commit_id":"c948488b5f23d71afe94839f581d91d3fd10e8d9"},{"author":{"_account_id":1001810,"name":"Daniel Anselmi","email":"danselmi@gmx.ch","username":"danselmi"},"change_message_id":"788db654e2e772af265b4bf120866169f2bd9595","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"ac4c0b80_ee5f3d67","in_reply_to":"3baed68c_7d5d0bfa","updated":"2025-08-28 15:02:15.000000000","message":"Doesn\u0027t look too bad!\nDo you have PulseView installed, is it working with PulseView?\n\nI will check later today with sigrok-cli - and get back to you.","commit_id":"c948488b5f23d71afe94839f581d91d3fd10e8d9"},{"author":{"_account_id":1000853,"name":"Marc Schink","display_name":"Marc Schink","email":"dev@zapb.de","username":"zapb"},"change_message_id":"7227ff18796949d83c21f3feb762d6ded0c2474c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"10faf773_e0762f99","in_reply_to":"5b4afe34_8ba357a9","updated":"2025-08-28 07:47:18.000000000","message":"I tried the CoSom example but get the following error:\n\n`/usr/bin/ghdl-mcode:error: unknown command \u0027--bind\u0027, try \u0027help\u0027`\n\nI have ghdl 5.0.1, released March 2025. What version do I need? Everything else that I forget to install / configure?","commit_id":"c948488b5f23d71afe94839f581d91d3fd10e8d9"},{"author":{"_account_id":1001810,"name":"Daniel Anselmi","email":"danselmi@gmx.ch","username":"danselmi"},"change_message_id":"2a389b0199dc8a37c79e5ccda72899320139bb23","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"7f7351a3_f9044a9e","in_reply_to":"9c5c58a7_48c85b6c","updated":"2025-08-29 10:42:36.000000000","message":"Great! You have some progress.\n\nI was able to debug yesterday, it\u0027s indeed a problem with the ipdbg driver in sigrok. I was able to pin it down a bit. sigrok-cli opens the socket to openocd-ipdbg-server before it closes it from the scan. This is different in pulseview.\nThanks for reporting!\n\nBack to OpenOCD, the `ipdbg create-hub` command was introduced after 0.12.0. So the command you mentioned was never in a release this way. Do we really need a backward compatibility to unreleased version?","commit_id":"c948488b5f23d71afe94839f581d91d3fd10e8d9"},{"author":{"_account_id":1000853,"name":"Marc Schink","display_name":"Marc Schink","email":"dev@zapb.de","username":"zapb"},"change_message_id":"3e0b6e0ba32e39005bdea7179885df10ffa9056f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"9c5c58a7_48c85b6c","in_reply_to":"ac4c0b80_ee5f3d67","updated":"2025-08-29 08:15:54.000000000","message":"Well, I did not expect PulseView to work but it works... :D I think there is something bogus with the ipdbg implementation in sigrok but that\u0027s all definitely out of scope / context here - just as feedback for your project.\n\nThanks for the testing support Daniel!\n\nWhat does not work is the old command syntax to ensure backwards compatibility. This command should still work but raise a deprecation warning `cosim.ipdbghub ipdbg start -tool 0 -port 4242`.","commit_id":"c948488b5f23d71afe94839f581d91d3fd10e8d9"},{"author":{"_account_id":1000853,"name":"Marc Schink","display_name":"Marc Schink","email":"dev@zapb.de","username":"zapb"},"change_message_id":"18198265de733b2ac565b8ddfd1063c12b5af7da","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"73cd0965_c1d74227","updated":"2025-08-30 08:19:12.000000000","message":"Ah, good point. I agree but on the other hand we have quite some users that use Git master version. Antonio, what do you think?","commit_id":"841cb4475244bea1c92430f52a8160d3b1816166"},{"author":{"_account_id":1001810,"name":"Daniel Anselmi","email":"danselmi@gmx.ch","username":"danselmi"},"change_message_id":"48e4f33209c38cf7613e9b225b2c77a193b00839","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"ede09d82_14fc7f1f","updated":"2025-10-21 20:44:09.000000000","message":"thanks!","commit_id":"841cb4475244bea1c92430f52a8160d3b1816166"},{"author":{"_account_id":1000021,"name":"Antonio Borneo","email":"borneo.antonio@gmail.com","username":"borneoa"},"change_message_id":"de32b5fc0f1b591cde9e276052ec19acd4cd116b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"ecfd5ea8_cca5ebf5","in_reply_to":"53a83996_ff063f9c","updated":"2025-10-18 12:37:45.000000000","message":"The master branch has to be considered as work in progress.\nWe have already some case where a syntax change during the development is not reported as deprecated.\nFor me there is no need to add the deprecation.","commit_id":"841cb4475244bea1c92430f52a8160d3b1816166"},{"author":{"_account_id":1001810,"name":"Daniel Anselmi","email":"danselmi@gmx.ch","username":"danselmi"},"change_message_id":"33e79e78525f2a9b0d811ccec7ea0ef6cdad08fb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"53a83996_ff063f9c","in_reply_to":"73cd0965_c1d74227","updated":"2025-10-17 09:13:52.000000000","message":"\u003e Ah, good point. I agree but on the other hand we have quite some users that use Git master version. Antonio, what do you think?\n\nAntonio?","commit_id":"841cb4475244bea1c92430f52a8160d3b1816166"}],"src/server/ipdbg.c":[{"author":{"_account_id":1000853,"name":"Marc Schink","display_name":"Marc Schink","email":"dev@zapb.de","username":"zapb"},"change_message_id":"be287ffc9d74f0f1af78627da4e824f06f62f0a9","unresolved":true,"context_lines":[{"line_number":926,"context_line":"}"},{"line_number":927,"context_line":""},{"line_number":928,"context_line":"static const struct command_registration ipdbg_hub_subcommand_handlers[] \u003d {"},{"line_number":929,"context_line":"\t{"},{"line_number":930,"context_line":"\t\t.name \u003d \"start\","},{"line_number":931,"context_line":"\t\t.mode \u003d COMMAND_EXEC,"},{"line_number":932,"context_line":"\t\t.handler \u003d handle_ipdbg_start_command,"}],"source_content_type":"text/x-csrc","patch_set":2,"id":"f666c669_74a88454","line":929,"updated":"2025-08-13 10:21:49.000000000","message":"I agree with removing the `ipdbg` subcomment here.","commit_id":"34f407aafcf3cd7acc7fb3cd1ab3f4cb345c8007"},{"author":{"_account_id":1001810,"name":"Daniel Anselmi","email":"danselmi@gmx.ch","username":"danselmi"},"change_message_id":"496d7a35e11c3d90da24d8d1e6f8b84ff2fd838e","unresolved":false,"context_lines":[{"line_number":926,"context_line":"}"},{"line_number":927,"context_line":""},{"line_number":928,"context_line":"static const struct command_registration ipdbg_hub_subcommand_handlers[] \u003d {"},{"line_number":929,"context_line":"\t{"},{"line_number":930,"context_line":"\t\t.name \u003d \"start\","},{"line_number":931,"context_line":"\t\t.mode \u003d COMMAND_EXEC,"},{"line_number":932,"context_line":"\t\t.handler \u003d handle_ipdbg_start_command,"}],"source_content_type":"text/x-csrc","patch_set":2,"id":"69e4fd7b_215571fb","line":929,"in_reply_to":"f666c669_74a88454","updated":"2025-08-18 21:08:25.000000000","message":"Ack","commit_id":"34f407aafcf3cd7acc7fb3cd1ab3f4cb345c8007"},{"author":{"_account_id":1000853,"name":"Marc Schink","display_name":"Marc Schink","email":"dev@zapb.de","username":"zapb"},"change_message_id":"be287ffc9d74f0f1af78627da4e824f06f62f0a9","unresolved":true,"context_lines":[{"line_number":1118,"context_line":"}"},{"line_number":1119,"context_line":""},{"line_number":1120,"context_line":"static const struct command_registration ipdbg_command_handlers[] \u003d {"},{"line_number":1121,"context_line":"\t{"},{"line_number":1122,"context_line":"\t\t.name \u003d \"create-ipdbg-hub\","},{"line_number":1123,"context_line":"\t\t.mode \u003d COMMAND_ANY,"},{"line_number":1124,"context_line":"\t\t.handler \u003d handle_ipdbg_create_hub_command,"}],"source_content_type":"text/x-csrc","patch_set":2,"id":"0780bf53_4ead969b","line":1121,"updated":"2025-08-13 10:21:49.000000000","message":"I don\u0027t agree with this. What is the reason to remove the top level command `ipdbg`? At the moment it is similar to `target create`.","commit_id":"34f407aafcf3cd7acc7fb3cd1ab3f4cb345c8007"},{"author":{"_account_id":1001810,"name":"Daniel Anselmi","email":"danselmi@gmx.ch","username":"danselmi"},"change_message_id":"496d7a35e11c3d90da24d8d1e6f8b84ff2fd838e","unresolved":false,"context_lines":[{"line_number":1118,"context_line":"}"},{"line_number":1119,"context_line":""},{"line_number":1120,"context_line":"static const struct command_registration ipdbg_command_handlers[] \u003d {"},{"line_number":1121,"context_line":"\t{"},{"line_number":1122,"context_line":"\t\t.name \u003d \"create-ipdbg-hub\","},{"line_number":1123,"context_line":"\t\t.mode \u003d COMMAND_ANY,"},{"line_number":1124,"context_line":"\t\t.handler \u003d handle_ipdbg_create_hub_command,"}],"source_content_type":"text/x-csrc","patch_set":2,"id":"0be2ffb5_6c9611bc","line":1121,"in_reply_to":"0780bf53_4ead969b","updated":"2025-08-18 21:08:25.000000000","message":"Ack","commit_id":"34f407aafcf3cd7acc7fb3cd1ab3f4cb345c8007"}]}