)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1000021,"name":"Antonio Borneo","email":"borneo.antonio@gmail.com","username":"borneoa"},"change_message_id":"c1163729b725f5c1c690509a83f8ebd4e55e6e34","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"eb6f1d71_75e5c1ac","updated":"2025-09-13 13:25:03.000000000","message":"I see we have https://review.openocd.org/c/openocd/+/8686\n`flash/nor: flash driver for PIC32CZ family`\nDoes it mean that patch is useless? Is there anything there we want to keep","commit_id":"31d5b4b33025d1672b3592f7808a07681d8fc2c1"},{"author":{"_account_id":1000687,"name":"Tomas Vanek","display_name":"Tomas Vanek","email":"vanekt@fbl.cz","username":"vanekt"},"change_message_id":"ffaa4f03a4fe6e8597b3ebcc01e0b84b80687923","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"d1e59e1d_4a4a0bd7","updated":"2025-09-10 11:29:20.000000000","message":"Thnaks, Michale.\n\nYou may also eventually consider submitting another patch with a \"redirect\" config for PIC32CZ CA70 which just sources `target/atsamv.cfg`. Be aware of new rules for vendor directories https://openocd.org/doc/doxygen/html/config_files.html","commit_id":"31d5b4b33025d1672b3592f7808a07681d8fc2c1"},{"author":{"_account_id":1002282,"name":"Michal Lenc","email":"michallenc@seznam.cz","username":"michallenc"},"change_message_id":"c737267a690016b9cfd7beb032487f5e336fcf28","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"2b636790_99754f2a","in_reply_to":"0aead76c_d9e3d35a","updated":"2025-09-13 14:52:18.000000000","message":"Yes, unfortunately, the naming is a bit confusing from Microchip side. They took SAM series from Atmel after acquiring it, but it seems they are now replacing it with their own PIC series. So they introduced 32 bits ARM MCUs PIC32CZ CA70, CA80 and CA90.\n\nPIC32CZ CA70 is binary and pin to pin compatible with the old Atmel\u0027s SAM series, but PIC32CZ CA80 and CA90 are completely different from CA70 and thus Atmel\u0027s SAM regarding peripherals, including the flash driver. So we need the patch Antonio mentioned for CA80 and CA90, but CA70 needs the old SAM code.\n\nHappy to see there is a patch for CA80/CA90 btw, I plan to play with that series sometime in the future.","commit_id":"31d5b4b33025d1672b3592f7808a07681d8fc2c1"},{"author":{"_account_id":1000687,"name":"Tomas Vanek","display_name":"Tomas Vanek","email":"vanekt@fbl.cz","username":"vanekt"},"change_message_id":"3a3eab7767f43bbd0f60c22dc23eacfc0d43852c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"fa96aca9_56c95498","in_reply_to":"2b636790_99754f2a","updated":"2026-01-08 05:32:21.000000000","message":"Ack","commit_id":"31d5b4b33025d1672b3592f7808a07681d8fc2c1"},{"author":{"_account_id":1002282,"name":"Michal Lenc","email":"michallenc@seznam.cz","username":"michallenc"},"change_message_id":"c737267a690016b9cfd7beb032487f5e336fcf28","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"53218dd5_abfa0d36","in_reply_to":"d1e59e1d_4a4a0bd7","updated":"2025-09-13 14:52:18.000000000","message":"\u003e You may also eventually consider submitting another patch with a \"redirect\" config for PIC32CZ CA70 which just sources `target/atsamv.cfg`.\n\nInitially thought about adding pic32 cfg, but didn\u0027t want to duplicate the original one. But it\u0027s true I can just reference it. \n\n\u003e Be aware of new rules for vendor directories https://openocd.org/doc/doxygen/html/config_files.html\n\nSo if I understand it correctly, the goal is to move all cfg files under vendor subdirectories, thus, lets say `target/microchip/pic32czca70.cfg`?","commit_id":"31d5b4b33025d1672b3592f7808a07681d8fc2c1"},{"author":{"_account_id":1000687,"name":"Tomas Vanek","display_name":"Tomas Vanek","email":"vanekt@fbl.cz","username":"vanekt"},"change_message_id":"c08a80adbb5ef739c05a5360dcc0f6d04fb7547a","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"0aead76c_d9e3d35a","in_reply_to":"eb6f1d71_75e5c1ac","updated":"2025-09-13 13:57:22.000000000","message":"Hmm, honestly I forgot completely. Thanks for finding it, Antonio.\n\nI\u0027m not familiar with Microchip MCU markings. #8686 looks like it handles PIC32CZ CA80 and CA90 series and they are substantially different from CA70.\nAm I right, Michal?","commit_id":"31d5b4b33025d1672b3592f7808a07681d8fc2c1"}],"doc/openocd.texi":[{"author":{"_account_id":1000021,"name":"Antonio Borneo","email":"borneo.antonio@gmail.com","username":"borneoa"},"change_message_id":"289927f12d076d401be5c3a141456a8c2d35c6e4","unresolved":true,"context_lines":[{"line_number":6959,"context_line":"@cindex atsamv"},{"line_number":6960,"context_line":"All members of the ATSAMV7x, ATSAMS70, and ATSAME70 families from"},{"line_number":6961,"context_line":"Atmel include internal flash and use ARM\u0027s Cortex-M7 core."},{"line_number":6962,"context_line":"This driver uses the same command names/syntax as @xref{at91sam3}."},{"line_number":6963,"context_line":""},{"line_number":6964,"context_line":"@example"},{"line_number":6965,"context_line":"flash bank $_FLASHNAME atsamv 0x00400000 0 0 0 $_TARGETNAME"}],"source_content_type":"text/x-texinfo","patch_set":1,"id":"7949640e_84b7a309","line":6962,"updated":"2025-09-01 08:41:54.000000000","message":"I think it could be more appropriate to move here the reference to PIC32CZ CA70 series, together with the list of the other devices.\nMaybe also mentianning that Atmel has been acquired by Microchip","commit_id":"29a3cd1d940f790d83eac481d2db536fe9e22e6d"},{"author":{"_account_id":1002282,"name":"Michal Lenc","email":"michallenc@seznam.cz","username":"michallenc"},"change_message_id":"5f2dc4b40ea8bca4dba3d114fdf821dfcc23620a","unresolved":false,"context_lines":[{"line_number":6959,"context_line":"@cindex atsamv"},{"line_number":6960,"context_line":"All members of the ATSAMV7x, ATSAMS70, and ATSAME70 families from"},{"line_number":6961,"context_line":"Atmel include internal flash and use ARM\u0027s Cortex-M7 core."},{"line_number":6962,"context_line":"This driver uses the same command names/syntax as @xref{at91sam3}."},{"line_number":6963,"context_line":""},{"line_number":6964,"context_line":"@example"},{"line_number":6965,"context_line":"flash bank $_FLASHNAME atsamv 0x00400000 0 0 0 $_TARGETNAME"}],"source_content_type":"text/x-texinfo","patch_set":1,"id":"91a64a3e_e9c84429","line":6962,"in_reply_to":"7949640e_84b7a309","updated":"2025-09-04 11:40:24.000000000","message":"Good point, moved.","commit_id":"29a3cd1d940f790d83eac481d2db536fe9e22e6d"}]}
