fix and improve e220 driver

sago35 commented 1 year ago

ざっと変更提案を作ってみた。正しいかは不明。 merge しなくてもよいので、適当に処理してください。 ※PR 作ったほうがわかりやすいかな、という事で PR にしています

変更箇所は以下。

SetMode() はマニュアルでは 1ms で大丈夫とあるが、手元の環境では 2ms でも失敗している
- 10ms は実測して問題なかった大きめの値
ReadRegister() は 0xC1 が正しい
ReadRegister() 内のループは明らかに良くないので修正したが、無限ループ入るのでこれはこれで良くない
- 要検討
- 手前の 100ms wait は要らない
2 byte 値に対する処理が誤っていたので全体的に修正
uart は examples としては machine.DefaultUART を使うのがよさそう
examples/e220/tx の例は、アドレスは 0xFFFF に対して送ったほうがテスト簡単そう

あと、残り気づいたところは以下。

examples/e220/conf の例は、 ReadConfig() 使って config 読み出しにすると良さそう
M0 / M1 をつなぐつもりなら、 examples/e220/tx などでも config 読み出しをしておくのがよさそう
io.Reader と io.Writer になるように作るとよりよい
- ただし machine.UART は ↓ の問題があるのでうまく作る必要がある
- https://github.com/tinygo-org/tinygo/pull/2739

akif999 commented 1 year ago

変更いただいた箇所は一旦取り込むことでよいと考えたため、 merge します。残る TODO は積んでおきます。

akif999 commented 1 year ago

以下はコメントで TODO として積んだ。

ReadRegister() 内のループは明らかに良くないので修正したが、無限ループ入るのでこれはこれで良くない要検討手前の 100ms wait は要らない

examples/e220/conf の例は、 ReadConfig() 使って config 読み出しにすると良さそう M0 / M1 をつなぐつもりなら、 examples/e220/tx などでも config 読み出しをしておくのがよさそう io.Reader と io.Writer になるように作るとよりよい

akif999 / drivers

fix and improve e220 driver #1