ciniml / rust-dap

CMSIS-DAP Rust implementation
Apache License 2.0
88 stars 10 forks source link

書き込まれているファームウェアのバージョンがわかるようにしたい #63

Open KOBA789 opened 5 months ago

KOBA789 commented 5 months ago

いつも便利に rust-dap を使わせてもらっています。 便利すぎるので、私の会社では何人もの開発者が rust-dap を使っているのですが、そうすると古参のメンバーは古いバージョンの rust-dap 使ってて差異で微妙にハマる、みたいなことが稀に起きます。

iSerialNumber に Git の SHA1 先頭7文字を埋めるみたいな感じでバージョン確認できるようになると嬉しいなぁと思っています。 余裕があればパッチ書いて Pull Request を送ろうと思います。

いまのところの実装方針:

ご意見ください。

ciniml commented 5 months ago

よさそうです。iSerialNumber だとバージョン変わるとシリアルでの対象指定が変わってしまいそうですけど、他に入れられる場所ありましたっけね…

KOBA789 commented 5 months ago

たしかに。シリアルナンバーで対象指定している場合は困りますね。 ちなみに iManufacturer や iProduct は Windows が VID:PID ごとにキャッシュするのでダメそうです。

ciniml commented 5 months ago

キャッシュ… Windowsめ…

ciniml commented 5 months ago

うーん、とするとfeatureで指定したら iSerialNumber にバージョン含めるようにするとかですかね。どっちデフォルトにするのがいいか悩みますが。

ciniml commented 5 months ago

ものとしてはUSBのレイヤより DAP_Info コマンドの 0x09 Product Firmware Version で返すべきな気はしますね。

https://arm-software.github.io/CMSIS_5/DAP/html/group__DAP__Info.html