)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1000687,"name":"Tomas Vanek","display_name":"Tomas Vanek","email":"vanekt@fbl.cz","username":"vanekt"},"change_message_id":"60d32fcd19a797ec56dec0faf348c44daad3e4b2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"20f0cd7f_81f22aee","updated":"2023-12-06 13:57:54.000000000","message":"Makes sense to me.\nTested with a couple of Xplained boards (03eb:2111 and 03eb:2169)\n\nUnfortunately I\u0027m not able to assess whether the VID:PID list is complete\nor some other adapters with 512 byte packets are around.\n\nI would prefer parsing the HID report descriptor as suggested in the comment.\nIt is feasible but requires to re-implement a highly system dependent part of HIDAPI library in OpenOCD to read the descriptor...","commit_id":"ecf567039a997d9a8245e62eea94a5195970b6c0"},{"author":{"_account_id":1000687,"name":"Tomas Vanek","display_name":"Tomas Vanek","email":"vanekt@fbl.cz","username":"vanekt"},"change_message_id":"a459126b4401a06633b9970e110c4179dd59f6b6","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"29d4a6e3_4a53bac9","in_reply_to":"09aa7c17_3d94d391","updated":"2023-12-14 06:02:05.000000000","message":"\u003e Could you please add a pointer to pyedbglib to the commit message?\n\nOr even better to VID PID table comment","commit_id":"ecf567039a997d9a8245e62eea94a5195970b6c0"},{"author":{"_account_id":1002215,"name":"Samuel Dewan","email":"samdewan@me.com","username":"samueldewan"},"change_message_id":"9f69c91c95883ec2f23da806ab5b40b3db574229","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"df6267e0_a880c5b3","in_reply_to":"20f0cd7f_81f22aee","updated":"2023-12-14 03:02:11.000000000","message":"I agree entirely that parsing the HID report descriptor would be better. In fact even the Atmel EDBG based tools that normally use 512 byte reports can be configured to use 64 byte reports, so detection is the only method that will ever be foolproof. That\u0027s what I originally set out to implement, but I quickly discovered that (as you mentioned) getting the report descriptor is somewhat difficult.\n\nAs of libhidapi 0.14 (released this May) there is a `hid_get_report_descriptor()` function, but this version of the library doesn\u0027t seem to be widely shipped yet. So I decided to propose this as a sort of interim solution.\n\nAs for the completeness of the list, it was pulled from Microchip\u0027s pyedbglib. I can\u0027t beside that it has every Atmel probe listed, but I doubt that there is a more complete publicly available source.","commit_id":"ecf567039a997d9a8245e62eea94a5195970b6c0"},{"author":{"_account_id":1002215,"name":"Samuel Dewan","email":"samdewan@me.com","username":"samueldewan"},"change_message_id":"776da75f67956a0e21bd9f09f4c8257e9f5e7014","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"78b79880_acd841bf","in_reply_to":"29d4a6e3_4a53bac9","updated":"2023-12-15 00:09:22.000000000","message":"Done","commit_id":"ecf567039a997d9a8245e62eea94a5195970b6c0"},{"author":{"_account_id":1000687,"name":"Tomas Vanek","display_name":"Tomas Vanek","email":"vanekt@fbl.cz","username":"vanekt"},"change_message_id":"1d0abe7376ea6a8358469fe1db8a88519fcea6d1","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"09aa7c17_3d94d391","in_reply_to":"df6267e0_a880c5b3","updated":"2023-12-14 05:52:34.000000000","message":"\u003e In fact even the Atmel EDBG based tools that normally use 512 byte reports can be configured to use 64 byte reports...\n\nCan they? I didn\u0027t know that.\n\n\u003e As of libhidapi 0.14 (released this May) there is a `hid_get_report_descriptor()` function, but this version of the library doesn\u0027t seem to be widely shipped yet. So I decided to propose this as a sort of interim solution.\n\nAgreed. In future we can parse the descriptor if we have libhidapi ver \u003e\u003d 0.14\nand use this vid pid list for older hidapi.\n\n\u003e As for the completeness of the list, it was pulled from Microchip\u0027s pyedbglib. I can\u0027t beside that it has every Atmel probe listed, but I doubt that there is a more complete publicly available source.\n\nGood to know. Could you please add a pointer to pyedbglib to the commit message?\nThanks.","commit_id":"ecf567039a997d9a8245e62eea94a5195970b6c0"},{"author":{"_account_id":1002215,"name":"Samuel Dewan","email":"samdewan@me.com","username":"samueldewan"},"change_message_id":"776da75f67956a0e21bd9f09f4c8257e9f5e7014","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"a7dbce89_2e047cf0","updated":"2023-12-15 00:09:22.000000000","message":"I have added a reference to pyedbglib in both the commit message and the VID PID table.","commit_id":"3e11654efe08d25304327fb2cc3bc994b6162249"}]}
