)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1000021,"name":"Antonio Borneo","email":"borneo.antonio@gmail.com","username":"borneoa"},"change_message_id":"71fc9bb1883583dcce88b56d77ef36e496117491","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"15f3c1c0_9258c131","updated":"2023-08-15 15:35:50.000000000","message":"Missing command in doc/openocd.texi\nPlus, grep refresh reports:\n\n@deffn {Command} {gowin reload} pld_name\nLoad the bitstream from external memory for FPGA @var{pld_name}. A.k.a. refresh.\n\n@deffn {Command} {virtex2 program} pld_name\nLoad the bitstream from external memory for FPGA @var{pld_name}. A.k.a. refresh.\n\nthis patch adds \"lattice refresh\"!\nAre thys related?\nAny way to stick to only one command? User has to learn a new command name for every FPGA type?","commit_id":"e06dc59f364df31a6974269e5bc9d4768fade818"},{"author":{"_account_id":1001810,"name":"Daniel Anselmi","email":"danselmi@gmx.ch","username":"danselmi"},"change_message_id":"c9ff514f6765bcac1112e50b4fd9b6456ec642a9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"7f30d62a_29c182bf","updated":"2023-08-15 18:59:06.000000000","message":"Thanks Antonio for your feedback and time.","commit_id":"e06dc59f364df31a6974269e5bc9d4768fade818"},{"author":{"_account_id":1001810,"name":"Daniel Anselmi","email":"danselmi@gmx.ch","username":"danselmi"},"change_message_id":"c9ff514f6765bcac1112e50b4fd9b6456ec642a9","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"ce1e9c49_630076c6","in_reply_to":"15f3c1c0_9258c131","updated":"2023-08-15 18:59:06.000000000","message":"They are the same functionality, yes. \nThe commands are called the same as the corresponding vendor calls them.\n\nThey all came after the 0.12.0 release. So I think we can simply rename them without providing a backward compatible version. Right?\n\nI think we can choose any of the three and mention the vendor naming in the documentation. Which shall I take?","commit_id":"e06dc59f364df31a6974269e5bc9d4768fade818"},{"author":{"_account_id":1001810,"name":"Daniel Anselmi","email":"danselmi@gmx.ch","username":"danselmi"},"change_message_id":"e78baaf0fc9efff28caa2f4c59300be2ec738fda","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"ac5ac861_3a000b61","in_reply_to":"ce1e9c49_630076c6","updated":"2023-08-22 22:38:02.000000000","message":"Its {virtex2/gowin/lattice} refresh now.\nNo backward compatible commands provided since old commands were not part of a release yet.","commit_id":"e06dc59f364df31a6974269e5bc9d4768fade818"},{"author":{"_account_id":1000021,"name":"Antonio Borneo","email":"borneo.antonio@gmail.com","username":"borneoa"},"change_message_id":"2f69ac349b2d67cb9b145c57816ab59ed2f86fac","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"a70dede6_40700226","updated":"2023-09-09 16:39:43.000000000","message":"thanks!","commit_id":"4a7d8cfd2f4c46de0e92c87d0e2f57104be7b725"}],"doc/openocd.texi":[{"author":{"_account_id":1000021,"name":"Antonio Borneo","email":"borneo.antonio@gmail.com","username":"borneoa"},"change_message_id":"eaec6a86679c35bcf6a4d0c22caa4fe55b725933","unresolved":true,"context_lines":[{"line_number":8685,"context_line":"Description of the arguments can be found at command @command{virtex2 set_instr_codes}."},{"line_number":8686,"context_line":"@end deffn"},{"line_number":8687,"context_line":""},{"line_number":8688,"context_line":"@deffn {Command} {virtex2 refresh} pld_name"},{"line_number":8689,"context_line":"Load the bitstream from external memory for FPGA @var{pld_name}. A.k.a. program."},{"line_number":8690,"context_line":"@end deffn"},{"line_number":8691,"context_line":"@end deffn"}],"source_content_type":"text/x-texinfo","patch_set":5,"id":"78d26b69_138438ff","line":8688,"updated":"2023-08-27 14:34:33.000000000","message":"Every change in command syntax requires a deprecation of old syntax and a fallback in TCL to the new syntax.\nYou should add a procedure\nproc \"virtex2 program\" args {\necho \"DEPRECATED! use \u0027virtex2 refresh ...\u0027, not \u0027virtex2 program ...\u0027\"\neval virtex2 refresh $args\n}\n\nI see already the deprecation of \"pld device\" in src/jtag/startup.tcl, so this deprecation can go there too.","commit_id":"f9b142f8003ad0108731d36fbf1b20bcaa914a02"},{"author":{"_account_id":1000021,"name":"Antonio Borneo","email":"borneo.antonio@gmail.com","username":"borneoa"},"change_message_id":"2f69ac349b2d67cb9b145c57816ab59ed2f86fac","unresolved":false,"context_lines":[{"line_number":8685,"context_line":"Description of the arguments can be found at command @command{virtex2 set_instr_codes}."},{"line_number":8686,"context_line":"@end deffn"},{"line_number":8687,"context_line":""},{"line_number":8688,"context_line":"@deffn {Command} {virtex2 refresh} pld_name"},{"line_number":8689,"context_line":"Load the bitstream from external memory for FPGA @var{pld_name}. A.k.a. program."},{"line_number":8690,"context_line":"@end deffn"},{"line_number":8691,"context_line":"@end deffn"}],"source_content_type":"text/x-texinfo","patch_set":5,"id":"e85834a3_f2563485","line":8688,"in_reply_to":"0975e1b1_3bdbb7c3","updated":"2023-09-09 16:39:43.000000000","message":"You are right, no need for a fallback","commit_id":"f9b142f8003ad0108731d36fbf1b20bcaa914a02"},{"author":{"_account_id":1001810,"name":"Daniel Anselmi","email":"danselmi@gmx.ch","username":"danselmi"},"change_message_id":"ccb631959d80fbd70b5ff3ab10824dbc1341cb5e","unresolved":true,"context_lines":[{"line_number":8685,"context_line":"Description of the arguments can be found at command @command{virtex2 set_instr_codes}."},{"line_number":8686,"context_line":"@end deffn"},{"line_number":8687,"context_line":""},{"line_number":8688,"context_line":"@deffn {Command} {virtex2 refresh} pld_name"},{"line_number":8689,"context_line":"Load the bitstream from external memory for FPGA @var{pld_name}. A.k.a. program."},{"line_number":8690,"context_line":"@end deffn"},{"line_number":8691,"context_line":"@end deffn"}],"source_content_type":"text/x-texinfo","patch_set":5,"id":"0975e1b1_3bdbb7c3","line":8688,"in_reply_to":"78d26b69_138438ff","updated":"2023-08-27 18:12:17.000000000","message":"it was not in any release yet, should we anyhow add the fallback?","commit_id":"f9b142f8003ad0108731d36fbf1b20bcaa914a02"},{"author":{"_account_id":1000021,"name":"Antonio Borneo","email":"borneo.antonio@gmail.com","username":"borneoa"},"change_message_id":"eaec6a86679c35bcf6a4d0c22caa4fe55b725933","unresolved":true,"context_lines":[{"line_number":8775,"context_line":"for FPGA @var{pld_name}."},{"line_number":8776,"context_line":"@end deffn"},{"line_number":8777,"context_line":""},{"line_number":8778,"context_line":"@deffn {Command} {gowin refresh} pld_name"},{"line_number":8779,"context_line":"Load the bitstream from external memory for"},{"line_number":8780,"context_line":"FPGA @var{pld_name}. A.k.a. reload."},{"line_number":8781,"context_line":"@end deffn"}],"source_content_type":"text/x-texinfo","patch_set":5,"id":"eb645da5_3f0391e4","line":8778,"updated":"2023-08-27 14:34:33.000000000","message":"same here:\nproc \"gowin reload\" args {\necho \"DEPRECATED! use \u0027gowin refresh ...\u0027, not \u0027gowin reload ...\u0027\"\neval gowin refresh $args\n}","commit_id":"f9b142f8003ad0108731d36fbf1b20bcaa914a02"},{"author":{"_account_id":1000021,"name":"Antonio Borneo","email":"borneo.antonio@gmail.com","username":"borneoa"},"change_message_id":"2f69ac349b2d67cb9b145c57816ab59ed2f86fac","unresolved":false,"context_lines":[{"line_number":8775,"context_line":"for FPGA @var{pld_name}."},{"line_number":8776,"context_line":"@end deffn"},{"line_number":8777,"context_line":""},{"line_number":8778,"context_line":"@deffn {Command} {gowin refresh} pld_name"},{"line_number":8779,"context_line":"Load the bitstream from external memory for"},{"line_number":8780,"context_line":"FPGA @var{pld_name}. A.k.a. reload."},{"line_number":8781,"context_line":"@end deffn"}],"source_content_type":"text/x-texinfo","patch_set":5,"id":"5f678379_eb1882bd","line":8778,"in_reply_to":"54ddd813_b0abed4e","updated":"2023-09-09 16:39:43.000000000","message":"Ack","commit_id":"f9b142f8003ad0108731d36fbf1b20bcaa914a02"},{"author":{"_account_id":1001810,"name":"Daniel Anselmi","email":"danselmi@gmx.ch","username":"danselmi"},"change_message_id":"ccb631959d80fbd70b5ff3ab10824dbc1341cb5e","unresolved":true,"context_lines":[{"line_number":8775,"context_line":"for FPGA @var{pld_name}."},{"line_number":8776,"context_line":"@end deffn"},{"line_number":8777,"context_line":""},{"line_number":8778,"context_line":"@deffn {Command} {gowin refresh} pld_name"},{"line_number":8779,"context_line":"Load the bitstream from external memory for"},{"line_number":8780,"context_line":"FPGA @var{pld_name}. A.k.a. reload."},{"line_number":8781,"context_line":"@end deffn"}],"source_content_type":"text/x-texinfo","patch_set":5,"id":"54ddd813_b0abed4e","line":8778,"in_reply_to":"eb645da5_3f0391e4","updated":"2023-08-27 18:12:17.000000000","message":"same question here.","commit_id":"f9b142f8003ad0108731d36fbf1b20bcaa914a02"}],"src/pld/lattice_cmd.h":[{"author":{"_account_id":1000021,"name":"Antonio Borneo","email":"borneo.antonio@gmail.com","username":"borneoa"},"change_message_id":"eaec6a86679c35bcf6a4d0c22caa4fe55b725933","unresolved":true,"context_lines":[{"line_number":14,"context_line":"#define LSC_READ_STATUS      0x3C"},{"line_number":15,"context_line":"#define LSC_INIT_ADDRESS     0x46"},{"line_number":16,"context_line":"#define LSC_BITSTREAM_BURST  0x7A"},{"line_number":17,"context_line":"#define LSC_REFRESH          0x79"},{"line_number":18,"context_line":"#define READ_USERCODE        0xC0"},{"line_number":19,"context_line":"#define ISC_ENABLE           0xC6"},{"line_number":20,"context_line":""}],"source_content_type":"text/x-csrc","patch_set":5,"id":"7a34277b_b8d1a513","line":17,"updated":"2023-08-27 14:34:33.000000000","message":"so far the commands where in order by value, maybe just by chance.\nAny reason to put the new one here?","commit_id":"f9b142f8003ad0108731d36fbf1b20bcaa914a02"},{"author":{"_account_id":1001810,"name":"Daniel Anselmi","email":"danselmi@gmx.ch","username":"danselmi"},"change_message_id":"61a16be290b6363b59d9264542b6f37b8319348c","unresolved":false,"context_lines":[{"line_number":14,"context_line":"#define LSC_READ_STATUS      0x3C"},{"line_number":15,"context_line":"#define LSC_INIT_ADDRESS     0x46"},{"line_number":16,"context_line":"#define LSC_BITSTREAM_BURST  0x7A"},{"line_number":17,"context_line":"#define LSC_REFRESH          0x79"},{"line_number":18,"context_line":"#define READ_USERCODE        0xC0"},{"line_number":19,"context_line":"#define ISC_ENABLE           0xC6"},{"line_number":20,"context_line":""}],"source_content_type":"text/x-csrc","patch_set":5,"id":"f0a5f2dc_70f2595d","line":17,"in_reply_to":"7a34277b_b8d1a513","updated":"2023-08-27 19:06:42.000000000","message":"Done","commit_id":"f9b142f8003ad0108731d36fbf1b20bcaa914a02"}]}
