ciniml / rust-dap

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

Move rust-dap sources to rust-dap subdir. #15

Closed ciniml closed 2 years ago

ciniml commented 2 years ago

Move RP2040 common items to the dedicated rust-dap-rp2040 crate.

elfmimi commented 2 years ago

以下の二つの部分で swclk と swdio の順番が入れ替わっているので、 https://github.com/ciniml/rust-dap/blob/5fd6a7aab4f6636a3a95322248b7fd6c85a8fdde/boards/xiao_rp2040/src/main.rs#L63 https://github.com/ciniml/rust-dap/blob/5fd6a7aab4f6636a3a95322248b7fd6c85a8fdde/rust-dap-rp2040/src/util.rs#L69-L75 最終的なピン割り当ては正しいのですが、 initialize_usb() の型引数の名前と実態が入れ替わってしまっています。

ciniml commented 2 years ago

以下の二つの部分で swclk と swdio の順番が入れ替わっているので、

https://github.com/ciniml/rust-dap/blob/5fd6a7aab4f6636a3a95322248b7fd6c85a8fdde/boards/xiao_rp2040/src/main.rs#L63

https://github.com/ciniml/rust-dap/blob/5fd6a7aab4f6636a3a95322248b7fd6c85a8fdde/rust-dap-rp2040/src/util.rs#L69-L75

最終的なピン割り当ては正しいのですが、 initialize_usb() の型引数の名前と実態が入れ替わってしまっています。

おっ、そういえばXIAO RP2040で動確してなかった… ご指摘ありがとうございます。修正して確認しておきます。

ciniml commented 2 years ago

ちがった… これ修正なしでも結果的にそのまま動くのか…

ciniml commented 2 years ago

パラメータ名入れ替わっているのを修正

ciniml commented 2 years ago

マージしてしまおう。